Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented Dec 15, 2025

This commit introduces several improvements to the findAngularJsonFiles function within the list_projects tool. An isIgnorableFileError helper was introduced to consistently handle file system errors (EACCES, EPERM, ENOENT, EBUSY) across stat and readdir calls, making the traversal more resilient to transient issues and unavailable directories. A check was implemented to ensure that symbolic links are only traversed if their resolved target paths are valid and remain within the defined search roots.

@clydin clydin marked this pull request as ready for review December 15, 2025 20:02
…al and symlink handling

This commit introduces several improvements to the `findAngularJsonFiles` function within the `list_projects` tool.
An `isIgnorableFileError` helper was introduced to consistently handle file system errors (EACCES, EPERM, ENOENT, EBUSY) across `stat` and `readdir` calls, making the traversal more resilient to transient issues and unavailable directories.
A check was implemented to ensure that symbolic links are only traversed if their resolved target paths are valid and remain within the defined search roots.
@clydin clydin force-pushed the mcp/list-projects-symlinks branch from 061f145 to 19cdc76 Compare December 15, 2025 20:05
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Dec 16, 2025
@clydin clydin added target: minor This PR is targeted for the next minor release and removed target: patch This PR is targeted for the next patch release labels Dec 16, 2025
@clydin clydin merged commit 3480966 into angular:main Dec 16, 2025
37 checks passed
@clydin
Copy link
Member Author

clydin commented Dec 16, 2025

This PR was merged into the repository. The changes were merged into the following branches:

@clydin clydin deleted the mcp/list-projects-symlinks branch December 16, 2025 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: @angular/cli target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants