Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Dec 24, 2024

This is an attempt at fixing the bug where newly created files aren’t available to @-mention.


Important

Replaces file event listeners with a file system watcher in WorkspaceTracker for more reliable file event handling, with updated tests.

  • Behavior:
    • Replaces file event listeners with a file system watcher in WorkspaceTracker.
    • Watches for file creation, deletion, and changes using createFileSystemWatcher('**').
    • Updates workspace on file events by calling addFilePath() and removeFilePath().
  • Tests:
    • Adds WorkspaceTracker.test.ts to test file creation, deletion, and change events.
    • Mocks createFileSystemWatcher in Cline.test.ts to simulate file events.
  • Misc:
    • Removes onDidCreateFiles, onDidDeleteFiles, and onDidRenameFiles listeners.
    • Adds the watcher to disposables for proper cleanup.
    • Updates README.md to mention the use of a file system watcher.

This description was created by Ellipsis for 947adc4. It will automatically update as commits are pushed.

@changeset-bot
Copy link

changeset-bot bot commented Dec 24, 2024

🦋 Changeset detected

Latest commit: 947adc4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
roo-cline Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mrubens mrubens merged commit aaa7d43 into main Dec 24, 2024
4 checks passed
@mrubens mrubens deleted the file_system_watcher branch December 24, 2024 22:17
roomote pushed a commit that referenced this pull request Jul 8, 2025
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.

3 participants