Critic-MCP — Безжалостный критик кода

Open-source сервер Model Context Protocol (MCP), который рецензирует — только для чтения — код, созданный другими AI-ассистентами программирования (Cursor, OpenCode, Cline и др.).

Critic-MCP — это «второй взгляд»: он никогда не исправляет ваш код, он только безжалостно его критикует. Он предоставляет единственный инструмент (review_code) и не имеет абсолютно никаких возможностей записи в файлы.

Что он делает?

Инструмент review_code сравнивает отправленный вами код с исходным требованием (намерением) и через LLM создает отчет с рецензией, содержащий следующие разделы:

• Вердикт: APPROVED | MODIFICATION_REQUIRED | REJECTED

• Отсутствующие требования — разрыв между намерением и кодом

• Проблемы безопасности — SQL-инъекции, XSS, повышение привилегий, хардкоженные секреты

• Проблемы с крайними случаями — null/пустые входные данные, граничные значения, ошибки на единицу, состояния гонки

• Проблемы производительности — N+1 запросы, утечки памяти, избыточные вычисления

• Прочие замечания + Обязательные к исправлению пункты (в порядке приоритета)

Установка — два шага

Требование: Node.js >= 20

Шаг 1: Аутентификация (однократно)

Запустите интерактивную настройку, которая работает так же, как aws configure или gh auth login:

npx -y critic-mcp auth

Она спрашивает, какого провайдера вы используете (gemini / openai / deepseek), запрашивает ваш API-ключ и сохраняет оба в ~/.critic-mcp.json в вашей домашней директории (права 0600 на Unix).

Шаг 2: Добавьте его в вашу IDE

Добавьте только это в настройки MCP вашей IDE:

{ "command": "npx", "args": ["-y", "critic-mcp"] }

Смотрите раздел Интеграция с AI-ассистентами для деталей, специфичных для клиента. Вот и всё — теперь ваши ключи хранятся в одном месте, вне любой конфигурации IDE.

Ключи никогда не записываются в конфиги IDE. Когда сервер запускается, он сначала смотрит в process.env, затем в ~/.critic-mcp.json; если ключ не найден ни в одном из них, он направляет вас к npx critic-mcp auth.

Локальная разработка (установка из исходников)

git clone https://github.com/layermedya/Critic-MCP.git
cd Critic-MCP
npm ci
npm run build
node dist/index.js auth   # authenticate against your own build

Команды

npm run build       # TypeScript compilation
npm run typecheck   # Type checking
npm test            # Vitest unit tests
npm run test:watch  # Tests in watch mode
npm start           # Start the server on stdio
npm run inspect     # Manual testing in the browser via MCP Inspector

Переменные окружения (опционально)

Все они опциональны; обычный путь для API-ключей — npx critic-mcp auth. Переменные окружения всегда имеют приоритет над конфигурационным файлом (для CI/серверных настроек).

Интеграция с AI-ассистентами

Ни одна из приведенных ниже конфигураций не содержит ключей; вы проходите аутентификацию один раз с помощью команды auth (шаг 1 выше). npx требует, чтобы пакет был опубликован на npm; для локального клона вы можете использовать "command": "node", "args": ["ABSOLUTE_PATH/dist/index.js"].

Cursor

В файле .cursor/mcp.json на уровне проекта (или глобальном ~/.cursor/mcp.json):

{
  "mcpServers": {
    "critic": {
      "command": "npx",
      "args": ["-y", "critic-mcp"]
    }
  }
}

Альтернативно: Settings → MCP → Add new MCP server, затем вставьте JSON.

OpenCode

В файле .opencode/opencode.json на уровне проекта или глобальном ~/.config/opencode/opencode.json:

{
  "mcp": {
    "critic": {
      "type": "local",
      "command": ["npx", "-y", "critic-mcp"],
      "enabled": true
    }
  }
}

OpenCode использует ключ mcp (не mcpServers) и поле environment (не env); command должен быть массивом. Вам больше не нужно записывать ключи в блок environment.

Cline (расширение для VS Code)

Откройте панель Cline → вкладка MCP Servers → Edit Global MCP или Edit Project MCP, затем отредактируйте JSON:

{
  "mcpServers": {
    "critic": {
      "command": "npx",
      "args": ["-y", "critic-mcp"],
      "disabled": false,
      "autoApprove": ["review_code"]
    }
  }
}

autoApprove позволяет Cline запускать review_code без подтверждения; это безопасно, так как инструмент никогда не записывает файлы.

Continue.dev

Добавьте MCP сервер в ~/.continue/config.json (транспорт stdio поддерживается независимо от вашей версии Continue):

{
  "experimental": {
    "modelContextProtocolServers": [
      {
        "transport": {
          "type": "stdio",
          "command": "npx",
          "args": ["-y", "critic-mcp"]
        }
      }
    ]
  }
}

Ручной тестовый сценарий

examples/bad_code.js — это пример Express, который намеренно содержит SQL-инъекцию, XSS и N+1 запросы; examples/intent.txt содержит исходное требование. Запустите его из любого клиента следующим образом:

«Проверь код в examples/bad_code.js с помощью инструмента review_code. Требование: examples/intent.txt»

Ожидается, что критик обнаружит как минимум следующее:

• КРИТИЧЕСКИ: db.query("SELECT * FROM users WHERE email = '" ...) — SQL-инъекция

• КРИТИЧЕСКИ: res.send(comment.body) — сохраненный XSS

• ВЫСОКИЙ: Отдельный запрос для каждого пользователя — проблема N+1

Архитектура

src/index.ts   -> MCP server, zod validation, error handling + `auth` argv routing
src/cli.ts     -> Interactive authentication flow (`critic-mcp auth`)
src/config.ts  -> Global config (~/.critic-mcp.json) + credential resolution (env → file)
src/prompt.ts  -> Ruthless Critic system prompt + chunked-review prompts
src/llm.ts     -> Provider layer + timeout protection + map-reduce orchestration
src/chunker.ts -> Line-ending based chunking (for code above the limit)

Рецензирование по частям (map-reduce)

Когда code_snippet превышает CHUNK_SIZE (по умолчанию 30 000 символов), система автоматически переключается на поток map-reduce:

1. Map: Код разбивается по границам строк; каждый фрагмент отправляется в LLM конкурентно (по умолчанию 3 параллельных запроса, настраивается через CRITIC_CONCURRENCY). Сбой одного фрагмента никогда не останавливает всю рецензию.

2. Reduce: Все возвращенные частичные анализы объединяются с помощью промпта «Синтезатор» — который никогда не ослабляет выводы и никогда не возвращает APPROVED, если одна часть сообщает о КРИТИЧЕСКОМ — в один финальный отчет.

Сервер возвращает только строковый отчет; он не имеет возможности записи в файлы и никогда не открывает наружу сетевого клиента.

Лицензия

MIT