Skip to content

Fix cherry-pick workflow: stash sticky-comment helper before branch switch - #178

Merged
kriszyp merged 1 commit into
mainfrom
claude/optimistic-turing-84f32a
May 19, 2026
Merged

Fix cherry-pick workflow: stash sticky-comment helper before branch switch#178
kriszyp merged 1 commit into
mainfrom
claude/optimistic-turing-84f32a

Conversation

@kriszyp

@kriszyp kriszyp commented May 19, 2026

Copy link
Copy Markdown
Member

Summary

The cherry-pick workflow checks out main initially so the .github/scripts/upsert-sticky-comment.js helper (added in #151) is in the tree, but the Cherry-pick step then runs git checkout -B <branch> origin/<release> and swaps the working tree to the release branch. The release branch (v5.0) doesn't contain that helper, so every subsequent node .github/scripts/upsert-sticky-comment.js call fails with MODULE_NOT_FOUND — see run 26107620269.

Fix: copy the helper into $RUNNER_TEMP right after the main checkout, and invoke it via $UPSERT_STICKY in all four call sites. That path survives the branch switch.

report-cherry-pick-tests.yml doesn't switch branches, so no change there.

Test plan

  • Label a PR with patch and confirm the sticky comment is posted (no MODULE_NOT_FOUND).
  • Unlabel the same PR and confirm the cancellation sticky comment posts.
  • Verify the success path (tests-running sticky comment) on a clean cherry-pick.

🤖 Generated with Claude Code

…branch

The Cherry-pick step runs `git checkout -B <branch> origin/<release>`, which
swaps the working tree to the release branch. That branch doesn't contain
`.github/scripts/upsert-sticky-comment.js`, so every subsequent invocation
fails with MODULE_NOT_FOUND. Copy the helper into `$RUNNER_TEMP` right after
the `main` checkout and reference it from there in all call sites.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@kriszyp
kriszyp requested review from a team as code owners May 19, 2026 17:38
@claude

claude Bot commented May 19, 2026

Copy link
Copy Markdown
Contributor

Reviewed; no blockers found.

@kriszyp
kriszyp merged commit 04ff1ea into main May 19, 2026
19 of 23 checks passed
@kriszyp
kriszyp deleted the claude/optimistic-turing-84f32a branch May 19, 2026 19:21
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