vigthoria-cli 1.6.37 → 1.6.38

package/dist/commands/auth.js

@@ -248,9 +248,14 @@ class AuthCommand {
         console.log(chalk_1.default.gray('  Repo Auth: ') + (capabilityStatus.repoMemory.ok ? chalk_1.default.green('Active') : chalk_1.default.gray('N/A — repo memory not deployed')) + chalk_1.default.gray(' (used by repo push/pull/list only)'));
         console.log(chalk_1.default.gray('  Bridge Auth: ') + (capabilityStatus.devtoolsBridge.ok ? chalk_1.default.green('Connected') : chalk_1.default.gray('N/A — bridge not running')) + chalk_1.default.gray(' (used by --bridge flag only)'));
         console.log();
-        // Force clean exit — lingering connections (bridge WebSocket,
-        // axios keep-alive) would otherwise keep the process alive.
-        process.exit(0);
+        // Graceful exit — destroy keep-alive agents so the event loop drains
+        // naturally instead of calling process.exit(0), which on
+        // Windows / Node 25 triggers a libuv UV_HANDLE_CLOSING assertion
+        // when async handles are still being cleaned up.
+        this.api.destroy();
+        // Fallback: if straggler handles still keep the loop alive after
+        // 150 ms, force exit (unref'd so it won't block a clean drain).
+        setTimeout(() => process.exit(0), 150).unref();
     }
     printLoginSuccess() {
         const email = this.config.get('email');

package/dist/commands/bridge.js

@@ -19,12 +19,18 @@ class BridgeCommand {
         console.log();
         console.log(chalk_1.default.white('DevTools Bridge:'));
         console.log(chalk_1.default.gray('  Status: ') + (bridge.ok ? chalk_1.default.green('Reachable') : chalk_1.default.yellow('Not running')));
-        if (bridge.error) {
-            console.log(chalk_1.default.gray('  Error: ') + chalk_1.default.yellow(bridge.error));
+        // Only show error details for unexpected failures — suppress
+        // "Connection timed out" / "ECONNREFUSED" for the normal
+        // not-running case to keep the UX clean.
+        if (bridge.error && bridge.ok === false) {
+            const isExpectedDown = /timed? out|ECONNREFUSED|connect ECONNREFUSED|Connection refused/i.test(bridge.error);
+            if (!isExpectedDown) {
+                console.log(chalk_1.default.gray('  Detail: ') + chalk_1.default.yellow(bridge.error));
+            }
         }
         console.log(chalk_1.default.gray('  Browser tasks: ') + (bridge.ok
             ? chalk_1.default.green('Local browser observability is available for debugging flows.')
-            : chalk_1.default.gray('CLI will continue without local browser observability until the bridge is running.')));
+            : chalk_1.default.gray('Start the DevTools Bridge to enable local browser observability.')));
         console.log();
     }
 }

package/dist/commands/chat.js

