Server has 10 tools with basic schemas and descriptions, but significant quality gaps. Most tools lack parameter descriptions (e.g., get_itinerary_details has no description for 'itinerary_id'). Descriptions are present but generic (avg ~80 chars, below 194-char baseline). No output schemas documented. Error handling is minimal, tools return bare error dicts without recovery guidance. Three tools (get_recommended_itinerary_resource, find_itineraries, get_itinerary_details) appear in multiple files with inconsistent definitions, suggesting incomplete refactoring. create_itinerary has complex nested array params (daily_plans) with no schema validation or field documentation.

• Add explicit output schema documentation to every tool. Example: 'Returns: {"id": int, "name": str, "nights": int, "daily_plans": [{"day": int, "hotel": {...}, "activities": [...]}]}'
• Add parameter descriptions to all tools. For get_itinerary_details, document: 'itinerary_id: The unique identifier of the itinerary to retrieve (integer, required)'
• Expand create_itinerary's daily_plans parameter schema. Specify: 'daily_plans: Array of objects, each with {day_number: int (1-8), hotel_id: int, transfer_id: int (optional), activity_ids: [int], notes: str (optional)}'
• Replace bare error dicts with actionable recovery guidance. E.g., 'Nights must be between 2 and 8, got {nights}. Call list_available_durations() to see valid options.'
• Consolidate duplicate tool definitions. Keep one canonical version in app/mcp/server.py; remove conflicting definitions from claude_mcp_server.py.