#1208 Clear the visualizations lint violations and retire six rule deferrals

[williamthorsen]

What

Fixes a subset of deferred lint violations in the factory workspace package and restores the severity of the associated rules to "error" when a strict-lint check is run.

Why

Factory capped eleven lint rules at warn, which also stopped strict lint from re-promoting them, so a
violation of any of them could land in the package without failing a check. Two thirds of the backlog sat
under one directory, which meant clearing it alone was enough to restore error-level enforcement for six of
the eleven rules.

Details

♻️ Refactoring

• Deleted the four deprecated catwalk dimension aliases (CATWALK_Y, GROUND_Y, CHUTE_TOP, CHUTE_BOT).
  demos/catwalk/ declares its own copies and is unaffected.
• Extracted the killed-actor suppression that both choreographers carried byte-for-byte into
  visualizations/shared/run-animation-suppressing-errors.ts, an awaiting wrapper that replaces eight
  .catch() chains.
• Converted the three scenes' promise chains into async private methods, collapsing CatwalkScene's and
  FactoryFloorScene's buffered-diff drain from a pair of duplicated handlers into one unconditional call.
• Replaced the module-level let bindings in both sprite loaders with never-reassigned const containers,
  and rewrote office-sprite-loader's .finally() chain as an awaiting wrapper that keeps the shared
  in-flight slot's release exactly once.
• Shared one sprite-load helper across all three scenes, parameterized by failure message so each scene
  keeps its own log text.
• Hoisted the repeated orchestrator !== undefined guards in both applyImmediate functions, swapped
  pure-read operand pairs in the mappers, and replaced || membership chains with module-level Sets.

🧪 Tests

• Added unit tests for the shared animation wrapper covering the suppressed, logged, and non-Error
  rejection paths, and for the shared sprite-load helper.
• Added two CatwalkScene tests covering the buffered-diff drain after both a resolved and a rejected
  choreography.
• Retargeted the dimension assertions at the surviving RAIL_Y and GROUND_LINE_Y, and converted the
  deferred-promise scaffolding to Promise.withResolvers().

⚙️ Tooling

• Retired six entries from factory's deferred-lint list, leaving the five rules that still have violations
  elsewhere in the package.

Closes #1208

[github-actions]

Dependency audit

Production dependency audit passed.