vigthoria-cli 1.6.20 → 1.6.22

package/dist/utils/bridge-client.js

@@ -0,0 +1,278 @@
+"use strict";
+/**
+ * Vigthoria CLI → DevTools Bridge Telemetry Client
+ *
+ * Connects the local CLI to the remote bridge server in "commando" mode,
+ * streaming real-time activity (commands, tool calls, model responses,
+ * file edits, errors) and receiving admin-issued commands.
+ *
+ * Design principles:
+ *  - Fire-and-forget: never blocks the CLI main flow
+ *  - Auto-reconnects with exponential back-off
+ *  - Opt-in via --bridge <url> flag
+ *  - Sensitive data (API keys, tokens) is never transmitted
+ */
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.BridgeClient = void 0;
+exports.getBridgeClient = getBridgeClient;
+const ws_1 = __importDefault(require("ws"));
+const os = __importStar(require("os"));
+// ── Singleton accessor ───────────────────────────────────────────────
+let _instance = null;
+/** Get the active bridge client (may be null if --bridge was not used). */
+function getBridgeClient() {
+    return _instance;
+}
+// ── BridgeClient ─────────────────────────────────────────────────────
+class BridgeClient {
+    ws = null;
+    url;
+    apiKey;
+    machineLabel;
+    clientId;
+    connected = false;
+    reconnectTimer = null;
+    heartbeatTimer = null;
+    queue = []; // buffered events while disconnected
+    maxQueueSize = 500;
+    reconnectDelay = 2000; // ms, doubles on failure up to 30 s
+    destroyed = false;
+    onAdminCommand;
+    constructor(opts) {
+        this.url = opts.bridgeUrl.replace(/\/$/, '');
+        if (!this.url.includes('/ws')) {
+            this.url = this.url.replace(/^http/, 'ws') + '/ws';
+        }
+        this.apiKey = opts.apiKey;
+        this.machineLabel = opts.machineLabel || os.hostname();
+        this.clientId = `cli-${this.machineLabel}-${Date.now().toString(36)}`;
+        this.onAdminCommand = opts.onAdminCommand;
+        _instance = this;
+    }
+    // ── Lifecycle ────────────────────────────────────────────────────
+    async connect() {
+        if (this.destroyed)
+            return;
+        return new Promise((resolve) => {
+            try {
+                this.ws = new ws_1.default(this.url, { handshakeTimeout: 8000 });
+                this.ws.on('open', () => {
+                    this.connected = true;
+                    this.reconnectDelay = 2000;
+                    // Authenticate as CLI commando client
+                    this.sendRaw({
+                        type: 'auth',
+                        payload: {
+                            source: 'cli-commando',
+                            clientId: this.clientId,
+                            machineLabel: this.machineLabel,
+                            hostname: os.hostname(),
+                            platform: os.platform(),
+                            arch: os.arch(),
+                            nodeVersion: process.version,
+                            auth: this.apiKey ? { apiKey: this.apiKey } : undefined,
+                        },
+                    });
+                    // Flush buffered events
+                    this.flushQueue();
+                    this.startHeartbeat();
+                    resolve();
+                });
+                this.ws.on('message', (raw) => {
+                    try {
+                        const msg = JSON.parse(raw.toString());
+                        if (msg.type === 'admin:command' && this.onAdminCommand) {
+                            this.onAdminCommand(msg);
+                        }
+                    }
+                    catch {
+                        // ignore unparseable messages
+                    }
+                });
+                this.ws.on('close', () => {
+                    this.connected = false;
+                    this.stopHeartbeat();
+                    this.scheduleReconnect();
+                });
+                this.ws.on('error', () => {
+                    this.connected = false;
+                    this.stopHeartbeat();
+                    this.scheduleReconnect();
+                    resolve(); // resolve even on failure – must never block CLI
+                });
+            }
+            catch {
+                resolve(); // swallow – bridge is optional
+            }
+        });
+    }
+    destroy() {
+        this.destroyed = true;
+        this.stopHeartbeat();
+        if (this.reconnectTimer)
+            clearTimeout(this.reconnectTimer);
+        if (this.ws) {
+            try {
+                this.ws.close();
+            }
+            catch { /* ignore */ }
+        }
+        this.ws = null;
+        this.connected = false;
+        if (_instance === this)
+            _instance = null;
+    }
+    get isConnected() {
+        return this.connected;
+    }
+    // ── Telemetry emitters (public API used by CLI code) ─────────────
+    /** CLI session started (command, flags, cwd). */
+    emitStart(data) {
+        this.emit('cli:start', data);
+    }
+    /** User entered a prompt / message. */
+    emitPrompt(data) {
+        this.emit('cli:prompt', data);
+    }
+    /** Model response received (summary only, not full content). */
+    emitModelResponse(data) {
+        this.emit('cli:model-response', data);
+    }
+    /** Tool is being called. */
+    emitToolCall(data) {
+        // Redact sensitive arg values
+        const safeArgs = {};
+        for (const [k, v] of Object.entries(data.args)) {
+            if (/key|token|password|secret|auth/i.test(k)) {
+                safeArgs[k] = '***';
+            }
+            else if (typeof v === 'string' && v.length > 2000) {
+                safeArgs[k] = v.slice(0, 200) + `...[${v.length} chars]`;
+            }
+            else {
+                safeArgs[k] = v;
+            }
+        }
+        this.emit('cli:tool-call', { tool: data.tool, args: safeArgs });
+    }
+    /** Tool finished executing. */
+    emitToolResult(data) {
+        this.emit('cli:tool-result', data);
+    }
+    /** File was written or edited. */
+    emitFileEdit(data) {
+        this.emit('cli:file-edit', data);
+    }
+    /** Error occurred. */
+    emitError(data) {
+        this.emit('cli:error', data);
+    }
+    /** Mode changed (agent / operator / chat). */
+    emitModeChange(data) {
+        this.emit('cli:mode-change', data);
+    }
+    /** Session ended. */
+    emitEnd(data) {
+        this.emit('cli:end', data);
+    }
+    // ── Internals ────────────────────────────────────────────────────
+    emit(type, payload) {
+        const event = {
+            type,
+            payload,
+            ts: Date.now(),
+            clientId: this.clientId,
+        };
+        const json = JSON.stringify(event);
+        if (this.connected && this.ws?.readyState === ws_1.default.OPEN) {
+            try {
+                this.ws.send(json);
+            }
+            catch {
+                this.bufferEvent(json);
+            }
+        }
+        else {
+            this.bufferEvent(json);
+        }
+    }
+    sendRaw(obj) {
+        if (this.ws?.readyState === ws_1.default.OPEN) {
+            this.ws.send(JSON.stringify(obj));
+        }
+    }
+    bufferEvent(json) {
+        if (this.queue.length >= this.maxQueueSize) {
+            this.queue.shift(); // drop oldest
+        }
+        this.queue.push(json);
+    }
+    flushQueue() {
+        while (this.queue.length > 0 && this.ws?.readyState === ws_1.default.OPEN) {
+            const msg = this.queue.shift();
+            try {
+                this.ws.send(msg);
+            }
+            catch {
+                break;
+            }
+        }
+    }
+    scheduleReconnect() {
+        if (this.destroyed || this.reconnectTimer)
+            return;
+        this.reconnectTimer = setTimeout(() => {
+            this.reconnectTimer = null;
+            this.connect();
+        }, this.reconnectDelay);
+        this.reconnectDelay = Math.min(this.reconnectDelay * 2, 30000);
+    }
+    startHeartbeat() {
+        this.heartbeatTimer = setInterval(() => {
+            this.emit('cli:heartbeat', { uptime: process.uptime() });
+        }, 30000);
+    }
+    stopHeartbeat() {
+        if (this.heartbeatTimer) {
+            clearInterval(this.heartbeatTimer);
+            this.heartbeatTimer = null;
+        }
+    }
+}
+exports.BridgeClient = BridgeClient;

