omnius 1.0.224 → 1.0.226

package/dist/index.js

@@ -560040,6 +560040,15 @@ var init_agenticRunner = __esm({
       _fileWritesThisRun = 0;
       _backwardPassCyclesUsed = 0;
       _lastBackwardPassVerdict = null;
+      // REG-54: completion-gate anti-collapse circuit breaker. The no-progress and
+      // completion-provenance gates can hold task_complete indefinitely. When a
+      // collapsed/weak model resubmits the same (or a similar) completion summary,
+      // the gate rejects it every turn and the model re-emits the identical
+      // task_complete forever — a mode-collapse loop. Unlike the backward-pass
+      // critic (REG-47), these gates had no cycle bound. We track consecutive holds
+      // of the same summary and fail open after a small threshold so the run
+      // terminates instead of spinning. Reset whenever a completion is allowed.
+      _completionHoldState = { count: 0, lastKey: "" };
       // ── WO-AM-01/04/10: Associative memory stores ──
       // Episode store: every tool call → persistent episode with importance + decay
       // Temporal KG: entities + relations with temporal validity (valid_from/valid_until)
@@ -564982,6 +564991,52 @@ TASK: ${scrubbedTask}` : scrubbedTask;
           });
           return true;
         };
+        const completionHoldEscapeMax = (() => {
+          const raw = Number(process.env["OMNIUS_COMPLETION_HOLD_MAX"]);
+          return Number.isFinite(raw) && raw >= 1 ? Math.floor(raw) : 3;
+        })();
+        const holdTaskCompleteGates = (args, turn) => {
+          const held = holdNoProgressTaskComplete(args, turn) || holdProvenanceTaskComplete(args, turn);
+          if (!held) {
+            this._completionHoldState.count = 0;
+            this._completionHoldState.lastKey = "";
+            return false;
+          }
+          const key = (extractTaskCompleteSummary(args) || "").trim().slice(0, 200);
+          if (key && key === this._completionHoldState.lastKey) {
+            this._completionHoldState.count += 1;
+          } else {
+            this._completionHoldState.lastKey = key;
+            this._completionHoldState.count = 1;
+          }
+          if (this._completionHoldState.count >= completionHoldEscapeMax) {
+            messages2.push({
+              role: "system",
+              content: `[COMPLETION GATE ESCAPE] task_complete was held ${this._completionHoldState.count} times for the same summary without new progress. The gate feedback above is not being acted on, so completion is now ALLOWED to avoid a stall/mode-collapse loop. If the summary lacks evidence, that limitation stands in the final result.`
+            });
+            this.emit({
+              type: "status",
+              content: `completion gates yielding after ${this._completionHoldState.count} repeated holds to prevent a mode-collapse loop`,
+              turn,
+              timestamp: (/* @__PURE__ */ new Date()).toISOString()
+            });
+            this.emit({
+              type: "adversary_reaction",
+              adversary: {
+                class: "guidance",
+                shortText: "Completion gate escape (anti-loop)",
+                confidence: 0.9,
+                details: `Same task_complete summary held ${this._completionHoldState.count}× — failing open to terminate the run.`
+              },
+              turn,
+              timestamp: (/* @__PURE__ */ new Date()).toISOString()
+            });
+            this._completionHoldState.count = 0;
+            this._completionHoldState.lastKey = "";
+            return false;
+          }
+          return true;
+        };
         const emitBackwardPassAdvisory = (feedback, turn) => {
           messages2.push({
             role: "system",
@@ -568543,7 +568598,7 @@ ${sr.result.output}`;
                         timestamp: (/* @__PURE__ */ new Date()).toISOString()
                       });
                     } else {
-                      if (holdNoProgressTaskComplete(matchTc.arguments, turn) || holdProvenanceTaskComplete(matchTc.arguments, turn)) {
+                      if (holdTaskCompleteGates(matchTc.arguments, turn)) {
                         continue;
                       }
                       const _bp1 = await this._runBackwardPassReview(turn);
@@ -568598,7 +568653,7 @@ ${sr.result.output}`;
                           timestamp: (/* @__PURE__ */ new Date()).toISOString()
                         });
                       } else {
-                        if (holdNoProgressTaskComplete(r2.tc.arguments, turn) || holdProvenanceTaskComplete(r2.tc.arguments, turn)) {
+                        if (holdTaskCompleteGates(r2.tc.arguments, turn)) {
                           continue;
                         }
                         const _bp2 = await this._runBackwardPassReview(turn);
@@ -568690,7 +568745,7 @@ ${sr.result.output}`;
                           timestamp: (/* @__PURE__ */ new Date()).toISOString()
                         });
                       } else {
-                        if (holdNoProgressTaskComplete(r2.tc.arguments, turn) || holdProvenanceTaskComplete(r2.tc.arguments, turn)) {
+                        if (holdTaskCompleteGates(r2.tc.arguments, turn)) {
                           continue;
                         }
                         const _bp3 = await this._runBackwardPassReview(turn);
@@ -568911,7 +568966,7 @@ Call task_complete(summary="...") NOW with whatever you have.`
             const adversaryAddedGuidance = this.pendingUserMessages.length > pendingBeforeAdversary;
             if (/task.?complete|all tests pass/i.test(content)) {
               const completionArgs = { summary: content };
-              if (holdNoProgressTaskComplete(completionArgs, turn) || holdProvenanceTaskComplete(completionArgs, turn)) {
+              if (holdTaskCompleteGates(completionArgs, turn)) {
                 continue;
               }
               completed = true;
@@ -569482,7 +569537,7 @@ Full content available via: repl_exec(code="data = retrieve('${handleId}')")  or
                       timestamp: (/* @__PURE__ */ new Date()).toISOString()
                     });
                   } else {
-                    if (holdNoProgressTaskComplete(tc.arguments, turn) || holdProvenanceTaskComplete(tc.arguments, turn)) {
+                    if (holdTaskCompleteGates(tc.arguments, turn)) {
                       continue;
                     }
                     const _bp4 = await this._runBackwardPassReview(turn);
@@ -569535,7 +569590,7 @@ Full content available via: repl_exec(code="data = retrieve('${handleId}')")  or
                   });
                 } else {
                   const completionArgs = { summary: content };
-                  if (holdNoProgressTaskComplete(completionArgs, turn) || holdProvenanceTaskComplete(completionArgs, turn)) {
+                  if (holdTaskCompleteGates(completionArgs, turn)) {
                     continue;
                   }
                   completed = true;
@@ -637758,13 +637813,15 @@ var init_emotion_engine = __esm({
           const prompt = [
             "You are the agent's private reflection process deciding whether to contact the Telegram admin.",
             "",
+            "CRITICAL ROLE BOUNDARY: You are NOT executing the task below and you must not try to. You have exactly two tools — telegram_admin_message and task_complete — and that is correct and complete for your role. You do not have, and do not need, file/shell/edit/code tools. Never report that you are 'blocked', that you 'lack tools', or ask for tool access — that is never your job and the main agent (which has the real tools) handles the work. The task is shown only as CONTEXT so you can judge whether the admin would want a message right now.",
+            "",
             "If no message is useful, call task_complete with NO_MESSAGE.",
             "If a message is useful, call telegram_admin_message with the exact text to send, then call task_complete.",
-            "Send a message when the agent is blocked, needs a preference/secret/clarification, is about to make a risky choice, or has a high-value status update the admin should know now.",
-            "Do not send routine task-complete, success-streak, mood, or cheerleading updates.",
-            "If you message, write it naturally in first person, under 700 characters. Ask a direct question only when user input would change the next action.",
+            "Send a message when the MAIN AGENT is genuinely blocked, needs a preference/secret/clarification from the admin, is about to make a risky choice, or has a high-value status update the admin should know now.",
+            "Do not send routine task-complete, success-streak, mood, or cheerleading updates. Do not send messages about your own (the reflection process's) tools or capabilities.",
+            "If you message, write it naturally in first person as the agent, under 700 characters. Ask a direct question only when admin input would change the next action.",
             "",
-            `Current task: ${this.currentTask || "(none recorded)"}`,
+            `Task the MAIN AGENT is working on (context only — NOT for you to perform): ${this.currentTask || "(none recorded)"}`,
             `Last event: ${event.type}${event.toolName ? `:${event.toolName}` : ""}${event.success === false ? " failed" : event.success === true ? " succeeded" : ""}`,
             `Event content: ${String(event.content ?? "").slice(0, 500)}`,
             `Recent activity: ${this.describeRecentActivity() || "(none)"}`,
@@ -637783,8 +637840,20 @@ var init_emotion_engine = __esm({
       cleanReflectionMessage(raw) {
         const text = raw.replace(/^["'`]+|["'`]+$/g, "").trim();
         if (!text || /^NO_MESSAGE\.?$/i.test(text)) return "";
+        if (this.isReflectionToolComplaint(text)) return "";
         return text.slice(0, 900);
       }
+      /**
+       * Detect reflection-process meta-complaints about its own (intentionally
+       * minimal) tool surface. These are mode-collapse artifacts, not genuine
+       * admin outreach, and must be suppressed.
+       */
+      isReflectionToolComplaint(text) {
+        const t2 = text.toLowerCase();
+        const mentionsTools = /\btask_complete\b/.test(t2) || /\btelegram_admin_message\b/.test(t2) || /\b(file_read|file_edit|file_write|shell|file_patch|batch_edit)\b/.test(t2) || /\btool ?set\b/.test(t2) || /\btools?\b/.test(t2);
+        if (!mentionsTools) return false;
+        return /\b(don'?t|do not|doesn'?t|did not|didn'?t|cannot|can'?t|unable to|no longer)\b[^.]*\b(have|access|use)\b/.test(t2) || /\b(provide|grant|give|need|require|request)\b[^.]*\b(access|tools?|tool ?set)\b/.test(t2) || /\bi (only )?have (only )?(access to )?(task_complete|telegram_admin_message)/.test(t2) || /\bblocked\b[^.]*\b(tools?|access)\b/.test(t2) || /\bonly (have|with) (reflection|admin|task_complete|telegram)/.test(t2);
+      }
       /** Summarize recent tool activity into a brief phrase */
       describeRecentActivity() {
         if (this.recentTools.length === 0) return "";

package/npm-shrinkwrap.json

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

package/package.json

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