Agent skill
shared-review-workflow
Severity definitions, approval gate protocol, and fix constraints shared across all review and test skills.
Install this agent skill to your Project
npx add-skill https://github.com/majiayu000/claude-skill-registry/tree/main/skills/other/other/shared-review-workflow
SKILL.md
Purpose
Define the common review workflow shared by code-review, design-review, and unit-test skills across all platforms.
Severity definitions
| Severity | Meaning |
|---|---|
| must-fix | Security vulnerability, broken functionality, blocks deployment or App Store submission |
| should-fix | Standards violation, maintainability concern, tech debt |
| nice-to-have | Style preference, minor improvement, optional optimization |
Approval gate
If issues are found and --no-fix is not set:
"Found X issues (Y must-fix). Approve fixes? (yes/no)"
Do not modify code before approval.
Fix and confirm (if approved)
- Apply fixes
- Re-run automated gates (lint, format, typecheck)
- Run unit tests to confirm no regressions
- Report final status
Fix constraints
- Don't add dependencies unless required
- Don't weaken assertions to make tests pass
- Prefer targeted fixes over broad refactors
- Don't disable or skip tests to make them pass
Recommended Agent Skills
Expand your agent's capabilities with these related and highly-rated skills.
majiayu000/claude-skill-registry
agent-ops-spec
Manage specification documents in .agent/specs/. Use when user provides requirements, acceptance criteria, or feature descriptions that need to be tracked and validated against implementation.
majiayu000/claude-skill-registry
agent-ops-state
Maintain .agent state files. Use at session start, after meaningful steps, and before concluding: read/update constitution/memory/focus/issues/baseline consistently.
majiayu000/claude-skill-registry
agent-ops-spec
Manage specification documents in .agent/specs/. Use when user provides requirements, acceptance criteria, or feature descriptions that need to be tracked and validated against implementation.
majiayu000/claude-skill-registry
agent-ops-testing
Test strategy, execution, and coverage analysis. Use when designing tests, running test suites, or analyzing test results beyond baseline checks.
majiayu000/claude-skill-registry
agent-ops-testing
Test strategy, execution, and coverage analysis. Use when designing tests, running test suites, or analyzing test results beyond baseline checks.
majiayu000/claude-skill-registry
agent-ops-state
Maintain .agent state files. Use at session start, after meaningful steps, and before concluding: read/update constitution/memory/focus/issues/baseline consistently.
Didn't find tool you were looking for?