The server provides 4 tools for South African pharmaceutical intelligence with reasonable structure but noticeable gaps. All tools have descriptions and input schemas present in the code. Naming is clear and verb-based (get_, search_, analyse_). Descriptions are adequate but generic, they lack actionable context for when to use each tool vs others. Schema completeness varies: some parameters have good constraints (enums for category, filter_type, aggregate_by, sort_by) but descriptions for parameters are inconsistent in quality. Error handling is present but basic, returns string errors without structured guidance. Output is markdown tables, which is inefficient for LLM parsing compared to structured JSON. No tool annotations (readOnlyHint/destructiveHint) despite all being read-only operations. Tool composition is weak: analyse_ndoh_market and analyse_private_market are near-duplicates with identical parameter structures but different domains, this creates cognitive load for the LLM.

• Consolidate analyse_ndoh_market and analyse_private_market into a single 'analyse_market' tool with a 'market_sector' enum parameter (public|private). This eliminates duplication and reduces LLM confusion.
• Return structured JSON objects instead of markdown tables. Example: {"records": [{"company_name": "Cipla", "product_name": "...", ...}], "total": 42, "limit": 15, "offset": 0}. Document the output schema in each tool's return type.
• Rewrite tool descriptions to include WHEN-to-use context. Example: 'Search for products registered to a specific company. Use this after calling get_licensed_companies to find a company's portfolio. Returns product names, APIs, license numbers, and registration dates.'
• Implement structured error responses. Example: {"error": "not_found", "message": "Company 'XYZ' not found. Try a partial name or call get_licensed_companies to browse.", "recoverable": true}.
• Add readOnlyHint: true to all tool definitions in fastmcp to declare these as safe for retry.
• Clarify 'query' parameter: change default from null to empty string (""), and document: 'Leave empty to retrieve all records. Partial matches supported.'
• Add response field chaining: if search_sahpra_products returns company_name and product_id, ensure analyse_private_market can accept product_id as a search parameter for drill-down analysis.
• Add pagination guidance: document that limit should be kept <=100 to prevent context window exhaustion. Provide a note in each tool: 'Large result sets are capped at 100 records per page. Use offset for pagination.'