agent-rev 0.2.3 → 0.2.5

package/dist/commands/repl.js

@@ -224,19 +224,18 @@ function detectModels(cliName) {
             case 'agent-rev':
             case 'agent-explorer': {
                 try {
-                    const out = execSync(`${cliName} --list-models 2>/dev/null`, { encoding: 'utf-8' });
-                    const models = out.trim().split('\n').filter(Boolean);
-                    if (models.length)
-                        return models;
+                    const cfg = JSON.parse(fsSync.readFileSync(path.join(os.homedir(), `.${cliName}`, 'cli-config.json'), 'utf-8'));
+                    if (cfg.coordinatorModel)
+                        return [cfg.coordinatorModel];
                 }
                 catch { }
-                // fallback: list from opencode since that's typically the underlying CLI
                 try {
-                    const out = execSync('opencode models 2>/dev/null', { encoding: 'utf-8' });
-                    return out.trim().split('\n').filter(Boolean);
+                    const cfg = JSON.parse(fsSync.readFileSync(path.join(os.homedir(), '.agent-mp', 'cli-config.json'), 'utf-8'));
+                    if (cfg.coordinatorModel)
+                        return [cfg.coordinatorModel];
                 }
                 catch { }
-                return ['configured'];
+                return ['(configure with /model after login)'];
             }
             default: return ['default'];
         }
@@ -938,6 +937,24 @@ export async function runRepl(resumeSession) {
         roles: welcomeRoles,
     }) + '\n');
     process.stdout.write(renderHelpHint() + '\n\n');
+    // Print coordinator header BEFORE fi.setup() so no blank lines appear between them
+    try {
+        const dir = process.cwd();
+        const config = await loadProjectConfig(dir);
+        if (config.roles?.orchestrator?.cli) {
+            process.stdout.write('\n');
+            process.stdout.write(chalk.bold.cyan('═'.repeat(46)) + '\n');
+            process.stdout.write(chalk.bold.cyan(`  COORDINADOR — ${config.project}`) + '\n');
+            process.stdout.write(chalk.bold.cyan('═'.repeat(46)) + '\n');
+            process.stdout.write(chalk.blue(`  → Rol: COORDINADOR (solo orquesta, nunca ejecuta)`) + '\n');
+            process.stdout.write(chalk.dim('  ' + '─'.repeat(42)) + '\n');
+            process.stdout.write(chalk.bold(`    ── FASE 0 — Clarificacion ──`) + '\n');
+            process.stdout.write(chalk.dim(`  → El coordinador va a conversar con vos para entender la tarea.`) + '\n');
+            process.stdout.write(chalk.dim(`  → Cuando el coordinador tenga claro el objetivo, te va a pedir confirmación para lanzar el plan.`) + '\n');
+            process.stdout.write(chalk.dim('  ' + '─'.repeat(42)) + '\n');
+        }
+    }
+    catch { }
     // Fixed-bottom input — owns the last 3 terminal rows permanently
     const fi = new FixedInput();
     fi.setup();
@@ -970,24 +987,6 @@ export async function runRepl(resumeSession) {
             resume();
         }
     };
