Skip to content

Show owner/repo:branch in PR header instead of owner:branch#8276

Merged
alexr00 merged 7 commits intomainfrom
copilot/update-pr-header-repo-info
Dec 16, 2025
Merged

Show owner/repo:branch in PR header instead of owner:branch#8276
alexr00 merged 7 commits intomainfrom
copilot/update-pr-header-repo-info

Conversation

Copy link
Contributor

Copilot AI commented Dec 10, 2025

In multi-repo workspaces, the PR header displays theking2:main from theking2:reduce-logging, making it unclear which repository the PR targets.

Changes

  • Display layer only: Modified pullRequestOverview.ts and activityBarViewProvider.ts to construct the owner/repo:branch format from existing GitHubRef properties (owner, name, ref)
  • No data layer changes: Does not modify GraphQL queries, interfaces, or utils functions

Impact

Before:

theking2 wants to merge changes into theking2:main from theking2:reduce-logging

After:

theking2 wants to merge changes into theking2/repo-name:main from theking2/repo-name:reduce-logging

This is a display-only change that makes it immediately clear which repository a PR belongs to, without affecting the underlying data structures.

Original prompt

This section details on the original issue you should resolve

<issue_title>make clear in which repo a pull request is stored</issue_title>
<issue_description>Image

This pull request was made in workspace with mutliple projects all being repos. The screen is not clear where the pull request was created or which changes were made.

Solution: more verbosety of the repo that is actually affected by the PR. </issue_description>

<agent_instructions>In the PR header, we show the remote name before the branch name.. Instead we should show the fulll owner/repoName before the branch name. </agent_instructions>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Make clear in which repo a pull request is stored Show owner/repo:branch in PR header instead of owner:branch Dec 10, 2025
Copilot AI requested a review from alexr00 December 10, 2025 14:50
Copy link
Member

@alexr00 alexr00 left a comment

Choose a reason for hiding this comment

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

@copilot your change is too deep. Just make a change in the PR overview to show the owener and repo, not in the queries.

Copilot AI and others added 2 commits December 15, 2025 17:06
Instead of modifying GraphQL queries and utils, construct the owner/repo:branch format directly in the PR overview and activity bar view provider using existing owner, name, and ref properties from GitHubRef.

Co-authored-by: alexr00 <[email protected]>
Copy link
Contributor Author

Copilot AI commented Dec 15, 2025

@copilot your change is too deep. Just make a change in the PR overview to show the owener and repo, not in the queries.

Changed to a simpler approach - now only modifying the display layer in pullRequestOverview.ts and activityBarViewProvider.ts by constructing the owner/repo:branch format from existing GitHubRef properties (owner, name, ref). Reverted all GraphQL query and utils changes. See commit 45d68c1.

Copilot AI requested a review from alexr00 December 15, 2025 17:11
@alexr00 alexr00 marked this pull request as ready for review December 16, 2025 10:44
@alexr00 alexr00 enabled auto-merge (squash) December 16, 2025 10:45
@vs-code-engineering vs-code-engineering bot added this to the December / January 2026 milestone Dec 16, 2025
@alexr00 alexr00 merged commit bc7d7ad into main Dec 16, 2025
6 checks passed
@alexr00 alexr00 deleted the copilot/update-pr-header-repo-info branch December 16, 2025 10:53
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.

make clear in which repo a pull request is stored

3 participants