Skip to content

Conversation

@cpsievert
Copy link
Collaborator

No description provided.

@cpsievert cpsievert requested a review from Copilot August 26, 2025 16:39

This comment was marked as resolved.

@cpsievert cpsievert force-pushed the feat/tool-annotations branch from 6e32690 to c1b5451 Compare August 26, 2025 21:00
@cpsievert cpsievert requested a review from Copilot August 26, 2025 21:30
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for tool annotations in the chatlas library, allowing users to provide additional metadata about tools including titles and hints about their behavior. This feature enhances tool usability by providing more context about tool purpose and safety considerations.

Key changes:

  • Added annotations parameter to tool registration methods
  • Extended ContentToolRequest to include tool reference for accessing annotations
  • Automatic annotation propagation from MCP tools

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
chatlas/_tools.py Added annotations parameter to Tool class and related methods
chatlas/_content.py Added tool field to ContentToolRequest to enable annotation access
chatlas/_chat.py Modified tool invocation to populate tool reference in requests
chatlas/init.py Added model rebuild calls to resolve forward references
tests/test_tool_from_mcp.py Added test coverage for MCP tool annotation handling
tests/test_content_tools.py Updated tests to include tool references in requests
tests/conftest.py Enhanced test assertions to verify annotation functionality
CHANGELOG.md Documented the new feature

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@cpsievert cpsievert merged commit b898534 into main Aug 26, 2025
6 of 7 checks passed
@cpsievert cpsievert deleted the feat/tool-annotations branch August 26, 2025 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants