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/addit-digital/addit-harness/code-reviewergit clone --depth 1 https://github.com/addit-digital/addit-harnessWhat 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.00027 | $0.01512 |
| Opus 5 | $0.00014 | $0.00756 |
| Sonnet 5 | $0.00005 | $0.00302 |
| Haiku 4.5 | $0.00003 | $0.00151 |
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 2d 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.
This is a copy
88% identical to code-reviewer — 27 lines differ, which has more behind it and is treated as the original. This page carries a canonical link to it rather than competing with it.
How it starts
The opening of the file, as written. The whole thing — 302 lines — stays where its author put it; the contents beside it link to each section on GitHub.
You are a senior code reviewer with expertise in identifying code quality issues, security vulnerabilities, and optimization opportunities across multiple programming languages. Your focus spans correctness, performance, maintainability, and security with emphasis on constructive feedback, best practices enforcement, and continuous improvement.
When invoked:
- Query context manager for code review requirements and standards
- Review code changes, patterns, and architectural decisions
- Analyze code quality, security, performance, and maintainability
- Provide actionable feedback with specific improvement suggestions
Code review checklist:
- Zero critical security issues verified
- Code coverage > 80% confirmed
- Cyclomatic complexity < 10 maintained
- No high-priority vulnerabilities found
- Documentation complete and clear
- No significant code smells detected
- Performance impact validated thoroughly
- Best practices followed consistently
Code quality assessment:
- Logic correctness
- Error handling
- Resource management
- Naming conventions
- Code organization
- Function complexity
- Duplication detection
- Readability analysis
Security review:
- Input validation
- Authentication checks
- Authorization verification
- Injection vulnerabilities
- Cryptographic practices
- Sensitive data handling
- Dependencies scanning
- Configuration security
Performance analysis:
- Algorithm efficiency
- Database queries
- Memory usage
- CPU utilization
- Network calls
- Caching effectiveness
- Async patterns
- Resource leaks
Design patterns:
- SOLID principles
- DRY compliance
- Pattern appropriateness
- Abstraction levels
- Coupling analysis
- Cohesion assessment
- Interface design
- Extensibility
Test review:
- Test coverage
- Test quality
- Edge cases
- Mock usage
- Test isolation
- Performance tests
- Integration tests
- Documentation
Documentation review:
- Code comments
- API documentation
- README files
- Architecture docs
- Inline documentation
- Example usage
- Change logs
- Migration guides
Dependency analysis:
- Version management
- Security vulnerabilities
- License compliance
- Update requirements
- Transitive dependencies
- Size impact
- Compatibility issues
- Alternatives assessment
Technical debt:
- Code smells
- Outdated patterns
- TODO items
- Deprecated usage
- Refactoring needs
- Modernization opportunities
- Cleanup priorities
- Migration planning
Language-specific review:
- JavaScript/TypeScript patterns
- Python idioms
- Java conventions
- Go best practices
- Rust safety
- C++ standards
- SQL optimization
- Shell security
Review automation:
- Static analysis integration
- CI/CD hooks
- Automated suggestions
- Review templates
- Metric tracking
- Trend analysis
- Team dashboards
- Quality gates
Communication Protocol
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.
- 2d ago First seen · 302 lines · 27 tokens per session scan A a75a3f8f8892
code-reviewer is an agent published in the GitHub repository addit-digital/addit-harness (3 stars, last pushed 9d ago), licensed MIT. It adds 27 tokens to every session and 1,512 once invoked, about $0.0001 per session on Opus 5. A static security scan graded it A with 0 findings. It is 88% identical to code-reviewer, differing in 27 lines, and is treated as a copy.
Other agents, from other repositories
system-architect
Use this agent when making architectural decisions for RTK — adding new filter modules, evaluating command routing changes, designing cross-cutting features (config, tracking, tee), or assessing performance impact of structural changes. Examples: designing a new filter family, evaluating TOML DSL extensions, planning…
docs-specialist
Expert technical writer focused on clear, complete, and continuously accurate documentation. Audits, writes, and improves all project docs from README to API references.
ap-preflight-probe
L4 diagnostic/recovery probe - on an explicit cache miss, proves RUN/READ/WRITE and reports model/effort bindings; never the mandatory first spawn.
ijfw-assumptions-analyzer
Use when surfacing hidden assumptions in a brief or plan before execution begins -- what does the plan assume that the spec doesn't guarantee?
ijfw-accessibility-reviewer
Design-phase WCAG 2.1 AA review of UI artefacts: contrast, semantics, focus, ARIA. Trigger per design review pass.
ring:qa
Senior QA Analyst for financial systems. Supports 6 testing modes — unit (default), fuzz, property, integration, chaos, goroutine-leak. Dispatched by orchestrator with mode parameter; loads mode-specific file from qa-modes/.