Agent skill
scan-code-review-rules
[Documentation] Scan project and populate/sync docs/project-reference/code-review-rules.md with code conventions, anti-patterns, architecture rules, and review checklists.
Install this agent skill to your Project
npx add-skill https://github.com/duc01226/EasyPlatform/tree/main/.claude/skills/scan-code-review-rules
SKILL.md
[IMPORTANT] Use
TaskCreateto break ALL work into small tasks BEFORE starting — including tasks for each file read. This prevents context loss from long files. For simple tasks, AI MUST ask user whether to skip.
Prerequisites: MUST READ before executing:
Scan & Update Reference Doc — Read existing doc first, scan codebase for current state, diff against doc content, update only changed sections, preserve manual annotations. MUST READ
.claude/skills/shared/scan-and-update-reference-doc-protocol.mdfor full protocol and checklists.
Quick Summary
Goal: Scan project codebase for established conventions, lint rules, common patterns, and anti-patterns, then populate docs/project-reference/code-review-rules.md with actionable review rules and checklists. (content auto-injected by hook — check for [Injected: ...] header before reading)
Workflow:
- Read — Load current target doc, detect init vs sync mode
- Scan — Discover conventions and patterns via parallel sub-agents
- Report — Write findings to external report file
- Generate — Build/update reference doc from report
- Verify — Validate rules reference real code patterns
Key Rules:
- Generic — works with any language/framework combination
- Derive rules from ACTUAL codebase patterns, not generic best practices
- Every rule should have a "DO" example from the project and a "DON'T" counterexample
- Focus on project-specific conventions that differ from framework defaults
Be skeptical. Apply critical thinking, sequential thinking. Every claim needs traced proof, confidence percentages (Idea should be more than 80%).
Scan Code Review Rules
Phase 0: Read & Assess
- Read
docs/project-reference/code-review-rules.md - Detect mode: init (placeholder) or sync (populated)
- If sync: extract existing sections and note what's already well-documented
Phase 1: Plan Scan Strategy
Discover code quality infrastructure:
- Linter configs (
.eslintrc,.editorconfig,stylecop.json,.prettierrc,ruff.toml) - CI quality gates (build scripts, test requirements, coverage thresholds)
- Code analysis configs (SonarQube, CodeClimate, custom analyzers)
- Existing code standards docs (CONTRIBUTING.md, CODING_STANDARDS.md)
- Git hooks (pre-commit, husky configs)
Use docs/project-config.json if available for architecture rules and naming conventions.
Phase 2: Execute Scan (Parallel Sub-Agents)
Launch 3 Explore agents in parallel:
Agent 1: Backend Rules
- Grep for naming conventions (class suffixes, method prefixes, interface naming)
- Find common base classes and when they're used vs not used
- Discover error handling patterns (try-catch, Result types, error middleware)
- Find dependency injection patterns (registration conventions, lifetime choices)
- Look for anti-patterns (direct DB access from controllers, business logic in wrong layer)
- Identify logging conventions (structured logging, log levels, correlation IDs)
Agent 2: Frontend Rules
- Grep for component conventions (naming, file organization, template patterns)
- Find state management rules (what goes in store vs component vs service)
- Discover styling conventions (BEM, CSS modules, utility classes, naming)
- Find subscription/memory management patterns (cleanup, unsubscribe)
- Look for accessibility patterns (ARIA, semantic HTML, keyboard navigation)
- Identify performance patterns (lazy loading, change detection, memoization)
Agent 3: Architecture Rules
- Find layer boundaries (what imports what, dependency direction)
- Discover cross-service communication patterns (direct calls vs messages)
- Find shared code conventions (what's shared vs duplicated)
- Look for testing conventions (test naming, test organization, mock patterns)
- Identify security patterns (auth checks, input validation, output encoding)
- Find configuration patterns (env vars, config files, secrets management)
Write all findings to: plans/reports/scan-code-review-rules-{YYMMDD}-{HHMM}-report.md
Phase 3: Analyze & Generate
Read the report. Build these sections:
Target Sections
| Section | Content |
|---|---|
| Critical Rules | Top 5-10 rules that cause the most bugs/issues if violated |
| Backend Rules | Naming, patterns, error handling, DI conventions with DO/DON'T examples |
| Frontend Rules | Component, state, styling, cleanup conventions with DO/DON'T examples |
| Architecture Rules | Layer boundaries, cross-service rules, shared code conventions |
| Anti-Patterns | Common mistakes found in codebase with explanations and fixes |
| Decision Trees | Flowcharts for common decisions (which base class, where to put logic, etc.) |
| Checklists | PR review checklists for backend, frontend, and cross-cutting concerns |
Content Rules
- Every rule must have a "DO" code example from the actual project
- Every rule should have a "DON'T" counterexample (real or realistic)
- Use
file:linereferences for all code examples - Prioritize rules by impact (bugs prevented, not style preferences)
- Decision trees can use markdown flowchart format or nested bullet lists
Phase 4: Write & Verify
- Write updated doc with
<!-- Last scanned: YYYY-MM-DD -->at top - Verify: 5 code example file paths exist (Glob check)
- Verify: anti-pattern examples are realistic (not fabricated)
- Report: sections updated, rules count, anti-patterns discovered
Closing Reminders
- MUST break work into small todo tasks using
TaskCreateBEFORE starting - MUST search codebase for 3+ similar patterns before creating new code
- MUST cite
file:lineevidence for every claim (confidence >80% to act) - MUST add a final review todo task to verify work quality
- MUST execute two review rounds (Round 1: understand, Round 2: catch missed issues) MANDATORY IMPORTANT MUST READ the following files before starting:
- MUST READ
.claude/skills/shared/scan-and-update-reference-doc-protocol.mdbefore starting
Recommended Agent Skills
Expand your agent's capabilities with these related and highly-rated skills.
fix-parallel
[Implementation] Analyze & fix issues with parallel fullstack-developer agents
ask
[Utilities] Answer technical and architectural questions.
claude-code
[Utilities] Claude Code CLI setup, configuration, troubleshooting, and feature guidance. Triggers on claude code setup, hook not firing, MCP connection, context limit, skill creation, slash command setup.
workflow-deployment
[Workflow] Trigger Deployment & Infrastructure workflow — CI/CD pipelines, Docker, Kubernetes setup and deployment.
workflow-idea-to-pbi
[Workflow] Trigger Idea to PBI workflow — po/ba workflow: capture idea, refine to pbi, create stories, prioritize.
easy-claude-help
[Utilities] Configuration guide for the easy-claude framework — explain settings, guide users through configuring .ck.json.
Didn't find tool you were looking for?