Skip to content

fix(search): include restored files in global search - #2830

Merged
bajrangCoder merged 2 commits into
mainfrom
fix/2825-restored-files-search
Aug 29, 2026
Merged

fix(search): include restored files in global search#2830
bajrangCoder merged 2 commits into
mainfrom
fix/2825-restored-files-search

Conversation

@bajrangCoder

Copy link
Copy Markdown
Member

Fixes: #2825

@greptile-apps

greptile-apps Bot commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR prevents unloaded restored-file placeholders from overriding filesystem content during global search.

  • Requires an editor file to be fully loaded before worker-based search reads its document.
  • Applies the same loaded-state filtering to native-search overlays.

Confidence Score: 4/5

The remote-file loading race should be fixed before merging because global search can temporarily ignore recovered unsaved content.

The new worker-path guard falls back to remote source content during an observable interval when the editor session already contains authoritative recovery text but has not yet been marked loaded.

Files Needing Attention: src/sidebarApps/searchInFiles/index.js

Important Files Changed

Filename Overview
src/sidebarApps/searchInFiles/index.js Adds loaded-state guards to both search-content paths, but the worker guard can temporarily discard authoritative recovery text for asynchronously loading remote files.

Reviews (1): Last reviewed commit: "fix(search): include restored files in g..." | Re-trigger Greptile

Comment thread src/sidebarApps/searchInFiles/index.js Outdated
@bajrangCoder

This comment was marked as outdated.

@bajrangCoder
bajrangCoder added this pull request to the merge queue Aug 29, 2026
Merged via the queue into main with commit c7a9e34 Aug 29, 2026
10 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in The Code Board - Acode Aug 29, 2026
@bajrangCoder
bajrangCoder deleted the fix/2825-restored-files-search branch August 29, 2026 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[Bug] Already opened files excluded from mass-file text search results

1 participant