vigthoria-cli 1.6.26 → 1.6.28

package/dist/commands/chat.js

@@ -421,11 +421,33 @@ class ChatCommand {
             return;
         }
         if (options.prompt) {
+            // Wrap in a timeout to guarantee bridge cleanup even if the agent
+            // loop hangs (e.g. model takes forever on a turn).
+            const BRIDGE_TIMEOUT_MS = 180_000; // 3 minutes max for a single prompt
+            let timedOut = false;
+            const timeoutId = options.bridge
+                ? setTimeout(() => {
+                    timedOut = true;
+                    const b = (0, bridge_client_js_1.getBridgeClient)();
+                    if (b) {
+                        b.emitEnd({ reason: 'timeout' });
+                        b.destroy();
+                    }
+                    if (!this.jsonOutput) {
+                        this.logger.error('Bridge prompt timed out after 3 minutes.');
+                    }
+                    process.exitCode = 1;
+                }, BRIDGE_TIMEOUT_MS)
+                : null;
             await this.handleDirectPrompt(options.prompt);
-            const bridge = (0, bridge_client_js_1.getBridgeClient)();
-            if (bridge) {
-                bridge.emitEnd({ reason: 'prompt-complete' });
-                bridge.destroy();
+            if (timeoutId)
+                clearTimeout(timeoutId);
+            if (!timedOut) {
+                const bridge = (0, bridge_client_js_1.getBridgeClient)();
+                if (bridge) {
+                    bridge.emitEnd({ reason: 'prompt-complete' });
+                    bridge.destroy();
+                }
             }
             return;
         }
@@ -980,13 +1002,15 @@ class ChatCommand {
                 if (toolCalls.length === 0) {
                     // Phase 5: Quality gate — if the agent tries to conclude on the first
                     // turn without any discovery, push it to gather evidence first.
-                    if (turn === 0 && this.agentToolEvidence.discovery === 0 && this.isDiagnosticPrompt(prompt)) {
+                    // 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)) {
                         this.messages.push({
                             role: 'system',
                             content: [
                                 'Quality gate: you concluded without using any discovery tools (list_dir, glob, read_file, grep).',
-                                'Before answering a diagnostic or audit question, you MUST inspect the project with tools.',
-                                'Use list_dir and read_file to gather concrete evidence, then provide your answer.',
+                                '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.',
                             ].join('\n'),
                         });
                         this.directToolContinuationCount += 1;

package/dist/commands/edit.d.ts

@@ -23,9 +23,11 @@ export declare class EditCommand {
     private extractCode;
     /**
      * Detect and remove duplicated content in model output.
-     * Models sometimes output the code twice (original + modified).
+     * Models sometimes output the code twice or three times.
+     * Applies iteratively until no further duplicates are found.
      */
     private deduplicateCode;
+    private deduplicateOnce;
     private showDiffAndConfirm;
     private showFullDiff;
     private applyFix;

package/dist/commands/edit.js

@@ -133,7 +133,24 @@ Return the complete modified file content:`,
         try {
             const result = await this.api.fixCode(file.content, file.language, options.type);
             spinner.stop();
+            // Deduplicate the fixed output — models sometimes repeat code 2-3x
+            result.fixed = this.deduplicateCode(result.fixed);
             if (result.changes.length === 0) {
+                // Even if the diff engine found no structured changes, the fixed
+                // code may still differ (e.g. single-char operator swap).  Show a
+                // diff and let the user decide instead of silently discarding.
+                if (result.fixed && result.fixed !== file.content) {
+                    this.logger.section('Found 1 issue(s)');
+                    console.log(chalk_1.default.yellow('1. Operator/character-level fix detected'));
+                    console.log();
+                    if (options.apply) {
+                        await this.applyFix(file.path, file.content, result.fixed);
+                    }
+                    else {
+                        await this.showDiffAndConfirm(file.path, file.content, result.fixed);
+                    }
+                    return;
+                }
                 this.logger.success('No issues found!');
                 return;
             }
@@ -180,14 +197,39 @@ Return the complete modified file content:`,
     }
     /**
      * Detect and remove duplicated content in model output.
-     * Models sometimes output the code twice (original + modified).
+     * Models sometimes output the code twice or three times.
+     * Applies iteratively until no further duplicates are found.
      */
     deduplicateCode(code) {
+        let result = code;
+        // Iterate up to 3 times to handle triple+ duplication
+        for (let pass = 0; pass < 3; pass++) {
+            const deduped = this.deduplicateOnce(result);
+            if (deduped === result)
+                break;
+            result = deduped;
+        }
+        return result;
+    }
+    deduplicateOnce(code) {
         const lines = code.split('\n');
         const len = lines.length;
         if (len < 4)
             return code;
-        // Check if the second half is a near-duplicate of the first half
+        // Pass 1: Remove adjacent duplicate non-empty lines (model stutter)
+        const deduped = [];
+        for (let i = 0; i < lines.length; i++) {
+            const trimmed = lines[i].trim();
+            const prevTrimmed = deduped.length > 0 ? deduped[deduped.length - 1].trim() : null;
+            if (trimmed && trimmed === prevTrimmed) {
+                continue; // skip consecutive duplicate
+            }
+            deduped.push(lines[i]);
+        }
+        if (deduped.length < lines.length) {
+            return deduped.join('\n');
+        }
+        // Pass 2: Check if the second half is a near-duplicate of the first half
         for (let splitAt = Math.floor(len * 0.4); splitAt <= Math.ceil(len * 0.6); splitAt++) {
             const firstHalf = lines.slice(0, splitAt);
             const secondHalf = lines.slice(splitAt).filter(l => l.trim() !== '');

package/dist/index.js

@@ -98,7 +98,7 @@ function getVersion() {
     catch (e) {
         // Fallback to hardcoded version
     }
-    return '1.6.26';
+    return '1.6.27';
 }
 const VERSION = getVersion();
 /**
@@ -576,10 +576,9 @@ async function main() {
             force: options.force
         });
     });
-    // Default repo action shows list
-    repoCommand.action(async () => {
-        const repo = new repo_js_1.RepoCommand(config, logger);
-        await repo.list({});
+    // Default repo action shows help with available subcommands
+    repoCommand.action(() => {
+        repoCommand.outputHelp();
     });
     // ==================== DEPLOY COMMANDS ====================
     // Deploy command - Host projects on Vigthoria

package/dist/utils/api.js

@@ -50,7 +50,7 @@ class APIClient {
         // Main Vigthoria Coder API (coder.vigthoria.io)
         this.client = axios_1.default.create({
             baseURL: config.get('apiUrl'),
-            timeout: 120000,
+            timeout: 300000, // 5 minutes — covers review, fix, and agent relay flows
             httpsAgent,
             headers: {
                 'Content-Type': 'application/json',
@@ -3374,7 +3374,7 @@ document.addEventListener('DOMContentLoaded', () => {
      * for a pure programming language like JavaScript, Python, etc.
      */
     codeContainsDomPollution(code) {
-        const domPatterns = /document\.(createElement|querySelector|getElementById|getElementsBy|body|head|addEventListener)|innerHTML|\.style\.(cssText|position|transform|animation)|@keyframes|\.appendChild|\.removeChild|window\.(onload|addEventListener)/;
+        const domPatterns = /document\.(createElement|querySelector|getElementById|getElementsBy|body|head|addEventListener)|innerHTML|\.style\.(cssText|position|transform|animation)|@keyframes|\.appendChild|\.removeChild|window\.(onload|addEventListener)|CSSAnimation|new\s+Animation\b|\.getAnimations\s*\(|\.animate\s*\(/;
         return domPatterns.test(code);
     }
     /**
@@ -3388,7 +3388,7 @@ document.addEventListener('DOMContentLoaded', () => {
         let braceDepth = 0;
         for (const line of lines) {
             // Detect start of DOM blocks
-            if (/document\.|\.style\.|\.appendChild|\.removeChild|\.textContent\s*=|@keyframes|addEventListener/.test(line) && !insideDomBlock) {
+            if (/document\.|\.style\.|\.appendChild|\.removeChild|\.textContent\s*=|@keyframes|addEventListener|CSSAnimation|\.getAnimations|\.animate\s*\(/.test(line) && !insideDomBlock) {
                 insideDomBlock = true;
                 braceDepth = 0;
             }
@@ -3456,7 +3456,18 @@ document.addEventListener('DOMContentLoaded', () => {
         // even when the server only reports style issues like console.log.
         const heuristic = this.heuristicCodeIssues(code, language);
         for (const h of heuristic) {
-            // Avoid duplicating issues the server already reported on the same line
+            // Always include critical logic bugs (severity error) from heuristics
+            // regardless of server results — these catch wrong-operator bugs the
+            // server frequently misses.
+            if (h.severity === 'error') {
+                const exactDuplicate = issues.some((existing) => existing.line === h.line && existing.message === h.message);
+                if (!exactDuplicate) {
+                    issues.push(h);
+                }
+                continue;
+            }
+            // For non-critical heuristics, avoid duplicating issues the server
+            // already reported on the same line with the same type.
             const isDuplicate = issues.some((existing) => existing.line === h.line && existing.type === h.type);
             if (!isDuplicate) {
                 issues.push(h);
@@ -3578,7 +3589,7 @@ document.addEventListener('DOMContentLoaded', () => {
             if (fixType === 'bugs' || fixType === 'logic')
                 return issue.type === 'logic' || issue.severity === 'error';
             if (fixType === 'syntax')
-                return issue.severity === 'error';
+                return issue.type !== 'logic' && issue.severity === 'error';
             if (fixType === 'style')
                 return issue.type === 'style' || issue.type === 'quality';
             if (fixType === 'security')
@@ -3619,7 +3630,11 @@ document.addEventListener('DOMContentLoaded', () => {
         // a semantic diff using LCS so inserted/removed lines don't cause
         // every subsequent line to appear as changed.
         if (changes.length === 0 && fixed !== code) {
-            changes = this.computeSemanticDiff(code, fixed, allHints);
+            // Use a clean reason string — strip verbose static-analysis hints
+            const cleanReason = relevantIssues.length > 0
+                ? relevantIssues.map(i => i.message).join('; ')
+                : 'AI-suggested fix';
+            changes = this.computeSemanticDiff(code, fixed, cleanReason);
         }
         return { fixed, changes };
     }

package/package.json

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