Repository navigation
docs(adr): ADR-003 trust tiers as admission gates plus signing (Proposed) - #160
Conversation
…sed) Records the owner's 2026-10-07 rulings on the tier enum: each tier is a cumulative set of admission requirements (Ayo, then Teranga, then Shield) plus a signing authority (the owner key for Teranga and Shield). The rule is enforced in two places: a required CI check here, and the boj-server loader against the signed index. All 11 Teranga assignments stay. The 10 that fall short (9 are manifest-only, 1 has no tests) are listed in a shrink-only exemption ratchet, declared before enforcement starts. No manifest changes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019j8She9eTFx54r6aL6sCHP Signed-off-by: Jonathan D.A. Jewell <6759885+hyperpolymath@users.noreply.github.com>
|
Caution Review failedFailed to post review comments. GitHub was unavailable or timed out while CodeRabbit was posting the review. Please request a new review later if the pull request still needs one. This happened while posting 1 inline comment. Use ⚙️ Run configuration
📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. ⏰ Context from checks skipped due to timeout. (13)
📝 SummarySummary by CodeRabbit
WalkthroughADR-003 defines cumulative admission requirements and signing rules for Ayo, Teranga, and Shield. It specifies CI and loader enforcement, records current assignments and exemptions, and lists follow-up work and rejected alternatives. ChangesTrust-tier admission
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Security Architecture ReviewSecurity architecture risk: 🔵 Low · up to This proposed decision establishes security-relevant admission rules but changes no executable behavior. Risk is low because the authorization and recovery guarantees for temporary exemptions remain to be demonstrated before enforcement. Retained concerns Security review detailsSecurity Blast Radius
Security Findings and Attack Paths
Trust Boundaries and Controls
Resilience and Maintainability Implications
Hardening Proposals
🚥 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 reads the tiers by moonlit light, Comment ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
|
Summary
Adds
docs/decisions/ADR-003-trust-tiers.adocwith status Proposed. It records the owner's 2026-10-07 rulings on the existingtierenum (Teranga / Shield / Ayo), which nothing enforces today:orchestrator-lsp-mcp, which has no tests) are listed in a shrink-only exemption ratchet, declared before enforcement starts.No manifest, code or workflow changes. The ADR becomes Accepted when the CI check lands.
How has this been verified?
test -d <path>/{abi,ffi,adapter,tests}for every ledger row with"tier":"Teranga"(ledger from docs(split): Phase 0 survey and per-cartridge JSON Lines ledger #159).7dd5897d.tieris consumed only byrouter.exL70–73 andcatalogue.zigMenuTier.loader.zigloadCartridgetakes an optionalexpected_hashand checks no signature..github/workflows/cartridge-schema.ymlexists.asciidoctor --failure-level=WARNreturned rc 0.📌 New pins
Head
51c2d0ab2c2aab6d33ef83a0d7aea584a775d4e5. No action, lockfile or digest pins are added or changed.Schema-validation note
cartridge.jsonmanifests.cartridge.json. The schema description change fortieris a follow-up inhyperpolymath/standards.Refs
🤖 Generated with Claude Code
https://claude.ai/code/session_019j8She9eTFx54r6aL6sCHP