Skip to content

refactor: keep org→token map out of git (host-mounted map + convention)#12

Merged
MrChrisRodriguez merged 1 commit into
mainfrom
chore/externalize-org-token-map
Jun 4, 2026
Merged

refactor: keep org→token map out of git (host-mounted map + convention)#12
MrChrisRodriguez merged 1 commit into
mainfrom
chore/externalize-org-token-map

Conversation

@MrChrisRodriguez

Copy link
Copy Markdown
Owner

What

Follow-up to #11. The credential router previously hardcoded org names (a case with the real orgs), which meant the client list landed in the git history of every repo built from this template. This removes them: the helper is now fully generic and resolves the token env var per org by:

  1. Host map file ~/.config/devcontainer/github-token-map (mounted read-only, not in git) — org=ENV_VAR_NAME lines, for tokens whose name doesn't follow the convention.
  2. Convention<ORG>_GITHUB_TOKEN (org upper-cased, non-alphanumeric → _).
  3. FallbackGITHUB_TOKEN.

Why

Keeps client/org names out of every descendant's history. New children never inherit them (init wipes history; sync copies current content), so scrubbing here means they're gone going forward.

Changed

  • git-credential-org-router.sh — generic resolver (map → convention → fallback); no org names.
  • setup-git-credentials.sh — generic echo.
  • AUTH-PERSISTENCE.md / secrets.example — describe the mechanism + host map file; org names removed.

Token values are still read from env at push time and never written to disk.

🤖 Generated with Claude Code

Router no longer hardcodes org names: resolve the token env var via a
host map file (~/.config/devcontainer/github-token-map), else the
<ORG>_GITHUB_TOKEN convention, else GITHUB_TOKEN. Scrub org/client names
from the docs so they never enter a child repo's history.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@MrChrisRodriguez MrChrisRodriguez merged commit 98e085d into main Jun 4, 2026
1 check passed
@MrChrisRodriguez MrChrisRodriguez deleted the chore/externalize-org-token-map branch June 4, 2026 21:07
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