Instruction MCP Server

# Code Review Checklist ## Before Reviewing - [ ] Understand the context and goals of the PR - [ ] Check if there's a linked issue or ticket - [ ] Note the scope: is this a bug fix, feature, or refactor? ## Code Quality ### Correctness - [ ] Does the code do what it's supposed to do? - [ ] Are edge cases handled? - [ ] Are error conditions handled gracefully? ### Readability - [ ] Is the code easy to understand? - [ ] Are names descriptive and consistent? - [ ] Is the code well-organized? ### Maintainability - [ ] Is there unnecessary complexity? - [ ] Is there code duplication that should be refactored? - [ ] Are dependencies reasonable? ## Security - [ ] No hardcoded secrets or credentials - [ ] User input is validated and sanitized - [ ] SQL queries use parameterized statements - [ ] Sensitive data is handled appropriately ## Testing - [ ] Are there tests for new functionality? - [ ] Do existing tests still pass? - [ ] Are edge cases tested? ## Performance - [ ] Are there obvious performance issues? - [ ] Are database queries efficient? - [ ] Is there unnecessary work in loops? ## Feedback Guidelines - Be specific and actionable - Explain the "why" behind suggestions - Distinguish between blocking issues and nits - Acknowledge good work