agent-rev 0.5.32 → 0.5.34

package/dist/commands/repl.js

@@ -855,6 +855,9 @@ function cmdHelp(fi) {
         { key: '/run rev <id>', value: 'Run only reviewer' },
         { key: '/run explorer [task]', value: 'Run only explorer' },
         { key: '/explorer [task]', value: 'Run explorer (shortcut)' },
+        { key: '/orch <feature-id>', value: 'Lanza SOLO orchestrator — genera plan.json' },
+        { key: '/impl <task-id>', value: 'Lanza SOLO implementor — ejecuta plan existente' },
+        { key: '/rev <task-id>', value: 'Lanza SOLO reviewer — valida implementación' },
         { key: '/model', value: 'Cambiar modelo del coordinador' },
         { key: '/provider', value: 'Cambiar proveedor activo (Gemini / Qwen)' },
         { key: '/login', value: 'Configurar API key (Qwen o Gemini)' },
@@ -1196,6 +1199,59 @@ export async function runRepl(resumeSession) {
                 });
                 break;
             }
+            // Comandos cortos para lanzar roles directamente
+            case 'orch': {
+                // /orch <feature-id> — lanza solo orchestrator sin preguntas
+                const featureId = args.join(' ').trim();
+                if (!featureId) {
+                    fi.println(chalk.red('  Uso: /orch <feature-id>'));
+                    break;
+                }
+                await withEngine(async (engine) => {
+                    const result = await engine.runOrchestrator(`.agent/docs/${featureId}`, undefined, featureId);
+                    fi.println(chalk.green(`  ✓ Task ID: ${result.taskId}`));
+                    fi.println(chalk.green(`  ✓ Plan: .agent/tasks/${result.taskId}/plan.json`));
+                });
+                break;
+            }
+            case 'impl': {
+                // /impl <task-id> — lanza solo implementor con plan existente
+                const taskId = args.join(' ').trim();
+                if (!taskId) {
+                    fi.println(chalk.red('  Uso: /impl <task-id>'));
+                    break;
+                }
+                await withEngine(async (engine) => {
+                    const taskDir = path.join(engine['projectDir'], '.agent', 'tasks', taskId);
+                    if (!await fileExists(taskDir)) {
+                        fi.println(chalk.red(`  ✗ Task "${taskId}" no existe`));
+                        return;
+                    }
+                    const plan = await readJson(path.join(taskDir, 'plan.json'));
+                    await engine.runImplementor(taskId, plan);
+                    fi.println(chalk.green(`  ✓ Implementación completada`));
+                });
+                break;
+            }
+            case 'rev': {
+                // /rev <task-id> — lanza solo reviewer
+                const taskId = args.join(' ').trim();
+                if (!taskId) {
+                    fi.println(chalk.red('  Uso: /rev <task-id>'));
+                    break;
+                }
+                await withEngine(async (engine) => {
+                    const taskDir = path.join(engine['projectDir'], '.agent', 'tasks', taskId);
+                    if (!await fileExists(taskDir)) {
+                        fi.println(chalk.red(`  ✗ Task "${taskId}" no existe`));
+                        return;
+                    }
+                    const plan = await readJson(path.join(taskDir, 'plan.json'));
+                    const progress = await readJson(path.join(taskDir, 'progress.json'));
+                    await engine.runReviewer(taskId, plan, progress);
+                });
+                break;
+            }
             case 'models':
             case 'model':
                 await withRl((rl) => cmdModels(args[0], fi, rl));

package/dist/core/engine.js

@@ -1043,6 +1043,31 @@ REGLA: Al terminar, reporta todo lo que encontraste de forma clara y estructurad
         const taskId = await this.generateTaskId(task, featureId);
         const taskDir = path.join(this.projectDir, '.agent', 'tasks', taskId);
         log.phase(1, 'Planificacion', this.config.roles.orchestrator.cli, this.config.roles.orchestrator.model);
