uw-code-review

分类: 开发与编程 | 上传者: devdavv | 下载: 0 | 版本: v1.0（最新）

Unity C# 代码的预提交代码质量审查。根据规则、CODING_STANDARDS.md、NAMING_CONVENTIONS.md 和 GDD Gherkin 测试覆盖率进行检查。当代码已编写且测试正在通过但在提交之前时使用。在 /uw-cmd-implement-feature 的步骤 8.5 自动运行。触发“检查此代码”、“检查我的实现”、“提交前检查”、“代码检查”或“提交前检查”。

更新日志: Source: GitHub https://github.com/devdavv/unity-ai-workflow

目录结构

当前层级: tree/main/.claude/skills/uw-code-review/

📄 SKILL.md 3.3 KB

SKILL.md

---
name: uw-code-review
description: Pre-commit code quality review for Unity C# code. Checks against Rules, CODING_STANDARDS.md, NAMING_CONVENTIONS.md, and GDD Gherkin test coverage. Use when code has been written and tests are passing but before committing. Runs automatically at Step 8.5 of /uw-cmd-implement-feature. Triggers on "review this code", "check my implementation", "pre-commit review", "code review", or "check before commit".
---

# Code Review

Read the target files, then run this checklist. Output one structured report per review. Fix all issues before committing.

## Checklist

### 1. Rules Compliance
- [ ] All Inspector fields use `[SerializeField] private` — no `public` fields on MonoBehaviours
- [ ] All component references cached in `Awake()` or `Start()` — never in `Update/FixedUpdate/LateUpdate`
- [ ] `GameDebug` used instead of `Debug.Log` / `Debug.LogError` / `Debug.LogWarning`
- [ ] New Input System only — no `Input.GetKey*` / `Input.GetAxis*`
- [ ] `TryGetComponent<T>()` preferred over `GetComponent<T>()` where component may be absent
- [ ] No Unity API calls from background threads
- [ ] No `.meta` file modifications

### 2. Naming Conventions
- [ ] Private fields: `_camelCase`
- [ ] Public properties & methods: `PascalCase`
- [ ] Parameters & local variables: `camelCase`
- [ ] Booleans: `is/has/can/should` prefix (e.g. `_isGrounded`, `CanJump`)
- [ ] Events: `On` prefix (e.g. `OnDeath`)
- [ ] Static fields: `s_camelCase`
- [ ] Class name matches file name

### 3. Code Quality
- [ ] No magic numbers or hardcoded strings — use constants, ScriptableObjects, or serialized fields
- [ ] No `FindObjectOfType<T>()` or `GameObject.Find()` (except in editor-only code)
- [ ] No cross-assembly references that violate the TDD's asmdef dependency graph
- [ ] K&R braces (opening brace on same line)
- [ ] 4-space indentation (no tabs)

### 4. Performance
- [ ] No `new` allocations inside `Update/FixedUpdate/LateUpdate` (no GC pressure in hot paths)
- [ ] Particles are pooled or have Stop Action set to `Destroy` with pool backing
- [ ] DOTween/PrimeTween tweens are killed in `OnDestroy` or `OnDisable`
- [ ] No `string` concatenation in hot paths — use `GameDebug` with conditional compilation

### 5. Test Coverage
- [ ] Every GDD Gherkin `Given/When/Then` scenario for this feature has a corresponding NUnit test
- [ ] EditMode tests for pure logic; PlayMode tests for MonoBehaviour lifecycle
- [ ] All tests currently passing

### 6. Game Feel
- [ ] If "later" was chosen in `/uw-cmd-implement-feature` Step 3: `// TODO(gamefeel): [feature] — pending` comment exists in the main script
- [ ] If "later" was chosen: a pending row exists in `docs/GFD.md` Feedback Matrix
- [ ] If "now" was chosen: Rule of Three satisfied (visual + audio + kinesthetic minimum)

## Output Format

```
## Code Review — {FeatureName}

### Rules          [✅ Pass | ⚠️ Issues]
### Naming            [✅ Pass | ⚠️ Issues]
### Code Quality      [✅ Pass | ⚠️ Issues]
### Performance       [✅ Pass | ⚠️ Issues]
### Test Coverage     [✅ X/Y scenarios | ⚠️ Missing]
### Game Feel         [✅ Integrated | ⏳ TODO filed | ⚠️ Missing]

Issues to fix before commit:
- [list each issue with file and line, or "None"]
```

Do not commit until **Issues to fix** is "None".

登录后下载/点赞/收藏 ❤ 10 | ★ 0

uw-code-review

目录结构

SKILL.md

举报内容

提示