Skip to content

Conversation

jruales
Copy link
Contributor

@jruales jruales commented Oct 21, 2025

The link wasn't working before
image

Fixes microsoft/vscode#272449
Partial fix for: microsoft/vscode#252415

@jruales jruales marked this pull request as ready for review October 21, 2025 10:40
@Copilot Copilot AI review requested due to automatic review settings October 21, 2025 10:40
Copy link
Contributor

@Copilot 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

This PR fixes a broken Markdown link in the Run Command tool that displays confirmation messages to users. The previous implementation incorrectly encoded command arguments, preventing the link from working properly when users clicked it.

Key Changes:

  • Updated command URI generation to use the commandUri helper function instead of manual JSON encoding
  • Adjusted the trusted command configuration to match the Quick Open command being linked

@jruales jruales requested a review from bhavyaus October 21, 2025 13:32
Copy link
Contributor

@bhavyaus bhavyaus left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@jruales jruales force-pushed the dev/jruales/fix-run-command-tool-md-link branch from ff109b3 to d07ca7c Compare October 21, 2025 17:46
@jruales jruales added this pull request to the merge queue Oct 21, 2025
github-merge-queue bot pushed a commit that referenced this pull request Oct 21, 2025
* Fix Markdown link in Run Command tool

* Import commandUri in vscodeCmdTool.tsx

Add import for commandUri from linkify commands

* Use commandId instead of name

* Small comment

Add comments to clarify command ID handling in Quick Open.
@jruales jruales removed this pull request from the merge queue due to a manual request Oct 21, 2025
Add import for commandUri from linkify commands
Add comments to clarify command ID handling in Quick Open.
@jruales jruales force-pushed the dev/jruales/fix-run-command-tool-md-link branch from d07ca7c to 1d85808 Compare October 21, 2025 23:18
@jruales jruales enabled auto-merge October 21, 2025 23:19
@jruales jruales added this pull request to the merge queue Oct 21, 2025
Merged via the queue into main with commit 82a95eb Oct 21, 2025
16 checks passed
@jruales jruales deleted the dev/jruales/fix-run-command-tool-md-link branch October 21, 2025 23:36
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.

Link broken in Run Command tool confirmation text

3 participants