Skip to content

Define shared diffr evidence and explicit Review display targets - #369

Closed
sidkmenon wants to merge 5 commits into
codex/diagram-lensesfrom
codex/diffr-evidence-contract
Closed

sidkmenon wants to merge 5 commits into
codex/diagram-lensesfrom
codex/diffr-evidence-contract

Conversation

@sidkmenon

@sidkmenon sidkmenon commented Sep 19, 2026 •

Copy link
Copy Markdown
Contributor

Accept Source or SearchResultData through existing authoring contracts. Separate display targets from navigation coordinates, preserve complete comparisons with independent display, and retain compatibility for existing snapshots. Pin shared types/schema to the rewritten diffr commit.

Review order: #369 → #381 → #370. Builds on codex/diagram-lenses; consumes diffr #13.

Read commit by commit:

  1. 3e6b6bf81 — Define portable code evidence and explicit display targets.
  2. fb710009c — Accept diffr evidence in existing authoring blocks.

Verification: all application source blobs match the previously tested Review tip. Only dependency manifests/lockfile now pin the identical rewritten diffr tree. Fresh validation after repinning: frozen-lockfile install succeeded and all 107 focused source/API/local-data tests passed. Prior native validation: 20 presentation tests, typechecks and the dev build passed, and saved evidence was inspected in the running app. Review-order documentation was refreshed.

This history is organized by API boundaries. Intermediate contract/usage commits may intentionally precede their implementations and need not compile individually.

AI assistance: authored and reorganized with Codex.

Port PR #245 onto main, retaining diffr v3 folding, alignment, streaming counts, and configuration. Stream exact pinned trees through the JSON review API, support absent sides, and use the structural provider in Monaco while preserving the ordinary unified renderer.

Agent-Session: 01a0abcb-f97f-7431-a8d0-9e7a2bb0d192
Agent-Session: 01a0ac29-e501-7282-941e-9d999e389f97
Agent-Session: 01a0ac09-05a3-70e3-8f9c-5c52947ccb2f
Agent-Session: 01a0ac8d-66c7-77c2-bea1-b1bbba7cd7d8
Agent-Session: 01a0abcb-f97f-7431-a8d0-9e7a2bb0d192
Agent-Session: 01a0ac29-e501-7282-941e-9d999e389f97
Agent-Session: 01a0ac09-05a3-70e3-8f9c-5c52947ccb2f
Agent-Session: 01a0ac8d-66c7-77c2-bea1-b1bbba7cd7d8
Agent-Session: 01a0ac29-e501-7282-941e-9d999e389f97
Agent-Session: 01a0b19a-2fda-72f0-9918-b540e7e4f1f9
@sidkmenon
sidkmenon added this pull request to stack #371 September 19, 2026 02:12
@sidkmenon sidkmenon changed the title codex/diffr evidence contract Define direct diffr evidence in Review authoring contracts Sep 19, 2026
Consume the final shared diffr DTO and schema at its rewritten immutable revision. Keep Source navigation separate from displayed evidence, represent range/result targets explicitly, and accept legacy saved payloads without manufacturing missing source trees. Package contracts precede their consumers.

AI assistance: reorganized with Codex.
Use Source or SearchResultData in code peeks, sequence and stack frames, database operations and file lenses. Test shared sources, independent display, legacy reads and rejected payloads against the public contract.

AI assistance: reorganized with Codex.
@sidkmenon
sidkmenon force-pushed the codex/diffr-evidence-contract branch from bb635df to fb71000 Compare September 19, 2026 21:50
@sidkmenon
sidkmenon removed this pull request from stack #371 September 19, 2026 21:50
@sidkmenon
sidkmenon added this pull request to stack #382 September 19, 2026 21:51
@sidkmenon sidkmenon changed the title Define direct diffr evidence in Review authoring contracts Define shared diffr evidence and explicit Review display targets Sep 19, 2026
An error occurred while trying to automatically change base from codex/diagram-lenses to diffr-wire-v2 September 21, 2026 13:25
@thesiti92 thesiti92 closed this Sep 24, 2026
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.

3 participants