Skip to content

feat: confine the template to the tsforge phaser pack - #37

Merged
agjs merged 1 commit into
mainfrom
chore/tsforge-phaser-pack
Aug 22, 2026
Merged

feat: confine the template to the tsforge phaser pack#37
agjs merged 1 commit into
mainfrom
chore/tsforge-phaser-pack

Conversation

@agjs

@agjs agjs commented Aug 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

Makes this template pass (and locally echo) the new tsforge phaser rule pack.

  • Brand scene keys via asSceneKey (Boot / World)
  • new:scene emits import * as Phaser, branded keys, and SHUTDOWN dispose
  • Remove S/R keyboard keys in WorldScene dispose
  • ESLint no-restricted-syntax overlay for ignoreDestroy, raw scene.start("…"), window.addEventListener, and raw timers
  • arch-invariants grep for ignoreDestroy = true
  • Docs: pack is shipped; still no .tsforge/scaffold-manifest.json

Companion: tsforge Phaser pack PR (agjs/tsforge). This repo does not depend on unpublished @agjs/tsforge; bun run check covers the syntactic subset.

Test plan

  • bun run check (typecheck, lint, format, knip, catalog, depcruise, 55 tests)
  • bun run test:smoke (canvas boot + loop still alive)
  • bun run new:scene TempProbe output lints/formats; deleted after
  • Cross-repo: tsforge phaser pack against this src/ → 0 errors, 0 warnings

Brand scene keys, dispose S/R keys on shutdown, emit the same patterns
from new:scene, and add a syntactic ESLint overlay so bun run check
catches ignoreDestroy, raw scene-key literals, and unmanaged timers
without depending on unpublished tsforge.
@agjs

agjs commented Aug 22, 2026

Copy link
Copy Markdown
Collaborator Author

Companion pack: boringstack-xyz/tsforge#340

@agjs
agjs merged commit 52c5862 into main Aug 22, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant