Critical gaps across all dimensions. Tool definitions in src/mcp/tools.ts lack input schemas entirely, only names and descriptions are visible. No parameter types, constraints, or output schemas documented. Descriptions are present but generic (10-80 chars, below the 194-char baseline). No error handling guidance, no security annotations, no composition hints. Tools operate on sensitive data (memory, facts) with destructive operations (forget) but lack permission gates or audit trail documentation. The server appears to be a stub or incomplete implementation.

• Add full JSON Schema inputSchema to each tool definition with type, description, and required fields for all parameters. Example: {"type": "object", "properties": {"accountId": {"type": "string", "description": "Account identifier (UUID or email domain)"}, "customerId": {"type": "string", "description": "Customer identifier (UUID or email)"}, "query": {"type": "string", "description": "Semantic search query (1-500 chars)"}}, "required": ["accountId", "customerId", "query"]}
• Expand tool descriptions to 150 - 250 chars explaining WHAT, WHEN, and WHAT-RETURNS. Example for recall_memory: 'Retrieve customer memory facts and recent events matching a semantic query. Use this to ground responses in prior context before composing a reply. Returns a bounded bundle of facts, events, and metadata to avoid context overflow.'
• Document output schemas for each tool. Example: recall_memory returns {"facts": [{"id": string, "predicate": string, "value": string, "confidence": number}], "events": [{"id": string, "timestamp": ISO8601, "summary": string}], "metadata": {"accountId": string, "customerId": string, "queryTokens": number}}
• Add error handling guidance to forget tool: 'If both factId and predicateClass are omitted, returns error "Must specify factId or predicateClass". If no facts match, returns {"deleted": 0, "message": "No facts found for criteria"}. Irreversible, consider adding a dry_run parameter.'
• Add tool annotations: recall_memory gets readOnlyHint=true; write_memory, distill_session, forget get destructiveHint=true. Document required permissions (e.g., 'read:memory', 'write:memory', 'delete:memory') in descriptions.