get_pull_review

Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?

Annotations already declare readOnlyHint=true, and the 'Get' wording is consistent with that, so there is no contradiction. The description adds little behavioral context beyond the read-only nature, but the output schema and annotation reduce the need for extensive behavior disclosure.

Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.

Is the description appropriately sized, front-loaded, and free of redundancy?

The description is a single, direct sentence with no filler. The core action and resource are front-loaded, making it easy to scan.

Shorter descriptions cost fewer tokens and are easier for agents to parse. Every sentence should earn its place.

Given the tool's complexity, does the description cover enough for an agent to succeed on first attempt?

For a simple read-only getter with an output schema and readOnlyHint annotation, the description is minimally adequate. However, it does not clarify how the parameters map to the pull request and review, nor does it reference list_pull_reviews as a prerequisite source of review IDs.

Complex tools with many parameters or behaviors need more documentation. Simple tools need less. This dimension scales expectations accordingly.

Does the description clarify parameter syntax, constraints, interactions, or defaults beyond what the schema provides?

Schema description coverage is 0%, and the description does not explain any of the four parameters. In particular, 'index' is ambiguous (likely the pull request number) and 'review_id' is not clarified, leaving the agent to rely on parameter names and inference.

Input schemas describe structure but not intent. Descriptions should explain non-obvious parameter relationships and valid value ranges.

Does the description clearly state what the tool does and how it differs from similar tools?

The description uses the specific verb 'Get' and clearly identifies the resource: 'a specific review on a pull request.' This distinguishes it from sibling tools like list_pull_reviews, which returns all reviews, and get_pull_request, which returns the pull request itself.

Agents choose between tools based on descriptions. A clear purpose with a specific verb and resource helps agents select the right tool.

Does the description explain when to use this tool, when not to, or what alternatives exist?

The phrase 'specific review' implies this tool is for retrieving a single review by ID, but there is no explicit guidance about when to use it versus list_pull_reviews or create_pull_review. No alternative tools or exclusion conditions are mentioned.

Agents often have multiple tools that could apply. Explicit usage guidance like "use X instead of Y when Z" prevents misuse.