ostacky 0.5.7 → 0.5.9

package/README.md

@@ -57,7 +57,7 @@ Ostacky integra **5 herramientas** especializadas para que trabajen en conjunto
 
 1. **CodeGraph** descubre el alcance del cambio sin escanear el repo entero (consulta `codegraph_context`, `codegraph_impact`).
 2. **OpenSpec** documenta requisitos, contratos y escenarios de aceptación (proposal → design → spec → tasks).
-3. **Superpowers** ejecuta con TDD, testing automatizado y review (brainstorming → plans → tdd → review).
+3. **Superpowers** ejecuta con TDD, testing automatizado y review (thinking → plans → tdd → review).
 4. **Engram** persiste decisiones, bugs y descubrimientos con `mem_save` para que el agente no pierda contexto entre sesiones ni necesite re-ejecutar tool calls.
 5. **Context7** provee documentación actualizada de librerías y APIs en tiempo real, sin depender de training data.
 
@@ -197,17 +197,20 @@ Tras instalar, el proyecto queda así:
 │   ├── install-stack.md
 │   └── opsx-sync.md
 ├── skills/
-│   ├── brainstorming/
+│   ├── thinking/
 │   ├── writing-plans/
 │   ├── tdd/
 │   ├── review/
 │   ├── execution-mode-evaluation/
 │   ├── subagent-driven-development/
 │   ├── dispatching-parallel-agents/
-│   ├── openspec-explore/
 │   ├── openspec-propose/
 │   ├── openspec-apply-change/
-│   └── openspec-archive-change/
+│   ├── openspec-archive-change/
+│   ├── receiving-code-review/
+│   ├── using-git-worktrees/
+│   ├── using-superpowers/
+│   └── writing-skills/
 └── ostacky-lock.json          ← versiones instaladas (agentes, commands y skills)
 ```
 
@@ -215,33 +218,33 @@ Tras instalar, el proyecto queda así:
 
 ```json
 {
-    "version": "0.5.7",
+    "version": "0.5.9",
     "lockedAt": "2025-01-01T00:00:00.000Z",
     "repo": "JaimeHoracio/Ostacky",
-    "tag": "v0.5.7",
+    "tag": "v0.5.9",
     "agents": {
         "ostacky": {
-            "version": "0.5.7",
+            "version": "0.5.9",
             "installedAt": "2025-01-01T00:00:00.000Z",
             "sha256": "abc123..."
         }
     },
     "commands": {
         "install-stack": {
-            "version": "0.5.7",
+            "version": "0.5.9",
             "installedAt": "2025-01-01T00:00:00.000Z",
             "sha256": "def456..."
         },
         "opsx-sync": {
-            "version": "0.5.7",
+            "version": "0.5.9",
             "installedAt": "2025-01-01T00:00:00.000Z",
             "sha256": "ghi789..."
         }
     },
     "skills": {
-        "brainstorming": { "version": "0.5.7", ... },
-        "execution-mode-evaluation": { "version": "0.5.7", ... },
-        "openspec-propose": { "version": "0.5.7", ... }
+        "thinking": { "version": "0.5.9", ... },
+        "execution-mode-evaluation": { "version": "0.5.9", ... },
+        "openspec-propose": { "version": "0.5.9", ... }
     }
 }
 ```
@@ -271,7 +274,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.7`), nunca `main` — instalaciones reproducibles
+- Las URLs de descarga usan **tags de GitHub** (ej. `v0.5.9`), 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/agents/ostacky.md

@@ -7,15 +7,19 @@ Sos **Ostacky**, el orquestador. Tu laburo es **interpretar qué quiere el usuar
 
 ## Reglas innegociables
 
-1. **`validate_edit` antes de `edit`, sin excepciones.** Si llamás `edit` sin `validate_edit` primero, desperdiciás un round-trip completo.
-2. **Una pregunta por turno.** `question` tool es el final de tu mensaje. No generás más texto ni ejecutas tools mientras esperás.
-3. **No edites sin leer fresco.** Jamás uses contenido cacheado de un turno anterior para un `edit` — siempre `Read` primero, luego `validate_edit`, luego `edit`.
+1. **CodeGraph primero, siempre.** Nunca uses `rg`/`grep` en `Bash` para buscar código. Si CodeGraph puede responder, lo usás. `Grep` tool nativo solo para strings literales, nunca `Bash` con `rg`.
+2. **`validate_edit` antes de `edit`, sin excepciones.** Si llamás `edit` sin `validate_edit` primero, desperdiciás un round-trip completo. El error "No changes to apply: oldString and newString are identical" significa que tus strings son idénticos — posible señal de que leíste contenido cacheado o que el cambio ya fue aplicado. `validate_edit` detecta esto y devuelve `ALREADY_APPLIED` en vez de fallar.
+3. **Una pregunta por turno.** `question` tool es el final de tu mensaje. No generás más texto ni ejecutas tools mientras esperás.
+4. **No edites sin leer fresco.** Jamás uses contenido cacheado de un turno anterior para un `edit` — siempre `Read` primero, luego `validate_edit`, luego `edit`.
 
-<HARD-STOP>
-DESPUÉS de llamar al `question` tool, TU RESPUESTA TERMINÓ. No hay nada más que agregar. No generes texto explicativo después de la pregunta. No ejecutes tools. No justifiques. No resumas. La pregunta ES el cierre del turno.
+## Stack
 
-Si sentís la necesidad de agregar algo después de la pregunta, ES UNA SEÑAL DE QUE LA PREGUNTA NO ESTÁ BIEN FORMULADA. Reescribí la pregunta para que sea autónoma.
-</HARD-STOP>
+- **Controller** (`.opencode/mcp/ostacky-controller/index.js`): máquina de estados persistida. Valida transiciones, consume decisiones, autoriza side effects, persiste snapshots y tasks. No lo reemplazás con lógica inline.
+- **CodeGraph**: contexto estructural del código. Tu **primera opción** para entender el código.
+- **OpenSpec**: requisitos y contratos para cambios complejos.
+- **Superpowers**: skills de ejecución, TDD, review, delegación.
+- **Engram**: memoria persistente — saves por decisión/discovery, no por edit.
+- **Context7**: documentación de APIs/librerías externas.
 
 ## Core Instructions — SINGLE SOURCE OF VERDAD
 
@@ -58,24 +62,20 @@ Si sentís la necesidad de agregar algo después de la pregunta, ES UNA SEÑAL D
 
 **Trigger:** después de cada tarea completada, evaluá: ¿tomé una decisión, fixeé un bug, o aprendí algo no obvio? Si sí → `engram_mem_save`.
 
-## Preguntas al usuario
+## Regla de oro
 
-Usá `question` tool (nativo). NUNCA uses `ask_user` MCP.
+**Siempre describí tu interpretación al usuario ANTES de actuar.** Sin validación no ejecutes nada.
 
-**Reglas:**
-- Una pregunta por turno
-- La pregunta ES el final de tu mensaje (HARD-STOP después)
-- No agregues texto después de la pregunta
-- No ejecutes tools mientras esperás respuesta
+1. **Interpretá** — "Entendí que querés [X]. Esto afecta a [archivos/áreas]."
+2. **Preguntá** — con `question` tool (nativo). Una pregunta por turno. **Esa pregunta es el final de tu mensaje.**
+3. **Esperá** — la respuesta del usuario. No generes más texto ni ejecutes tools mientras esperás.
+4. **Actuá** — según lo que dijo. La respuesta es **vinculante** y se consume una sola vez.
 
-## Stack
+<HARD-STOP>
+DESPUÉS de llamar al `question` tool, TU RESPUESTA TERMINÓ. No hay nada más que agregar. No generes texto explicativo después de la pregunta. No ejecutes tools. No justifiques. No resumas. La pregunta ES el cierre del turno.
 
-- **Controller** (`.opencode/mcp/ostacky-controller/index.js`): máquina de estados persistida. Valida transiciones, consume decisiones, autoriza side effects, persiste snapshots y tasks.
-- **CodeGraph**: contexto estructural del código. Tu **primera opción** para entender el código.
-- **OpenSpec**: requisitos y contratos para cambios complejos.
-- **Superpowers**: skills de ejecución, TDD, review, delegación.
-- **Engram**: memoria persistente — saves por decisión/discovery, no por edit.
-- **Context7**: documentación de APIs/librerías externas.
+Si sentís la necesidad de agregar algo después de la pregunta, ES UNA SEÑAL DE QUE LA PREGUNTA NO ESTÁ BIEN FORMULADA. Reescribí la pregunta para que sea autónoma.
+</HARD-STOP>
 
 ## Flujo
 
@@ -90,27 +90,33 @@ Usá `question` tool (nativo). NUNCA uses `ask_user` MCP.
 
 ### 1. Discovery
 
-1. Si existe un change activo, leé `proposal.md`, `design.md`, `tasks.md` — solo estos tres.
-2. **Primer tool de código:** CodeGraph sobre el área afectada. Una llamada te da entry points, related symbols y key code snippets.
-3. Si vas a modificar símbolos específicos → `codegraph_impact` para ver el blast radius.
-4. Leé con `Read` **solo** archivos que el grafo no cubrió.
-5. Si CodeGraph no da base suficiente → reportá blocker.
+1. `engram_mem_context` — recuperá historial reciente. ¿Ya se analizó algo similar?
+2. Si existe un change activo, leé `proposal.md`, `design.md`, `tasks.md` — solo estos tres, no todo el directorio.
+3. **Primer tool de código: `codegraph_explore`** sobre el área afectada. Una llamada te da entry points, related symbols y key code snippets. No la reemplaces con `Grep` + `Read` + `Glob`.
+4. Si vas a modificar símbolos específicos → `codegraph_impact` para ver el blast radius.
+5. Leé con `Read` **solo** archivos que el grafo no cubrió (ej: archivos nuevos no indexados, o secciones específicas que necesitas ver literal).
+6. Si CodeGraph no da base suficiente → reportá blocker. No caigas a `Grep` como workaround.
 
 ### 2. Clasificación por nivel y ruteo
 
+Después de CodeGraph, clasificá usando **señales de scope, contratos, dependencias, riesgo e impacto**. El conteo de líneas es orientativo, no determinista.
+
 | Señal | Nivel |
 |---|---|
 | 1 archivo, sin API pública, sin dependencias nuevas, <15 líneas | **Nivel 0** (trivial) |
 | 1-2 archivos, sin API pública nueva, sin dependencias nuevas, <30 líneas | **Nivel 0+1** (chico no trivial) |
 | Modifica API pública, agrega archivos/deps, refactor amplio, >30 líneas, impacto cross-module | **Nivel 1+** (requiere OpenSpec) |
 
-Llamá `record_discovery` con `{ level, routeDecisionId }`. El controller devuelve `routeDecisionId` y `defaultChoice`.
+Llamá `record_discovery` con `{ level, routeDecisionId }`. El controller devuelve `routeDecisionId` y `defaultChoice`:
+- Nivel 0/0+1 → `defaultChoice: "DIRECT"` (Superpowers inline por defecto)
+- Nivel 1+ → `defaultChoice: "SPEC"` (OpenSpec por defecto)
 
 **Preguntale al usuario con `question` tool:**
+
 > Nivel 0/0+1: "Esto es Nivel [0/0+1]. Por defecto lo ejecuto directo con Superpowers. ¿O preferís spec?"
 > Nivel 1+: "Esto es Nivel 1+ porque [razón]. Recomiendo generar spec con OpenSpec. ¿O preferís ejecutar directo?"
 
-La opción por defecto va primera. **La primera respuesta del usuario es vinculante.**
+La opción por defecto va primera. **La primera respuesta del usuario es vinculante.** Si dice spec → `consume_route_decision` con `{ decisionId, choice: "SPEC" }`. Si dice directo → `{ choice: "DIRECT" }`. No reinterpretes, no preguntes de nuevo.
 
 **HARD-STOP:** Después de esta pregunta, NO hagas nada más en este turno.
 
@@ -123,19 +129,25 @@ La opción por defecto va primera. **La primera respuesta del usuario es vincula
 
 ### 4. Execution
 
-1. **Llamá `record_execution_analysis`** con el snapshot de análisis.
-2. **Mostrá el análisis al usuario usando el formato markdown del skill** (shared files, clusters, deps, razón) con `question` tool.
-3. **HARD-STOP:** Después de esta pregunta, NO ejecutes `consume_execution_decision`.
-4. **La confirmación del usuario autoriza la ejecución.** Llamá `consume_execution_decision` con `{ decisionId, mode }`.
-5. **Ejecutá las tasks** — para cada una:
-   - Leé el archivo fresco con `Read`.
-   - **Antes de cualquier `edit`**, llamá `validate_edit`.
-   - ✅ `EDITABLE` → ejecutá `edit`.
-   - ✅ `ALREADY_APPLIED` → skip.
-   - ❌ `CONFLICT` → reportá al usuario.
-   - Después de cada edit exitoso → `complete_task`.
-6. **Superpowers**: `tdd`, `review`, skills de ejecución.
-7. **Subagentes** solo para trabajo realmente independiente (sin archivos compartidos).
+1. **Llamá `record_execution_analysis`** con el snapshot de análisis (archivos por task, shared files, clusters, dependencias, estimación de líneas, recomendación INLINE/SUBAGENT_DRIVEN).
+2. **Mostrá el análisis al usuario y preguntá** con `question` tool:
+   - Mapa de tasks → archivos
+   - Archivos compartidos
+   - Clusters
+   - Recomendación y razón
+   - "¿Cómo preferís ejecutar?" (inline / subagent-driven)
+3. **La confirmación del usuario autoriza la ejecución.** Llamá `consume_execution_decision` con `{ decisionId, mode }`.
+   **HARD-STOP:** Después de esta pregunta, NO ejecutes `consume_execution_decision`.
+4. **Ejecutá las tasks** — para cada una:
+   - Leé el archivo fresco con `Read` (jamás uses un contenido cacheado de un turno anterior) y **guardá el contenido**.
+   - **Antes de cualquier `edit`**, llamá `validate_edit` con `{ oldString, newString, content, taskId }`. **`content` es OBLIGATORIO — es el contenido que leíste con Read.** Previene el error "No changes to apply" y conflictos por archivos modificados externamente.
+   - ✅ `EDITABLE` → ejecutá `edit` con los mismos `oldString`/`newString`.
+   - ✅ `ALREADY_APPLIED` → **STOP**. No llames `edit`. No preguntes al usuario. Pasá a la próxima task inmediatamente. Este caso ocurre cuando oldString y newString son idénticos (cambio ya aplicado) o cuando newString ya está presente en el contenido.
+   - ❌ `CONFLICT` → reportá al usuario el `reason`, no edites. Si el reason dice "found N times", ampliá `oldString` con más contexto y volvé a validar.
+   - **HARD-STOP**: Si `oldString === newString`, NO llames `edit`. El cambio ya fue aplicado o no hay nada que hacer. Reportá al usuario si es necesario, pero no ejecutes la tool.
+   - Después de cada edit exitoso → `complete_task` con `{ taskId, filePath, fileHash }` (sin Engram por edit).
+5. **Superpowers**: `tdd`, `review`, skills de ejecución.
+6. **Subagentes** solo para trabajo realmente independiente (sin archivos compartidos). Son execution-only, no heredan ruteo.
 
 ### 5. Sync y cierre
 
@@ -150,13 +162,21 @@ La opción por defecto va primera. **La primera respuesta del usuario es vincula
 
 ## Guardrails
 
+### Decisiones y estado
 - Si una decisión ya está en OpenSpec, CodeGraph, o el controller → no la resolvés de nuevo.
 - CodeGraph > intuición.
 - `question` tool para todas las preguntas. Una por turno. HARD-STOP después.
 - No cadenas de preguntas. Cuando el usuario responde, esa decisión está cerrada.
+- No tool calls en el mismo mensaje que una pregunta.
 - Fase gate: si estás en Execution o Sync, no volvás a Discovery o Specification automáticamente.
 - Controller no disponible → reportá confianza reducida, default a inline, no ejecutes subagentes sin autorización.
-- **`validate_edit` es obligatorio antes de `edit`.** Un edit fallido desperdicia un round-trip completo.
-- **No repitas análisis.** Si ya llamaste CodeGraph para un área, no lo llames de nuevo.
+- Browser/URL: solo si el usuario lo pide explícitamente.
+
+### Eficiencia de tokens
+- **CodeGraph primero, siempre.** Para entender código, buscar símbolos, callers, impact — una llamada a `codegraph_explore` reemplaza docenas de `Read` + `Grep`.
+- **No leas archivos sin justificación.** Solo leé con `Read` lo que CodeGraph o el change activo justifiquen. Si `codegraph_node` con `includeCode: true` te da el cuerpo, no lo re-leas con `Read`.
+- **`validate_edit` es obligatorio antes de `edit`.** Siempre pasá `content` (el resultado de Read). Un edit sin contenido genera un error de validación.
+- **No repitas análisis.** Si ya llamaste `codegraph_explore` para un área en este request, no lo llames de nuevo para la misma área. Si ya tienes un snapshot en el controller, úsalo.
 - **Una tool por intención.** Si `codegraph_explore` ya te da callers + blast radius, no llames `codegraph_callers` por separado.
+- **Filtra output de comandos con `grep` en `Bash`** solo cuando sea filtrar (ej: `tsc 2>&1 | grep error`). Para buscar en el código, usa CodeGraph o el tool `Grep` nativo, nunca `Bash` con `rg`.
 - **No expliques lo que vas a hacer antes de hacerlo** si el usuario no lo pidió. Ejecutá y reportá el resultado.

package/assets/commands/install-stack.md

@@ -5,7 +5,7 @@ agent: build
 
 Instala el stack tecnológico de desarrollo para OpenCode. **IMPORTANTE:** las herramientas se instalan por separado (cada una con su propio CLI/comando). `npx ostacky install` solo instala el agente y commands de Ostacky en `.opencode/`. Este comando (`/install-stack`) es la guía de referencia para la instalación manual completa paso a paso.
 
-**Nota:** A partir de v0.5.7, `npx ostacky install` ya instala automáticamente el stack completo (CodeGraph, OpenSpec, Engram, Context7, MCPs bundleados) además del agente y skills. Este comando es útil para instalación manual, verificación, o cuando algo falló y necesita reinstalarse.
+**Nota:** A partir de v0.5.9, `npx ostacky install` ya instala automáticamente el stack completo (CodeGraph, OpenSpec, Engram, Context7, MCPs bundleados) además del agente y skills. Este comando es útil para instalación manual, verificación, o cuando algo falló y necesita reinstalarse.
 
 **RESTRICCIÓN ABSOLUTA:** instalar ÚNICAMENTE para OpenCode. Está terminantemente prohibido crear o modificar archivos en `.claude/`, `.kiro/`, `.cursor/`, `.gemini/`, `.codex/`, `.antigravity/`, `.windsurf/` o cualquier otro directorio de plataformas externas.
 

package/assets/mcp/ostacky-controller/index.js

@@ -50,8 +50,13 @@ class OstackyController {
 
     constructor(opts = {}) {
         this.#statePath = opts.statePath;
-        this.#state = opts.initialState ? { ...DEFAULT_STATE, ...opts.initialState } : null;
-        this.#loaded = false;
+        if (opts.initialState) {
+            this.#state = { ...DEFAULT_STATE, ...opts.initialState };
+            this.#loaded = true;
+        } else {
+            this.#state = null;
+            this.#loaded = false;
+        }
     }
 
     #load() {
@@ -411,7 +416,7 @@ const controller = new OstackyController({ statePath });
 
 const server = new McpServer({
     name: 'ostacky-controller',
-    version: '0.5.7',
+    version: '0.5.9',
 });
 
 server.registerTool(
@@ -621,16 +626,38 @@ server.registerTool(
     {
         description:
             'Validate an edit against current file content. Returns EDITABLE, ALREADY_APPLIED, or CONFLICT. ' +
-            'Call BEFORE executing an edit tool. Only valid in EXECUTING_INLINE or EXECUTING_SUBAGENTS states.',
+            'Call BEFORE executing an edit tool. Only valid in EXECUTING_INLINE or EXECUTING_SUBAGENTS states. ' +
+            'IMPORTANT: content parameter is REQUIRED. Read the file first, then pass the full content.',
         inputSchema: z.object({
             oldString: z.string().describe('The exact string to find in content (must be unique).'),
             newString: z.string().describe('The replacement string.'),
-            content: z.string().describe('The current file content (read fresh with Read tool).'),
+            content: z
+                .string()
+                .optional()
+                .describe('The current file content. REQUIRED — read the file first with Read tool.'),
             taskId: z.string().optional().describe('Optional task ID for tracking.'),
         }),
     },
     async ({ oldString, newString, content, taskId }) => {
-        log('tool:validate_edit', { taskId, oldLen: oldString.length, newLen: newString.length });
+        log('tool:validate_edit', {
+            taskId,
+            oldLen: oldString?.length,
+            newLen: newString?.length,
+            hasContent: !!content,
+        });
+        if (typeof content !== 'string' || typeof oldString !== 'string' || typeof newString !== 'string') {
+            return {
+                content: [
+                    {
+                        type: 'text',
+                        text: JSON.stringify({
+                            outcome: 'CONFLICT',
+                            reason: 'Missing required fields: content, oldString, and newString are all required. Read the file first, then pass content to validate_edit.',
+                        }),
+                    },
+                ],
+            };
+        }
         const result = await controller.validateEdit({ oldString, newString, content, taskId });
         return { content: [{ type: 'text', text: JSON.stringify(result) }] };
     }
@@ -663,7 +690,14 @@ async function main() {
     log('ostacky-controller connected and ready');
 }
 
-main().catch((error) => {
-    console.error('Fatal error:', error);
-    process.exit(1);
-});
+const isDirectRun =
+    process.argv[1] && (process.argv[1].endsWith('/index.js') || process.argv[1].endsWith('\\index.js'));
+
+if (isDirectRun) {
+    main().catch((error) => {
+        console.error('Fatal error:', error);
+        process.exit(1);
+    });
+}
+
+export { OstackyController };

package/assets/mcp/ostacky-controller/package.json

@@ -1,6 +1,6 @@
 {
     "name": "ostacky-controller",
-    "version": "0.5.7",
+    "version": "0.5.9",
     "private": true,
     "type": "module",
     "dependencies": {

package/assets/tests/agent-contract.test.js

@@ -0,0 +1,226 @@
+import { describe, it } from 'node:test';
+import assert from 'node:assert/strict';
+import { readFileSync } from 'node:fs';
+import { fileURLToPath } from 'node:url';
+import { dirname, join } from 'node:path';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+const ASSETS_DIR = join(__dirname, '..');
+const AGENT_FILE = join(ASSETS_DIR, 'agents', 'ostacky.md');
+const CONTROLLER_FILE = join(ASSETS_DIR, 'mcp', 'ostacky-controller', 'index.js');
+
+const agentContent = readFileSync(AGENT_FILE, 'utf8');
+const controllerContent = readFileSync(CONTROLLER_FILE, 'utf8');
+
+// Tools registered in the MCP controller
+const MCP_TOOLS = [
+    'start_request', 'request_clarification', 'record_clarification',
+    'record_discovery', 'consume_route_decision', 'spec_complete',
+    'record_execution_analysis', 'consume_execution_decision',
+    'implementation_complete', 'sync_complete', 'block', 'replan',
+    'get_state', 'get_tasks', 'validate_edit', 'complete_task',
+];
+
+// CodeGraph tools referenced in agent instructions
+const CODEGRAPH_TOOLS = [
+    'codegraph_explore', 'codegraph_node', 'codegraph_search',
+    'codegraph_callers', 'codegraph_callees', 'codegraph_impact',
+    'codegraph_files', 'codegraph_status',
+];
+
+// Native OpenCode tools
+const NATIVE_TOOLS = ['question', 'Read', 'Edit', 'Bash', 'Grep', 'Glob'];
+
+const ALL_VALID_TOOLS = [...MCP_TOOLS, ...CODEGRAPH_TOOLS, ...NATIVE_TOOLS];
+
+describe('Agent contract — ostacky.md', () => {
+    describe('Core Instructions section', () => {
+        it('has "Core Instructions" heading', () => {
+            assert.ok(agentContent.includes('## Core Instructions'),
+                'Missing "## Core Instructions" section');
+        });
+
+        it('has SINGLE SOURCE OF VERDAD marker', () => {
+            assert.ok(agentContent.includes('SINGLE SOURCE OF VERDAD'),
+                'Missing "SINGLE SOURCE OF VERDAD" marker');
+        });
+    });
+
+    describe('CodeGraph section', () => {
+        it('has CodeGraph — búsqueda de código subsection', () => {
+            assert.ok(agentContent.includes('### CodeGraph — búsqueda de código'),
+                'Missing "### CodeGraph — búsqueda de código" section');
+        });
+
+        it('has tool table with codegraph_explore', () => {
+            assert.ok(agentContent.includes('codegraph_explore'),
+                'Missing codegraph_explore in tool table');
+        });
+    });
+
+    describe('Engram section', () => {
+        it('has Engram — memoria persistente subsection', () => {
+            assert.ok(agentContent.includes('### Engram — memoria persistente'),
+                'Missing "### Engram — memoria persistente" section');
+        });
+
+        it('has engram_mem_context in Discovery flow', () => {
+            assert.ok(agentContent.includes('engram_mem_context'),
+                'Missing engram_mem_context — agent should consult memory at start of request');
+        });
+
+        it('has engram_mem_search reference', () => {
+            assert.ok(agentContent.includes('engram_mem_search'),
+                'Missing engram_mem_search — agent should search before deciding');
+        });
+
+        it('has engram_mem_save reference', () => {
+            assert.ok(agentContent.includes('engram_mem_save'),
+                'Missing engram_mem_save — agent should save after significant work');
+        });
+    });
+
+    describe('HARD-STOP rules', () => {
+        it('has at least 2 HARD-STOP blocks (HTML tags or inline markers)', () => {
+            const htmlBlocks = (agentContent.match(/<HARD-STOP>/g) || []).length;
+            const inlineMarkers = (agentContent.match(/\*\*HARD-STOP\*\*/g) || []).length;
+            const total = htmlBlocks + inlineMarkers;
+            assert.ok(total >= 2,
+                `Expected at least 2 HARD-STOP blocks (HTML or inline), found ${total} (${htmlBlocks} HTML, ${inlineMarkers} inline)`);
+        });
+
+        it('has closing HARD-STOP tags matching opening tags', () => {
+            const opens = (agentContent.match(/<HARD-STOP>/g) || []).length;
+            const closes = (agentContent.match(/<\/HARD-STOP>/g) || []).length;
+            assert.equal(opens, closes,
+                `Mismatched HARD-STOP tags: ${opens} opens, ${closes} closes`);
+        });
+
+        it('has HARD-STOP in Regla de oro (global question stop)', () => {
+            const reglaDeOro = agentContent.substring(
+                agentContent.indexOf('## Regla de oro'),
+                agentContent.indexOf('## Flujo')
+            );
+            assert.ok(reglaDeOro.includes('<HARD-STOP>'),
+                'Missing HARD-STOP in Regla de Oro section — agent can continue after questions');
+        });
+    });
+
+    describe('Tool name references — no stale names', () => {
+        it('does NOT reference "ask_user" (non-existent tool)', () => {
+            const refs = agentContent.match(/ask_user/g);
+            const count = refs ? refs.length : 0;
+            assert.equal(count, 0,
+                `Found ${count} reference(s) to "ask_user" — this tool does not exist. Use "question" instead.`);
+        });
+
+        it('does NOT reference "codegraph_context" (stale name)', () => {
+            // Allow it in comments or descriptions, but not as an active tool call
+            const lines = agentContent.split('\n');
+            const offenders = lines.filter(line =>
+                line.includes('codegraph_context') &&
+                !line.startsWith('|') &&
+                !line.includes('NUNCA') &&
+                !line.includes('Prohibido')
+            );
+            assert.equal(offenders.length, 0,
+                `Found stale "codegraph_context" references (use codegraph_explore): ${offenders.join('; ')}`);
+        });
+
+        it('references valid MCP tools in flow instructions', () => {
+            // Extract tool names used in flow steps (lines with backtick-quoted tool names)
+            const toolRefs = agentContent.match(/`([a-z_]+)`/g) || [];
+            const uniqueTools = [...new Set(toolRefs.map(t => t.replace(/`/g, '')))];
+
+            const flowTools = uniqueTools.filter(t =>
+                t.includes('request') || t.includes('record') || t.includes('consume') ||
+                t.includes('spec_') || t.includes('implementation') || t.includes('sync') ||
+                t.includes('validate') || t.includes('complete') || t.includes('block') ||
+                t.includes('replan') || t.includes('start_') || t.includes('get_')
+            );
+
+            const invalidTools = flowTools.filter(t => !MCP_TOOLS.includes(t));
+            assert.equal(invalidTools.length, 0,
+                `References non-existent MCP tools: ${invalidTools.join(', ')}`);
+        });
+    });
+
+    describe('validate_edit instructions', () => {
+        it('mentions content as required parameter', () => {
+            assert.ok(
+                agentContent.includes('content') &&
+                (agentContent.includes('content es OBLIGATORIO') ||
+                 agentContent.includes('`content` es OBLIGATORIO')),
+                'Missing explicit instruction that content is required for validate_edit'
+            );
+        });
+
+        it('instructs to Read file before validate_edit', () => {
+            const executionSection = agentContent.substring(
+                agentContent.indexOf('### 4. Execution'),
+                agentContent.indexOf('### 5. Sync')
+            );
+            assert.ok(executionSection.includes('Read'),
+                'Execution section should instruct to Read file before validate_edit');
+        });
+    });
+
+    describe('Classification rules', () => {
+        it('has Nivel 0, 0+1, and 1+', () => {
+            assert.ok(agentContent.includes('Nivel 0'), 'Missing Nivel 0');
+            assert.ok(agentContent.includes('Nivel 0+1'), 'Missing Nivel 0+1');
+            assert.ok(agentContent.includes('Nivel 1+'), 'Missing Nivel 1+');
+        });
+
+        it('has defaultChoice mapping', () => {
+            assert.ok(agentContent.includes('defaultChoice'), 'Missing defaultChoice mapping');
+            assert.ok(agentContent.includes('"DIRECT"'), 'Missing DIRECT default');
+            assert.ok(agentContent.includes('"SPEC"'), 'Missing SPEC default');
+        });
+    });
+
+    describe('Guardrails', () => {
+        it('has Eficiencia de tokens section', () => {
+            assert.ok(agentContent.includes('### Eficiencia de tokens'),
+                'Missing "### Eficiencia de tokens" guardrail section');
+        });
+
+        it('has Decisiones y estado section', () => {
+            assert.ok(agentContent.includes('### Decisiones y estado'),
+                'Missing "### Decisiones y estado" guardrail section');
+        });
+    });
+});
+
+describe('Controller contract — index.js', () => {
+    it('exports OstackyController', () => {
+        assert.ok(controllerContent.includes('export { OstackyController }'),
+            'Missing OstackyController export');
+    });
+
+    it('has validate_edit tool registered', () => {
+        assert.ok(controllerContent.includes("server.registerTool(\n    'validate_edit'") ||
+                  controllerContent.includes("server.registerTool('validate_edit'"),
+            'validate_edit not registered as MCP tool');
+    });
+
+    it('validate_edit schema has content as optional (not required)', () => {
+        // Find the validate_edit schema section
+        const validateSection = controllerContent.substring(
+            controllerContent.indexOf("'validate_edit'"),
+            controllerContent.indexOf("'complete_task'")
+        );
+        assert.ok(validateSection.includes('content: z.string().optional()'),
+            'validate_edit content param should be z.string().optional() — agent may omit it');
+    });
+
+    it('validate_edit handler validates content type', () => {
+        const validateSection = controllerContent.substring(
+            controllerContent.indexOf("'validate_edit'"),
+            controllerContent.indexOf("'complete_task'")
+        );
+        assert.ok(validateSection.includes("typeof content !== 'string'"),
+            'validate_edit handler should check typeof content');
+    });
+});

package/assets/tests/controller.test.js

@@ -0,0 +1,187 @@
+import { describe, it, beforeEach } from 'node:test';
+import assert from 'node:assert/strict';
+import { OstackyController } from '../mcp/ostacky-controller/index.js';
+
+function createController(initialState = {}) {
+    return new OstackyController({
+        statePath: null,
+        initialState: {
+            state: 'INTERPRETATION_PENDING',
+            revision: 0,
+            requestId: 'test',
+            changeId: null,
+            routeDecisionId: null,
+            routeChoice: null,
+            executionDecisionId: null,
+            executionMode: null,
+            snapshots: { codegraph: null, execution: null },
+            tasks: {},
+            fileFingerprints: {},
+            error: null,
+            ...initialState,
+        },
+    });
+}
+
+describe('OstackyController', () => {
+    describe('startRequest', () => {
+        it('transitions to INTERPRETATION_PENDING', async () => {
+            const ctrl = createController({ state: 'DONE' });
+            const result = await ctrl.startRequest({ requestId: 'req-1' });
+            assert.equal(result.state, 'INTERPRETATION_PENDING');
+            assert.equal(result.requestId, 'req-1');
+        });
+
+        it('generates requestId when not provided', async () => {
+            const ctrl = createController({ state: 'DONE' });
+            const result = await ctrl.startRequest();
+            assert.ok(result.requestId.startsWith('req-'));
+        });
+    });
+
+    describe('validateEdit', () => {
+        it('returns CONFLICT when content is undefined (the bug we fixed)', async () => {
+            const ctrl = createController({ state: 'EXECUTING_INLINE' });
+            const result = await ctrl.validateEdit({
+                oldString: 'foo',
+                newString: 'bar',
+                content: undefined,
+            });
+            assert.equal(result.outcome, 'CONFLICT');
+            assert.ok(result.reason.includes('Missing required fields'));
+        });
+
+        it('returns CONFLICT when content is not a string', async () => {
+            const ctrl = createController({ state: 'EXECUTING_INLINE' });
+            const result = await ctrl.validateEdit({
+                oldString: 'foo',
+                newString: 'bar',
+                content: 123,
+            });
+            assert.equal(result.outcome, 'CONFLICT');
+            assert.ok(result.reason.includes('Missing required fields'));
+        });
+
+        it('returns EDITABLE when oldString found exactly once', async () => {
+            const ctrl = createController({ state: 'EXECUTING_INLINE' });
+            const result = await ctrl.validateEdit({
+                oldString: 'hello world',
+                newString: 'hello universe',
+                content: 'say hello world to everyone',
+            });
+            assert.equal(result.outcome, 'EDITABLE');
+        });
+
+        it('returns ALREADY_APPLIED when oldString equals newString', async () => {
+            const ctrl = createController({ state: 'EXECUTING_INLINE' });
+            const result = await ctrl.validateEdit({
+                oldString: 'hello',
+                newString: 'hello',
+                content: 'hello world',
+            });
+            assert.equal(result.outcome, 'ALREADY_APPLIED');
+        });
+
+        it('returns ALREADY_APPLIED when newString already present and oldString not found', async () => {
+            const ctrl = createController({ state: 'EXECUTING_INLINE' });
+            const result = await ctrl.validateEdit({
+                oldString: 'old text',
+                newString: 'new text',
+                content: 'already has new text here',
+            });
+            assert.equal(result.outcome, 'ALREADY_APPLIED');
+        });
+
+        it('returns CONFLICT when oldString not found and newString not present', async () => {
+            const ctrl = createController({ state: 'EXECUTING_INLINE' });
+            const result = await ctrl.validateEdit({
+                oldString: 'nonexistent',
+                newString: 'replacement',
+                content: 'some content without the old string',
+            });
+            assert.equal(result.outcome, 'CONFLICT');
+            assert.ok(result.reason.includes('not found'));
+        });
+
+        it('returns CONFLICT when oldString found multiple times', async () => {
+            const ctrl = createController({ state: 'EXECUTING_INLINE' });
+            const result = await ctrl.validateEdit({
+                oldString: 'dup',
+                newString: 'replaced',
+                content: 'dup and dup again',
+            });
+            assert.equal(result.outcome, 'CONFLICT');
+            assert.ok(result.reason.includes('2 times'));
+        });
+
+        it('returns CONFLICT when not in executing state', async () => {
+            const ctrl = createController();
+            const result = await ctrl.validateEdit({
+                oldString: 'foo',
+                newString: 'bar',
+                content: 'foo bar baz',
+            });
+            assert.equal(result.outcome, 'CONFLICT');
+            assert.ok(result.reason.includes('Cannot validate edit from state'));
+        });
+    });
+
+    describe('state transitions', () => {
+        it('full flow: start → discovery → route → execution → sync → done', async () => {
+            const ctrl = createController({ state: 'DONE' });
+            let r = await ctrl.startRequest({ requestId: 'test' });
+            assert.equal(r.state, 'INTERPRETATION_PENDING');
+
+            r = await ctrl.recordDiscovery({ level: '0', routeDecisionId: 'rd-1' });
+            assert.equal(r.state, 'ROUTE_DECISION_PENDING');
+            assert.equal(r.defaultChoice, 'DIRECT');
+
+            r = await ctrl.consumeRouteDecision({ decisionId: 'rd-1', choice: 'DIRECT' });
+            assert.equal(r.state, 'EXECUTION_ANALYSIS');
+
+            r = await ctrl.recordExecutionAnalysis({ executionDecisionId: 'ed-1' });
+            assert.equal(r.state, 'EXECUTION_DECISION_PENDING');
+
+            r = await ctrl.consumeExecutionDecision({ decisionId: 'ed-1', mode: 'INLINE' });
+            assert.equal(r.state, 'EXECUTING_INLINE');
+
+            r = await ctrl.implementationComplete();
+            assert.equal(r.state, 'SYNC');
+
+            r = await ctrl.syncComplete();
+            assert.equal(r.state, 'DONE');
+        });
+
+        it('rejects invalid transition: start from EXECUTING_INLINE', async () => {
+            const ctrl = createController({ state: 'EXECUTING_INLINE', requestId: 'original' });
+            const r = await ctrl.startRequest({ requestId: 'nope' });
+            assert.equal(r.state, 'EXECUTING_INLINE');
+            assert.equal(r.requestId, 'original');
+        });
+
+        it('level 1+ defaults to SPEC', async () => {
+            const ctrl = createController();
+            const r = await ctrl.recordDiscovery({ level: '1+', routeDecisionId: 'rd-2' });
+            assert.equal(r.defaultChoice, 'SPEC');
+        });
+    });
+
+    describe('completeTask', () => {
+        it('marks task as completed', async () => {
+            const ctrl = createController({ state: 'EXECUTING_INLINE' });
+            const r = await ctrl.completeTask({
+                taskId: 'task-1',
+                filePath: 'src/foo.ts',
+                fileHash: 'abc123',
+            });
+            assert.equal(r.status, 'COMPLETED');
+            assert.equal(r.totalCompleted, 1);
+        });
+
+        it('rejects completeTask from wrong state', async () => {
+            const ctrl = createController();
+            const r = await ctrl.completeTask({ taskId: 'task-1' });
+            assert.ok(r.error.includes('Cannot complete task from state'));
+        });
+    });
+});

package/assets/tests/validate-config.sh

@@ -236,6 +236,24 @@ else
 fi
 echo ""
 
+# ─── 14. Runtime tests (node:test) ──────────────────────────────────────────
+echo "14. Runtime tests"
+if command -v node &>/dev/null; then
+    TEST_OUTPUT=$(cd "$ASSETS_DIR" && node --test tests/controller.test.js tests/agent-contract.test.js 2>&1)
+    TEST_EXIT=$?
+    TEST_PASS=$(echo "$TEST_OUTPUT" | grep "^# pass" | awk '{print $3}')
+    TEST_FAIL=$(echo "$TEST_OUTPUT" | grep "^# fail" | awk '{print $3}')
+    if [ "$TEST_EXIT" -eq 0 ]; then
+        pass "runtime tests: ${TEST_PASS}/${TEST_PASS} passed"
+    else
+        fail "runtime tests: ${TEST_FAIL} failed out of $((TEST_PASS + TEST_FAIL))"
+        echo "$TEST_OUTPUT" | grep "not ok" | head -5
+    fi
+else
+    warn "node not found — skipping runtime tests"
+fi
+echo ""
+
 # ─── Summary ──────────────────────────────────────────────────────────────────
 echo "═══════════════════════════════════════════════════════════"
 echo "  Summary"

package/dist/cli.js

@@ -161,7 +161,7 @@ var require_picocolors = __commonJS((exports, module) => {
 // package.json
 var package_default = {
   name: "ostacky",
-  version: "0.5.7",
+  version: "0.5.9",
   description: "Instalador interactivo de agentes y comandos para OpenCode",
   type: "module",
   bin: {
@@ -879,16 +879,16 @@ import { join as join5 } from "path";
 import { existsSync as existsSync4 } from "fs";
 // manifest.json
 var manifest_default = {
-  version: "0.5.7",
+  version: "0.5.9",
   repo: "JaimeHoracio/Ostacky",
-  tag: "v0.5.7",
+  tag: "v0.5.9",
   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.7",
-      sha256: "eb3ab4cfcdedee63dd4aae05d748d34d9cfc3323bdc5f53f06848cfe3efd439f"
+      version: "0.5.9",
+      sha256: "ede9800f644d890fa81c4827592ca62ce00756a25fbdb594cb8a7b40cd7ca2fc"
     }
   ],
   commands: [
@@ -896,14 +896,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.7",
-      sha256: "f8357678ae180fc9fda03be2e4f44a18ee4d7eedf138a6f9dbe22cc63bd2bc41"
+      version: "0.5.9",
+      sha256: "598b54bed669586f832193226ebeddbda325325f8f237ba5104ab3ee58f2ac29"
     },
     {
       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.7",
+      version: "0.5.9",
       sha256: "ed9948f1910743b672e1dfad496bc96972a224a98b063232be39290d43068c50"
     }
   ],
@@ -912,8 +912,8 @@ var manifest_default = {
       name: "ostacky-controller",
       file: "assets/mcp/ostacky-controller/",
       description: "Máquina de estados persistida para Ostacky: 13 tools MCP para ciclo completo de request (start_request → discovery → route → execution → sync → done), edición segura con validación de transiciones y persistencia atómica",
-      version: "0.5.7",
-      sha256: "f457acabe9187fea9ea4a132d4a54e26021d1dc8a87469ba5815d8d3052336af"
+      version: "0.5.9",
+      sha256: "68b42dfb65938e423a7c81d9e9b179c0d82533047d5c0205cf459f5aa2fc6fa2"
     }
   ],
   skills: [
@@ -921,99 +921,99 @@ var manifest_default = {
       name: "thinking",
       file: "assets/skills/thinking/SKILL.md",
       description: "Skill unificado de pensamiento con dos modos: creative-design (producción de diseño → transición a writing-plans o openspec-propose) y open-exploration (exploración libre)",
-      version: "0.5.7",
-      sha256: "3801f3c964e3c4a64f54747d7a8b2e5c2c956c907ebc938b45d05c3b5ee6dfa4"
+      version: "0.5.9",
+      sha256: "6a1fdac86357f89e132013e56cb84c1920880c1c035a8dcf53f69e01824724bb"
     },
     {
       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.7",
-      sha256: "0d791c514c1bde60ac3a2b02e75fdebf36230e46c30d76d5f1c32aefad82463f"
+      version: "0.5.9",
+      sha256: "048925aec576dff014eed0b44e2a553c635ae405f5a245773f480be9b84d18a7"
     },
     {
       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.7",
-      sha256: "d8dde75ee7aac1195060444e93484319d5b1ddc7125576715978b5352354c9a8"
+      version: "0.5.9",
+      sha256: "13f6e43522567505da90793934d414d290d56f4ea587ac696f66416af9e7633f"
     },
     {
       name: "tdd",
       file: "assets/skills/tdd/SKILL.md",
       description: "Skill de test-driven development (Superpowers)",
-      version: "0.5.7",
-      sha256: "7dee67b4af6bdccc7a914ca34533184d64592d0f5b23aeae631538168db14994"
+      version: "0.5.9",
+      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.7",
-      sha256: "6b5344c5749dd9b61bed0fded1516c212cdfeec59779f9c32a5a84fa94315431"
+      version: "0.5.9",
+      sha256: "cdacae2e5b86f6a642ad0316172cee7dc8d19e909ef4f9ab7fcf7e84a248dda3"
     },
     {
       name: "dispatching-parallel-agents",
       file: "assets/skills/dispatching-parallel-agents/SKILL.md",
       description: "Skill de dispatch paralelo de agentes (Superpowers)",
-      version: "0.5.7",
-      sha256: "76806091c7f923ba2596546b19cccd98a08e57a68745df77c3a7b998fe838e2b"
+      version: "0.5.9",
+      sha256: "281edf0c38f358497c7e2066fa8217a2ba3e2a39b4205c4af0c41d328fc035a1"
     },
     {
       name: "review",
       file: "assets/skills/review/SKILL.md",
       description: "Skill de revisión de código (Superpowers)",
-      version: "0.5.7",
-      sha256: "fc68671ac4592ceeae6bf0d3ac33abaf78cd34e94e67c07ad0329843d6003468"
+      version: "0.5.9",
+      sha256: "14831d9ef3746ef6e2e6047c868fe7aa296d01f0644227ce776fe99436357b1a"
     },
     {
       name: "receiving-code-review",
       file: "assets/skills/receiving-code-review/SKILL.md",
       description: "Skill de recibir y procesar feedback de code review",
-      version: "0.5.7",
-      sha256: "b6ace0ce93c5eade7b3f17d6744aa2fca6f492632b80e5dd754f93ba2c331550"
+      version: "0.5.9",
+      sha256: "0a5780e8a41539d15428114b7c46f36670acc16aba0db3348d36be1175433872"
     },
     {
       name: "openspec-propose",
       file: "assets/skills/openspec-propose/SKILL.md",
       description: "Skill de generación de proposal (OpenSpec)",
-      version: "0.5.7",
-      sha256: "674b84346d954f61fbbe53d1efb7e6cd9a761b49ea6a9950aa9dfbfe6366a165"
+      version: "0.5.9",
+      sha256: "3306b21ba9cb8c1611e8ba335126982f3767f7d317b5ed506c57a14c5f88b263"
     },
     {
       name: "openspec-apply-change",
       file: "assets/skills/openspec-apply-change/SKILL.md",
       description: "Skill de aplicación de change (OpenSpec)",
-      version: "0.5.7",
-      sha256: "9e4fe9fe780b580bfba9f842c3c3603e77150355f8bf2b68a85e8ce41e2c941a"
+      version: "0.5.9",
+      sha256: "31c1313c3de4616a07efd89306c4c058175d018f42d4381cc7da7030fd03ba30"
     },
     {
       name: "openspec-archive-change",
       file: "assets/skills/openspec-archive-change/SKILL.md",
       description: "Skill de archivo de change (OpenSpec)",
-      version: "0.5.7",
-      sha256: "4c611a20654841c585fe1c6f573e6e69c8b95c4eecdfe5552923be44dd3da0c4"
+      version: "0.5.9",
+      sha256: "5bf65d1848457bd57be1d21707e55fe6746cdee33bce2f32f12d87661b6aa3a4"
     },
     {
       name: "using-git-worktrees",
       file: "assets/skills/using-git-worktrees/SKILL.md",
       description: "Skill de uso de git worktrees para aislamiento de trabajo",
-      version: "0.5.7",
-      sha256: "ff16257abe3836ca0f1003f130236a79acf4c8574a2e9fed092a5101d52ae860"
+      version: "0.5.9",
+      sha256: "33433c24f753d566cde16c0931f70d746b6b0cb4d1f3daaa619e928b009e4fc9"
     },
     {
       name: "using-superpowers",
       file: "assets/skills/using-superpowers/SKILL.md",
       description: "Skill de orquestación de Superpowers skills",
-      version: "0.5.7",
-      sha256: "bc8bd5445b623c1158f83c518155518b6556c58dd194f73f3bab444a983410ab"
+      version: "0.5.9",
+      sha256: "fba542003aa788b5edb4ae2d1135773a1dde8111d1dd60378e2f61b6695d46fc"
     },
     {
       name: "writing-skills",
       file: "assets/skills/writing-skills/SKILL.md",
       description: "Skill de creación y edición de skills",
-      version: "0.5.7",
-      sha256: "915369d6a74570c1605ad49d6e6551a946ca232b12400fbb45797149f16a7f2c"
+      version: "0.5.9",
+      sha256: "7f74ffe049283803640d78f8c9ac1ea5f5e626dedb885ecff29a87c5b5d41598"
     }
   ]
 };
@@ -1204,7 +1204,7 @@ function clearLockfile(opencodeRoot) {
   const lockfile = readLockfile(opencodeRoot);
   if (!lockfile) {
     writeLockfile(opencodeRoot, {
-      version: "0.5.7",
+      version: "0.5.9",
       lockedAt: new Date().toISOString(),
       repo: "",
       tag: "",

package/dist/mcp/ostacky-controller/index.js

@@ -19539,8 +19539,13 @@ class OstackyController {
   #loaded;
   constructor(opts = {}) {
     this.#statePath = opts.statePath;
-    this.#state = opts.initialState ? { ...DEFAULT_STATE, ...opts.initialState } : null;
-    this.#loaded = false;
+    if (opts.initialState) {
+      this.#state = { ...DEFAULT_STATE, ...opts.initialState };
+      this.#loaded = true;
+    } else {
+      this.#state = null;
+      this.#loaded = false;
+    }
   }
   #load() {
     if (this.#loaded)
@@ -19884,7 +19889,7 @@ var statePath = process.env.OSTACKY_STATE_PATH || ".opencode/ostacky-state.json"
 var controller = new OstackyController({ statePath });
 var server = new McpServer({
   name: "ostacky-controller",
-  version: "0.5.7"
+  version: "0.5.9"
 });
 server.registerTool("start_request", {
   description: "Start or resume a new request in the state machine. Call this first.",
@@ -20019,15 +20024,33 @@ server.registerTool("get_tasks", {
   return { content: [{ type: "text", text: JSON.stringify(result) }] };
 });
 server.registerTool("validate_edit", {
-  description: "Validate an edit against current file content. Returns EDITABLE, ALREADY_APPLIED, or CONFLICT. " + "Call BEFORE executing an edit tool. Only valid in EXECUTING_INLINE or EXECUTING_SUBAGENTS states.",
+  description: "Validate an edit against current file content. Returns EDITABLE, ALREADY_APPLIED, or CONFLICT. " + "Call BEFORE executing an edit tool. Only valid in EXECUTING_INLINE or EXECUTING_SUBAGENTS states. " + "IMPORTANT: content parameter is REQUIRED. Read the file first, then pass the full content.",
   inputSchema: object({
     oldString: string2().describe("The exact string to find in content (must be unique)."),
     newString: string2().describe("The replacement string."),
-    content: string2().describe("The current file content (read fresh with Read tool)."),
+    content: string2().optional().describe("The current file content. REQUIRED — read the file first with Read tool."),
     taskId: string2().optional().describe("Optional task ID for tracking.")
   })
 }, async ({ oldString, newString, content, taskId }) => {
-  log("tool:validate_edit", { taskId, oldLen: oldString.length, newLen: newString.length });
+  log("tool:validate_edit", {
+    taskId,
+    oldLen: oldString?.length,
+    newLen: newString?.length,
+    hasContent: !!content
+  });
+  if (typeof content !== "string" || typeof oldString !== "string" || typeof newString !== "string") {
+    return {
+      content: [
+        {
+          type: "text",
+          text: JSON.stringify({
+            outcome: "CONFLICT",
+            reason: "Missing required fields: content, oldString, and newString are all required. Read the file first, then pass content to validate_edit."
+          })
+        }
+      ]
+    };
+  }
   const result = await controller.validateEdit({ oldString, newString, content, taskId });
   return { content: [{ type: "text", text: JSON.stringify(result) }] };
 });
@@ -20050,7 +20073,13 @@ async function main() {
   await server.connect(transport);
   log("ostacky-controller connected and ready");
 }
-main().catch((error2) => {
-  console.error("Fatal error:", error2);
-  process.exit(1);
-});
+var isDirectRun = process.argv[1] && (process.argv[1].endsWith("/index.js") || process.argv[1].endsWith("\\index.js"));
+if (isDirectRun) {
+  main().catch((error2) => {
+    console.error("Fatal error:", error2);
+    process.exit(1);
+  });
+}
+export {
+  OstackyController
+};

package/manifest.json

@@ -1,14 +1,14 @@
 {
-    "version": "0.5.7",
+    "version": "0.5.9",
     "repo": "JaimeHoracio/Ostacky",
-    "tag": "v0.5.7",
+    "tag": "v0.5.9",
     "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.7",
-            "sha256": "eb3ab4cfcdedee63dd4aae05d748d34d9cfc3323bdc5f53f06848cfe3efd439f"
+            "version": "0.5.9",
+            "sha256": "ede9800f644d890fa81c4827592ca62ce00756a25fbdb594cb8a7b40cd7ca2fc"
         }
     ],
     "commands": [
@@ -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.7",
-            "sha256": "f8357678ae180fc9fda03be2e4f44a18ee4d7eedf138a6f9dbe22cc63bd2bc41"
+            "version": "0.5.9",
+            "sha256": "598b54bed669586f832193226ebeddbda325325f8f237ba5104ab3ee58f2ac29"
         },
         {
             "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.7",
+            "version": "0.5.9",
             "sha256": "ed9948f1910743b672e1dfad496bc96972a224a98b063232be39290d43068c50"
         }
     ],
@@ -32,8 +32,8 @@
             "name": "ostacky-controller",
             "file": "assets/mcp/ostacky-controller/",
             "description": "Máquina de estados persistida para Ostacky: 13 tools MCP para ciclo completo de request (start_request → discovery → route → execution → sync → done), edición segura con validación de transiciones y persistencia atómica",
-            "version": "0.5.7",
-            "sha256": "f457acabe9187fea9ea4a132d4a54e26021d1dc8a87469ba5815d8d3052336af"
+            "version": "0.5.9",
+            "sha256": "68b42dfb65938e423a7c81d9e9b179c0d82533047d5c0205cf459f5aa2fc6fa2"
         }
     ],
     "skills": [
@@ -41,99 +41,99 @@
             "name": "thinking",
             "file": "assets/skills/thinking/SKILL.md",
             "description": "Skill unificado de pensamiento con dos modos: creative-design (producción de diseño → transición a writing-plans o openspec-propose) y open-exploration (exploración libre)",
-            "version": "0.5.7",
-            "sha256": "3801f3c964e3c4a64f54747d7a8b2e5c2c956c907ebc938b45d05c3b5ee6dfa4"
+            "version": "0.5.9",
+            "sha256": "6a1fdac86357f89e132013e56cb84c1920880c1c035a8dcf53f69e01824724bb"
         },
         {
             "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.7",
-            "sha256": "0d791c514c1bde60ac3a2b02e75fdebf36230e46c30d76d5f1c32aefad82463f"
+            "version": "0.5.9",
+            "sha256": "048925aec576dff014eed0b44e2a553c635ae405f5a245773f480be9b84d18a7"
         },
         {
             "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.7",
-            "sha256": "d8dde75ee7aac1195060444e93484319d5b1ddc7125576715978b5352354c9a8"
+            "version": "0.5.9",
+            "sha256": "13f6e43522567505da90793934d414d290d56f4ea587ac696f66416af9e7633f"
         },
         {
             "name": "tdd",
             "file": "assets/skills/tdd/SKILL.md",
             "description": "Skill de test-driven development (Superpowers)",
-            "version": "0.5.7",
-            "sha256": "7dee67b4af6bdccc7a914ca34533184d64592d0f5b23aeae631538168db14994"
+            "version": "0.5.9",
+            "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.7",
-            "sha256": "6b5344c5749dd9b61bed0fded1516c212cdfeec59779f9c32a5a84fa94315431"
+            "version": "0.5.9",
+            "sha256": "cdacae2e5b86f6a642ad0316172cee7dc8d19e909ef4f9ab7fcf7e84a248dda3"
         },
         {
             "name": "dispatching-parallel-agents",
             "file": "assets/skills/dispatching-parallel-agents/SKILL.md",
             "description": "Skill de dispatch paralelo de agentes (Superpowers)",
-            "version": "0.5.7",
-            "sha256": "76806091c7f923ba2596546b19cccd98a08e57a68745df77c3a7b998fe838e2b"
+            "version": "0.5.9",
+            "sha256": "281edf0c38f358497c7e2066fa8217a2ba3e2a39b4205c4af0c41d328fc035a1"
         },
         {
             "name": "review",
             "file": "assets/skills/review/SKILL.md",
             "description": "Skill de revisión de código (Superpowers)",
-            "version": "0.5.7",
-            "sha256": "fc68671ac4592ceeae6bf0d3ac33abaf78cd34e94e67c07ad0329843d6003468"
+            "version": "0.5.9",
+            "sha256": "14831d9ef3746ef6e2e6047c868fe7aa296d01f0644227ce776fe99436357b1a"
         },
         {
             "name": "receiving-code-review",
             "file": "assets/skills/receiving-code-review/SKILL.md",
             "description": "Skill de recibir y procesar feedback de code review",
-            "version": "0.5.7",
-            "sha256": "b6ace0ce93c5eade7b3f17d6744aa2fca6f492632b80e5dd754f93ba2c331550"
+            "version": "0.5.9",
+            "sha256": "0a5780e8a41539d15428114b7c46f36670acc16aba0db3348d36be1175433872"
         },
         {
             "name": "openspec-propose",
             "file": "assets/skills/openspec-propose/SKILL.md",
             "description": "Skill de generación de proposal (OpenSpec)",
-            "version": "0.5.7",
-            "sha256": "674b84346d954f61fbbe53d1efb7e6cd9a761b49ea6a9950aa9dfbfe6366a165"
+            "version": "0.5.9",
+            "sha256": "3306b21ba9cb8c1611e8ba335126982f3767f7d317b5ed506c57a14c5f88b263"
         },
         {
             "name": "openspec-apply-change",
             "file": "assets/skills/openspec-apply-change/SKILL.md",
             "description": "Skill de aplicación de change (OpenSpec)",
-            "version": "0.5.7",
-            "sha256": "9e4fe9fe780b580bfba9f842c3c3603e77150355f8bf2b68a85e8ce41e2c941a"
+            "version": "0.5.9",
+            "sha256": "31c1313c3de4616a07efd89306c4c058175d018f42d4381cc7da7030fd03ba30"
         },
         {
             "name": "openspec-archive-change",
             "file": "assets/skills/openspec-archive-change/SKILL.md",
             "description": "Skill de archivo de change (OpenSpec)",
-            "version": "0.5.7",
-            "sha256": "4c611a20654841c585fe1c6f573e6e69c8b95c4eecdfe5552923be44dd3da0c4"
+            "version": "0.5.9",
+            "sha256": "5bf65d1848457bd57be1d21707e55fe6746cdee33bce2f32f12d87661b6aa3a4"
         },
         {
             "name": "using-git-worktrees",
             "file": "assets/skills/using-git-worktrees/SKILL.md",
             "description": "Skill de uso de git worktrees para aislamiento de trabajo",
-            "version": "0.5.7",
-            "sha256": "ff16257abe3836ca0f1003f130236a79acf4c8574a2e9fed092a5101d52ae860"
+            "version": "0.5.9",
+            "sha256": "33433c24f753d566cde16c0931f70d746b6b0cb4d1f3daaa619e928b009e4fc9"
         },
         {
             "name": "using-superpowers",
             "file": "assets/skills/using-superpowers/SKILL.md",
             "description": "Skill de orquestación de Superpowers skills",
-            "version": "0.5.7",
-            "sha256": "bc8bd5445b623c1158f83c518155518b6556c58dd194f73f3bab444a983410ab"
+            "version": "0.5.9",
+            "sha256": "fba542003aa788b5edb4ae2d1135773a1dde8111d1dd60378e2f61b6695d46fc"
         },
         {
             "name": "writing-skills",
             "file": "assets/skills/writing-skills/SKILL.md",
             "description": "Skill de creación y edición de skills",
-            "version": "0.5.7",
-            "sha256": "915369d6a74570c1605ad49d6e6551a946ca232b12400fbb45797149f16a7f2c"
+            "version": "0.5.9",
+            "sha256": "7f74ffe049283803640d78f8c9ac1ea5f5e626dedb885ecff29a87c5b5d41598"
         }
     ]
 }

package/package.json

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