+        // VERIFICAR SI YA EXISTE UN PLAN PARA ESTA TAREA
+        const existingPlanPath = path.join(taskDir, 'plan.json');
+        if (await fileExists(existingPlanPath)) {
+            const existingPlan = await readJson(existingPlanPath);
+            console.log(chalk.yellow('\n  ⚠️  Ya existe un plan para esta tarea:'));
+            console.log(chalk.dim(`  Descripción: ${existingPlan.description?.slice(0, 100) || 'N/A'}`));
+            console.log(chalk.dim(`  Steps: ${existingPlan.steps?.length || 0}`));
+            const action = await ask('\n  ¿Qué querés hacer? (v=ver plan completo / r=regenerar / m=mantener y continuar): ', this.rl);
+            if (action.toLowerCase() === 'v') {
+                console.log('');
+                console.log(chalk.cyan('  === plan.json ==='));
+                console.log(chalk.white(JSON.stringify(existingPlan, null, 2).slice(0, 2000)));
+                console.log(chalk.cyan('  ================='));
+                const retry = await ask('  ¿Regenerar igual? (y/n): ', this.rl);
+                if (retry.toLowerCase() !== 'y') {
+                    log.info('Usando plan existente.');
+                    return { taskId, plan: existingPlan };
+                }
+            }
+            else if (action.toLowerCase() === 'm') {
+                log.info('Usando plan existente.');
+                return { taskId, plan: existingPlan };
+            }
+            // Si es 'r', continúa generando nuevo plan
+        }
         const context = await this.buildOrchestratorContext();
         let specContent = '';
         if (specPath && await fileExists(specPath)) {
@@ -1060,10 +1085,15 @@ ${context}
 INSTRUCCIONES CRÍTICAS:
 1. Si hay SPEC.MD, ESE ES EL DOCUMENTO PRIMARIO — define completamente la tarea. Los archivos raw son solo contexto histórico, NO los uses para definir la implementación.
 2. Ignorá cualquier información que NO tenga relación directa con la tarea del spec.md (ej: listas de modelos, ejemplos genéricos, etc.).
-3. Generá steps CONCRETOS y DIRECTOS para implementar lo que dice el spec.md — sin sobre-analizar.
-4. Si el spec.md dice "script que muestra la hora", los steps son: (1) crear el script, (2) hacerlo ejecutable. No más.
-5. Definí acceptance_criteria medibles leyendo archivos.
-6. Responde SOLO con este JSON exacto, sin texto adicional:
+3. **GENERÁ STEPS DE IMPLEMENTACIÓN CONCRETOS** — NO steps sobre "leer", "analizar", "identificar". El implementor ya sabe leer.
+   - ❌ MAL: "Leer spec.md para identificar requerimientos"
+   - ❌ MAL: "Analizar archivos raw como contexto"
+   - ✅ BIEN: "Crear script clock.sh con formato HH:MM:SS, sleep 1, trap SIGINT"
+   - ✅ BIEN: "Hacer ejecutable el script con chmod +x"
+4. Cada step debe tener una descripción de **ACCIÓN** (crear, modificar, instalar, configurar) y los archivos específicos que se van a crear/modificar.
+5. Si el spec.md dice "script que muestra la hora", los steps son: (1) crear el script, (2) hacerlo ejecutable. No más.
+6. Definí acceptance_criteria medibles leyendo archivos.
+7. Responde SOLO con este JSON exacto, sin texto adicional:
 
 {"plan":{"task_id":"${taskId}","description":"${task}","steps":[{"num":1,"description":"descripcion concreta del paso","files":["archivo/a/crear.ts"],"status":"pending"}],"acceptance_criteria":["criterio verificable"],"deliberation":{"needed":false,"question":""}}}`;
         const res = await this.runWithFallback('orchestrator', prompt, 'Planificacion');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-rev",
-  "version": "0.5.32",
+  "version": "0.5.34",
   "description": "Deterministic multi-agent CLI orchestrator — plan, code, review",
   "type": "module",
   "main": "./dist/index.js",