Skip to content

fix(@angular/cli): cache MCP best practices content and add tool annotations #30840

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 6, 2025

Conversation

clydin
Copy link
Member

@clydin clydin commented Aug 6, 2025

Caches the content of the best practices instructions to avoid redundant file reads on subsequent uses of the tool. This can provide a minor performance improvement in cases where the tool is used multiple times during a single CLI process.

Also, annotations have been added to the text to provide additional context for the assistant and allow for more tailored display of the information.

…tations

Caches the content of the best practices instructions to avoid redundant file reads on subsequent uses of the tool. This can provide a minor performance improvement in cases where the tool is used multiple times during a single CLI process.

Also, annotations have been added to the text to provide additional context for the assistant and allow for more tailored display of the information.
@clydin clydin added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Aug 6, 2025
@clydin clydin requested a review from devversion August 6, 2025 14:21
@clydin clydin 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 Aug 6, 2025
@clydin clydin merged commit 51d56f7 into angular:main Aug 6, 2025
33 checks passed
@clydin
Copy link
Member Author

clydin commented Aug 6, 2025

The changes were merged into the following branches: main, 20.1.x

@clydin clydin deleted the mcp/best-prac-annotations branch August 6, 2025 15:10
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.

2 participants