Skip to content

board: V1 contiguous-u24 GUID tail forbidden — V3 4+12 facet mandatory (migration marked)#645

Merged
AdaWorldAPI merged 1 commit into
mainfrom
claude/v3-substrate-migration-review-o0yoxv
Jul 4, 2026
Merged

board: V1 contiguous-u24 GUID tail forbidden — V3 4+12 facet mandatory (migration marked)#645
AdaWorldAPI merged 1 commit into
mainfrom
claude/v3-substrate-migration-review-o0yoxv

Conversation

@AdaWorldAPI

@AdaWorldAPI AdaWorldAPI commented Jul 4, 2026

Copy link
Copy Markdown
Owner

What

Marks the flat V1 GUID key-tail (family(u24) ++ identity(u24)) as forbidden for new units and records the V1→V3 migration of the two live residues as MANDATORY (deferred in timing, not obligation). Docs/board only — no code churn this PR.

Why

The tail bytes 10..16 of the 16-byte NodeGuid have three historical shapes:

  • V1 (FORBIDDEN for new): family(u24) ++ identity(u24) — two contiguous flat integer fields, no axis. NodeGuid::new / family() / identity().
  • V2: leaf·family·identity — three flat u16s (deprecated integer read).
  • V3 (REQUIRED for new): the content-blind 4+12 facetclassid(4) + 12-byte payload. The 12B is a dumb byte register the ClassView projects, holding every sanctioned reading at once: 6×(u8:u8) rails / 4×(u8:u8:u8) SPO / 3×(u8:u8:u8:u8) odoo quads (6·2 = 4·3 = 3·4 = 12). u8:u8 = two separate bytes, never widened to u16/u24.

A flat u24 has no axis — it cannot carry a rail (X:Y) — so it is a structurally retired shape, not a width variant. Operator-locked E-V3-FACET-4-PLUS-12 (2026-07-02) + this session's ruling.

