Getting it into your agent
One page per mod, every tool's command on it. A separate URL per tool would split the same page into five that compete with each other.
npx agentmods add agents/reviewtoolkits/cext-review-toolkit/error-path-analyzergit clone --depth 1 https://github.com/ReviewToolkits/cext-review-toolkitWrote this? Show the measurements
A badge with what this costs and how it scanned, read live from this page, so it follows the numbers instead of freezing them. Markdown for a README, HTML for a documentation site or a project page.
[](https://agentmods.dev/agents/reviewtoolkits/cext-review-toolkit/error-path-analyzer)<a href="https://agentmods.dev/agents/reviewtoolkits/cext-review-toolkit/error-path-analyzer"><img src="https://agentmods.dev/badge/agents/reviewtoolkits/cext-review-toolkit/error-path-analyzer.svg" alt="Measured on agentmods" height="20"></a>What it costs to keep this loaded
Counted locally with the o200k_base tokenizer, which is exact for GPT models; Claude uses its own tokenizer and its counts differ. Treat this as one consistent yardstick across the catalogue rather than a bill. Prices are per million input tokens.
| Model | Per session | Once invoked |
|---|---|---|
| Fable 5 | $0.00086 | $0.03281 |
| Opus 5 | $0.00043 | $0.01640 |
| Sonnet 5 | $0.00017 | $0.00656 |
| Haiku 4.5 | $0.00009 | $0.00328 |
Grade A, and why
error-path-analyzer scanned grade A with 0 findings against 26 rules in 11 categories — prompt injection, anti-refusal, data exfiltration, privilege escalation, supply chain, agent snooping, system-prompt leakage, SSRF and excessive agency — measured 3d ago.
A static scan of the body, not an audit. Every finding is printed with the line that produced it so you can judge whether it matters here. A mod is markdown that instructs an agent; that is exactly why what it instructs is worth reading.
Nothing flagged
None of the 26 patterns this scan looks for appear in this file: no shell pipes, no recursive deletes, no credential paths, no hidden text, no instruction-override or anti-refusal phrasing, no agent-config snooping. That is not a guarantee, it is the absence of the things that are checkable.
How it starts
The opening of the file, as written. The whole thing — 188 lines — stays where its author put it; the contents beside it link to each section on GitHub.
You are an expert in Python/C API error handling conventions for C extension code. Your goal is to find error handling bugs -- missing NULL checks, returning error status without setting an exception, exception clobbering, and unchecked argument parsing -- in extension modules.
Preflight Orientation (read first)
If reports/<extension>_v1/preflight/generated_code_map.md exists, read it before Phase 1. The generated-code-mapper has already classified files (hand-written vs generator-emitted), catalogued ACCEPTABLE generator-runtime idioms with grep regexes, and surfaced project-specific patterns that flip finding classifications. Apply its orientation to:
- Skip generator-emitted files unless the mapper escalated specific lines
- Filter findings matching the mapper's ACCEPTABLE-idiom regexes
- Use project-specific patterns to flip classifications (e.g., uvloop's RAII context-object dismisses Q2 "no Release in this function" findings)
- Cross-reference any Q1–Q5 finding IDs the mapper triaged
If no preflight exists, proceed normally.
Key Concepts
The Python/C API error convention is:
- Most API functions return
NULL(forPyObject*) or-1(forint) on error, and set an exception viaPyErr_SetStringor similar before returning. - Extension functions must check return values from API calls and propagate errors by returning
NULLor-1. - Before returning an error indicator, an exception must be set. Returning
NULLwithout an active exception causes an obscureSystemError: returned a result with an error setor, worse, undefined behavior. PyErr_Clear()must be used with extreme care -- it discards the active exception. Misuse can silently swallow real errors.PyArg_ParseTupleandPyArg_ParseTupleAndKeywordsreturn 0 on failure and set an appropriateTypeError/ValueError. The caller must check the return value and propagate.
Analysis Phases
Phase 1: Automated Scan and Prioritized Triage
Run the error path scanner:
What this file has done since we first saw it
Hashed on every crawl. A supply-chain change to an agent config is a question of when, not whether, so the history is kept rather than the latest state alone.
- 3d ago First seen · 188 lines · 0 tokens per session scan A 009b95ac5b95
error-path-analyzer is an agent published in the GitHub repository ReviewToolkits/cext-review-toolkit (28 stars, last pushed 1mo ago), licensed MIT. It adds 86 tokens to every session and 3,281 once invoked, about $0.0004 per session on Opus 5. A static security scan graded it A with 0 findings. No closer match exists in the catalogue, so it is treated as the original; first seen 2026-08-30.
Other agents, from other repositories
Demonstrate
Agent for demonstrating VS Code features.
playwright-test-generator
Use this agent when you need to create automated browser tests using Playwright Examples: Context: User wants to generate a test for the test plan item.
analyzer
Analyze blind comparison results to understand WHY the winner won and generate improvement suggestions.
grader
Evaluate expectations against an execution transcript and outputs.
comparator
Compare two outputs WITHOUT knowing which skill produced them.
.NET-Notebook-Migration-Agent
Expert .NET and documentation transformation agent that migrates Polyglot Jupyter notebooks into clean Markdown and companion .NET sample code.