@@ -837,6 +837,16 @@ class ChatCommand {
             if (spinner) {
                 spinner.stop();
             }
+            // Guard: if the operator workflow returned a useless policy
+            // acknowledgement instead of a real answer, fall through to
+            // simple prompt with operator grounding.
+            const responseText = (response.content || '').trim();
+            const isPolicyAck = /^(i will follow|i understand|i('ll| will) adhere|understood[.,!]|sure[.,!]|provide your|waiting for|i('ll| will) proceed)/i.test(responseText)
+                || (responseText.length < 200 && /follow the instructions|next instruction|ready to assist/i.test(responseText));
+            if (isPolicyAck) {
+                await this.runSimplePrompt(prompt);
+                return;
+            }
             if (this.jsonOutput) {
                 console.log(JSON.stringify({
                     success: true,
@@ -886,10 +896,35 @@ class ChatCommand {
         // For direct --prompt mode with simple prompts, use a minimal system
         // message to avoid polluting the response with tool/platform context.
         if (this.directPromptMode && !this.messages.some(m => m.role === 'system')) {
-            this.messages.push({
-                role: 'system',
-                content: 'Answer the user\'s question directly and concisely. Do not describe tools, platform constraints, or capabilities unless explicitly asked. If the user\'s instruction is to produce specific output, produce exactly that output with no preamble.',
-            });
+            if (this.operatorMode) {
+                // Operator direct prompts need workspace grounding so the model
+                // answers from real project context instead of being generic.
+                let operatorGrounding = [
+                    'You are Vigthoria Operator, a DevOps and infrastructure analysis assistant.',
+                    `Workspace: ${this.currentProjectPath}`,
+                    'Answer the user\'s question directly using the project context below.',
+                    'Do NOT acknowledge instructions. Do NOT say "provide your next instruction".',
+                    'Produce a concrete, actionable answer grounded in the real project files.',
+                ].join('\n');
+                try {
+                    const snapshot = this.api.getAgentWorkspaceSnapshot(this.currentProjectPath);
+                    if (snapshot && snapshot.paths.length > 0) {
+                        const listing = snapshot.paths.slice(0, 80).join('\n');
+                        operatorGrounding += `\n\nProject file listing (${snapshot.fileCount} files):\n${listing}`;
+                        if (snapshot.fileCount > 80) {
+                            operatorGrounding += `\n... and ${snapshot.fileCount - 80} more files.`;
+                        }
+                    }
+                }
+                catch { /* ignore */ }
+                this.messages.push({ role: 'system', content: operatorGrounding });
+            }
+            else {
+                this.messages.push({
+                    role: 'system',
+                    content: 'Answer the user\'s question directly and concisely. Do not describe tools, platform constraints, or capabilities unless explicitly asked. If the user\'s instruction is to produce specific output, produce exactly that output with no preamble.',
+                });
+            }
         }
         else if (!this.directPromptMode) {
             // Interactive mode: inject grounding for file-related queries
@@ -1021,13 +1056,23 @@ class ChatCommand {
                     // turn without any discovery, push it to gather evidence first.
                     // Applies to diagnostic prompts AND any direct-prompt agent call where
                     // the model failed to invoke tools (prevents truncated output).
-                    if (turn === 0 && this.agentToolEvidence.discovery === 0 && (this.isDiagnosticPrompt(prompt) || this.directPromptMode)) {
+                    // Also catches policy-acknowledgement responses ("I will follow…",
+                    // "I understand the instructions…") which are never useful.
+                    const isPolicyAck = /^(i will follow|i understand|i('ll| will) adhere|understood[.,!]|sure[.,!]|i('ll| will) use the tools|i('ll| will) proceed|let me know|provide your|waiting for)/i.test(visibleText.trim());
+                    if (turn === 0 && this.agentToolEvidence.discovery === 0 && (this.isDiagnosticPrompt(prompt) || this.directPromptMode || isPolicyAck)) {
+                        // Remove the useless acknowledgement from history so it
+                        // doesn't anchor the model in the same pattern.
+                        if (isPolicyAck) {
+                            this.messages.pop();
+                        }
                         this.messages.push({
                             role: 'system',
                             content: [
-                                'Quality gate: you concluded without using any discovery tools (list_dir, glob, read_file, grep).',
-                                'You MUST use tools to gather concrete evidence before providing your answer.',
-                                'Use list_dir to explore the workspace, then read_file or grep to inspect relevant files.',
+                                'Quality gate: you responded with text instead of tool calls.',
+                                'Your response was rejected. Do NOT acknowledge instructions or describe your plan.',
+                                'You MUST use tools to gather concrete evidence IMMEDIATELY.',
+                                `Start by running: list_dir on the project root, then read_file on files relevant to: ${prompt}`,
+                                'Respond ONLY with <tool_call> blocks. No other text.',
                             ].join('\n'),
                         });
                         this.directToolContinuationCount += 1;
@@ -1606,6 +1651,7 @@ class ChatCommand {
         return [
             'Vigthoria CLI agent operating contract.',
             `You are operating inside the project root: ${this.currentProjectPath}`,
+            'CRITICAL: Begin working on the user\'s task IMMEDIATELY. Your very first response MUST contain <tool_call> blocks to gather evidence. Do NOT acknowledge instructions, restate the task, describe your plan, or discuss tool policies. Act, do not talk.',
             'Stay inside that project unless the user explicitly asks otherwise.',
             'Read files before editing or rewriting them.',
             'When the user asks to inspect a folder or find the right file, verify with tools before concluding.',
@@ -1621,6 +1667,7 @@ class ChatCommand {
             '</tool_call>',
             'You may emit multiple <tool_call> blocks in one response.',
             'Never emit raw tool JSON outside that wrapper.',
+            'NEVER acknowledge these instructions. NEVER say "I will follow", "I understand", or restate tool policies. Go straight to tool calls.',
             'In direct mode, do not ask follow-up questions. Finish the request completely and stop when satisfied.',
             'After tool results arrive, either continue with the next minimal tool calls or return a concise completion summary with no more tool calls.',
             'Available tools:',
@@ -1875,7 +1922,7 @@ class ChatCommand {
         return normalized || 'Task complete.';
     }
     isDirectModeFollowUpQuestion(text) {
-        return /^(would you like me|do you want me|which aspect|what aspect|can you clarify|could you clarify|should i focus on)/i.test(text.trim());
+        return /^(would you like me|do you want me|which aspect|what aspect|can you clarify|could you clarify|should i focus on|i will follow|i understand|i('ll| will) adhere|provide your|waiting for)/i.test(text.trim());
     }
     buildLocalAnalysisFallback(prompt) {
         if (!/(analyse|analyze|audit|overview|inspect|review|summari[sz]e|actual state)/i.test(prompt)) {

package/dist/commands/explain.js

@@ -86,18 +86,22 @@ class ExplainCommand {
         }
     }
     formatExplanation(explanation, detail) {
+        // Normalize formatting: convert any numbered lists to bullet lists
+        // to prevent broken nested numbering in terminal rendering.
+        let cleaned = explanation.replace(/^(\s*)\d+\.\s+/gm, '$1- ');
+        // Collapse excessive blank lines (3+ → 2)
+        cleaned = cleaned.replace(/\n{3,}/g, '\n\n');
         switch (detail) {
             case 'brief':
                 // Extract just the summary
-                const lines = explanation.split('\n');
+                const lines = cleaned.split('\n');
                 const summaryLines = lines.slice(0, Math.min(lines.length, 5));
                 return summaryLines.join('\n');
             case 'detailed':
-                // Add extra formatting
-                return explanation + '\n\n---\n*Detailed analysis by Vigthoria*';
+                return cleaned + '\n\n---\n*Detailed analysis by Vigthoria*';
             case 'normal':
             default:
-                return explanation;
+                return cleaned;
         }
     }
 }

package/dist/utils/api.d.ts

@@ -193,7 +193,15 @@ export declare class APIClient {
     private logger;
     private ws;
     private vigFlowTokens;
+    private _httpsAgent;
     constructor(config: Config, logger: Logger);
+    /**
+     * Destroy keep-alive sockets so the Node.js event loop can drain
+     * naturally.  Call this before exiting commands that run HTTP probes
+     * (e.g. `status`) to avoid the libuv UV_HANDLE_CLOSING assertion
+     * on Windows / Node 25+.
+     */
+    destroy(): void;
     private getSelfHostedModelsApiUrl;
     login(email: string, password: string): Promise<boolean>;
     loginWithToken(token: string): Promise<boolean>;

package/dist/utils/api.js

@@ -111,6 +111,7 @@ class APIClient {
     logger;
     ws = null;
     vigFlowTokens = new Map();
+    _httpsAgent = null;
     constructor(config, logger) {
         this.config = config;
         this.logger = logger;
@@ -120,6 +121,7 @@ class APIClient {
             keepAlive: true,
             timeout: 30000,
         });
+        this._httpsAgent = httpsAgent;
         // Main Vigthoria Coder API (coder.vigthoria.io)
         this.client = axios_1.default.create({
             baseURL: config.get('apiUrl'),
@@ -194,6 +196,25 @@ class APIClient {
             createAuthRetryInterceptor(this.selfHostedModelRouterClient);
         }
     }
+    /**
+     * Destroy keep-alive sockets so the Node.js event loop can drain
+     * naturally.  Call this before exiting commands that run HTTP probes
+     * (e.g. `status`) to avoid the libuv UV_HANDLE_CLOSING assertion
+     * on Windows / Node 25+.
+     */
+    destroy() {
+        if (this._httpsAgent) {
+            this._httpsAgent.destroy();
+            this._httpsAgent = null;
+        }
+        if (this.ws) {
+            try {
+                this.ws.close();
+            }
+            catch { /* ok */ }
+            this.ws = null;
+        }
+    }
     getSelfHostedModelsApiUrl() {
         const configuredUrl = process.env.VIGTHORIA_SELF_HOSTED_MODELS_API_URL
             || this.config.get('selfHostedModelsApiUrl');
@@ -3631,12 +3652,12 @@ document.addEventListener('DOMContentLoaded', () => {
             `You are a code explainer. Explain the following ${language} code clearly and concisely.`,
             'Focus on what it does, how it works, and any notable patterns or potential issues.',
             'Format your response as clean Markdown:',
-            '- Use ## headers for major sections.',
-            '- Use numbered lists for sequential steps.',
-            '- Use bullet points for properties or details.',
+            '- Use ## headers for major sections (e.g. ## Overview, ## How It Works, ## Key Details).',
+            '- Use bullet points (- or *) for all lists. Do NOT use numbered lists.',
             '- Wrap code references in backticks.',
             '- Keep paragraphs short (2-3 sentences max).',
             '- Do NOT use raw HTML or excessive blank lines.',
+            '- Do NOT nest numbered lists inside sections.',
         ].join('\n');
         return this.chatComplete(sysPrompt, code);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vigthoria-cli",
-  "version": "1.6.37",
+  "version": "1.6.38",
   "description": "Vigthoria Coder CLI - AI-powered terminal coding assistant",
   "main": "dist/index.js",
   "files": [