Conformance sweep (3 read-only repo audits)

  • ruff: already V3 (ruff_spo_address::Facet = classid(4) + 6×(is_a:part_of)). No V1. No-op.
  • OGAR: code already V3; only docs pinned/defended V1 → superseded in the OGAR canon PR (CLAUDE.md P0 regraded, NODEGUID-CANON-AUDIT F-3 inverted, DISCOVERY-MAP D-V1-TAIL-RETIRED).
  • lance-graph: exactly two live V1 producers — contract/src/ocr.rs:121 (Integrated cognitive planner reference map + ValueSchema presets + FULL POC default #496 OCR keystone) + contract/src/aiwar.rs:104 — both hardcode NodeGuid::new instead of mint_for. Behaviorally fine today (V1-default classids); MANDATORY to migrate when their output paths move onto a V3-marked classid.

Changes (all append-only / regrade-in-place)

  • .claude/board/EPIPHANIES.mdE-V1-TAIL-FORBIDDEN-V3-IS-CONTENT-BLIND-1 (RULING).
  • .claude/board/ISSUES.mdISS-V1-TAIL-RESIDUE (OPEN — MIGRATION MANDATORY), names the two mint sites.
  • CLAUDE.md — CANON block gets an append-only supersession note marking the u24 tail V1-LEGACY/read-only, pointing to the V3 4+12 facet. The pin is not deleted.

Reads of the u24 tail on existing V1 rows survive unchanged (I-LEGACY-API-FEATURE-GATED); only new mints are forbidden.

🤖 Generated with Claude Code


Generated by Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated project guidance to clarify the supported tail format for new units and mark the older layout as legacy-only.
    • Added migration notes for existing entries, including required routing for new minting and read behavior.
    • Recorded a new issue-log entry tracking remaining legacy usage and the needed migration guardrails.

…atory

The flat key-tail shape `family(u24) ++ identity(u24)` (bytes 10..16 of the
16-byte NodeGuid) is V1-LEGACY and forbidden for new units. A flat u24 has no
axis — it cannot carry a rail (`X:Y`) — so it is a structurally retired shape,
not a width variant. Every new unit is the V3 content-blind 4+12 facet
(`classid(4) + 12-byte payload`); the 12B is a dumb byte register the ClassView
projects, holding every sanctioned reading at once (6×u8:u8 rails / 4×u8:u8:u8
SPO / 3×u8:u8:u8:u8 quads; u8:u8 stays two bytes, never widened).

Reads of the u24 tail on existing V1 rows survive unchanged
(I-LEGACY-API-FEATURE-GATED); only new mints are forbidden.

- EPIPHANIES: E-V1-TAIL-FORBIDDEN-V3-IS-CONTENT-BLIND-1 (RULING, operator) —
  the three tail shapes, why u24 is forbidden, the 3-repo conformance sweep
  (ruff already-V3 no-op, OGAR docs superseded / code already V3, lance-graph
  2 live residues), and the mandatory follow-ups.
- ISSUES: ISS-V1-TAIL-RESIDUE (OPEN — MIGRATION MANDATORY) — names the two
  pre-existing NodeGuid::new (V1) mint sites (ocr.rs:121, aiwar.rs:104) that
  must route through mint_for onto a V3-marked classid; reads stay.
- CLAUDE.md CANON block: append-only ⊘ supersession note marking the u24 tail
  V1-LEGACY/read-only, pointing to the V3 4+12 facet; the pin is not deleted.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01MLBnPuScZy6w9di2QEjsXM
@coderabbitai

coderabbitai Bot commented Jul 4, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 15dfb2a3-3342-4a2b-9313-1b757ce2b877

📥 Commits

Reviewing files that changed from the base of the PR and between 7384bc3 and ef6dd16.

📒 Files selected for processing (3)
  • .claude/board/EPIPHANIES.md
  • .claude/board/ISSUES.md
  • CLAUDE.md

📝 Walkthrough

Walkthrough

This PR updates project documentation across three markdown files to formally forbid the legacy V1 flat family(u24)+identity(u24) tail for new units, mandate the V3 content-blind classid+payload facet, log a tracked migration issue for two existing mint sites, and update canonical layout comments accordingly.

Changes

Tail-Variant Spec Ruling Documentation

Layer / File(s) Summary
EPIPHANIES ruling: V1 forbidden, V3 defined
.claude/board/EPIPHANIES.md
Documents forbidding the V1 flat tail for new units, defines V3's byte-register semantics, reports a conformance sweep, and lists mandatory follow-ups with cross-references.
DeepNSM convergence finding
.claude/board/EPIPHANIES.md
Adds a finding clarifying DeepNSM is an encoder filling V3 tenants rather than a migration onto V3.
Issues log: NodeGuid migration tracking
.claude/board/ISSUES.md
Logs a mandatory issue requiring two existing NodeGuid::new mint sites to migrate to mint_for/tail_variant dispatch, plus a grep guard and a feature-gating blocker note.
CLAUDE.md canon layout supersession
CLAUDE.md
Adds a SUPERSEDED migration-mandatory directive and revises the key-tail layout comment to distinguish the retired V1 tail from the new V3 content-blind facet.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Poem

A rabbit scribes with careful paws,
"Forbid the tail with u24 flaws!"
V3 now holds the content-blind crown,
Twelve dumb bytes, rails handed down.
Two old mint sites await their turn,
The board records what we must learn. 🐇📜


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@AdaWorldAPI AdaWorldAPI merged commit 938d51e into main Jul 4, 2026
1 of 2 checks passed
AdaWorldAPI pushed a commit that referenced this pull request Jul 4, 2026
… ruff phases

The full operator-directed review pipeline, complete: Fable-5 draft ->
5x Sonnet PR-history drift audit (lance-graph #618-#645, OGAR #139-#151,
ruff #33-#41, ndarray ruled out) -> 5x Opus savant review (iron-rule,
dto-soa, prior-art, cascade-impact, creative-explorer) -> 3x Opus brutal
review (overclaim RESTATE-10, dilution REPAIR-3, baton CATCH-CRITICAL-1)
-> all findings folded into the ratified text.

Registry: 2 new parent laws (E-BOUNDARY-FUSE-1,
E-REGISTRY-COMPLETENESS-ORACLE-1) + 14 instance rows, all PENDING the
epiphany-brainstorm-council (deliverable B1); 9 OVERLAPS rows carry
mandatory cross-refs, 2 rows file at CONJECTURE. Includes the
self-caught E-PLAN-GROUND-STATE-IS-A-RANGE-1 (ruff main moved #39->#41
mid-draft; cite the executable, never the prose).

Deliverables: unified ruff phase sequence (A1 mints a NEW branch per
BH-4 -- never re-point the shared branch name; A2a freeze at derived
count 62; A4 rescoped as a feature; A7 blocked on Q-A7 after BH-5's
meta-membrane finding), council filing + V3-folder drafts with 7-day
fallback, OGAR doctrine bullets, and consumer-side baton homes (the
ruff board file carries the plan pointer + gate markers).

Board hygiene in same commit: INTEGRATION_PLANS.md prepend +
STATUS_BOARD.md D-EPI-* rows. Doc-only; execution queued behind D1->A1.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RhpwkHGgia2TuDFvdnuQdE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants