fix(cli): show multi-file apply_patch diffs#17080
fix(cli): show multi-file apply_patch diffs#17080german-swan wants to merge 3 commits intoanomalyco:devfrom
Conversation
|
Thanks for your contribution! This PR doesn't have a linked issue. All PRs must reference an existing issue. Please:
See CONTRIBUTING.md for details. |
|
The following comment was made by an LLM, it may be inaccurate: Based on my search, I found one potentially related PR: PR #15893: fix: re-read files after formatting in apply_patch This PR is related to All other search results returned PR #17080 itself or unrelated PRs. |
|
Thanks for updating your PR! It now meets our contributing guidelines. 👍 |
|
This is a good catch, ill peep tmr |
|
Is this still being considered for merge? |
Issue for this PR
Closes #17076
Type of change
What does this PR do?
apply_patchalready returns per-file diff metadata, but the CLI permission view still treated patch approvals like a single-file edit and only showed one diff block. This updates the TUI approval renderer to usemetadata.fileswhen present, keep each file path as a header, and place the whole patch set inside one shared scroll area so multi-file reviews are easier to navigate.It also adds explicit
apply_patchrendering inrunmode so non-interactive CLI output prints one diff block per changed file instead of falling back to generic tool output.How did you verify your code works?
bun run typecheckinpackages/opencodeapply_patchlocally in TUI and confirmed all file diffs render in one scrollable view with per-file headersScreenshots / recordings
Checklist