Code Review Agent MCP

ИИ-агент для ревью кода, который не приукрашивает. Если ваш код плохой — он скажет это. Прямо.

MCP-сервер (Model Context Protocol) на базе ИИ, который ревьюит ваш код так, как опытные мейнтейнеры ревьюят патчи в серьёзных проектах: технично, прямо, без дипломатии.

Что он делает

• ИИ-ревью кода — ревьюит фрагменты кода, файлы, git-диффы и коммиты

• Возвращает замечания с метками серьёзности: CRITICAL, MAJOR, MINOR, NIT, CLEAN

• Указывает конкретные номера строк для каждого замечания

• Объясняет, почему что-то плохо, а затем — как это исправить

• Завершает чётким вердиктом: «Ship it.» или «Do not merge until...»

• Никакой ложной похвалы. Никаких оговорок. Никаких извинений. Никакой воды.

Related MCP server: greybeard

Зачем?

Большинство ИИ-ревьюеров кода обучены быть вежливыми. Они предваряют критику словами «отличная работа, но...», смягчают её фразами «возможно, вам стоит рассмотреть...», разбавляют замечания «это потенциально может быть проблемой».

Так серьёзное ревью кода не работает. В реальных проектах (Linux kernel, PostgreSQL, Redis, SQLite) мейнтейнеры ревьюят патчи напрямую. Они находят проблемы и чётко о них заявляют. Они существуют не для того, чтобы автору было приятно, — они существуют, чтобы код стал лучше.

Этот MCP воплощает эту традицию. Личность и есть продукт.

Anti-RLHF инженерия

LLM обучены с помощью RLHF быть вежливыми. Этот ИИ-агент борется с этим на трёх уровнях:

1. Системный промпт — 12 ключевых принципов (код, а не кодер; вердикт в первую очередь; метки серьёзности; указание строк; никакой ложной похвалы; никаких оговорок; никаких извинений; никакой воды; второе лицо; никаких упоминаний демографии; никаких отказов; никаких выдуманных багов)

2. Пост-процессор — удаляет 40+ запрещённых фраз (оговорки, смягчения, извинения, вода) из вывода LLM

3. Валидатор — проверяет, что в выводе есть метки серьёзности, указания строк и вердикт; помечает расплывчатые формулировки проблем как сигналы галлюцинаций

Инструменты

Уровни жёсткости

Установка

pip install code-review-agent-mcp

Или с помощью uv:

uv pip install code-review-agent-mcp

Конфигурация

Claude Desktop

Добавьте в claude_desktop_config.json:

{
  "mcpServers": {
    "code-review-agent": {
      "command": "python",
      "args": ["-m", "code_review_agent.server"]
    }
  }
}

Или если установлено через pip:

{
  "mcpServers": {
    "code-review-agent": {
      "command": "code-review-agent-mcp"
    }
  }
}

Cursor

Добавьте в .cursor/mcp.json:

{
  "mcpServers": {
    "code-review-agent": {
      "command": "python",
      "args": ["-m", "code_review_agent.server"]
    }
  }
}

Примеры использования

Ревью фрагмента кода

User: Review this code for me

def get_user(username):
    import sqlite3
    conn = sqlite3.connect("users.db")
    cursor = conn.cursor()
    query = f"SELECT * FROM users WHERE username = '{username}'"
    cursor.execute(query)
    return cursor.fetchone()

Ответ MCP:

## Code Review: snippet

### Findings

**CRITICAL** `snippet:7` — SQL injection
The query uses an f-string with user input, allowing SQL injection. Use parameterized queries: `cursor.execute("SELECT * FROM users WHERE username = ?", (username,))`.

### Verdict

Do not merge until CRITICAL is fixed.

Ревью файла

User: Review src/auth.py

MCP calls review_file with file_path="src/auth.py"
Returns blunt review with line citations.

Ревью коммита

User: Review the last commit

MCP calls review_commit with commit_ref="HEAD"
Returns blunt review of the diff.

Метки серьёзности

Безопасность

Этот MCP-сервер реализует песочницу безопасности:

• Доступ к файлам по умолчанию ограничен текущей рабочей директорией

• Чувствительные пути (.ssh, .aws, .env, /etc/passwd и т.д.) отклоняются

• Git-ссылки проверяются по строгому белому списку символов для предотвращения инъекции опций

• Вызовы подпроцессов используют shell=False и отключают глобальную git-конфигурацию

Полную модель угроз см. в SECURITY.md.

Разработка

# Install in development mode
pip install -e ".[dev]"

# Run tests
pytest

# Run tests with coverage
pytest --cov=code_review_agent

Бенчмарк-фрагменты

Директория benchmarks/ содержит 5 регрессионных фрагментов, проверяющих ревьюера:

1. SQL-инъекция — ожидается CRITICAL, указание строки, «Do not merge»

2. Изменяемый аргумент по умолчанию — ожидается MAJOR

3. Чистый код (бинарный поиск) — ожидается CLEAN, «Ship it» (тест на анти-галлюцинации)

4. Проглоченное исключение — ожидается MAJOR

5. Ошибка на единицу — ожидается MAJOR

Бенчмарк с чистым кодом — самый важный: он ловит галлюцинации. Если ревьюер выдумывает баги в корректном коде, анти-RLHF система сломана.

Лицензия

MIT

Благодарности

Этот проект воплощает традицию ревью кода мейнтейнеров ядра — методологию, применяемую многими старшими инженерами во многих проектах (Linux kernel, PostgreSQL, Redis, SQLite и других). Мы ссылаемся на традицию, а не на какого-либо конкретного практика.