Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Dec 23, 2024

This is an attempt at fixing the bug where newly created files aren’t available to @-mention. Would love to better understand the root cause though, and I’m not sure how resource-intensive the periodic refresh is if we want to go down that path. Might be better to do it less frequently.


Important

Adds periodic refresh in WorkspaceTracker.ts to detect new files for @-mention, with tests and npm registry configuration.

  • Behavior:
    • Adds periodic refresh in WorkspaceTracker.ts to detect new files every second using setInterval.
    • Updates workspace if new files are detected, making them available for @-mention.
  • Testing:
    • Adds tests in WorkspaceTracker.test.ts to verify periodic file detection and workspace update.
    • Ensures refresh interval is cleaned up on dispose.
  • Misc:
    • Adds .npmrc file to specify npm registry.

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

@changeset-bot
Copy link

changeset-bot bot commented Dec 23, 2024

⚠️ No Changeset found

Latest commit: ac872d6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@mrubens mrubens marked this pull request as draft December 23, 2024 15:56
@mrubens
Copy link
Collaborator Author

mrubens commented Dec 24, 2024

Closing in favor of #203

@mrubens mrubens closed this Dec 24, 2024
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