This shell-based MCP server has critical definition gaps. Both tools lack formal input schemas (parameters are documented in descriptions only, not as JSON Schema). Tool descriptions are present but generic (gather-context: 156 chars, recent-files: 189 chars). No output schemas are documented. Parameters lack type constraints and validation guidance. No error handling patterns, no recovery guidance, and no security considerations (shell scripts executing user-provided paths invite injection). The server is not registered as a proper MCP tool provider, tool definitions appear to be shell scripts invoked directly, not MCP-compliant tool registrations with structured schemas.

• Define formal JSON Schema for both tools. gather-context input: {project_root: {type: string, description: 'Absolute or relative path to project root. Must be a readable directory.', minLength: 1, maxLength: 256}, ...}. recent-files input: {project_root: {...}, minutes: {type: integer, minimum: 1, maximum: 10080, description: 'Time window in minutes (1-10080, ~7 days max)'}.
• Document output schemas. gather-context returns {type: object, properties: {project_type: {type: string, enum: [Node.js, Python, Rust, Go, Java, Unknown]}, project_name: {type: string}, git_branch: {type: string}, git_status: {type: object, changed_files: integer, recent_commits: array}, directory_structure: array}}. recent-files returns {type: object, properties: {files: {type: array, items: {type: string}}, total_count: integer, time_window_minutes: integer}}.
• Add input validation and error handling. Validate project_root is a readable directory (not a symlink escape, not absolute path outside allowed scope). Return structured errors: {error: 'invalid_path', message: 'project_root must be a readable directory', suggestion: 'Verify the path exists and you have read permissions'}.
• Sanitize shell execution. Use realpath to resolve symlinks and validate the resolved path is within an allowed parent. Reject paths containing '..' or starting with '/'. Escape all user input passed to shell commands.
• Add parameter descriptions with constraints. 'minutes: Time window in minutes to search (1 - 10080, default 30). Larger windows may be slow on large repos.' 'project_root: Root directory path (relative or absolute, max 256 chars, must be readable). Defaults to current directory.'