Korean-contracts has clear intent (PII masking for contract workflows) but suffers from critical definition gaps. Tool names lack action verbs (mask_personal_info is acceptable, but save_contract and load_contract_for_review are vague about what 'save' and 'load' mean in context). Descriptions are present but generic, they explain the Korean workflow rather than guiding LLM tool selection. Critically, input schemas are INFERRED from docstrings, not explicitly visible in the code. The `contract_data` parameter in mask_personal_info accepts a dict with no schema constraints, LLMs cannot validate structure. Output schemas are undocumented. Error handling is minimal (only session_id lookup failure shown). No parameter descriptions for most inputs (e.g., output_dir defaults to ~/Desktop with no guidance on format). Security concern: session_id is a weak identifier stored in-memory; no audit trail or permission checks.

• Define explicit JSON Schema for `contract_data` input. Specify required fields (employeeName, employeeIdFront, etc.), types (string, int), and constraints. Example: {"type": "object", "required": ["employeeName", "baseSalary"], "properties": {"employeeName": {"type": "string", "description": "Full name of employee (e.g., 홍길동)"}, ...}}
• Document output schemas for all tools. For mask_personal_info, return {"session_id": string, "masked": object, "message": string}. For save_contract, return {"status": string, "txt_path": string, "docx_path": string|null, "message": string}. Include field descriptions.
• Rename tools for clarity: 'mask_personal_info' → 'create_masked_contract' (emphasizes creation); 'save_contract' → 'restore_and_save_contract' (clarifies restoration step); 'load_contract_for_review' → 'read_and_mask_contract' (explicit read+mask); 'save_reviewed_contract' → 'restore_and_save_reviewed_contract'.
• Add parameter descriptions and constraints. For output_dir: 'Absolute or relative path where contract files are saved. Defaults to ~/Desktop. Must be writable.' For contract_type: 'Filename prefix (e.g., employment-contract, freelance-agreement). Alphanumeric and hyphens only.' For file_path: 'Absolute or relative path to contract file. Supported formats: .txt, .docx, .md. File must exist and be readable.'
• Enhance error handling with recovery guidance. Example: 'Session expired (session_id={session_id}). Call create_masked_contract() to generate a new session.' For file errors: 'File not found: {file_path}. Check path and permissions.' For docx parsing: 'Failed to parse DOCX: {error}. Ensure file is valid Office format. Try converting to .txt first.'