@polymorphism-tech/morph-spec 8.2.0 → 8.2.1

package/README.md

@@ -3,7 +3,7 @@
 > Spec-driven development framework for multi-stack projects. Turns feature requests into implementation-ready code through structured, AI-orchestrated phases.
 
 **Package:** `@polymorphism-tech/morph-spec`
-**Version:** 8.2.0  
+**Version:** 8.2.1  
 **Requires:** Node.js 22+, Claude Code
 
 ---
@@ -394,4 +394,4 @@ Code generated by morph-spec (contracts, templates, implementation output) belon
 
 ---
 
-*morph-spec v8.2.0 by [Polymorphism Tech](https://polymorphism.tech)*
+*morph-spec v8.2.1 by [Polymorphism Tech](https://polymorphism.tech)*

package/bin/morph-spec.js

@@ -186,7 +186,7 @@ program
 program
   .command('score <feature> <task> <score>')
   .description('Record a task eval score in feature.json (taskScores, server-side timestamp)')
-  .option('--dims <dims>', 'Four comma-separated dimension scores in order a,c,q,t (architecture,contracts,code-quality,test-coverage)')
+  .option('--dims <dims...>', 'Four dimension scores in order a,c,q,t (architecture,contracts,code-quality,test-coverage) — comma- or space-separated; quote under PowerShell: --dims "9.5,9,9,9"')
   .option('--json', 'Output as JSON')
   .action((feature, task, score, options) => scoreCommand(feature, task, score, options));
 

package/claude-plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "morph-spec",
-  "version": "8.2.0",
+  "version": "8.2.1",
   "displayName": "MORPH-SPEC Framework",
   "description": "Spec-driven development with 38 agents and 8-phase workflow for .NET/Blazor/Next.js/Azure",
   "publisher": "polymorphism-tech",

package/docs/CHEATSHEET.md

@@ -239,4 +239,4 @@ These files are never edited directly. Use CLI commands or `morph-spec update` i
 
 ---
 
-*morph-spec v8.2.0 by Polymorphism Tech*
+*morph-spec v8.2.1 by Polymorphism Tech*

package/docs/QUICKSTART.md

@@ -256,4 +256,4 @@ morph-spec doctor
 
 ---
 
-*morph-spec v8.2.0 by Polymorphism Tech*
+*morph-spec v8.2.1 by Polymorphism Tech*

package/framework/CLAUDE.md

@@ -26,7 +26,7 @@ Se não houver feature ativa, pergunte ao usuário qual feature trabalhar.
 - Apresentar gates com o template fixo definido em MORPH.md / morph-plan.
 - Registrar decisões não triviais em `decisions.md`.
 - Aplicar `morph-eval` após cada tarefa; silêncio em score ≥ 9 é o sinal correto. Score < 9 dispara o loop de correção autônomo (re-implementa, sem pausa), não escala ao humano entre tarefas. No Gate 3, o score composto da feature vem do **avaliador independente** despachado pela `morph-review` (persona `evaluator`, contexto fresco) — nunca de auto-avaliação de quem implementou.
-- Incluir as 3 cláusulas do contrato do relatório final em todo dispatch de sub-agent (**ato final = o relatório é a ÚLTIMA MENSAGEM DE TEXTO do sub-agent, com os 4 itens obrigatórios (o quê / arquivos / problemas / auto-score 0-10) — ela é o valor de retorno e chega inteira ao orquestrador; não chamar `SendMessage`, que gera um evento a mais e custa um turno inútil**; não tocar tasks.json/feature.json/board; sem build/test da suíte compartilhada em batch paralelo) — templates em `morph-implement` §4 e `morph-apply` §3b.
+- Incluir as 3 cláusulas do contrato do relatório final em todo dispatch de sub-agent (**o sub-agent GRAVA o relatório em arquivo canônico ANTES de encerrar — `.morph/features/{f}/3-implement/reports/{taskId}.md`, ou `4-review/evaluator-report.md` para o avaliador — com os 4 itens obrigatórios (o quê / arquivos / problemas / auto-score 0-10); esse arquivo é o valor de retorno autoritativo, a última mensagem de texto é cortesia best-effort e pode não chegar inteira; ainda assim não chamar `SendMessage`, que gera um evento a mais e custa um turno inútil**; não tocar tasks.json/feature.json/board; sem build/test da suíte compartilhada em batch paralelo) — templates em `morph-implement` §4 e `morph-apply` §3b. **O orquestrador lê o arquivo ao retomar controle** — nunca confia só na mensagem de retorno da tool.
 
 ## Slash commands
 
@@ -45,7 +45,7 @@ Se não houver feature ativa, pergunte ao usuário qual feature trabalhar.
 - `morph-spec advance <feature>` — aprova o gate da fase corrente **e** cria a pasta da próxima fase num passo só (`--design` toma o ramo opcional de uiux a partir da proposta).
 - `morph-spec archive <feature>` — arquiva a feature (atômico, git-aware); é o que `/morph-archive` chama.
 - `morph-spec dag <feature>` — análise do DAG de tasks.json com decisão sequencial vs paralelo **ponderada por effort** (raízes S vão inline, nunca ganham agente próprio).
-- `morph-spec score <feature> <task> <score> [--dims a,c,q,t]` — grava `taskScores` no feature.json com timestamp server-side; **nunca edite taskScores à mão**.
+- `morph-spec score <feature> <task> <score> [--dims a,c,q,t]` — grava `taskScores` no feature.json com timestamp server-side; **nunca edite taskScores à mão**. Em PowerShell, cite o valor de `--dims` entre aspas (`--dims "9.5,9,9,9"`) — sem aspas, a lista pode chegar como tokens separados; o parser aceita ambas as formas, mas citar evita qualquer ambiguidade.
 - `morph-spec gate-decision <feature> <gate> --pause <bool> [--reason <txt>] [--signals <json>]` — grava `gateDecisions` no feature.json com timestamp server-side (shape em `framework/schemas/feature.schema.json`); **nunca edite gateDecisions à mão** — o Edit manual colide com o hook state-sync.
 - `morph-spec create <feature> --type <feature|bug|chore|hotfix> [--request "<txt>"] [--worktree]` — cria a estrutura da feature + `feature.json` canônico explicitamente (nascimento explícito no início da proposal; o hook state-sync continua cobrindo o caminho implícito como fallback). `--type` classifica direto; `--request "<pedido>"` roda o detector determinístico (empate/sem sinal sai com exit 2 e candidatos — a LLM decide e re-roda com `--type`); `--worktree` já nasce a feature no seu worktree próprio.
 - `morph-spec retype <feature> <newType>` — muda o tipo de trabalho mid-flight; desfaz só os gates carimbados por política, preserva aprovações humanas; bloqueado após o Gate 3.

package/framework/MORPH.md

@@ -269,14 +269,32 @@ Inline (sem dispatch) é aceitável para tasks curtas (effort S/M) **dentro do d
 - Contexto de outras tarefas não relacionadas
 - Outputs de ferramentas não pertinentes à tarefa
 
-**O que receber de volta:**
+**O ARQUIVO é o retorno, a última mensagem é cortesia.** Em uso real (feature `config-tenant-split`,
+MORPH_GHLBrain, 2026-07-22 — overnight com dispatch de sub-agents), o evento recebido de volta do
+sub-agent foi só um `idle_notification`, sem o texto — o orquestrador teve que reconstruir o
+relatório do disco e pedir via `SendMessage` que o avaliador o persistisse. A última mensagem de
+texto **pode não chegar** inteira ao orquestrador; por isso ela deixou de ser o canal
+autoritativo. O sub-agent **sempre** grava o relatório também em um arquivo canônico:
+
+- Task de implementação: `.morph/features/{feature}/3-implement/reports/{taskId}.md` (um arquivo
+  por task concluída, mesmo quando uma única sessão de sub-agent cobre várias tasks de um fluxo).
+- Avaliador do Gate 3 (`morph-evaluator`): `.morph/features/{feature}/4-review/evaluator-report.md`
+  — o relatório BRUTO do avaliador; a `morph-review` o lê e sintetiza o `review-report.md`
+  apresentado no Gate 3 (os dois nunca colidem: um é saída crua do sub-agent, o outro é a curadoria
+  do orquestrador).
+
+O orquestrador **lê o arquivo** ao retomar controle após o dispatch — nunca confia apenas na
+mensagem de retorno da tool. Se a mensagem de texto final chegou inteira, ótimo (economiza uma
+leitura); se não chegou, o arquivo já está lá.
+
+**O que receber de volta (no arquivo, e na mensagem final quando ela chega):**
 - O que foi feito (2-3 frases)
 - Arquivos modificados (lista)
 - Problemas encontrados e como foram resolvidos
 - Auto-score 0-10 com justificativa em uma linha
 
 **Contrato do relatório final** (cláusulas que todo prompt de dispatch inclui — ver templates em `morph-implement` §4 e `morph-apply` §3b):
-1. **ATO FINAL: o seu relatório é a sua ÚLTIMA MENSAGEM DE TEXTO** — ela é o valor de retorno do sub-agent e chega inteira ao orquestrador. **Não chame `SendMessage`**: ele gera um evento a mais (o teammate-message) além da notificação de término, custando um turno inútil ao orquestrador. Conteúdo obrigatório: (1) o que foi feito em 2-3 frases, (2) arquivos modificados, (3) problemas encontrados e como foram resolvidos, (4) auto-score 0-10 com justificativa em uma linha. Relatório oco ou sem os 4 itens é **falha de protocolo** — registrada no recap.
+1. **GRAVE o relatório no arquivo canônico do seu dispatch ANTES de encerrar** — é o valor de retorno autoritativo desta sessão, não a sua última mensagem de texto (que é cortesia best-effort e pode não chegar inteira ao orquestrador). Conteúdo obrigatório, idêntico no arquivo: (1) o que foi feito em 2-3 frases, (2) arquivos modificados, (3) problemas encontrados e como foram resolvidos, (4) auto-score 0-10 com justificativa em uma linha. Relatório ausente, oco, ou sem os 4 itens é **falha de protocolo** — registrada no recap. Ainda assim **não chame `SendMessage`**: ele gera um evento a mais (o teammate-message) além da notificação de término, custando um turno inútil ao orquestrador — a mensagem de texto final (cortesia) já é entregue automaticamente.
 2. Não tocar em `tasks.json`, `feature.json`, `state.json` nem no board de sessão (TaskCreate/TaskUpdate) — todo estado é gravado exclusivamente pelo orquestrador.
 3. **Verificação em batch paralelo — condicional ao isolamento:** em batch **sem isolamento** (agentes na mesma working tree), não rode build/test da suíte compartilhada — a verificação (`morph-spec verify`) é do orquestrador, uma vez, pós-batch. Em batch **com `isolation: "worktree"`** (quando o `morph-spec dag` recomenda, por colisão de `outputs` entre fluxos), a regra se inverte: rode build/test **dentro do seu worktree**, commite o trabalho na branch do worktree como último ato antes do relatório, reporte a branch + status de build, e **nunca** faça merge nem toque na árvore principal — a integração é do orquestrador.
 

package/framework/hooks/claude-code/core/protect-spec-files.js

@@ -52,7 +52,13 @@ function diffPlanContract(oldDoc, newDoc, mutableFields) {
   const violations = [];
   const mutable = new Set(mutableFields);
 
-  if (oldDoc.version !== newDoc.version) {
+  // Backfilling a previously-absent `version` (undefined → any value) is a
+  // benign schema-tooling update, not a plan-contract change — it's what
+  // lets legacy tasks.json adopt the field so `morph-spec dag` can analyze
+  // it without a lenient fallback. An actual change between two SET
+  // versions is still a real contract change and stays blocked.
+  const versionBackfilled = oldDoc.version === undefined && newDoc.version !== undefined;
+  if (!versionBackfilled && oldDoc.version !== newDoc.version) {
     violations.push(`'version' changed (${oldDoc.version} → ${newDoc.version})`);
   }
   if (oldDoc.feature !== newDoc.feature) {

package/framework/hooks/shared/build-check.js

@@ -84,7 +84,14 @@ export function getBuildSteps(stack, { cwd, frontendPath, backendPath } = {}) {
 
   if (isFrontendStack(s)) {
     const frontendDir = frontendPath ? join(cwd, frontendPath) : cwd;
-    if (existsSync(join(frontendDir, 'tsconfig.json'))) {
+    // node_modules guard (same fail-open as post-edit-typecheck.js): a fresh worktree before
+    // `morph-spec worktree link` has tsconfig.json (versioned) but no toolchain — running
+    // `npx tsc` there fetches an unrelated "tsc" package and fails noisily. No toolchain →
+    // no tsc step; the real typecheck runs wherever node_modules exists (root/CI).
+    if (
+      existsSync(join(frontendDir, 'tsconfig.json')) &&
+      existsSync(join(frontendDir, 'node_modules'))
+    ) {
       steps.push({ command: TSC_BUILD_CMD, cwd: frontendDir });
     }
   }

package/framework/hooks/shared/payload-utils.js

@@ -93,7 +93,16 @@ export function getToolName(payload) {
  * Build a normalized signature for loop detection.
  * - Edit/Write: "Edit:normalized/file/path"
  * - Bash: "Bash:first two words" (e.g., "Bash:npm test")
- * - Other: just the tool name
+ * - Every other tool WITH input: "Tool:<truncated JSON of tool_input>"
+ * - No tool_input at all: just the tool name
+ *
+ * The general (non-Edit/Write/Bash) case matters: a burst of `Read`/`Skill`/
+ * `Grep`/`Agent` calls against DIFFERENT targets (phase-context loading,
+ * post-task evaluation reads, parallel subagent dispatch) is normal work, not
+ * a loop. Collapsing every one of them to the bare tool name is what made
+ * loop-detect fire on exactly that legitimate traffic — a real, reported
+ * false positive. Folding in the actual input (rather than hand-listing a
+ * target field per tool) fixes it uniformly for every tool at once.
  *
  * @param {Object} payload - Hook payload from stdin
  * @returns {string} Signature string
@@ -116,5 +125,10 @@ export function getCommandSignature(payload) {
     return words ? `Bash:${words}` : 'Bash';
   }
 
-  return tool;
+  try {
+    const json = JSON.stringify(input).replace(/\\/g, '/');
+    return json ? `${tool}:${json.slice(0, 200)}` : tool;
+  } catch {
+    return tool;
+  }
 }

package/framework/hooks/shared/phase-utils.js

@@ -35,6 +35,7 @@ export const OUTPUT_PHASE_MAP = {
   decisions: 'plan',
   recap: 'implement',
   reviewReport: 'review',
+  evaluatorReport: 'review',
 };
 
 /** Map filename → output type (camelCase) */
@@ -52,6 +53,7 @@ export const FILENAME_TO_OUTPUT = {
   'decisions.md': 'decisions',
   'recap.md': 'recap',
   'review-report.md': 'reviewReport',
+  'evaluator-report.md': 'evaluatorReport',
 };
 
 /** Protected spec files and the approval gate that locks them */

package/framework/skills/core/create-umbraco-lp/SKILL.md

@@ -4,7 +4,7 @@ description: Guia a criação de uma landing page nova ponta-a-ponta a partir do
 version: core
 when-to-invoke: ao iniciar um novo projeto de LP com o template Umbraco genérico
 user-invokable: true
-cliVersion: "8.2.0"
+cliVersion: "8.2.1"
 ---
 
 # create-umbraco-lp

package/framework/skills/core/migrate-lp-to-umbraco/SKILL.md

@@ -4,7 +4,7 @@ description: Migra uma landing page existente (React/Vite, Next.js estático ou
 version: core
 when-to-invoke: ao receber instrução "use /migrate-lp-to-umbraco {projeto}" do usuário, ou quando o usuário descrever necessidade de migrar uma LP para Umbraco
 user-invokable: true
-cliVersion: "8.2.0"
+cliVersion: "8.2.1"
 ---
 
 # migrate-lp-to-umbraco

package/framework/skills/core/morph-apply/SKILL.md

@@ -4,7 +4,7 @@ description: Orquestra a execução autônoma de tasks de uma feature morph-spec
 version: core
 when-to-invoke: após Gate 2 aprovado, ao receber instrução "use /morph-apply" ou "implementar feature"
 user-invokable: true
-cliVersion: "8.2.0"
+cliVersion: "8.2.1"
 ---
 
 # morph-apply
@@ -135,11 +135,16 @@ dispatch Agent (subagent_type: "morph-{id}") with prompt:
     Não modifique spec.md, mandate.md.
 
     Contrato do relatório final:
-    1. ATO FINAL — o seu relatório é a sua ÚLTIMA MENSAGEM DE TEXTO: ela é o
-       valor de retorno desta sessão e chega inteira ao orquestrador. NÃO chame
-       SendMessage: ele gera um evento a mais além da notificação de término,
-       custando um turno inútil ao orquestrador. Escreva o relatório e encerre.
-       Relatório oco, ou sem os itens abaixo, é falha de protocolo.
+    1. GRAVE um relatório por task concluída em
+       `.morph/features/{feature}/3-implement/reports/{taskId}.md` ANTES de
+       encerrar — mesmo quando esta sessão cobre várias tasks do fluxo, um
+       arquivo por task. Esse arquivo é o valor de retorno autoritativo; a sua
+       última mensagem de texto é cortesia best-effort e pode não chegar
+       inteira ao orquestrador (o evento recebido às vezes é só um
+       idle_notification, sem texto). Ainda assim NÃO chame SendMessage: ele
+       gera um evento a mais além da notificação de término, custando um turno
+       inútil ao orquestrador. Relatório ausente, oco, ou sem os itens abaixo,
+       é falha de protocolo.
     2. NÃO toque em tasks.json, feature.json, state.json nem no board de sessão
        (TaskCreate/TaskUpdate) — todo estado (status de task, scores, gates) é
        gravado exclusivamente pelo orquestrador.
@@ -163,7 +168,7 @@ dispatch Agent (subagent_type: "morph-{id}") with prompt:
 
 > **Alternativa quando os fluxos tocam os mesmos projetos/arquivos** (ex.: o mesmo csproj de testes compartilhado): siga a §3b-w abaixo em vez desta seção — despache com `isolation: "worktree"`, cada agente trabalha numa working tree própria e o orquestrador integra ao final.
 
-Aguardar todos os agentes terminarem — o relatório de cada um chega como o resultado do seu dispatch. Se um relatório vier oco ou sem os itens da cláusula 1, registre a falha de protocolo no recap. Depois, sequencialmente:
+Aguardar todos os agentes terminarem, depois **ler primeiro** `.morph/features/{feature}/3-implement/reports/{taskId}.md` de cada task — é o valor de retorno autoritativo; a mensagem de texto do dispatch, se chegou inteira, é só um atalho que economiza essa leitura. Se um arquivo vier ausente, oco, ou sem os itens da cláusula 1, registre a falha de protocolo no recap. Depois, sequencialmente:
 - **Rodar `morph-spec verify {feature} --json` uma única vez, pós-batch** (escopo feature) — os agentes não a rodaram (cláusula 3). Vermelho: use `verify.feedback` para corrigir/redespachar ANTES de qualquer morph-eval. (Com isolamento por worktree, cada agente já rodou o verify dentro do seu worktree; o orquestrador roda o verify compartilhado UMA vez após o merge — ver §3b-w.)
 - Para cada relatório retornado:
   - Aplicar `Skill(morph-eval)`.

package/framework/skills/core/morph-brainstorming/SKILL.md

@@ -4,7 +4,7 @@ description: Entende o negócio, os usuários e a arquitetura antes de qualquer
 version: core
 when-to-invoke: ao receber um novo request de feature, antes de morph-plan
 user-invokable: true
-cliVersion: "8.2.0"
+cliVersion: "8.2.1"
 ---
 
 # morph-brainstorming

package/framework/skills/core/morph-eval/SKILL.md

@@ -4,7 +4,7 @@ description: Avalia a qualidade de uma tarefa ou fase com rubricas graduadas 0-1
 version: core
 when-to-invoke: ao completar uma tarefa ou ao finalizar uma fase de implementação
 user-invokable: false
-cliVersion: "8.2.0"
+cliVersion: "8.2.1"
 ---
 
 # morph-eval

package/framework/skills/core/morph-hotfix/SKILL.md

@@ -4,7 +4,7 @@ description: ADW cirúrgico de hotfix — resolve uma queda/bug crítico de prod
 version: core
 when-to-invoke: ao receber "/morph-hotfix {feature}", quando a morph-proposal classifica o pedido como workType hotfix (Passo 0), ou o usuário diz "produção caiu / hotfix / urgente"
 user-invokable: true
-cliVersion: "8.2.0"
+cliVersion: "8.2.1"
 ---
 
 # morph-hotfix

package/framework/skills/core/morph-implement/SKILL.md

@@ -4,7 +4,7 @@ description: Guia a execução de cada tarefa de implementação com standard e
 version: core
 when-to-invoke: ao iniciar cada task de implementação listada em tasks.json
 user-invokable: false
-cliVersion: "8.2.0"
+cliVersion: "8.2.1"
 ---
 
 # morph-implement
@@ -59,11 +59,14 @@ Ao iniciar cada task de `tasks.json` — antes de escrever qualquer código. Nor
    Done quando: [doneCriteria do tasks.json]
 
    Contrato do relatório final:
-   1. ATO FINAL — o seu relatório é a sua ÚLTIMA MENSAGEM DE TEXTO: ela é o
-      valor de retorno desta sessão e chega inteira ao orquestrador. NÃO chame
-      SendMessage: ele gera um evento a mais além da notificação de término,
-      custando um turno inútil ao orquestrador. Escreva o relatório e encerre.
-      Relatório oco, ou sem os 4 itens abaixo, é falha de protocolo.
+   1. GRAVE seu relatório em `.morph/features/{feature}/3-implement/reports/[ID].md`
+      ANTES de encerrar — esse arquivo é o valor de retorno autoritativo desta
+      sessão. A sua última mensagem de texto é cortesia best-effort: pode não
+      chegar inteira ao orquestrador (o evento recebido às vezes é só um
+      idle_notification, sem texto). Ainda assim NÃO chame SendMessage: ele
+      gera um evento a mais além da notificação de término, custando um turno
+      inútil ao orquestrador. Relatório ausente, oco, ou sem os 4 itens abaixo,
+      é falha de protocolo.
    2. NÃO toque em tasks.json, feature.json, state.json nem no board de sessão
       (TaskCreate/TaskUpdate) — todo estado é gravado exclusivamente pelo
       orquestrador.
@@ -74,11 +77,18 @@ Ao iniciar cada task de `tasks.json` — antes de escrever qualquer código. Nor
       commite na branch do worktree como último ato antes do relatório
       e reporte a branch — nunca faça merge.
 
-   Seu relatório final deve conter APENAS: (1) o que foi feito em 2-3 frases,
-   (2) arquivos modificados, (3) problemas encontrados e como foram resolvidos,
-   (4) auto-score 0-10.
+   Seu relatório (no arquivo, e na mensagem final quando ela chega) deve conter
+   APENAS: (1) o que foi feito em 2-3 frases, (2) arquivos modificados,
+   (3) problemas encontrados e como foram resolvidos, (4) auto-score 0-10.
    ```
-   Após receber o retorno: o auto-score é **triagem, não aprovação** — se < 7, re-despache com o feedback específico (ou corrija inline) antes de prosseguir. Em qualquer caso, a task só avança quando `morph-eval` der score ≥ 9 (barra única do MORPH.md §0). Integre o relatório ao recap.md e vá para o passo 7.
+   Após o retorno do dispatch: **leia primeiro** `.morph/features/{feature}/3-implement/reports/[ID].md`
+   — é o valor de retorno autoritativo. Trate a mensagem de texto do sub-agent, se chegou inteira,
+   como um atalho que economiza essa leitura — nunca como a única fonte. Arquivo ausente é falha de
+   protocolo (registre no recap e re-despache/complete a partir do que o sub-agent já fez, se
+   recuperável). O auto-score é **triagem, não aprovação** — se < 7, re-despache com o feedback
+   específico (ou corrija inline) antes de prosseguir. Em qualquer caso, a task só avança quando
+   `morph-eval` der score ≥ 9 (barra única do MORPH.md §0). Integre o relatório ao recap.md e vá
+   para o passo 7.
 
    **Se não — siga para o passo 5.**
 
@@ -136,6 +146,8 @@ Ao iniciar cada task de `tasks.json` — antes de escrever qualquer código. Nor
 
 ## Output esperado
 - Código implementado conforme o standard consultado (direto ou via sub-agent)
+- Quando despachada: `.morph/features/{feature}/3-implement/reports/{taskId}.md` — o relatório
+  autoritativo do sub-agent, canal durável (passo 4)
 - `.morph/features/{feature}/3-implement/recap.md` atualizado com entrada da task concluída
 - `.morph/features/{feature}/2-plan/tasks.json` com status da task atualizado para `done`
 - `.morph/features/{feature}/2-plan/decisions.md` atualizado se houver decisão não trivial

package/framework/skills/core/morph-plan/SKILL.md

@@ -4,7 +4,7 @@ description: Guia a produção dos três artefatos de planejamento de uma featur
 version: core
 when-to-invoke: ao iniciar a fase plan de uma feature, após o Gate 1 (proposal aprovado)
 user-invokable: true
-cliVersion: "8.2.0"
+cliVersion: "8.2.1"
 ---
 
 # morph-plan

package/framework/skills/core/morph-proposal/SKILL.md

@@ -4,7 +4,7 @@ description: Orquestra a fase de proposta+plano de uma feature morph-spec. Encad
 version: core
 when-to-invoke: ao receber instrução "use /morph-proposal" do usuário, ou ao iniciar uma nova feature
 user-invokable: true
-cliVersion: "8.2.0"
+cliVersion: "8.2.1"
 ---
 
 # morph-proposal

package/framework/skills/core/morph-review/SKILL.md

@@ -4,7 +4,7 @@ description: Conduz a fase de revisão e produz o review-report.md para o Gate 3
 version: core
 when-to-invoke: ao iniciar a fase review, após todas as tasks de tasks.json estarem com status `done`
 user-invokable: true
-cliVersion: "8.2.0"
+cliVersion: "8.2.1"
 ---
 
 # morph-review
@@ -56,18 +56,22 @@ e o `recap.md` estar atualizado. Normalmente invocada pela skill orquestradora `
          seguindo as bandas 0-10 de cada uma.
          NÃO reescreva código. NÃO toque em tasks.json, feature.json nem no board
          de sessão.
-         Contrato do relatório final — ATO FINAL: o seu relatório é a sua
-         ÚLTIMA MENSAGEM DE TEXTO; ela é o valor de retorno desta sessão e
-         chega inteira ao orquestrador. NÃO chame SendMessage: ele gera um
-         evento a mais além da notificação de término, custando um turno
-         inútil ao orquestrador. Escreva o relatório e encerre.
-         Seu relatório final deve conter: score 0-10 por dimensão, score
-         composto (arch×0.30 + contracts×0.25 + quality×0.25 + tests×0.20), e
-         feedback priorizado (bloqueantes primeiro) para qualquer dimensão < 9.
-         Relatório oco, ou sem esses itens, é falha de protocolo.
+         Contrato do relatório final: GRAVE seu relatório em
+         `.morph/features/{feature}/4-review/evaluator-report.md` ANTES de
+         encerrar — esse arquivo é o valor de retorno autoritativo desta
+         sessão. A sua última mensagem de texto é cortesia best-effort: pode
+         não chegar inteira ao orquestrador (o evento recebido às vezes é só
+         um idle_notification, sem texto). Ainda assim NÃO chame SendMessage:
+         ele gera um evento a mais além da notificação de término, custando
+         um turno inútil ao orquestrador.
+         Seu relatório (no arquivo, e na mensagem final quando ela chega)
+         deve conter: score 0-10 por dimensão, score composto (arch×0.30 +
+         contracts×0.25 + quality×0.25 + tests×0.20), e feedback priorizado
+         (bloqueantes primeiro) para qualquer dimensão < 9. Relatório
+         ausente, oco, ou sem esses itens, é falha de protocolo.
      ```
    - **Fallback (tipo não registrado):** se `morph-evaluator` não estiver na sua lista de subagent types, despache `general-purpose` colando o campo `persona` do agente `evaluator` (Read em `.morph/framework/agents.json`) no topo do prompt — e **registre o fallback no review-report** (seção de método).
-   - Os scores por dimensão e o composto do review-report vêm **desse retorno**, não da sua auto-avaliação. Se o relatório vier oco ou sem os scores por dimensão, registre a falha de protocolo na seção de método do review-report e re-despache.
+   - **Leia primeiro** `.morph/features/{feature}/4-review/evaluator-report.md` — é o valor de retorno autoritativo, não a mensagem de texto do dispatch (que é cortesia, útil só quando chega inteira). Os scores por dimensão e o composto do review-report vêm **desse arquivo**, não da sua auto-avaliação. Se o arquivo vier ausente, oco, ou sem os scores por dimensão, registre a falha de protocolo na seção de método do review-report e re-despache.
 
 5. Produza `review-report.md` em `.morph/features/{feature}/4-review/` com:
    - Scores por dimensão (arquitetura, contratos, qualidade, cobertura)
@@ -117,7 +121,10 @@ e o `recap.md` estar atualizado. Normalmente invocada pela skill orquestradora `
    No caminho manual, o hook state-sync detecta a aprovação em answers de AskUserQuestion e grava `approvalGates.review = { approved: true, timestamp, approvedBy: "manual" }` no `feature.json` autoritativo da feature. Em auto sem AskUserQuestion, aprove via `morph-spec approve {feature} review --mode auto` — nunca via Edit à mão de state.json.
 
 ## Output esperado
-- `.morph/features/{feature}/4-review/review-report.md` com scores e recomendação
+- `.morph/features/{feature}/4-review/evaluator-report.md` — relatório bruto e autoritativo do
+  `morph-evaluator` despachado no passo 4 (canal durável, ver MORPH.md §6)
+- `.morph/features/{feature}/4-review/review-report.md` — síntese do orquestrador a partir do
+  evaluator-report, com scores e recomendação
 - Trace da fase com `phase_summary` preenchido
 - Gate 3 registrado aprovado em `.morph/features/{feature}/feature.json` (autoritativo), na forma canônica `{ approved, timestamp, approvedBy }` — via `morph-spec approve` em auto, ou pelo hook state-sync no caminho manual
 

package/framework/skills/core/morph-stack-scan/SKILL.md

@@ -4,7 +4,7 @@ description: Varre o codebase existente, detecta stack atual, compara com o stac
 version: core
 when-to-invoke: ao iniciar uma feature em projeto brownfield (codebase já existente)
 user-invokable: true
-cliVersion: "8.2.0"
+cliVersion: "8.2.1"
 ---
 
 # morph-stack-scan

package/framework/skills/core/morph-standards/SKILL.md

@@ -4,7 +4,7 @@ description: Localiza e carrega um standard específico do STANDARDS.json sob de
 version: core
 when-to-invoke: quando precisar consultar um standard específico durante a implementação
 user-invokable: true
-cliVersion: "8.2.0"
+cliVersion: "8.2.1"
 ---
 
 # morph-standards

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@polymorphism-tech/morph-spec",
-  "version": "8.2.0",
+  "version": "8.2.1",
   "description": "MORPH-SPEC: NLH (Natural Language Harness) for spec-driven development with Claude Code",
   "keywords": [
     "claude-code",

package/src/commands/project/worktree.js

@@ -145,6 +145,38 @@ function getDateSuffix() {
 
 // ── Infrastructure linking ───────────────────────────────────────────────────
 
+/**
+ * The infra directories morph itself links into a worktree — the ONLY ones
+ * `setupInfraLinks`/`worktreeLinkCommand --all`/`unlinkWorktreeInfraLinks` treat
+ * as KNOWN, and therefore the only ones `finish` may auto-remove via the
+ * already-proven-safe unlink mechanism (never by widening the reparse-point
+ * scan — see `findReparsePoints`).
+ *
+ * Always: `node_modules`, `.claude`. Conditionally: `.morph/framework` (client
+ * projects only) and `{project.frontendPath}/node_modules` — a monorepo
+ * frontend (e.g. a Next.js app under `apps/catalog-admin`) commonly has its OWN
+ * node_modules, a separate `npm install` from the root's, invisible to a static
+ * list. Fail-open: a missing/unreadable config.json just means no frontendPath.
+ *
+ * @param {string} mainCwd - Absolute path to the main project root
+ * @returns {string[]} relative paths, root-first
+ */
+function resolveInfraLinkDirs(mainCwd) {
+  const dirs = ['node_modules', '.claude'];
+  if (existsSync(join(mainCwd, '.morph', 'framework'))) {
+    dirs.push(join('.morph', 'framework'));
+  }
+  try {
+    const configPath = join(mainCwd, '.morph', 'config', 'config.json');
+    if (existsSync(configPath)) {
+      const config = JSON.parse(readFileSync(configPath, 'utf8'));
+      const frontendPath = config?.project?.frontendPath;
+      if (frontendPath) dirs.push(join(frontendPath, 'node_modules'));
+    }
+  } catch { /* fail-open — no frontendPath to add */ }
+  return dirs;
+}
+
 /**
  * Create junctions in the worktree for shared infrastructure directories
  * (node_modules/, .claude/, .morph/framework/).
@@ -169,16 +201,9 @@ export function setupInfraLinks(worktreePath, mainCwd, opts = {}) {
   const mainAbs = resolve(mainCwd);
   const wtAbs = resolve(worktreePath);
 
-  let dirLinks;
-  if (Array.isArray(opts.dirs) && opts.dirs.length) {
-    dirLinks = opts.dirs;
-  } else {
-    dirLinks = ['node_modules', '.claude'];
-    // .morph/framework/ — client projects (in framework repo, framework/ is tracked)
-    if (existsSync(join(mainAbs, '.morph', 'framework'))) {
-      dirLinks.push(join('.morph', 'framework'));
-    }
-  }
+  const dirLinks = (Array.isArray(opts.dirs) && opts.dirs.length)
+    ? opts.dirs
+    : resolveInfraLinkDirs(mainAbs);
 
   for (const rel of dirLinks) {
     const target = join(mainAbs, rel);
@@ -530,7 +555,7 @@ export function worktreeLinkCommand(options = {}) {
   }
 
   const dirs = options.all
-    ? ['node_modules', '.claude', join('.morph', 'framework')]
+    ? resolveInfraLinkDirs(root)
     : ['node_modules'];
 
   const alreadyPresent = {};
@@ -624,12 +649,10 @@ function findWorktreeForFeature(feature, root) {
   return { path: match.path, branch: actual, expectedBranch };
 }
 
-/** The infra paths `setupInfraLinks` junctions into a worktree. */
-const INFRA_LINK_RELS = ['node_modules', '.claude', join('.morph', 'framework')];
-
 /**
  * Drop the infra junctions a worktree carries (`node_modules`, `.claude`,
- * `.morph/framework`) WITHOUT following them.
+ * `.morph/framework`, and the `project.frontendPath` node_modules if
+ * configured) WITHOUT following them.
  *
  * CRITICAL — this is what keeps `worktree finish` from destroying the project.
  * `git worktree remove` refuses a worktree still holding these ("contains
@@ -646,12 +669,19 @@ const INFRA_LINK_RELS = ['node_modules', '.claude', join('.morph', 'framework')]
  * target's contents, and `rmdirSync` fails on a real non-empty directory — the
  * safety that stops this from ever deleting genuine files.
  *
+ * This is deliberately narrow to the KNOWN inventory from `resolveInfraLinkDirs`
+ * — never widened to auto-remove any reparse point `findReparsePoints` might
+ * discover elsewhere. An unknown/nested link (npm link, npm workspaces) must
+ * still decline the removal; see the CANARY tests pinning that in worktree.test.js.
+ *
  * @param {string} worktreePath
+ * @param {string} mainCwd - Absolute path to the main project root, to resolve
+ *   `project.frontendPath` from config.json
  * @returns {string[]} relative paths actually unlinked
  */
-export function unlinkWorktreeInfraLinks(worktreePath) {
+export function unlinkWorktreeInfraLinks(worktreePath, mainCwd) {
   const removed = [];
-  for (const rel of INFRA_LINK_RELS) {
+  for (const rel of resolveInfraLinkDirs(mainCwd)) {
     const p = join(worktreePath, rel);
     let st;
     try {
@@ -746,9 +776,60 @@ function findReparsePoints(dir, base = dir, acc = { points: [], unprovable: [] }
  */
 const IGNORED_SHIELDABLE_PREFIXES = ['.morph/'];
 
+/**
+ * Well-known, regenerable build artifacts — `bin/`, `obj/` (dotnet), a real
+ * `node_modules` left behind outside the setup-created junction, and
+ * `*.tsbuildinfo` (tsc incremental cache). These are the everyday leftovers of
+ * running a build/test cycle inside a worktree, not user data: they are fully
+ * reproducible from source, and requiring `--discard-ignored` for every finish
+ * that ever ran a build turns the flag from "explicit consent for a real risk"
+ * into a rubber stamp nobody reads anymore. Auto-discarding them (like any
+ * `discarded` entry, WITHOUT a backup — see the note below) is safe precisely
+ * because "regenerable" is a stronger guarantee than "backed up".
+ *
+ * Deliberately narrow — do NOT add speculative patterns (`dist/`, `.next/`)
+ * beyond what's proven to be pure noise; anything else stays in `blocking` by
+ * default, requiring the explicit `--discard-ignored` consent.
+ *
+ * @param {string} p - forward-slash relative path from `git status --ignored`
+ */
+/**
+ * Whether a dirty root-side path is EXACTLY a sibling feature's live worktree
+ * registration — see the call site (Validation 3 in worktreeFinishCommand)
+ * for why this is safe to exempt from the whole-root clean gate rather than a
+ * widened hole.
+ *
+ * PROVES it rather than assuming: the path must be exactly that sibling's
+ * `feature.json`, must parse, and its `worktree.path` must point at a
+ * currently-existing directory. A parse failure, a missing registration, or a
+ * STALE pointer (worktree already removed) is NOT exempted.
+ *
+ * @param {string} relPath - forward-slash path relative to `root`
+ * @param {string} root - absolute path to the primary root
+ * @param {string} feature - the feature THIS finish is acting on (never exempted — its own scope is the shed's job)
+ */
+function isLiveSiblingWorktreeRegistration(relPath, root, feature) {
+  const m = relPath.match(/^\.morph\/features\/([^/]+)\/feature\.json$/);
+  if (!m || m[1] === feature) return false;
+  try {
+    const fj = JSON.parse(readFileSync(join(root, relPath), 'utf8'));
+    const wtPath = fj?.worktree?.path;
+    return typeof wtPath === 'string' && wtPath.length > 0 && existsSync(wtPath);
+  } catch {
+    return false;
+  }
+}
+
+function isDisposableBuildArtifact(p) {
+  if (/\.tsbuildinfo$/.test(p)) return true;
+  const segments = p.split('/').filter(Boolean);
+  return segments.some((s) => s === 'bin' || s === 'obj' || s === 'node_modules');
+}
+
 /**
  * Split the worktree's GITIGNORED leftovers into what this command may remove
- * (morph's own, after a backup) and what it must refuse to destroy.
+ * (morph's own, after a backup, or a known disposable build artifact, without
+ * one) and what it must refuse to destroy.
  *
  * Why this exists: `git worktree remove` deletes gitignored files outright, and
  * the clean-check that gates it uses `git status --porcelain`, which omits them
@@ -762,34 +843,46 @@ const IGNORED_SHIELDABLE_PREFIXES = ['.morph/'];
  *
  * @param {string} worktreePath
  * @param {{ discardIgnored?: boolean }} options
- * @returns {{ shieldable: string[], blocking: string[] }}
+ * @returns {{ shieldable: string[], blocking: string[], discarded: string[] }}
  */
 function classifyIgnoredLeftovers(worktreePath, options = {}) {
   let out;
   try {
     // `-z` + execFile: NUL-delimited, no shell, no C-style quoting to undo —
     // paths with spaces or non-ASCII (a pt-BR codebase has both) stay intact.
-    out = execFileSync('git', ['status', '--porcelain', '--ignored', '-z'], { cwd: worktreePath, stdio: 'pipe' }).toString();
+    // `=matching` (not the traditional default): an ignored directory whose own
+    // name matches no pattern — e.g. `MyProj/`, ignored only because EVERYTHING
+    // inside it (`bin/`, `obj/`) does — otherwise collapses into one opaque
+    // `MyProj/` entry, hiding the actual disposable-artifact names from
+    // isDisposableBuildArtifact. A directory that IS itself the matched pattern
+    // (`node_modules/`) still reports as one entry either way — verified
+    // empirically, not assumed.
+    out = execFileSync('git', ['status', '--porcelain', '--ignored=matching', '-z'], { cwd: worktreePath, stdio: 'pipe' }).toString();
   } catch {
-    return { shieldable: [], blocking: ['<git status --ignored falhou — não foi possível enumerar o que seria destruído>'] };
+    return { shieldable: [], blocking: ['<git status --ignored falhou — não foi possível enumerar o que seria destruído>'], discarded: [] };
   }
 
   const shieldable = [];
-  const blocking = [];
+  const blockingRest = [];
+  const disposable = [];
   for (const record of out.split('\0')) {
     if (!record || record.length < 4) continue;
     if (record.slice(0, 2) !== '!!') continue; // only ignored entries; the rest is the clean-check's job
     const p = record.slice(3).replace(/\\/g, '/');
     if (IGNORED_SHIELDABLE_PREFIXES.some((prefix) => p.startsWith(prefix))) shieldable.push(p);
-    else blocking.push(p);
+    else if (isDisposableBuildArtifact(p)) disposable.push(p);
+    else blockingRest.push(p);
   }
-  // Explicit consent clears the block — but the list is NOT thrown away. These
-  // are the files that exist nowhere else, and they are the ones about to die;
-  // reporting them is the least this can do. (Backing them up instead is not an
-  // option: this bucket is where `node_modules`/`dist`/`.next` land, so a copy
-  // is unbounded. The user declared them disposable — they still get told.)
-  if (options.discardIgnored) return { shieldable, blocking: [], discarded: blocking };
-  return { shieldable, blocking, discarded: [] };
+  // Explicit consent clears the rest of the block too — but the list is NOT
+  // thrown away. These are the files that exist nowhere else, and they are the
+  // ones about to die; reporting them is the least this can do. (Backing them
+  // up instead is not an option: this bucket is where arbitrarily large
+  // leftovers land, so a copy is unbounded. The user declared them
+  // disposable — they still get told.)
+  if (options.discardIgnored) return { shieldable, blocking: [], discarded: [...disposable, ...blockingRest] };
+  // Known build artifacts are discarded by default, no flag required — they
+  // are reproducible from source, unlike anything else in `blockingRest`.
+  return { shieldable, blocking: blockingRest, discarded: disposable };
 }
 
 /**
@@ -1015,15 +1108,34 @@ export async function worktreeFinishCommand(feature, options = {}) {
     }
     : {};
 
-  // Validation 3: primary root must be clean.
-  let rootStatus;
+  // Validation 3: primary root must be clean — except a SIBLING feature's own
+  // LIVE worktree registration (see isLiveSiblingWorktreeRegistration). That
+  // artifact is how `runWorktreeSetup`/`create --worktree` are DESIGNED to
+  // leave the root for that sibling's entire life (decidePersistence in
+  // verify.js and the anti-dupe guard above both read the root's copy for
+  // `.worktree.path`), so refusing THIS feature's finish over it would block
+  // every OTHER feature for as long as any sibling stays open in its own
+  // worktree — the exact "orphaned scaffold collides with finish" friction
+  // from real multi-feature use. Anything else is still unexplained dirt and
+  // still blocks, fail-closed, exactly as before.
+  let rootStatusRaw;
   try {
-    rootStatus = execFileSync('git', ['status', '--porcelain'], { cwd: root, stdio: 'pipe' }).toString();
+    // `--untracked-files=all`, not the default `normal`: the default collapses
+    // an entirely-untracked directory into one entry (e.g. `.morph/features/
+    // sibling-feat/`, verified empirically) — which would never match the
+    // exact `.../feature.json` path isLiveSiblingWorktreeRegistration expects,
+    // silently defeating the exemption for the very shape it exists to allow.
+    rootStatusRaw = execFileSync('git', ['status', '--porcelain', '--untracked-files=all', '-z'], { cwd: root, stdio: 'pipe' }).toString();
   } catch (err) {
     console.log(JSON.stringify({ finished: false, error: 'primary-root-status-failed', detail: (err.message || '').slice(0, 200), feature, ...backupNote }));
     process.exit(1);
   }
-  if (rootStatus.trim() !== '') {
+  const unexplainedDirt = rootStatusRaw.split('\0').filter((record) => {
+    if (!record || record.length < 4) return false;
+    const p = record.slice(3).replace(/\\/g, '/');
+    return !isLiveSiblingWorktreeRegistration(p, root, feature);
+  });
+  if (unexplainedDirt.length) {
     console.log(JSON.stringify({
       finished: false,
       error: 'primary-root-dirty',
@@ -1075,7 +1187,7 @@ export async function worktreeFinishCommand(feature, options = {}) {
   // Both gates below therefore PROVE-or-DECLINE. Declining costs a stale
   // directory the user can delete in one command; being wrong costs data that
   // exists nowhere else.
-  unlinkWorktreeInfraLinks(worktreePath);
+  unlinkWorktreeInfraLinks(worktreePath, root);
 
   const scan = findReparsePoints(worktreePath);
   const ignored = classifyIgnoredLeftovers(worktreePath, options);
@@ -1202,7 +1314,9 @@ export async function worktreeFinishCommand(feature, options = {}) {
     ...(worktreeRemoved && ignored.discarded.length
       ? {
         discardedIgnored: ignored.discarded,
-        discardedNote: `--discard-ignored: os caminhos acima eram gitignored, existiam só no worktree e foram DESTRUÍDOS sem backup (não são copiados por poderem ser arbitrariamente grandes)`,
+        discardedNote: options.discardIgnored
+          ? `--discard-ignored: os caminhos acima eram gitignored, existiam só no worktree e foram DESTRUÍDOS sem backup (não são copiados por poderem ser arbitrariamente grandes)`
+          : `artefatos de build reconhecidos (bin/, obj/, node_modules/, *.tsbuildinfo) — descartados por padrão, sem backup (regeneráveis a partir do source)`,
       }
       : {}),
     ...(worktreeRemoved

package/src/commands/state/score.js

@@ -39,9 +39,20 @@ export async function scoreCommand(featureName, taskId, scoreArg, options = {})
 
     let dimensions;
     if (options.dims) {
-      const parts = String(options.dims).split(',').map(s => s.trim());
+      // Commander captures --dims as variadic (`<dims...>`), so an UNQUOTED
+      // comma list typed in PowerShell — `--dims 9.5,9,9,9` — that arrives as
+      // 4 separate argv tokens (confirmed: Commander binds only the first
+      // token to a non-variadic option and silently drops the rest) is not
+      // lost. Accept either shape: one comma-joined string (the documented,
+      // quoted form) or N separate tokens — split on both separators so a
+      // mix of the two never matters either.
+      const raw = Array.isArray(options.dims) ? options.dims.join(' ') : String(options.dims);
+      const parts = raw.split(/[,\s]+/).filter(Boolean);
       if (parts.length !== DIM_KEYS.length) {
-        console.error(chalk.red(`\n❌ --dims expects ${DIM_KEYS.length} comma-separated values in order a,c,q,t (architecture,contracts,code-quality,test-coverage)\n`));
+        console.error(chalk.red(
+          `\n❌ --dims expects ${DIM_KEYS.length} values in order a,c,q,t (architecture,contracts,code-quality,test-coverage) — got ${parts.length} (${JSON.stringify(parts)}).\n` +
+          `   Quote the value under PowerShell to keep it as one argument: --dims "9.5,9,9,9"\n`
+        ));
         process.exit(1);
       }
       dimensions = {};

package/src/commands/tasks/dag.js

@@ -1,7 +1,7 @@
 import chalk from 'chalk';
 import { existsSync } from 'fs';
 import { join } from 'path';
-import { parseTasksJson } from '../../lib/tasks/tasks-json-parser.js';
+import { parseTasksJsonLenient } from '../../lib/tasks/tasks-json-parser.js';
 import { planDispatch } from '../../lib/tasks/dispatch-planner.js';
 import { renderMermaidFlowchart } from '../../lib/tasks/mermaid-renderer.js';
 
@@ -24,8 +24,16 @@ export async function dagCommand(featureName, options = {}) {
       process.exit(1);
     }
 
-    // parseTasksJson validates the schema and throws with the error list.
-    const data = await parseTasksJson(tasksPath);
+    // parseTasksJsonLenient validates the schema and throws with the error
+    // list, but defaults a missing `version` (legacy plans predating the
+    // field) instead of failing — the DAG analysis doesn't depend on it.
+    const { data, versionDefaulted } = await parseTasksJsonLenient(tasksPath);
+    if (versionDefaulted) {
+      console.error(chalk.yellow(
+        `⚠ tasks.json has no 'version' field — defaulting to "1.0" for this analysis.\n` +
+        `  Consider adding it back explicitly (safe: protect-spec-files allows backfilling it).`
+      ));
+    }
 
     // --mermaid short-circuits before the dispatch analysis: visualizing the
     // plan's DAG doesn't need a sequential-vs-parallel decision.

package/src/core/paths/output-schema.js

@@ -49,6 +49,11 @@ export const OUTPUT_SCHEMA = {
 
   // 4-review/
   reviewReport:   { filename: 'review-report.md',      phaseDir: '4-review',    phase: 'review',    protected: false },
+  // Durable report the dispatched morph-evaluator sub-agent writes itself
+  // (MORPH.md §6) — the orchestrator reads it and synthesizes reviewReport
+  // from it; the two never collide because one is raw sub-agent output and
+  // the other is the orchestrator's curated Gate 3 presentation.
+  evaluatorReport:{ filename: 'evaluator-report.md',   phaseDir: '4-review',    phase: 'review',    protected: false },
 };
 
 // ============================================================================
@@ -89,6 +94,29 @@ export function getFeatureDir(featureName) {
   return join('.morph', 'features', featureName).replace(/\\/g, '/');
 }
 
+/**
+ * Per-task sub-agent report — the durable "return value" channel for a
+ * dispatched implementation task (MORPH.md §6). A sub-agent's final text
+ * message may never reach the orchestrator (idle_notification with no text
+ * is an observed failure mode); the file it writes here always does. Doesn't
+ * fit OUTPUT_SCHEMA's one-fixed-filename-per-type model since the filename
+ * is parameterized by taskId, so it's a sibling helper instead of a schema
+ * entry.
+ *
+ * @param {string} featureName
+ * @param {string} taskId - e.g. "T7"
+ * @returns {string} morph-relative, forward-slash path
+ */
+export function getTaskReportPath(featureName, taskId) {
+  return join('.morph', 'features', featureName, '3-implement', 'reports', `${taskId}.md`)
+    .replace(/\\/g, '/');
+}
+
+/** Absolute filesystem variant of {@link getTaskReportPath}. */
+export function getAbsoluteTaskReportPath(projectRoot, featureName, taskId) {
+  return join(projectRoot, '.morph', 'features', featureName, '3-implement', 'reports', `${taskId}.md`);
+}
+
 /**
  * Get all output paths for a feature, ready for state.json `outputs` block.
  */

package/src/lib/stack/stack-profile.js

@@ -51,24 +51,55 @@ function readStack(cwd) {
   }
 }
 
+/** Merge two or more profiles' validator lists (deduped) and boolean flags. */
+function mergeProfiles(profiles) {
+  return {
+    isDotnet: profiles.some((p) => p.isDotnet),
+    isNextjs: profiles.some((p) => p.isNextjs),
+    defaultValidators: [...new Set(profiles.flatMap((p) => p.defaultValidators))],
+    checkpointValidators: [...new Set(profiles.flatMap((p) => p.checkpointValidators))],
+    buildCommand: profiles.map((p) => p.buildCommand).find(Boolean) ?? null,
+  };
+}
+
 /**
  * Get the StackProfile for the current project.
+ *
+ * `stack` may be a compound value (e.g. "dotnet-nextjs", "dotnet+nextjs") —
+ * the framework's own convention for monorepos (see stack-filter.js's
+ * getProjectTags, which splits project.stack the same way). Each segment is
+ * resolved independently and matching profiles are merged, instead of the
+ * whole compound string failing every exact-match check and silently
+ * collapsing to the empty profile (0 default validators).
  */
 export function getStackProfile(cwd, stackOverride) {
   const root = cwd ?? process.cwd();
   const raw = stackOverride ?? readStack(root);
   const stack = raw?.toLowerCase() ?? 'unknown';
 
-  let profile;
-  if (NEXTJS_STACKS.has(stack)) {
-    profile = PROFILES.nextjs;
-  } else if (NO_VALIDATOR_STACKS.has(stack)) {
-    profile = PROFILES.empty;
-  } else if (DOTNET_STACKS.has(stack) || stack === 'unknown') {
-    profile = PROFILES.dotnet;
-  } else {
-    profile = PROFILES.empty;
+  const segments = stack.split(/[-+]/).filter(Boolean);
+
+  // Single-segment stack (the overwhelming common case): resolve exactly as
+  // before, byte-for-byte, including the 'unknown' → dotnet fallback.
+  if (segments.length <= 1) {
+    let profile;
+    if (NEXTJS_STACKS.has(stack)) {
+      profile = PROFILES.nextjs;
+    } else if (NO_VALIDATOR_STACKS.has(stack)) {
+      profile = PROFILES.empty;
+    } else if (DOTNET_STACKS.has(stack) || stack === 'unknown') {
+      profile = PROFILES.dotnet;
+    } else {
+      profile = PROFILES.empty;
+    }
+    return { stack, ...profile };
   }
 
+  // Compound stack: resolve each recognized segment and merge.
+  const matched = [];
+  if (segments.some((s) => DOTNET_STACKS.has(s))) matched.push(PROFILES.dotnet);
+  if (segments.some((s) => NEXTJS_STACKS.has(s))) matched.push(PROFILES.nextjs);
+
+  const profile = matched.length > 0 ? mergeProfiles(matched) : PROFILES.empty;
   return { stack, ...profile };
 }

package/src/lib/tasks/tasks-json-parser.js

@@ -98,6 +98,27 @@ export async function parseTasksJson(filePath) {
   return data;
 }
 
+/**
+ * Lenient variant for read-only analysis commands (e.g. `morph-spec dag`):
+ * a missing `version` field is defaulted to "1.0" instead of failing
+ * validation — legacy tasks.json predating the field is still analyzable.
+ * All other schema errors still throw exactly like `parseTasksJson`.
+ *
+ * @param {string} filePath - Absolute path to tasks.json
+ * @returns {Promise<{ data: object, versionDefaulted: boolean }>}
+ */
+export async function parseTasksJsonLenient(filePath) {
+  const data = await fs.readJson(filePath);
+  const versionDefaulted = data.version === undefined;
+  if (versionDefaulted) data.version = '1.0';
+
+  const { valid, errors } = validateTasksJson(data);
+  if (!valid) {
+    throw new Error(`Invalid tasks.json:\n  ${errors.join('\n  ')}`);
+  }
+  return { data, versionDefaulted };
+}
+
 /**
  * Synchronous variant — used by hooks.
  */

package/src/lib/tasks/test-runner.js

@@ -4,13 +4,16 @@
  * Resolution order for test command:
  *   1. .morph/config/config.json → project.testCommand
  *   2. .sln or .csproj present at project root → "dotnet test"
- *   3. package.json present AND has scripts.test → "npm test"
- *   4. null (no test suite detected)
+ *   3. Nested *Tests.csproj anywhere in the tree (monorepo without a root
+ *      .sln) → "dotnet test <proj> --nologo -v q" for each match
+ *   4. package.json present AND has scripts.test → "npm test"
+ *   5. null (no test suite detected)
  */
 
 import { execSync } from 'child_process';
 import { existsSync, readFileSync, readdirSync } from 'fs';
 import { join } from 'path';
+import { globSync } from 'glob';
 
 /**
  * Resolve the test command for a project.
@@ -46,7 +49,26 @@ export function resolveTestCommand(projectPath = '.') {
     // Unreadable project directory — fall through to next detection
   }
 
-  // 3. package.json present AND has scripts.test → npm test
+  // 3. No root-level project file — look for a nested test project
+  // (e.g. tests/MyApp.Tests/MyApp.Tests.csproj), common in monorepos that
+  // have no root .sln. Run each matched project explicitly rather than a
+  // bare `dotnet test` (which finds nothing to build at the root).
+  try {
+    const testProjects = globSync('**/*Tests.csproj', {
+      cwd: projectPath,
+      ignore: ['**/bin/**', '**/obj/**', '**/node_modules/**'],
+    });
+    if (testProjects.length > 0) {
+      return testProjects
+        .sort()
+        .map((p) => `dotnet test "${p}" --nologo -v q`)
+        .join(' && ');
+    }
+  } catch {
+    // Glob failure — fall through to next detection
+  }
+
+  // 4. package.json present AND has scripts.test → npm test
   const packageJsonPath = join(projectPath, 'package.json');
   if (existsSync(packageJsonPath)) {
     try {
@@ -59,7 +81,7 @@ export function resolveTestCommand(projectPath = '.') {
     }
   }
 
-  // 4. Nothing found
+  // 5. Nothing found
   return null;
 }
 

package/src/lib/validators/validation-runner.js

@@ -124,7 +124,10 @@ export async function runValidation(projectPath, featureName, options = {}) {
     const _profileForContracts = getStackProfile(projectPath);
     const contractsType = resolveContractsOutputType(_profileForContracts.stack);
     const contractsPath = getAbsoluteOutputPath(projectPath, featureName, contractsType);
-    if (existsSync(contractsPath) && !_profileForContracts.isNextjs) {
+    // Positive check on isDotnet (contract-compliance is a VSA/.NET concern) —
+    // NOT `!isNextjs`, which used to wrongly suppress it for a compound
+    // "dotnet-nextjs" stack that has a .NET backend but is ALSO isNextjs.
+    if (existsSync(contractsPath) && _profileForContracts.isDotnet) {
       validatorIds.push('contract-compliance');
     }
   }