fix(ci): repoint io-triage to wave-foundation-public mirror - #37
Conversation
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Your free trial PR review limit of 300 PRs has been reached. Please upgrade your plan to continue using CodeAnt AI. |
|
ⓘ Qodo reviews are paused because your workspace is out of credits. Ask your workspace admin to add credits to resume reviews. Manage billing |
There was a problem hiding this comment.
Sorry @yakimoto, this account has used its review budget of 2,500,000 diff characters for the last 7 days.
You can request another review in 1 day and 4 hours by commenting @sourcery-ai review.
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_eda893f5-6559-4a69-808c-39839f5a1b58) |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideFix issue triage startup failures in the public repository by switching all workflow and script dependencies to the public wave-foundation mirror at the same immutable SHA. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Team Run ID: 📒 Files selected for processing (1)
Included review availability: 0 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour. 📜 Recent review details⏰ Context from checks skipped due to timeout. (3)
🧰 Additional context used🪛 zizmor (1.29.0).github/workflows/issue-ops-triage.yml[warning] 17-17: permissions without explanatory comments (undocumented-permissions): needs an explanatory comment (undocumented-permissions) 🔇 Additional comments (2)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe issue triage job now uses the public ChangesIssue operations triage
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This localized CI-only change repoints issue triage to the pinned public mirror without changing application behavior, secrets, or permissions. No actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation 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 0 files. (1 skipped: 1 unsupported.) ✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
Code Review ✅ ApprovedFixes invisible issue triage on this public repo by repointing the OptionsAuto-apply is off → Gitar will not commit updates to this branch. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Gitar |
ApprovabilityVerdict: Would Approve Macroscope's review found this PR approvable — The PR only retargets an existing issue-ops workflow and its pinned script checkout from an inaccessible private repository to the matching public SHA; triggers, permissions, secrets, and application code remain unchanged. Its runtime effect is limited to restoring the intended issue-event triage automation. Not approved because:
Review your spending limits in Billing settings. You can add or adjust custom eligibility rules. Learn more. |
Symptom
wave-av/sdk-python'sissue-ops-triagelane has zero recorded runs at all (gh run list --repo wave-av/sdk-python --workflow issue-ops-triage.yml --json conclusionreturns[]). Not "it fails" — it never even shows up as an attempt. Issue triage has never run on this repo.Cause
wave-av/sdk-pythonis a public repository.issue-ops-triage.ymlcalled a reusable workflow hosted inwave-av/wave-foundation, a private repository. A public repo cannotuses:a reusable workflow in a private one — the call dies atstartup_failure: no jobs, no logs, and no check appears on the triggering issue event at all. It doesn't go red; it vanishes, which is exactly whygh run listshows nothing rather than a string of failures. That invisibility is why this went unnoticed.Fix — three pointers, moved together
uses:→wave-av/wave-foundation-public/.github/workflows/io-triage.yml@f63b91566ab311b09599db6195da8c353cab1f69with.scripts_repo:→wave-av/wave-foundation-publicwith.scripts_ref:→"f63b91566ab311b09599db6195da8c353cab1f69"Pointers 2 and 3 are not optional.
io-triage.ymldoesn't embed its scripts — itsparse-checkoutsscripts/issue-opsfromscripts_repoatscripts_refat runtime. Moving onlyuses:clears the startup_failure and dies one step later on a checkout of a private repo this token can't read. All three land together or the lane fails a different way.Provenance
This exact repoint (same three fields, same pinned mirror SHA) is the pilot fix already proven green at
wave-av/adk— its first successfulissue-ops-triagerun in 9 runs all-time, 2026-09-01T13:07. The pilot took five rounds (startup_failure → bad scripts_ref → missinglib.sh→ missingclassify.sh/dedup.sh/trust-tier.sh→ green); all seven scripts in the closure are now published in the mirror, so this repoint should go green first try.Verification
By run outcome, not the PR check rollup — a startup_failure produces no rollup entry:
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.Note
Low Risk
CI wiring only—no application code, secrets, or permissions change beyond pointing at a published public mirror SHA already validated on another repo.
Overview
Fixes invisible issue triage on this public repo by moving the reusable workflow and runtime script checkout off the private
wave-av/wave-foundationrepo to the public mirrorwave-av/wave-foundation-public, all pinned to commitf63b915.The change updates three coordinated fields in
issue-ops-triage.yml: theuses:workflow reference,with.scripts_repo, andwith.scripts_ref. Public repositories cannot call reusable workflows in private repos (startup failure with no runs recorded); the script inputs must move withuses:becauseio-triagesparse-checkoutsscripts/issue-opsfrom that repo at runtime.Reviewed by Cursor Bugbot for commit f8f0f77. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by Sourcery
Enable issue triage in the public repository by using the pinned workflow and scripts from the public foundation mirror.
Bug Fixes:
CI: