@mstar-harness/opencode 0.7.0 → 0.7.4
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/INSTALL.md +2 -5
- package/harness-commands/iteration-drive.md +15 -8
- package/harness-commands/iteration-start.md +45 -10
- package/harness-skills/mstar-branch-worktree/SKILL.md +4 -4
- package/harness-skills/mstar-coding-behavior/SKILL.md +27 -4
- package/harness-skills/mstar-compound/SKILL.md +2 -0
- package/harness-skills/mstar-dispatch-gates/SKILL.md +13 -4
- package/harness-skills/mstar-dispatch-gates/references/leaf-executor-checklist.md +1 -1
- package/harness-skills/mstar-harness-core/SKILL.md +8 -5
- package/harness-skills/mstar-harness-core/references/open-harness-principles.md +3 -3
- package/harness-skills/mstar-host/references/cursor-plan-mode-bridge.md +1 -1
- package/harness-skills/mstar-host/references/cursor.md +2 -2
- package/harness-skills/mstar-host/references/opencode.md +2 -2
- package/harness-skills/mstar-iteration/SKILL.md +87 -28
- package/harness-skills/mstar-iteration/references/iteration-compass-template.md +14 -8
- package/harness-skills/mstar-phase-gates/SKILL.md +3 -3
- package/harness-skills/mstar-plan-artifacts/references/plan-files-and-reports.md +1 -1
- package/harness-skills/mstar-plan-conventions/SKILL.md +3 -3
- package/harness-skills/mstar-roles/SKILL.md +4 -4
- package/harness-skills/mstar-roles/references/architect.md +7 -11
- package/harness-skills/mstar-roles/references/frontend-dev.md +2 -2
- package/harness-skills/mstar-roles/references/fullstack-dev-shared.md +2 -2
- package/harness-skills/mstar-roles/references/ops-engineer.md +2 -2
- package/harness-skills/mstar-roles/references/product-manager.md +7 -9
- package/harness-skills/mstar-roles/references/project-manager/dispatch-and-assignment.md +3 -3
- package/harness-skills/mstar-roles/references/project-manager.md +4 -5
- package/harness-skills/mstar-roles/references/prompt-engineer.md +11 -5
- package/harness-skills/mstar-roles/references/qa-engineer.md +2 -2
- package/harness-skills/mstar-roles/references/qc-specialist-shared.md +1 -1
- package/harness-skills/mstar-roles/references/writing-specialist.md +2 -2
- package/harness-skills/mstar-skill-authoring/SKILL.md +186 -0
- package/harness-skills/pm/SKILL.md +21 -1
- package/package.json +1 -1
- package/harness-skills/mstar-superpowers-align/SKILL.md +0 -150
- package/harness-skills/mstar-superpowers-align/references/per-role-matrix.md +0 -99
- package/harness-skills/mstar-superpowers-align/references/tension-table.md +0 -19
package/INSTALL.md
CHANGED
|
@@ -6,14 +6,11 @@
|
|
|
6
6
|
|
|
7
7
|
## Installation
|
|
8
8
|
|
|
9
|
-
Add Morning Star to the `plugin` array in your `opencode.json` (global or project-level)
|
|
10
|
-
|
|
11
|
-
If you use Superpowers together, use:
|
|
9
|
+
Add Morning Star to the `plugin` array in your `opencode.json` (global or project-level):
|
|
12
10
|
|
|
13
11
|
```json
|
|
14
12
|
{
|
|
15
13
|
"plugin": [
|
|
16
|
-
"superpowers@git+https://github.com/obra/superpowers.git",
|
|
17
14
|
"@mstar-harness/opencode@latest"
|
|
18
15
|
]
|
|
19
16
|
}
|
|
@@ -64,7 +61,7 @@ The `@mstar-harness/cli` package (`npx @mstar-harness/cli init`) migrates that e
|
|
|
64
61
|
|
|
65
62
|
### Plugin not loading
|
|
66
63
|
|
|
67
|
-
1. Check logs: `opencode run --print-logs "hello" 2>&1 | grep -i
|
|
64
|
+
1. Check logs: `opencode run --print-logs "hello" 2>&1 | grep -i mstar`
|
|
68
65
|
2. Verify the plugin line in your `opencode.json`
|
|
69
66
|
3. Make sure you're running a recent version of OpenCode
|
|
70
67
|
|
|
@@ -13,9 +13,10 @@ Drive the active Morning Star iteration forward. The canonical flow is in **`mst
|
|
|
13
13
|
1. `mstar-harness-core`
|
|
14
14
|
2. `mstar-roles` → `references/project-manager.md`
|
|
15
15
|
3. `skills/pm/SKILL.md` → **§ Host entry** + **§ Boot**(PM role identity + dispatch-first rules)
|
|
16
|
-
4. `mstar-iteration` → **§ Phase 2
|
|
17
|
-
5. `mstar-
|
|
18
|
-
6. `mstar-
|
|
16
|
+
4. `mstar-iteration` → **§ Phase 2** + **§ Phase 3**(close 为独立 gate,§3.0–§3.5)
|
|
17
|
+
5. `mstar-compound` — Phase 3 §3.2 前加载(含 Phase 6 索引登记)
|
|
18
|
+
6. `mstar-dispatch-gates` + host reference
|
|
19
|
+
7. `mstar-plan-artifacts`, `mstar-plan-conventions`, `mstar-branch-worktree`
|
|
19
20
|
|
|
20
21
|
## Phase 2: Autonomous Execute
|
|
21
22
|
|
|
@@ -39,12 +40,16 @@ Execute **`mstar-iteration` § Phase 2** exactly. Summary:
|
|
|
39
40
|
|
|
40
41
|
## Phase 3: iteration-close
|
|
41
42
|
|
|
42
|
-
|
|
43
|
+
当 **every** plan 为 `Done` 时,**STOP** per-plan loop,打印 `## Phase 3: iteration-close`,按 **`mstar-iteration` § Phase 3** 逐步执行。final plan 的 closure 只能提供输入,不能替代 close gate。
|
|
43
44
|
|
|
44
|
-
|
|
45
|
-
|
|
46
|
-
|
|
47
|
-
|
|
45
|
+
| Step | Section | 要点 |
|
|
46
|
+
|------|---------|------|
|
|
47
|
+
| 1 | §3.0 / §3.0.5 | Phase 边界;legacy compass 规范化(frontmatter + `## Roadmap Position`) |
|
|
48
|
+
| 2 | §3.1 | 打印 close entry checklist,全部 `[x]` |
|
|
49
|
+
| 3 | §3.2 | Compound;新增 doc 完成 `mstar-compound` Phase 6(`{KNOWLEDGE_DIR}/README.md`) |
|
|
50
|
+
| 4 | §3.3 | `## Roadmap Position` current iteration → `delivered`;STRATEGY/tracker 按需更新 |
|
|
51
|
+
| 5 | §3.4 | frontmatter `status: completed` + `end_date`;Compound Summary + Retrospective |
|
|
52
|
+
| 6 | §3.5 | 打印 close exit checklist → commit → push |
|
|
48
53
|
|
|
49
54
|
```bash
|
|
50
55
|
git add {ITERATION_DIR}/ {KNOWLEDGE_DIR}/ CONCEPTS.md
|
|
@@ -52,6 +57,8 @@ git commit -m "chore(iteration): close <iteration-id> — compound round, roadma
|
|
|
52
57
|
git push origin <spec_integration_branch>
|
|
53
58
|
```
|
|
54
59
|
|
|
60
|
+
**Not Done until**: §3.5 checklist `[x]` + compass frontmatter `completed`.
|
|
61
|
+
|
|
55
62
|
## Completion: Create PR
|
|
56
63
|
|
|
57
64
|
All iteration-close changes committed to integration branch:
|
|
@@ -1,21 +1,24 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: iteration-start
|
|
3
|
-
description: Start a new harness iteration — research backlog, lock direction with grill-me, produce compass/plans, run
|
|
3
|
+
description: Start a new harness iteration — research backlog, lock direction with grill-me, produce compass/plans, run mandatory Review & Edit chain via Task dispatch (product-manager → architect → writing-specialist each review-and-edit, then PM final lock), then integration branch. Not Done until review chain completes and compass is locked.
|
|
4
4
|
agent: project-manager
|
|
5
5
|
---
|
|
6
6
|
|
|
7
7
|
# Start Iteration
|
|
8
8
|
|
|
9
|
-
Start a new Morning Star harness iteration.
|
|
9
|
+
Start a new Morning Star harness iteration. **Not Done until the Review & Edit chain runs via dispatched roles and PM lock — not when compass files are first written.**
|
|
10
|
+
|
|
11
|
+
Detailed workflow → **`mstar-iteration` § Phase 1: iteration-start**(含 §1.6 Review & Edit chain);per-plan Prepare gates → **`mstar-phase-gates`**(specify → clarify → plan)。
|
|
10
12
|
|
|
11
13
|
## 0. Boot
|
|
12
14
|
|
|
13
15
|
1. `mstar-harness-core`
|
|
14
16
|
2. `mstar-roles` → `references/project-manager.md`
|
|
15
17
|
3. `skills/pm/SKILL.md` → **§ Host entry** + **§ Boot**
|
|
16
|
-
4. `mstar-iteration` → **§ Phase 1: iteration-start**(迭代范围、compass
|
|
17
|
-
5. `mstar-
|
|
18
|
-
6. `mstar-
|
|
18
|
+
4. `mstar-iteration` → **§ Phase 1: iteration-start**(迭代范围、compass 模板、§1.6 Review & Edit chain、状态初始化)
|
|
19
|
+
5. `mstar-dispatch-gates` → iteration-start review chain + implement dispatch
|
|
20
|
+
6. `mstar-phase-gates` → Prepare(specify → clarify → plan)
|
|
21
|
+
7. `mstar-plan-conventions`, `mstar-plan-artifacts`
|
|
19
22
|
|
|
20
23
|
## 1. Research
|
|
21
24
|
|
|
@@ -63,17 +66,49 @@ Produce harness artifacts per **`mstar-iteration` § 1.3 创建迭代 compass**
|
|
|
63
66
|
- Register all plans in `{HARNESS_DIR}/status.json`(per `mstar-plan-artifacts`)
|
|
64
67
|
- Update `{ITERATION_DIR}/README.md` index(per `mstar-iteration` § 1.4)
|
|
65
68
|
|
|
66
|
-
## 5. Review & Edit Chain
|
|
69
|
+
## 5. Review & Edit Chain(HARD GATE — do not commit before this)
|
|
70
|
+
|
|
71
|
+
**STOP**: Do not run §6 Integration Branch until **all** rows below are true.
|
|
67
72
|
|
|
68
73
|
Each role below **reviews and directly edits** the documents. Do not just flag issues — apply the fixes yourself. PM only steps in for the final lock.
|
|
69
74
|
|
|
70
|
-
|
|
71
|
-
|
|
72
|
-
|
|
73
|
-
|
|
75
|
+
| # | Role | Required action |
|
|
76
|
+
|---|------|-----------------|
|
|
77
|
+
| 5.1 | **@product-manager** | Task subagent: **edit** compass, plans, `{SPECS_DIR}/`; scope, UX, priorities |
|
|
78
|
+
| 5.2 | **@architect** | Task subagent: **edit** compass, plans, specs; contracts, SQL, module boundaries |
|
|
79
|
+
| 5.3 | **@writing-specialist** | Task subagent: **edit** all iteration docs; terminology, structure, clarity |
|
|
80
|
+
| 5.4 | **@project-manager** | Merge subagent edits; resolve conflicts; **lock** compass (`status: locked`); confirm Prepare gates |
|
|
81
|
+
|
|
82
|
+
**Evidence of done** = edited compass / plans / specs on disk + compass `status: locked`. **No** separate iteration review reports under `reports/` — unlike per-plan QC, there is no downstream audit chain to preserve.
|
|
83
|
+
|
|
84
|
+
**Tool rule (Cursor / hosts with Task)**:
|
|
85
|
+
- Steps 5.1–5.3: **MUST** use `Task` (parallel when independent). See **`mstar-dispatch-gates`** · iteration-start review chain.
|
|
86
|
+
- PM thread **MUST NOT** substitute by performing all three specialist edits itself.
|
|
87
|
+
- Exception: user explicitly waives subagent dispatch ("PM-only review").
|
|
88
|
+
|
|
89
|
+
**Prepare gate (per plan in compass)**:
|
|
90
|
+
- [ ] specify / clarify / plan = done on each plan file
|
|
91
|
+
- [ ] `primary_spec` path exists (if declared)
|
|
92
|
+
- [ ] `blocked_by` / sequential deps documented
|
|
93
|
+
|
|
94
|
+
Only after 5.4 → proceed to §6.
|
|
95
|
+
|
|
96
|
+
### iteration-start pre-commit checklist
|
|
97
|
+
|
|
98
|
+
PM must print this block before §6; all `[ ]` must be `[x]`:
|
|
99
|
+
|
|
100
|
+
- [ ] grill-me decisions recorded in compass
|
|
101
|
+
- [ ] Draft compass + plans + `status.json` registered
|
|
102
|
+
- [ ] @product-manager Task completed — compass / plans / specs edited
|
|
103
|
+
- [ ] @architect Task completed — compass / specs edited
|
|
104
|
+
- [ ] @writing-specialist Task completed — iteration docs edited
|
|
105
|
+
- [ ] PM final lock: compass `status: locked`; Prepare gates pass (blocked plans documented)
|
|
106
|
+
- [ ] **THEN**: git commit + push `iteration/<iteration-id>`
|
|
74
107
|
|
|
75
108
|
## 6. Integration Branch
|
|
76
109
|
|
|
110
|
+
**Precondition**: §5 complete — compass `status: locked`; specialist Tasks returned; Prepare gates confirmed.
|
|
111
|
+
|
|
77
112
|
- Create `iteration/<iteration-id>` from `main`
|
|
78
113
|
- Register `spec_integration_branch` in `{HARNESS_DIR}/status.json`
|
|
79
114
|
- Commit all documents to the integration branch and push to remote
|
|
@@ -105,16 +105,16 @@ description: Morning Star business-repo Git feature branches, same-repo concurre
|
|
|
105
105
|
|
|
106
106
|
当 **`@project-manager` 在同一调度轮次内并发启动多个** subagent(含宿主侧「并行 Task / 并行 subagent」),且 **≥2 个承接方**可能对 **同一 Git 仓库的同一工作区(同一 cwd 检出目录)**产生写文件或 `git commit` 级改动时:
|
|
107
107
|
|
|
108
|
-
- **必须**为每条并发写流使用
|
|
108
|
+
- **必须**为每条并发写流使用 **独立检出目录**:优先使用宿主原生 worktree/checkout 隔离能力;没有原生能力时使用 `git worktree`,并按本 skill 的目录、分支和 QC/QA 对齐规则执行。
|
|
109
109
|
- **必须**与既有分支门禁一致:每个可写承接方的 Assignment 仍须含 PM 已批准的 **`Working branch`** / **`Branch policy`**;在某一 worktree 内 **不得**擅自 `checkout` 到未授权分支或私自新建分支。
|
|
110
|
-
- **PM 须在 Assignment 中写清**各并发写流的 **检出约定**(例如预期 **`Worktree path`** /
|
|
110
|
+
- **PM 须在 Assignment 中写清**各并发写流的 **检出约定**(例如预期 **`Worktree path`** / 命名规则,或「由承接方创建/使用隔离 worktree 并在 Completion Report 回报路径」),避免多代理默认共享同一目录导致互相覆盖、冲突或半写入状态。
|
|
111
111
|
- **同仓、同一 plan、≥2 可写并行轨**:**推荐**在首次向各轨下发实现 Assignment **之前**,先由 PM 与用户确认 **`Branch policy`**,并 **明确 plan 集成分支与各轨 topic 分支的关系**(见下节 **「推荐默认编排:先建 plan 集成分支,再挂各 worktree」**),再为各轨约定 **`git worktree`**。这样 QC 前可把各轨 **自然归并**到同一条 **`HEAD`**,减少「多头分支、无合并靶」导致的误派。
|
|
112
112
|
|
|
113
113
|
**可不强制新开 worktree** 的情形包括:并发流 **全部为只读**;各写入者针对 **不同 Git 仓库根**;或写入 **串行**(同一时刻仅一个代理持有该仓工作区)。
|
|
114
114
|
|
|
115
115
|
### 并发 subagent 与同仓工作树(对齐)
|
|
116
116
|
|
|
117
|
-
当多个可写 subagent **并发**修改 **同一仓库** 时,**不得**共用同一检出目录作为写入 cwd。PM 在分派前应规划
|
|
117
|
+
当多个可写 subagent **并发**修改 **同一仓库** 时,**不得**共用同一检出目录作为写入 cwd。PM 在分派前应规划 worktree/checkout 隔离,并在各承接方 Assignment 中写明 **`Working branch`** / **`Branch policy`** 及 **检出路径约定**(或要求回报实际 worktree 路径)。单分支决策权仍仅属 PM;worktree 只解决「目录与工作区隔离」,不替代分支授权。
|
|
118
118
|
|
|
119
119
|
**同仓、同一 plan、多可写并行轨(推荐)**:在挂齐各轨 `git worktree` **之前**,先与用户确认并写明 **plan 集成分支**(从商定 `<base>` 创建)及各轨 **topic 分支** 如何从该线分出或如何 **merge 回** 该线;QC 前再将待一并验收的提交 **全部归并**到 PM 指定为 QC **`Working branch`** 的那条分支的 **`HEAD`**。分步说明与示例命名边界见下节 **「推荐默认编排:先建 plan 集成分支,再挂各 worktree」**。
|
|
120
120
|
|
|
@@ -137,7 +137,7 @@ description: Morning Star business-repo Git feature branches, same-repo concurre
|
|
|
137
137
|
|
|
138
138
|
开发在 **feature 分支**上完成(往往在 **独立 worktree** 中实现)后,**QC 审查与 QA 验证针对的都是这份 feature**,而不是 `main` 或任意未对齐的默认 cwd。
|
|
139
139
|
|
|
140
|
-
- **`@project-manager`** 分派 **QC** 时须在 Assignment 写明与待审实现一致的 **`Working branch`**,并写明 **`Review cwd` / `Worktree path`**:**优先**沿用开发 **Completion Report** 中回报的业务仓 **实现检出路径**(即「该 feature 的 worktree」)**当且仅当**该路径上的检出分支 **`HEAD` 已包含本轮待审的全部提交**(含曾发生在其他并行 worktree、现已归并到该分支的变更)。否则 **必须**改用 **集成完成后的** `Working branch`
|
|
140
|
+
- **`@project-manager`** 分派 **QC** 时须在 Assignment 写明与待审实现一致的 **`Working branch`**,并写明 **`Review cwd` / `Worktree path`**:**优先**沿用开发 **Completion Report** 中回报的业务仓 **实现检出路径**(即「该 feature 的 worktree」)**当且仅当**该路径上的检出分支 **`HEAD` 已包含本轮待审的全部提交**(含曾发生在其他并行 worktree、现已归并到该分支的变更)。否则 **必须**改用 **集成完成后的** `Working branch` 与对应检出路径(或在该分支上 **另开** 审查专用 worktree)。若开发未用 worktree,则写明单一明确的业务仓根路径。若审查需与开发目录 **物理分离** 但仍审 **同一分支**,可指示在 **`Working branch`** 上 **另加** 一个 worktree 专供审查(只读使用业务仓)。**多流并行开发**时的前置归并、**推荐默认编排(plan 集成分支先行)** 与误派禁令见上一小节。
|
|
141
141
|
- **三票审同一功能(强制对齐)**:分派 **QC 三审**时,除上述字段外,**必须**在 **三份 Assignment 中逐字写入相同**的 **`plan_id`** 与 **`Review range` / `Diff basis`**:
|
|
142
142
|
- **`plan_id`**:与 `{PLAN_DIR}/reports/<plan-id>/` 及主 **Plan Path** 一致;无 `{PLAN_DIR}` 流程时写 **`plan_id: N/A`**,并另给一行 **`Feature / scope label`**(不可歧义,足以与并行其它 feature 区分)。
|
|
143
143
|
- **`Review range` / `Diff basis`**:明确本次审查所针对的 **diff/提交范围**(例如 `merge-base: origin/main` + `tip: HEAD`;或 `rev-range: <full-40>..<full-40>`;或一句 `equivalent to: git diff <merge-base>...HEAD`,以团队可复现为准)。**三名 reviewer 的 Assignment 间该字段必须完全一致**;**@qa-engineer** 验证同一 feature 时 **复用同一 `plan_id` 与同一 `Review range` / `Diff basis`**。**热修 / QC 单审**路径也须含 **同一组字段**,仅承接方份数为 1。
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: mstar-coding-behavior
|
|
3
|
-
description: Morning Star (启明星) 跨角色通用编码行为准则 —— Think Before Coding(先读懂再改、显式假设、不静默猜测、读 imports/test/项目模式)、Simplicity First(YAGNI 优先不写代码、The Ladder 决策层级含依赖评估、删除优于添加、简洁优于聪明、5 项具名反模式速查、`simplify:` 标记天花板与升级路径、最小耐久切片)、Surgical Changes(改动可追溯、Bug 修根因先 grep 所有调用点、不顺手重构、不 piggyback)、Debugging(读完整报错与栈追踪、先复现、一步一测、根因分析、Bug 修复前先写复现测试、卡住时坦白)、Goal-Driven Execution(非平凡逻辑必留一个可运行检查、模糊请求转可验证结果、测行为不测实现、已有测试前后对比、不能测试说理由、Step → verify 微模板、分批留 roadmap)、Communication(说做了什么及为什么、标记顾虑、精确表达不确定性、不解释已知的、commit message 质量)。任何实现、调试、重构、审查任务都应优先 Read 本 skill;`@fullstack-dev` / `@frontend-dev` / `@fullstack-dev-2` / `@architect` / `@qa-engineer` / `@ops-engineer` / `@prompt-engineer` 动手前必读;QC 审查员核对变更是否只做了该做的手术时必读。本 skill 不覆盖分支门禁、QC/QA 路由、Assignment 权限、Done 所有权等不变量(那些以 `mstar-harness-core` 为准)。
|
|
3
|
+
description: Morning Star (启明星) 跨角色通用编码行为准则 —— Think Before Coding(先读懂再改、显式假设、不静默猜测、读 imports/test/项目模式)、Simplicity First(YAGNI 优先不写代码、The Ladder 决策层级含依赖评估、删除优于添加、简洁优于聪明、5 项具名反模式速查、`simplify:` 标记天花板与升级路径、最小耐久切片)、Surgical Changes(改动可追溯、Bug 修根因先 grep 所有调用点、不顺手重构、不 piggyback)、Debugging(读完整报错与栈追踪、先复现、一步一测、根因分析、Bug 修复前先写复现测试、卡住时坦白)、Review Feedback Handling(先核实反馈再改、逐项处理、可用证据反驳错误建议)、Goal-Driven Execution(非平凡逻辑必留一个可运行检查、模糊请求转可验证结果、测行为不测实现、已有测试前后对比、不能测试说理由、Step → verify 微模板、分批留 roadmap)、Communication(说做了什么及为什么、标记顾虑、精确表达不确定性、不解释已知的、commit message 质量)。任何实现、调试、重构、审查任务都应优先 Read 本 skill;`@fullstack-dev` / `@frontend-dev` / `@fullstack-dev-2` / `@architect` / `@qa-engineer` / `@ops-engineer` / `@prompt-engineer` 动手前必读;QC 审查员核对变更是否只做了该做的手术时必读。本 skill 不覆盖分支门禁、QC/QA 路由、Assignment 权限、Done 所有权等不变量(那些以 `mstar-harness-core` 为准)。
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
## Load order(必读顺序)
|
|
@@ -166,7 +166,31 @@ Micro template:
|
|
|
166
166
|
- **Test behavior, not implementation.** A test that checks whether a constructor sets properties is worthless. A test that checks whether validation actually rejects bad input is valuable. Focus on the interesting cases.
|
|
167
167
|
- **If you cannot write a test, say why.** "I cannot easily test this because the database calls are tightly coupled to the business logic" is useful information that may signal a need for restructuring. Do not skip testing without an explanation.
|
|
168
168
|
|
|
169
|
-
## 6)
|
|
169
|
+
## 6) Review Feedback Handling
|
|
170
|
+
|
|
171
|
+
Core idea: review feedback is technical input, not an order to perform unverified edits.
|
|
172
|
+
|
|
173
|
+
When receiving code review, QA, CI, or human feedback:
|
|
174
|
+
|
|
175
|
+
1. Read all feedback before editing.
|
|
176
|
+
2. Clarify ambiguous items before partial implementation.
|
|
177
|
+
3. Verify each suggestion against codebase reality.
|
|
178
|
+
4. Apply technically correct feedback one item at a time.
|
|
179
|
+
5. Test each fix individually where practical.
|
|
180
|
+
6. Push back with evidence when feedback is incorrect, obsolete, risky, out of scope, or violates YAGNI.
|
|
181
|
+
|
|
182
|
+
Feedback priority:
|
|
183
|
+
|
|
184
|
+
| Feedback type | Handling |
|
|
185
|
+
|---|---|
|
|
186
|
+
| Security, correctness, data loss, build/test failure | Fix or escalate before proceeding. |
|
|
187
|
+
| Scope mismatch, reviewer misunderstanding, obsolete assumption | Verify and push back with evidence. |
|
|
188
|
+
| Style-only suggestion | Apply only if it matches project conventions or is requested by the user/PM. |
|
|
189
|
+
| New feature disguised as review | Route through PM/plan unless explicitly in scope. |
|
|
190
|
+
|
|
191
|
+
Do not perform agreement. State the technical action, the verification result, or the technical reason for disagreement.
|
|
192
|
+
|
|
193
|
+
## 7) Communication
|
|
170
194
|
|
|
171
195
|
Core idea: how you communicate about code matters as much as the code itself.
|
|
172
196
|
|
|
@@ -178,8 +202,7 @@ Core idea: how you communicate about code matters as much as the code itself.
|
|
|
178
202
|
|
|
179
203
|
## Integration Notes
|
|
180
204
|
|
|
181
|
-
- This skill
|
|
182
|
-
- It must not be used to bypass:
|
|
205
|
+
- This skill must not be used to bypass:
|
|
183
206
|
- branch constraints,
|
|
184
207
|
- QC/QA gate definitions,
|
|
185
208
|
- assignment authority,
|
|
@@ -211,6 +211,8 @@ If `CONCEPTS.md` doesn't exist yet, ask whether to seed it. A seed populates the
|
|
|
211
211
|
|
|
212
212
|
2. If `plan_id` was provided, optionally update `status.json` metadata to reference this doc under `knowledge_refs`.
|
|
213
213
|
|
|
214
|
+
**iteration-close gate**: `mstar-iteration` §3.2 #5 — **each** new doc in the compound round must complete this phase; do not skip for lightweight captures.
|
|
215
|
+
|
|
214
216
|
## Phase 7: Refresh trigger
|
|
215
217
|
|
|
216
218
|
After capturing, check if the new learning suggests an older doc may now be stale (contradicted, superseded, or in a refactored domain). If so, recommend:
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: mstar-dispatch-gates
|
|
3
|
-
description: Morning Star 派发与委派门禁 —— 仅 PM 可增派 subagent、`Execute as` 与 `Delegation`、承接方反递归 NEVER 红线、同条消息 N 次 invoke、QC 三审禁止串行 rollout、Assignment 文案≠派发(invoke 条数须对齐)、未齐不发(emit zero until batch-ready)。**必须**在 `@project-manager` 每轮派发、QC 三审并发、双轨 implement、或 leaf 角色疑惑能否 Task 时 Read;所有非 PM 承接方动手前必读反递归与自检。同仓 worktree 与 QC 检出对齐见 `mstar-branch-worktree
|
|
3
|
+
description: Morning Star 派发与委派门禁 —— 仅 PM 可增派 subagent、`Execute as` 与 `Delegation`、承接方反递归 NEVER 红线、同条消息 N 次 invoke、QC 三审禁止串行 rollout、Assignment 文案≠派发(invoke 条数须对齐)、未齐不发(emit zero until batch-ready)。**必须**在 `@project-manager` 每轮派发、QC 三审并发、双轨 implement、或 leaf 角色疑惑能否 Task 时 Read;所有非 PM 承接方动手前必读反递归与自检。同仓 worktree 与 QC 检出对齐见 `mstar-branch-worktree`。宿主细则见 `mstar-host`(`references/parallel-dispatch.md` 与各宿主 reference)。
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
## Load order(必读顺序)
|
|
@@ -14,7 +14,7 @@ description: Morning Star 派发与委派门禁 —— 仅 PM 可增派 subagent
|
|
|
14
14
|
> **先回到你 Assignment 最顶部的 `**IDENTITY**` 块重读一遍** —— 那里已经告诉你:你是谁、你不是谁、Task 工具不属于你。本节是那个身份断言的加固版。
|
|
15
15
|
|
|
16
16
|
**身份断言(牢记,不动摇):**
|
|
17
|
-
- **你就是** `Execute as` 写的那个角色。你**不是 PM**,不是调度者,不是
|
|
17
|
+
- **你就是** `Execute as` 写的那个角色。你**不是 PM**,不是调度者,不是 parallel-agent dispatch 的使用者。
|
|
18
18
|
- Task/subagent 工具即使在你的工具列表里出现,它**不属于你**。把它当作你没见过的工具 —— 不是"不能用",是"你没有"。
|
|
19
19
|
- 所有工作由你**本人**在当前会话内完成(Read / Write / Edit / Shell / Grep / Glob)。任何"这样做更高效"的想法都不构成 dispatch 的理由。
|
|
20
20
|
- 遇到超出能力范围的需求 → 写 `## Blocked` 回报 PM,**不是**尝试 dispatch。Blocked 是你能发出的正确信号,dispatch 不是。
|
|
@@ -27,7 +27,7 @@ description: Morning Star 派发与委派门禁 —— 仅 PM 可增派 subagent
|
|
|
27
27
|
- **NEVER** 把 Assignment 里出现的 **任何** `@<role>`、反引号 `` `<role-id>` ``、**Handoff**、**QA note**、**Completion Report** 模板里的角色名、路由表下游角色当成「立刻 invoke」的指令;这些是**叙事 / 路由文档 / 后续 PM 编排意图**,不是命令。
|
|
28
28
|
- **NEVER** 把「分解为多个计划 / 多 phase / 多 track」等**设计产物层面**的并行或拆分读成「应 invoke 与子会话数量对应的多个 subagent」。**纸面产物**由本会话写盘完成;并行**调度**由 PM 在后续轮次决定。
|
|
29
29
|
- **NEVER** 因宿主**暴露**了 `Task` 或若干 `subagent_type` 名称就推断可以调用。**工具可用 ≠ 授权使用**;授权只来自 **`Delegation: allowed (...)`**。
|
|
30
|
-
- **NEVER**(非 PM)主动执行
|
|
30
|
+
- **NEVER**(非 PM)主动执行 parallel-agent dispatch 来分派子代理;需要并行时回报 PM。
|
|
31
31
|
- **DO NOT** 在 Assignment 缺少 `Execute as` / `Delegation` / `Who runs this turn` 时自行「补齐」为 PM;缺字段时按 **leaf executor** 解释:亲自完成或 **`Blocked`**。
|
|
32
32
|
- **DO NOT** 用「Assignment 太长 / 像编排稿」当作分派依据;先交付本会话任务再回报,分派由 PM 下一轮决定。
|
|
33
33
|
|
|
@@ -47,7 +47,7 @@ description: Morning Star 派发与委派门禁 —— 仅 PM 可增派 subagent
|
|
|
47
47
|
- 额外代理仅以 **`Delegation: allowed (...)`** 为准;未显式写时视为 **`Delegation: forbidden`**。
|
|
48
48
|
- Assignment 正文中的 `@xxx` 默认按「文本引用」解释,**不**视为自动调用命令。
|
|
49
49
|
- 承接方若判断必须增加 subagent,应先回报 **`Blocked`** 请 PM 重分派。
|
|
50
|
-
-
|
|
50
|
+
- Per-task informal review, when PM explicitly allows it, must not use `@qc-specialist*`; use `@general` / `generalPurpose` or PM-marked informal `@qa-engineer`. Formal QC remains `mstar-review-qc`.
|
|
51
51
|
|
|
52
52
|
## 并发分派完整性门禁(PM 强制)
|
|
53
53
|
|
|
@@ -70,11 +70,20 @@ description: Morning Star 派发与委派门禁 —— 仅 PM 可增派 subagent
|
|
|
70
70
|
- **QC 三审**在 feature 开发完成后执行;三名 reviewer 共用同一组 `Review cwd` / `Working branch` / `plan_id` / `Review range` / `Diff basis`(**`mstar-branch-worktree`**)。
|
|
71
71
|
- **同一 plan 多 batch**:默认整 plan 交付完成跑一轮完整三审;**fix 后默认 targeted re-review**(N = 被指派的 QC 席位数,同条消息发满 N);**仅** Assignment 写明 **`QC re-review: full tri-review`** 时复跑三审且用新文件名(**`mstar-plan-artifacts/references/plan-files-and-reports.md`**、**`mstar-review-qc`**)。
|
|
72
72
|
|
|
73
|
+
## iteration-start Review & Edit chain
|
|
74
|
+
|
|
75
|
+
与 QC 三审同理:**dispatch-first** 适用于文档审查角色,不限于 implement/QC。
|
|
76
|
+
|
|
77
|
+
- PM 写 compass/plans **初稿**;**@product-manager**、**@architect**、**@writing-specialist** 各通过 Task **直接编辑** compass / plans / specs(**不**另写 `reports/` 审查报告)。
|
|
78
|
+
- PM 线程跳过派发、自行完成三角色全部编辑 = **反模式**(见 **`mstar-harness-core`** 反模式索引、`mstar-iteration` §1.6)。
|
|
79
|
+
- Steps 5.1–5.3 须 **Task**(可并行);PM lock(5.4)在 PM 线程完成。**不得**在 review chain 完成前 commit integration 分支。
|
|
80
|
+
|
|
73
81
|
## 反模式(派发)
|
|
74
82
|
|
|
75
83
|
- QC 三审拆在多条消息或等 #1 返回再发 #2/#3。
|
|
76
84
|
- 仅 1 次 invoke 却声称「并行三审已启动」。
|
|
77
85
|
- 递归同角色 subagent;把 Handoff / 多轨编排措辞当 invoke。
|
|
86
|
+
- iteration-start 在 Review & Edit chain 完成前 commit;PM 代做三角色全部编辑而不派发 Task。
|
|
78
87
|
|
|
79
88
|
## References
|
|
80
89
|
|
|
@@ -16,7 +16,7 @@ Before any Task/subagent call (if I somehow forget the preamble):
|
|
|
16
16
|
4. Is `subagent_type` equal to my `Execute as`? If yes → **forbidden** (recursive dispatch).
|
|
17
17
|
5. Am I treating `@roles`, Handoff, QA note, Completion Report roles, or multi-plan/multi-track **design text** as invoke commands? If yes → **stop**; deliver in-session.
|
|
18
18
|
6. Am I invoking because the tool exists? **Available ≠ authorized.**
|
|
19
|
-
7. Need parallel work or PM-only
|
|
19
|
+
7. Need parallel work or PM-only dispatch? → **`Blocked`**; PM dispatches on the next round.
|
|
20
20
|
|
|
21
21
|
If blocked, report: `## Blocked — recursive dispatch refused (<which NEVER or reason>)`
|
|
22
22
|
|
|
@@ -33,7 +33,7 @@ description: Morning Star (启明星) harness **强制全局入口** —— 信
|
|
|
33
33
|
| 角色 | 始终 | 按任务追加(典型) |
|
|
34
34
|
|------|------|-------------------|
|
|
35
35
|
| **全部** | 本 skill | — |
|
|
36
|
-
| **`@project-manager`** | 本 skill | `mstar-dispatch-gates`、`mstar-phase-gates`、`mstar-plan-conventions`、`mstar-
|
|
36
|
+
| **`@project-manager`** | 本 skill | `mstar-dispatch-gates`、`mstar-phase-gates`、`mstar-plan-conventions`、`mstar-roles`;派 QC 前 `mstar-review-qc`;并行/审查 `mstar-branch-worktree`;plan/status/reports `mstar-plan-artifacts`;UI 类 plan Prepare 阶段 `mstar-design-md`(DESIGN.md 门禁);新建/大改 skill 时 `mstar-skill-authoring`;迭代管理 `mstar-iteration`(iteration-start/drive/close 含 compound);战略性工作 `mstar-strategy`。**不**读 `mstar-coding-behavior` |
|
|
37
37
|
| **实现/审查/QA/运维** | 本 skill + `mstar-coding-behavior` + 角色 ref | 有 git 写:`mstar-branch-worktree`;有 plan 路径:`mstar-plan-conventions`(路径符号节);QC/QA:`mstar-review-qc`;改 status/residual:`mstar-plan-artifacts`;UI 任务:`mstar-design-md`(读取 DESIGN.md tokens);写入知识库 `{KNOWLEDGE_DIR}`:`mstar-compound`(PM 触发) |
|
|
38
38
|
| **leaf 承接方** | 上栏 + **`mstar-dispatch-gates`**(反递归节) | — |
|
|
39
39
|
|
|
@@ -71,7 +71,7 @@ PM 在 Assignment 写 **`Task category`**(主类 + 可选 `secondary`):
|
|
|
71
71
|
|
|
72
72
|
## 长任务纪律
|
|
73
73
|
|
|
74
|
-
可追踪清单(plan `tasks` 或 Todo);偏离时 PM
|
|
74
|
+
可追踪清单(plan `tasks` 或 Todo);偏离时 PM 拉回;完成前须可核对证据(实现侧自检见 **`mstar-coding-behavior`**;门禁证据见 **`mstar-phase-gates`** / **`mstar-review-qc`**)。
|
|
75
75
|
|
|
76
76
|
**Durable Roadmap Gate**:凡声明“分批 / 后续 / next plan / later / temporary workaround”的非热修任务,必须在 `{PLAN_DIR}` 主 plan、CreatePlan mirror、`status.json`/residual、或 PM Task Board 中写清后续路线(批次、依赖、owner/触发条件、完成定义)。只在对话或 Completion Report 里说“以后做”不算可追踪,不能进入 implement GO 或 Done。
|
|
77
77
|
|
|
@@ -88,11 +88,11 @@ PM 在 Assignment 写 **`Task category`**(主类 + 可选 `secondary`):
|
|
|
88
88
|
| `mstar-plan-artifacts` | 主 plan、reports、`status.json`、residual、knowledge、Done 归档 |
|
|
89
89
|
| `mstar-design-md` | DESIGN.md 设计系统规范 —— 创建/审计/维护 design tokens,三级检查清单,light/dark 双主题 |
|
|
90
90
|
| `mstar-review-qc` | QC 工作流、模板、verdict、residual 留档、deep review 透镜 |
|
|
91
|
-
| `mstar-coding-behavior` | Think / Simplicity / Surgical / Debugging / Goal-Driven / Communication |
|
|
91
|
+
| `mstar-coding-behavior` | Think / Simplicity / Surgical / Debugging / Review Feedback / Goal-Driven / Communication |
|
|
92
92
|
| `mstar-compound` | 知识结晶 —— 已解决问题→结构化知识文档,双轨(Bug/Knowledge),「是否值得结晶」自检清单,重叠检测,可发现性检查,CONCEPTS.md 协同 |
|
|
93
93
|
| `mstar-compound-refresh` | 知识维护 —— 审查/更新/合并/删除 `{KNOWLEDGE_DIR}` 文档 |
|
|
94
94
|
| `mstar-strategy` | `STRATEGY.md` 全局战略方向 —— 产品愿景、技术方向、决策原则 |
|
|
95
|
-
| `mstar-
|
|
95
|
+
| `mstar-skill-authoring` | mstar-native skill authoring: trigger contracts, progressive disclosure, pressure scenarios, behavior-change evidence |
|
|
96
96
|
| `mstar-roles` | 角色正文 hub |
|
|
97
97
|
| `mstar-host` | 宿主适配(自动识别;`references/opencode.md` / `cursor.md` / `codex.md` / `parallel-dispatch.md`) |
|
|
98
98
|
|
|
@@ -114,7 +114,7 @@ Read **`mstar-host`** after this skill; detect host per its table, then Read the
|
|
|
114
114
|
- 业务仓默认功能分支(Assignment `Branch policy` 例外)→ **`mstar-branch-worktree`**。
|
|
115
115
|
- **Dev 三角**:`@fullstack-dev` 后端主导;UI → `@frontend-dev`;第二轨 → `@fullstack-dev-2`(`mstar-roles` PM 节)。
|
|
116
116
|
- 工期仅 agent-oriented → **`mstar-plan-conventions`** · effort-estimation。
|
|
117
|
-
-
|
|
117
|
+
- plan-writing artifacts land in `{PLAN_DIR}`, not external default plan directories.
|
|
118
118
|
- PM Assignment 键名英文;任务正文可中文;产出/报告默认英文。
|
|
119
119
|
|
|
120
120
|
## 库文档(Context7)
|
|
@@ -139,6 +139,9 @@ Read **`mstar-host`** after this skill; detect host per its table, then Read the
|
|
|
139
139
|
| 无证据宣称完成 | `mstar-coding-behavior` / verification |
|
|
140
140
|
| CreatePlan 不落盘 / 无 `{HARNESS_DIR}` mirror | `mstar-host` · `cursor-plan-mode-bridge` |
|
|
141
141
|
| 临时方案 / 后续计划只写在对话里 | `mstar-phase-gates` · Durable Roadmap Gate |
|
|
142
|
+
| iteration-start commit before review chain | `mstar-iteration` §1.6;PM 模拟三角色编辑而不派发 Task |
|
|
143
|
+
| Phase 3 折叠进 final plan closure / 跳过 §3.1 gate | `mstar-iteration` §3.0–§3.5;`iteration-drive` Phase 3 |
|
|
144
|
+
| iteration-close 无 frontmatter completed / 漏 compound Phase 6 | `mstar-iteration` §3.0.5、§3.4、§3.2 #5 |
|
|
142
145
|
|
|
143
146
|
## 可选:OpenViking Memory
|
|
144
147
|
|
|
@@ -13,9 +13,9 @@
|
|
|
13
13
|
| **`clarify` 核心纪律** | 逐方面核对至共享理解;沿设计决策树逐枝、一次一决;能探索代码库则先探索;每问带推荐答案 | **`mstar-phase-gates`** SKILL.md(含 Phase Gate Playbook) |
|
|
14
14
|
| **按任务类别选能力与模型** | 视觉/深读/快改/硬逻辑等用不同强项 | `mstar-harness-core` SKILL.md「Task category」;Assignment 字段 **`Task category`**;宿主侧按角色配置 model(如 OpenCode 的 `opencode.json`) |
|
|
15
15
|
| **可验证编辑** | 减少「凭记忆 Patch」导致的漂移与损坏 | `mstar-phase-gates` SKILL.md「可验证编辑与上下文纪律」:读后再改、失败则重读 |
|
|
16
|
-
| **持续推进与可核对完成** | 长任务有清单、有关门证据,避免空转 | `mstar-
|
|
16
|
+
| **持续推进与可核对完成** | 长任务有清单、有关门证据,避免空转 | `mstar-phase-gates` / `mstar-review-qc`;实现侧自检见 `mstar-coding-behavior` |
|
|
17
17
|
| **编码行为约束(轻量)** | 降低静默假设、过度设计与无关改动 | `mstar-coding-behavior`:Think Before Coding / Simplicity First / Surgical Changes / Goal-Driven Execution |
|
|
18
|
-
| **并行与边界** | 多线任务不踩同一写归属、不绕过分支门禁;**开发**阶段同仓多可写并发须独立 **`git worktree`**;**QC / QA** 在 **同一检出**(`Review cwd`)与 **同一 `plan_id` + `Review range` / `Diff basis`** 上审查与验证,保证三票同一功能 | `mstar-branch-worktree`;`mstar-review-qc`;`mstar-
|
|
18
|
+
| **并行与边界** | 多线任务不踩同一写归属、不绕过分支门禁;**开发**阶段同仓多可写并发须独立 **`git worktree`**;**QC / QA** 在 **同一检出**(`Review cwd`)与 **同一 `plan_id` + `Review range` / `Diff basis`** 上审查与验证,保证三票同一功能 | `mstar-branch-worktree`;`mstar-review-qc`;`mstar-dispatch-gates` |
|
|
19
19
|
| **分层上下文(可选)** | 大仓库用目录级 `AGENTS.md` 降噪;根 `AGENTS.md` 维护边界见下文专节 | `mstar-harness-core` SKILL.md「分层上下文」;由业务项目维护者按需添加 |
|
|
20
20
|
| **结构化澄清(按宿主)** | 向用户澄清/抉择时,**有 `question` 类能力则优先**;否则结构化正文;长问兜底 | `mstar-phase-gates` SKILL.md Prepare · `clarify`;当前宿主的 `mstar-host` skill;`mstar-roles` · `project-manager` |
|
|
21
21
|
|
|
@@ -71,6 +71,6 @@
|
|
|
71
71
|
## 延伸阅读
|
|
72
72
|
|
|
73
73
|
- 按能力选配 MCP/skills:OpenCode → `mstar-host` `references/opencode.md`;其它宿主见 `mstar-host` 检测表
|
|
74
|
-
-
|
|
74
|
+
- Evidence and checkpoints: `mstar-phase-gates` / `mstar-review-qc`; implementation self-checks: `mstar-coding-behavior`
|
|
75
75
|
- 库文档检索共享协议:`mstar-harness-core` `references/library-docs-protocol.md`
|
|
76
76
|
- 跨角色编码行为准则(轻量、可复用):`mstar-coding-behavior`
|
|
@@ -197,7 +197,7 @@ When `/pm` runs under Plan mode:
|
|
|
197
197
|
|
|
198
198
|
## Related skills
|
|
199
199
|
|
|
200
|
-
- `mstar-plan-conventions` — discovery, init,
|
|
200
|
+
- `mstar-plan-conventions` — discovery, init, plan-writing path gate
|
|
201
201
|
- `mstar-plan-artifacts` — `status.json`, reports, checkboxes, residual
|
|
202
202
|
- `mstar-phase-gates` — Prepare / Execute order
|
|
203
203
|
- `mstar-roles/references/project-manager/dispatch-and-assignment.md` — Checkpoint: commit → Completion Report → Status Update
|
|
@@ -61,9 +61,9 @@ Enforcement: `rules/mstar-cursor-plan-mode.mdc` when plugin active.
|
|
|
61
61
|
- No `question` tool: structured Markdown or Cursor UI.
|
|
62
62
|
- “Question asked” ≠ clarify done; high-impact ambiguity → `Blocked` or escalation.
|
|
63
63
|
|
|
64
|
-
##
|
|
64
|
+
## Execution Discipline
|
|
65
65
|
|
|
66
|
-
|
|
66
|
+
Implementation roles use `mstar-coding-behavior` for RCA, test-first checks, review feedback, and completion evidence. Plan checkpoints remain in `mstar-phase-gates`; external skill plugins are not required.
|
|
67
67
|
|
|
68
68
|
## Gotchas
|
|
69
69
|
|
|
@@ -8,7 +8,7 @@ Parallel PM dispatch: **`parallel-dispatch.md`** (read in dispatch rounds).
|
|
|
8
8
|
|
|
9
9
|
- **Role shell**: `agents/<id>.md` referenced by `opencode.json` `agent.<id>` (frontmatter + role binding only).
|
|
10
10
|
- **Role body**: `mstar-roles` `references/<id>.md` (or shared references + parameters).
|
|
11
|
-
-
|
|
11
|
+
- Implementation evidence and RCA behavior: `mstar-coding-behavior`.
|
|
12
12
|
|
|
13
13
|
## OpenCode-specific capabilities
|
|
14
14
|
|
|
@@ -48,7 +48,7 @@ Aligned with `mstar-harness-core` `references/open-harness-principles.md`. Editi
|
|
|
48
48
|
| **Repo graph** | Impact / PR risk | e.g. GitNexus |
|
|
49
49
|
| **Browser / E2E** | Observable QA evidence | agent-browser, Playwright |
|
|
50
50
|
| **Git workflow** | Atomic commits, branch closure | git-commit, finishing-a-development-branch |
|
|
51
|
-
| **Systematic debugging** | RCA before fix | `mstar-
|
|
51
|
+
| **Systematic debugging** | RCA before fix | `mstar-coding-behavior` |
|
|
52
52
|
| **OpenViking memory** | Long-term memory tools | Only if `memsearch` present; `openviking-memory-plugin.md` |
|
|
53
53
|
|
|
54
54
|
**Not recommended**: overlapping search MCPs; extra tools to mask missing harness baseline.
|