Skip to content

feat(cloud-agent): send saved diff reviews to chats - #5817

Draft
eshurakov wants to merge 1 commit into
eshurakov/boundless-sphinxfrom
eshurakov/spry-mangrove
Draft

feat(cloud-agent): send saved diff reviews to chats#5817
eshurakov wants to merge 1 commit into
eshurakov/boundless-sphinxfrom
eshurakov/spry-mangrove

Conversation

@eshurakov

@eshurakov eshurakov commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add saved-diff line/range comments, cross-file drafts, and one batched Send review to agent action.
  • Preserve immutable reviewed context and explicit stale-review handling across file and chat switches.
  • Pin and authorize the destination worktree/chat, retaining the same message identity for uncertain retries.

Verification

No manual browser or live-provider tests were rerun during this restack. Existing pending work was preserved in named stash/backup refs; local handoff and evidence files remain unchanged and excluded.

Visual Changes

Adds direct diff selection, inline feedback, draft review controls, and a review dialog. No screenshots were captured or uploaded during this restack.

Reviewer Notes

Layer 6 of 6 in stack #5814, directly on eshurakov/boundless-sphinx. One feature commit. The merge preserves both commit-card and review wiring, plus upstream autoCommit: true and expected-worktree authorization checks.

Final combined verification passed: web lint, full web typecheck with tRPC declaration build, and eight setup-free Jest suites / 241 tests, including the native review wrappers. All 29 terminal-navigation tests pass after adding the missing review-boundary mocks to their existing fixture; this correction changes no product behavior. The standalone native review suites (77 tests) and review-send/client suites (131 tests) also passed earlier. Formatting, whitespace, and independent merge review passed. No database setup or browser evidence is implied by these isolated tests.

@eshurakov
eshurakov force-pushed the eshurakov/spry-mangrove branch from 987fa7c to 14b8fdb Compare September 1, 2026 19:41
@eshurakov
eshurakov force-pushed the eshurakov/spry-mangrove branch from 14b8fdb to b22f8fa Compare September 2, 2026 07:49
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