ci: run release-please via CLI instead of the googleapis action#324
Open
max-parke-scale wants to merge 1 commit into
Open
ci: run release-please via CLI instead of the googleapis action#324max-parke-scale wants to merge 1 commit into
max-parke-scale wants to merge 1 commit into
Conversation
03180ce to
70d3124
Compare
The googleapis/release-please-action isn't on the org Actions allow-list, so the release-please workflow added in #321 failed at startup (no release PR cut). Run the release-please CLI under actions/setup-node (allow-listed) instead — same manifest-mode behavior (release-pr + github-release), no third-party action. Verified the CLI commands/flags against release-please@16. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
70d3124 to
b21deb7
Compare
NiteshDhanpal
approved these changes
Jun 18, 2026
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
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.
What
Rewrites the release-please workflow (added in #321) to run the release-please CLI under
actions/setup-node, instead ofgoogleapis/release-please-action.Why
After #321 merged, the workflow failed at startup on every push to
main(startup_failure, no logs, no release PR cut). The merged YAML is valid and the action SHA resolves — the cause is the org Actions allow-list:googleapis/release-please-actionisn't on it. (Every other workflow here uses onlyactions/,astral-sh/,docker/,stainless-api/,codecov/,dorny/; the Actions-policy API is admin-only/403 for me, so I couldn't read it directly, but the signature is unambiguous.)Fix
Run
npx release-please@16 release-pr+github-release(manifest mode — same as the action did internally) underactions/setup-node@v4, which is allow-listed (actions/*is used throughout the repo). No third-party action → allow-list-proof.Verified the CLI commands (
release-pr/github-release;manifest-pr/manifest-releaseare deprecated aliases) and flags (--token,--repo-url,--config-file,--manifest-file) againstrelease-please@16.After merge
Runs on
main; once afeat/fixlands (or viaworkflow_dispatch) it opens the first release PR → merging that cuts the firstvX.Y.Ztag. Config + manifest are unchanged from #321.🧑💻🤖 — posted via Claude Code
Greptile Summary
googleapis/release-please-actionworkflow step with directrelease-please@16CLI commands.actions/setup-node@v4before running release PR and GitHub release commands.issues: writefor label management.Confidence Score: 5/5
The workflow change is narrowly scoped to replacing a blocked GitHub Action with equivalent CLI invocations.
Only one CI workflow file changed, the release-please config and manifest remain unchanged, and no code issues were identified.
What T-Rex did
Comments Outside Diff (1)
.github/workflows/release-please.yml, line 12-14 (link)The CLI still applies and removes release-please labels on release PRs, and those calls go through GitHub's Issues API. This workflow only grants
contents: writeandpull-requests: write, so the job can fail with a permissions error when it tries to add or remove labels likeautorelease: pending. Addissues: writehere so the CLI has the same label permissions the release flow needs.Prompt To Fix With AI
Reviews (2): Last reviewed commit: "ci: run release-please via CLI instead o..." | Re-trigger Greptile