code-review

Enforce TypeScript, database, and MCP protocol standards during pull requests.

2|1|Updated Oct 3, 2025
One-click install
npx skills add https://github.com/joelmnz/mcp-markdown-manager --skill code-review-joelmnz
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/joelmnz/mcp-markdown-manager/tree/main/.opencode/skills/code-review
Command: npx skills add https://github.com/joelmnz/mcp-markdown-manager --skill code-review-joelmnz

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Code review conformance for the MCP Markdown Manager project ensures architectural discipline, strong typing, secure data handling, correct MCP protocol usage, and comprehensive tests by guiding reviewers through a standardized checklist.

Core Features & Use Cases

  • Architecture / Boundaries: Ensure routes/handlers call the service facade and avoid direct DB access.
  • TypeScript / Imports: Enforce strict typing and ESModule imports with no type suppressions.
  • Database Operations: Require parameterized queries, proper client handling, and error management.
  • Authentication & Security: Enforce auth scopes for write/read endpoints and safe token usage.
  • MCP Protocol: Validate MCP tool definitions, argument validation, and response formats.
  • Embeddings / Performance: Ensure embedding tasks are non-blocking and gracefully degrade.

Quick Start

Perform the PR review using this checklist to ensure architecture, typing discipline, security, and MCP protocol compliance before merging.

Frequently Asked Questions about code-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I enforce architectural boundaries and prevent direct database access during PR reviews?

To enforce architectural boundaries during PR reviews, validate that routes and handlers exclusively call the service facade, ensuring no direct database access occurs outside the designated service layer.

How do I validate TypeScript discipline and ESModule imports in a pull request?

Validate TypeScript discipline in a pull request by enforcing strict typing across all modules, requiring explicit ESModule imports, and rejecting any type suppressions or bypasses.

What's the best way to check for secure database operations and parameterized queries in code reviews?

The best way to check for secure database operations is to verify that all queries are parameterized, database client handling is properly managed, and comprehensive error management is implemented.

How do I ensure MCP protocol compliance for tool definitions and argument validation?

Ensure MCP protocol compliance by validating that tool definitions are correctly structured, arguments are rigorously validated, and response formats strictly match the expected MCP protocol standards.

Does this code review checklist cover authentication scopes for read and write endpoints?

Yes, the code review checklist covers authentication scopes by enforcing specific auth scopes for both write and read endpoints, alongside verifying safe token usage across the application.

Why should embedding tasks be non-blocking and gracefully degrade in backend architectures?

Embedding tasks should be non-blocking to prevent performance bottlenecks in backend architectures, ensuring they gracefully degrade and do not disrupt primary application workflows under load.