Skip to content

feat: turn on live Sentry mode for Glitch Invaders - #27

Merged
theycallmeswift merged 1 commit into
mainfrom
feat/glitch-invaders-live-sentry
Oct 19, 2026
Merged

theycallmeswift merged 1 commit into
mainfrom
feat/glitch-invaders-live-sentry

Conversation

@theycallmeswift

@theycallmeswift theycallmeswift commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Fills in Glitch Invaders' sentry.json with MLH's glitch-invaders project in the major-league-hacking-xc org, so a station whose event vault holds a real sentry token plays it in live Sentry mode. The two tests that assumed the shipped file holds placeholders now check the shipped values' shape, and the demo-mode test writes its own placeholder copy.

Decide

  • Merge only after a live playthrough on a station. Recommend: play it once on the test Mac from this branch's commit (the event config can pin it), confirm each Incident Console shows real Sentry evidence, then merge and tag v0.1.0.

Ready gate

Step Evidence
Tests and lint macOS, Node 26.9: npm test 258 passing, 0 failing; npm run verify ok for all 4 packages; npm run lint 0 issues ✅
Red run With the real values and the old tests, the package … ships placeholder Sentry settings and setup stays in demo mode … placeholders fail; with the updated tests both pass ✅
Sibling sweep Searched sentry.json and placeholder in test/: makeLivePackage now builds on a shared makePackageWithSettings; no other test reads the shipped file ✅
Docs and specs None: the README's Live Sentry mode section already describes filled-in values ✅
Standalone run setup with SENTRY_AUTH_TOKEN set prints ready at http://127.0.0.1:4373/ (live Sentry mode), and cleanup stops it. Not run: a live Pi playthrough against Sentry; that's the Decide item ⚠️
Editorial pass Reread against docs/style/ ✅
Skipped tests None ✅
Secrets, permissions DSN, org, and project are public values; no token in the repo. The token lives only in the event vault's sentry service ✅

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

Co-authored-by: Claude <noreply@anthropic.com>
@theycallmeswift
theycallmeswift marked this pull request as ready for review October 19, 2026 02:39
@theycallmeswift
theycallmeswift merged commit 51a7c14 into main Oct 19, 2026
1 check 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