@protolabsai/proto 0.55.1 → 0.55.2

package/cli.js

@@ -168684,7 +168684,7 @@ __export(geminiContentGenerator_exports, {
   createGeminiContentGenerator: () => createGeminiContentGenerator
 });
 function createGeminiContentGenerator(config2, gcConfig) {
-  const version2 = "0.55.1";
+  const version2 = "0.55.2";
   const userAgent2 = config2.userAgent || `QwenCode/${version2} (${process.platform}; ${process.arch})`;
   const baseHeaders = {
     "User-Agent": userAgent2
@@ -175339,10 +175339,10 @@ function parseRule(raw2) {
   const normalized2 = trimmed2.replace(/:(\*)/, " $1");
   const openParen = normalized2.indexOf("(");
   if (openParen === -1) {
-    const canonicalName2 = resolveToolName(normalized2);
+    const canonicalName3 = resolveToolName(normalized2);
     return {
       raw: trimmed2,
-      toolName: canonicalName2
+      toolName: canonicalName3
     };
   }
   const toolPart = normalized2.substring(0, openParen).trim();
@@ -175350,11 +175350,11 @@ function parseRule(raw2) {
     return { raw: trimmed2, toolName: resolveToolName(toolPart), invalid: true };
   }
   const specifier = normalized2.substring(openParen + 1, normalized2.length - 1);
-  const canonicalName = resolveToolName(toolPart);
-  const specifierKind = specifier ? getSpecifierKind(canonicalName) : void 0;
+  const canonicalName2 = resolveToolName(toolPart);
+  const specifierKind = specifier ? getSpecifierKind(canonicalName2) : void 0;
   return {
     raw: trimmed2,
-    toolName: canonicalName,
+    toolName: canonicalName2,
     specifier,
     specifierKind
   };
@@ -175371,9 +175371,9 @@ function getRuleDisplayName(canonicalToolName) {
   return CANONICAL_TO_RULE_DISPLAY[canonicalToolName] ?? canonicalToolName;
 }
 function buildPermissionRules(ctx) {
-  const canonicalName = resolveToolName(ctx.toolName);
-  const displayName = getRuleDisplayName(canonicalName);
-  const kind2 = getSpecifierKind(canonicalName);
+  const canonicalName2 = resolveToolName(ctx.toolName);
+  const displayName = getRuleDisplayName(canonicalName2);
+  const kind2 = getSpecifierKind(canonicalName2);
   switch (kind2) {
     case "command":
       if (ctx.command) {
@@ -175382,7 +175382,7 @@ function buildPermissionRules(ctx) {
       return [displayName];
     case "path":
       if (ctx.filePath) {
-        const dirPath = FILE_TARGETED_TOOLS.has(canonicalName) ? path21.dirname(ctx.filePath) : ctx.filePath;
+        const dirPath = FILE_TARGETED_TOOLS.has(canonicalName2) ? path21.dirname(ctx.filePath) : ctx.filePath;
         const specifier = dirPath.startsWith("/") ? `/${dirPath}/**` : `${dirPath}/**`;
         return [`${displayName}(${specifier})`];
       }
@@ -175425,8 +175425,8 @@ function buildHumanReadableRuleLabel(rules) {
     const displayName = rule.substring(0, parenIdx);
     const specifier = rule.substring(parenIdx + 1, rule.length - 1);
     const verb = DISPLAY_NAME_TO_VERB[displayName] ?? displayName.toLowerCase();
-    const canonicalName = Object.entries(CANONICAL_TO_RULE_DISPLAY).find(([, v2]) => v2 === displayName)?.[0];
-    const kind2 = canonicalName ? getSpecifierKind(canonicalName) : "literal";
+    const canonicalName2 = Object.entries(CANONICAL_TO_RULE_DISPLAY).find(([, v2]) => v2 === displayName)?.[0];
+    const kind2 = canonicalName2 ? getSpecifierKind(canonicalName2) : "literal";
     switch (kind2) {
       case "path": {
         const cleanPath = cleanPathSpecifier(specifier);
@@ -176637,14 +176637,14 @@ var init_tool_utils = __esm({
         set2.add(normalizeIdentifier(alias));
       }, "addAlias");
       for (const key of toolNameKeys) {
-        const canonicalName = ToolNames[key];
+        const canonicalName2 = ToolNames[key];
         const displayName = ToolDisplayNames[key];
         const aliases = /* @__PURE__ */ new Set();
-        addAlias(aliases, canonicalName);
+        addAlias(aliases, canonicalName2);
         addAlias(aliases, displayName);
         addAlias(aliases, `${displayName}Tool`);
         for (const [legacyName, mappedName] of Object.entries(ToolNamesMigration)) {
-          if (mappedName === canonicalName) {
+          if (mappedName === canonicalName2) {
             addAlias(aliases, legacyName);
           }
         }
@@ -176653,7 +176653,7 @@ var init_tool_utils = __esm({
             addAlias(aliases, legacyDisplay);
           }
         }
-        map3.set(canonicalName, aliases);
+        map3.set(canonicalName2, aliases);
       }
       return map3;
     })();
@@ -197087,12 +197087,13 @@ var init_completion_checker = __esm({
 });
 
 // packages/core/dist/src/goal/types.js
-var MAX_GOAL_CONDITION_LENGTH, GOAL_CLEAR_ALIASES;
+var MAX_GOAL_CONDITION_LENGTH, GOAL_STALL_LIMIT, GOAL_CLEAR_ALIASES;
 var init_types10 = __esm({
   "packages/core/dist/src/goal/types.js"() {
     "use strict";
     init_esbuild_shims();
     MAX_GOAL_CONDITION_LENGTH = 4e3;
+    GOAL_STALL_LIMIT = 3;
     GOAL_CLEAR_ALIASES = [
       "clear",
       "stop",
@@ -197108,6 +197109,17 @@ var init_types10 = __esm({
 function truncate2(s5, max) {
   return s5.length <= max ? s5 : `${s5.slice(0, max - 1)}\u2026`;
 }
+function normalizeReason(reason) {
+  const collapsed = reason.trim().toLowerCase().replace(/\s+/g, " ");
+  let end = collapsed.length;
+  while (end > 0 && isTrailingPunctuation(collapsed[end - 1])) {
+    end -= 1;
+  }
+  return collapsed.slice(0, end);
+}
+function isTrailingPunctuation(ch) {
+  return ch === "." || ch === "!" || ch === "?";
+}
 var debugLogger39, GoalManager;
 var init_GoalManager = __esm({
   "packages/core/dist/src/goal/GoalManager.js"() {
@@ -197123,6 +197135,10 @@ var init_GoalManager = __esm({
       active;
       lastAchieved;
       lastFailed;
+      /** Normalized text of the last not-met reason, for stall detection. */
+      lastReasonNormalized;
+      /** Consecutive not-met evaluations repeating `lastReasonNormalized`. */
+      repeatedReasonCount = 0;
       /**
        * Set a new goal. Replaces any active goal. Returns the created state.
        * Throws if the condition is empty or exceeds the length limit.
@@ -197144,6 +197160,7 @@ var init_GoalManager = __esm({
           turnCount: 0,
           tokensSpent: 0
         };
+        this.resetStall();
         return { ...this.active };
       }
       /**
@@ -197155,6 +197172,7 @@ var init_GoalManager = __esm({
           return void 0;
         const cleared = { ...this.active };
         this.active = void 0;
+        this.resetStall();
         debugLogger39.info(`Cleared goal "${truncate2(cleared.condition, 60)}" after ${cleared.turnCount} turns.`);
         return cleared;
       }
@@ -197171,6 +197189,7 @@ var init_GoalManager = __esm({
         };
         this.lastAchieved = achieved;
         this.active = void 0;
+        this.resetStall();
         debugLogger39.info(`Goal achieved after ${achieved.turnCount} turns: "${truncate2(achieved.condition, 60)}".`);
         return achieved;
       }
@@ -197189,6 +197208,7 @@ var init_GoalManager = __esm({
         };
         this.lastFailed = failed;
         this.active = void 0;
+        this.resetStall();
         debugLogger39.info(`Goal abandoned as impossible after ${failed.turnCount} turns: "${truncate2(failed.condition, 60)}" (${truncate2(reason, 80)}).`);
         return failed;
       }
@@ -197198,15 +197218,30 @@ var init_GoalManager = __esm({
           this.active = { ...this.active, turnCount: this.active.turnCount + 1 };
         }
       }
-      /** Record the result of an evaluation against the active goal. */
+      /**
+       * Record the result of an evaluation against the active goal. Returns the
+       * number of consecutive not-met evaluations that have repeated the same
+       * reason -- the stall signal a caller compares against {@link GOAL_STALL_LIMIT}.
+       * A converging goal sees the reason change as different pieces complete; a
+       * stuck goal repeats one complaint while the agent churns. Returns 0 when the
+       * goal is met or no goal is active.
+       */
       recordEvaluation(result) {
         if (!this.active)
-          return;
+          return 0;
         this.active = {
           ...this.active,
           lastReason: result.reason,
           tokensSpent: this.active.tokensSpent + result.tokensUsed
         };
+        if (result.met) {
+          this.resetStall();
+          return 0;
+        }
+        const norm = normalizeReason(result.reason);
+        this.repeatedReasonCount = norm === this.lastReasonNormalized ? this.repeatedReasonCount + 1 : 1;
+        this.lastReasonNormalized = norm;
+        return this.repeatedReasonCount;
       }
       hasActiveGoal() {
         return this.active !== void 0;
@@ -197225,9 +197260,17 @@ var init_GoalManager = __esm({
         this.active = void 0;
         this.lastAchieved = void 0;
         this.lastFailed = void 0;
+        this.resetStall();
+      }
+      /** Clear stall-detection bookkeeping. */
+      resetStall() {
+        this.lastReasonNormalized = void 0;
+        this.repeatedReasonCount = 0;
       }
     };
     __name(truncate2, "truncate");
+    __name(normalizeReason, "normalizeReason");
+    __name(isTrailingPunctuation, "isTrailingPunctuation");
   }
 });
 
@@ -197276,7 +197319,7 @@ function buildEvaluatorPrompt(context2) {
     "Condition:",
     context2.condition,
     "",
-    "Recent tool calls:",
+    "Tool activity so far (repeats collapsed; actions are state-changing, inspections are read-only):",
     context2.toolCallSummary || "(none)",
     "",
     "Final assistant message:",
@@ -197371,9 +197414,9 @@ Respond ONLY with valid JSON matching one of these shapes:
 {"met": false, "impossible": true, "reason": "one short sentence on why it can never be satisfied"}
 
 Rules:
-- Set "met": true only when the condition is clearly demonstrated by the visible evidence (a test output, an exit code, a file count, an empty queue, etc.).
-- If the assistant only claims the work is done without showing evidence, set "met": false and ask for the missing evidence in "reason".
-- If the condition cannot be verified from what is shown, set "met": false and explain what would prove it in "reason".
+- Set "met": true when the visible evidence shows the condition is satisfied. Evidence can be a command result (a test output, an exit code, a count, an empty queue) OR the requested artifact itself (files created or edited, content written, a result documented, output shown in the transcript). The existence of the deliverable the condition asked for IS evidence -- do not demand a test or an exit code for a goal that does not call for one.
+- Judge the condition exactly as written. If it asks to create, write, explore, or document something and the transcript shows that artifact exists, set "met": true. Do not invent acceptance criteria stricter than what the user stated.
+- Set "met": false only when the condition is genuinely unsatisfied, or when the assistant asserts success with nothing in the transcript backing it. When false, name the specific missing piece in "reason".
 - Only set "impossible": true when the condition is genuinely unachievable in this session: it is self-contradictory, depends on an unavailable resource or capability, or the assistant has exhausted reasonable approaches and the transcript confirms there is no path forward. The assistant claiming the goal is impossible is evidence, not proof -- independently confirm it rather than deferring to the assistant's self-assessment. Do NOT set it just because progress is slow or evidence is currently missing. When in doubt, return {"met": false} without "impossible".
 - "impossible" is only meaningful alongside "met": false. Never set it when "met" is true.
 - Keep "reason" to one short sentence. It will be shown to the assistant as guidance for the next turn.`;
@@ -197386,6 +197429,139 @@ Rules:
   }
 });
 
+// packages/core/dist/src/goal/toolCallSummary.js
+function summarizeToolCallsForGoal(records) {
+  if (records.length === 0)
+    return "";
+  const byKey = /* @__PURE__ */ new Map();
+  records.forEach((record2, index) => {
+    const name4 = canonicalName(record2.name);
+    const target = extractTarget(record2.input);
+    const key = `${name4} ${target}`;
+    const existing = byKey.get(key);
+    if (existing) {
+      existing.count += 1;
+      existing.lastIndex = index;
+      if (record2.success)
+        existing.okCount += 1;
+      else
+        existing.failCount += 1;
+    } else {
+      byKey.set(key, {
+        name: name4,
+        target,
+        readOnly: READ_ONLY_TOOLS.has(name4),
+        count: 1,
+        okCount: record2.success ? 1 : 0,
+        failCount: record2.success ? 0 : 1,
+        lastIndex: index
+      });
+    }
+  });
+  const distinct = [...byKey.values()].sort((a2, b2) => a2.lastIndex - b2.lastIndex);
+  const actions = distinct.filter((c4) => !c4.readOnly);
+  const reads = distinct.filter((c4) => c4.readOnly);
+  const sections = [];
+  const actionSection = renderSection("Actions taken (most recent last)", actions, MAX_ACTION_LINES);
+  if (actionSection)
+    sections.push(actionSection);
+  const readSection = renderSection("Inspections / reads (most recent last)", reads, MAX_READ_LINES);
+  if (readSection)
+    sections.push(readSection);
+  return sections.join("\n");
+}
+function canonicalName(name4) {
+  const migrated = ToolNamesMigration[name4];
+  return migrated ?? name4;
+}
+function extractTarget(input) {
+  if (!input)
+    return "";
+  for (const key of TARGET_KEYS) {
+    const value = input[key];
+    if (typeof value === "string" && value.trim().length > 0) {
+      return truncateTarget(value);
+    }
+  }
+  return "";
+}
+function truncateTarget(value) {
+  const collapsed = value.replace(/\s+/g, " ").trim();
+  return collapsed.length <= MAX_TARGET_LEN ? collapsed : `${collapsed.slice(0, MAX_TARGET_LEN - 3)}...`;
+}
+function renderSection(label, entries, limit2) {
+  if (entries.length === 0)
+    return "";
+  const kept = entries.length > limit2 ? entries.slice(-limit2) : entries;
+  const omitted = entries.length - kept.length;
+  const header = omitted > 0 ? `${label}; ${omitted} earlier omitted:` : `${label}:`;
+  const lines = kept.map((c4) => `- ${renderLine(c4)}`);
+  return [header, ...lines].join("\n");
+}
+function renderLine(call) {
+  const target = call.target ? ` ${call.target}` : "";
+  const status = renderStatus(call);
+  const repeat = call.count > 1 ? ` (x${call.count})` : "";
+  return `${call.name}${target} ${status}${repeat}`;
+}
+function renderStatus(call) {
+  if (call.failCount === 0)
+    return "[ok]";
+  if (call.okCount === 0)
+    return "[failed]";
+  return `[${call.okCount} ok, ${call.failCount} failed]`;
+}
+var MAX_ACTION_LINES, MAX_READ_LINES, MAX_TARGET_LEN, READ_ONLY_TOOLS, TARGET_KEYS;
+var init_toolCallSummary = __esm({
+  "packages/core/dist/src/goal/toolCallSummary.js"() {
+    "use strict";
+    init_esbuild_shims();
+    init_tool_names();
+    MAX_ACTION_LINES = 30;
+    MAX_READ_LINES = 12;
+    MAX_TARGET_LEN = 120;
+    READ_ONLY_TOOLS = /* @__PURE__ */ new Set([
+      ToolNames.READ_FILE,
+      ToolNames.GREP,
+      ToolNames.GLOB,
+      ToolNames.LS,
+      ToolNames.WEB_FETCH,
+      ToolNames.WEB_SEARCH,
+      ToolNames.LSP,
+      ToolNames.REPO_MAP,
+      ToolNames.TASK_GET,
+      ToolNames.TASK_LIST,
+      ToolNames.TASK_OUTPUT,
+      ToolNames.TASK_READY,
+      // Real read-only tools without a ToolNames entry:
+      "read_many_files"
+    ]);
+    TARGET_KEYS = [
+      "file_path",
+      "absolute_path",
+      "notebook_path",
+      "path",
+      "command",
+      "pattern",
+      "url",
+      "query",
+      "prompt",
+      "description",
+      "skill",
+      "fact",
+      "title",
+      "task_id"
+    ];
+    __name(summarizeToolCallsForGoal, "summarizeToolCallsForGoal");
+    __name(canonicalName, "canonicalName");
+    __name(extractTarget, "extractTarget");
+    __name(truncateTarget, "truncateTarget");
+    __name(renderSection, "renderSection");
+    __name(renderLine, "renderLine");
+    __name(renderStatus, "renderStatus");
+  }
+});
+
 // packages/core/dist/src/goal/index.js
 var init_goal = __esm({
   "packages/core/dist/src/goal/index.js"() {
@@ -197393,6 +197569,7 @@ var init_goal = __esm({
     init_esbuild_shims();
     init_GoalManager();
     init_goalEvaluator();
+    init_toolCallSummary();
     init_types10();
   }
 });
@@ -197992,11 +198169,7 @@ Please address these issues before finishing.`;
           const active = goalManager.getActiveGoal();
           if (active) {
             goalManager.recordTurn();
-            const toolCallSummary = goalToolCalls.slice(-20).map((t5) => {
-              const status = t5.success ? "ok" : "failed";
-              const cmd = typeof t5.input?.["command"] === "string" ? `: ${t5.input["command"].slice(0, 200)}` : "";
-              return `- ${t5.name} [${status}]${cmd}`;
-            }).join("\n");
+            const toolCallSummary = summarizeToolCallsForGoal(goalToolCalls);
             let evalResult;
             try {
               evalResult = await evaluateGoal(this.getContentGeneratorOrFail(), this.config.getModel(), {
@@ -198008,12 +198181,20 @@ Please address these issues before finishing.`;
               this.config.getDebugLogger().warn(`[goal] evaluator threw for condition "${active.condition.slice(0, 60)}": ${err3 instanceof Error ? err3.message : String(err3)}`);
             }
             if (evalResult) {
-              goalManager.recordEvaluation(evalResult);
+              const repeatedReason = goalManager.recordEvaluation(evalResult);
               if (evalResult.met) {
                 goalManager.markAchieved();
               } else if (evalResult.impossible) {
                 goalManager.markImpossible(evalResult.reason);
                 this.config.getDebugLogger().info(`[goal] abandoned as impossible: ${evalResult.reason.slice(0, 120)}`);
+              } else if (repeatedReason >= GOAL_STALL_LIMIT) {
+                goalManager.clearGoal();
+                const stallReason = `Stopping the /goal loop: the completion check returned the same unmet verdict ${repeatedReason} turns in a row without converging ("${evalResult.reason}"). The goal "${active.condition}" is now cleared. Briefly tell the user where things stand and what they should decide or check next, then stop.`;
+                const stallRequest = [{ text: stallReason }];
+                const stallResult = yield* this.sendMessageStream(stallRequest, signal, prompt_id, { type: SendMessageType.GoalContinuation }, boundedTurns - 1);
+                if (ownsTurnSpan)
+                  endTurnSpan("ok");
+                return stallResult;
               } else {
                 const continueReason = `Goal not yet met. Evaluator: ${evalResult.reason}
 
@@ -250739,13 +250920,13 @@ var init_permission_manager = __esm({
        * deny specific invocations at runtime.
        */
       async isToolEnabled(toolName) {
-        const canonicalName = resolveToolName(toolName);
+        const canonicalName2 = resolveToolName(toolName);
         if (this.coreToolsAllowList !== null && this.coreToolsAllowList.size > 0) {
-          if (!this.coreToolsAllowList.has(canonicalName)) {
+          if (!this.coreToolsAllowList.has(canonicalName2)) {
             return false;
           }
         }
-        const decision = await this.evaluate({ toolName: canonicalName });
+        const decision = await this.evaluate({ toolName: canonicalName2 });
         return decision !== "deny";
       }
       /**
@@ -288360,6 +288541,7 @@ __export(dist_exports, {
   FlashFallbackEvent: () => FlashFallbackEvent,
   ForbiddenError: () => ForbiddenError,
   GOAL_CLEAR_ALIASES: () => GOAL_CLEAR_ALIASES,
+  GOAL_STALL_LIMIT: () => GOAL_STALL_LIMIT,
   GOOGLE_ACCOUNTS_FILENAME: () => GOOGLE_ACCOUNTS_FILENAME,
   GeminiChat: () => GeminiChat,
   GeminiClient: () => GeminiClient,
@@ -288413,9 +288595,11 @@ __export(dist_exports, {
   LspServerManager: () => LspServerManager,
   LspTool: () => LspTool,
   MAINLINE_CODER_MODEL: () => MAINLINE_CODER_MODEL,
+  MAX_ACTION_LINES: () => MAX_ACTION_LINES,
   MAX_ENTRYPOINT_BYTES: () => MAX_ENTRYPOINT_BYTES,
   MAX_ENTRYPOINT_LINES: () => MAX_ENTRYPOINT_LINES,
   MAX_GOAL_CONDITION_LENGTH: () => MAX_GOAL_CONDITION_LENGTH,
+  MAX_READ_LINES: () => MAX_READ_LINES,
   MCPDiscoveryState: () => MCPDiscoveryState,
   MCPOAuthProvider: () => MCPOAuthProvider,
   MCPOAuthTokenStorage: () => MCPOAuthTokenStorage,
@@ -288967,6 +289151,7 @@ __export(dist_exports, {
   stripStartupContext: () => stripStartupContext,
   subagentGenerator: () => subagentGenerator,
   substituteHookVariables: () => substituteHookVariables,
+  summarizeToolCallsForGoal: () => summarizeToolCallsForGoal,
   teamRegistry: () => teamRegistry,
   templateString: () => templateString,
   tildeifyPath: () => tildeifyPath,
@@ -416789,7 +416974,7 @@ __name(getPackageJson, "getPackageJson");
 // packages/cli/src/utils/version.ts
 async function getCliVersion() {
   const pkgJson = await getPackageJson();
-  return "0.55.1";
+  return "0.55.2";
 }
 __name(getCliVersion, "getCliVersion");
 
@@ -424989,7 +425174,7 @@ var formatDuration = /* @__PURE__ */ __name((milliseconds) => {
 
 // packages/cli/src/generated/git-commit.ts
 init_esbuild_shims();
-var GIT_COMMIT_INFO = "a8772314d";
+var GIT_COMMIT_INFO = "c065df84f";
 
 // packages/cli/src/utils/systemInfo.ts
 async function getNpmVersion() {
@@ -475732,7 +475917,7 @@ var BaseTextInput = /* @__PURE__ */ __name(({
   borderColor,
   topRightLabel,
   isActive = true,
-  renderLine = defaultRenderLine
+  renderLine: renderLine2 = defaultRenderLine
 }) => {
   const handleKey = (0, import_react116.useCallback)(
     (key) => {
@@ -475824,7 +476009,7 @@ var BaseTextInput = /* @__PURE__ */ __name(({
           ] }) : /* @__PURE__ */ (0, import_jsx_runtime114.jsx)(Text3, { color: theme.text.secondary, children: placeholder }) : linesToRender.map((lineText, idx) => {
             const absoluteVisualIndex = scrollVisualRow + idx;
             const isOnCursorLine = absoluteVisualIndex === cursorVisualRow;
-            return /* @__PURE__ */ (0, import_jsx_runtime114.jsx)(Box_default, { height: 1, children: renderLine({
+            return /* @__PURE__ */ (0, import_jsx_runtime114.jsx)(Box_default, { height: 1, children: renderLine2({
               lineText,
               isOnCursorLine,
               cursorCol: cursorVisualCol,
@@ -493878,7 +494063,7 @@ var QwenAgent = class {
   async initialize(args2) {
     this.clientCapabilities = args2.clientCapabilities;
     const authMethods = buildAuthMethods();
-    const version2 = "0.55.1";
+    const version2 = "0.55.2";
     return {
       protocolVersion: PROTOCOL_VERSION,
       agentInfo: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@protolabsai/proto",
-  "version": "0.55.1",
+  "version": "0.55.2",
   "description": "proto - AI-powered coding agent",
   "repository": {
     "type": "git",
@@ -21,7 +21,7 @@
     "bundled"
   ],
   "config": {
-    "sandboxImageUri": "ghcr.io/qwenlm/qwen-code:0.55.1"
+    "sandboxImageUri": "ghcr.io/qwenlm/qwen-code:0.55.2"
   },
   "dependencies": {},
   "optionalDependencies": {