The server defines a single tool 'get-user-review' with basic structure but critical gaps in schema completeness, parameter documentation, and output specification. The tool name follows verb_noun convention (get-user-review) which is acceptable, but the schema shows only two input parameters with minimal descriptions. No output schema is documented. The tool description is terse (102 chars) and lacks context on WHEN to use it or WHAT it returns. Parameter descriptions are generic ('Description of the work performed', 'Optional context about the work') without specifying format, constraints, or examples. The implementation appears to spawn a child process to invoke Claude API, but this integration is not exposed in the tool definition, it's hidden from the agent's perspective, which violates transparency. No error handling guidance is documented. No idempotency or safety markers are present.
Get a detailed review of user work based on description and optional context
No output schema documented. Tool returns the result of spawning a Claude API call, but the agent has no specification of the response structure, field names, or data types.
Tool description is too brief (102 chars) and lacks context. Does not state WHAT is returned, WHEN to use this tool, or any prerequisites. Rubric baseline for good descriptions: 194 chars median, 34-392 char range for p10-p90.
Parameter descriptions lack actionable detail. 'Description of the work performed' and 'Optional context about the work' do not specify format, length limits, or expected structure. Rubric requires descriptions to answer: what type of input? What format? What constraints?
Input schema lacks proper validation constraints. No enum, pattern, minLength, maxLength, or type narrowing beyond 'string'. Schema score capped at 20 because parameters lack type definitions beyond primitive types.
| Scored | Grade | Overall | Spec posture | Rubric |
|---|---|---|---|---|
| 2026-09-21 | F | 33 | 2026-07-28+ | v2 |
| 2026-03-09 | F | 41 | - | v1 |
Tool spawns a subprocess to call Claude API with hardcoded prompts. The tool definition does not disclose this multi-step process, its dependencies, potential failure modes, or retry behavior. This violates transparency and makes error handling unpredictable.
No error handling guidance. Tool definition includes no error classification, recovery paths, or user-fixable remediation suggestions. If the Claude subprocess fails, the agent has no instruction on what to do next.
No idempotency or safety markers. Tool modifies no state but this is not declared. No 'idempotentHint' or 'readOnlyHint' annotation (per current MCP spec 2026-07-28).
Hardcoded review prompts in source code suggest the tool is not configurable. If reviews must be customized, this design does not support it, the agent cannot adjust tone, criteria, or focus without code changes.