This server has severe definition quality gaps. Tool names lack action verbs (discoverCourses, syncCourse, syncGoogleDrive are acceptable but parsePortal is vague). Most critically, NO input schemas are visible in the source code, only informal descriptions in the evaluation prompt. The downloadFile tool shows a schema in the prompt, but the actual tool registration code is not provided. Without seeing explicit tool definitions in index.js or the MCP server entry point, tool existence is inferred. Descriptions are present but generic (e.g., 'Discovers enrolled courses' lacks WHEN/WHY context). No output schemas documented. No error handling guidance. No parameter validation rules visible. This is a custom STDIO server with minimal MCP integration evidence.

• Create explicit MCP tool registration with full JSON Schema for all 5 tools. Include type, description, and constraints for every parameter. Example: {"type": "object", "properties": {"courseId": {"type": "string", "description": "Numeric course ID or course name (string). If name, will be resolved via courses.json lookup."}}, "required": ["courseId"]}
• Expand tool descriptions to 100-200 characters. Include WHEN to use (e.g., 'Call this first to enumerate available courses before syncing'), WHAT it returns (e.g., 'Returns array of {id, name} objects'), and any prerequisites (e.g., 'Requires valid credentials in ~/lms_creds.txt').
• Implement secret injection: move CRED_FILE and SESSION_FILE paths to server-side environment variables or a secure vault. Never expose credential paths as tool parameters. Document the expected format of lms_creds.txt (username\npassword) in server setup docs, not in tool descriptions.
• Add output schema documentation for each tool. Example for discoverCourses: 'Returns {courses: [{id: string, name: string}], total: number}'. Document pagination if results exceed 50 items.
• Implement error classification and recovery guidance. For syncCourse, catch 'Course not found' and return 'Course ID not found. Call discoverCourses() to list available courses.' For downloadFile, catch network errors and return 'Download failed (timeout). Retry with a shorter timeout or check the URL.' Add a dry-run mode for destructive operations.