mcpField
F
factory-review

factory-review

github.com/jwilgerAgent 工作流582 次安装17 周前更新
代码仓库

关于 factory-review

用于 factory-mode 构建阶段的审查 Skill，将审计记录、PR、回顾、指标和待处理升级整理为简洁的人工审查材料。它还会校验配置变更，并要求确认后才应用。

主要功能

  1. 01审计记录构建摘要
  2. 02已合并 PR 摘要
  3. 03回顾中的模式归纳
  4. 04质量趋势报告
  5. 05升级事项背景与审查
  6. 06经确认的配置调整

适用场景

  1. 01审查 factory 构建结果
  2. 02快速查看已合并 PR 与质量门禁
  3. 03识别反复返工的模式
  4. 04决定如何处理待解决的升级事项

安装

复制下面的命令，在终端中运行。

$npx skills add https://github.com/jwilger/agent-skills --skill factory-review

安装后在 Claude Code 中输入/factory-review确认已加载。

详细介绍

Factory Review

价值： 沟通——工厂模式将人的注意力集中在重要决策上。这项技能可确保审查界面清晰、简洁且便于采取行动，让人能够进行有意义的监督，而不必重新阅读每一行代码。

目的

规范工厂模式构建第 3 阶段中的人工审查体验。生成审计跟踪摘要、PR 摘要、回顾综合分析、质量趋势报告和自主性调节界面。目标是在尽可能短的时间内提供高信号审查。

实践

审计跟踪摘要

从 .factory/audit-trail/ 读取信息，并提供简洁的构建摘要：

  • 已完成的切片： 统计数量并列出（按切片 ID 和标题）
  • 返工率： 需要返工周期的切片百分比
  • 质量门失败分布： 按质量门类型统计失败次数（tdd、review、ci、mutation）
  • 待处理的升级事项： 统计数量，并简要说明每项未解决的升级事项

采用仪表盘式摘要格式。先列数字，只有在人要求时才提供详细信息。

PR 摘要

为构建中每个已合并的 PR 生成一段摘要：

  1. 更改内容： 用通俗语言描述功能或修复
  2. 所属切片： 此 PR 实现的垂直切片 ID
  3. 协作搭档： 共同完成此 PR 的两位工程师
  4. 质量门结果： 每项质量门的通过/失败情况（tdd、review、ci、mutation）
  5. 返工次数： 所有质量门通过前经历的返工周期数

每份摘要控制在 3～5 句话。对于典型的构建会话，人工应能在一分钟内浏览完所有 PR 摘要。

回顾综合分析

汇总团队回顾中的发现（存储在 .factory/audit-trail/retrospectives/），并呈现其中的规律：

  • 反复出现的返工原因： 哪些问题引发了最多返工？
  • 协作搭档的成效： 哪些搭档组合产生的返工周期最少？
  • 领域热点： 领域模型的哪些方面引发了最多讨论或修订？
  • 流程阻力： 团队提出了哪些阻碍因素或改进建议？

按主题分组，以项目符号列出。不要加入主观评论——如实报告团队的说法。

质量趋势跟踪

从 .factory/audit-trail/metrics/ 读取信息，并呈现趋势：

指标当前上一次趋势
变异分数%%上升/下降/稳定
返工率%%上升/下降/稳定
每个切片的周期时间时长时长上升/下降/稳定
按类型统计的质量门失败率数量数量上升/下降/稳定

“上一次”指最近一次已完成的构建会话。如果不存在上一次会话，则省略对比列。

调节界面

在审查期间接受对 .factory/config.yaml 的调整。对于每项拟议更改：

  1. 验证： 检查更改是否一致（例如，不能在 conservative 自主性级别启用自动合并；不能禁用其他设置所依赖的质量门）
  2. 说明影响： 修改此设置会如何改变行为？请具体说明。
  3. 应用或拒绝： 如果有效，则应用更改。如果无效，则解释原因并建议最接近的有效替代方案。

绝不静默应用配置更改。写入 .factory/config.yaml 前，务必先征得人工确认。

升级事项审查

完整呈现每项待处理升级事项的上下文：

  1. 触发的质量门： 触发升级事项的质量门
  2. 返工尝试次数： 已尝试的返工周期数
  3. 已尝试的措施： 简要总结每次返工尝试
  4. 当前状态： 当前代码的情况（差异或描述）
  5. 建议： 团队建议的解决方案（如有）

由人工决定：解决、覆盖，或退回以进行更多返工。

执行说明

所有模式下均为建议性要求。摘要和趋势报告属于自行遵守的格式规范。

硬性约束：

  • 不得静默更改配置：[H]

约束

  • “不发表主观评论”：报告团队说了什么，而不是你对团队所说内容的看法。“团队讨论了 X，并决定 Y”是在报告。“团队明智地决定了 Y”是在发表主观评论。“团队讨论了 X，但忽略了关于 Z 的重要要点”是在发表主观评论。如果你有顾虑，应将其单独作为自己的分析提出，并明确标注。
  • 配置验证：“绝不静默应用配置更改”意味着每项配置更改都必须经过用户明确确认，即使用户之前说过“继续配置即可”。每项更改都要单独确认。配置更改不接受一揽子授权。

验证

完成一次工厂审查会话后，请验证：

  • 已提交包含当前构建指标的审计跟踪摘要
  • 每个已合并的 PR 都有摘要（未跳过任何 PR）
  • 回顾综合涵盖了本次会话中的所有团队回顾
  • 质量趋势包含与上次会话的对比（如有）
  • 已完整呈现所有待处理的升级事项及其背景
  • 应用任何配置更改前都已进行验证
  • 人员已确认或否决每项升级事项

如果未满足任何一项标准，请重新检查相关实践。

依赖项

此技能需要 pipeline 技能提供工厂模式基础设施。 它与以下内容集成：

  • **pipeline：**读取 pipeline 编排器在第 2 阶段生成的审计跟踪文件和指标
  • **ensemble-team：**回顾综合会读取团队回顾输出；配对效果数据来自配对历史记录
  • **ci-integration：**门禁失败数据包含来自 ci-integration 技能的 CI_RESULT 数据包

缺少依赖项？请使用以下命令安装：

npx skills add jwilger/agent-skills --skill pipeline

评论

支持 Markdown