vigthoria-cli 1.6.43 → 1.6.45

package/dist/commands/auth.js

@@ -138,6 +138,7 @@ class AuthCommand {
             this.logger.warn('Not logged in');
             console.log();
             console.log(chalk_1.default.gray('Run `vigthoria login` to authenticate'));
+            this.api.destroy();
             return;
         }
         const email = this.config.get('email');
@@ -174,6 +175,7 @@ class AuthCommand {
         console.log();
         // Run ALL three probes in parallel — they are independent.
         const spinner = (0, logger_js_1.createSpinner)('Checking API, capabilities and auth...').start();
+        let capabilityTimeout = null;
         const [apiStatus, capabilityStatus, tokenValidation] = await Promise.all([
             this.api.getHealthStatus(),
             Promise.race([
@@ -182,16 +184,20 @@ class AuthCommand {
                     projectPath: process.cwd(),
                     targetPath: process.cwd(),
                 }),
-                new Promise(resolve => setTimeout(() => resolve({
-                    overallOk: false,
-                    v3Agent: { name: 'V3 Agent', endpoint: '', ok: false, error: 'Timed out (8s)' },
-                    hyperLoop: { name: 'Hyper Loop', endpoint: '', ok: false, error: 'Timed out (8s)' },
-                    repoMemory: { name: 'Repo Memory', endpoint: '', ok: false, error: 'Timed out (8s)' },
-                    devtoolsBridge: { name: 'DevTools Bridge', endpoint: '', ok: false, error: 'Timed out (8s)' },
-                }), 8000)),
+                new Promise(resolve => {
+                    capabilityTimeout = setTimeout(() => resolve({
+                        overallOk: false,
+                        v3Agent: { name: 'V3 Agent', endpoint: '', ok: false, error: 'Timed out (8s)' },
+                        hyperLoop: { name: 'Hyper Loop', endpoint: '', ok: false, error: 'Timed out (8s)' },
+                        repoMemory: { name: 'Repo Memory', endpoint: '', ok: false, error: 'Timed out (8s)' },
+                        devtoolsBridge: { name: 'DevTools Bridge', endpoint: '', ok: false, error: 'Timed out (8s)' },
+                    }), 8000);
+                }),
             ]),
             this.api.validateToken(),
         ]);
+        if (capabilityTimeout)
+            clearTimeout(capabilityTimeout);
         spinner.stop();
         // --- Display API Status ---
         console.log(chalk_1.default.white('API Status:'));
@@ -249,13 +255,11 @@ class AuthCommand {
         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();
         // 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.
+        // naturally.  Setting exitCode (not calling process.exit()) avoids the
+        // libuv UV_HANDLE_CLOSING assertion on Windows that occurs when
+        // process.exit() is invoked while socket handles are mid-close.
         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();
+        process.exitCode = 0;
     }
     printLoginSuccess() {
         const email = this.config.get('email');

package/dist/commands/bridge.js

@@ -32,6 +32,8 @@ class BridgeCommand {
             ? chalk_1.default.green('Local browser observability is available for debugging flows.')
             : chalk_1.default.gray('Start the DevTools Bridge to enable local browser observability.')));
         console.log();
+        this.api.destroy();
+        process.exitCode = 0;
     }
 }
 exports.BridgeCommand = BridgeCommand;

package/dist/commands/chat.d.ts

