refactor(@angular/cli): migrate MCP zoneless tool to host abstraction - #33133
Conversation
45dc998 to
225778b
Compare
There was a problem hiding this comment.
Code Review
This pull request refactors the zoneless migration tool to use a Host abstraction for file system operations instead of direct node:fs calls. This involves updating several functions and tests to accept and utilize the Host interface. A review comment identifies a potential type mismatch in the Host implementation following the update of the readFile signature to use BufferEncoding.
Migrate the zoneless migration tool to use the Host abstraction for file system operations including reading files, checking stats, and globbing. This ensures that the tool adheres to the allowed roots configured in the MCP server. By removing direct dependencies on native Node.js fs modules, the tool now benefits from the centralized injection and restriction model provided by the Host implementation.
225778b to
1671504
Compare
|
This PR was merged into the repository. The changes were merged into the following branches:
|
|
This pull request has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Migrate the zoneless migration tool to use the Host abstraction for file system operations including reading files, checking stats, and globbing. This ensures that the tool adheres to the allowed roots configured in the MCP server. By removing direct dependencies on native Node.js fs modules, the tool now benefits from the centralized injection and restriction model provided by the Host implementation.