This server demonstrates good foundational quality with three well-named tools, proper schema definitions, and solid descriptions. All tools follow verb-noun naming conventions (get-weather, get-products, get-product-by-id). Input schemas are properly defined using Zod with type information and parameter descriptions. Tool annotations (readOnlyHint, openWorldHint, destructiveHint) are correctly applied. However, output schemas are not formally documented in the tool definitions, responses are returned as untyped structuredContent/content arrays without schema documentation. Error handling is present but minimal, returning only error messages without guidance on recovery steps. Parameter descriptions are adequate but lack constraint details (e.g., perPage defaults, search term format). Composition is clean, each tool has one clear responsibility.

• Add formal output schema documentation to all three tools. For get-weather, document that the response includes {location, temperature, humidity, windSpeed}. For get-products, document the product array structure and total count. This enables LLMs to know exactly what fields to extract.
• Enhance parameter descriptions with constraints and defaults. Change 'Number of products to return (default: 10)' to 'Number of products to return per page (default: 10, range: 1-100).' Add format hints for search and category parameters.
• Improve error messages with recovery guidance. Instead of returning {error: 'Failed to fetch weather'}, return {error: 'Failed to fetch weather for London. Ensure the city name is spelled correctly or try a major city like London, Paris, or Tokyo.'}
• Add parameter interdependencies to descriptions. For get-products, note that search and category are mutually compatible filters, not exclusive options.
• Document result limits and pagination. Specify that get-products returns at most perPage items, and include a next_cursor or page number in responses to guide pagination calls.

Inferred effective spec: 2026-07-28+.

• Relies on Stateful initialize / Mcp-Session-Id (removed; protocol is stateless) - make each request self-contained