Skip to content
This repository was archived by the owner on Sep 28, 2026. It is now read-only.

practices:Matt 四条实践复制入库——grilling、domain-modeling、tdd、code-review 改中文与两节骨架,评测齐(#118) - #125

Merged
Eridanus117 merged 2 commits into
mainfrom
Eridanus117/as118-practices
Sep 19, 2026
Merged

Eridanus117 merged 2 commits into
mainfrom
Eridanus117/as118-practices

Conversation

@Eridanus117

Copy link
Copy Markdown
Owner

做了什么

  • 新增插件 plugins/practices(0.1.0):四条实践 skill grilling、domain-modeling、tdd、code-review,正文按 MIT 从 mattpocock/skills(revision 6acc160)复制入库,改成中文、两节骨架(什么时候用、步骤带完成判据)、正面陈述,术语按 plugins/CONTEXT.md;每条头部注来源仓、revision、许可与「与原版的差别」。随附文件进 references/(grilling 的一题形状;domain-modeling 的 CONTEXT/ADR 格式;tdd 的好坏测试与 mock 边界;code-review 的 Fowler 坏味道基线)。
  • plugins/practices/LICENSE-mattpocock:vendor 的 MIT 原文;plugins/practices/SOURCE:来源仓、版本、revision(与 vendor/mattpocock/SOURCE.json 一致)、改写范围、跟上游的做法。
  • 十个 claude plugin eval 用例(grilling 2、domain-modeling 3、tdd 3、code-review 2,各一正一反),evals/README.md 有用例构成、基线观察、实跑记录;基线、门评测与重跑的 aggregate-result.json 都洗过用户名后提交。
  • 装配:.claude-plugin/marketplace.json 加 practices;profiles/daily 的四条从 vendor/mattpocock/skills/<名> 改指 plugins/practices/skills/<名>;profiles/all 改指 grilling 并补上另外三条(all 是仓内插件的全集,这三条现在在仓内)。plugins/skill-imports.toml 没动(它是 .cap 退役后的历史声明,skill-evaluation.test.ts 断言它仍指 vendor)。
  • 顺手:skill-authoring 升 0.1.2,scrub-eval-results.ts 补两种此前洗不掉的写法(执行记录当字符串再嵌进 JSON 的四反斜杠、Claude Code 的目录 slug C--Users-<名>),分隔符与用户名片段抽成常量;ADR-0007「确认」节补实际结果;plugins/README.md 插件清单加 practices。

判两件事(按 skill-authoring):四条都不是路由 skill,各只讲自己那件事的做法,不改别的 skill 什么时候被用;都是模型可拿的(model-invoked),frontmatter 只有 name 与 description,由 flow/implement/grill-with-docs 这类规程点名。flow 现在点名的 tdd、code-review 就是这四个名字,manifest 改指之后不需要改字。

为什么

Closes #118。ADR-0007(负责人 2026-09-18 定:允许复制、抄别人)选了「复制入库」而不是引用 vendor:正文要能改中文、贴本仓写法,四条不必破例进工作方法地图;代价是上游更新要按 SOURCE 里的 revision 手动 diff。ADR-0006 定了术语只用有出处的业界词、实践与规程分两层,plugins/practices 是实践层的起点(agent-system#115 第三批)。方案对齐记录在 desk#152。

与原版的实质差别都写在各条头部;三处最要紧:grilling 原版一轮问整个前沿,本工作区常驻规则定一轮只问一题,改成一轮一题;code-review 原版只允许并行子代理、要求 git rev-parse 必须解析,这里允许 git 不可用时用主人贴来的 diff、派不了子代理时顺序做两轴并写明未隔离;domain-modeling 的 CONTEXT/ADR 格式先按仓里既有的(本仓词条带出处、ADR 用 MADR),没有才用原版默认。

怎么验证的

  • node plugins/tests/skills.test.ts:退出 0,PASS: 13 个 Plugin、27 个 Skill,目录页一致,词汇表 26 条带英文与出处。

  • claude plugin validate plugins/practices、claude plugin validate plugins/skill-authoring:都 Validation passed。

  • node plugins/tests/skill-evaluation.test.ts:3 过 0 败。python contracts/tools/validate.py:validation passed: 3 schemas, 6 examples, 2 Issue Forms, 12 execution-loop tests。python -m scripts.entry_sync check --scope repository:三项 OK。python scripts/test_federated_entry.py:通过。

  • 公共面门禁 bun packages/mounts/src/cli.ts assess-public-tree --repo . --rev HEAD --policy .github/public-tree-policy.json:status: allowed (scanned 1546)(第一个 commit 上跑的;第二个 commit 只改文档与脚本)。

  • 评测(默认模型 claude-opus-5,裁判 sonnet,每用例 3 次两组,--max-cost-usd 20 每条 skill;实际 grilling 5.07、domain-modeling 9.47、tdd 9.27、code-review 5.80 美元)。基线(SKILL.md 占位,results/2026-09-19-baseline/):对照组在 tdd「先定接缝」「认出耦合样板」、code-review「两轴分开」、domain-modeling「指出冲突后问一句就停」上都是 0,观察写在 evals/README.md。门评测(results/2026-09-19-gate/<skill>/ 加按用例重跑 -rerun、-rerun2、-rerun3):

    用例 有 skill 组 对照组 差值
    grilling-plan-one-question 0.78 0.56 +0.22
    grilling-facts-self-served 1.0 0 +1.0
    domain-modeling-glossary-conflict 1.0 1.0 0
    domain-modeling-update-inline 1.0 1.0 0
    domain-modeling-just-reading 1.0 1.0 0
    tdd-seams-first 1.0 0.11 +0.89
    tdd-first-slice 1.0 1.0 0
    tdd-coupled-test 1.0 0 +1.0
    code-review-two-axes 1.0 0.33 +0.67
    code-review-missing-fixed-point 1.0 1.0 0

    平均差值 +0.38;五条两组同分的作回归守卫(其中两条是 grader 放宽到只判可观测行为后对照组才过的,放宽理由在 README「迭代过程」)。门上待裁的一条:grilling-plan-one-question 0.78 没到 1.0 硬线——两轮重跑各有一次三票 FAIL,那两条回复都只递了一题、带推荐、问完就停,裁判把决定树里带问号的子项数成了多问;grader 已两次改写仍没拦住,再跑只是换随机数。是否按硬线再跑由负责人定(与 flow-steps:第 4、6、7 段的四条步——建工作树加基线、验证取证、接审查意见、收尾;flow 升 0.1.1 改点名(#113) #114 里 worktree-baseline-open 0.83 同一处理)。

  • 结果 JSON 都跑过 scrub-eval-results.ts,grep Morni 零命中;脚本对已提交结果幂等(再跑 0 文件改动)。

  • 两轴审查(practices:code-review 的做法,两个子代理并行)已接:Standards 轴违例 2 条——「固定点」「竖切/横切」补出处;三条多面 llm grader 按 flow-steps:第 4、6、7 段的四条步——建工作树加基线、验证取证、接审查意见、收尾;flow 升 0.1.1 改点名(#113) #114 先例作判断题留着并写进 README。Spec 轴 3 条——SOURCE 声称各头部列了差别但只有 grilling 列了(三条头部各补一句);code-review 两处为沙箱放宽的规则补声明并在降级时写明未隔离;profiles/all 新增三条在上面说明。判断题采纳:code-review「为什么分两轴」并入来源段;tdd 游离句并进第 1 步;scrub 脚本抽常量。审查后正文的四处改动没有重跑评测(都是头部说明、句子搬家和一句降级提示)。

  • 沙箱里 Claude Code 自带一条 code-review,本插件同名 skill 在沙箱里叫 practices:code-review,两个 code-review 用例的提示词因此点名 practices:code-review。装到本机后走 junction 名字是 code-review,与自带那条的先后要在上线清单里查(此前 vendor 的同名 skill 已是这个处境)。

怎么回退

revert 本 PR:plugins/practices 整目录、marketplace 条目、两份 manifest 的改指与新增、skill-authoring 的脚本改动与版本号、ADR-0007 确认节、plugins/README.md 一起回去;profiles/daily 回到指 vendor/mattpocock/skills/<名>。本 PR 不改 agent-config,本机 junction 由上线清单驱动,revert 后无需清理外部状态。

🤖 Generated with Claude Code

zaurakworks and others added 2 commits September 19, 2026 02:40
…与两节骨架,十个评测用例加门评测(#118)

- 新增插件 plugins/practices 0.1.0:四条实践 skill,正文按 MIT 从 mattpocock/skills revision 6acc160 复制并改写,LICENSE-mattpocock 与 SOURCE 记来源
- 评测:十个 claude plugin eval 用例,基线、门评测与重跑结果已洗过提交,README 记用例构成、基线观察、实跑记录
- 装配:marketplace 加 practices;daily/all manifest 的四条从 vendor 改指 plugins/practices
- skill-authoring 0.1.2:scrub-eval-results.ts 补两种路径写法(双重转义、目录 slug)
- ADR-0007 确认节补实际结果;plugins/README 插件清单加 practices

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…符常量(#118)

- Standards 轴违例:「固定点」「竖切/横切」补英文原词与出处;README 旧名「步」改「活动」
- Spec 轴:SOURCE 声称各头部列差别但只有 grilling 列了——domain-modeling、tdd、code-review 头部各补一句;code-review 第 4 步派不了子代理时在报告开头写明「两轴未隔离」
- 判断题采纳:code-review「为什么分两轴」并入来源段一句;tdd 探索前读 CONTEXT.md 并进第 1 步;scrub-eval-results.ts 分隔符与用户名片段抽成常量
- 判断题保留并写进 evals/README:多面 llm grader 与 axes-separate(同 flow-steps 先例)

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

票4 practices plugin 起步:复制 Matt 四条实践入库

2 participants