Critic-MCP — Der gnadenlose Code-Kritiker

Ein Open-Source-Model Context Protocol (MCP)-Server, der den von anderen KI-Coding-Assistenten (Cursor, OpenCode, Cline, etc.) produzierten Code überprüft – schreibgeschützt.

Critic-MCP ist ein „zweites Augenpaar“: Es repariert niemals Ihren Code, es kritisiert ihn nur gnadenlos. Es stellt ein einziges Tool (review_code) zur Verfügung und hat absolut keine Dateischreibfähigkeit.

Was macht es?

Das review_code-Tool vergleicht den von Ihnen gesendeten Code mit der ursprünglichen Anforderung (Absicht) und erstellt über ein LLM einen Überprüfungsbericht mit den folgenden Abschnitten:

• Urteil: APPROVED | MODIFICATION_REQUIRED | REJECTED

• Fehlende Anforderungen — die Lücke zwischen Absicht und Code

• Sicherheitsergebnisse — SQL-Injection, XSS, Privilegieneskalation, hartcodierte Geheimnisse

• Grenzfall-Ergebnisse — null/leere Eingaben, Grenzwerte, Off-by-One, Race Conditions

• Leistungsergebnisse — N+1-Abfragen, Speicherlecks, redundante Berechnungen

• Weitere Ergebnisse + Muss-korrigiert-Elemente (in Prioritätsreihenfolge)

Related MCP server: codereview-mcp

Installation — Zwei Schritte

Voraussetzung: Node.js >= 20

Schritt 1: Authentifizieren (einmalig)

Führen Sie die interaktive Einrichtung aus, die genau wie aws configure oder gh auth login funktioniert:

npx -y critic-mcp auth

Es fragt, welchen Anbieter Sie verwenden (gemini / openai / deepseek), fordert Ihren API-Schlüssel an und speichert beide in ~/.critic-mcp.json in Ihrem Home-Verzeichnis (0600-Berechtigungen unter Unix).

Schritt 2: Fügen Sie es Ihrer IDE hinzu

Fügen Sie nur dies zu den MCP-Einstellungen Ihrer IDE hinzu:

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

Siehe Abschnitt KI-Assistent-Integration für client-spezifische Details. Das war's – Ihre Schlüssel befinden sich jetzt an einem Ort, außerhalb jeder IDE-Konfiguration.

Schlüssel werden niemals in IDE-Konfigurationen geschrieben. Wenn der Server startet, sucht er zuerst in process.env, dann in ~/.critic-mcp.json; wird in keinem ein Schlüssel gefunden, werden Sie zu npx critic-mcp auth weitergeleitet.

Lokale Entwicklung (Installation aus dem Quellcode)

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

Befehle

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

Umgebungsvariablen (optional)

Alle sind optional; der normale Weg für API-Schlüssel ist npx critic-mcp auth. Umgebungsvariablen haben immer Vorrang vor der Konfigurationsdatei (für CI/Server-Setups).

KI-Assistent-Integration

Keine der untenstehenden Konfigurationen enthält Schlüssel; Sie authentifizieren sich einmalig über den auth-Befehl (Schritt 1 oben). npx erfordert, dass das Paket auf npm veröffentlicht ist; für einen lokalen Klon können Sie stattdessen "command": "node", "args": ["ABSOLUTE_PATH/dist/index.js"] verwenden.

Cursor

In der projektweiten .cursor/mcp.json (oder der globalen ~/.cursor/mcp.json):

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

Alternativ: Einstellungen → MCP → Neuen MCP-Server hinzufügen, dann das JSON einfügen.

OpenCode

In der projektweiten .opencode/opencode.json oder der globalen ~/.config/opencode/opencode.json:

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

OpenCode verwendet den mcp-Schlüssel (nicht mcpServers) und das environment-Feld (nicht env); command muss ein Array sein. Sie müssen keine Schlüssel mehr in einen environment-Block schreiben.

Cline (VS Code-Erweiterung)

Öffnen Sie das Cline-Panel → Registerkarte MCP Servers → Globales MCP bearbeiten oder Projekt-MCP bearbeiten, dann bearbeiten Sie das JSON:

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

autoApprove erlaubt Cline, review_code ohne Bestätigung auszuführen; das ist sicher, da das Tool niemals Dateien schreibt.

Continue.dev

Fügen Sie den MCP-Server zu ~/.continue/config.json hinzu (stdio-Transport wird unabhängig von Ihrer Continue-Version unterstützt):

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

Manuelles Testszenario

examples/bad_code.js ist ein Express-Beispiel, das absichtlich SQL-Injection, XSS und N+1-Abfragen enthält; examples/intent.txt enthält die ursprüngliche Anforderung. Rufen Sie es von einem beliebigen Client wie folgt auf:

"Überprüfen Sie den Code in examples/bad_code.js mit dem review_code-Tool. Anforderung: examples/intent.txt"

Erwarten Sie, dass der Kritiker mindestens Folgendes erkennt:

• KRITISCH: db.query("SELECT * FROM users WHERE email = '" ...) — SQL-Injection

• KRITISCH: res.send(comment.body) — gespeichertes XSS

• HOCH: Eine separate Abfrage pro Benutzer — N+1-Problem

Architektur

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)

Chunked-Überprüfung (Map-Reduce)

Wenn code_snippet CHUNK_SIZE (Standard 30.000 Zeichen) überschreitet, wechselt das System automatisch zu einem Map-Reduce-Ablauf:

1. Map: Der Code wird an Zeilengrenzen aufgeteilt; jeder Block wird gleichzeitig an das LLM gesendet (Standard 3 parallele Anfragen, konfigurierbar über CRITIC_CONCURRENCY). Ein einzelner Blockfehler stoppt niemals die gesamte Überprüfung.

2. Reduce: Alle zurückgegebenen Teilanalysen werden durch den "Synthesizer"-Prompt zusammengeführt – der niemals Ergebnisse abschwächt und niemals APPROVED zurückgibt, wenn ein einzelner Teil KRITISCH meldet – zu einem endgültigen Bericht.

Der Server gibt nur einen String-Bericht zurück; er hat keine Dateischreibfähigkeit und setzt niemals einen Netzwerkclient nach außen frei.

Lizenz

MIT