@heihei0299/matt-skills 2.1.13 → 3.0.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +3 -2
- package/bin/cli.js +40 -92
- package/package.json +1 -1
- package/template/.opencode/CONTEXT.md +7 -7
- package/template/.pi/CONTEXT.md +7 -7
- package/template/.agents/skills/ask-matt/PHASE-BOUNDARIES.md +0 -55
- package/template/.agents/skills/ask-matt/SKILL.md +0 -90
- package/template/.agents/skills/ask-matt/agents/openai.yaml +0 -5
- package/template/.agents/skills/code-review/SKILL.md +0 -87
- package/template/.agents/skills/code-review/agents/openai.yaml +0 -3
- package/template/.agents/skills/codebase-design/DEEPENING.md +0 -37
- package/template/.agents/skills/codebase-design/DESIGN-IT-TWICE.md +0 -44
- package/template/.agents/skills/codebase-design/SKILL.md +0 -114
- package/template/.agents/skills/codebase-design/agents/openai.yaml +0 -3
- package/template/.agents/skills/diagnose-fix/SKILL.md +0 -43
- package/template/.agents/skills/diagnose-fix/agents/openai.yaml +0 -5
- package/template/.agents/skills/diagnose-fix/references/anti-patterns.md +0 -21
- package/template/.agents/skills/diagnosing-bugs/SKILL.md +0 -138
- package/template/.agents/skills/diagnosing-bugs/agents/openai.yaml +0 -3
- package/template/.agents/skills/diagnosing-bugs/scripts/hitl-loop.template.sh +0 -44
- package/template/.agents/skills/domain-modeling/ADR-FORMAT.md +0 -47
- package/template/.agents/skills/domain-modeling/CONTEXT-FORMAT.md +0 -60
- package/template/.agents/skills/domain-modeling/SKILL.md +0 -74
- package/template/.agents/skills/domain-modeling/agents/openai.yaml +0 -3
- package/template/.agents/skills/grill-me/SKILL.md +0 -7
- package/template/.agents/skills/grill-me/agents/openai.yaml +0 -5
- package/template/.agents/skills/grill-to-spec/SKILL.md +0 -55
- package/template/.agents/skills/grill-to-spec/agents/openai.yaml +0 -5
- package/template/.agents/skills/grill-to-spec/references/rules.md +0 -47
- package/template/.agents/skills/grill-with-docs/SKILL.md +0 -7
- package/template/.agents/skills/grill-with-docs/agents/openai.yaml +0 -5
- package/template/.agents/skills/grilling/SKILL.md +0 -28
- package/template/.agents/skills/grilling/agents/openai.yaml +0 -3
- package/template/.agents/skills/handoff/SKILL.md +0 -16
- package/template/.agents/skills/handoff/agents/openai.yaml +0 -5
- package/template/.agents/skills/implement/SKILL.md +0 -15
- package/template/.agents/skills/implement/agents/openai.yaml +0 -5
- package/template/.agents/skills/implement-review-loop/SKILL.md +0 -34
- package/template/.agents/skills/implement-review-loop/agents/openai.yaml +0 -5
- package/template/.agents/skills/improve-codebase-architecture/HTML-REPORT.md +0 -123
- package/template/.agents/skills/improve-codebase-architecture/SKILL.md +0 -71
- package/template/.agents/skills/improve-codebase-architecture/agents/openai.yaml +0 -5
- package/template/.agents/skills/instance-test/SKILL.md +0 -70
- package/template/.agents/skills/instance-test/agents/openai.yaml +0 -5
- package/template/.agents/skills/instance-test/references/instances.md +0 -75
- package/template/.agents/skills/prototype/LOGIC.md +0 -67
- package/template/.agents/skills/prototype/SKILL.md +0 -26
- package/template/.agents/skills/prototype/UI.md +0 -112
- package/template/.agents/skills/prototype/agents/openai.yaml +0 -3
- package/template/.agents/skills/research/SKILL.md +0 -12
- package/template/.agents/skills/research/agents/openai.yaml +0 -3
- package/template/.agents/skills/resolving-merge-conflicts/SKILL.md +0 -14
- package/template/.agents/skills/resolving-merge-conflicts/agents/openai.yaml +0 -3
- package/template/.agents/skills/scaffold-functional-test/SKILL.md +0 -64
- package/template/.agents/skills/scaffold-functional-test/agents/openai.yaml +0 -5
- package/template/.agents/skills/scaffold-functional-test/references/schema.md +0 -80
- package/template/.agents/skills/setup-matt-pocock-skills/SKILL.md +0 -116
- package/template/.agents/skills/setup-matt-pocock-skills/agents/openai.yaml +0 -5
- package/template/.agents/skills/setup-matt-pocock-skills/domain.md +0 -51
- package/template/.agents/skills/setup-matt-pocock-skills/issue-tracker-github.md +0 -45
- package/template/.agents/skills/setup-matt-pocock-skills/issue-tracker-gitlab.md +0 -46
- package/template/.agents/skills/setup-matt-pocock-skills/issue-tracker-local.md +0 -30
- package/template/.agents/skills/setup-matt-pocock-skills/triage-labels.md +0 -15
- package/template/.agents/skills/show-me/SKILL.md +0 -28
- package/template/.agents/skills/tdd/SKILL.md +0 -38
- package/template/.agents/skills/tdd/agents/openai.yaml +0 -3
- package/template/.agents/skills/tdd/mocking.md +0 -59
- package/template/.agents/skills/tdd/tests.md +0 -77
- package/template/.agents/skills/tdd-implement/SKILL.md +0 -81
- package/template/.agents/skills/tdd-implement/agents/openai.yaml +0 -5
- package/template/.agents/skills/tdd-implement/references/finalize.md +0 -16
- package/template/.agents/skills/tdd-implement/references/orchestration.md +0 -54
- package/template/.agents/skills/tdd-implement/references/verify.md +0 -15
- package/template/.agents/skills/teach/GLOSSARY-FORMAT.md +0 -35
- package/template/.agents/skills/teach/LEARNING-RECORD-FORMAT.md +0 -46
- package/template/.agents/skills/teach/MISSION-FORMAT.md +0 -31
- package/template/.agents/skills/teach/RESOURCES-FORMAT.md +0 -32
- package/template/.agents/skills/teach/SKILL.md +0 -140
- package/template/.agents/skills/teach/agents/openai.yaml +0 -5
- package/template/.agents/skills/to-questionnaire/SKILL.md +0 -54
- package/template/.agents/skills/to-questionnaire/agents/openai.yaml +0 -5
- package/template/.agents/skills/to-spec/SKILL.md +0 -75
- package/template/.agents/skills/to-spec/agents/openai.yaml +0 -5
- package/template/.agents/skills/to-tickets/SKILL.md +0 -105
- package/template/.agents/skills/to-tickets/agents/openai.yaml +0 -5
- package/template/.agents/skills/triage/AGENT-BRIEF.md +0 -207
- package/template/.agents/skills/triage/OUT-OF-SCOPE.md +0 -105
- package/template/.agents/skills/triage/SKILL.md +0 -112
- package/template/.agents/skills/triage/agents/openai.yaml +0 -5
- package/template/.agents/skills/wait-what/SKILL.md +0 -7
- package/template/.agents/skills/wait-what/agents/openai.yaml +0 -5
- package/template/.agents/skills/wayfinder/SKILL.md +0 -128
- package/template/.agents/skills/wayfinder/agents/openai.yaml +0 -5
- package/template/.agents/skills/wizard/SKILL.md +0 -44
- package/template/.agents/skills/wizard/agents/openai.yaml +0 -3
- package/template/.agents/skills/wizard/template.sh +0 -204
- package/template/.agents/skills/writing-for-agents/SKILL-MECHANICS.md +0 -22
- package/template/.agents/skills/writing-for-agents/SKILL.md +0 -81
- package/template/.agents/skills/writing-for-agents/agents/openai.yaml +0 -3
|
@@ -1,55 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: grill-to-spec
|
|
3
|
-
description: "Router:编排 grill-with-docs → to-spec,把模糊想法打磨成可执行 Spec。Use when the user asks to grill/design/polish an idea into a spec——只产出领域文档与 spec,不写代码。"
|
|
4
|
-
disable-model-invocation: true
|
|
5
|
-
---
|
|
6
|
-
|
|
7
|
-
# Grill to Spec
|
|
8
|
-
|
|
9
|
-
只做两个上游 skill 的编排:先把想法打磨成共识,再把共识发布成 spec。本 skill 不写代码、不修改源码或测试、不自动 commit。
|
|
10
|
-
|
|
11
|
-
## 流程
|
|
12
|
-
|
|
13
|
-
### 0. 预检
|
|
14
|
-
|
|
15
|
-
- 只读确认 issue tracker、目标上下文、feature slug 和已有 spec/issue 状态。
|
|
16
|
-
- tracker 未配置、slug 不明确或目标路径不可写时,在任何 ADR/spec/issue 写入前报告阻塞。
|
|
17
|
-
- 已有同一 feature 的产物先读取并比较;相同共识不重复发布,设计变化进入变更流程。
|
|
18
|
-
|
|
19
|
-
### ① 形成共识
|
|
20
|
-
|
|
21
|
-
调用 [`grill-with-docs`](.agents/skills/grill-with-docs/SKILL.md),由 `grilling` 与 `domain-modeling` 完成采访、术语和设计决策。
|
|
22
|
-
|
|
23
|
-
- glossary 按上游规则 inline 更新;
|
|
24
|
-
- 只有同时满足 ADR 条件时才提出 ADR,但 ADR 等最终决策清单确认后再写入;
|
|
25
|
-
- 在本阶段让 [`to-spec`](.agents/skills/to-spec/SKILL.md) 做代码理解和 seam 分析;seam 是共识的一部分,不单独制造发布确认;
|
|
26
|
-
- 形成精简决策清单:目标、范围、关键选择、seam、未纳入范围和待验证假设;只展示清单,不展示任何 ADR/spec/issue 正文或草稿;
|
|
27
|
-
- 用户确认决策清单后,立即进入阶段 ②。
|
|
28
|
-
|
|
29
|
-
出口:决策清单已确认,glossary 已按需更新;尚未确认的 ADR/spec/issue 不写入。
|
|
30
|
-
|
|
31
|
-
### ② 发布 spec
|
|
32
|
-
|
|
33
|
-
将已确认的决策清单和 seam 分析交给 `to-spec`;此处只综合、写入和发布,不重新采访或再次确认 seam。
|
|
34
|
-
|
|
35
|
-
- 按 ADR → spec → issue 的顺序执行;issue 发布成功后设置 `ready-for-agent`;格式细则只读取 [`references/rules.md`](references/rules.md)。
|
|
36
|
-
- 相同 feature 复用已有产物,设计变化按 tracker 的更新语义保留历史;
|
|
37
|
-
- 任一步失败都保留已成功写入的内容,记录状态和失败点,重跑时从第一个未完成出口继续;不回滚、不重复发布。
|
|
38
|
-
|
|
39
|
-
出口:ADR/spec/issue 已写入或发布,只报告路径或标识、状态和未纳入范围,不复制正文。
|
|
40
|
-
|
|
41
|
-
## 回合连续性
|
|
42
|
-
|
|
43
|
-
本 skill 是 Long-Horizon Skill。预检、阶段 ①、代码理解、决策清单确认后的阶段 ② 在同一任务链连续执行;进度汇报和阶段切换不是回合终点。仅在需要设计确认、决策清单确认、外部阻塞、用户主动停止或整个 skill 出口时暂停;确认完成后不要求用户额外回复“继续”。
|
|
44
|
-
|
|
45
|
-
## 本 skill 独有门禁
|
|
46
|
-
|
|
47
|
-
- 文档正文永不展示;用户只确认精简决策清单,发布后只接收元数据报告;
|
|
48
|
-
- ADR/spec/issue 不设置额外草稿确认;
|
|
49
|
-
- 不写代码、不修改源码或测试、不自动 commit;实现 tickets 交给 `to-tickets`。
|
|
50
|
-
|
|
51
|
-
## 异常
|
|
52
|
-
|
|
53
|
-
- 用户放弃或没有可形成 spec 的主题时终止;
|
|
54
|
-
- issue tracker 未配置时报告配置阻塞,不绕过发布;
|
|
55
|
-
- 用户改变已确认设计时回到阶段 ①,按变更语义保留历史,不静默覆盖。
|
|
@@ -1,47 +0,0 @@
|
|
|
1
|
-
# 守则:Grill-to-Spec 本地增量规则
|
|
2
|
-
|
|
3
|
-
本文件只保存 `grill-to-spec` 相对上游 `grill-with-docs` / `to-spec` 的**增量约束**。Spec 的章节、User Story 形状、Implementation Decisions 等格式全部以 `to-spec` 为唯一事实源,本文件不复制上游模板。
|
|
4
|
-
|
|
5
|
-
## 预检与状态规则
|
|
6
|
-
|
|
7
|
-
- 写入前只读确认 issue tracker、目标上下文、feature slug、已有 spec/issue 和当前状态;tracker 未配置或目标不可写时不写 ADR/spec/issue。
|
|
8
|
-
- feature slug 是 spec 路径和 issue 幂等键;候选不明确或冲突时才询问。
|
|
9
|
-
- 记录共识、ADR、spec、issue、`ready-for-agent` 出口;重跑从第一个未完成出口继续,不重复已成功动作。
|
|
10
|
-
- 已有相同共识的 feature 只报告已有路径或标识;设计变化走变更流程,不创建重复 issue。
|
|
11
|
-
|
|
12
|
-
## Glossary 增量规则
|
|
13
|
-
|
|
14
|
-
- 懒创建:首个术语解析时才建 `CONTEXT.md`;多上下文时先确认归属,归属不清则询问。
|
|
15
|
-
- 只收本上下文特有术语;定义 WHAT 非 HOW,避免把 glossary 变成实现草稿。
|
|
16
|
-
- glossary 可按上游流程 inline 更新,不额外增加确认轮次;设计变化时按历史保留规则修正。
|
|
17
|
-
|
|
18
|
-
## ADR 增量规则
|
|
19
|
-
|
|
20
|
-
- 只有同时满足“难逆转 / 无上下文费解 / 存在真实权衡”时才提议 ADR。
|
|
21
|
-
- ADR 不像 glossary 一样静默 inline 更新;等最终决策清单确认后直接落盘,不展示正文,不增加独立确认轮次。
|
|
22
|
-
- 用户改变决策时不静默覆盖旧 ADR;按项目 ADR 规则追加、废弃或标记 superseded。
|
|
23
|
-
|
|
24
|
-
## Spec 增量规则
|
|
25
|
-
|
|
26
|
-
- Spec 的结构与字段全部委托 `to-spec`,本文件不维护第二份模板。
|
|
27
|
-
- 代码理解和 seam 分析属于共识阶段;seam 纳入最终决策清单,不在发布阶段再次询问。
|
|
28
|
-
- 决策清单确认后直接写入 spec;不展示 spec 正文或草稿。
|
|
29
|
-
- 全文沿用已经确认的 glossary 词汇,并尊重所触区域既有 ADR。
|
|
30
|
-
|
|
31
|
-
## Issue 增量规则
|
|
32
|
-
|
|
33
|
-
- 按已配置的 issue tracker 直接发布唯一 spec issue;implementation tickets 交给 `to-tickets`。
|
|
34
|
-
- issue 正文不在对话中展示;发布成功后设置 `ready-for-agent`。
|
|
35
|
-
- issue 创建成功但 label/status 更新失败时保留 issue,报告标识和失败点,不删除、不误报为 ready。
|
|
36
|
-
- 设计变化委托 tracker 的原生更新语义:本地追加变更记录,远端按 body/comment/关联关系更新;不另造版本模板。
|
|
37
|
-
|
|
38
|
-
## 失败与 Git 边界
|
|
39
|
-
|
|
40
|
-
- 任一步失败都保留已成功写入的内容并报告部分状态;不做跨文件或跨 tracker 回滚,不盲目重试。
|
|
41
|
-
- 本 skill 不自动创建 Git commit;发布文档与 Git 提交是两个独立出口。
|
|
42
|
-
|
|
43
|
-
## 反模式
|
|
44
|
-
|
|
45
|
-
- 不复制 `to-spec` 的章节清单、User Story 模板或 Implementation Decisions 细则。
|
|
46
|
-
- 不产出 Glossary / ADR / Spec 之外的额外设计文件。
|
|
47
|
-
- 不把本文件当逐条朗读的对话脚本;它只约束本地增量行为。
|
|
@@ -1,28 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: grilling
|
|
3
|
-
description: Grill the user relentlessly about a plan, decision, or idea. Use when the user wants to stress-test their thinking, or uses any 'grill' trigger phrases.
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
Interview the user relentlessly until you reach a shared understanding. Map this as a **design tree**: every decision branches into the decisions that hang off it.
|
|
7
|
-
|
|
8
|
-
Work the tree in **rounds**. The **frontier** is every decision whose prerequisites are already settled: the questions you can ask _now_ without guessing at answers you haven't heard yet. Ask the whole frontier in one round: number each question and give your recommended answer. Then wait for the user's answers before the next round.
|
|
9
|
-
|
|
10
|
-
Format a round like so:
|
|
11
|
-
|
|
12
|
-
```
|
|
13
|
-
❓ **Q1** - **<question title>**: <question body, might be multiple paragraphs, including multiple choices>
|
|
14
|
-
|
|
15
|
-
➡️ <your recommended answer>
|
|
16
|
-
|
|
17
|
-
---
|
|
18
|
-
|
|
19
|
-
❓ **Q2** - **<question title>**: <question body, might be multiple paragraphs, including multiple choices>
|
|
20
|
-
|
|
21
|
-
➡️ <your recommended answer>
|
|
22
|
-
```
|
|
23
|
-
|
|
24
|
-
Each round the user answers reshapes the tree: settled decisions push the frontier outward and unblock questions that depended on them. Recompute the frontier and ask the next round. A question whose answer depends on another question still open in this round belongs to a _later_ round, not this one.
|
|
25
|
-
|
|
26
|
-
Finding _facts_ is your job, never the user's. When a frontier question needs a fact from the environment (filesystem, tools, etc.), dispatch a sub-agent to find it; don't ask the user for anything you could look up yourself. Don't block on it: a running exploration is an unsettled prerequisite, so only the questions downstream of it wait for the sub-agent to report; ask the rest of the frontier now. The _decisions_ are the user's: put each to them and wait.
|
|
27
|
-
|
|
28
|
-
The session is done when the frontier is empty: every branch of the design tree visited, nothing left silently assumed. Do not act on it until the user confirms you have reached a shared understanding.
|
|
@@ -1,16 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: handoff
|
|
3
|
-
description: Compact the current conversation into a handoff document for another agent to pick up.
|
|
4
|
-
argument-hint: "What will the next session be used for?"
|
|
5
|
-
disable-model-invocation: true
|
|
6
|
-
---
|
|
7
|
-
|
|
8
|
-
Write a handoff document summarising the current conversation so a fresh agent can continue the work. Save to the temporary directory of the user's OS - not the current workspace.
|
|
9
|
-
|
|
10
|
-
Include a "suggested skills" section in the document, naming which skills the next agent should call the Skill tool for.
|
|
11
|
-
|
|
12
|
-
Do not duplicate content already captured in other artifacts (specs, plans, ADRs, issues, commits, diffs). Reference them by path or URL instead.
|
|
13
|
-
|
|
14
|
-
Redact any sensitive information, such as API keys, passwords, or personally identifiable information.
|
|
15
|
-
|
|
16
|
-
If the user passed arguments, treat them as a description of what the next session will focus on and tailor the doc accordingly.
|
|
@@ -1,15 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: implement
|
|
3
|
-
description: "Implement a piece of work based on a spec or set of tickets."
|
|
4
|
-
disable-model-invocation: true
|
|
5
|
-
---
|
|
6
|
-
|
|
7
|
-
Implement the work described by the user in the spec or tickets.
|
|
8
|
-
|
|
9
|
-
Use /tdd where possible, at pre-agreed seams.
|
|
10
|
-
|
|
11
|
-
Run typechecking regularly, single test files regularly, and the full test suite once at the end.
|
|
12
|
-
|
|
13
|
-
Once done, use /code-review to review the work.
|
|
14
|
-
|
|
15
|
-
Commit your work to the current branch.
|
|
@@ -1,34 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: implement-review-loop
|
|
3
|
-
description: "Implement changes with one executor model and one independent reviewer model, iterating on only the changed surface until no actionable issues remain."
|
|
4
|
-
disable-model-invocation: true
|
|
5
|
-
---
|
|
6
|
-
|
|
7
|
-
Use two distinct roles:
|
|
8
|
-
|
|
9
|
-
- **Executor**: implements the requested change, fixes findings, and runs the smallest relevant verification.
|
|
10
|
-
- **Reviewer**: independently reviews the latest diff and relevant verification results. It does not perform the implementation.
|
|
11
|
-
|
|
12
|
-
Workflow:
|
|
13
|
-
|
|
14
|
-
```text
|
|
15
|
-
Executor implements
|
|
16
|
-
→ targeted verification
|
|
17
|
-
→ Reviewer reviews latest diff
|
|
18
|
-
→ if findings exist: Executor fixes only those findings
|
|
19
|
-
→ targeted re-verification
|
|
20
|
-
→ Reviewer re-reviews changed parts and unresolved findings
|
|
21
|
-
→ repeat until Reviewer reports no actionable issues
|
|
22
|
-
```
|
|
23
|
-
|
|
24
|
-
Rules:
|
|
25
|
-
|
|
26
|
-
- Keep implementation, testing, and review scoped to the current change and directly affected paths.
|
|
27
|
-
- Prefer targeted tests: changed test files, affected modules, regression cases, or the original reproduction path.
|
|
28
|
-
- Do not rerun equivalent checks after every small edit.
|
|
29
|
-
- Reviewer should inspect the latest diff first, then only enough surrounding code to validate behavior and integration.
|
|
30
|
-
- On subsequent rounds, review the new changes plus unresolved findings; do not restart a whole-project review.
|
|
31
|
-
- Expand test or review scope only when the change crosses modules, modifies shared/public contracts or core infrastructure, targeted evidence is insufficient, final release/merge validation requires it, or the user explicitly requests it.
|
|
32
|
-
- Findings must be concrete and actionable. Separate blockers from optional improvements.
|
|
33
|
-
- Stop the loop when the Reviewer has no actionable findings and targeted verification is green.
|
|
34
|
-
- Do not let the Reviewer silently become the Executor; preserve role independence throughout the loop.
|
|
@@ -1,123 +0,0 @@
|
|
|
1
|
-
# HTML Report Format
|
|
2
|
-
|
|
3
|
-
The architectural review is rendered as a single self-contained HTML file in the OS temp directory. Tailwind and Mermaid both come from CDNs. Mermaid handles graph-shaped diagrams reliably; hand-built divs and inline SVG handle the more editorial visuals (mass diagrams, cross-sections). Mix the two: don't lean on Mermaid for everything, it'll start to look generic.
|
|
4
|
-
|
|
5
|
-
## Scaffold
|
|
6
|
-
|
|
7
|
-
```html
|
|
8
|
-
<!doctype html>
|
|
9
|
-
<html lang="en">
|
|
10
|
-
<head>
|
|
11
|
-
<meta charset="utf-8" />
|
|
12
|
-
<title>Architecture review for {{repo name}}</title>
|
|
13
|
-
<script src="https://cdn.tailwindcss.com"></script>
|
|
14
|
-
<script type="module">
|
|
15
|
-
import mermaid from "https://cdn.jsdelivr.net/npm/mermaid@11/dist/mermaid.esm.min.mjs";
|
|
16
|
-
mermaid.initialize({ startOnLoad: true, theme: "neutral", securityLevel: "loose" });
|
|
17
|
-
</script>
|
|
18
|
-
<style>
|
|
19
|
-
/* small custom layer for things Tailwind doesn't cover cleanly:
|
|
20
|
-
dashed seam lines, hand-drawn-feeling arrow heads, etc. */
|
|
21
|
-
.seam { stroke-dasharray: 4 4; }
|
|
22
|
-
.leak { stroke: #dc2626; }
|
|
23
|
-
.deep { background: linear-gradient(135deg, #0f172a, #1e293b); }
|
|
24
|
-
</style>
|
|
25
|
-
</head>
|
|
26
|
-
<body class="bg-stone-50 text-slate-900 font-sans">
|
|
27
|
-
<main class="max-w-5xl mx-auto px-6 py-12 space-y-12">
|
|
28
|
-
<header>...</header>
|
|
29
|
-
<section id="candidates" class="space-y-10">...</section>
|
|
30
|
-
<section id="top-recommendation">...</section>
|
|
31
|
-
</main>
|
|
32
|
-
</body>
|
|
33
|
-
</html>
|
|
34
|
-
```
|
|
35
|
-
|
|
36
|
-
## Header
|
|
37
|
-
|
|
38
|
-
Repo name, date, and a compact legend: solid box = module, dashed line = seam, red arrow = leakage, thick dark box = deep module. No introduction paragraph. Straight into the candidates.
|
|
39
|
-
|
|
40
|
-
## Candidate card
|
|
41
|
-
|
|
42
|
-
The diagrams carry the weight. Prose is sparse, plain, and uses the glossary terms (from the `/codebase-design` skill) without ceremony.
|
|
43
|
-
|
|
44
|
-
Each candidate is one `<article>`:
|
|
45
|
-
|
|
46
|
-
- **Title**: short, names the deepening (e.g. "Collapse the Order intake pipeline").
|
|
47
|
-
- **Badge row**: recommendation strength (`Strong` = emerald, `Worth exploring` = amber, `Speculative` = slate), plus a tag for the dependency category (`in-process`, `local-substitutable`, `ports & adapters`, `mock`).
|
|
48
|
-
- **Files**: monospaced list, `font-mono text-sm`.
|
|
49
|
-
- **Before / After diagram**: the centrepiece. Two columns, side by side. See patterns below.
|
|
50
|
-
- **Problem**: one sentence. What hurts.
|
|
51
|
-
- **Solution**: one sentence. What changes.
|
|
52
|
-
- **Wins**: bullets, ≤6 words each. e.g. "Tests hit one interface", "Pricing logic stops leaking", "Delete 4 shallow wrappers".
|
|
53
|
-
- **ADR callout** (if applicable): one line in an amber-tinted box.
|
|
54
|
-
|
|
55
|
-
No paragraphs of explanation. If the diagram needs a paragraph to be understood, redraw the diagram.
|
|
56
|
-
|
|
57
|
-
## Diagram patterns
|
|
58
|
-
|
|
59
|
-
Pick the pattern that fits the candidate. Mix them. Don't make every diagram look the same. Variety is part of the point.
|
|
60
|
-
|
|
61
|
-
### Mermaid graph (the workhorse for dependencies / call flow)
|
|
62
|
-
|
|
63
|
-
Use a Mermaid `flowchart` or `graph` when the point is "X calls Y calls Z, and look at the mess." Wrap it in a Tailwind-styled card so it doesn't feel parachuted in. Style with classDef to colour leakage edges red and the deep module dark. Sequence diagrams work well for "before: 6 round-trips; after: 1."
|
|
64
|
-
|
|
65
|
-
```html
|
|
66
|
-
<div class="rounded-lg border border-slate-200 bg-white p-4">
|
|
67
|
-
<pre class="mermaid">
|
|
68
|
-
flowchart LR
|
|
69
|
-
A[OrderHandler] --> B[OrderValidator]
|
|
70
|
-
B --> C[OrderRepo]
|
|
71
|
-
C -.leak.-> D[PricingClient]
|
|
72
|
-
classDef leak stroke:#dc2626,stroke-width:2px;
|
|
73
|
-
class C,D leak
|
|
74
|
-
</pre>
|
|
75
|
-
</div>
|
|
76
|
-
```
|
|
77
|
-
|
|
78
|
-
### Hand-built boxes-and-arrows (when Mermaid's layout fights you)
|
|
79
|
-
|
|
80
|
-
Modules as `<div>`s with borders and labels. Arrows as inline SVG `<line>` or `<path>` elements positioned absolutely over a relative container. Reach for this when you want the "after" diagram to feel like one thick-bordered deep module with greyed-out internals, since Mermaid won't render that with the right weight.
|
|
81
|
-
|
|
82
|
-
### Cross-section (good for layered shallowness)
|
|
83
|
-
|
|
84
|
-
Stack horizontal bands (`h-12 border-l-4`) to show layers a call passes through. Before: 6 thin layers each doing nothing. After: 1 thick band labelled with the consolidated responsibility.
|
|
85
|
-
|
|
86
|
-
### Mass diagram (good for "interface as wide as implementation")
|
|
87
|
-
|
|
88
|
-
Two rectangles per module: one for interface surface area, one for implementation. Before: interface rectangle is nearly as tall as the implementation rectangle (shallow). After: interface rectangle is short, implementation rectangle is tall (deep).
|
|
89
|
-
|
|
90
|
-
### Call-graph collapse
|
|
91
|
-
|
|
92
|
-
Before: a tree of function calls rendered as nested boxes. After: the same tree collapsed into one box, with the now-internal calls shown faded inside it.
|
|
93
|
-
|
|
94
|
-
## Style guidance
|
|
95
|
-
|
|
96
|
-
- Lean editorial, not corporate-dashboard. Generous whitespace. Serif optional for headings (`font-serif` works well with stone/slate).
|
|
97
|
-
- Colour sparingly: one accent (emerald or indigo) plus red for leakage and amber for warnings.
|
|
98
|
-
- Keep diagrams ~320px tall so before/after sits comfortably side by side without scrolling.
|
|
99
|
-
- Use `text-xs uppercase tracking-wider` for module labels inside diagrams, so they read as schematic, not as UI.
|
|
100
|
-
- The only scripts are the Tailwind CDN and the Mermaid ESM import. The report is otherwise static: no app code, no interactivity beyond Mermaid's own rendering.
|
|
101
|
-
|
|
102
|
-
## Top recommendation section
|
|
103
|
-
|
|
104
|
-
One larger card. Candidate name, one sentence on why, anchor link to its card. That's it.
|
|
105
|
-
|
|
106
|
-
## Tone
|
|
107
|
-
|
|
108
|
-
Plain English, concise, but the architectural nouns and verbs come straight from the `/codebase-design` skill. Concision is not an excuse to drift.
|
|
109
|
-
|
|
110
|
-
**Use exactly:** module, interface, implementation, depth, deep, shallow, seam, adapter, leverage, locality.
|
|
111
|
-
|
|
112
|
-
**Never substitute:** component, service, unit (for module) · API, signature (for interface) · boundary (for seam) · layer, wrapper (for module, when you mean module).
|
|
113
|
-
|
|
114
|
-
**Phrasings that fit the style:**
|
|
115
|
-
|
|
116
|
-
- "Order intake module is shallow: interface nearly matches the implementation."
|
|
117
|
-
- "Pricing leaks across the seam."
|
|
118
|
-
- "Deepen: one interface, one place to test."
|
|
119
|
-
- "Two adapters justify the seam: HTTP in prod, in-memory in tests."
|
|
120
|
-
|
|
121
|
-
**Wins bullets** name the gain in glossary terms: *"locality: bugs concentrate in one module"*, *"leverage: one interface, N call sites"*, *"interface shrinks; implementation absorbs the wrappers"*. Don't write *"easier to maintain"* or *"cleaner code"*, because those terms aren't in the glossary and don't earn their place.
|
|
122
|
-
|
|
123
|
-
No hedging, no throat-clearing, no "it's worth noting that…". If a sentence could be a bullet, make it a bullet. If a bullet could be cut, cut it. If a term isn't in the `/codebase-design` glossary, reach for one that is before inventing a new one.
|
|
@@ -1,71 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: improve-codebase-architecture
|
|
3
|
-
description: Scan a codebase for deepening opportunities, present them as a visual HTML report, then grill through whichever one you pick.
|
|
4
|
-
disable-model-invocation: true
|
|
5
|
-
---
|
|
6
|
-
|
|
7
|
-
# Improve Codebase Architecture
|
|
8
|
-
|
|
9
|
-
Surface architectural friction and propose **deepening opportunities**: refactors that turn shallow modules into deep ones. The aim is testability and AI-navigability.
|
|
10
|
-
|
|
11
|
-
This command is _informed_ by the project's domain model and built on a shared design vocabulary:
|
|
12
|
-
|
|
13
|
-
- Call the Skill tool with "codebase-design" for the architecture vocabulary (**module**, **interface**, **depth**, **seam**, **adapter**, **leverage**, **locality**) and its principles (the deletion test, "the interface is the test surface", "one adapter = hypothetical seam, two = real"). Use these terms exactly in every suggestion, and don't drift into "component," "service," "API," or "boundary."
|
|
14
|
-
- The domain language in `CONTEXT.md` gives names to good seams; ADRs in `docs/adr/` record decisions this command should not re-litigate.
|
|
15
|
-
|
|
16
|
-
## Process
|
|
17
|
-
|
|
18
|
-
### 1. Explore
|
|
19
|
-
|
|
20
|
-
**Scope before you scan: YAGNI.** Deepening a module pays off by making future changes to it easier, so put extra weight on the parts of the codebase that have recently changed. Decide *where* to look before you look:
|
|
21
|
-
|
|
22
|
-
- If the user named a direction (a module, a subsystem, a pain point), take it, and skip the inference below.
|
|
23
|
-
- Otherwise, walk back a good stretch of the commit history (`git log --oneline`) to find the codebase's hot spots, the files and areas that keep coming up, and let those paths pull your attention first. If the changes are scattered with no clear hot spot, widen the net.
|
|
24
|
-
|
|
25
|
-
Read the project's domain glossary (`CONTEXT.md`) and any ADRs in the area you're touching first.
|
|
26
|
-
|
|
27
|
-
Then spawn a sub-agent to walk the codebase. Don't follow rigid heuristics; explore organically and note where you experience friction:
|
|
28
|
-
|
|
29
|
-
- Where does understanding one concept require bouncing between many small modules?
|
|
30
|
-
- Where are modules **shallow**, with an interface nearly as complex as the implementation?
|
|
31
|
-
- Where have pure functions been extracted just for testability, but the real bugs hide in how they're called (no **locality**)?
|
|
32
|
-
- Where do tightly-coupled modules leak across their seams?
|
|
33
|
-
- Which parts of the codebase are untested, or hard to test through their current interface?
|
|
34
|
-
|
|
35
|
-
Apply the **deletion test** to anything you suspect is shallow: would deleting it concentrate complexity, or just move it? A "yes, concentrates" is the signal you want.
|
|
36
|
-
|
|
37
|
-
### 2. Present candidates as an HTML report
|
|
38
|
-
|
|
39
|
-
Write a self-contained HTML file to the OS temp directory so nothing lands in the repo. Resolve the temp dir from `$TMPDIR`, falling back to `/tmp` (or `%TEMP%` on Windows), and write to `<tmpdir>/architecture-review-<timestamp>.html` so each run gets a fresh file. Open it for the user (`xdg-open <path>` on Linux, `open <path>` on macOS, `start <path>` on Windows) and tell them the absolute path.
|
|
40
|
-
|
|
41
|
-
The report uses **Tailwind via CDN** for layout and styling, and **Mermaid via CDN** for diagrams where a graph/flow/sequence reliably communicates the structure. Mix Mermaid with hand-crafted CSS/SVG visuals: use Mermaid when relationships are graph-shaped (call graphs, dependencies, sequences), and hand-built divs/SVG when you want something more editorial (mass diagrams, cross-sections, collapse animations). Each candidate gets a **before/after visualisation**. Be visual.
|
|
42
|
-
|
|
43
|
-
For each candidate, render a card with:
|
|
44
|
-
|
|
45
|
-
- **Files**: which files/modules are involved
|
|
46
|
-
- **Problem**: why the current architecture is causing friction
|
|
47
|
-
- **Solution**: plain English description of what would change
|
|
48
|
-
- **Benefits**: explained in terms of locality and leverage, and how tests would improve
|
|
49
|
-
- **Before / After diagram**: side-by-side, custom-drawn, illustrating the shallowness and the deepening
|
|
50
|
-
- **Recommendation strength**: one of `Strong`, `Worth exploring`, `Speculative`, rendered as a badge
|
|
51
|
-
|
|
52
|
-
End the report with a **Top recommendation** section: which candidate you'd tackle first and why.
|
|
53
|
-
|
|
54
|
-
**Use CONTEXT.md vocabulary for the domain, and the `/codebase-design` vocabulary for the architecture.** If `CONTEXT.md` defines "Order," talk about "the Order intake module," not "the FooBarHandler," and not "the Order service."
|
|
55
|
-
|
|
56
|
-
**ADR conflicts**: if a candidate contradicts an existing ADR, only surface it when the friction is real enough to warrant revisiting the ADR. Mark it clearly in the card (e.g. a warning callout: _"contradicts ADR-0007, but worth reopening because…"_). Don't list every theoretical refactor an ADR forbids.
|
|
57
|
-
|
|
58
|
-
See [HTML-REPORT.md](HTML-REPORT.md) for the full HTML scaffold, diagram patterns, and styling guidance.
|
|
59
|
-
|
|
60
|
-
Do NOT propose interfaces yet. After the file is written, ask the user: "Which of these would you like to explore?"
|
|
61
|
-
|
|
62
|
-
### 3. Grilling loop
|
|
63
|
-
|
|
64
|
-
Once the user picks a candidate, call the Skill tool with "grilling" to walk the decision tree with them: constraints, dependencies, the shape of the deepened module, what sits behind the seam, what tests survive.
|
|
65
|
-
|
|
66
|
-
Side effects happen inline as decisions crystallize; call the Skill tool with "domain-modeling" to keep the domain model current as you go:
|
|
67
|
-
|
|
68
|
-
- **Naming a deepened module after a concept not in `CONTEXT.md`?** Add the term to `CONTEXT.md`. Create the file lazily if it doesn't exist.
|
|
69
|
-
- **Sharpening a fuzzy term during the conversation?** Update `CONTEXT.md` right there.
|
|
70
|
-
- **User rejects the candidate with a load-bearing reason?** Offer an ADR, framed as: _"Want me to record this as an ADR so future architecture reviews don't re-suggest it?"_ Only offer when the reason would actually be needed by a future explorer to avoid re-suggesting the same thing; skip ephemeral reasons ("not worth it right now") and self-evident ones.
|
|
71
|
-
- **Want to explore alternative interfaces for the deepened module?** Call the Skill tool with "codebase-design" and use its design-it-twice parallel sub-agent pattern.
|
|
@@ -1,70 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: instance-test
|
|
3
|
-
disable-model-invocation: true
|
|
4
|
-
description: "matt-skills 专属功能测试示范(由 scaffold-functional-test 从 spec 生成)— 验证 sync 合并 update 后的行为;仅显式调用"
|
|
5
|
-
---
|
|
6
|
-
|
|
7
|
-
# Instance Test — matt-skills 专属示范
|
|
8
|
-
|
|
9
|
-
本 skill 是 **matt-skills 专属**的功能测试示范,由 `scaffold-functional-test` 从 `.scratch/sync-merge-update/spec.md` 生成(见 `references/instances.md` 头部 `spec hash` + `generatedAt`)。它是生成器产出形态的示例,不随 Template Snapshot 分发,仅保留于 Workspace。旧通用执行器文案已废弃。
|
|
10
|
-
|
|
11
|
-
兼容别名:`instance-test` 保留原名以兼容历史调用,实际为 `matt-functional-test` 的示范实现。
|
|
12
|
-
|
|
13
|
-
## Steps
|
|
14
|
-
|
|
15
|
-
### 1. Gather instances
|
|
16
|
-
|
|
17
|
-
实例集已由生成器按**受控扩展模型**落盘于 `references/instances.md`(头部含 `spec hash` + `generatedAt`,每实例含**溯源** `spec.md` 章节/行号,`<!-- manual -->` 段受保护)。
|
|
18
|
-
|
|
19
|
-
执行前校验指纹:若当前 spec 的 `spec hash` 与 `references/instances.md` 头部不一致,提示「spec 已变更,建议重跑 scaffold-functional-test」但不自动覆盖,需用户显式确认才 regenerate(AI 先给 diff 建议)。
|
|
20
|
-
|
|
21
|
-
每实例声明:`prompt/command/expected files/content/expected stdout phrases/expected exit code` 必选,`setup/env/timeout/type/teardown` 可选,默认 `type: cli`。
|
|
22
|
-
|
|
23
|
-
完成:实例清单已固定(含溯源与指纹),`<!-- manual -->` 段未被覆盖。
|
|
24
|
-
|
|
25
|
-
### 2. Run instances
|
|
26
|
-
|
|
27
|
-
For each **instance** in order:
|
|
28
|
-
|
|
29
|
-
1. `mktemp -d` 隔离目录(或项目支持的 `git worktree` / `--dest`),单线程串行,不并行。
|
|
30
|
-
2. 执行实例的 `command` 与可选 `setup`,捕获 stdout/stderr 与 exit code。
|
|
31
|
-
3. 快照 `expected` 声明的文件与副作用。
|
|
32
|
-
|
|
33
|
-
一个 **instance** 一次,失败不阻断后续,产物不碰撞。
|
|
34
|
-
|
|
35
|
-
完成:每实例均有独立 run dir 与捕获输出。
|
|
36
|
-
|
|
37
|
-
### 3. Evaluate
|
|
38
|
-
|
|
39
|
-
对比每实例的 actual vs expected:
|
|
40
|
-
|
|
41
|
-
- 文件存在性/内容(`test -f`/`grep -q`/`diff`)
|
|
42
|
-
- Stdout/stderr 含预期短语
|
|
43
|
-
- Exit code 一致
|
|
44
|
-
- 扩展字段(`env`/`timeout`/`type`)行为符合声明
|
|
45
|
-
|
|
46
|
-
标记 `PASS`/`FAIL`,附 `expected vs actual` diff 与 run dir 证据。
|
|
47
|
-
|
|
48
|
-
完成:每实例均有 `PASS` 或 `FAIL` 且含证据。
|
|
49
|
-
|
|
50
|
-
### 4. Report
|
|
51
|
-
|
|
52
|
-
对话内汇总:
|
|
53
|
-
|
|
54
|
-
- `PASS m/n` + per-instance evidence
|
|
55
|
-
- 失败项列出 gap(expected vs actual)与 run dir 复现路径
|
|
56
|
-
- 成功默认清理临时目录、失败默认保留;`--keep` 保留全部;`--report` 显式开启才落盘报告文件
|
|
57
|
-
|
|
58
|
-
不以文件刷屏——默认输出在对话,报告文件仅显式开启才写。
|
|
59
|
-
|
|
60
|
-
## 实例来源
|
|
61
|
-
|
|
62
|
-
- 源 spec:`.scratch/sync-merge-update/spec.md`(`spec hash` 见 `references/instances.md` 头部)
|
|
63
|
-
- 推导策略:混合推导(验收标准锚点 + 需求/接口/边界补充),每实例含溯源,无溯源视为幻觉
|
|
64
|
-
- 手工段:`<!-- manual -->` 保护
|
|
65
|
-
|
|
66
|
-
## 引用
|
|
67
|
-
|
|
68
|
-
- 生成器:`scaffold-functional-test`(读 spec 产出本 skill)
|
|
69
|
-
- 领域术语:`CONTEXT.md`
|
|
70
|
-
- 技能设计:`docs/agents/skill-design.md`
|
|
@@ -1,75 +0,0 @@
|
|
|
1
|
-
# Instances for matt-skills — sync 行为功能测试(由 scaffold-functional-test 生成)
|
|
2
|
-
|
|
3
|
-
> 源 spec:`.scratch/sync-merge-update/spec.md`
|
|
4
|
-
> spec hash: `062a76fc872d` # .scratch/sync-merge-update/spec.md 的 sha256 前 12 位
|
|
5
|
-
> generatedAt: 2026-05-11
|
|
6
|
-
> 推导策略:混合推导(验收标准锚点 + 需求/行为补充),每实例含溯源,无溯源视为幻觉
|
|
7
|
-
|
|
8
|
-
本文件由 `scaffold-functional-test` 按**受控扩展模型**生成:必选 `prompt/command/expected files/content/expected stdout phrases/expected exit code`,可选 `setup/env/timeout/type/teardown`,默认 `type: cli`。执行语义:`mktemp -d` 隔离、单线程串行、`PASS m/n` 汇总、证据含 `expected vs actual` diff + `run dir`。
|
|
9
|
-
|
|
10
|
-
执行前校验:对比当前 `.scratch/sync-merge-update/spec.md` 的 hash 与本文件头部 `spec hash`,不一致时提示「spec 已变更,建议重跑 scaffold-functional-test」但不自动覆盖。
|
|
11
|
-
|
|
12
|
-
---
|
|
13
|
-
|
|
14
|
-
## 1. sync 默认 check(无参不写盘)
|
|
15
|
-
|
|
16
|
-
- Prompt: 验证 `matt-skills sync` 无参等价 check,打印表且不改 AGENTS.md
|
|
17
|
-
- 溯源: spec.md — 需求/行为「`matt-skills sync` 无参:等价 `check`」+ 验收标准「`sync` 无参在已定制的 `pi-switch` 仓库上不改 `AGENTS.md`」
|
|
18
|
-
- type: cli
|
|
19
|
-
- setup: `node bin/cli.js init --dest <tmp>` 后手工改 `AGENTS.md` 加入 `tdd-implement` 定制行
|
|
20
|
-
- Command: `node bin/cli.js sync --dest <tmp>`(无参)
|
|
21
|
-
- Expected:
|
|
22
|
-
- `git diff HEAD -- AGENTS.md` 为空(`AGENTS.md` 未被覆盖)
|
|
23
|
-
- stdout 含 `上游 HEAD` 与 `新增/更新/删除/一致` 表头
|
|
24
|
-
- stdout 含 `--json` 可解析提示或表格行
|
|
25
|
-
- exit 0 或 1(有差异时 exit 1,判 exit code 符合 check 语义)
|
|
26
|
-
- Expected files/content: `AGENTS.md` 保留定制行,无 `AGENTS.md.bak` 新增
|
|
27
|
-
- Expected stdout phrases: `上游 HEAD`, `一致`
|
|
28
|
-
- Expected exit code: 1(有差异时)/ 0(无差异时)— 按实现定义,测试以实际 check 语义为准
|
|
29
|
-
|
|
30
|
-
## 2. sync --apply 安全增量(AGENTS.md 跳过、上游强制覆盖不删)
|
|
31
|
-
|
|
32
|
-
- Prompt: 验证 `sync --apply` 为安全增量,`AGENTS.md` 定制跳过、上游技能被覆盖但 remove 列表不删
|
|
33
|
-
- 溯源: spec.md — 需求/行为「`sync --apply`:安全增量写盘。`AGENTS.md` 若含独有路由则跳过;上游技能 `rm+cp force` 覆盖,跳过 `PROPRIETARY`,不执行 `remove`」+ 验收标准「`sync --apply` 后上游技能被强制更新为上游 `HEAD`,`remove` 列表的技能仍保留」
|
|
34
|
-
- type: cli
|
|
35
|
-
- setup: 在 `<tmp>` 放置旧版上游技能 `test-skill` 过期文件,并手工改 `AGENTS.md`
|
|
36
|
-
- Command: `node bin/cli.js sync --apply --dest <tmp>`
|
|
37
|
-
- Expected:
|
|
38
|
-
- `AGENTS.md` 仍含定制行(未被模板覆盖)
|
|
39
|
-
- 上游技能文件已更新为上游 HEAD 内容(`diff` 无旧版残留)
|
|
40
|
-
- `remove` 列表中的技能目录仍存在(未被删除)
|
|
41
|
-
- Expected files/content: `AGENTS.md` 定制行存在;`test-skill` 被覆盖为新版;无 `AGENTS.md.bak`(安全档不备份)或按实现保留但不覆盖
|
|
42
|
-
- Expected stdout phrases: `已同步` 或 `已更新` 或 `同步`
|
|
43
|
-
- Expected exit code: 0
|
|
44
|
-
- timeout: 30000
|
|
45
|
-
|
|
46
|
-
## 3. sync --force 硬盖(AGENTS.md 备份后覆盖、全量 add/update/remove)
|
|
47
|
-
|
|
48
|
-
- Prompt: 验证 `sync --force` 硬盖,`AGENTS.md` 备份后被模板覆盖、技能与模板全量同步含删除
|
|
49
|
-
- 溯源: spec.md — 需求/行为「`sync --force`:硬盖。`AGENTS.md` 先 `backupIfExists → .bak` 再 `cp -r force`;技能与模板均 `add/update/remove` 全做」+ 验收标准「`sync --force` 后 `AGENTS.md` 变为模板且 `AGENTS.md.bak` 存在,`remove` 列表的技能被删除」
|
|
50
|
-
- type: cli
|
|
51
|
-
- setup: 在 `<tmp>` 放置 `AGENTS.md` 定制行 + 一个上游已删的本地技能 `obsolete-skill/`
|
|
52
|
-
- Command: `node bin/cli.js sync --force --dest <tmp>`
|
|
53
|
-
- Expected:
|
|
54
|
-
- `AGENTS.md` 已被模板覆盖(定制行消失,与 `template/AGENTS.md` 一致)
|
|
55
|
-
- `AGENTS.md.bak` 存在且含定制行备份
|
|
56
|
-
- `obsolete-skill/` 已被删除
|
|
57
|
-
- Expected files/content: `AGENTS.md` 内容等于 `template/AGENTS.md`;`AGENTS.md.bak` 存在
|
|
58
|
-
- Expected stdout phrases: `已覆盖` 或 `硬盖`
|
|
59
|
-
- Expected exit code: 0
|
|
60
|
-
|
|
61
|
-
## 4. update 已合并到 sync --apply(删除分支、提示已合并)
|
|
62
|
-
|
|
63
|
-
- Prompt: 验证 `matt-skills update` 已删除,执行后报错提示已合并到 `sync --apply`,且 `--help` 不再列 `update`
|
|
64
|
-
- 溯源: spec.md — 需求/行为「`matt-skills update`:删除该分支,`main` 中 `command === 'update'` 改为 `stderr: 'update 已合并到 sync --apply'` 且 `exit 1`,`--help` 不再列 `update`」+ 验收标准「`matt-skills update` 执行后报错提示已合并,`--help` 无 `update`」
|
|
65
|
-
- type: cli
|
|
66
|
-
- Command: `node bin/cli.js update 2>&1; echo "exit:$?"` 与 `node bin/cli.js --help`
|
|
67
|
-
- Expected:
|
|
68
|
-
- `update` 命令 stdout/stderr 含 `已合并到 sync --apply` 且 exit 1
|
|
69
|
-
- `--help` 输出不含独立的 `update` 子命令行(不匹配 `^\s*update`)
|
|
70
|
-
- Expected stdout phrases: `已合并到 sync --apply`
|
|
71
|
-
- Expected exit code: 1(`update` 分支)
|
|
72
|
-
- env: {}
|
|
73
|
-
|
|
74
|
-
<!-- manual -->
|
|
75
|
-
<!-- 以下为人工定制实例保护段:由开发者手写,scaffold-functional-test 重生成时不覆盖此段以上的内容。如需新增手工实例,请在此段后追加。 -->
|