Beagle SCM

COMMIT-006: jab commit: must inline the full diff (tree vs parent) hunks, like C COMMIT-002

The C commit: view inlines the commit's full diff as relayed graf hunks after the metadata (COMMIT-002, landed df596d0f). The JS commit: (be/views/commit/commit.js) emits ONLY the metadata hunk (commit/tree/parent/author/body) — no diff. So jab commit:?<sha> shows who/when/why but not WHAT changed. Port COMMIT-002's diff-relay to the JS view. Method: Issues.

Input

Context

• C: COMMIT-002 (df596d0f) — commit: inlines the full diff as relayed graf hunks (keeper/graf), superseding COMMIT-001's diff:?<sha> link.
• JS be/views/commit/commit.js: emits the metadata hunk only (the recent COMMIT-003/004/005 + the BRO-006 tree/parent U-links work). No diff hunks.
• The JS diff: VIEW is inert (ctx.views unset, JS-071); but the diff GENERATION (graf weave/diff binding, used by be/views/diff/diff.js) may work independently of the dead view-routing — the producer can call it directly.

Goals

• After the metadata, jab commit:?<sha> inlines the FULL diff of the commit's tree vs its parent's tree, as relayed hunks — matching C be commit:.
• ALL commits diff vs the FIRST parent (parent¹): non-merge = its one parent, MERGE (2+ parents) = parent¹ (the mainline net change, git --first-parent). Root (0 parents): diff vs the empty tree (all-added) or skip.

Constraints

• Reuse the graf diff machinery (be/views/diff/diff.js's hunk generation / the graf binding); do NOT depend on the inert diff VIEW routing (JS-071). PRESERVE COMMIT-003/004/005 + the BRO-006 tree/parent U-links (uncommitted in commit.js).
• Plain/colour/tlv parity with C be commit: where feasible. Repro-first (CLAUDE §17).

WIP

Design decisions

• RULING (user 2026-06-26): the diff is ALWAYS commit.tree vs parent[0].tree (the FIRST parent) — merges included (git --first-parent; consistent with beagle's first-parent log spine, LOG-001). No merge special-case, no skip.
• DRIVE: call the diff VIEW handler (../diff/diff.js) with a PINNED views spec {mode:range, fromSha:parent[0], toSha:sha} — bypasses dead ctx.views.
• ATTACH: PLAIN folds the diff TEXT (out.chunk) into the ONE metadata record (HUNK plain adds a sep PER record; native has none mid-stream). COLOR/TLV feed the diff's own records after metadata (no sep; native tlv = 2 hunks).
• ROOT (0 parents): SKIP (no base; native's root diff is wt-driven/odd — skip is simpler). All others (incl. merges) diff vs parent[0].

TODOs

• Repro-first: jab commit:?<sha> 1-parent shows NO diff (RED, verified); after, the diff hunks follow the metadata (asserts vs native be commit:).
• Inline the diff vs parent¹ for ALL commits (non-merge + merge alike, real 2-parent merge tested vs native); root → SKIP.
• Preserve metadata + U-links + plain parity; be/ suite green (bar 1 pre- existing unrelated stale xfail be-js-bro-universal).

Blockers and bummers

• Merge base DECIDED (user): the FIRST parent (parent¹), git --first-parent style — uniform with the non-merge path. JS-071 (dead diff view): call the diff generator directly, not via ctx.views.

Outcome

• LANDED: be aef4caef (commit.js) + be/test 6a8c8b3c (links+diff tests). Fix: be/views/commit/commit.js — require ../diff/diff.js (l.39); gate + diffPlainText/diffFeedRecords/diffSpecRow helpers (l.267-307); inline at the handler feed (l.382-407): inline = parents.length>=1 ? parents[0], PLAIN folds diff text into the 1 record, COLOR/TLV feed diff records after.
• Test: be/test/commit/diff/run.sh (CONFIGURE_DEPENDS, no CMakeLists edit) — 1-parent inline+native-parity, root SKIP, real 2-parent merge first-parent diff vs native. RED-verified (no '+++' pre-fix). be-js-commit-diff GREEN.
• Preserved: COMMIT-003/004/005 metadata + BRO-006 U-links (render/resolve/ links GREEN). links/check.js hunks.length==1→>=1 (FORCED one-line relax: native commit: --tlv now = 2 hunks; U-target logic on hunks[0] unchanged). Color = best-effort (be --color pages via bro, not a producer oracle).