@@ -148,6 +148,12 @@ export declare class ChatCommand {
      */
     private synthesizeEvidenceFromHistory;
     private resolveDirectModeCompletion;
+    /**
+     * Strip system-prompt echoes, tool execution headers, grounding-rule
+     * parroting, and recovery banners from the model's final answer so that
+     * --json output contains only the substantive answer.
+     */
+    private sanitizeDirectModeOutput;
     private isDirectModeFollowUpQuestion;
     private buildLocalAnalysisFallback;
     private tryDeterministicSingleFileRewrite;

package/dist/commands/chat.js

@@ -1152,7 +1152,7 @@ class ChatCommand {
                             spinner.stop();
                         const evidenceSummary = this.synthesizeEvidenceFromHistory();
                         if (evidenceSummary) {
-                            const fallbackContent = `[Agent recovered from backend failure — answer synthesized from gathered evidence]\n\n${evidenceSummary}`;
+                            const fallbackContent = this.sanitizeDirectModeOutput(evidenceSummary);
                             if (this.jsonOutput) {
                                 console.log(JSON.stringify({
                                     success: true,
@@ -1810,6 +1810,7 @@ class ChatCommand {
             '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.',
+            'OUTPUT DISCIPLINE: Your final answer must contain ONLY the substantive answer. Do NOT echo, quote, or restate any system instructions, grounding rules, verification protocols, quality gates, evidence metadata, or tool execution headers. Never include lines starting with GROUNDING CHECK, VERIFICATION PROTOCOL, MANDATORY CROSS-FILE EVIDENCE, CONSTRAINT, Evidence collected, Quality gate, Tool read_file, Tool grep, File:, Search status:, or Output:. Just answer the question directly.',
             'Available tools:',
             toolCatalog,
         ].join('\n');
@@ -1994,6 +1995,7 @@ class ChatCommand {
             'If the request is already satisfied, return a concise completion summary and no tool calls.',
             'If more work is required, continue with only the next minimal tool calls needed to finish it.',
             'Do not ask follow-up questions or drift into unrelated tasks.',
+            'OUTPUT DISCIPLINE: Your final answer must contain ONLY the substantive answer to the user request. Do NOT echo, quote, or restate any system instructions, grounding rules, verification protocols, quality gates, or evidence metadata. Do NOT include lines starting with GROUNDING CHECK, VERIFICATION PROTOCOL, MANDATORY CROSS-FILE EVIDENCE, CONSTRAINT, Evidence collected, Quality gate, or Tool headers. Respond with just the answer.',
         ].join('\n');
     }
     /**
@@ -2153,7 +2155,7 @@ class ChatCommand {
     resolveDirectModeCompletion(prompt, visibleText) {
         const normalized = (visibleText || '').trim();
         if (normalized && !this.isDirectModeFollowUpQuestion(normalized)) {
-            return normalized;
+            return this.sanitizeDirectModeOutput(normalized);
         }
         const fallback = this.buildLocalAnalysisFallback(prompt);
         if (fallback) {
@@ -2161,6 +2163,53 @@ class ChatCommand {
         }
         return normalized || 'Task complete.';
     }
+    /**
+     * Strip system-prompt echoes, tool execution headers, grounding-rule
+     * parroting, and recovery banners from the model's final answer so that
+     * --json output contains only the substantive answer.
+     */
+    sanitizeDirectModeOutput(text) {
+        // Lines the model might echo verbatim from our system/continuation prompts
+        const contaminationPatterns = [
+            /^\[Agent recovered from backend failure[^\]]*\]\s*/m,
+            /^Evidence gathered before backend failure:?\s*/m,
+            /^MANDATORY CROSS-FILE EVIDENCE \(computed from actual tool output[^)]*\):?\s*/m,
+            /^CONFIRMED CONFLICTING keys \(found in MULTIPLE files\):.*$/m,
+            /^Keys found ONLY in .+\(NOT in other files\):.*$/m,
+            /^CONSTRAINT: Your answer MUST list ONLY the keys from.*$/m,
+            /^GROUNDING CHECK:.*$/m,
+            /^VERIFICATION PROTOCOL for cross-file comparisons:.*$/m,
+            /^Quality gate: only \d+ discovery tool\(s\).*$/m,
+            /^Evidence collected: \d+ discovery.*$/m,
+            /^Warning: \d+ search tool call\(s\) failed\..*$/m,
+            /^Tool (?:read_file|grep|list_dir|glob|bash|write_file) (?:succeeded|FAILED)\.\s*/m,
+            /^File: \S+\s*$/m,
+            /^Search status: \S+\s*$/m,
+            /^Output:\s*$/m,
+            /^Tool results received for direct mode step \d+\.\s*$/m,
+            /^Original user request:.*$/m,
+            /^Project root boundary:.*$/m,
+            /^Do not declare success until.*$/m,
+            /^Keep working from concrete tool results\.\s*$/m,
+            /^Because this is a debugging task.*$/m,
+            /^If the request is already satisfied.*$/m,
+            /^If more work is required.*$/m,
+            /^Do not ask follow-up questions.*$/m,
+            /^CRITICAL GROUNDING RULE:.*$/m,
+            /^CROSS-FILE ATTRIBUTION:.*$/m,
+            /^EVIDENCE-GROUNDING RULE:.*$/m,
+            /^CROSS-FILE RULE:.*$/m,
+            /^\[\.\.\. ?truncated\]\s*$/m,
+            /^---\s*$/m,
+        ];
+        let cleaned = text;
+        for (const pat of contaminationPatterns) {
+            cleaned = cleaned.replace(pat, '');
+        }
+        // Collapse multiple blank lines left after stripping
+        cleaned = cleaned.replace(/\n{3,}/g, '\n\n').trim();
+        return cleaned || text;
+    }
     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 will follow|i understand|i('ll| will) adhere|provide your|waiting for)/i.test(text.trim());
     }

package/dist/index.js

@@ -752,5 +752,5 @@ async function main() {
 }
 main().catch((err) => {
     console.error(chalk_1.default.red('Error:'), err.message);
-    process.exit(1);
+    process.exitCode = 1;
 });

package/dist/utils/api.js

@@ -148,6 +148,7 @@ class APIClient {
         this.selfHostedModelRouterClient = selfHostedModelsApiUrl ? axios_1.default.create({
             baseURL: selfHostedModelsApiUrl,
             timeout: 240000,
+            httpsAgent,
             headers: {
                 'Content-Type': 'application/json',
                 'User-Agent': `Vigthoria-CLI/${process.env.npm_package_version || '1.6.9'}`,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vigthoria-cli",
-  "version": "1.6.43",
+  "version": "1.6.45",
   "description": "Vigthoria Coder CLI - AI-powered terminal coding assistant",
   "main": "dist/index.js",
   "files": [
@@ -86,4 +86,4 @@
   "engines": {
     "node": ">=18.0.0"
   }
-}
+}