Skip to content

fix: unset checkout extraheader so bot token can push release tags#21227

Draft
AztecBot wants to merge 4 commits intonextfrom
claudebox/566d801ace30e9e8-2
Draft

fix: unset checkout extraheader so bot token can push release tags#21227
AztecBot wants to merge 4 commits intonextfrom
claudebox/566d801ace30e9e8-2

Conversation

@AztecBot
Copy link
Collaborator

@AztecBot AztecBot commented Mar 6, 2026

Instead of broadening permissions to contents: write, unset the extraheader that actions/checkout persists (using the workflow's github.token with contents: read). This keeps the workflow token tight while letting handle_release_pr use the bot token (AZTEC_BOT_GITHUB_TOKEN) for git push.

See full analysis: https://gist.github.com/AztecBot/3606d126292f2f2544270306ea359fb0

randyquaye and others added 3 commits March 6, 2026 12:18
- Move SSM registration wait and shutdown timer from bootstrap_ssm into
  aws_request_instance_type for consistency with the SSH path.
- Extract ssm_send_command as a reusable script (send, poll, stream output,
  return remote exit code).
- Use mktemp for SSM script files to avoid race conditions between
  concurrent SSM commands.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This helps with some of the claudebox MCP sandbox work.
@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label Mar 6, 2026
Instead of broadening permissions to contents:write, unset the
extraheader that actions/checkout persists. This keeps the workflow
token tight (contents:read) while letting handle_release_pr use the
bot token for git push.
@AztecBot AztecBot changed the title fix: grant contents:write for release PR tag push fix: unset checkout extraheader so bot token can push release tags Mar 6, 2026
Base automatically changed from rq/ssm-migration to next March 7, 2026 00:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants