Repository navigation
fix(k9): bring the 18 K9 files to the validator's contract - #82
Conversation
Validate K9 contracts failed on main with 20 errors in 18 files (scaffold debt
from the template, not from any recent change). Brought to 0 errors and 0
warnings, also under INPUT_STRICT=true, by the same moves systemet already made:
- K9! magic line first, on the 8 contractile, arrival-pack and coaptation
files and on methodology-guard.
- signature_required = true in the security block of the four Hunt-tier
contractiles (must, bust, intend, trust), as validator rule K9-S008 asks.
- methodology-guard gains a Yard-leash pedigree block.
- coordination.k9 and session/custom-checks.k9 were YAML under a K9
extension, so they could not carry a magic line. They become
coordination.k9.ncl and session/custom-checks.k9.ncl, Nickel records with
a Kennel pedigree and the same content, copied from systemet.
- container/stapeln/deploy.k9.ncl still holds unfilled {{PLACEHOLDERS}}, so
it is a template, not a contract: renamed to deploy.k9.ncl.in, as systemet
did.
- root-allow.txt and session/README.adoc follow the renames.
Deliberately not changed: the methodology-guard copyright line still carries
{{CURRENT_YEAR}} {{AUTHOR}} placeholders. Filling it is a licence-header edit,
which is manual and owner-only. The SPDX line is present, so the validator is
satisfied. The comments in PLAYBOOK.a2ml and the historical governance docs
still say coordination.k9; they are canon-derived or dated records.
No consumer in this repo evaluates the .k9.ncl files with nickel (the arrival
pack and coaptation scripts read a2ml and the deed, not these), so a leading
K9! line cannot break a runtime path here. validate-template.sh output is
identical before and after; the arrival-pack and coaptation drift gates and the
root-shape check pass.
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01J1XmMC49T8sZ3uUuvrgB7y
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 8 minutes. View limit details
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. |
|
|
🤖 Generated with Claude Code https://claude.ai/code/session_01J1XmMC49T8sZ3uUuvrgB7y Generated by Claude Code |
Codex Review Summary: ❌ FailedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
What
Validate K9 contractshas failed on main with 20 errors in 18 files. This brings it to 0 errors, 0 warnings (also underINPUT_STRICT=true), using the same moves systemet already made.Changes
K9!magic line first on the 8 contractile / arrival-pack / coaptation K9 files and onmethodology-guard.signature_required = truein the security block of the four Hunt-tier contractiles (must,bust,intend,trust), per validator rule K9-S008.methodology-guardgains a Yard-leash pedigree block.coordination.k9andsession/custom-checks.k9were YAML under a K9 extension and could not carry a magic line. They becomecoordination.k9.nclandsession/custom-checks.k9.ncl: Nickel records with a Kennel pedigree, same content, copied from systemet.container/stapeln/deploy.k9.nclstill holds unfilled{{PLACEHOLDERS}}, so it is a template, not a contract: renamed todeploy.k9.ncl.in(as systemet did).root-allow.txtandsession/README.adocfollow the renames.Deliberately not changed
methodology-guardcopyright line still carries{{CURRENT_YEAR}} {{AUTHOR}}placeholders. Filling it is a licence-header edit, which is manual and owner-only. The SPDX line is present, so the validator passes.PLAYBOOK.a2mlcomments and the historical governance docs still saycoordination.k9(canon-derived / dated records).Verified
.github/hooks/validate-k9.sh: 17 files scanned, 0 errors, 0 warnings (strict too).check-root-shape.sh, arrival-packverify.sh, coaptationverify.sh: pass.validate-template.shoutput identical before and after..k9.nclfiles withnickel, so a leadingK9!cannot break a runtime path here.Generated by Claude Code