Skip to content

fix(@angular/cli): ignore EBADF file system errors during MCP project scan#33002

Merged
clydin merged 1 commit intoangular:mainfrom
clydin:mcp/project-ignore-fs-errors
Apr 15, 2026
Merged

fix(@angular/cli): ignore EBADF file system errors during MCP project scan#33002
clydin merged 1 commit intoangular:mainfrom
clydin:mcp/project-ignore-fs-errors

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented Apr 14, 2026

File crawling concurrency occasionally surfaces temporary EBADF descriptors on specialized filesystems. This catches and ignores them, guaranteeing completion of background discovery loops.

… scan

File crawling concurrency occasionally surfaces temporary EBADF descriptors on specialized filesystems. This catches and ignores them, guaranteeing completion of background discovery loops.
@clydin clydin added the target: patch This PR is targeted for the next patch release label Apr 14, 2026
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Apr 14, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds 'EBADF' to the IGNORED_FILE_SYSTEM_ERRORS set in the Angular CLI's MCP tools, ensuring that bad file descriptor errors are treated as ignorable during file system operations. I have no feedback to provide as there are no review comments to evaluate.

@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Apr 15, 2026
@clydin clydin merged commit ead60d3 into angular:main Apr 15, 2026
39 checks passed
@clydin
Copy link
Copy Markdown
Member Author

clydin commented Apr 15, 2026

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

@clydin clydin deleted the mcp/project-ignore-fs-errors branch April 15, 2026 13:48
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: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

list_projects MCP tool fails with EBADF: bad file descriptor, lstat '/dev/fd/12' when MCP roots include /dev/fd/*

2 participants