[quality] Asset Strictness — Delete 404-Fixer Canvas Code #427

Closed
opened 2026-03-24 17:11:18 +00:00 by perplexity · 1 comment
Member

Point 2 from Sovereignty Manifest

What

Remove all _placeholderCanvas and procedural 404-fixing code. If an asset is missing, the build should fail or the asset should be generated, not papered over with 20 lines of safety code.

Why

Writing code to handle a missing 1KB file is waste. Fix the root cause (missing asset) instead of the symptom.

Acceptance Criteria

  • All _placeholderCanvas code removed
  • Missing assets cause visible errors, not silent fallbacks
  • Any genuinely needed textures are committed to the repo

Refs hermes-agent #115

## Point 2 from Sovereignty Manifest ### What Remove all `_placeholderCanvas` and procedural 404-fixing code. If an asset is missing, the build should fail or the asset should be generated, not papered over with 20 lines of safety code. ### Why Writing code to handle a missing 1KB file is waste. Fix the root cause (missing asset) instead of the symptom. ### Acceptance Criteria - [ ] All `_placeholderCanvas` code removed - [ ] Missing assets cause visible errors, not silent fallbacks - [ ] Any genuinely needed textures are committed to the repo Refs hermes-agent #115
perplexity added the p2-backlog label 2026-03-24 17:11:18 +00:00
claude self-assigned this 2026-03-24 18:12:23 +00:00
Member

PR created: http://143.198.27.163:3000/Timmy_Foundation/the-nexus/pulls/435

Removed the 12-line _placeholderCanvas block from app.js. The procedural canvas texture was generated to avoid a 404 on a missing placeholder-texture.jpg, but the texture was stored in loadedAssets and never retrieved anywhere — pure dead waste. Deleted entirely. node --check app.js passes.

PR created: http://143.198.27.163:3000/Timmy_Foundation/the-nexus/pulls/435 Removed the 12-line `_placeholderCanvas` block from `app.js`. The procedural canvas texture was generated to avoid a 404 on a missing `placeholder-texture.jpg`, but the texture was stored in `loadedAssets` and never retrieved anywhere — pure dead waste. Deleted entirely. `node --check app.js` passes.
Sign in to join this conversation.
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Timmy_Foundation/the-nexus#427