vigthoria-cli 1.6.39 → 1.6.41

package/dist/commands/chat.d.ts

@@ -55,8 +55,9 @@ export declare class ChatCommand {
     private isBrowserTaskPrompt;
     /**
      * Returns true when a prompt can be answered directly without the full
-     * BMAD operator workflow.  Matches analysis/lookup questions.
-     * Infrastructure action verbs are already filtered at the routing level.
+     * BMAD operator workflow.  Matches analysis/lookup questions as well as
+     * short echo/smoke-test prompts that don't require code generation,
+     * file scanning, or infrastructure changes.
      */
     private isOperatorDirectAnswerable;
     private inferAgentTaskType;
@@ -92,6 +93,13 @@ export declare class ChatCommand {
     private formatWorkflowTargetResult;
     private runWorkflowTargetPrompt;
     private runOperatorTurn;
+    /**
+     * Fast-path for operator prompts that don't need the full BMAD workflow.
+     * Uses the regular chat API with a strong operator system message so
+     * the user gets an instant, grounded answer instead of waiting for the
+     * BMAD orchestrator to scan the workspace.
+     */
+    private runOperatorDirectAnswer;
     private runSimplePrompt;
     private runAgentTurn;
     private runLocalAgentLoop;

package/dist/commands/chat.js

@@ -82,8 +82,8 @@ class ChatCommand {
             return;
         }
         if (enabledMode === 'operator') {
-            if (this.currentModel === 'agent' || !this.currentModel) {
-                this.currentModel = 'code-8b';
+            if (this.currentModel === 'agent' || this.currentModel === 'code-8b' || !this.currentModel) {
+                this.currentModel = 'code';
             }
             return;
         }
@@ -108,10 +108,12 @@ class ChatCommand {
     resolveInitialModel(options) {
         const requestedModel = String(options.model || '').trim();
         if (requestedModel) {
-            return requestedModel;
+            return options.operator === true && requestedModel === 'code-8b'
+                ? 'code'
+                : requestedModel;
         }
         if (options.operator === true) {
-            return 'code-8b';
+            return 'code';
         }
         if (options.agent === true) {
             return 'agent';
@@ -208,11 +210,17 @@ class ChatCommand {
     }
     /**
      * Returns true when a prompt can be answered directly without the full
-     * BMAD operator workflow.  Matches analysis/lookup questions.
-     * Infrastructure action verbs are already filtered at the routing level.
+     * BMAD operator workflow.  Matches analysis/lookup questions as well as
+     * short echo/smoke-test prompts that don't require code generation,
+     * file scanning, or infrastructure changes.
      */
     isOperatorDirectAnswerable(prompt) {
-        return this.isAnalysisLookupPrompt(prompt.trim());
+        const trimmed = prompt.trim();
+        // Short prompts (under 200 chars) that do not contain action verbs
+        // targeting infrastructure are almost always direct-answerable.
+        const isShortNonAction = trimmed.length < 200
+            && !/\b(deploy|provision|scale|migrate|rollback|restart|install|configure|create|build|scaffold|generate|set up|tear down|spin up|initialize|bootstrap)\b/i.test(trimmed);
+        return this.isAnalysisLookupPrompt(trimmed) || isShortNonAction;
     }
     inferAgentTaskType(prompt) {
         if (this.isDiagnosticPrompt(prompt))
@@ -228,7 +236,7 @@ class ChatCommand {
             instructions.push('Platform: Windows. Use list_dir, glob, read_file, and the grep tool for searching.', 'The grep tool handles Windows automatically — do not use bash to call grep, findstr, or Select-String manually.', 'Do not use bash for Unix commands (cat, head, tail, awk, sed, wc).', 'Use read_file to inspect file contents instead of shell commands.', 'All file paths use forward slashes internally.');
         }
         if (this.isDiagnosticPrompt(prompt)) {
-            instructions.push('Diagnostic mode is active.', 'Treat this as a debugging task, not a generic code review or feature request.', 'Start with concrete evidence: logs, runtime errors, config, launch files, and exact symbol references.', 'If log files exist, inspect them before proposing fixes.', 'Do not claim a file, definition, asset, or symbol is missing until you verify that with tools.', 'If a prior diagnosis mentioned a missing symbol or YAML entry, re-check the actual files before repeating it.', 'Prefer grep plus read_file around the exact references involved in the failure.', 'Separate your reasoning into: Evidence, Confirmed Cause, and Remaining Hypotheses.', 'Do not suggest speculative fixes when the current evidence contradicts them.');
+            instructions.push('Diagnostic mode is active.', 'Treat this as a debugging task, not a generic code review or feature request.', 'Start with concrete evidence: logs, runtime errors, config, launch files, and exact symbol references.', 'If log files exist, inspect them before proposing fixes.', 'Do not claim a file, definition, asset, or symbol is missing until you verify that with tools.', 'If a prior diagnosis mentioned a missing symbol or YAML entry, re-check the actual files before repeating it.', 'Prefer grep plus read_file around the exact references involved in the failure.', 'Separate your reasoning into: Evidence, Confirmed Cause, and Remaining Hypotheses.', 'Do not suggest speculative fixes when the current evidence contradicts them.', 'CRITICAL GROUNDING RULE: Every key name, variable name, symbol, or identifier you mention in your final answer MUST appear verbatim in the tool output you received. If a key/symbol does NOT appear in tool output, you MUST NOT mention it as involved in any conflict or issue.', 'CROSS-FILE ATTRIBUTION: When reporting conflicts between two files, a key/symbol is conflicting ONLY if it appears in BOTH files. Read each file carefully and list only the exact keys that appear in the relevant handler/function of EACH file. Do not assume that because one file handles a key, the other file does too.', 'When reporting conflicts between files, cite the exact file name, line number, and the exact string/key from the tool output. Do not paraphrase or substitute key names.', 'Before concluding, re-check: (1) does every key/symbol in my answer actually appear in the evidence I gathered? (2) for each claimed conflict, did I verify the key appears in BOTH files? If not, correct your answer.');
         }
         if (this.isBrowserTaskPrompt(prompt)) {
             instructions.push('Browser-debug mode is active.', 'Prefer concrete browser evidence such as console errors, network failures, DOM state, and websocket behavior.', 'If a DevTools Bridge is available, use it as the primary browser observability path.');
@@ -673,13 +681,6 @@ class ChatCommand {
                 this.logger.error(this.operatorAccessMessage());
                 return;
             }
-            // Smart routing: infrastructure action verbs always need the BMAD
-            // workflow.  Everything else can be answered with a direct chat.
-            const isInfraAction = /(deploy|provision|scale|replicas|rollback|roll back|migrate|tear.?down|restart|stop\s+\w+|start\s+\w+|configure|set.?up|upgrade|pipeline)/i.test(prompt.trim());
-            if (!isInfraAction && (this.isSimpleDirectPrompt(prompt) || this.isOperatorDirectAnswerable(prompt))) {
-                await this.runSimplePrompt(prompt);
-                return;
-            }
             await this.runOperatorTurn(prompt);
             return;
         }
@@ -788,37 +789,20 @@ class ChatCommand {
             this.logger.error(this.operatorAccessMessage());
             return;
         }
+        // ── Fast-path: direct-answerable prompts skip the full BMAD workflow ──
+        // The BMAD orchestrator scans the entire workspace (can be 1000+ files)
+        // even for trivial prompts like "Reply with exactly: OK".  Route these
+        // through the regular chat API with a strong operator system message
+        // so the user gets an instant, grounded answer.
+        if (this.isOperatorDirectAnswerable(prompt)) {
+            await this.runOperatorDirectAnswer(prompt);
+            return;
+        }
         (0, bridge_client_js_1.getBridgeClient)()?.emitPrompt({ prompt, mode: 'operator', model: this.currentModel });
         const runtimeContext = await this.getPromptRuntimeContext(prompt);
         const spinner = this.jsonOutput ? null : (0, logger_js_1.createSpinner)({ text: 'Thinking like an operator...', spinner: 'clock' }).start();
-        const isLookup = this.isAnalysisLookupPrompt(prompt);
-        const workflowType = this.isDiagnosticPrompt(prompt) || isLookup
-            ? 'analysis_only'
-            : 'full_autonomy';
-        // For lightweight lookup tasks, prepend a grounding constraint so
-        // the operator returns a concise answer rather than a verbose report.
-        let executionPrompt = this.buildExecutionPrompt(prompt);
-        if (isLookup) {
-            // Inject workspace file listing into the prompt so the operator
-            // can answer "which file defines X" without a full agent loop.
-            let fileListing = '';
-            try {
-                const snapshot = this.api.getAgentWorkspaceSnapshot(this.currentProjectPath);
-                if (snapshot && snapshot.paths.length > 0) {
-                    fileListing = '\n\nWorkspace file listing:\n' + snapshot.paths.slice(0, 80).join('\n');
-                }
-            }
-            catch { /* ignore */ }
-            executionPrompt = [
-                'INSTRUCTION: This is a lightweight lookup task. Return ONLY the concise answer the user asked for.',
-                'Do NOT produce a certification report, audit, or analysis blob.',
-                'Do NOT return files_analyzed counts or summary statistics.',
-                'Just answer the question directly.',
-                fileListing,
-                '',
-                executionPrompt,
-            ].join('\n');
-        }
+        const workflowType = 'full';
+        const executionPrompt = this.buildExecutionPrompt(prompt);
         try {
             const response = await this.api.runOperatorWorkflow(executionPrompt, {
                 workspacePath: this.currentProjectPath,
@@ -838,14 +822,12 @@ class ChatCommand {
                 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.
+            // acknowledgement instead of a real grounded answer, fail closed.
             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;
+                throw new api_js_1.CLIError('Operator workflow returned a non-actionable acknowledgement instead of a grounded result.', 'model_backend');
             }
             if (this.jsonOutput) {
                 console.log(JSON.stringify({
@@ -891,6 +873,83 @@ class ChatCommand {
             }
         }
     }
+    /**
+     * Fast-path for operator prompts that don't need the full BMAD workflow.
+     * Uses the regular chat API with a strong operator system message so
+     * the user gets an instant, grounded answer instead of waiting for the
+     * BMAD orchestrator to scan the workspace.
+     */
+    async runOperatorDirectAnswer(prompt) {
+        (0, bridge_client_js_1.getBridgeClient)()?.emitPrompt({ prompt, mode: 'operator-direct', model: this.currentModel });
+        const spinner = this.jsonOutput ? null : (0, logger_js_1.createSpinner)({ text: 'Operator (direct)...', spinner: 'clock' }).start();
+        try {
+            let operatorGrounding = [
+                'You are Vigthoria Operator, a DevOps and infrastructure analysis assistant.',
+                `Workspace: ${this.currentProjectPath}`,
+                'Answer the user\'s question directly with a concrete, actionable answer.',
+                'Do NOT acknowledge instructions. Do NOT say "provide your next instruction".',
+                'If asked to produce exact output, produce exactly that output with no preamble.',
+            ].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 */ }
+            const chatMessages = [
+                { role: 'system', content: operatorGrounding },
+                ...this.getMessagesForModel().filter(m => m.role !== 'system'),
+                { role: 'user', content: prompt },
+            ];
+            const response = await this.api.chat(chatMessages, this.currentModel);
+            if (spinner)
+                spinner.stop();
+            const content = (response.message || '').trim();
+            const isPolicyAck = /^(i will follow|i understand|i('ll| will) adhere|understood[.,!]|sure[.,!]|provide your|waiting for|i('ll| will) proceed)/i.test(content)
+                || (content.length < 200 && /follow the instructions|next instruction|ready to assist/i.test(content));
+            if (isPolicyAck || !content) {
+                throw new api_js_1.CLIError('Operator returned a non-actionable acknowledgement instead of a grounded result.', 'model_backend');
+            }
+            if (this.jsonOutput) {
+                console.log(JSON.stringify({
+                    success: true,
+                    mode: 'operator',
+                    content,
+                    metadata: { source: 'operator-direct', mode: 'operator' },
+                }, null, 2));
+            }
+            else {
+                console.log(content);
+            }
+            this.messages.push({ role: 'assistant', content });
+            this.saveSession();
+        }
+        catch (error) {
+            if (spinner)
+                spinner.stop();
+            const cliErr = error instanceof api_js_1.CLIError ? error : (0, api_js_1.classifyError)(error);
+            const errorMsg = (0, api_js_1.formatCLIError)(cliErr);
+            if (this.jsonOutput) {
+                process.exitCode = 1;
+                console.log(JSON.stringify({
+                    success: false,
+                    mode: 'operator',
+                    content: '',
+                    error: errorMsg,
+                    errorCategory: cliErr.category,
+                }, null, 2));
+            }
+            else {
+                this.logger.error('Operator direct answer failed');
+                this.logger.error(errorMsg);
+            }
+        }
+    }
     async runSimplePrompt(prompt) {
         this.lastActionableUserInput = prompt;
         // For direct --prompt mode with simple prompts, use a minimal system
@@ -1661,6 +1720,8 @@ class ChatCommand {
             'When diagnosing failures, prefer evidence-backed conclusions over broad repo scans.',
             'If the user corrects you, treat that as evidence that your previous assumption was wrong and re-check with tools.',
             'Do not stop after a partial answer when more direct inspection is possible.',
+            'EVIDENCE-GROUNDING RULE: Every identifier, key name, variable, or symbol you cite in your final answer MUST appear verbatim in tool output you received during this session. Never invent or guess identifiers. If you read a file and saw "KeyA" and "KeyS" on specific lines, cite those exact strings — do not substitute "KeyR" or any other key that was not in the output.',
+            'CROSS-FILE RULE: When comparing two or more files, only claim a key/symbol is conflicting or shared if it literally appears in the relevant function/handler of EACH file you read. Finding a key in File A does not mean it also exists in File B — verify each file independently. Use grep to confirm a key exists in a file before claiming that file handles it.',
             'When you need a tool, emit only one or more tool wrappers in this exact format:',
             '<tool_call>',
             '{"tool":"read_file","args":{"path":"relative/path.ext"}}',
@@ -1844,6 +1905,8 @@ class ChatCommand {
             'Do not declare success until the exact user question has been answered with tool-backed evidence.',
             'If a user is asking which file is correct or most recent, keep inspecting until you can justify the answer from actual results.',
             diagnosticMode ? 'Because this is a debugging task, prefer logs, runtime evidence, and exact symbol references over generic fixes.' : 'Keep working from concrete tool results.',
+            'GROUNDING CHECK: Before writing your final answer, verify that every identifier, key name, or symbol you mention actually appeared in tool output above. Do not invent identifiers that were not in the evidence. If you saw "KeyA" and "KeyS" in the file contents, use those exact names — never substitute a different key. For cross-file comparisons, confirm each claimed conflict key appears in the handler/function of BOTH files — not just one.',
+            'VERIFICATION PROTOCOL for cross-file comparisons: If your answer claims a key/identifier appears in multiple files, you MUST first use grep to search for that exact identifier in each file BEFORE including it in your answer. Only include keys/identifiers that grep confirms exist in each file. Example: to verify "KeyW" is in InputManager.js, use grep for "KeyW" in that file. If grep finds no match, do NOT claim that file handles "KeyW".',
             '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.',

package/dist/index.js

@@ -284,7 +284,7 @@ async function main() {
         .command('operator')
         .alias('op')
         .description('Start BMAD operator mode for infrastructure and system workflows')
-        .option('-m, --model <model>', 'Select operator model (code-8b, code, agent)')
+        .option('-m, --model <model>', 'Select operator model (code, agent, cloud)')
         .option('-p, --project <path>', 'Set project context path')
         .option('--new-project [name]', 'Create or use a managed local workspace folder when no --project path is given')
         .option('--prompt <text>', 'Run a single operator prompt directly and exit')

package/dist/utils/api.js

@@ -2958,7 +2958,7 @@ document.addEventListener('DOMContentLoaded', () => {
                             workspace: { path: workspacePath },
                             workspace_path: workspacePath,
                             workspace_summary: workspaceSummary,
-                            model: this.resolveModelId(executionContext.model || 'code-8b'),
+                            model: this.resolveModelId(executionContext.model || 'code'),
                             history: executionContext.history || [],
                             executionSurface: executionContext.executionSurface || 'cli',
                             clientSurface: executionContext.clientSurface || 'cli',
@@ -2969,7 +2969,7 @@ document.addEventListener('DOMContentLoaded', () => {
                             rawPrompt: executionContext.rawPrompt || null,
                             requestStartedAt: executionContext.requestStartedAt,
                         },
-                        workflow_type: executionContext.workflowType || 'analysis_only',
+                        workflow_type: executionContext.workflowType || 'full',
                         options: {
                             stream: true,
                             save_to_vigflow: executionContext.savePlanToVigFlow === true,

package/package.json

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