-    // Show coordinator header before the REPL loop (once, at startup)
-    try {
-        const dir = process.cwd();
-        const config = await loadProjectConfig(dir);
-        if (config.roles?.orchestrator?.cli) {
-            fi.println('');
-            fi.println(chalk.bold.cyan('═'.repeat(46)));
-            fi.println(chalk.bold.cyan(`  COORDINADOR — ${config.project}`));
-            fi.println(chalk.bold.cyan('═'.repeat(46)));
-            fi.println(chalk.blue(`  → Rol: COORDINADOR (solo orquesta, nunca ejecuta)`));
-            fi.println(chalk.dim('  ' + '─'.repeat(42)));
-            fi.println(chalk.bold(`    ── FASE 0 — Clarificacion ──`));
-            fi.println(chalk.dim(`  → El coordinador va a conversar con vos para entender la tarea.`));
-            fi.println(chalk.dim(`  → Cuando el coordinador tenga claro el objetivo, te va a pedir confirmación para lanzar el plan.`));
-            fi.println(chalk.dim('  ' + '─'.repeat(42)));
-        }
-    }
-    catch { }
     // Command handler — used by the main loop AND by the engine during mid-conversation
     const handleCmd = async (trimmed) => {
         const parts = trimmed.slice(1).split(/\s+/);

package/dist/commands/setup.js

@@ -1,6 +1,8 @@
 import * as readline from 'readline';
 import * as path from 'path';
+import * as os from 'os';
 import * as fs from 'fs/promises';
+import * as fsSync from 'fs';
 import chalk from 'chalk';
 import { execSync } from 'child_process';
 import { CLI_REGISTRY } from '../types.js';
@@ -61,19 +63,25 @@ function detectModels(cliName) {
             case 'agent-impl':
             case 'agent-rev':
             case 'agent-explorer': {
+                // Try to read the saved coordinator model from the agent's own config
                 try {
-                    const out = execSync(`${cliName} --list-models 2>/dev/null`, { encoding: 'utf-8' });
-                    const models = out.trim().split('\n').filter(Boolean);
-                    if (models.length)
-                        return models;
+                    const agentHome = path.join(os.homedir(), `.${cliName}`);
+                    const cfgRaw = fsSync.readFileSync(path.join(agentHome, 'cli-config.json'), 'utf-8');
+                    const cfg = JSON.parse(cfgRaw);
+                    if (cfg.coordinatorModel)
+                        return [cfg.coordinatorModel];
                 }
                 catch { }
+                // Try to read from shared agent-mp config as fallback
                 try {
-                    const out = execSync('opencode models 2>/dev/null', { encoding: 'utf-8' });
-                    return out.trim().split('\n').filter(Boolean);
+                    const agentHome = path.join(os.homedir(), '.agent-mp');
+                    const cfgRaw = fsSync.readFileSync(path.join(agentHome, 'cli-config.json'), 'utf-8');
+                    const cfg = JSON.parse(cfgRaw);
+                    if (cfg.coordinatorModel)
+                        return [cfg.coordinatorModel];
                 }
                 catch { }
-                return ['configured'];
+                return ['(configure with /model after login)'];
             }
             default:
                 return ['default'];

package/dist/index.js

@@ -68,8 +68,26 @@ const ROLE_BINS = {
 };
 const nativeRole = ROLE_BINS[PKG_NAME];
 if (nativeRole) {
-    // Parse --model flag if provided
     const args = process.argv.slice(2);
+    // --login: OAuth login for this role's account
+    if (args.includes('--login') || args.includes('login')) {
+        const { qwenLogin } = await import('./utils/qwen-auth.js');
+        const ok = await qwenLogin();
+        process.exit(ok ? 0 : 1);
+    }
+    // --status: show auth status
+    if (args.includes('--status') || args.includes('status')) {
+        const { qwenAuthStatus } = await import('./utils/qwen-auth.js');
+        const status = await qwenAuthStatus();
+        if (status.authenticated) {
+            console.log(chalk.green(`  ✓ ${PKG_NAME}: authenticated`));
+        }
+        else {
+            console.log(chalk.yellow(`  ✗ ${PKG_NAME}: not authenticated — run: ${PKG_NAME} --login`));
+        }
+        process.exit(0);
+    }
+    // Parse --model flag if provided
     const modelIdx = args.findIndex(a => a === '--model' || a === '-m');
     let model;
     if (modelIdx !== -1 && args[modelIdx + 1]) {
@@ -77,8 +95,11 @@ if (nativeRole) {
     }
     const taskArg = args.filter((a, i) => !a.startsWith('-') && i !== modelIdx + 1).join(' ').trim();
     if (!taskArg) {
-        console.error(chalk.red(`  Usage: ${PKG_NAME} [--model <model>] "<task description or task-id>"`));
-        process.exit(1);
+        console.log(chalk.bold.cyan(`\n  ${PKG_NAME} — ${nativeRole} agent\n`));
+        console.log(chalk.dim(`  Usage:   ${PKG_NAME} [--model <model>] "<task>"`));
+        console.log(chalk.dim(`  Login:   ${PKG_NAME} --login`));
+        console.log(chalk.dim(`  Status:  ${PKG_NAME} --status\n`));
+        process.exit(0);
     }
     await runRole(nativeRole, taskArg, model);
     process.exit(0);

package/package.json

@@ -1,21 +1,17 @@
 {
   "name": "agent-rev",
-  "version": "0.2.3",
-  "description": "Reviewer agent — validates and reviews code written by the implementor",
+  "version": "0.2.5",
+  "description": "",
   "type": "module",
   "main": "./dist/index.js",
-  "files": [
-    "dist/"
-  ],
-  "bin": {
-    "agent-rev": "dist/index.js"
-  },
+  "files": ["dist/"],
+  "bin": { "agent-rev": "dist/index.js" },
   "scripts": {
     "build": "tsc && echo '#!/usr/bin/env node' | cat - dist/index.js > dist/index.tmp && mv dist/index.tmp dist/index.js && chmod +x dist/index.js",
     "dev": "tsx src/index.ts",
     "prepublishOnly": "npm run build"
   },
-  "keywords": ["ai", "agent", "reviewer", "multi-agent", "cli", "coding"],
+  "keywords": ["ai","agent","multi-agent","cli","coding"],
   "license": "MIT",
   "dependencies": {
     "@anthropic-ai/sdk": "^0.39.0",
@@ -31,7 +27,5 @@
     "tsx": "^4.19.3",
     "typescript": "^5.7.3"
   },
-  "engines": {
-    "node": ">=18.0.0"
-  }
+  "engines": { "node": ">=18.0.0" }
 }