Skip to content

ci: delete Claude review tracking comment when the run is cancelled - #11914

Open
ovr wants to merge 2 commits into
masterfrom
delete-review-comment-on-cancel
Open

ovr wants to merge 2 commits into
masterfrom
delete-review-comment-on-cancel

Conversation

@ovr

@ovr ovr commented Sep 17, 2026

Copy link
Copy Markdown
Member

cancel-in-progress kills a superseded Claude review mid-run, leaving its "Claude Code is working…" tracking comment stuck on the PR forever. This adds a cleanup step guarded by cancelled() — such steps still run during the cancellation grace period — that finds the comment by the run link the action keeps in its body and deletes it. The step is continue-on-error, so a failed cleanup never reds the job, and it no-ops when cancellation happens before the comment exists.

Check List

  • Tests have been run in packages where changes have been made if available
  • Linter has been run for changed code (actionlint)
  • Tests for the changes have been added if not covered yet
  • Docs have been added / updated if required

🤖 Generated with Claude Code

ovr and others added 2 commits September 17, 2026 15:36
`cancel-in-progress` kills a superseded review mid-run, leaving its
"Claude Code is working…" comment stuck on the PR forever. Steps guarded
by `cancelled()` still run during the cancellation grace period, so clean
up the comment there. The action keeps the run link in the comment body
for the comment's whole lifetime, which makes it the marker for the
comment this run owns.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

This branch has not been deployed

No deployments
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