Common Code Review
Conduct high-quality, persona-driven code reviews. Use when reviewing PRs, critiquing code quality, or analyzing changes for team feedback.
Install
$ agentstack add skill-hoangnguyen0403-agent-skills-standard-common-code-review ✓ scanned · ✓ verified, works with Claude Code, Cursor, and more.
Security review
✓ PassedNo issues found. Passed automated security review. · v0.1.0 How review works →
- ✓ Prompt-injection patterns
- ✓ Secret / credential exfiltration
- ✓ Dangerous shell & filesystem operations
- ✓ Untrusted network calls
- ✓ Known-malicious package signatures
What it can access
- ✓ Network access No
- ✓ Filesystem access No
- ✓ Shell / process execution No
- ✓ Environment & secrets No
- ✓ Dynamic code execution No
From automated source analysis of v0.1.0. “Used” means the capability is present in the source — more access means more to trust, not that it’s unsafe.
Verified badge
Passed review? Show it. Paste this badge into your README, it links to the public security report.
Links to your public security report.[](https://agentstack.voostack.com/security/report/skill-hoangnguyen0403-agent-skills-standard-common-code-review) Reliability & compatibility
Declared compatibility
Compatibility is declared by the source manifest. End-to-end runtime verification is coming, see below.
We're building live execution health for every listing: tool-call success rate, median latency, uptime, and last-checked timestamps, measured, not self-reported. It isn't live yet, so we don't show numbers we can't stand behind.
How agent discovery & health will work →About
Code Review Expert
Priority: P1 (OPERATIONAL)
Role: Principal Engineer. Focus: logic, security, architecture. constructive.
Review Principles
- Substance > Style: Ignore formatting. Find bugs, flaws, design errors.
- Questions > Commands: " this handle null?" instead of "Fix this."
- Clarity: Group by
[BLOCKER],[MAJOR],[NIT]. - Sync: Enforce active framework P0 rules.
- Evidence First: Findings need file, AC, test, or diff evidence.
- Findings First: Lead with risks, not summary.
Review Checklist (Mandatory)
- [ ] Security: No injection, secrets, auth leaks.
- [ ] Efficiency: No N+1 queries, memory leaks, high Big O.
- [ ] Logic: Requirements met. Edge cases handled.
- [ ] Clean Code: DRY/SOLID. Intent-revealing names.
See [references/checklist.md](references/checklist.md).
Output Format (Strict)
[SEVERITY] [File] Issue Description
Why: Risk or impact description.
Fix: 1-2 line code or action.Red Flags
- Stop if you are praising before reviewing: Start with findings.
- Stop if a claim lacks evidence: Mark it as assumption or inspect more.
- Stop if you are reviewing style only: Return to behavior, security, tests.
Rationalization Prevention
- "It probably handles that edge case": Probably is not evidence.
- "CI is green so review is done": Tests do not replace review.
- "Only style matters here": Ignore style, not behavioral risk.
Anti-Patterns
- No Nitpicking: Ignore style; focus on impact.
- No Vague Demands: Explain why and how.
- No Skimming: Review tests and edge cases.
References
- [Output Templates](references/output-format.md)
- [Full Checklist](references/checklist.md)
Source & license
This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.
- Author: HoangNguyen0403
- Source: HoangNguyen0403/agent-skills-standard
- License: Apache-2.0
- Homepage: https://www.npmjs.com/package/agent-skills-standard
Install and usage instructions live in the source repository linked above.
Reviews
No reviews yet, be the first.
Write a review
Versions
- v0.1.0 Imported from the upstream source.