Skip to content

CmdPal: Improve indexer plain query search#46907

Open
jiripolasek wants to merge 3 commits intomainfrom
dev/jpolasek/f/42153-improve-file-search-query
Open

CmdPal: Improve indexer plain query search#46907
jiripolasek wants to merge 3 commits intomainfrom
dev/jpolasek/f/42153-improve-file-search-query

Conversation

@jiripolasek
Copy link
Copy Markdown
Collaborator

@jiripolasek jiripolasek commented Apr 11, 2026

Summary of the Pull Request

This PR improves File Search:

  • Improves simple free-text Windows Search queries with implicit filename broadening while preserving structured AQS input.
  • Adds resilient fallback behavior for noisy or punctuation-heavy searches by retrying with literal filename matching (fixes failed searches with & or other symbols).
  • Surfaces Windows Search availability and indexing-status notices in the indexer page and fallback item -- if the Windows Search service is down or unreachable, we show this to the user.
  • Extends production time logging.
  • Adds documentation of query transformation for maintainers.
  • Adds some unit tests to pretend that we care.

Pictures? Pictures!

Error notices:

image image

PR Checklist

  • Communication: I've discussed this with core contributors already. If the work hasn't been agreed, this work might be rejected
  • Tests: Added/updated and all pass
  • Localization: All end-user-facing strings can be localized
  • Dev docs: Added/updated
  • New binaries: Added on the required places
  • Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

- Improves simple free-text Windows Search queries with implicit filename broadening while preserving structured AQS input.
- Adds resilient fallback behavior for noisy or punctuation-heavy searches by retrying with literal filename matching.
- Surfaces Windows Search availability and indexing-status notices in the indexer page and fallback item.
- Tightens production logging for datasource, query-execution, and catalog-status failures.
- Documents the query-handling heuristics and the paging/notice invariants for future maintainers.
@jiripolasek jiripolasek added the Product-Command Palette Refers to the Command Palette utility label Apr 11, 2026
Examples:

- `foo-bar`
- `20220409-tontrager.xlsx`
- `foo-bar`
- `20220409-tontrager.xlsx`

Users usually expect broadening here, because `tontrager` should still find `20220409-tontrager.xlsx`.
- `foo-bar`
- `20220409-tontrager.xlsx`

Users usually expect broadening here, because `tontrager` should still find `20220409-tontrager.xlsx`.
@jiripolasek
Copy link
Copy Markdown
Collaborator Author

@niels9001 I'm adding a new item with the message instead of replacing EmptyContent with the notice -- I feel that if we put it into EmptyContent, users would dismiss it as actual empty content. Thoughts?

image

@niels9001
Copy link
Copy Markdown
Collaborator

@niels9001 I'm adding a new item with the message instead of replacing EmptyContent with the notice -- I feel that if we put it into EmptyContent, users would dismiss it as actual empty content. Thoughts?

image

@jiripolasek Yeah this works.. but can't we show empty content with an error icon instead of the File Explorer icon (that would show up in the actual empty state)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Product-Command Palette Refers to the Command Palette utility

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Command Palette file search not working Ampersands (&) in Command Palette File Search

3 participants