principal-code-review
Skill0 starsChanged 5 months agoMITView on GitHub
Review application source code or a diff at a senior/principal bar for correctness, reliability, performance, security, and observability. Use when the user asks to "review this code", review a function/PR/diff, or hunt for bugs and edge cases in a specific piece of code. NOT for high-level system architecture (use architecture-review).
In this repository12 other files it gives its agents.
- adr-writerSkill
- architecture-reviewSkill
- gcp-architectureSkill
- incident-postmortemSkill
- interview-modeSkill
- leadership-lensSkill
- learning-coachSkill
- llm-app-architectureSkill
- mlops-pipeline-reviewSkill
- ml-system-designSkill
- observability-reviewSkill
- rag-system-designSkill
The file
Files
Code Review Standards
Review the code across every dimension. Classify findings by severity.
Correctness
- Edge cases
- Null / empty handling
- Failure paths
Reliability
- Retry strategy
- Timeout handling
- Circuit breaking
Performance
- Time / space complexity
- Memory allocation
- Database access patterns (N+1, missing indexes)
Security
- Injection
- Secrets handling
- Access control
Observability
- Metrics
- Logs
- Traces
Output (grouped by severity)
- Critical issues
- Major issues
- Minor issues
- Suggestions
Does it work?
Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.
No reviews yet
People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.
- It worked
- Partly
- It didn't
Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.
Your trail
Files you open will collect here, so you can find them again.
Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.
Now do this for your own.
Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.