Skip to content

Commit c249d10

Browse files
committed
fix(docs): rename V2 Changes subsection to Layout Notes in graph.rs
The subsection referenced the old DIND_STATE_HASH_V2 versioning scheme which created ambiguity next to the echo:state_root:v1 header. Renamed to Layout Notes per CodeRabbit review feedback.
1 parent 80f0be8 commit c249d10

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

CHANGELOG.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,9 @@
2525
knowledge as a structured implementation companion to ADR-0007.
2626
- **Stale Hash Domain:** Updated three stale `DIND_STATE_HASH_V2` references in
2727
`graph.rs` doc comment and `ADR-0007-impl.md` §2.1/§7 to match the actual
28-
domain prefix `echo:state_root:v1` defined in `domain.rs`.
28+
domain prefix `echo:state_root:v1` defined in `domain.rs`. Renamed the
29+
adjacent `V2 Changes` subsection to `Layout Notes` to remove versioning
30+
ambiguity.
2931
- **Module Count:** Fixed off-by-one module count in `ADR-0007-impl.md` metadata
3032
and §1 prose (36 → 37) and added qualifier noting tables cover key modules only.
3133
- **Stale Design Doc:** Deleted `docs/WARP-GRAPH.md` (1,219-line chat transcript

crates/warp-core/src/graph.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -483,9 +483,9 @@ impl GraphStore {
483483
/// 4. Edge Count (u64 LE)
484484
/// 5. Edges (sorted by `EdgeId`): `b"E\0"` + `EdgeId` + From + To + Type + Attachment(if any)
485485
///
486-
/// # V2 Changes
486+
/// # Layout Notes
487487
///
488-
/// V2 uses `u64` for all counts and lengths (node count, edge count, blob length)
488+
/// Uses `u64` for all counts and lengths (node count, edge count, blob length)
489489
/// to align with the WSC format and avoid truncation issues with large graphs.
490490
#[must_use]
491491
pub fn canonical_state_hash(&self) -> Hash {

0 commit comments

Comments
 (0)