一键导入
code-review
Review code quality, patterns, performance, and best practices. Provides structured feedback with actionable recommendations.
用 Codex 或 Claude 帮你安装 复制这段 Prompt,粘贴到 Codex、Claude 或其他助手里,让它检查 Skill 页面并帮你完成安装。
菜单
Review code quality, patterns, performance, and best practices. Provides structured feedback with actionable recommendations.
用 Codex 或 Claude 帮你安装 复制这段 Prompt,粘贴到 Codex、Claude 或其他助手里,让它检查 Skill 页面并帮你完成安装。
基于 SOC 职业分类
| name | code-review |
| description | Review code quality, patterns, performance, and best practices. Provides structured feedback with actionable recommendations. |
| license | MIT |
审查代码质量、模式和最佳实践,提供可操作的建议。
| 命令 | 行动 |
|---|---|
/code-review | 审查最近更改 |
/code-review [path] | 审查特定目录 |
/code-review [file] | 审查特定文件 |
ASK:
审查范围:
A) 最近更改(git diff)
B) 特定目录: [path]
C) 特定文件: [file]
D) 整个src/目录
STOP: Wait for user's choice.
TRY these tools (skip if unavailable):
| 工具 | 命令 | 目的 |
|---|---|---|
| LSP Diagnostics | lsp_diagnostics | 类型错误、语法错误 |
| Git Diff | git diff | 查看更改 |
IF tools unavailable: Say "自动检查工具不可用,将进行手动审查"
REPORT:
自动检查结果:
━━━━━━━━━━━━━━━━━━━━━━
语法错误: [N]
类型警告: [N]
其他问题: [N]
━━━━━━━━━━━━━━━━━━━━━━
STOP: Show results, ask "要继续手动审查吗?"
READ: Target files
核心检查项 (精简版):
| 检查项 | 状态 |
|---|---|
| 功能正确? | ✅/⚠️/❌ |
| 边缘情况处理? | ✅/⚠️/❌ |
| 空值安全? | ✅/⚠️/❌ |
| 检查项 | 状态 |
|---|---|
| 命名规范? | ✅/⚠️/❌ |
| 函数职责单一? | ✅/⚠️/❌ |
| 无重复代码? | ✅/⚠️/❌ |
| 检查项 | 状态 |
|---|---|
| 无不必要分配? | ✅/⚠️/❌ |
| 不在_process做重活? | ✅/⚠️/❌ |
| 检查项 | 状态 |
|---|---|
| 静态类型使用? | ✅/⚠️/❌ |
| @onready用于节点引用? | ✅/⚠️/❌ |
| 信号使用恰当? | ✅/⚠️/❌ |
STOP: Present findings, ask "要深入分析某个问题吗?"
SEVERITY LEVELS:
| 级别 | 描述 | 示例 |
|---|---|---|
| 🔴 阻塞 | Bug、崩溃、安全问题 | 空指针、资源泄漏 |
| 🟡 重要 | 性能、可维护性 | 重复代码、缺少类型 |
| 🟢 建议 | 风格、小改进 | 命名不一致 |
| 💡 信息 | 教育性提示 | 最佳实践建议 |
# 代码审查: [File/Directory]
## 摘要
[1-2句话总体评价]
## 🔴 阻塞问题
### [问题标题]
- **文件**: [file:line]
- **问题**: [描述]
- **修复**: [如何修复]
## 🟡 重要问题
### [问题标题]
- **文件**: [file:line]
- **问题**: [描述]
- **修复**: [如何修复]
## 🟢 建议
- [文件:line] [改进建议]
## ✅ 亮点
- [做得好的地方]
## 统计
| 指标 | 值 |
|------|-----|
| 审查文件 | [N] |
| 审查行数 | [N] |
| 阻塞问题 | [N] |
| 重要问题 | [N] |
| 建议 | [N] |
ASK: "需要我帮助修复这些问题吗?"
STOP: Wait for user's response.
IF user wants help fixing:
IF user wants tasks:
STOP: Review complete.
# 静态类型
var health: int = 100
func take_damage(amount: int) -> void:
pass
# @onready 缓存
@onready var health_bar: ProgressBar = %HealthBar
# 类型化信号
signal health_changed(new_health: int, max_health: int)
# 禁用不需要的_process
func _ready():
set_process(false)
| 反模式 | 问题 | 解决 |
|---|---|---|
| God类 | 职责过多 | 拆分职责 |
| 深继承 | 难理解 | 使用组合 |
| $在循环中 | 性能差 | @onready缓存 |
| 无类型 | 无编译帮助 | 添加类型 |
| 魔法数字 | 含义不清 | 命名常量 |
| Error | Fallback |
|---|---|
| LSP unavailable | Skip automated checks, manual review only |
| File not found | Ask user to provide correct path |
| Too many issues | Focus on blocking/important, offer to review rest later |
| User wants to quit | Summarize what was found so far |
| 检查类型 | 关键问题 |
|---|---|
| 正确性 | 它能工作吗? |
| 质量 | 可读可维护吗? |
| 性能 | 会拖慢游戏吗? |
| Godot | 符合引擎最佳实践? |
as any 抑制类型错误The Godot Engine Specialist is the authority on all Godot-specific patterns, APIs, and optimization techniques. Guides C# (primary) vs GDScript (optional) vs GDExtension decisions, ensures proper use of Godot's node/scene architecture, signals, and resources.
快速恢复项目上下文 — 新会话中读取已有配置,无需重新走 /start 流程。
First-time onboarding — asks where you are, then guides you to the right workflow. No assumptions.
引导式美术圣经编写。创建视觉规范文档,作为所有资产制作的基础。在 /brainstorm 完成后、/map-systems 或 GDD 编写之前运行。
美术与程序协调桥梁。处理资产规格、命名规范、导入管道和美术-程序沟通。
从 GDD、关卡文档或角色档案生成资产规格。包含 AI 图像生成提示词和详细规格。在美术圣经和 GDD 批准后、制作开始前运行。