Skip to content

contents: write so the job token can arm auto-merge - #14

Merged
jeremy merged 1 commit into
mainfrom
ci/automerge-contents-write
Jul 27, 2026
Merged

contents: write so the job token can arm auto-merge#14
jeremy merged 1 commit into
mainfrom
ci/automerge-contents-write

Conversation

@jeremy

@jeremy jeremy commented Jul 27, 2026

Copy link
Copy Markdown
Member

Exercise round 3 (run 30309501264) passed every invariant — repair PR #466 adopted, head/author verified after the API-catchup retry — and failed only at enablePullRequestAutoMerge: Resource not accessible by integration. Arming auto-merge requires contents: write on the job token (merging is a contents operation; identical grant to the fleet's dependabot-auto-merge workflows). The token remains unable to push workflow files, and the repair branch push stays on the environment-scoped deploy key. Caller permission blocks updated to match in the re-pin PRs.

Exercise round 3 passed every invariant (repair PR adopted, head/author
verified after the API-catchup retry) and failed only on
enablePullRequestAutoMerge: Resource not accessible by integration.
Arming auto-merge is a contents operation — the same grant the
dependabot-auto-merge workflows carry. The token still cannot push
workflow files; the repair branch push stays on the deploy key.
Copilot AI review requested due to automatic review settings July 27, 2026 22:07
@jeremy
jeremy merged commit d7a9888 into main Jul 27, 2026
6 checks passed
@jeremy
jeremy deleted the ci/automerge-contents-write branch July 27, 2026 22:09

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the reusable Dependabot “sync action pin comments” workflow to request contents: write on the job token so it can successfully enable PR auto-merge (which GitHub treats as a contents-level operation), addressing the Resource not accessible by integration failure during enablePullRequestAutoMerge.

Changes:

  • Update the documented caller permissions example to include contents: write.
  • Change the job token permissions from contents: read to contents: write, with clarified inline rationale tying it specifically to arming auto-merge.

Tip

If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

jeremy added a commit to basecamp/basecamp-sdk that referenced this pull request Jul 27, 2026
basecamp/.github#14: enablePullRequestAutoMerge needs contents: write
on the job token (round 3 passed every other gate). Exercise round 4.
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.

2 participants