Skip to main content
Glama
Sign In
mcp-rtl-review

mcp-rtl-review

by zesun33
OverviewSchemaSourceRelated ServersDiscussionsAdmin
TypeScript
Local

rtl_check_assignments

Audit Verilog/SystemVerilog files to detect blocking assignments in sequential blocks and non-blocking assignments in combinational blocks, ensuring correct RTL coding discipline.

Instructions

Audits Verilog/SystemVerilog source files specifically for assignment discipline violations: blocking '=' in sequential clocked blocks, or non-blocking '<=' in combinational blocks.

Tool call template

Replace each placeholder with a value matching the input schema. Only required parameters are included; add optional parameters as needed.

{
  "id": 1,
  "jsonrpc": "2.0",
  "method": "tools/call",
  "params": {
    "arguments": {
      "verilog_sources": "<verilog_sources>"
    },
    "name": "rtl_check_assignments"
  }
}

Input Schema

NameRequiredDescriptionDefault
cwdNoOptional working directory where source files reside.
top_moduleNoTop module name for AST hierarchy analysis.
verilog_sourcesYesList of Verilog/SystemVerilog source files to audit.

Schema Changelog

Changes observed during successful MCP inspections.

  1. First observedv0.1.0

TDQS

A3.7/5.0
Behavior3/5

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

With no annotations present, the description carries the burden of behavioral disclosure. It implies a read-only audit by using 'Audits' and specifies the exact violation categories, but it does not state whether files are modified, what side effects occur, or how violations are reported.

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 sentence that front-loads the action and resource, then immediately specifies the exact checks performed. Every word is necessary, with no fluff or redundancy.

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?

The tool has no output schema, so the description should ideally clarify what an audit result looks like. It explains what is audited but not how violations are returned (e.g., exit code, report, or structured list). It is adequate for a simple static-analysis tool but leaves a notable gap.

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?

The input schema has 100% coverage of all parameters, so the description adds no additional parameter meaning. The baseline of 3 is appropriate because the schema already documents the three parameters adequately.

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?

The description states a specific verb ('Audits') and a specific resource ('Verilog/SystemVerilog source files'), and further narrows the scope to 'assignment discipline violations.' It clearly explains what the tool does, though it does not explicitly distinguish itself from sibling tools like rtl_check_widths.

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?

The phrase 'specifically for assignment discipline violations' gives clear context for when to use this tool over more general or width-focused checks. However, it does not explicitly name alternatives or state when not to use it, so it falls short of full guidance.

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

Deploy Server

Other Tools