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
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Jan 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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