Skip to content

fix(framework): authenticate the Actions run-branch push explicitly (#1085) - #1088

Merged
suleimansh merged 1 commit into
mainfrom
fix/1085-push-auth
Jul 23, 2026
Merged

fix(framework): authenticate the Actions run-branch push explicitly (#1085)#1088
suleimansh merged 1 commit into
mainfrom
fix/1085-push-auth

Conversation

@suleimansh

Copy link
Copy Markdown
Contributor

Follow-up to #1087. Live-testing the two-turn Actions flow showed the run's agent commits fine, but the new "Push the run branch" step failed:

remote: Invalid username or token. Password authentication is not supported for Git operations.
fatal: Authentication failed for 'https://github.com/gemstack-land/gemstack.git/'

The step used a plain git push origin, which relies on the credentials actions/checkout persists. The claude-code-action step runs its own git setup during the run and leaves those unusable, so the later push has no valid auth. The commit and no-op-skip logic were correct; only the push auth was wrong.

Fix: push through an x-access-token:<workflow-token> URL (masked in logs), so the push carries its own auth regardless of what the agent step left in the git config.

Workflow-only change, no changeset. Verifies after merge (workflow-validation gate): I'll re-run the two-turn test and confirm readCode reads the pushed file and turn 2 builds on turn 1's branch.

Refs #1085 #1087

The Push the run branch step (#1085) used a plain `git push origin`, which relies on
the checkout's persisted credentials. The claude-code-action step runs its own git
setup and leaves those unusable, so the push failed with "Authentication failed for
github.com". Push through an x-access-token URL built from the workflow token instead.

Refs #1085 #1087
@suleimansh suleimansh added bug Something isn't working priority: medium Worth doing, not urgent the-framework ♻️ labels Jul 23, 2026
@suleimansh suleimansh self-assigned this Jul 23, 2026
@suleimansh
suleimansh marked this pull request as ready for review July 23, 2026 19:06
@suleimansh
suleimansh merged commit e9671ab into main Jul 23, 2026
2 checks passed
@suleimansh
suleimansh deleted the fix/1085-push-auth branch July 23, 2026 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working priority: medium Worth doing, not urgent the-framework ♻️

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant