ostacky 0.5.0 → 0.5.1

package/README.md

@@ -215,33 +215,33 @@ Tras instalar, el proyecto queda así:
 
 ```json
 {
-    "version": "0.5.0",
+    "version": "0.5.1",
     "lockedAt": "2025-01-01T00:00:00.000Z",
     "repo": "JaimeHoracio/Ostacky",
-    "tag": "v0.5.0",
+    "tag": "v0.5.1",
     "agents": {
         "ostacky": {
-            "version": "0.5.0",
+            "version": "0.5.1",
             "installedAt": "2025-01-01T00:00:00.000Z",
             "sha256": "abc123..."
         }
     },
     "commands": {
         "install-stack": {
-            "version": "0.5.0",
+            "version": "0.5.1",
             "installedAt": "2025-01-01T00:00:00.000Z",
             "sha256": "def456..."
         },
         "opsx-sync": {
-            "version": "0.5.0",
+            "version": "0.5.1",
             "installedAt": "2025-01-01T00:00:00.000Z",
             "sha256": "ghi789..."
         }
     },
     "skills": {
-        "brainstorming": { "version": "0.5.0", ... },
-        "execution-mode-evaluation": { "version": "0.5.0", ... },
-        "openspec-propose": { "version": "0.5.0", ... }
+        "brainstorming": { "version": "0.5.1", ... },
+        "execution-mode-evaluation": { "version": "0.5.1", ... },
+        "openspec-propose": { "version": "0.5.1", ... }
     }
 }
 ```
@@ -271,7 +271,7 @@ Es opcional y solo necesario si algo falló durante la instalación o si querés
 ## Seguridad
 
 - `opencode.jsonc` se versiona en el repo para compartir permisos y MCP de forma reproducible.
-- Las URLs de descarga usan **tags de GitHub** (ej. `v0.5.0`), nunca `main` — instalaciones reproducibles
+- Las URLs de descarga usan **tags de GitHub** (ej. `v0.5.1`), nunca `main` — instalaciones reproducibles
 - Cada path de archivo descargado es validado para prevenir **path traversal**
 - Los archivos incluyen **checksum SHA-256** opcional; si el manifest lo define, el contenido se verifica antes de escribir
 - El cache local (`~/.opencode/cache/`) también valida integridad al servir archivos cacheados

package/assets/mcp/ask-user-server/index.js

@@ -167,7 +167,7 @@ async function handleRequest(msg) {
                     },
                     serverInfo: {
                         name: 'ask-user-server',
-                        version: '0.5.0',
+                        version: '0.5.1',
                     },
                 });
                 break;

package/assets/mcp/ask-user-server/package.json

@@ -1,6 +1,6 @@
 {
     "name": "ask-user-server",
-    "version": "0.5.0",
+    "version": "0.5.1",
     "private": true,
     "type": "module"
 }

package/dist/cli.js

@@ -161,7 +161,7 @@ var require_picocolors = __commonJS((exports, module) => {
 // package.json
 var package_default = {
   name: "ostacky",
-  version: "0.5.0",
+  version: "0.5.1",
   description: "Instalador interactivo de agentes y comandos para OpenCode",
   type: "module",
   bin: {
@@ -876,15 +876,15 @@ var L2 = () => {
 import { join as join5 } from "path";
 // manifest.json
 var manifest_default = {
-  version: "0.5.0",
+  version: "0.5.1",
   repo: "JaimeHoracio/Ostacky",
-  tag: "v0.5.0",
+  tag: "v0.5.1",
   agents: [
     {
       name: "ostacky",
       file: "assets/agents/ostacky.md",
       description: "Orquestador principal con ruteo por nivel de impacto, máquina de estados persistida (controller MCP), edición segura con 3 outcomes, y delegación en OpenSpec + Superpowers",
-      version: "0.5.0",
+      version: "0.5.1",
       sha256: "355c729ee13f0cd99b9a16e0c203dd83808ec9fb5a764012521794011a91a9f1"
     }
   ],
@@ -893,14 +893,14 @@ var manifest_default = {
       name: "install-stack",
       file: "assets/commands/install-stack.md",
       description: "Instala el stack tecnológico del proyecto (CodeGraph, skills, OpenSpec, Engram, Context7, controller MCP)",
-      version: "0.5.0",
+      version: "0.5.1",
       sha256: "e8b75efe505766d47dd4d38cfb055b6801552cd457d99bce018c196a346e4560"
     },
     {
       name: "opsx-sync",
       file: "assets/commands/opsx-sync.md",
       description: "Sincroniza delta specs del change activo sin inicializar CodeGraph si ya existe índice",
-      version: "0.5.0",
+      version: "0.5.1",
       sha256: "ed9948f1910743b672e1dfad496bc96972a224a98b063232be39290d43068c50"
     }
   ],
@@ -909,15 +909,15 @@ var manifest_default = {
       name: "ask-user",
       file: "assets/mcp/ask-user-server/",
       description: "MCP server con tool ask_user para consultar al usuario con bloqueo de ejecución",
-      version: "0.5.0",
+      version: "0.5.1",
       sha256: "74deeadfde30dfd9e3fe5e6fb75ab9e024aa7150a40518b2f40354c392e8fb5d"
     },
     {
       name: "ostacky-controller",
       file: "assets/mcp/ostacky-controller/",
       description: "Máquina de estados persistida para Ostacky: validación de transiciones, consumo de decisiones, autorización de side effects, edición segura y snapshots reutilizables",
-      version: "0.5.0",
-      sha256: "7652bb70376a71157c35dda5a5c24f3671b0cf5922b2fef1065a40e7600d694e"
+      version: "0.5.1",
+      sha256: "c66291cf170a7579e611580e8ac6f4bb15857d7e71b43fb793d92ea3a0013951"
     }
   ],
   skills: [
@@ -925,77 +925,77 @@ var manifest_default = {
       name: "execution-mode-evaluation",
       file: "assets/skills/execution-mode-evaluation/SKILL.md",
       description: "Skill de análisis de modo de ejecución — output reconciliado con controller snapshot contract (recommendation field)",
-      version: "0.5.0",
+      version: "0.5.1",
       sha256: "0bf115b939c906e276040a9bd741c1b84d658da29e3c3fa2cb5e74e4c72792a1"
     },
     {
       name: "brainstorming",
       file: "assets/skills/brainstorming/SKILL.md",
       description: "Skill de descubrimiento y exploración (Superpowers)",
-      version: "0.5.0",
+      version: "0.5.1",
       sha256: "ad15458c54912007ae3f708c698a11f0a4a58e2a7c7860a1ae8874db573c9e98"
     },
     {
       name: "writing-plans",
       file: "assets/skills/writing-plans/SKILL.md",
       description: "Skill de planificación de implementación (Superpowers) — execution handoff removido, Ostacky decide modo de ejecución",
-      version: "0.5.0",
+      version: "0.5.1",
       sha256: "408bca4652677d59bb35cdbe7459b01555b4ca11bf44217d95f98f3641af88b2"
     },
     {
       name: "tdd",
       file: "assets/skills/tdd/SKILL.md",
       description: "Skill de test-driven development (Superpowers)",
-      version: "0.5.0",
+      version: "0.5.1",
       sha256: "7eacd8ee81dc5c0b85065c0392e37ee10314661f7edb59dd8e70a9e0dae8f371"
     },
     {
       name: "subagent-driven-development",
       file: "assets/skills/subagent-driven-development/SKILL.md",
       description: "Skill de ejecución con subagentes (Superpowers) — ejecuta solo después de confirmación del coordinador Ostacky",
-      version: "0.5.0",
+      version: "0.5.1",
       sha256: "2d0c880deae34ab6882c93f6c8de666013498f0e84d7741116dd0835e2780a1b"
     },
     {
       name: "dispatching-parallel-agents",
       file: "assets/skills/dispatching-parallel-agents/SKILL.md",
       description: "Skill de dispatch paralelo de agentes (Superpowers)",
-      version: "0.5.0",
+      version: "0.5.1",
       sha256: "281edf0c38f358497c7e2066fa8217a2ba3e2a39b4205c4af0c41d328fc035a1"
     },
     {
       name: "review",
       file: "assets/skills/review/SKILL.md",
       description: "Skill de revisión de código (Superpowers)",
-      version: "0.5.0",
+      version: "0.5.1",
       sha256: "c78d3beff45e9bb6b5e9e1d7155910f379e082d7aaf3c61b90d651d09e0d2e2a"
     },
     {
       name: "openspec-explore",
       file: "assets/skills/openspec-explore/SKILL.md",
       description: "Skill de exploración pre-spec (OpenSpec)",
-      version: "0.5.0",
+      version: "0.5.1",
       sha256: "1c75cab8672c2eb8b12f27a0b31f0b7fdb73c21c72583c1e8a566e09fe4108a6"
     },
     {
       name: "openspec-propose",
       file: "assets/skills/openspec-propose/SKILL.md",
       description: "Skill de generación de proposal (OpenSpec)",
-      version: "0.5.0",
+      version: "0.5.1",
       sha256: "ab88e9aa22db09559ccf7987d08adc0469b7c6fbe46498e63a3b64926f19e66b"
     },
     {
       name: "openspec-apply-change",
       file: "assets/skills/openspec-apply-change/SKILL.md",
       description: "Skill de aplicación de change (OpenSpec)",
-      version: "0.5.0",
+      version: "0.5.1",
       sha256: "847c2f88a66e8d5946c1fec422a5367c89eca5340839b659bbaaeb6f0bdac487"
     },
     {
       name: "openspec-archive-change",
       file: "assets/skills/openspec-archive-change/SKILL.md",
       description: "Skill de archivo de change (OpenSpec)",
-      version: "0.5.0",
+      version: "0.5.1",
       sha256: "3d35905ec40a081b8fd77b144426e540c67b1f50cc5801b655313a4fdfaa087a"
     }
   ]
@@ -1186,7 +1186,7 @@ function clearLockfile(opencodeRoot) {
   const lockfile = readLockfile(opencodeRoot);
   if (!lockfile) {
     writeLockfile(opencodeRoot, {
-      version: "0.5.0",
+      version: "0.5.1",
       lockedAt: new Date().toISOString(),
       repo: "",
       tag: "",

package/manifest.json

@@ -1,13 +1,13 @@
 {
-    "version": "0.5.0",
+    "version": "0.5.1",
     "repo": "JaimeHoracio/Ostacky",
-    "tag": "v0.5.0",
+    "tag": "v0.5.1",
     "agents": [
         {
             "name": "ostacky",
             "file": "assets/agents/ostacky.md",
             "description": "Orquestador principal con ruteo por nivel de impacto, máquina de estados persistida (controller MCP), edición segura con 3 outcomes, y delegación en OpenSpec + Superpowers",
-            "version": "0.5.0",
+            "version": "0.5.1",
             "sha256": "355c729ee13f0cd99b9a16e0c203dd83808ec9fb5a764012521794011a91a9f1"
         }
     ],
@@ -16,14 +16,14 @@
             "name": "install-stack",
             "file": "assets/commands/install-stack.md",
             "description": "Instala el stack tecnológico del proyecto (CodeGraph, skills, OpenSpec, Engram, Context7, controller MCP)",
-            "version": "0.5.0",
+            "version": "0.5.1",
             "sha256": "e8b75efe505766d47dd4d38cfb055b6801552cd457d99bce018c196a346e4560"
         },
         {
             "name": "opsx-sync",
             "file": "assets/commands/opsx-sync.md",
             "description": "Sincroniza delta specs del change activo sin inicializar CodeGraph si ya existe índice",
-            "version": "0.5.0",
+            "version": "0.5.1",
             "sha256": "ed9948f1910743b672e1dfad496bc96972a224a98b063232be39290d43068c50"
         }
     ],
@@ -32,15 +32,15 @@
             "name": "ask-user",
             "file": "assets/mcp/ask-user-server/",
             "description": "MCP server con tool ask_user para consultar al usuario con bloqueo de ejecución",
-            "version": "0.5.0",
+            "version": "0.5.1",
             "sha256": "74deeadfde30dfd9e3fe5e6fb75ab9e024aa7150a40518b2f40354c392e8fb5d"
         },
         {
             "name": "ostacky-controller",
             "file": "assets/mcp/ostacky-controller/",
             "description": "Máquina de estados persistida para Ostacky: validación de transiciones, consumo de decisiones, autorización de side effects, edición segura y snapshots reutilizables",
-            "version": "0.5.0",
-            "sha256": "7652bb70376a71157c35dda5a5c24f3671b0cf5922b2fef1065a40e7600d694e"
+            "version": "0.5.1",
+            "sha256": "c66291cf170a7579e611580e8ac6f4bb15857d7e71b43fb793d92ea3a0013951"
         }
     ],
     "skills": [
@@ -48,77 +48,77 @@
             "name": "execution-mode-evaluation",
             "file": "assets/skills/execution-mode-evaluation/SKILL.md",
             "description": "Skill de análisis de modo de ejecución — output reconciliado con controller snapshot contract (recommendation field)",
-            "version": "0.5.0",
+            "version": "0.5.1",
             "sha256": "0bf115b939c906e276040a9bd741c1b84d658da29e3c3fa2cb5e74e4c72792a1"
         },
         {
             "name": "brainstorming",
             "file": "assets/skills/brainstorming/SKILL.md",
             "description": "Skill de descubrimiento y exploración (Superpowers)",
-            "version": "0.5.0",
+            "version": "0.5.1",
             "sha256": "ad15458c54912007ae3f708c698a11f0a4a58e2a7c7860a1ae8874db573c9e98"
         },
         {
             "name": "writing-plans",
             "file": "assets/skills/writing-plans/SKILL.md",
             "description": "Skill de planificación de implementación (Superpowers) — execution handoff removido, Ostacky decide modo de ejecución",
-            "version": "0.5.0",
+            "version": "0.5.1",
             "sha256": "408bca4652677d59bb35cdbe7459b01555b4ca11bf44217d95f98f3641af88b2"
         },
         {
             "name": "tdd",
             "file": "assets/skills/tdd/SKILL.md",
             "description": "Skill de test-driven development (Superpowers)",
-            "version": "0.5.0",
+            "version": "0.5.1",
             "sha256": "7eacd8ee81dc5c0b85065c0392e37ee10314661f7edb59dd8e70a9e0dae8f371"
         },
         {
             "name": "subagent-driven-development",
             "file": "assets/skills/subagent-driven-development/SKILL.md",
             "description": "Skill de ejecución con subagentes (Superpowers) — ejecuta solo después de confirmación del coordinador Ostacky",
-            "version": "0.5.0",
+            "version": "0.5.1",
             "sha256": "2d0c880deae34ab6882c93f6c8de666013498f0e84d7741116dd0835e2780a1b"
         },
         {
             "name": "dispatching-parallel-agents",
             "file": "assets/skills/dispatching-parallel-agents/SKILL.md",
             "description": "Skill de dispatch paralelo de agentes (Superpowers)",
-            "version": "0.5.0",
+            "version": "0.5.1",
             "sha256": "281edf0c38f358497c7e2066fa8217a2ba3e2a39b4205c4af0c41d328fc035a1"
         },
         {
             "name": "review",
             "file": "assets/skills/review/SKILL.md",
             "description": "Skill de revisión de código (Superpowers)",
-            "version": "0.5.0",
+            "version": "0.5.1",
             "sha256": "c78d3beff45e9bb6b5e9e1d7155910f379e082d7aaf3c61b90d651d09e0d2e2a"
         },
         {
             "name": "openspec-explore",
             "file": "assets/skills/openspec-explore/SKILL.md",
             "description": "Skill de exploración pre-spec (OpenSpec)",
-            "version": "0.5.0",
+            "version": "0.5.1",
             "sha256": "1c75cab8672c2eb8b12f27a0b31f0b7fdb73c21c72583c1e8a566e09fe4108a6"
         },
         {
             "name": "openspec-propose",
             "file": "assets/skills/openspec-propose/SKILL.md",
             "description": "Skill de generación de proposal (OpenSpec)",
-            "version": "0.5.0",
+            "version": "0.5.1",
             "sha256": "ab88e9aa22db09559ccf7987d08adc0469b7c6fbe46498e63a3b64926f19e66b"
         },
         {
             "name": "openspec-apply-change",
             "file": "assets/skills/openspec-apply-change/SKILL.md",
             "description": "Skill de aplicación de change (OpenSpec)",
-            "version": "0.5.0",
+            "version": "0.5.1",
             "sha256": "847c2f88a66e8d5946c1fec422a5367c89eca5340839b659bbaaeb6f0bdac487"
         },
         {
             "name": "openspec-archive-change",
             "file": "assets/skills/openspec-archive-change/SKILL.md",
             "description": "Skill de archivo de change (OpenSpec)",
-            "version": "0.5.0",
+            "version": "0.5.1",
             "sha256": "3d35905ec40a081b8fd77b144426e540c67b1f50cc5801b655313a4fdfaa087a"
         }
     ]

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "ostacky",
-    "version": "0.5.0",
+    "version": "0.5.1",
     "description": "Instalador interactivo de agentes y comandos para OpenCode",
     "type": "module",
     "bin": {

package/assets/mcp/ostacky-controller/controller.test.mjs

@@ -1,447 +0,0 @@
-import { mkdtemp, rm, writeFile, readFile } from "node:fs/promises";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { describe, it, beforeEach, afterEach } from "node:test";
-import assert from "node:assert/strict";
-import {
-  OstackyController,
-  buildExecutionSnapshot,
-  RESULTS,
-  STATES,
-} from "./index.js";
-
-describe("OstackyController", () => {
-  let stateDir;
-
-  beforeEach(async () => {
-    stateDir = await mkdtemp(join(tmpdir(), "ostacky-controller-"));
-  });
-
-  afterEach(async () => {
-    await rm(stateDir, { recursive: true, force: true });
-  });
-
-  it("consumes the route decision once and authorizes the selected route", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    const started = await controller.startRequest({ requestId: "req-1" });
-    const discovered = await controller.recordDiscovery({
-      level: "0+1",
-      routeDecisionId: "route-1",
-    });
-
-    assert.strictEqual(started.state, STATES.INTERPRETATION_PENDING);
-    assert.strictEqual(discovered.state, STATES.ROUTE_DECISION_PENDING);
-
-    const selected = await controller.consumeRouteDecision({
-      decisionId: "route-1",
-      choice: "SPEC",
-    });
-
-    assert.strictEqual(selected.status, RESULTS.OK);
-    assert.strictEqual(selected.state, STATES.SPECIFICATION);
-    assert.strictEqual((await controller.authorize("openspec-propose")).status, RESULTS.OK);
-
-    const repeated = await controller.consumeRouteDecision({
-      decisionId: "route-1",
-      choice: "DIRECT",
-    });
-
-    assert.strictEqual(repeated.status, RESULTS.DECISION_ALREADY_CONSUMED);
-    assert.strictEqual(repeated.state, STATES.SPECIFICATION);
-  });
-
-  it("rejects side effects before a valid route decision", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    await controller.startRequest({ requestId: "req-2" });
-
-    const result = await controller.authorize("openspec-propose");
-
-    assert.strictEqual(result.status, RESULTS.INVALID_TRANSITION);
-  });
-
-  it("validates editable, already-applied, and conflicting edits", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-
-    assert.strictEqual(
-      controller.validateEdit({
-        oldString: "old",
-        newString: "new",
-        content: "before old after",
-      }).status,
-      RESULTS.EDITABLE,
-    );
-
-    assert.strictEqual(
-      controller.validateEdit({
-        oldString: "same",
-        newString: "same",
-        content: "same",
-      }).status,
-      RESULTS.ALREADY_APPLIED,
-    );
-
-    assert.strictEqual(
-      controller.validateEdit({
-        oldString: "old",
-        newString: "new",
-        content: "before new after",
-      }).status,
-      RESULTS.ALREADY_APPLIED,
-    );
-
-    assert.strictEqual(
-      controller.validateEdit({
-        oldString: "old",
-        newString: "new",
-        content: "unchanged",
-      }).status,
-      RESULTS.CONFLICT,
-    );
-  });
-
-  it("persists state and restores it in a new controller instance", async () => {
-    const statePath = join(stateDir, "state.json");
-    const first = new OstackyController({ statePath });
-    await first.startRequest({ requestId: "req-3", changeId: "change-3" });
-    await first.recordDiscovery({ level: "1+", routeDecisionId: "route-3" });
-    await first.consumeRouteDecision({ decisionId: "route-3", choice: "DIRECT" });
-
-    const restored = new OstackyController({ statePath });
-    const state = await restored.getState();
-
-    assert.strictEqual(state.requestId, "req-3");
-    assert.strictEqual(state.changeId, "change-3");
-    assert.strictEqual(state.state, STATES.EXECUTION_ANALYSIS);
-  });
-
-  it("resumes active flow without resetting on repeated startRequest", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    await controller.startRequest({ requestId: "req-4" });
-    await controller.recordDiscovery({ level: "0+1", routeDecisionId: "route-4" });
-    await controller.consumeRouteDecision({ decisionId: "route-4", choice: "SPEC" });
-
-    // Second startRequest on existing flow — should resume, not reset
-    const resume = await controller.startRequest({ requestId: "req-5" });
-
-    assert.strictEqual(resume.state, STATES.SPECIFICATION);
-    assert.strictEqual(resume.requestId, "req-4"); // preserved original
-  });
-
-  it("rejects duplicate route decision consumption with same decisionId", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    await controller.startRequest({ requestId: "req-6" });
-    await controller.recordDiscovery({ level: "0+1", routeDecisionId: "route-6" });
-
-    const first = await controller.consumeRouteDecision({ decisionId: "route-6", choice: "SPEC" });
-    assert.strictEqual(first.status, RESULTS.OK);
-
-    const second = await controller.consumeRouteDecision({ decisionId: "route-6", choice: "SPEC" });
-    assert.strictEqual(second.status, RESULTS.DECISION_ALREADY_CONSUMED);
-
-    const third = await controller.consumeRouteDecision({ decisionId: "route-6", choice: "DIRECT" });
-    assert.strictEqual(third.status, RESULTS.DECISION_ALREADY_CONSUMED);
-    assert.strictEqual(third.state, STATES.SPECIFICATION); // still SPEC
-  });
-
-  it("rejects consumeRouteDecision with wrong decision ID", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    await controller.startRequest({ requestId: "req-7" });
-    await controller.recordDiscovery({ level: "0+1", routeDecisionId: "route-7" });
-
-    const result = await controller.consumeRouteDecision({ decisionId: "wrong-id", choice: "SPEC" });
-    assert.strictEqual(result.status, RESULTS.INVALID_TRANSITION);
-    assert.match(result.reason, /Decision ID mismatch/);
-  });
-
-  it("supports clarification flow: interpret → clarify → discover", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    await controller.startRequest({ requestId: "req-8" });
-
-    const clarified = await controller.requestClarification({ question: "What area?" });
-    assert.strictEqual(clarified.state, STATES.CLARIFICATION_PENDING);
-
-    const answered = await controller.recordClarification();
-    assert.strictEqual(answered.state, STATES.DISCOVERY);
-
-    const discovered = await controller.recordDiscovery({ level: "1+", routeDecisionId: "route-8" });
-    assert.strictEqual(discovered.state, STATES.ROUTE_DECISION_PENDING);
-  });
-
-  it("supports cancel flow via block", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    await controller.startRequest({ requestId: "req-9" });
-
-    const blocked = await controller.block({ reason: "Cancelled by user" });
-    assert.strictEqual(blocked.state, STATES.BLOCKED);
-
-    const state = await controller.getState();
-    assert.strictEqual(state.error, "Cancelled by user");
-  });
-
-  it("supports replan from BLOCKED back to INTERPRETATION_PENDING", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    await controller.startRequest({ requestId: "req-10", changeId: "change-10" });
-    await controller.block({ reason: "Blocked" });
-
-    const replanned = await controller.replan({ reason: "Retry with new approach" });
-    assert.strictEqual(replanned.state, STATES.INTERPRETATION_PENDING);
-
-    const state = await controller.getState();
-    // Replan preserves requestId and changeId but resets workflow
-    assert.strictEqual(state.requestId, "req-10");
-    assert.strictEqual(state.changeId, "change-10");
-    assert.strictEqual(state.routeDecisionId, null);
-    assert.strictEqual(state.routeChoice, null);
-  });
-
-  it("records execution analysis and consumes execution decision", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    await controller.startRequest({ requestId: "req-11" });
-    await controller.recordDiscovery({ level: "0+1", routeDecisionId: "route-11" });
-    await controller.consumeRouteDecision({ decisionId: "route-11", choice: "DIRECT" });
-
-    // Now in EXECUTION_ANALYSIS
-    const analysis = await controller.recordExecutionAnalysis({
-      executionDecisionId: "exec-11",
-      snapshot: {
-        recommendation: "INLINE",
-        sharedFiles: ["assets/agents/ostacky.md"],
-        estimatedLines: 30,
-        reasons: ["Single file change, no subagent overhead"],
-      },
-    });
-    assert.strictEqual(analysis.state, STATES.EXECUTION_DECISION_PENDING);
-    assert.strictEqual(analysis.executionDecisionId, "exec-11");
-
-    const confirmed = await controller.consumeExecutionDecision({
-      decisionId: "exec-11",
-      mode: "INLINE",
-    });
-    assert.strictEqual(confirmed.status, RESULTS.OK);
-    assert.strictEqual(confirmed.state, STATES.EXECUTING_INLINE);
-    assert.strictEqual(confirmed.executionMode, "INLINE");
-    assert.ok(confirmed.allowedActions.includes("execution-start"));
-  });
-
-  it("rejects stale revisions via validateSnapshot", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    const before = await controller.getState();
-    const revBefore = before.revision;
-
-    await controller.startRequest({ requestId: "req-12" });
-
-    const after = await controller.getState();
-    const revAfter = after.revision;
-
-    // Revision changed after mutation
-    assert.notStrictEqual(revAfter, revBefore);
-
-    // Old revision should be stale
-    const valid = await controller.validateSnapshot(revBefore);
-    assert.strictEqual(valid.valid, false);
-    assert.strictEqual(valid.currentRevision, revAfter);
-  });
-
-  it("enforces recommendation → user confirmation → execution mode consistency (cannot diverge)", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    await controller.startRequest({ requestId: "req-14" });
-    await controller.recordDiscovery({ level: "0+1", routeDecisionId: "route-14" });
-    await controller.consumeRouteDecision({ decisionId: "route-14", choice: "DIRECT" });
-
-    // Record analysis with INLINE recommendation
-    await controller.recordExecutionAnalysis({
-      executionDecisionId: "exec-14",
-      snapshot: buildExecutionSnapshot({
-        recommendation: "INLINE",
-        filesPerTask: { "1": ["a.ts"] },
-        sharedFiles: {},
-        fileClusters: [["1"]],
-        taskCount: 1,
-        clusterCount: 1,
-        estLines: 5,
-        reasons: ["Single task, no shared files"],
-      }),
-    });
-
-    // User confirms SUBAGENT_DRIVEN despite INLINE recommendation — user is the authority
-    const confirmed = await controller.consumeExecutionDecision({
-      decisionId: "exec-14",
-      mode: "SUBAGENT_DRIVEN",
-    });
-    assert.strictEqual(confirmed.status, RESULTS.OK);
-    assert.strictEqual(confirmed.state, STATES.EXECUTING_SUBAGENTS);
-    assert.strictEqual(confirmed.executionMode, "SUBAGENT_DRIVEN");
-
-    // Verify persisted mode matches confirmed mode (not recommendation)
-    const state = await controller.getState();
-    assert.strictEqual(state.executionMode, "SUBAGENT_DRIVEN");
-    // The recommendation is still stored in the snapshot
-    assert.strictEqual(state.snapshots.execution.recommendation, "INLINE");
-  });
-
-  it("prevents execution start without an execution decision", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    await controller.startRequest({ requestId: "req-15" });
-    await controller.recordDiscovery({ level: "1+", routeDecisionId: "route-15" });
-    await controller.consumeRouteDecision({ decisionId: "route-15", choice: "SPEC" });
-    await controller.specComplete();
-    // In EXECUTION_ANALYSIS now — no decision consumed yet
-
-    // Try to authorize execution-start before confirmation
-    const auth = await controller.authorize("execution-start");
-    assert.strictEqual(auth.status, RESULTS.INVALID_TRANSITION);
-    assert.strictEqual(auth.allowed, false);
-  });
-
-  it("gates edit authorization on EDITABLE validation result", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    await controller.startRequest({ requestId: "req-16" });
-    await controller.recordDiscovery({ level: "0+1", routeDecisionId: "route-16" });
-    await controller.consumeRouteDecision({ decisionId: "route-16", choice: "DIRECT" });
-    await controller.recordExecutionAnalysis({
-      executionDecisionId: "exec-16",
-      snapshot: { recommendation: "INLINE", sharedFiles: {}, estLines: 5, reasons: [] },
-    });
-    await controller.consumeExecutionDecision({ decisionId: "exec-16", mode: "INLINE" });
-
-    // Authorize with ALREADY_APPLIED — should be rejected
-    const badAuth = await controller.authorize("edit", { editResult: RESULTS.ALREADY_APPLIED });
-    assert.strictEqual(badAuth.status, RESULTS.ACTION_NOT_AUTHORIZED);
-    assert.strictEqual(badAuth.allowed, false);
-
-    // Authorize with EDITABLE — should pass
-    const goodAuth = await controller.authorize("edit", { editResult: RESULTS.EDITABLE });
-    assert.strictEqual(goodAuth.status, RESULTS.OK);
-    assert.strictEqual(goodAuth.allowed, true);
-
-    // Authorize without editResult — should be rejected
-    const noResultAuth = await controller.authorize("edit");
-    assert.strictEqual(noResultAuth.status, RESULTS.ACTION_NOT_AUTHORIZED);
-
-
-    // Authorize with CONFLICT — should be rejected
-    const conflictAuth = await controller.authorize("edit", { editResult: RESULTS.CONFLICT });
-    assert.strictEqual(conflictAuth.status, RESULTS.ACTION_NOT_AUTHORIZED);
-  });
-
-  it("authorizes task-complete and sync only from correct states", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    // From INTERPRETATION_PENDING — no ops authorized
-    assert.strictEqual((await controller.authorize("task-complete")).status, RESULTS.INVALID_TRANSITION);
-    assert.strictEqual((await controller.authorize("sync")).status, RESULTS.INVALID_TRANSITION);
-  });
-
-  it("buildExecutionSnapshot validates recommendation field", () => {
-    assert.throws(
-      () => buildExecutionSnapshot({ recommendation: "INVALID" }),
-      /Invalid recommendation/,
-    );
-
-    const valid = buildExecutionSnapshot({ recommendation: "INLINE", estLines: 10 });
-    assert.strictEqual(valid.recommendation, "INLINE");
-    assert.strictEqual(valid.estLines, 10);
-    assert.strictEqual(valid.taskCount, 0); // default
-    assert.deepStrictEqual(valid.filesPerTask, {});
-  });
-
-  it("records and retrieves CodeGraph snapshots for phase reuse", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-
-    await controller.recordCodegraphSnapshot({
-      context: { symbols: ["AuthService"], files: ["src/auth.ts"] },
-      calls: ["codegraph_context"],
-    });
-
-    const snapshot = await controller.getCodegraphSnapshot();
-    assert.ok(snapshot);
-    assert.ok(snapshot.capturedAt);
-    assert.deepStrictEqual(snapshot.data.context.symbols, ["AuthService"]);
-  });
-
-  it("detects stale file fingerprints and returns REPLAN_REQUIRED", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    // Use the revision after a state transition (startRequest increments revision)
-    await controller.startRequest({ requestId: "req-fp" });
-    const rev = (await controller.getState()).revision;
-
-    // Record fingerprints for a file
-    await controller.recordFileFingerprints({
-      "src/auth.ts": { mtime: 1000, size: 500 },
-    });
-
-    // Same fingerprints — valid
-    const valid = await controller.validateSnapshot(rev, {
-      "src/auth.ts": { mtime: 1000, size: 500 },
-    });
-    assert.strictEqual(valid.valid, true);
-
-    // Changed mtime — stale
-    const stale = await controller.validateSnapshot(rev, {
-      "src/auth.ts": { mtime: 2000, size: 500 },
-    });
-    assert.strictEqual(stale.valid, false);
-    assert.strictEqual(stale.status, RESULTS.REPLAN_REQUIRED);
-    assert.deepStrictEqual(stale.staleFingerprints, ["src/auth.ts"]);
-  });
-
-  it("records and retrieves Engram keys for boundary persistence", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-
-    await controller.recordEngramKey({
-      changeKey: "change/redesign-ostacky-orchestration",
-      taskKey: "task/controller-core",
-    });
-
-    const keys = await controller.getEngramKeys();
-    assert.strictEqual(keys.changeKey, "change/redesign-ostacky-orchestration");
-    assert.strictEqual(keys.taskKey, "task/controller-core");
-  });
-
-  it("increments and retrieves instrumentation counters", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-
-    const val1 = await controller.incrementCounter("codegraph_calls");
-    assert.strictEqual(val1, 1);
-
-    const val2 = await controller.incrementCounter("codegraph_calls");
-    assert.strictEqual(val2, 2);
-
-    const val3 = await controller.incrementCounter("subagent_dispatches", 3);
-    assert.strictEqual(val3, 3);
-
-    const counters = await controller.getCounters();
-    assert.strictEqual(counters.codegraph_calls, 2);
-    assert.strictEqual(counters.subagent_dispatches, 3);
-
-    await controller.resetCounters();
-    const empty = await controller.getCounters();
-    assert.deepStrictEqual(empty, {});
-  });
-
-  it("supports complete task lifecycle: execute, complete task, sync, done", async () => {
-    const controller = new OstackyController({ statePath: join(stateDir, "state.json") });
-    await controller.startRequest({ requestId: "req-13" });
-    await controller.recordDiscovery({ level: "0+1", routeDecisionId: "route-13" });
-    await controller.consumeRouteDecision({ decisionId: "route-13", choice: "DIRECT" });
-    await controller.recordExecutionAnalysis({
-      executionDecisionId: "exec-13",
-      snapshot: { recommendation: "INLINE", sharedFiles: [], estimatedLines: 10, reasons: [] },
-    });
-    await controller.consumeExecutionDecision({ decisionId: "exec-13", mode: "INLINE" });
-
-    const taskResult = await controller.completeTask({ taskId: "task-1", note: "Implemented controller" });
-    assert.strictEqual(taskResult.status, RESULTS.OK);
-    assert.ok(taskResult.taskState.completedAt);
-
-    const tasks = await controller.getTasks();
-    assert.ok(tasks["task-1"]);
-    assert.strictEqual(tasks["task-1"].note, "Implemented controller");
-
-    // Complete implementation, sync, mark done
-    const implDone = await controller.implementationComplete();
-    assert.strictEqual(implDone.state, STATES.SYNC);
-
-    const syncDone = await controller.syncComplete();
-    assert.strictEqual(syncDone.state, STATES.DONE);
-  });
-});