Even the 2 tools with parameters (get_characters_in_house, display_text) lack parameter descriptions. Tool descriptions are present but generic (avg ~60 chars), falling short of the 194-char baseline for production tools. No output schemas are documented. Error handling is minimal, the mcpServer.js shows only basic try-catch with a generic 'API error' message, offering no recovery guidance. The tool set mixes disparate domains (Harry Potter API, GitHub Octodex, LED badge) without a coherent purpose or composition strategy. Most critically, 6 tools accept zero parameters and offer no way to customize behavior, making them low-utility for agent composition.

• Add input parameters to all no-parameter tools. At minimum, add limit/offset pagination params to list tools (get_all_characters, fetch_octocats, fetch_spells, get_hogwarts_staff, get_hogwarts_students) so agents can control result size.
• Write complete parameter descriptions for get_characters_in_house and display_text. For house param, document valid values (Gryffindor, Slytherin, Hufflepuff, Ravenclaw) as an enum. For text param, specify character limits (e.g. max 255 chars) and any formatting rules for icons.
• Expand tool descriptions to 100-250 chars. Answer: What does it do? When should the LLM use it? What structure does it return? Example: 'Retrieves all students currently enrolled at Hogwarts. Use this to find students by school when filtering by house is insufficient. Returns a list of student objects with name, house, and patronus fields.'
• Document output schemas for all tools. Add a returns section to each tool definition showing the structure of the response (e.g. {students: [{name: string, house: string, patronus: string}], total: number}).
• Implement result capping and pagination. Add limit (default 20, max 100) and offset (default 0) params to all list tools. Cap results at 50 items and return total count. Document this in tool descriptions.
• Improve error messages. Instead of 'API error: {message}', return structured errors with guidance: '{"error": "Invalid house name", "valid_options": ["Gryffindor", "Slytherin", "Hufflepuff", "Ravenclaw"], "recovery": "Retry with one of the valid house names."}'