Skip to main content
Glama
saidsef

GitHub PR Issue Analyser

by saidsef

Github Add Inline Pr Comment

github_add_inline_pr_comment

Adds an inline review comment to a line or range of lines in a pull request diff, enabling precise feedback on code changes without opening a full review.

Instructions

Adds an inline review comment to a line, or to a range of lines, in a file within a PR. The line must fall inside the PR's diff hunks: GitHub rejects a comment on an unchanged line outside any hunk, and on a path not in the diff. A deleted line exists only on side LEFT.

Input Schema

TableJSON Schema
NameRequiredDescriptionDefault
lineYes
pathYes
sideNoLEFT for a line the PR deletes, RIGHT for one it adds or leaves as contextRIGHT
pr_numberYes
repo_nameYes
repo_ownerYes
start_lineNoFirst line of a range ending at line. Omit to comment on line alone
start_sideNoSide start_line sits on. Omit to match side
comment_bodyYes

Output Schema

TableJSON Schema
NameRequiredDescriptionDefault
idYes
bodyYes
authorYes
html_urlYes
created_atYes

Schema Changelog

Changes observed during successful MCP inspections.

  1. Addedv42.0.0

TDQS

A4.2/5.0
Behavior4/5

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

Annotations already indicate a write operation, but the description adds valuable behavioral context: GitHub will reject comments on unchanged lines outside hunks or on paths not in the diff, and deleted lines exist only on side LEFT. This goes beyond the structured annotation hints and helps the agent anticipate failures.

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

Conciseness5/5

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

Three sentences, with the core purpose first and the critical constraints following. Every sentence contributes meaningful guidance and there is no filler or repetition of the name.

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

Completeness4/5

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

For a 9-parameter tool, the description covers the most error-prone constraints (diff hunks, side usage, range support) and the output schema is present to handle return details. It does not enumerate every parameter, but the self-explanatory names and schema descriptions cover the rest, making the description reasonably complete.

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

Parameters3/5

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

With schema description coverage at only 33%, the description compensates partly by explaining line/range semantics and the LEFT side for deleted lines. However, it does not add semantics for path, comment_body, or the repo identifiers, though those are fairly inferable from names. The low coverage keeps this from scoring higher.

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

Purpose5/5

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

The description states a specific action ('Adds an inline review comment'), a precise resource ('a line, or a range of lines, in a file within a PR'), and adds scope by explaining the diff-hunk requirement. It clearly differentiates this from sibling tools like github_add_pr_comments by emphasizing 'inline' and line/range targeting.

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

Usage Guidelines4/5

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

The description gives actionable preconditions: the target line must be inside a diff hunk, the path must be in the diff, and deleted lines require side LEFT. It does not explicitly name alternative tools, but the 'inline' and 'line/range' framing implies when this tool is appropriate versus general PR comment tools.

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