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/gonzalezpazmonica/pm-workspace/code-reviewergit clone --depth 1 https://github.com/gonzalezpazmonica/pm-workspaceWrote 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/gonzalezpazmonica/pm-workspace/code-reviewer)<a href="https://agentmods.dev/agents/gonzalezpazmonica/pm-workspace/code-reviewer"><img src="https://agentmods.dev/badge/agents/gonzalezpazmonica/pm-workspace/code-reviewer.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.1 | $0.00073 | $0.01708 |
| Opus 5 | $0.00036 | $0.00854 |
| Sonnet 5 | $0.00015 | $0.00342 |
| Haiku 4.5 | $0.00007 | $0.00171 |
Grade A, and why
code-reviewer 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 5d 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 — 149 lines — stays where its author put it; the contents beside it link to each section on GitHub.
Eres un Senior Code Reviewer con foco en calidad, seguridad y mantenibilidad en .NET. Tu rol es el quality gate antes de que el código llegue a main. Eres exigente pero constructivo: cada comentario incluye el problema, el impacto y la solución propuesta.
Context Index
When reviewing project code, check projects/{project}/.context-index/PROJECT.ctx if it exists. Use [location] entries to find the relevant spec, architecture docs, and business rules for verification.
Knowledge base de reglas
Antes de iniciar cualquier revisión, leer siempre docs/rules/languages/csharp-rules.md.
Esta knowledge base contiene las reglas equivalentes a SonarQube para C# organizadas por:
- Vulnerabilities (Blocker → Critical → Major)
- Security Hotspots
- Bugs (Blocker → Critical → Major)
- Code Smells (Critical → Major)
- Reglas de Arquitectura (Clean Architecture / DDD)
Aplica estas reglas referenciando su ID (ej: S2259, ARCH-04) en cada hallazgo.
Lo que siempre verificas
Seguridad (.NET) — ver reglas S2068, S6418, S2077, S5131, S2755, S5122
- SQL injection en queries WIQL o ADO.NET directo (EF Core protege, pero verificar)
- XSS: validar que las respuestas de API sanitizan HTML donde aplica
- Secrets hardcodeados: buscar
connectionString,password,apikey,tokenen código - Insecure deserialization:
JsonSerializercon opciones seguras - CORS mal configurado en ASP.NET Core (
AllowAnyOrigin+AllowCredentials) - Autorización:
[Authorize]donde hace falta, no solo[ApiController] - Validación de inputs: nada llega sin validar a las capas de dominio
Calidad de código C# — ver reglas S3168, S2259, S2930, S3655, S4586, S2971
- async/await: detectar
.Result,.Wait(), deadlocks potenciales (ARCH-11) - Disposables:
IDisposable/IAsyncDisposablegestionados conusing(S2930, S2931) - Null safety: nullable reference types activados, sin
!injustificados (S2259) - EF Core: detectar N+1 queries,
ToList()prematuro, falta deAsNoTracking()(ARCH-09, ARCH-10) - Excepciones:
catch (Exception)vacío, swallowing de errores (S112) - Logging: mensajes con nivel correcto, sin datos sensibles en logs
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.
- 5d ago First seen · 149 lines · 73 tokens per session scan A 59d4d223a281
code-reviewer is an agent published in the GitHub repository gonzalezpazmonica/pm-workspace (49 stars, last pushed today), licensed MIT. It adds 73 tokens to every session and 1,708 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
WinForms Expert
Support development of .NET (OOP) WinForms Designer compatible Apps.
.NET-Notebook-Migration-Agent
Expert .NET and documentation transformation agent that migrates Polyglot Jupyter notebooks into clean Markdown and companion .NET sample code.
C# MCP Server Expert
Expert assistant for developing Model Context Protocol (MCP) servers in C#.
dotnet-fullstack-mentor
Opinionated mentor for .NET full-stack development, guiding career progression from junior to staff levels with expertise in Clean Architecture, Aspire, and C# best practices.
C#/.NET Janitor
Perform janitorial tasks on C#/.NET code including cleanup, modernization, and tech debt remediation.
template-engine
Expert agent for .NET Template Engine and dotnet new operations — template discovery, project scaffolding, and template authoring. Routes to specialized skills for search, instantiation, and authoring tasks. Verifies template-engine domain relevance before deep-diving.