所谓无规矩不成方圆,前端时间在团队 code-review 中发现,推荐大家可以大致过一遍,形成自己的编码习惯 原文地址:juejin.cn/post/6850037282419392526 作者:lzg9527
review #会自动推送到我们的 gerrit 服务器上 然后相关负责人去 gerrit 上做 Code review,如果审核通过,则 Merge 到分支,若未通过,则可直接 Abandon 掉或者 Code-Review 这里演示先 Code-Review -1,然后 Code-Review -2,相当于本次提交不通过。 接下来,我们在做一下修改,再次提交(注意:这里我们要使用上一次提交的 change-id,意思是再次提交作为上一次的修改补丁),最后 Code-Review +1,Code-Review +2 审核通过本次提交 (以上都是用我一个人的账号进行 Code-Review,实际应用中,可以事先指定好各审核人。) 当然 gerrit 能做的还有很多,如权限控制,分组等等,这里只介绍了最简单的也是最核心的 Code-Review 功能的简单使用。
: 代码审查技能 - data-analysis: 数据分析技能 [Loaded] code-review ## 审查指南 当你进行代码审查时,请遵循以下步骤... " → 模型看到概览,判断需要 code-review 技能 → 模型调用 skill_load(skill="code-review", include_all_docs=true) → Tool /code-review": "1", "temp:skill:docs:my-agent/code-review": "*" } → 框架将 StateDelta 写入 Session 从 State 发现 code-review 已加载 c. 从 Repository 获取 code-review 的完整 Body + 所有文档 d. 注入到 Prompt: "[Loaded] code-review\n\n## 审查指南\n...\n\n[Doc] guide.md\n...
在gerrit配置中有两个计数器,Code-Review和Verified,一般情况项目会分别设置为+2,+1使代码审核通过。 Code-Review通常需要开发人员人工审核,而Verified设置项可以与测试工具结合,通过测试的指标则设置Verified +1,不通过则-1,如此可以在代码提交阶段提前介入测试,更早的发现问题。 访问权限在change范围内有效 Project Owners 访问权限在project范围内有效 Registered Users 所有在gerrit页面上成功登录的用户都属于该群组,通常被赋予Code-Review
方式 2:手动创建文件 格式示例: --- name: code-review description: 代码审查专家,检查代码质量和安全性 tools: Read, Grep, Glob, Bash 模型根据 description 自动识别并调用 手动触发: 输入 /a/agent-name 显式调用 串联调用: 在一条指令中顺序调度多个智能体 示例: 帮我审查这个接口的实现 → 自动识别并调用 code-review 智能体 /code-review 先使用 design subagent 完成系统设计,再使用 code-review subagent 完成代码 review 位置与作用域 位置 路径 作用域 用户级 触发: /c/code-review 能力: 对指定范围的代码执行结构化审查,输出质量、安全、性能等维度的评审意见 适用: 提交前代码审查、针对特定模块的质量把关、团队代码评审 三者协同与选择指南 分层协作管道 代理名 并行执行,上下文隔离 Plan Agent 内置 /p/plan 先规划再执行 Browser Agent 内置 /b/browser 网页自动化 Code Review Agent 内置 /c/code-review
文件,由两部分组成: Frontmatter(头部配置)—— 告诉 Claude 这个技能的元信息 Body(指令正文)—— 告诉 Claude 具体该做什么 下面是一个真实的例子: --- name:code-review 第三步:使用你的 Skill 保存文件后,在 Claude Code 中直接输入: /code-review Claude 就会按照你定义的审查清单,自动对代码进行全方位检查! 四、进阶技巧:打造专业级 Skills 技巧 1:多文件组织 当 Skill 变复杂时,拆分成多个文件: code-review/ ├── SKILL.md ← 主入口(保持精简
postive integer gitConfig: - #git config 1 - #git config 2 vote: on_success: Verified: 1 Code-Review : 2 on_failure: Verified: -1 Code-Review: -2 env: - #env1=foo - #env2=bar matrix: build
消息 https://sentry-dev.hacker-linner.com/commit-messages/ 代码审查 https://sentry-dev.hacker-linner.com/code-review
Skill的文件结构:展开代码语言:TXTAI代码解释code-review/├──SKILL.md←核心指令(必须)├──checklist.md←审查清单└──scripts/└──scan.sh← 只有你能手动触发(适合"发微博"这类有副作用的命令)user-invocable:false—只有Claude能自动调用(适合后台静默执行)一行安装:展开代码语言:BashAI代码解释cp-r03-skills/code-review /CLAUDE.md#4.安装代码审查技能cp-r03-skills/code-review~/.claude/skills/#5.在ClaudeCode里运行测试#/optimize#/commit想生成离线
**静态代码审查**:使用现有 code-review 技能检查代码规范 2.**运行时验证**:使用 Aspire MCP 工具检查应用日志和资源状态 3. **调用代码审查技能** - 参考现有的 [code-review](.. /code-review/SKILL.md) 技能 - 对修改和新文件执行全面审查 - 重点关注: - 严重级别:安全、数据库、异步编程 - 重要级别:多语言、DTO、控制器、服务类 - 文档更新及时 同步更新了 startup-framework.mdc 规范文档和 code-review/SKILL.md 技能文档,展示了简化配置方式并标注为"推荐"。 静态代码审查结果 ✅ 审查范围 修改的文件(14个): • .cursor/rules/startup-framework.mdc - 启动框架规范更新 • .cursor/skills/code-review
/skills/code-review"],#这一行...)skills/code-review/SKILL.md的内容:展开代码语言:MarkdownAI代码解释#CodeReviewStandardsWhenreviewingcode
有两个概念需要理解一个是Verified,另一个是Code-Review。Verifying代表着对代码进行实际编译、单元测试等,主要是自动化程序。 Code-Review是一个人来查看你的代码,确保这段代码的逻辑正确,编码风格良好等。
示例:创建代码评审Skills(.claude/skills/code-review/SKILL.md)展开代码语言:MarkdownAI代码解释#代码评审技能(/code-review)##功能描述自动评审指定文件的代码规范 ##调用方式/code-review<文件路径>##评审标准1.编码规范:符合项目ESLint规则,无语法错误2.逻辑检查:无空指针、未定义变量、死循环等潜在bug3.性能优化:避免冗余代码、无效查询、
论文把一次进化拆成七个阶段:Locate、Plan、Plan-Review、Implement、Code-Review、Task-Evaluate、Verdict。 最外层先有 pre-loop baseline,用来确定当前失败表现;然后进入 iteration loop,每一轮都执行固定的七阶段 pipeline;在 Plan-Review 和 Code-Review 不急着给修复方案;Plan 负责提出修改计划,明确要改哪些文件、增加什么逻辑、哪些部分不该动;Plan-Review 是第一个质量门,用来判断方案是否偏离架构或范围过窄;Implement 才真正修改代码;Code-Review 因此,MOSS 在 Code-Review 之后还会进行 runtime 验证。host-daemon 会构建候选镜像,然后启动多个临时 trial workers。
本质: 它是知识与业务流程的加载机制(如 code-review, data-analysis),让 Agent 保持轻量化核心的同时具备无限的专业扩展性。
完整字段说明如下,搭配示例代码:展开代码语言:YAMLAI代码解释---#1.核心基础配置(必选/推荐)name:skill-name#技能名,即斜杠命令名(仅允许小写、数字、连字符,如code-review 如禁用所有技能、仅允许指定技能、拒绝高危技能等,操作示例如下:展开代码语言:TXTAI代码解释#1.禁用所有技能(全局禁止调用任何技能)>/permissionsdenySkill#2.仅允许调用指定技能(如code-review 、pr-summary)>/permissionsallowSkill(code-review)>/permissionsallowSkill(pr-summary:*)#3.拒绝调用指定技能(如deploy-prod 7.1模板1:代码审查技能(项目级,团队共享)路径:.claude/skills/code-review/SKILL.md展开代码语言:YAMLAI代码解释---name:code-reviewdescription
第一种:本地命令 /code-review(v2.1.147 之前叫 /simplify) 在本地 Claude Code 会话里,对当前 diff 运行 /code-review 命令。 这三种用法不互斥,我们现在的实践是:本地 /code-review 做提交前自检,GitHub App 在 PR 开后自动 review 一次。两道门,覆盖不同粒度的问题。
例如在多 Agent 协作场景中,code-reviewer Agent 能用 code-review Skill,而 doc-writer Agent 不应该看到 code-review。 下面以同一个 code-review Skill 的 description 三个版本演化为例,说明哪些写法会显著影响命中行为。 tree ~/.claude/skills/code-review 看一遍目录结构,确保没有 Claude Code 私有文件(如 .claude-cache、.cm 等)混入。 例如同时存在 code-review 和 code-quality-check 两个 Skill,描述都涉及"评审代码",Agent 在用户说"看看这段代码"时无法稳定选择。工程对策有两种。 例如 code-review 和 code-quality-check 合并成一个,在正文中区分"如果是 commit 前评审走流程 A,如果是 PR review 走流程 B"。
3.2 commands:Slash 命令一键触发流程 commands 里已经把高频工作流封装成了: /tdd /plan /e2e /code-review /build-fix /refactor-clean 先装插件,直接用 /plan /tdd /code-review 跑一遍流程。 第二步,只保留你真正用得上的 agents 和 rules。
Superpowers:技能触发式 brainstorming → writing-plans → executing-plans → TDD → code-review ↓ 比如写代码前自动激活 TDD 技能,写完代码自动激活 code-review 技能。 Superpowers 的哲学是:流程胜过猜测。 你适合哪种范式?