Skip to content

Wire review history into local Codiff#2

Merged
xortive merged 0 commit into
codiff-review-historyfrom
codiff-local-review-history
Jul 22, 2026
Merged

Wire review history into local Codiff#2
xortive merged 0 commit into
codiff-review-historyfrom
codiff-local-review-history

Conversation

@xortive

@xortive xortive commented Jul 21, 2026

Copy link
Copy Markdown
Owner

Summary

  • add the shared GitHub review-history adapter
  • adopt shared walkthrough authoring in Electron and mount the shared local review host
  • add separate local GitLab and GitHub history backends
  • connect provider-neutral IPC, walkthrough generation, progress events, and local follow-up documentation

Stacked on #1.

Validation

  • focused local provider and host tests pass
  • vpr build passes at this revision
  • five broad App-render assertions remain stale and also fail against the previously pushed tree

banana banana banana

@xortive

xortive commented Jul 21, 2026

Copy link
Copy Markdown
Owner Author

This stack of pull requests is managed by jj-ryu.

@xortive xortive changed the title Document local review-history parity and follow-ups Wire review history into local Codiff Jul 21, 2026
@xortive
xortive force-pushed the codiff-review-history branch from 72c6b64 to 0bf5597 Compare July 21, 2026 23:05
@xortive
xortive force-pushed the codiff-local-review-history branch from a284052 to 0370ea6 Compare July 21, 2026 23:05
@xortive
xortive force-pushed the codiff-review-history branch 3 times, most recently from 42f52b8 to 0bf5597 Compare July 22, 2026 00:07
@xortive
xortive force-pushed the codiff-local-review-history branch from 0370ea6 to 8c070bc Compare July 22, 2026 03:44
@xortive
xortive merged commit 8c070bc into codiff-review-history Jul 22, 2026
@xortive
xortive force-pushed the codiff-review-history branch from 4f50ea7 to 8c070bc Compare July 22, 2026 03:44
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.

1 participant