@mstar-harness/opencode 1.0.5 → 1.1.0

package/harness-skills/mstar-roles/references/qc-specialist-shared.md

@@ -18,7 +18,7 @@ Before any non-trivial QC assignment, read in order:
 
 1. `mstar-harness-core`
 2. `mstar-dispatch-gates` + `mstar-branch-worktree`
-3. `mstar-plan-artifacts` (report paths and naming)
+3. `mstar-plan-artifacts` (review bundle paths and naming)
 4. Host: `mstar-host` → active host reference
 5. **`references/qc-specialist/reviewer-workflow.md`**
 6. **`references/qc-specialist/reviewer-checklist.md`**
@@ -49,8 +49,8 @@ If any item below matches, **stop** and return `Blocked` to `project-manager`:
 
 - **NEVER** invoke another QC seat or `{role_id}` again, nor `qa-engineer` / dev / `architect` / `project-manager`, unless `Delegation: allowed (...)` lists them.
 - **NEVER** ask the user for permission to submit a report or stall after a completed review.
-- **NEVER** modify business implementation/tests, `status.json` residual fields, or paths outside QC report whitelist.
-- **NEVER** `git add .` — stage **only** report files you changed.
+- **NEVER** modify business implementation/tests, `status.json` residual fields, or paths outside the Assignment-specified QC report path.
+- **NEVER** `git add .` or commit raw bundle reports by default.
 - **NEVER** close or archive residual entries in `status.json` from QC.
 - **NEVER** treat `Handoff` or routing prose as invoke instructions.
 - **NEVER** infer tool exposure implies authorization.
@@ -76,13 +76,13 @@ See **`references/qc-specialist/report-template.md`**. Machine **`severity`** en
 
 ## Report path (required)
 
-Write **`{PLAN_DIR}/reports/<plan-id>/{report_suffix}.md`** (tri: `qc1`…`qc3`; inline: `qc.md`). No `<plan-id>` filename prefix.
+Write the Assignment-provided path under **`{SDD_DIR}/review/{report_suffix}.md`** (tri: `qc1`…`qc3`; inline: `qc.md`). No `<plan-id>` filename prefix.
 
 ## Targeted re-review (same report file)
 
 When Assignment includes **`QC re-review: targeted`**:
 
-- Edit the **same** `{report_suffix}.md` — add **`## Revalidation`**, update frontmatter verdict/`generated_at`.
+- Edit the **same** bundle `{report_suffix}.md` — add **`## Revalidation`**, update frontmatter verdict/`generated_at`.
 - Do **not** create `qcN-rev2.md` on this path.
 - Full tri re-review → new basenames per `mstar-plan-artifacts/references/plan-files-and-reports.md`.
 
@@ -118,11 +118,11 @@ generated_at: "YYYY-MM-DD"
 
 ## Repository Write Scope
 
-QC may write only `{PLAN_DIR}/reports/**/*.md` per assignment.
+QC may write only the Assignment-specified review bundle `.md` path under `{SDD_DIR}/review/`.
 
 ### Git NEVER (QC reports)
 
-- **NEVER** claim complete without `git add` (report paths only) + `git commit` when required — real `git log -1 --oneline` in Completion Report.
+- **NEVER** commit raw `{SDD_DIR}/review/` reports unless Assignment explicitly says `Review archive mode: tracked reports`.
 - **NEVER** `git add .`
 
 ## Detailed References Index

package/harness-skills/mstar-sdd/SKILL.md

@@ -45,7 +45,7 @@ Batch all findings for the human in one message. If clean, proceed silently.
    - **`SDD implementer session: fresh`** (default) — new subagent; templates: `references/implementer-prompt.md`
    - **`SDD implementer session: sticky`** — first task: same as fresh + write `{SDD_DIR}/implementer-session.json` with `host_agent_id`; later tasks: host **resume** + `references/implementer-continuation-prompt.md` (see **`references/sticky-implementer-session.md`**)
 5. On `DONE`: `review-package BASE HEAD` → diff file
-6. Dispatch **fresh** task reviewer (brief, report, diff, Global Constraints) — `references/task-reviewer-prompt.md` — **never** sticky resume for reviewers
+6. Dispatch **fresh** task reviewer — **`subagent_type: generalPurpose`** (L2; **not** `qc-specialist*`) — brief, report, diff, Global Constraints — `references/task-reviewer-prompt.md` — **never** sticky resume for reviewers
 7. Fix loop for Critical/Important; re-review until approved
 8. Append `progress.md`; update `status.json` `task_commits[]` and `implementer-session.json` `last_task` if sticky
 9. Next task
@@ -81,8 +81,8 @@ Host mapping → **`mstar-host`** references (`model` / Task field).
 
 ## After all tasks
 
-1. `review-package MERGE_BASE HEAD` → branch diff in `{SDD_DIR}`
-2. PM dispatches **plan QC tri-review (L3)** — **`QC mode: full tri-review`**, **N=3** — with branch review-package path → **`mstar-review-qc`** · **`mstar-dispatch-gates`**. Layer SSOT → **`mstar-review-qc/references/review-responsibility-boundaries.md`**. PM writes `qc-consolidated.md`. **Mandatory whenever `Execution mode: sdd`** (single-plan or iteration).
+1. `review-package MERGE_BASE HEAD` → branch diff in `{SDD_DIR}/review/`
+2. PM dispatches **plan QC tri-review (L3)** — **`QC mode: full tri-review`**, **N=3** — with branch review-package path and report paths under `{SDD_DIR}/review/` → **`mstar-review-qc`** · **`mstar-dispatch-gates`**. Layer SSOT → **`mstar-review-qc/references/review-responsibility-boundaries.md`**. PM writes `{SDD_DIR}/review/qc-consolidated.md` and durable main-plan gate summary. **Mandatory whenever `Execution mode: sdd`** (single-plan or iteration).
 3. Critical/Important QC findings → **one** fix dispatch (full list), then targeted re-review
 4. QA gate → **`mstar-harness-core`** Done rules; PM **`mstar-roles/references/project-manager/qa-trigger-matrix.md`**
 

package/harness-skills/mstar-sdd/references/file-handoffs.md

@@ -58,10 +58,11 @@ After all tasks:
 
 ```bash
 MERGE_BASE=$(git merge-base <target-branch> HEAD)
-skills/mstar-sdd/scripts/review-package "$MERGE_BASE" HEAD "$SDD_DIR/branch-review-....diff"
+mkdir -p "$SDD_DIR/review"
+skills/mstar-sdd/scripts/review-package "$MERGE_BASE" HEAD "$SDD_DIR/review/branch-review-....diff"
 ```
 
-Pass **branch** diff path to QC dispatch — not task-level diffs.
+Pass **branch** diff path and bundle report paths (`$SDD_DIR/review/qc1.md` …) to QC dispatch — not task-level diffs. Raw QC/QA files stay in the gitignored review bundle; PM records durable summary and residuals in tracked plan/status artifacts.
 
 ## PM context hygiene
 

package/harness-skills/mstar-sdd/references/task-reviewer-prompt.md

@@ -4,6 +4,7 @@ One reviewer per task: spec compliance + code quality (`mstar-sdd`).
 
 ```
 Task / subagent:
+  subagent_type: generalPurpose
   description: "SDD review Task N (spec + quality)"
   model: [REQUIRED — standard tier default; capable if diff is large/subtle]
   prompt: |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mstar-harness/opencode",
-  "version": "1.0.5",
+  "version": "1.1.0",
   "description": "Morning Star harness OpenCode plugin (skills bootstrap and agent loading).",
   "license": "MIT",
   "repository": {