Skip to main content
Glama
drewrukin

llm-code-security-review-mcp

by drewrukin

LLM Code Security Review MCP Server

This MCP server is a companion tool for the llm-code-security-review methodology. It helps an LLM follow the review plan one step at a time. The LLM performs the security review; the server tracks progress, makes sure the LLM actually reads all source code assigned to each subsystem, and does not accept the review as complete until every plan step is finished.

The Short Version

  1. Load a review plan.

  2. Codex receives one subsystem and one security area at a time.

  3. The server checks that the assigned source was read before accepting the result.

  4. Progress is saved automatically. You can restart Codex, pause the review, or continue later without starting over.

  5. Findings must be reviewed separately. The server organizes the work; it does not decide whether a reported issue is real.

That is the whole job.

Related MCP server: DevFlow MCP Server

What You Need

  • Codex on Linux, macOS, or another POSIX system;

  • Python 3.11 or newer;

  • a review plan created with the companion methodology.

The server has no third-party Python dependencies.

Install It

git clone https://github.com/drewrukin/llm-code-security-review-mcp.git
cd llm-code-security-review-mcp
python3 -m venv .venv
. .venv/bin/activate
python3 -m pip install .

Add the server to ~/.codex/config.toml. Replace the example path with the absolute path to your clone:

[mcp_servers.llm-code-security-review-mcp]
command = "/absolute/path/to/llm-code-security-review-mcp/.venv/bin/llm-code-security-review-mcp-server"

Start a new Codex session after changing the configuration.

Run a Review

First, use the companion methodology to create a plan for your project. Then start a new Codex session and say:

Use the llm-code-security-review-mcp server to load <path-to-plan.json> and execute the review.

The server will provide the first task and keep the review moving until the plan is complete.

Development

Run the test suite with:

python3 -m unittest discover -s tests

Released changes are in CHANGELOG.md. Please report security problems privately as described in SECURITY.md.

License

Copyright 2026 Andrew Rukin (drewrukin). Licensed under the Apache License 2.0.

Available Tools

4 tools
get_taskA

Issue or repeat the one current security-review task. Obey its execution_instruction: read source serially in one-file or <=300-line calls, submit report_done for mechanical coverage verification, then immediately call get_task {} again.

ParametersJSON Schema
NameRequiredDescriptionDefault

No parameters

TDQS

A4.9/5.0
Behavior5/5

Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?

Beyond the annotations (openWorldHint=false, destructiveHint=false), the description reveals non-obvious behavior: get_task can be re-invoked to repeat the current task, its response carries an execution_instruction, and the agent is expected to loop back. This meaningfully helps the agent predict and act on the tool's behavior.

Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.

Conciseness5/5

Is the description appropriately sized, front-loaded, and free of redundancy?

The description is a single dense sentence that front-loads the core purpose and then packs the necessary behavioral workflow into the rest. Every clause adds information; there is no filler or restatement.

Shorter descriptions cost fewer tokens and are easier for agents to parse. Every sentence should earn its place.

Completeness5/5

Given the tool's complexity, does the description cover enough for an agent to succeed on first attempt?

For a zero-parameter tool with no output schema, the description is sufficient: it says what the tool does, what the agent should do with the returned task, and what to call next. The absence of detailed return-field documentation is acceptable because the execution_instruction is explicitly referenced.

Complex tools with many parameters or behaviors need more documentation. Simple tools need less. This dimension scales expectations accordingly.

Parameters4/5

Does the description clarify parameter syntax, constraints, interactions, or defaults beyond what the schema provides?

The tool has zero parameters and the schema fully covers this, so no parameter detail is needed. The description also reinforces the expected call shape by literally showing 'get_task {} again,' which is a small but useful confirmation.

Input schemas describe structure but not intent. Descriptions should explain non-obvious parameter relationships and valid value ranges.

Purpose5/5

Does the description clearly state what the tool does and how it differs from similar tools?

The description states a clear verb-resource pair: it issues or repeats 'the one current security-review task.' It also distinguishes itself from sibling tools by framing get_task as the source of the task and the loop entry point, with report_done as follow-up.

Agents choose between tools based on descriptions. A clear purpose with a specific verb and resource helps agents select the right tool.

Usage Guidelines5/5

Does the description explain when to use this tool, when not to, or what alternatives exist?

The description explicitly explains the workflow: read source serially, submit report_done for coverage verification, then immediately call get_task {} again. This gives the agent concrete when-to-use-next guidance and names the sibling tool involved, leaving little to inference.

Agents often have multiple tools that could apply. Explicit usage guidance like "use X instead of Y when Z" prevents misuse.

load_planA

Load the supplied security-review plan before execution.

ParametersJSON Schema
NameRequiredDescriptionDefault
pathYes

TDQS

A3.5/5.0
Behavior3/5

Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?

Annotations declare non-destructive and non-open-world, so the description only needs to add context. It adds that this is a load step in a security-review workflow, but doesn't explain side effects, what happens to the loaded plan, or validation behavior. That is a modest addition, not a rich one.

Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.

Conciseness5/5

Is the description appropriately sized, front-loaded, and free of redundancy?

One sentence, front-loaded with the action and resource. No wasted words.

Shorter descriptions cost fewer tokens and are easier for agents to parse. Every sentence should earn its place.

Completeness3/5

Given the tool's complexity, does the description cover enough for an agent to succeed on first attempt?

With one parameter, no output schema, and simple annotations, the description covers the basics but misses details like expected return, behavior on invalid paths, and the exact meaning of 'load.' For an agent to invoke this correctly, the path semantics and outcome should be clearer.

Complex tools with many parameters or behaviors need more documentation. Simple tools need less. This dimension scales expectations accordingly.

Parameters2/5

Does the description clarify parameter syntax, constraints, interactions, or defaults beyond what the schema provides?

The schema has 0% description coverage for the single 'path' parameter, so the description must compensate. It only refers to the plan as 'supplied,' implying the path is the plan location, but doesn't clarify path type, format, or how the plan is consumed. This leaves an agent guessing about a required parameter.

Input schemas describe structure but not intent. Descriptions should explain non-obvious parameter relationships and valid value ranges.

Purpose4/5

Does the description clearly state what the tool does and how it differs from similar tools?

States a specific verb ('Load') and resource ('security-review plan'), with a usage context ('before execution'). It clearly differs from siblings like get_task and status, though it doesn't name them. Slight ambiguity remains in what 'load' means operationally.

Agents choose between tools based on descriptions. A clear purpose with a specific verb and resource helps agents select the right tool.

Usage Guidelines4/5

Does the description explain when to use this tool, when not to, or what alternatives exist?

Provides a clear usage context: 'before execution' signals when to call this tool. It does not explicitly exclude alternatives or name siblings, but for a simple plan-loading step this is sufficient.

Agents often have multiple tools that could apply. Explicit usage guidance like "use X instead of Y when Z" prevents misuse.

report_doneA

Submit the structured result. This call mechanically verifies source coverage and returns the exact missing read when incomplete. For NO FINDINGS, finding_ids, evidence, and artifacts must all be empty. For FINDING or CORROBORATION, include only concise source ranges needed to prove the security effect. After an accepted result, immediately call get_task {} and continue until COMPLETE; after COMPLETE, call status {} and use only its diagnostics for final totals.

ParametersJSON Schema
NameRequiredDescriptionDefault
outcomeYes
step_idYes
summaryYes
evidenceYes
artifactsYes
finding_idsYes

TDQS

A4.4/5.0
Behavior4/5

Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?

The description discloses meaningful behavior beyond the annotations: it mechanically verifies source coverage, returns the exact missing read when incomplete, and enforces content constraints per outcome. This is especially valuable because annotations only indicate non-destructiveness and closed-world behavior, not the verification workflow.

Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.

Conciseness5/5

Is the description appropriately sized, front-loaded, and free of redundancy?

The description is dense but every sentence earns its place: purpose, validation behavior, per-outcome payload rules, and next-step workflow. It is front-loaded with the core action and avoids filler.

Shorter descriptions cost fewer tokens and are easier for agents to parse. Every sentence should earn its place.

Completeness4/5

Given the tool's complexity, does the description cover enough for an agent to succeed on first attempt?

For a structured submission tool with no output schema, the description provides strong workflow context and error behavior. It could be more complete by explaining what an 'accepted result' looks like and clarifying step_id/summary expectations, but overall it gives enough for an agent to operate correctly.

Complex tools with many parameters or behaviors need more documentation. Simple tools need less. This dimension scales expectations accordingly.

Parameters3/5

Does the description clarify parameter syntax, constraints, interactions, or defaults beyond what the schema provides?

With 0% schema description coverage, the description partially compensates by explaining outcome-specific requirements for finding_ids, evidence, and artifacts, and by describing evidence as concise source ranges. However, it leaves step_id, summary, and the evidence note field semantically unexplained, so the compensation is incomplete.

Input schemas describe structure but not intent. Descriptions should explain non-obvious parameter relationships and valid value ranges.

Purpose5/5

Does the description clearly state what the tool does and how it differs from similar tools?

The description opens with a specific verb and resource: 'Submit the structured result.' It goes on to explain that the tool verifies source coverage and reports missing reads, making its role clear. It also distinguishes itself from siblings by locating it in the workflow between get_task and status.

Agents choose between tools based on descriptions. A clear purpose with a specific verb and resource helps agents select the right tool.

Usage Guidelines5/5

Does the description explain when to use this tool, when not to, or what alternatives exist?

The description provides concrete when-to-use guidance: submit results, then call get_task until COMPLETE, then call status for final totals. It also gives outcome-specific formatting rules for NO FINDINGS versus FINDING/CORROBORATION, leaving little ambiguity about when and how to invoke the tool.

Agents often have multiple tools that could apply. Explicit usage guidance like "use X instead of Y when Z" prevents misuse.

statusA
Read-onlyIdempotent

Return read-only authoritative review progress and diagnostics. After the server reports COMPLETE, call status {} and use only its diagnostics for final totals.

ParametersJSON Schema
NameRequiredDescriptionDefault

No parameters

TDQS

A4.4/5.0
Behavior4/5

Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?

Annotations already declare readOnlyHint, openWorldHint, idempotentHint, and destructiveHint, so the safety profile is covered. The description adds meaningful behavioral context beyond annotations: the tool is authoritative for final totals, and its diagnostics should be exclusively trusted after COMPLETE. No contradiction with annotations.

Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.

Conciseness5/5

Is the description appropriately sized, front-loaded, and free of redundancy?

Two sentences, no wasted words. The core purpose and read-only nature are front-loaded, followed by the critical usage instruction. Every phrase contributes meaning.

Shorter descriptions cost fewer tokens and are easier for agents to parse. Every sentence should earn its place.

Completeness4/5

Given the tool's complexity, does the description cover enough for an agent to succeed on first attempt?

For a parameterless status tool, the description covers what it does, when to call it, and how to use its output for final totals. It does not describe the structure of diagnostics, but the authoritative usage guidance compensates enough given the simple interface and no output schema.

Complex tools with many parameters or behaviors need more documentation. Simple tools need less. This dimension scales expectations accordingly.

Parameters4/5

Does the description clarify parameter syntax, constraints, interactions, or defaults beyond what the schema provides?

The tool has zero parameters and the schema is fully descriptive with an empty object, so there is no parameter burden for the description to carry. The description even models the intended call as 'status {}', which is helpful. Baseline for 0 params is 4; no additional explanation is needed.

Input schemas describe structure but not intent. Descriptions should explain non-obvious parameter relationships and valid value ranges.

Purpose5/5

Does the description clearly state what the tool does and how it differs from similar tools?

Description states a specific verb ('Return') and resource ('read-only authoritative review progress and diagnostics'), clearly distinguishing it as the source of final totals. The phrase 'authoritative' positions it against siblings like get_task without ambiguity. It is immediately obvious what this tool does.

Agents choose between tools based on descriptions. A clear purpose with a specific verb and resource helps agents select the right tool.

Usage Guidelines4/5

Does the description explain when to use this tool, when not to, or what alternatives exist?

Explicitly states when to call it: 'After the server reports COMPLETE, call status {}'. This gives clear timing context and tells the agent to use its diagnostics for final totals. It does not explicitly name alternatives or when not to use it, so it falls short of a 5.

Agents often have multiple tools that could apply. Explicit usage guidance like "use X instead of Y when Z" prevents misuse.

Tool Schema Changelog

Recent tool additions, removals, and schema changes observed during successful MCP inspections. Dates show when Glama detected each change.

  1. 4 tool updatesv0.5.0
    • First observedget_task
    • First observedload_plan
    • First observedreport_done
    • First observedstatus

TDQS

A4.2/5.0
Disambiguation5/5

Each tool has a distinct role: load_plan initializes the review, get_task fetches the active task, status reports progress, and report_done submits results. Despite verbose instructions, the boundaries are clear and an agent should not confuse them.

Naming Consistency4/5

load_plan and get_task follow a clean verb_noun pattern, and all names use lowercase snake_case. status is a bare noun and report_done uses an adjective complement rather than a noun object, so the pattern is not perfect but remains readable and predictable.

Tool Count5/5

Four tools is well-scoped for a focused security-review orchestration server. Each tool maps to an essential phase of the workflow and none feels redundant or missing.

Completeness5/5

The tool surface covers the full intended lifecycle: load the plan, retrieve tasks, submit results, and check final status. There are no obvious dead ends or missing operations within the server's clearly defined orchestration purpose.

Maintenance

ActivityMaintained
ResponsivenessNo issues

Resources

Unclaimed servers have limited discoverability.

Looking for Admin?

If you are the server author, to access and configure the admin panel.

Related MCP Connectors

Related MCP Servers

  • A
    license
    D
    quality
    D
    maintenance
    An MCP server that implements a structured workflow for LLM-based coding, guiding development through feature clarification, documentation generation, phased implementation, and progress tracking.
    10
    67
    18
    MIT
  • A
    license
    Not graded
    quality
    D
    maintenance
    MCP server that securely reviews AI-generated code for vulnerabilities such as SQL injection, command injection, and hardcoded credentials.
    11
    Apache 2.0
  • A
    license
    Not graded
    quality
    F
    maintenance
    A lightweight MCP server for security reviews that injects security requirements before code generation, scans dependencies for CVEs, and verifies generated code without disrupting workflow.
    68
    MIT

Latest Blog Posts

MCP directory API

We provide all the information about MCP servers via our MCP API.

curl -X GET 'https://glama.ai/api/mcp/v1/servers/drewrukin/llm-code-security-review-mcp'

If you have feedback or need assistance with the MCP directory API, please join our Discord server