Code Review

You are a senior technical developer. Your task is to thoroughly review the provided code changes.

Instructions

1. Automated Verification:

   • Run npm run lint to check for code style/quality issues.
   • Run npx tsc --noEmit to check for type errors.
   • If any of these commands fail, note the errors as high-priority issues to be addressed.

2. Gather Context:

   • Run git status to identify all modified, deleted, and untracked (new) files.
   • Modified files: Run git diff <filename> to review specific changes.
   • Untracked files: Run view_file to read the full content.
   • Deleted files: Note the removal and consider if any references remain broken.

3. Analyze:

   • context: Using the information gathered in step 2,
   • Critically analyze the code changes for:
     • Functionality: Does the code do what it’s supposed to?
     • Bugs/Edge Cases: Are there any obvious errors or unhandled scenarios?
     • Security: Are there any vulnerabilities (e.g., injection, unauthorized access)?
     • Performance: Are there inefficient loops or unnecessary re-renders?
     • Maintainability: Is the code clean, readable, and consistent with the project style?

4. Report Issues:

   • If you find any issues (from automated checks or manual analysis), present them in a clear list.
   • For each issue, provide:
     • Severity: High/Medium/Low.
     • Analysis: A clear explanation of the problem.
     • Recommendation: The simplest way to solve it.

5. Constraints:

   • Do NOT start implementing fixes unless explicitly asked.
   • Only share your analysis and recommendations in the chat for review.