Skip to main content
Glama

AI 代码审查(按次计费)

review

审查代码并给出分级的修改建议。按次计费:业务成功才确认扣款,内容违规、模型超时或生成失败会自动全额退款。

Input Schema

NameRequiredDescriptionDefault
codeYes待审查的代码原文,最多 20000 字符。
focusNo审查重点,默认全部。all
languageNo代码语言,不填则自动判断。
requestIdNo幂等键。同一次业务重试时复用同一个值,可避免重复扣费。

Schema Changelog

Changes observed during successful MCP inspections.

  1. Added

TDQS

B3.4/5.0
Behavior4/5

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

With no annotations, the description carries the full behavioral burden and does disclose the commercially important trait: pay-per-call with charge confirmed only on business success, and automatic full refund on content violation, model timeout, or generation failure. That is real, non-obvious context. It stops short of covering authorization/prerequisites or what happens to submitted code, which is why it is not a 5.

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

Conciseness4/5

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

Two sentences, purpose front-loaded, zero filler. The billing sentence is dense but earns its place because it is the tool's only non-schema disclosure.

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

Completeness3/5

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

Billing risk is well covered, but for a tool with no output schema the description never characterizes the response — what 'graded' means (levels? counts? line references?) is undefined. It also omits practical limits and the idempotency expectation, leaving meaningful gaps for an agent planning a call.

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?

Schema description coverage is 100%, so the baseline is 3. The description adds nothing about code, focus, language, or the requestId idempotency key, all of which are already fully documented in the schema.

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

Purpose4/5

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

States a specific verb+resource+output shape: review code and return graded fix suggestions. That clearly separates it from codegen/testgen/docgen siblings, though it never names an alternative explicitly. It falls short of 5 only because the sibling differentiation is implicit rather than stated.

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

Usage Guidelines2/5

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

The second sentence is about billing mechanics, not about when to reach for this tool versus codegen or testgen. There is no 'use this when…' or 'do not use for…' guidance at all; an agent must infer usage purely from the one-line purpose.

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

Try in Browser

Glama MCP Gateway

Add one secure layer between your agents and this server.