ci(pipeline): re-pin ci-pipeline.yml to standards#989 squash SHA ed5e3f65 - #122
Conversation
This caller was pinned to 2eb9bb19, which predates hyperpolymath/standards#985. That commit fixed the ci-pipeline detect gate; every caller still on 2eb9bb19 runs the pre-fix gate and goes red on its next pull_request. Re-pins to f95130bb (the #985 squash merge) and updates the provenance comment to the matching blob 8bcdeaf2. Both lines move together: a SHA bump that leaves a stale blob comment behind is a phantom pin, unverifiable against the tree it claims to name. f95130bb was chosen over current main (d1bd7f42) deliberately: ci-pipeline.yml is byte-identical at both (blob 8bcdeaf2), f95130bb is the reviewed revision, and its only red checks are SonarCloud and the mirror trio -- all four listed in standards' own config/rulesets/gates.json never_required_workflows. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Ji1bq3TypfycfUPAR7hSxR
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📜 Recent review details⏰ Context from checks skipped due to timeout. (23)
🔇 Additional comments (1)
📝 SummarySummary by CodeRabbit
WalkthroughThe standards pipeline now invokes the shared CI workflow at commit ChangesStandards workflow update
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The pin update is ready to merge with no identified correctness or availability risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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. A rabbit checks the pipeline line Comment |
⛔ Do not merge — superseded by hyperpolymath/standards#989Converted to draft to keep it out of the merge queue. This is reversible: This PR re-pins Merging this would move the caller from a stale-but-latent Correct sequence: merge #989 first, then retarget this PR to #989's squash SHA 🤖 Generated with Claude Code |
|
✅ Coding Agent task started: View task and status The task will inspect the CI failures, validate its fix, and commit the fix to this branch automatically.
|
f95130bb predated hyperpolymath/standards#989 and carried both defects that PR cures: the exemption-ledger checkout read `ref: a2ff696a` (an abbreviated SHA, which `git fetch` cannot resolve) and `detect` did its own `exit 1`, so a ledgered repository went red regardless of its exemption. Re-pins to ed5e3f65 - the squash merge of #989 on standards' main - where the ledger is read at `${{ job.workflow_sha }}` and `report` is the single judge. Proven on the pilot before this wave: AcceleratorGate.jl#40, dispatch run 35770740241, printed `ledger read at 'ed5e3f651305dd1ce0d0b5d2d08b97a963634632'` - the callee's own commit, 40 hex, non-empty - and the run was green while correctly refusing. The provenance comment moves with the pin: a stale comment is a phantom pin. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Ji1bq3TypfycfUPAR7hSxR
What
Re-pins this repository's
standards-pipeline.ymlcaller from2eb9bb19tof95130bb— the squash merge of hyperpolymath/standards#985.Why
2eb9bb19predates #985, which fixed theci-pipelinedetect gate. Every callerstill on the old pin runs the pre-fix gate. The workflow is
pull_request-triggeredonly, so the failure is latent: it does not manifest until the next PR is opened here.
What moves
Two lines, together:
@ 2eb9bb19 — ci-pipeline.yml blob d2583657.@ f95130bb — ci-pipeline.yml blob 8bcdeaf2.uses:ref...ci-pipeline.yml@2eb9bb19......ci-pipeline.yml@f95130bb...A SHA bump that leaves the blob comment behind is a phantom pin — it names a tree
it can no longer be checked against — so both change or neither does.
Why
f95130bband not currentmainmainhas since moved tod1bd7f42(#988, a changelog fix).f95130bbwas chosendeliberately:
ci-pipeline.ymlis byte-identical at both — blob8bcdeaf2either way.f95130bbis the reviewed revision: the commit #985 was merged as.SonarCloud Code Analysisand the mirror trio(
mirror-gitea,mirror-disroot,mirror-codeberg) — all four are listed instandards' own
config/rulesets/gates.jsonundernever_required_workflows.Nothing ci-pipeline-related is red there.
ci-pipeline.ymlpins its own sub-reusables fully-qualified at571cc734rather than./, so this pin selects only whichci-pipeline.ymlis read — not its dependencies.Verification
provenance-comment shape were both present.
2/2changed lines, no residue of2eb9bb19ord2583657, and the new SHA present.%G?=G).🤖 Generated with Claude Code
https://claude.ai/code/session_01Ji1bq3TypfycfUPAR7hSxR