F7.1: intra-diagram mermaid diffing (#22) #28

2026-06-11T23:36:32Z

benstull commented

2026-06-11 23:36:32 +00:00

Summary

Refines the F7 rendered track-changes preview's whole-diagram "changed" badge
into a node/edge-level diff inside flowchart and sequence diagrams against the
F6 baseline (task #22; design = spec §11, INV-29..31).

Parsed-graph diff — new pure host modules parse a changed mermaid block's
before/after source into a typed element model, diff it, and re-emit the
current diagram source augmented with mermaid's own styling directives so
mermaid renders a colored diagram. Host stays pure (INV-22); webview unchanged
(INV-21); nothing persisted (INV-20).
- src/mermaidDiff.ts — dispatch + total fallback (never throws)
- src/mermaidFlowchartDiff.ts — node/edge diff -> classDef/class/linkStyle
- src/mermaidSequenceDiff.ts — participant/message diff -> rect rgb(...) runs
Flowchart + sequence this increment; every other type, wholly added/removed
blocks, and any parse failure fall back to the v1 badge (INV-30).
Removed elements ghosted in place (faded/dashed), re-injected at baseline
position (INV-31). Layout reflow accepted (no position pinning).
Legend beneath a diffed diagram (media/preview.css).

Test Plan

npm run typecheck — clean
npm test — 189 unit tests green
npm run test:e2e — 38 + 5 green (new: a changed flowchart augments the
emitted mermaid source via the renderHtmlFor seam)
Manual webview-render smoke (docs/MANUAL-SMOKE-F7.1.md) — colors paint

Fixes #22

## Summary Refines the F7 rendered track-changes preview's whole-diagram "changed" badge into a **node/edge-level diff inside flowchart and sequence diagrams** against the F6 baseline (task #22; design = spec §11, INV-29..31). - **Parsed-graph diff** — new pure host modules parse a changed mermaid block's before/after source into a typed element model, diff it, and re-emit the *current* diagram source **augmented with mermaid's own styling directives** so mermaid renders a colored diagram. Host stays pure (INV-22); webview unchanged (INV-21); nothing persisted (INV-20). - `src/mermaidDiff.ts` — dispatch + total fallback (never throws) - `src/mermaidFlowchartDiff.ts` — node/edge diff -> `classDef`/`class`/`linkStyle` - `src/mermaidSequenceDiff.ts` — participant/message diff -> `rect rgb(...)` runs - **Flowchart + sequence** this increment; every other type, wholly added/removed blocks, and any parse failure fall back to the v1 badge (INV-30). - **Removed elements ghosted in place** (faded/dashed), re-injected at baseline position (INV-31). Layout reflow accepted (no position pinning). - Legend beneath a diffed diagram (`media/preview.css`). ## Test Plan - [x] `npm run typecheck` — clean - [x] `npm test` — 189 unit tests green - [x] `npm run test:e2e` — 38 + 5 green (new: a changed flowchart augments the emitted mermaid source via the `renderHtmlFor` seam) - [ ] Manual webview-render smoke (`docs/MANUAL-SMOKE-F7.1.md`) — colors paint Fixes #22

benstull added 10 commits 2026-06-11 23:36:39 +00:00

plan(f7.1): intra-diagram mermaid diffing implementation plan (#22 ) c117552ea1

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

feat(f7.1): mermaid diff dispatcher + type detection (#22 ) d8ce4f12e1

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

feat(f7.1): flowchart parser (#22 ) 78352c2922

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

feat(f7.1): flowchart node/edge diff + styling emission (#22 ) 1454c7792a

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

feat(f7.1): sequence-diagram parser (#22 ) 96931f245d

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

feat(f7.1): sequence participant/message diff + rect emission (#22 ) e938572ebc

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

feat(f7.1): render augmented mermaid diff in changed-block branch (#22 ) 6702490497

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

feat(f7.1): mermaid diff legend styles (#22 ) ea96b4861c

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

test(f7.1): host E2E — changed flowchart augments emitted source (#22 ) 3b45e784c1

Adds a renderHtmlFor test seam to TrackChangesPreviewController.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

docs(f7.1): manual webview-render smoke (#22 ) 4c4be84f37

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

benstull merged commit 06d4f87bf7 into main

2026-06-11 23:36:58 +00:00

benstull deleted branch f7.1-intra-diagram-mermaid-diff

2026-06-11 23:36:58 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: benstull/vscode-cowriting-plugin#28