FCP-MCP has 37 tools with basic schemas and descriptions, but significant quality gaps limit production readiness. All tools have input schemas with type definitions and descriptions present, meeting minimum requirements. However, descriptions are often generic (10-50 chars) and lack actionable context for LLM selection. Parameter descriptions are minimal. Output schemas are not visible in the provided source. Tool naming follows verb_noun convention (fcpxml_validate, detect_gaps, fcp_undo) but lacks clarity in distinguishing similar operations (e.g., puppet_preset_* tools are repetitive). No error handling guidance, no pagination support for list-like tools, and no documented output structures. Security concerns: tools like fcp_keyboard_shortcut and fcp_menu_command accept free-form strings without validation, inviting injection attacks. The server mixes read-only and write operations without clear permission boundaries.

• Document output schemas for all 37 tools. Specify return type (object, array, string), required fields, and field types. Example: fcpxml_validate should return {valid: boolean, errors: string[], warnings: string[]}.
• Replace free-form string parameters with enums or constrained formats. fcp_keyboard_shortcut should accept enum values (cmd+space, shift+z, etc.) or document valid patterns. fcp_playback should use enum: ['play', 'pause', 'stop']. fcp_navigate should use enum: ['forward', 'backward', 'start', 'end'].
• Expand descriptions to 100-200 characters. Add WHEN to use, prerequisites, and side effects. Example: 'Validate FCPXML structure and content. Call before importing to catch malformed files. Returns list of validation errors and warnings. Idempotent, read-only.'
• Consolidate puppet_preset_* tools into a single puppet_preset tool with a preset_type enum parameter (idle, talk, walk, wave, bounce). Reduces cognitive load and simplifies LLM tool selection.
• Add error handling guidance to all write tools (fcp_undo, fcp_redo, fcp_import_xml, compressor_submit). Document: 'Returns error if operation fails. Retryable on timeout. Non-retryable if file not found or permission denied.'