Skip to content

refactor: auto-discover games using import.meta.glob#13

Merged
devp merged 2 commits intomainfrom
claude/self-registering-game-dirs-PuAIL
Feb 1, 2026
Merged

refactor: auto-discover games using import.meta.glob#13
devp merged 2 commits intomainfrom
claude/self-registering-game-dirs-PuAIL

Conversation

@devp
Copy link
Copy Markdown
Owner

@devp devp commented Feb 1, 2026

Move GameDefinition interface to types.ts and update registry.ts to
use Vite's import.meta.glob() for automatic game discovery. Each game
now exports a complete definition from its index.ts, eliminating the
need to manually register games in the registry file.

This enables easier parallel development since adding a new game no
longer requires editing a shared registry file.

https://claude.ai/code/session_01BB2Hp7Lo7ufqDVQktxYDD3

Move GameDefinition interface to types.ts and update registry.ts to
use Vite's import.meta.glob() for automatic game discovery. Each game
now exports a complete definition from its index.ts, eliminating the
need to manually register games in the registry file.

This enables easier parallel development since adding a new game no
longer requires editing a shared registry file.

https://claude.ai/code/session_01BB2Hp7Lo7ufqDVQktxYDD3
Update documentation to reflect the new auto-discovery pattern:
- Replace manual registry.ts registration with index.ts definition export
- Update Phase 8 to show the self-contained index.ts pattern
- Update pitfall about missing definition export

https://claude.ai/code/session_01BB2Hp7Lo7ufqDVQktxYDD3
@netlify
Copy link
Copy Markdown

netlify bot commented Feb 1, 2026

Deploy Preview for devp-fgj-js ready!

Name Link
🔨 Latest commit 582364c
🔍 Latest deploy log https://app.netlify.com/projects/devp-fgj-js/deploys/697f6789f2605a000835c5c8
😎 Deploy Preview https://deploy-preview-13--devp-fgj-js.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@devp devp merged commit 1e6798f into main Feb 1, 2026
5 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.

2 participants