You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
fix(ci): skip stale workflow results and quote headers - #960
The release workflow can receive CI results from commits made before release automation existed. Those checkouts lack the planning script, so release preparation fails.
Skip those historical CI results. Use consistent quoting for registry request headers to satisfy the lint rule.
The package size report can also finish after its PR closes or moves to another commit. Skip comments for those stale results.
No actionable comments were generated in the recent review. 🎉
ℹ️ Recent review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Advanced
Run ID: 11b8699b-b85f-48cc-af23-acc49f46802c
📥 Commits
Reviewing files that changed from the base of the PR and between 34e68ac and a1a7e89.
📒 Files selected for processing (2)
.github/workflows/release.yml
scripts/check-release-channel.ts
Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 1 remain after this review.
📝 Walkthrough
📝 Walkthrough
Walkthrough
The release-planning step skips the planner for workflow_run events when the checked-out commit lacks scripts/prepare-release.ts, and sets the release output to false. Other events continue to run the planner. The registry request now quotes the accept header key; its value and other request options are unchanged.
The historical-run guard skips planning when the checked-out commit lacks the planner, without triggering release actions. The header edit does not change registry requests; no actionable merge risk remains.
Pre-merge checks | 5
✅ Passed checks (5 passed)
Check name
Status
Explanation
Docstring Coverage
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
Title check
The title clearly summarizes both main changes: skipping stale release workflow results and quoting registry headers.
Description check
The description directly explains the stale workflow result problem and the registry header quoting change.
✨ Finishing Touches
📝 Generate docstrings
Commit to this branch
Create a new PR
🧪 Generate unit tests (beta)
Commit to this branch
Create a new PR
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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.
harlan-zw
changed the title
fix(ci): skip historical runs and quote registry headers
fix(ci): skip stale workflow results and quote headers
Oct 12, 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
harlan-agent-reviewApprove automated work for the current issue state or pull request head commit.
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.
🔗 Linked issue
Follow-up to #956.
📚 Description
The release workflow can receive CI results from commits made before release automation existed. Those checkouts lack the planning script, so release preparation fails.
Skip those historical CI results. Use consistent quoting for registry request headers to satisfy the lint rule.
The package size report can also finish after its PR closes or moves to another commit. Skip comments for those stale results.