Skip to content

feat(app): copy full file contents from review - #59785

Draft
lancy69 wants to merge 1 commit into
anomalyco:devfrom
lancy69:review-copy-content
Draft

lancy69 wants to merge 1 commit into
anomalyco:devfrom
lancy69:review-copy-content

Conversation

@lancy69

@lancy69 lancy69 commented Oct 28, 2026

Copy link
Copy Markdown

Issue for this PR

Closes #58878

Draft: design approval requested on the issue and still pending. Local implementation prepared with OpenAI Codex assistance.

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Copy complete file text from the review header rather than diff hunks. Provide success/error feedback and leave the clipboard unchanged for unavailable or binary content.

How did you verify your code works?

Three review E2E regressions passed. Chromium clipboard matched all 654 characters, including unchanged lines outside the hunk; RTL, empty content, and clipboard denial exercised.

bun run check passed with pinned Bun 1.4.4, including the pre-push check; existing lint warnings remain. TUI runtime captures use simulation-control rendering/input, not a host PTY.

Visual

<svg xmlns="http://www.w3.org/2000/svg" width="240" height="72" viewBox="0 0 240 72"><style>@keyframes pulse{0%,100%{opacity:1}50%{opacity:.45}}.pulse{animation:pulse 2s ease-in-out infinite}@media(prefers-reduced-motion:reduce){.pulse{animation:none}}</style><g fill="currentColor" font-family="monospace" font-size="12"><text x="8" y="22" opacity=".55">Review file</text><text x="8" y="51" class="pulse">→ Copy full text</text></g></svg>

Screenshots / recordings

Review copy action
RTL review

After-change captures from isolated fixtures; no before-change capture retained.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

This branch has not been deployed

No deployments
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.

[FEATURE]: web review-panel copy content button

1 participant