Skip to content

Improve discoverability of browser open tool#297724

Merged
kycutler merged 1 commit intomainfrom
kycutler/opensearch
Feb 25, 2026
Merged

Improve discoverability of browser open tool#297724
kycutler merged 1 commit intomainfrom
kycutler/opensearch

Conversation

@kycutler
Copy link
Contributor

Fixes #297723.

The issue is that the tool search tool only returns a limited number of results, and they may not be prioritized as expected. So we shouldn't reference the open tool name in other tools to keep the results clean.

Copilot AI review requested due to automatic review settings February 25, 2026 17:04
@kycutler kycutler self-assigned this Feb 25, 2026
@kycutler kycutler enabled auto-merge (squash) February 25, 2026 17:05
@vs-code-engineering vs-code-engineering bot added this to the February 2026 milestone Feb 25, 2026
Copy link
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

This PR improves the discoverability of the browser “open page” tool by removing explicit references to its tool id from other browser tool metadata, reducing noise in tool-search ranking.

Changes:

  • Updated pageId input schema descriptions across browser tools to avoid mentioning OpenPageToolId directly.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
src/vs/workbench/contrib/browserView/electron-browser/tools/typeBrowserTool.ts Rewords pageId schema description to avoid referencing the open tool id.
src/vs/workbench/contrib/browserView/electron-browser/tools/screenshotBrowserTool.ts Rewords pageId schema description to avoid referencing the open tool id.
src/vs/workbench/contrib/browserView/electron-browser/tools/runPlaywrightCodeTool.ts Rewords pageId schema description to avoid referencing the open tool id.
src/vs/workbench/contrib/browserView/electron-browser/tools/readBrowserTool.ts Rewords pageId schema description to avoid referencing the open tool id.
src/vs/workbench/contrib/browserView/electron-browser/tools/navigateBrowserTool.ts Rewords pageId schema description to avoid referencing the open tool id.
src/vs/workbench/contrib/browserView/electron-browser/tools/hoverElementTool.ts Rewords pageId schema description to avoid referencing the open tool id.
src/vs/workbench/contrib/browserView/electron-browser/tools/handleDialogBrowserTool.ts Rewords pageId schema description to avoid referencing the open tool id.
src/vs/workbench/contrib/browserView/electron-browser/tools/dragElementTool.ts Rewords pageId schema description to avoid referencing the open tool id.
src/vs/workbench/contrib/browserView/electron-browser/tools/clickBrowserTool.ts Rewords pageId schema description to avoid referencing the open tool id.

@kycutler kycutler merged commit 8be4c75 into main Feb 25, 2026
24 checks passed
@kycutler kycutler deleted the kycutler/opensearch branch February 25, 2026 17: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.

Agent has trouble discovering open_browser_page tool when deferred

3 participants