All skills
sanyuan0704 avatar

/code-review-expert

@ae75e91
by Shawn Yangsanyuan0704/sanyuan-skills3.9k stars
344

Expert code review of current git changes with a senior engineer lens. Detects SOLID violations, security risks, and proposes actionable improvements.

Use this Skill: https://skilld.dev/gh/sanyuan0704/sanyuan-skills/code-review-expert

This session only. Nothing lands on disk.

referencesremoval-plan.md

≈418 tokens on demand. Your agent reads this file only when SKILL.md points to it.

Removal and Iteration Plan Template

Priority Levels

  • P0: Immediate removal needed (security risk, significant cost, blocking other work)
  • P1: Remove in current sprint
  • P2: Backlog / next iteration

Safe to Remove Now

Item: [Name/Description]

Field Details
Location path/to/file.ts:line
Rationale Why this should be removed
Evidence Unused (no references), dead feature flag, deprecated API
Impact None / Low - no active consumers
Deletion steps 1. Remove code 2. Remove tests 3. Remove config
Verification Run tests, check no runtime errors, monitor logs

Defer Removal (Plan Required)

Item: [Name/Description]

Field Details
Location path/to/file.ts:line
Why defer Active consumers, needs migration, stakeholder sign-off
Preconditions Feature flag off for 2 weeks, telemetry shows 0 usage
Breaking changes List any API/contract changes
Migration plan Steps for consumers to migrate
Timeline Target date or sprint
Owner Person/team responsible
Validation Metrics to confirm safe removal (error rates, usage counts)
Rollback plan How to restore if issues found

Checklist Before Removal

  • Searched codebase for all references (rg, grep)
  • Checked for dynamic/reflection-based usage
  • Verified no external consumers (APIs, SDKs, docs)
  • Feature flag telemetry reviewed (if applicable)
  • Tests updated/removed
  • Documentation updated
  • Team notified (if shared code)

Source: SKILL.md on GitHub

No alerts16d5 checks · Risk SAFE
  • Gen Agent Trust Hub16d

    The 'Code Review Expert' skill provides structured analysis of code changes, focusing on security, performance, and design principles. It operates using local git tools and requires user approval before making any changes to the codebase, maintaining a safe and transparent workflow.

  • Socket16d

    No alerts

  • Snyk16d

    Risk: LOW · No issues

  • Runlayer7mo

    7 files scanned · No issues

  • ZeroLeaks5mo

    Score: 93/100 · 2 sections analyzed

Signed by skilld at ae75e91. This ties the file your Agent reads to that commit on GitHub. It does not review the instructions.

Last checked against GitHub 2 months ago.

Steadyupdated 7 months ago

README badge

README badge for sanyuan0704/sanyuan-skills/code-review-expert