Pin the toolchain to kaizen-loop v0.1.4 and verifier v0.1.1 - #219
Conversation
kaizen-loop v0.1.4 derives the verifier expectedRef from the pinned manifest (#362), so a pinned install no longer fails the verifier freshness preflight once verifier main moves ahead of the pinned tag. verifier v0.1.1 restores the mechanical verification evidence the verifier is supposed to weigh (verifier#221, fixes kaizen-loop#357).
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: kaizen-agents-org/coderabbit/.coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe onboarding manifest updates ChangesOnboarding version pins
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related issues
Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
PR GuardianMerge-ready. No fixes were needed — no actionable feedback was raised. Feedback inventory
Final gateTwo snapshots taken 35s apart after the stabilization wait were identical: Base is current with Notes on two items I did not treat as blockersCodex did not review. It is not in the required status checks CodeRabbit's "no linked issues found". The two issues this release closes, |
Pins the onboarding toolchain to a newly verified compatible set.
5e0379128fc7f4What each tag carries
kaizen-loop v0.1.4 derives the verifier
expectedReffrom the pinnedtoolchain manifest (kaizen-loop#362). Before this, a pinned install failed the
verifier freshness preflight as soon as verifier
mainmoved ahead of thepinned tag — the config said
refs/heads/mainwhile the installed build was atag. The config schema now accepts
refs/tags/as well asrefs/heads/, andannotated tags are peeled with
ls-remote ... ^{}.verifier v0.1.1 restores the mechanical verification evidence the verifier
is supposed to weigh (verifier#221, fixes kaizen-loop#357).
verify_commandsis populated from the verification log block and
evidence_gradeis upgradedto
executedwhen those commands are present.Verification
kaizen-loop at
5e03791— 672 tests passed,typecheckclean,check:distcurrent.verifier at
28fc7f4— the repository's full CI contract from a cleancheckout:
typecheck,test,build,test:package-entry,test:built-cli,schema:check,eval,fixture-run,eval:relaxations,the semantic eval CI gate, and
git diff --exit-codeclean.Producer/parser pairing. The verifier's parser only extracts commands from
a
<verification_logs_data>block wrapping a```markdownfence, andreturns empty silently on any mismatch — which would look exactly like the
original bug. kaizen-loop's real producer output was fed to the real parser:
block found, fence matched (length 4), records parsed. The two halves pair.
Clean install from this manifest into an isolated prefix, all three
components resolving into
$KAIZEN_HOME:kaizen doctor --repair --project <registered slug>— 12/12 checks pass,including
gh authandverifier agent, with verifiermain(28fc7f4)deliberately ahead of the pinned tag. That is the exact condition that failed
before kaizen-loop#362.
End-to-end run on a Rust repository (
s-hiraoku/topcoat-sandbox), which isthe evidence for kaizen-loop#357:
verify.logis 338 lines withtest result: ok. 3 passed. Before v0.1.1 thissame payload had
verify_commands: []andevidence_grade: "reported".Known limitation, stated deliberately
Configs generated by
kaizen initunder v0.1.4 useexpectedRef: refs/tags/..., which does not parse under v0.1.3 or earlier — the olderschema regex only accepts
refs/heads/. Adopters cannot downgrade afterrunning
initwith this set. This is recorded in the v0.1.4 tag message.Not covered by this PR
The smoke run above reached a verifier verdict but could not publish a PR:
the publication broker refuses the push with a generic failure. Filed
separately as #217 (workspace mode) and the broker-side temp-ownership
problem; kaizen-loop#368 records the same failure observed from inside a run.
Those are broker/workspace defects, independent of the three pinned tags.
Summary by CodeRabbit