Uh oh!
There was an error while loading. Please reload this page.
feat(verify): ADR-0060 P1 — reusable conformance-ledger helper + unify two ledgers - #2104
Merged
Merged
Conversation
…y two ledgers Promote the conformance-ledger discipline (hand-written twice: ADR-0056 D10 authz matrix, ADR-0058 D7 expression surface) to a reusable platform capability: - @objectstack/verify gains a `conformance` module: `ConformanceRow` + `checkLedger (rows, opts): string[]` — returns problems (empty = sound), so the helper carries no test-runner dependency. It encodes the shared invariants once (unique ids, valid state, enforced-has-enforcement, experimental/removed-has-note, proof-file- exists, high-risk-has-proof, exactly-one-cover) and the ratchet (discover the real surface from source; fail on unclassified or stale covers). 12-case unit test. - authz-conformance + expression-conformance refactored onto checkLedger: one call replaces the duplicated assertion logic. The expression ledger's `site` field is unified to `enforcement` and ExprSurface now `extends ConformanceRow`; its expression-specific invariants (mode/dialect/fail-policy, compile rows name the canonical compiler) stay local. Ratchet verified still has teeth. Green: full build 75/75, dogfood 133 (incl. 16 conformance tests). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
📓 Docs Drift CheckNo hand-written docs reference the 2 changed package(s). ✅ |
Uh oh!
There was an error while loading. Please reload this page.
This was referenced Jun 21, 2026
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ADR-0060 (#2103) P1: promote the conformance-ledger discipline to a reusable platform capability.
@objectstack/verifynow exportscheckLedger(rows, opts): string[]+ConformanceRow— the static complement to its runtime harness, encoding the invariants the platform hand-wrote twice (ADR-0056 D10 authz + ADR-0058 D7 expression): unique ids, valid state, enforced-has-site, experimental/removed-has-note, proof-file-exists, high-risk-has-proof, exactly-one-cover, and the discover ratchet (fail on unclassified/stale surfaces). Both existing ledgers are refactored onto it (the expressionsitefield unified toenforcement;ExprSurface extends ConformanceRow).Verification: full build 75/75 · dogfood 133 (incl. 16 conformance tests). Ratchet verified still has teeth. Next: P2 — validation-rule-surface ledger (pins ADR-0020 state_machine).
🤖 Generated with Claude Code