Skip to content

ci: remove unused release workflow permissions - #773

Merged
marandaneto merged 2 commits into
mainfrom
ci/remove-unused-workflow-permissions-20260910104257
Sep 10, 2026
Merged

ci: remove unused release workflow permissions#773
marandaneto merged 2 commits into
mainfrom
ci/remove-unused-workflow-permissions-20260910104257

Conversation

@marandaneto

@marandaneto marandaneto commented Sep 10, 2026

Copy link
Copy Markdown
Member

💡 Motivation and Context

The version-bump job grants the built-in GITHUB_TOKEN repository write access even though its repository writes use a separate GitHub App token.

Remove unused actions-write from the publisher. Retain publisher contents-write for tags and GitHub releases.

Lower version-bump to contents: read. Preserve the App-authenticated checkout and all App-token commit, tag and release operations. No authentication input or App permission changes.

Read-only workflow inheritance remains allowed. This reduces the built-in token's permissions, not the job's separate App-token authority or build/publish exposure. No package release or changeset is needed.

💚 How did you test it?

  • Checked every repository-write operation in the affected job and traced it to the App token, including the pinned action contracts.
  • Compared parsed YAML with the baseline and verified that only the intended permission keys changed. Triggers, steps, dependencies, secrets and token wiring are unchanged.
  • actionlint -shellcheck= -pyflakes= and git diff --check passed. Existing shell-lint diagnostics were checked against the baseline and left unchanged.
  • Reviewed the final branch diff. No live release or SDK tests were run for this permission-only change.

📝 Checklist

  • I reviewed the submitted code.
  • I added tests to verify the changes. No SDK tests apply to this permission-only change.
  • I updated the docs if needed. No documentation change is needed.
  • No breaking change or entry added to the changelog.

If releasing new changes

No package release or changeset is needed.

🤖 Agent context

Autonomy: Human-driven (agent-assisted)

Implemented with Pi and delegate agents using file tools, Git, GitHub CLI, Python/PyYAML and actionlint. The parent agent reviewed the final diff. Work was performed in a dedicated worktree. Autoreview was skipped because all changes are GitHub workflow permission administration. No session transcript was published. Human review is required.

@marandaneto marandaneto self-assigned this Sep 10, 2026
@marandaneto
marandaneto marked this pull request as ready for review September 10, 2026 08:51
@marandaneto
marandaneto requested a review from a team as a code owner September 10, 2026 08:51
@greptile-apps

greptile-apps Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "ci: remove unused release actions permis..." | Re-trigger Greptile

@marandaneto marandaneto changed the title ci: remove unused release actions permission ci: remove unused release workflow permissions Sep 10, 2026
@marandaneto
marandaneto merged commit cad92a5 into main Sep 10, 2026
16 of 17 checks passed
@marandaneto
marandaneto deleted the ci/remove-unused-workflow-permissions-20260910104257 branch September 10, 2026 12:17
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