CI: bump Actions pins, add track_progress, add claude-debug label toggle - #15
Open
jnasbyupgrade wants to merge 1 commit into
Open
Conversation
Combines three related CI-workflow improvements into one PR (touching only claude-code-review.yml and claude.yml): - Bump actions/checkout@v4 -> @v7 (current latest major) to clear the Node.js-20-deprecation warning it triggers on every run. Supersedes/ incorporates upstream PR Postgres-Extensions#13 and fork branch ci/bump-actions-versions, which made the same v4->v7 bump to the same two lines. anthropics/claude-code-action@v1 is still current (a floating v1 tag exists at v1.0.185) so it is left as-is. - Add track_progress: true to the claude-code-action step in claude-code-review.yml so it posts a live, updating checklist comment as it works instead of staying silent until the whole run finishes -- which, combined with this workflow's cost gate, could leave a PR dark for the better part of an hour. Disabled specifically for labeled-triggered runs (see below): the action's own track_progress validation only accepts opened/synchronize/reopened/ready_for_review for pull_request(_target) events and throws for any other action. - Add a claude-debug PR-label toggle so a maintainer can skip the cost gate and turn on full transcript output (show_full_output) by just labeling the PR, without editing/pushing the workflow file. The label is queried live via `gh pr view` inside the step rather than read from the event payload, since GitHub's "Re-run jobs" replays the original stored payload and would miss a label added afterward. `labeled` is added to the trigger types so applying the label alone starts a fresh run, scoped tightly in the job's `if:` so an unrelated label can't re-trigger this paid workflow. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
jnasbyupgrade
added
claude-debug
jnasbyupgrade
changed the title
This was referenced Aug 5, 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
1 participant
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.
Summary
Three small fixes to
claude-code-review.yml/claude.yml:actions/checkout@v4->@v7in both files, clearing the Node.js 20 deprecation warning it triggers on every run. Supersedes CI: bump GitHub Actions to latest versions #13 and theci/bump-actions-versionsbranch (same change) — close CI: bump GitHub Actions to latest versions #13 once this merges.track_progress: trueto theclaude-code-actionstep, so it posts a live, updating checklist comment instead of staying silent until the whole run finishes.claude-debugPR label: applying it skips the cost-gate wait and turns on full transcript output, with no workflow edit/push needed per debug iteration. The label is checked live viagh pr viewrather than the cached event payload, so it also works on a re-run.Since this workflow runs on
pull_request_target(see the file's own top-of-file comment on why), GitHub sources the whole file frommaster, not this PR's branch — so none of these three changes are exercised by this PR's ownclaude-reviewcheck. Verification happens on the next PR after this merges.Also found, not fixed here
claude-reviewcurrently fails at checkout for every PR from this fork (actions/checkout's "pwn request" refusal on apull_request_targetworkflow). Pre-existing, unrelated to this PR. Fixing it (allow-unsafe-pr-checkout: true) is a security-tradeoff decision, left for you.🤖 Generated with Claude Code