Code review
Skill 0xc000022070/agentic-flake/examples/3-home-manager/skills/code-review
Composable agent skills and project-scoped environments for Nix.
npx -y skills add 0xc000022070/agentic-flake --skill code-reviewAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
2 things to look at
- no licenseNo license file was found in the repository. Code published without one is not open source by default, so using it at work is a question for whoever answers licensing questions where you are.
- 1 stars1 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
What its author says it does
Copied from the file, not written here
Code review guidelines and quality standards
SKILL.md
1.8 KB, as published. Nobody here has run it
Code Review Guidelines
Standards for reviewing code across all projects.
What We Review
- Readability — Is the code clear and understandable?
- Correctness — Does it do what it's supposed to do?
- Performance — Any obvious inefficiencies?
- Security — Vulnerabilities or risky patterns?
- Tests — Adequate coverage for changes?
Code Clarity
Good Code
- Descriptive variable names
- Functions do one thing
- Comments explain "why", not "what"
- Consistent formatting
Common Issues
| Issue | Example | Fix |
|---|---|---|
| Unclear names | x = y + z | totalPrice = itemCost + tax |
| Deep nesting | 5+ levels of if | Extract functions, use guards |
| Magic numbers | if (age > 18) | const ADULT_AGE = 18 |
| Long functions | 100+ line function | Break into smaller functions |
| Missing tests | Critical logic without tests | Add comprehensive tests |
Performance Red Flags
- N+1 queries (loop with DB calls inside)
- Unnecessary data copies in loops
- Blocking operations on main thread
- Regular expressions without optimization
Security Checklist
- Input validation on user data
- No hardcoded secrets
- SQL queries use parameterized queries
- Auth/authorization checks present
- Error messages don't leak sensitive info
- Dependencies checked for vulnerabilities
Approval Criteria
Approve when:
- ✅ Code meets standards
- ✅ Tests are adequate
- ✅ No security issues
- ✅ Performance acceptable
- ✅ Documentation clear
Request changes when:
- ❌ Readability concerns
- ❌ Missing tests
- ❌ Security issues
- ❌ Performance degradation
- ❌ Breaking changes undocumented