rfc(0001): v0.4.1 — definitional completions and conformance-doc corrections#7
Conversation
…ections Draft patch per §6.3: clarifications only, no conformance change. Re-cut of superseded PRs #5/#6 against the v0.4.0 baseline (f70c796). - §1.1: complete the standard-admission-path definition with the §3.4 loop-origin authorization step the path already performs (NEW-N-3) - §5.6: annotate the evidence-field SHOULD-list with per-event-type MUST pointers to the upgrading paragraph, incl. principal_authorization on loop-originated memory_entry_admitted events (NEW-N-1) - negative/09 CASE.md: correct false 'warning only' claim (violation since v0.2.0); positive/05 CASE.md: cite §5.3 instead of 'v0.2 frame' - §11/CHANGELOG: v0.4.1 entries; sweep current-version refs to v0.4.1 (README, PRIMER, SPEC, faq, rfcs/README, conformance+validator READMEs, validate.js banner, package.json+lock); historical refs and the deep-dive's intentional v0.3.0 pin untouched Conformance: 6/6 positive, 36/36 negative, READY; both examples PASS. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Fresh-context Nova-lane review — verdict: PASSReviewed at pinned a. Draft-patch classification (new §6.3) — SOUND. NEW-N-3's §1.1 wording describes an obligation verbatim-present in §3.4 since the v0.3 snapshot; NEW-N-1's three bullet annotations match the unchanged MUST paragraph word-by-word (pre-change, the b. Internal consistency — CLEAN. §1.1 ↔ §3.4 agree; §5.6 bullets ↔ MUST paragraph agree on event types + loop-originated condition; reinforces §3.4's provenance-vs-authorization distinction. c. CASE.md corrections — NOW TRUE. Fixture 09 re-run: exit 1, d. Version sweep — COMPLETE. Residual e. Conformance — GREEN. 6/6 positive, 36/36 negative, READY; both examples PASS; banner v0.4.1. f. Changelog/§11 — ACCURATE. 'Violation since v0.2.0' verified against the v0.2 snapshot (line 134) and validator history. Findings
Remaining gateVal approval → mark ready → merge → tag Bead: threads-auu. |
There was a problem hiding this comment.
Pull request overview
Updates RFC-0001 and repository documentation to v0.4.1 as a draft patch, clarifying existing cross-references (no conformance-requirements change) and correcting conformance-suite documentation that previously misstated validator behavior.
Changes:
- Bump current-version references across the RFC, docs, validator banner, and package metadata to v0.4.1.
- Clarify RFC-0001 definitions and audit-log evidence-field bullets to explicitly point at existing MUST-level requirements.
- Fix conformance-case documentation anchors (notably the missing
MEMORY.mdcase) and add v0.4.1 entries to changelogs.
Reviewed changes
Copilot reviewed 13 out of 14 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| validators/validate.js | Updates CLI banner to v0.4.1 (version sweep). |
| validators/README.md | Updates validator documentation and sample output to v0.4.1. |
| tests/conformance/README.md | Updates conformance-suite title/claim text to v0.4.1. |
| tests/conformance/positive/05-tier-rich-ward/CASE.md | Reanchors tier-name citation to RFC-0001 §5.3. |
| tests/conformance/negative/09-missing-memory/CASE.md | Corrects description of missing MEMORY.md behavior and anchors to RFC-0001 §3.4. |
| SPEC.md | Updates “current normative specification” pointer to RFC-0001 v0.4.1. |
| rfcs/RFC-0001-familiar-contract.md | Bumps RFC to v0.4.1; clarifies definition + audit-log bullets; adds v0.4.1 changelog entry. |
| rfcs/README.md | Updates RFC index and citation examples to v0.4.1. |
| README.md | Updates badge and usage text to v0.4.1. |
| PRIMER.md | Updates normative spec reference to v0.4.1. |
| package.json | Bumps package version to 0.4.1. |
| package-lock.json | Bumps lockfile version fields to 0.4.1. |
| docs/faq.md | Updates FAQ references to RFC-0001 v0.4.1. |
| CHANGELOG.md | Adds 0.4.1 entry and summarizes the clarifications/doc corrections. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| } | ||
|
|
||
| console.log(`\n${bold('familiar-contract validator')} ${dim('v0.4.0')}`); | ||
| console.log(`\n${bold('familiar-contract validator')} ${dim('v0.4.1')}`); |
RFC-0001 v0.4.1 (draft patch)
Re-cut of superseded PRs #5/#6 against the v0.4.0 baseline — main moved beneath that stack when #4 merged (f70c796), absorbing their version sweeps but not the definitional nits or the conformance-doc falsehood, which land here.
Patch classification (new §6.3): draft patch — clarifications only; no conformance requirements change. Prior Nova-lane PASS on the identical normative delta: #6 (comment 5041608668).
Normative text (clarifying)
(MUST — see below)pointers; theprincipal_authorizationbullet's condition now includes loop-originatedmemory_entry_admittedevents (matching the MUST paragraph verbatim).Conformance-doc corrections
MEMORY.mdis reported "as a warning only" (violation since v0.2.0; verified against the v0.4.0 Ajv validator: exit 1, Persistent Memory ✗). Stale v0.2 anchor dropped.Version sweep → v0.4.1
RFC header/abstract/§6.1×2/§6.3/§10.1/footer + §11 entry; CHANGELOG [0.4.1]; README (badge + 5 claims); PRIMER.md; SPEC.md banner; docs/faq.md ×3; rfcs/README.md (table/citation/tag URL); tests/conformance/README.md; validators/README.md ×4; validate.js banner; package.json + lock. Untouched by design: historical changelog/§11/snapshot refs, PROVENANCE.md, docs/ward-deep-dive.md (intentionally pinned to the v0.3.0 snapshot), superpowers plan docs (historical).
Verification
bash tests/conformance/run-conformance.sh→ 6/6 positive, 36/36 negative, READYnode validators/validate.js examples/minimalandexamples/sage→ PASS, banner v0.4.1v0.4.0grep: historical positions onlyGates
Fresh-context Nova-lane review → Val approval → merge → tag
v0.4.1.Bead: threads-auu