Skip to content

Fix duplicated Fix entry points on marker hover#308488

Merged
jrieken merged 2 commits intomainfrom
copilot/fix-duplicate-fix-entry-points
Apr 8, 2026
Merged

Fix duplicated Fix entry points on marker hover#308488
jrieken merged 2 commits intomainfrom
copilot/fix-duplicate-fix-entry-points

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 8, 2026

Both the AI code action "Fix" (sparkle icon) and the menu-contributed FixDiagnosticsAction "Fix" (editSparkle icon) render simultaneously in the marker hover status bar, since renderMenuActions() is called unconditionally after the AI code action is added.

image

  • Move renderMenuActions() into an else branch of the aiCodeAction check in _renderMarkerStatusbar, so menu-contributed fix actions only render when no AI code action is present
  • When no AI code action exists (e.g. no Copilot), menu actions still render as before
  • When no quick fixes exist at all, menu actions still render as before (separate code path, unchanged)

Copilot AI requested review from Copilot and removed request for Copilot April 8, 2026 12:13
Copilot AI linked an issue Apr 8, 2026 that may be closed by this pull request
When an AI code action is present, skip rendering menu-contributed
actions (e.g. inline chat FixDiagnosticsAction) to avoid showing
two "Fix" buttons in the hover status bar.

Agent-Logs-Url: https://github.com/microsoft/vscode/sessions/52d215c6-9411-43f1-b6b6-15c542e0ed81

Co-authored-by: jrieken <1794099+jrieken@users.noreply.github.com>
Copilot AI requested review from Copilot and removed request for Copilot April 8, 2026 12:20
Copilot AI changed the title [WIP] Fix duplicated Fix entry points on hover Fix duplicated Fix entry points on marker hover Apr 8, 2026
Copilot AI requested a review from jrieken April 8, 2026 12:20
@jrieken jrieken marked this pull request as ready for review April 8, 2026 12:35
Copilot AI review requested due to automatic review settings April 8, 2026 12:35
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Prevents duplicate “Fix” entry points in the Problems/marker hover status bar by ensuring menu-contributed fix actions only render when no AI quick-fix action is present.

Changes:

  • Render menu-contributed marker-hover status bar actions only when an AI code action is not available.
  • Preserve existing behavior when there are no quick fixes (menu actions still render) and when the editor is read-only (menu actions still render).
Show a summary per file
File Description
src/vs/editor/contrib/hover/browser/markerHoverParticipant.ts Avoids rendering menu-contributed “Fix” actions when an AI quick-fix action is present, preventing duplicated status bar actions.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@jrieken jrieken merged commit d7286a4 into main Apr 8, 2026
27 checks passed
@jrieken jrieken deleted the copilot/fix-duplicate-fix-entry-points branch April 8, 2026 12:59
@vs-code-engineering vs-code-engineering Bot added this to the 1.116.0 milestone Apr 8, 2026
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.

Duplicated Fix entry points on hover

4 participants