@mstar-harness/opencode 3.3.0 → 3.4.0

package/harness-skills/mstar-roles/references/frontend-dev.md

@@ -3,6 +3,7 @@
 
 You are the frontend implementation owner for UI/components/interactions/accessibility/performance.
 You are dispatched by `project-manager` and report back with completion evidence.
+YAGNI is your coding philosophy; PDCA is your behavioral discipline.
 
 ## Non-Recursive Dispatch Rule (Hard)
 

package/harness-skills/mstar-roles/references/fullstack-dev-shared.md

@@ -11,8 +11,9 @@ Behavior is shared; track identity is parameterized.
 
 ## Role Mission
 
-Backend-led fullstack implementation with contract-aware collaboration.
-Dispatched by `project-manager`; returns completion report and evidence.
+You are `{role_id}`, a backend-led fullstack implementation role with contract-aware collaboration.
+You are dispatched by `project-manager` and return a completion report and evidence.
+YAGNI is your coding philosophy; PDCA is your behavioral discipline.
 
 ## Non-Recursive Dispatch Rule (Hard)
 

package/harness-skills/mstar-roles/references/ops-engineer.md

@@ -2,7 +2,7 @@
 ## Role Mission
 
 You are the operations/deployment role.
-Dispatched by `project-manager`; responsible for execution safety, observability, and rollback readiness.
+You are dispatched by `project-manager`, owning execution safety, observability, and rollback readiness.
 
 ## Non-Recursive Dispatch Rule (Hard)
 

package/harness-skills/mstar-roles/references/prompt-engineer.md

@@ -1,7 +1,7 @@
 
 ## Role Mission
 
-You design and optimize prompts, skills, and rules.
+You are `prompt-engineer`: you design and optimize prompts, skills, and rules.
 You are dispatched by `project-manager` and return structured prompt/rule artifacts with validation notes.
 
 ## Non-Recursive Dispatch Rule (Hard)

package/harness-skills/mstar-roles/references/qa-engineer.md

@@ -6,7 +6,7 @@ Detailed L4 procedures: `references/qa-engineer/*.md`.
 
 ## Role Mission
 
-L4 **acceptance seat**: map plan DoD to evidence, verify residuals when assigned, return reproducible QA outputs. PM dispatches you only when Assignment says **`QA gate: mandatory`** or **`QA gate: report-only`** (`references/project-manager/qa-trigger-matrix.md`).
+You are `qa-engineer`, the L4 **acceptance seat**: map plan DoD to evidence, verify residuals when assigned, return reproducible QA outputs. You are dispatched by `project-manager` only when Assignment says **`QA gate: mandatory`** or **`QA gate: report-only`** (`references/project-manager/qa-trigger-matrix.md`).
 
 ## Non-Recursive Dispatch Rule (Hard)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mstar-harness/opencode",
-  "version": "3.3.0",
+  "version": "3.4.0",
   "description": "Morning Star harness OpenCode plugin — skills bootstrap + engine-backed runtime hooks (status lint, dispatch validation, Enforcement: hard gates).",
   "license": "MIT",
   "repository": {

package/harness-commands/pr-deep-review.md

@@ -1,39 +0,0 @@
----
-name: pr-deep-review
-description: Use when asked to deeply review a pull request, branch, or diff before merge — deciding whether a change is safe to ship with evidence-backed findings, rather than a shallow "looks good" pass. Produces a `ship it` / `needs fixes` / `blocked` verdict. Also for a batch of sibling PRs. Do not use for self-checking a change you just authored.
-agent: project-manager
-input: "[pr|branch|scope]"
----
-
-# Deep PR Review
-
-Run a read-only, evidence-first deep review of a pull request, branch, or diff and decide whether it is safe to ship. When a PR number exists, posting the GitHub Review is **mandatory** — the review is not complete until comments land on the PR. Output: verdict + findings presented to the user, plus the posted review URL. Never auto-approve, never REQUEST_CHANGES, never merge.
-
-The verdict is **computed from the finding tally** (`must-fix` / `should-fix` / `nit` + `unverified`); `score_pct` is display-only feedback and never overrides it. Procedure and formula → **`references/pr-review.md`** § Verdict synthesis / Tally and derived score.
-
-**Read-only advisory.** The review does not enter the harness plan state machine (`Todo → InProgress → InReview → Done`). Reviewers never edit the worktree, never merge, and never approve-as-merge.
-
-## Boot
-
-1. `mstar-harness-core`
-2. `mstar-audit` → SKILL.md（common core）+ `references/pr-review.md`（`pr` variant 全量）
-3. `mstar-coding-behavior` (evidence discipline)
-4. `mstar-branch-worktree` (worktree isolation)
-5. `mstar-host` → active host reference (invoke capability for parallel subagents)
-
-## Routing（谁执行 review）
-
-| Context | Who runs the review |
-|---------|-------------------|
-| **Small PR / single pass** | PM dispatches a single `@code-reviewer` — review, then vet and synthesize the verdict |
-| **Batch of sibling PRs** | PM 按 PR 业务信息（业务域 / 变更面 / 技术栈）**平均分配**到四个席位：`@code-reviewer`（general）、`@fullstack-dev`、`@fullstack-dev-2`、`@frontend-dev` — 每个席位承载约 N/4 个 PR，摊薄同模型并发，降低 rate-limit。All worktrees created first, then all reviewers dispatched in one batch; each reviewer owns review + comment for its PRs only |
-
-All review seats (`code-reviewer` / `fullstack-dev` / `fullstack-dev-2` / `frontend-dev`) are **read-only** in this flow: never edit the reviewed worktree, never merge, never approve-as-merge. Implementer seats run in **Audit Mode** (shared contract → `mstar-roles` `references/_shared/leaf-executor-core.md`). PM dispatches; each reviewer executes the `pr` variant and returns findings + verdict to PM for consolidation.
-
-## Execute
-
-Execute **`mstar-audit`** § `pr` variant end to end（SKILL.md common core：recon + three-way attack & vet；variant detail：**`references/pr-review.md`** —— scope → guidance load → concern lenses → evidence → verdict → output）. Review is run in a dedicated worktree against a diff from the PR's **real base** — resolve the base per `references/pr-review.md` § Worktree isolation (never assume `main`).
-
-Review findings that need fixing can be turned into self-contained plans for the normal Prepare → Execute flow (reusing `mstar-audit` SKILL.md **`## Plan output (all variants)`** — same contract as the `pr` variant).
-
-Output verdict + findings to the user, with the posted GitHub Review URL. Posting procedure (when a PR number exists) → **`references/pr-review.md`** § Comment posting; each reviewer also saves the durable local report → **`references/pr-review.md`** § Local report archive. Never auto-approve or merge.