Skip to content

Conversation

clydin
Copy link
Member

@clydin clydin commented Sep 19, 2025

The previous description for the ai_tutor MCP tool was written from a user-centric perspective. This commit refactors the description to be LLM-centric, providing more explicit instructions for the assistant.

The new description clarifies that:

  • The tool's output is a system prompt that reprograms the assistant to adopt the "tutor" persona.
  • The tool acts as a RAG source, providing a new knowledge base.
  • The tutor persona supports special commands (e.g., "skip this section").

These changes align with the best practices for MCP tool descriptions and ensure the LLM has the proper context to use the tool effectively.

The previous description for the `ai_tutor` MCP tool was written from a user-centric perspective. This commit refactors the description to be LLM-centric, providing more explicit instructions for the assistant.

The new description clarifies that:
- The tool's output is a system prompt that reprograms the assistant to adopt the "tutor" persona.
- The tool acts as a RAG source, providing a new knowledge base.
- The tutor persona supports special commands (e.g., "skip this section").

These changes align with the best practices for MCP tool descriptions and ensure the LLM has the proper context to use the tool effectively.
@clydin clydin added target: major This PR is targeted for the next major release action: review The PR is still awaiting reviews from at least one requested reviewer labels Sep 19, 2025
@clydin clydin requested a review from devchas September 19, 2025 23:54
@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 Sep 22, 2025
@clydin clydin merged commit cb8980a into angular:main Sep 22, 2025
35 checks passed
@clydin clydin deleted the mcp/ai-tutor-description branch September 22, 2025 16:02
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: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants