‹ 首页

code-review

@wellapp-ai · 收录于 5 天前 · 上游提交 1 个月前

Review code changes against hard rules and conventions

适合你,如果需要在合并前确保代码质量和一致性

/ 通过 npx 安装 校验哈希
npx oh-my-skill add wellapp-ai/well/code-review
/ 通过 bash 安装
curl -fsSL https://oh-my-skill.com/install.sh | bash -s -- wellapp-ai/well/code-review
/ 已经装过?验证本机副本,不用重装
npx oh-my-skill verify wellapp-ai/well/code-review
安装目标可用 --agent / --scope 或 --to 明确指定;省略时只会在唯一已存在的 agent 目录上自动选择,零命中或多命中会停止并提示。content_hash 缺失或不一致均拒装。
340GitHub stars
~431上下文体积 · 单文件
索引托管

怎么用

商店整理自技能原文 · 版本 782601c · 表述以原文为准
它做什么

Claude 会在每次提交代码前自动审查变更,检查硬性规则(如禁用 any 类型、console.log、组件超 200 行等)、代码质量、约定和安全问题,输出带有严重等级的问题列表和修改建议。

什么时候触发

每次你提交代码时自动触发。也可用于审查他人 PR 或自我审查时手动调用。

装好后可以这样说
自动运行所有检查。
会检查硬性规则和约定。
检查硬编码密钥等。
技能原文 SKILL.md作者撰写 · MIT · 782601c

Code Review Skill

Systematically review code for quality, conventions, and potential issues.

When to Use
  • Before committing changes (auto-triggered)
  • Reviewing others' PRs
  • Self-review before pushing
  • Pre-push validation (Phase 0)

Phase 0: Pre-Push Validation

Before reviewing code, ensure build passes:

  • [ ] npm run typecheck - No type errors
  • [ ] npm run lint - No lint errors
  • [ ] No console.log in changed files
  • [ ] Tests pass (if applicable): npm run test

If any check fails, fix before proceeding to code review.

Quick check command:

npm run typecheck && npm run lint

Check for console.log:

git diff --cached --name-only | xargs grep -l "console.log" 2>/dev/null

Review Checklist
Hard Rules Check
  • [ ] No any type usage
  • [ ] No console.log statements
  • [ ] Components under 200 lines
  • [ ] No inline styles (Tailwind only)
  • [ ] No arbitrary values (px-[13px])
  • [ ] API follows 3-layer pattern
Code Quality Check
  • [ ] Clear variable/function names
  • [ ] No duplicated code
  • [ ] Proper error handling
  • [ ] Types are specific (not unknown everywhere)
Conventions Check
  • [ ] Follows existing patterns in codebase
  • [ ] Imports organized correctly
  • [ ] File in correct location (feature folder)
Security Check
  • [ ] No hardcoded secrets/API keys
  • [ ] No exposed sensitive data
  • [ ] Proper input validation
Output Format

Present findings as:


Code Review Summary

Files Changed: [count] Issues Found: [count by severity]

Critical (must fix)
  • [ ] file.ts:L42 - [issue description]
Warnings (should fix)
  • [ ] file.ts:L15 - [issue description]
Suggestions (nice to have)
  • [ ] file.ts:L88 - [suggestion]
Approved
  • [x] No blocking issues found

Proceed with commit? (yes / fix issues first)


Auto-Trigger

This skill is automatically invoked before every commit.

按 MIT 许可原样转载,未经改动 · 在 GitHub 查看 →

评论

登录即可评论;带「已验证安装」的,是发布者名下有本店的安装或持有记录。