omnius 1.0.270 → 1.0.271

package/dist/index.js

@@ -546546,7 +546546,10 @@ var init_agent_tool = __esm({
           lines.push(prompt);
           return lines.join("\n");
         })();
-        if (isolation === "worktree") {
+        const fullDefault = process.env["OMNIUS_FULL_SUBAGENT_DEFAULT"] !== "0";
+        const workCapableType = subagentType === "general" || subagentType === "coordinator";
+        const routeToSubprocess = isolation === "worktree" || fullDefault && runInBackground && workCapableType;
+        if (routeToSubprocess) {
           this.callbacks.onViewRegister?.(agentId, label);
           const spawn35 = this.callbacks.spawnSubprocess({
             id: agentId,
@@ -555351,6 +555354,31 @@ var init_textSanitize = __esm({
   }
 });
 
+// packages/orchestrator/dist/permissionRuleset.js
+function checkDoomLoop(context2) {
+  const history = context2.toolCallHistory;
+  if (!history || history.length < 3)
+    return null;
+  const lastCall = history[history.length - 1];
+  const recent = history.slice(-4);
+  let identicalCount = 1;
+  for (let i2 = history.length - 2; i2 >= Math.max(0, history.length - 4); i2--) {
+    const prev = history[i2];
+    if (prev.tool === lastCall.tool && JSON.stringify(prev.args) === JSON.stringify(lastCall.args)) {
+      identicalCount++;
+    }
+  }
+  if (identicalCount >= 3) {
+    return "ask";
+  }
+  return null;
+}
+var init_permissionRuleset = __esm({
+  "packages/orchestrator/dist/permissionRuleset.js"() {
+    "use strict";
+  }
+});
+
 // packages/orchestrator/dist/completionContract.js
 function normalizeText(value2, max = 500) {
   const text2 = String(value2 ?? "").trim().replace(/\s+/g, " ");
@@ -565337,6 +565365,7 @@ var init_agenticRunner = __esm({
   "packages/orchestrator/dist/agenticRunner.js"() {
     "use strict";
     init_textSanitize();
+    init_permissionRuleset();
     init_completionLedger();
     init_dist6();
     init_ollama_pool();
@@ -567832,10 +567861,12 @@ ${body}`;
               }
               const prior = seen.get(fp);
               if (prior) {
+                const priorContent = typeof messages2[prior.idx]?.content === "string" ? messages2[prior.idx].content : "";
+                const priorFailed = ERROR_MARKERS.test(priorContent);
                 pending2.push({
                   idx: prior.idx,
                   reason: "dedupe",
-                  replacement: `${DEDUPE_PREFIX} ${scanTurn} — duplicate ${name10}() call]`
+                  replacement: priorFailed ? `${DEDUPE_PREFIX} ${scanTurn} — duplicate ${name10}() call; the earlier call FAILED (same error as the retained copy below). Do not retry identically.]` : `${DEDUPE_PREFIX} ${scanTurn} — duplicate ${name10}() call]`
                 });
                 seen.set(fp, { turn: scanTurn, idx: resultIdx });
                 continue;
@@ -567847,10 +567878,11 @@ ${body}`;
                 if (priorResource && priorResource.idx !== resultIdx) {
                   const priorContent = typeof messages2[priorResource.idx]?.content === "string" ? messages2[priorResource.idx].content : "";
                   if (!priorContent.startsWith(PRUNE_PREFIX) && !priorContent.startsWith(DEDUPE_PREFIX) && !priorContent.startsWith(FILE_AGED_PREFIX)) {
+                    const priorResFailed = ERROR_MARKERS.test(priorContent);
                     pending2.push({
                       idx: priorResource.idx,
                       reason: "dedupe",
-                      replacement: `${DEDUPE_PREFIX} ${scanTurn} — semantic duplicate ${name10}() (same resource: ${rkey})]`
+                      replacement: priorResFailed ? `${DEDUPE_PREFIX} ${scanTurn} — semantic duplicate ${name10}() (same resource: ${rkey}); the earlier call FAILED. Do not retry identically.]` : `${DEDUPE_PREFIX} ${scanTurn} — semantic duplicate ${name10}() (same resource: ${rkey})]`
                     });
                   }
                 }
@@ -572790,6 +572822,44 @@ Corrective action: try a different approach first: read relevant files, adjust a
                 timestampMs: Date.now()
               });
               const _toolLogTailIdx = toolCallLog.length - 1;
+              if (tc.name !== "task_complete") {
+                const doomHistory = toolCallLog.slice(-5).map((e2) => ({ tool: e2.name, args: e2.argsKey }));
+                doomHistory.push({
+                  tool: tc.name,
+                  args: this._buildExactArgsKey(tc.arguments ?? {})
+                });
+                const doom = checkDoomLoop({
+                  permission: tc.name,
+                  target: "",
+                  toolCallHistory: doomHistory
+                });
+                if (doom === "ask" && cohort && cohort.failure >= 3 && cohort.success === 0) {
+                  const stopMsg = `[blocked: doom-loop] '${tc.name}' has been called with identical arguments 3+ times and has failed every time. This exact call was NOT executed. Stop repeating it. Choose ONE: (a) change the arguments or approach, (b) read/inspect to understand why it fails, or (c) if genuinely blocked, call task_complete and state the blocker plainly.`;
+                  this.emit({
+                    type: "tool_call",
+                    toolName: tc.name,
+                    toolArgs: tc.arguments,
+                    turn,
+                    timestamp: (/* @__PURE__ */ new Date()).toISOString()
+                  });
+                  const lastLog = toolCallLog[_toolLogTailIdx];
+                  if (lastLog) {
+                    lastLog.success = false;
+                    lastLog.mutated = false;
+                    lastLog.mutatedFiles = [];
+                    lastLog.outputPreview = stopMsg.slice(0, 100);
+                  }
+                  this.emit({
+                    type: "tool_result",
+                    toolName: tc.name,
+                    success: false,
+                    content: stopMsg.slice(0, 200),
+                    turn,
+                    timestamp: (/* @__PURE__ */ new Date()).toISOString()
+                  });
+                  return { tc, output: stopMsg, success: false };
+                }
+              }
               if (editFeedbackRequiredBeforeMoreEdits && this._isProjectEditTool(tc.name)) {
                 this.emit({
                   type: "tool_call",
@@ -573971,8 +574041,23 @@ Respond with EXACTLY this structure before your next tool call:
                 }
               }
               const isFileMutation = realFileMutation;
-              if (isFileMutation && dedupHitCount.size > 0) {
-                dedupHitCount.clear();
+              if (isFileMutation) {
+                if (realMutationPaths.length > 0) {
+                  for (const mp of realMutationPaths) {
+                    if (!mp)
+                      continue;
+                    for (const key of Array.from(dedupHitCount.keys())) {
+                      if (key.includes(mp))
+                        dedupHitCount.delete(key);
+                    }
+                    for (const key of Array.from(recentToolResults.keys())) {
+                      if (key.includes(mp))
+                        recentToolResults.delete(key);
+                    }
+                  }
+                } else if (dedupHitCount.size > 0) {
+                  dedupHitCount.clear();
+                }
               }
               if (isFileMutation && recentToolResults.size > 0) {
                 for (const key of Array.from(recentToolResults.keys())) {
@@ -602335,7 +602420,7 @@ var init_call_agent = __esm({
       }
       /** Initialize the runner with appropriate tools for the access tier */
       async init() {
-        const backend = new OllamaAgenticBackend(this.config.backendUrl, this.config.model, this.config.apiKey);
+        const backend = new OllamaAgenticBackend(this.config.backendUrl, this.config.model, this.config.apiKey, false);
         this.backend = backend;
         const feed = getActivityFeed();
         const systemPrompt = this.buildSystemPrompt();
@@ -705646,7 +705731,13 @@ Review its full output via sub_agent(action='output', id='${id}')`
             backend = new OllamaAgenticBackend(
               config.backendUrl,
               opts.model,
-              config.apiKey
+              config.apiKey,
+              // Sub-agents: force thinking OFF explicitly. Thinking models
+              // interleave <think> blocks that corrupt tool-call parsing for
+              // Qwen/Ollama. Relying on the constructor default is fragile
+              // (breaks if OMNIUS_ENABLE_THINKING=1 is inherited); pass false
+              // so sub-agent tool loops are always clean.
+              false
             );
           }
           const subTier = getModelTier(opts.model);
@@ -705683,6 +705774,10 @@ Review its full output via sub_agent(action='output', id='${id}')`
             `Sub-agent ${subAgentId}`,
             "sub"
           );
+          subRunner.onEvent((event) => {
+            const line = formatSubAgentEventForView(event);
+            if (line) statusBar.writeToAgentView(subAgentId, line);
+          });
           subRunner.registerTool(
             createTaskCompleteTool(subTier, repoRoot, true)
           );
@@ -705725,11 +705820,21 @@ ${result.summary}`
           };
         },
         spawnSubprocess: (opts) => {
-          const entry = spawnFullSubAgent(opts.task, {
-            model: opts.model,
-            backendUrl: config.backendUrl,
-            workingDir: opts.workingDir
-          });
+          const entry = spawnFullSubAgent(
+            opts.task,
+            {
+              model: opts.model,
+              backendUrl: config.backendUrl,
+              workingDir: opts.workingDir
+            },
+            // Stream the full sub-agent's stdout into its toolbar view so the
+            // clickable tab shows live output instead of staying empty.
+            (text2) => statusBar.writeToAgentView(opts.id, text2),
+            (id, exitCode) => statusBar.updateAgentViewStatus(
+              opts.id,
+              exitCode === 0 ? "completed" : "failed"
+            )
+          );
           return { id: entry.id, pid: entry.pid ?? 0 };
         },
         trackTask: (id, promise) => {

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "omnius",
-  "version": "1.0.270",
+  "version": "1.0.271",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "omnius",
-      "version": "1.0.270",
+      "version": "1.0.271",
       "bundleDependencies": [
         "image-to-ascii"
       ],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "omnius",
-  "version": "1.0.270",
+  "version": "1.0.271",
   "description": "AI coding agent powered by open-source models (Ollama/vLLM) — interactive TUI with agentic tool-calling loop",
   "type": "module",
   "main": "./dist/index.js",