Skip to content

(feature) Enable to filter search within changed files of source tree#314790

Merged
osortega merged 14 commits into
microsoft:mainfrom
dgercho:dgercho/search-include-changed-files
May 20, 2026
Merged

(feature) Enable to filter search within changed files of source tree#314790
osortega merged 14 commits into
microsoft:mainfrom
dgercho:dgercho/search-include-changed-files

Conversation

@dgercho
Copy link
Copy Markdown
Contributor

@dgercho dgercho commented May 6, 2026

Adds a "Search only in changed files (source control)" toggle button to the search panel's files to include row, alongside the existing "Search only in Open Editors" toggle. When enabled, search results are restricted to files that have uncommitted changes tracked. The toggle is disabled if there's no repository presented.

Copilot AI review requested due to automatic review settings May 6, 2026 17:17
@dgercho
Copy link
Copy Markdown
Contributor Author

dgercho commented May 6, 2026

@microsoft-github-policy-service agree

@dgercho dgercho marked this pull request as draft May 6, 2026 17:49
@dgercho dgercho marked this pull request as ready for review May 6, 2026 18:01
@dgercho
Copy link
Copy Markdown
Contributor Author

dgercho commented May 7, 2026

@osortega Would it be possible to merge this feature? I think it could be helpful for many users

@osortega osortega requested review from Copilot and removed request for Copilot May 15, 2026 23:30
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.

Copilot's findings

  • Files reviewed: 4/4 changed files
  • Comments generated: 4

Comment thread src/vs/workbench/services/search/common/queryBuilder.ts Outdated
Comment thread src/vs/workbench/contrib/search/browser/searchView.ts Outdated
Comment thread src/vs/workbench/contrib/search/browser/searchView.ts Outdated
Comment thread src/vs/workbench/contrib/search/browser/patternInputWidget.ts
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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@dgercho
Copy link
Copy Markdown
Contributor Author

dgercho commented May 16, 2026

@osortega Updated branch after copilot review.

Copy link
Copy Markdown
Contributor

@osortega osortega left a comment

Choose a reason for hiding this comment

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

This feature would be really useful for agentic flows, thanks for contributing.
Just a couple comments

Comment thread src/vs/workbench/contrib/search/browser/searchView.ts Outdated
Comment thread src/vs/workbench/contrib/search/browser/patternInputWidget.ts Outdated
@dgercho dgercho requested a review from osortega May 18, 2026 20:58
Comment thread src/vs/base/browser/ui/toggle/toggle.css
dmitrivMS
dmitrivMS previously approved these changes May 18, 2026
@dgercho dgercho requested a review from osortega May 19, 2026 17:26
Copy link
Copy Markdown
Contributor

@osortega osortega left a comment

Choose a reason for hiding this comment

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

@osortega osortega merged commit 5da06da into microsoft:main May 20, 2026
25 checks passed
@vs-code-engineering vs-code-engineering Bot added this to the 1.122.0 milestone May 20, 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.

6 participants