Skip to content

Fix internal-release infinite loop on main ([skip ci] on version-bump commit) - #378

Merged
Devesh-Skyflow merged 1 commit into
mainfrom
devesh/sk-3023-fix-release-loop-main
Jul 29, 2026
Merged

Fix internal-release infinite loop on main ([skip ci] on version-bump commit)#378
Devesh-Skyflow merged 1 commit into
mainfrom
devesh/sk-3023-fix-release-loop-main

Conversation

@Devesh-Skyflow

Copy link
Copy Markdown
Collaborator

Problem

The internal release path in shared-build-and-deploy.yml bumps the version and force-pushes an automated [AUTOMATED] Private Release …-dev-<sha> commit back to the branch using an admin PAT. GitHub does not trigger workflows for GITHUB_TOKEN pushes, but does for PAT-authored pushes — so each bump re-triggers the internal-release workflow → bump → push → infinite loop.

This is the same defect fixed on v3-release/26.7.29 in #377. Putting it on main ensures future release branches cut from main don't reintroduce the loop.

Fix

Append [skip ci] to the automated internal-release commit message so the bump push no longer re-triggers CI. The PAT stays (branch-protection bypass preserved); only the loop is broken.

Scope

Only the internal path (the confirmed loop). Beta/public paths are tag/release-triggered, not branch-push, so they don't loop the same way.

🤖 Generated with Claude Code

…ion-bump commit

Same fix as #377 (applied to v3-release/26.7.29), now on main so future
release branches cut from main don't reintroduce the loop.

The internal-release path bumps the version and force-pushes an automated
commit back to the branch using an admin PAT. PAT-authored pushes DO trigger
workflows (unlike GITHUB_TOKEN), so the bump re-triggers the release ->
bump -> push -> infinite loop. Appending [skip ci] to the automated commit
message breaks it while keeping the PAT (preserving the branch-protection
bypass fix).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

1 participant