Skip to content

Show all fetched resources during result review#325702

Open
TylerLeonhardt with Copilot wants to merge 2 commits into
mainfrom
copilot/security-fix-multiple-resources-review
Open

Show all fetched resources during result review#325702
TylerLeonhardt with Copilot wants to merge 2 commits into
mainfrom
copilot/security-fix-multiple-resources-review

Conversation

Copilot AI commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

When reviewing a multi-resource fetch, adjacent results were merged into one block and only the first resource URL was shown.

  • Preserve merging for untitled continuation chunks.
  • Render each titled result as a separate review block, keeping every resource URL associated with its content.
  • Add regression coverage for multiple titled results.

Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com>
Copilot AI requested review from Copilot and removed request for Copilot July 13, 2026 21:57
Copilot AI changed the title [WIP] Fix GitHub Copilot review to show multiple resources Show all fetched resources during result review Jul 13, 2026
Copilot AI requested a review from TylerLeonhardt July 13, 2026 22:05
@justschen

Copy link
Copy Markdown
Collaborator
Screenshot 2026-07-13 at 3 41 53 PM

it's really big, but this works

@justschen justschen marked this pull request as ready for review July 13, 2026 22:42
Copilot AI review requested due to automatic review settings July 13, 2026 22:42

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Separates titled tool-fetch results so every resource URL remains visible while retaining continuation merging.

Changes:

  • Stops merging when the next code result has a title.
  • Adds regression coverage for multiple titled results.
Show a summary per file
File Description
chatToolOutputContentSubPart.ts Splits titled outputs into distinct review blocks.
chatToolInputOutputContentPart.test.ts Tests rendering multiple titled outputs.

Review details

  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Medium

@TylerLeonhardt TylerLeonhardt enabled auto-merge (squash) July 13, 2026 23:01
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.

[security] GitHub Copilot fetch review only presents one of the resources for approval

4 participants