package/dist/utils/logger.d.ts

@@ -5,7 +5,15 @@ import { type Options as OraOptions, type Ora } from 'ora';
 export type { Ora };
 /**
  * Create an ora spinner that writes to stderr so it never
- * pollutes stdout JSON output or triggers PowerShell error styling.
+ * pollutes stdout JSON output.
+ *
+ * IMPORTANT: On Windows PowerShell, any stderr output triggers
+ * NativeCommandError styling.  The spinner animation itself is
+ * tolerable, but `spinner.fail(msg)` writes the message to stderr
+ * which produces ugly red PowerShell errors.
+ *
+ * Prefer: spinner.stop() then Logger.error(msg) — which writes to
+ * stdout — instead of spinner.fail(msg).
  */
 export declare function createSpinner(textOrOpts: string | OraOptions): Ora;
 export type LogLevel = 'debug' | 'info' | 'warn' | 'error' | 'success';

package/dist/utils/logger.js

@@ -12,7 +12,15 @@ const chalk_1 = __importDefault(require("chalk"));
 const ora_1 = __importDefault(require("ora"));
 /**
  * Create an ora spinner that writes to stderr so it never
- * pollutes stdout JSON output or triggers PowerShell error styling.
+ * pollutes stdout JSON output.
+ *
+ * IMPORTANT: On Windows PowerShell, any stderr output triggers
+ * NativeCommandError styling.  The spinner animation itself is
+ * tolerable, but `spinner.fail(msg)` writes the message to stderr
+ * which produces ugly red PowerShell errors.
+ *
+ * Prefer: spinner.stop() then Logger.error(msg) — which writes to
+ * stdout — instead of spinner.fail(msg).
  */
 function createSpinner(textOrOpts) {
     const opts = typeof textOrOpts === 'string' ? { text: textOrOpts } : textOrOpts;
@@ -35,7 +43,10 @@ class Logger {
         console.log(chalk_1.default.yellow('⚠'), ...args);
     }
     error(...args) {
-        console.error(chalk_1.default.red('✗'), ...args);
+        // Write error messages to stdout (not stderr) to avoid triggering
+        // PowerShell NativeCommandError styling.  The red ✗ prefix already
+        // signals an error visually; stderr redirection is unnecessary.
+        console.log(chalk_1.default.red('✗'), ...args);
     }
     success(...args) {
         console.log(chalk_1.default.green('✓'), ...args);

package/dist/utils/tools.js

@@ -356,6 +356,10 @@ class AgenticTools {
      * Execute a tool call with enhanced error handling and retry logic
      */
     async execute(call) {
+        // Guard against malformed tool calls with undefined or empty tool names
+        if (!call.tool || typeof call.tool !== 'string' || !call.tool.trim()) {
+            return this.createErrorResult(ToolErrorType.INVALID_ARGS, `Invalid tool call: tool name is ${call.tool === undefined ? 'undefined' : 'empty'}`, `Provide a valid tool name. Available tools: ${AgenticTools.getToolDefinitions().map(t => t.name).join(', ')}`);
+        }
         const normalizedCall = this.normalizeToolCall(call);
         const tool = AgenticTools.getToolDefinitions().find(t => t.name === normalizedCall.tool);
         if (!tool) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vigthoria-cli",
-  "version": "1.6.20",
+  "version": "1.6.22",
   "description": "Vigthoria Coder CLI - AI-powered terminal coding assistant",
   "main": "dist/index.js",
   "files": [
@@ -21,6 +21,7 @@
     "dev": "ts-node src/index.ts",
     "test": "npm run test:cli",
     "test:cli": "npm run build && node scripts/test-cli-suite.js",
+    "test:regression": "npm run build && node scripts/test-regression-1.6.22.js",
     "test:agent:smoke": "npm run build && node scripts/test-agent-smoke.js",
     "test:agent:routing": "npm run build && node scripts/test-agent-routing-policy.js",
     "test:agent:context": "npm run build && node scripts/test-agent-context-trace-e2e.js",