fix(spawn): scope out ambient ANTHROPIC_API_KEY from claude launches - #4
Merged
Merged
Conversation
herdr server sets ANTHROPIC_API_KEY in its own environment and passes it to every child pane, which makes claude prefer API-key billing over the OAuth claude.ai login and disables org connectors. env -u ANTHROPIC_API_KEY on the claude launch line unsets it for that one process, verified empirically (claude doctor) to satisfy claude's own auth-source check.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Intent
Scope out ambient ANTHROPIC_API_KEY from firstmate-launched claude worker panes so Claude Code prefers OAuth claude.ai login over API-key billing, restoring org connectors. Root cause and fix already diagnosed in backlog item nomistakes-connectors-authkey; captain approved skipping every claude-spawning pipeline step (review, test, document, lint) for this run since each hits the exact bug this fix resolves via the no-mistakes daemon's own agent invocations, and validating instead via the already-passing local test suite (27/27, including the 2 new cases added for this fix).
What Changed
910ecac fix(spawn): scope out ambient ANTHROPIC_API_KEY from claude launches
Testing
Pipeline
Updates from git push no-mistakes
✅ **intent** - passed
✅ No issues found.
✅ **Rebase** - passed
✅ No issues found.
⏭️ **Review** - skipped
Step was skipped.
⏭️ **Test** - skipped
Step was skipped.
⏭️ **Document** - skipped
Step was skipped.
⏭️ **Lint** - skipped
Step was skipped.
✅ **Push** - passed
✅ No issues found.