fix: fall back to inherited env for auth when inputs are empty#1342
Open
kirsanium wants to merge 1 commit into
Open
fix: fall back to inherited env for auth when inputs are empty#1342kirsanium wants to merge 1 commit into
kirsanium wants to merge 1 commit into
Conversation
The "Run Claude Code Action" step maps the auth inputs into env
unconditionally:
ANTHROPIC_API_KEY: ${{ inputs.anthropic_api_key }}
CLAUDE_CODE_OAUTH_TOKEN: ${{ inputs.claude_code_oauth_token }}
When the input is empty — whether because the caller supplied auth via
the step `env:` block (as reported in anthropics#676) or because the `with:` value
resolves empty in some runner/secret configurations — this assignment
overwrites the inherited env value with an empty string. validate-env
then fails with the misleading "Either ANTHROPIC_API_KEY or
CLAUDE_CODE_OAUTH_TOKEN is required" error even though the caller did
provide a token.
Fall back to the inherited env var when the input is empty, mirroring the
existing `${{ env.X }}` pattern already used a few lines below for
ANTHROPIC_BASE_URL / ANTHROPIC_CUSTOM_HEADERS. The input still takes
precedence; nothing changes for workflows that pass auth via `with:`.
Fixes anthropics#676
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.
The "Run Claude Code Action" step maps the auth inputs into env unconditionally:
When the input is empty — whether because the caller supplied auth via the step
env:block (as reported in #676) or because thewith:value resolves empty in some runner/secret configurations — this assignment overwrites the inherited env value with an empty string. validate-env then fails with the misleading "Either ANTHROPIC_API_KEY or CLAUDE_CODE_OAUTH_TOKEN is required" error even though the caller did provide a token.Fall back to the inherited env var when the input is empty, mirroring the existing
${{ env.X }}pattern already used a few lines below for ANTHROPIC_BASE_URL / ANTHROPIC_CUSTOM_HEADERS. The input still takes precedence; nothing changes for workflows that pass auth viawith:.Fixes #676