Skip to content

docs(glitch-invaders): say how to make the Sentry token - #40

Merged
theycallmeswift merged 1 commit into
mainfrom
claude/keen-feynman-nbyhbi
Nov 2, 2026
Merged

theycallmeswift merged 1 commit into
mainfrom
claude/keen-feynman-nbyhbi

Conversation

@theycallmeswift

@theycallmeswift theycallmeswift commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

The Glitch Invaders README listed the Sentry token's scopes but not how to get a token that has them. It now says to make one with an internal integration in the org the game reports to, since organization tokens have fixed CI scopes and a token from another org gets 403.

Decide

Nothing.

Ready gate

Step Evidence
Tests and lint Node 26.13.0: npm run verify ok for all 4 packages; npm test 299 pass, 0 fail; npm run lint 0 issues in 18 files ✓
Red run n/a: docs-only change ✓
Sibling sweep grep for event:read, org:read, auth token, SENTRY_AUTH_TOKEN outside tests and code: only challenges/glitch-invaders/README.md states the scopes; sentry.json's comment names the variable but not how to make the token ✓
Docs and specs challenges/glitch-invaders/README.md ✓
Standalone run n/a: README-only change; no package code, manifest, or hook changed ✓
Editorial pass Reread against docs/style/markdown.md: prose wrapped at 80, one rule stated once, no history or IDs ✓
Skipped tests None ✓
Secrets, permissions Not touched: no token or credential added; the text describes which Sentry permissions to grant ✓

Generated by Claude Code


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

The token needs read scopes an organization token can't carry, so the
README now names an internal integration in the game's own org.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01ViFVZZzWH6bDpJkxdRGgty
@coderabbitai

coderabbitai Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 3d7fb69b-acab-4996-afa9-0dee42962495
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@theycallmeswift
theycallmeswift marked this pull request as ready for review November 2, 2026 18:27
@theycallmeswift
theycallmeswift merged commit de7a7d3 into main Nov 2, 2026
2 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