Skip to content

feat: add worktree source control review#17

Merged
jln13x merged 1 commit into
personalfrom
t3code/worktree-diff-viewer
Jul 21, 2026
Merged

feat: add worktree source control review#17
jln13x merged 1 commit into
personalfrom
t3code/worktree-diff-viewer

Conversation

@jln13x

@jln13x jln13x commented Jul 21, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a VS Code-style worktree source control view with staged and unstaged sections
  • support file/all stage and unstage actions, confirmed discard, refresh, and single/all-file diffs
  • reuse the annotatable diff renderer so line comments can continue into chat
  • keep review comments in an isolated temporary draft until the user explicitly continues into chat
  • preserve upstream behavior behind the enableWorktreeSourceControl personal-fork flag

Regression review

  • rebased onto the latest personal branch
  • prevented opening the viewer from silently retargeting an existing chat draft or changing its branch
  • fixed compact nested rename paths so staged renames appear once under their destination path
  • carried the existing oversized-line render guard into the worktree viewer
  • removed unused rename metadata from the public status contract

Verification

  • vp check
  • vp run typecheck
  • vp run lint:mobile
  • vp test apps/server/src/vcs/GitVcsDriverCore.test.ts (41 tests)
  • focused web/contracts/mobile tests (127 tests)
  • vp run --filter @t3tools/web build

vp check reports only the repository's existing lint warnings; it has zero errors.

@github-actions github-actions Bot added size:XXL vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Jul 21, 2026
@jln13x
jln13x merged commit f07b0b7 into personal Jul 21, 2026
6 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant