Skip to content

fix(watch): byte-hash sync with build extension filter (#345)#377

Open
ASuresh0524 wants to merge 2 commits into
mainfrom
fix/watch-sync-scope-345
Open

fix(watch): byte-hash sync with build extension filter (#345)#377
ASuresh0524 wants to merge 2 commits into
mainfrom
fix/watch-sync-scope-345

Conversation

@ASuresh0524

Copy link
Copy Markdown
Collaborator

Replace SimpleDirectoryReader in change detection with extension-filtered filesystem walks and raw byte hashing. Store explicit files vs directories in sync_roots.json so a loose README.md no longer watches the whole repo.

Fixes #345

Checklist

  • Tests pass (uv run pytest)
  • Code formatted (ruff format and ruff check)
  • Pre-commit hooks pass (pre-commit run --all-files)

Replace SimpleDirectoryReader in change detection with extension-filtered
filesystem walks and raw byte hashing. Store explicit files vs directories
in sync_roots.json so a loose README.md no longer watches the whole repo.

Co-authored-by: Cursor <cursoragent@cursor.com>
@ASuresh0524

Copy link
Copy Markdown
Collaborator Author

@ArtifexSystems got a fix up for this — PR should be open shortly. watch no longer runs LlamaIndex extractors for change detection (byte hash + same ext filter as build), and loose files like README.md don't expand to watching the whole repo anymore. would love if you can sanity-check on your 4GB/WSL setup when you get a chance 🙏

Co-authored-by: Cursor <cursoragent@cursor.com>
@ASuresh0524 ASuresh0524 requested a review from yichuan-w June 18, 2026 02:18
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.

leann watch hard-crashes on media/binary files, and over-scans when a root-level path is indexed

1 participant