Fix zed irresponsive on symlinked directory events outside the editor#50746
Fix zed irresponsive on symlinked directory events outside the editor#50746probably-neb merged 18 commits intozed-industries:mainfrom
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1370f3ca27
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@prayanshchh I think the bigger issue is #45954, which would also handle #48729. Have you taken a look at #45954? Wdyt? |
|
I agree, I didn't see this issue, will change this PR to resolve that one |
|
Let us know when this is ready @prayanshchh! |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 790324d847
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
this is ready @probably-neb |
|
just checking, I am seeing more users facing the symlink issue can we get this reviewed? |
|
This PR may address the same issue as my PR here? No opinion about which one is better, just thought I'd mention it |
|
This PR may also relate to issue #27263 ? |
no worries lemme just push tests for this and lets review each other's PR just for fun..if u are down |
yes |
|
sorry for the delay on the follow up review! I got this one and #46338 mixed up |
|
@prayanshchh FYI, probably-ned mentioned he'll be merging this PR rather than mine, as the solution here is better, which I agree with. You don't have to review the implementation in my PR, but you can check if there are tests you want to bring over. |
|
will do! |
np! |
|
lmk if something more is needed from this. |
|
Great work. Thanks! |
…zed-industries#50746) Closes zed-industries#48729, closes zed-industries#27263, closes zed-industries#45954 This PR aims to make zed responsive on symlinked directory events outside the editor. Before you mark this PR as ready for review, make sure that you have: - [ ] Added a solid test coverage and/or screenshots from doing manual testing - [x] Done a self-review taking into account security and performance aspects - [ ] Aligned any UI changes with the [UI checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist) new-linked-folder is inside /zed-test/zed-project output of ls -ld new-linked-folder `lrwxr-xr-x 1 prayanshchhablani staff 42 28 Mar 23:20 new-linked-folder -> /Users/prayanshchhablani/new-target-folder` this shows new-linked-folder is a symlink folder whose target is new-target-folder which is outside the root dir of the project opened in zed. https://github.com/user-attachments/assets/ffebafc3-2fc4-4293-bdbf-3a894a45e276 Release Notes: - Fixed file watching of symlinks that point outside of the project/watched directory. Zed should now properly respond to changes in files in symlinked directories
Closes #48729, closes #27263, closes #45954
This PR aims to make zed responsive on symlinked directory events outside the editor.
Before you mark this PR as ready for review, make sure that you have:
new-linked-folder is inside /zed-test/zed-project
output of ls -ld new-linked-folder
lrwxr-xr-x 1 prayanshchhablani staff 42 28 Mar 23:20 new-linked-folder -> /Users/prayanshchhablani/new-target-folderthis shows new-linked-folder is a symlink folder whose target is new-target-folder which is outside the root dir of the project opened in zed.
Screen.Recording.2026-04-02.at.3.54.02.PM.mov
Release Notes: