llm-code-security-review-mcp
Click on "Install Server".
Wait a few minutes for the server to deploy. Once ready, it will show a "Started" state.
In the chat, type
@followed by the MCP server name and your instructions, e.g., "@llm-code-security-review-mcpLoad the review plan at /path/to/plan.json and begin the review."
That's it! The server will respond to your query, and you can continue using it as needed.
Here is a step-by-step guide with screenshots.
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
Load a review plan.
Codex receives one subsystem and one security area at a time.
The server checks that the assigned source was read before accepting the result.
Progress is saved automatically. You can restart Codex, pause the review, or continue later without starting over.
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 testsReleased 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 toolsget_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.
| Name | Required | Description | Default |
|---|---|---|---|
No parameters | |||
TDQS
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.
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.
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.
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.
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.
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.
| Name | Required | Description | Default |
|---|---|---|---|
| path | Yes |
TDQS
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.
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.
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.
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.
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.
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.
| Name | Required | Description | Default |
|---|---|---|---|
| outcome | Yes | ||
| step_id | Yes | ||
| summary | Yes | ||
| evidence | Yes | ||
| artifacts | Yes | ||
| finding_ids | Yes |
TDQS
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.
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.
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.
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.
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.
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.
statusARead-onlyIdempotent
Return read-only authoritative review progress and diagnostics. After the server reports COMPLETE, call status {} and use only its diagnostics for final totals.
| Name | Required | Description | Default |
|---|---|---|---|
No parameters | |||
TDQS
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.
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.
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.
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.
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.
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.
4 tool updates
v0.5.0- First observed
get_task - First observed
load_plan - First observed
report_done - First observed
status
TDQS
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.
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.
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.
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
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
MCP server providing access to the Scorecard API to evaluate and optimize LLM systems.
MCP server for generating rough-draft project plans from natural-language prompts.
MCP server for AI agents to plan, verify, and deploy Cloudflare-native apps.
MCP server for secureFlows: token-free URL builders and integration-linting tools for AI agents.
Related MCP Servers
- AlicenseDqualityDmaintenanceAn MCP server that implements a structured workflow for LLM-based coding, guiding development through feature clarification, documentation generation, phased implementation, and progress tracking.106718MIT
- AlicenseAqualityBmaintenanceMCP server that integrates DevFlow with AI code assistants to enforce structured development workflows including planning, task tracking, and code review gates.652471MIT
- AlicenseNot gradedqualityDmaintenanceMCP server that securely reviews AI-generated code for vulnerabilities such as SQL injection, command injection, and hardcoded credentials.11Apache 2.0
- AlicenseNot gradedqualityFmaintenanceA lightweight MCP server for security reviews that injects security requirements before code generation, scans dependencies for CVEs, and verifies generated code without disrupting workflow.68MIT
Latest Blog Posts
- Who's Calling? MCP Hosts Are an Identity Blind Spot (And the Spec Knows It)By Om-Shree-0709 on .mcpAgent IdentityOAuth 2.1
- Your AI Chatbot Just Exposed Your CEO's Salary to an InternBy Om-Shree-0709 on .Agent IdentityMCP SecurityOAuth Delegation
- Why MCP Servers Need Execution Sandboxing (And Why Your Current Stack Isn't Enough)By Om-Shree-0709 on .Agentic AiPrompt InjectionWebAssembly
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