zames_pro 2.10.2 → 2.10.4

package/dist/agent-loop.js

@@ -2,6 +2,7 @@ import { buildSystemPrompt } from './system-prompt.js';
 import { getGitContext, formatGitContext } from './gitTools.js';
 import { parseXmlToolCalls } from './xml-toolcall.js';
 import { translate } from './i18n.js';
+import { normText } from './browser.js';
 export async function runAgentLoop({ browser, tools, task, workdir, maxIterations = 40, freshChat = false, sendSystemPrompt = false, transcript = null, attachments = [], onThinking = () => { }, onAssistantThought = () => { }, onToolCall = () => { }, onToolResult = () => { }, onAssistantMessage = () => { }, onChatReady = () => { }, onWarning = () => { }, debugLog = false, locale = 'ru', }) {
     // UI callbacks must NEVER break the agent loop. A rendering error (a huge
     // tool result, a broken markdown frame, a closed terminal) used to throw
@@ -189,14 +190,22 @@ export async function runAgentLoop({ browser, tools, task, workdir, maxIteration
         // cut-off "Stale. Let me ..." or a truncated JSON). All three mean the
         // turn is unfinished: nudge instead of stopping.
         const wdEmpty = !String(rawResponse || '').trim();
+        // STALE is compared on NORMALIZED text: DeepSeek often echoes the
+        // previous answer with different markdown emphasis (`**done**` vs `done`),
+        // which defeated an exact match and made the loop run the SAME tool again —
+        // the "stopped after a tool call" signature with a duplicate call.
         const wdStale = justRanTool &&
             lastRaw.trim() !== '' &&
-            rawResponse.trim() === lastRaw.trim();
+            (rawResponse.trim() === lastRaw.trim() ||
+                normForStale(rawResponse) === normForStale(lastRaw));
         const wdNoCall = justRanTool &&
             !wdEmpty &&
             !wdStale &&
             parseToolCall(rawResponse) === null &&
             !responseLooksLikeToolCall(rawResponse);
+        // A stale answer is discarded even when it parses to a valid call: it is a
+        // duplicate of the previous turn, and re-running the tool would repeat
+        // side effects and stall the loop.
         if (!isFirst && (wdEmpty || wdStale || wdNoCall) && watchdogRetries < MAX_WATCHDOG_RETRIES) {
             watchdogRetries++;
             transcript?.log('watchdog_nudge', {
@@ -522,6 +531,16 @@ function isMeaningfulRespond(msg) {
         return false;
     return true;
 }
+// Normalize an answer for STALE comparison: collapse whitespace AND strip
+// markdown emphasis/code markers. DeepSeek echoes the previous turn with a
+// different emphasis (`**done**` vs `done`), which defeated an exact match and
+// made the loop re-run the same tool ("stopped after a tool call").
+function normForStale(s) {
+    return normText(s)
+        .replace(/[*_`#>]+/g, '')
+        .replace(/[ \t]+/g, ' ')
+        .trim();
+}
 // Text that promises a tool call in the future tense but contains no call
 // itself. DeepSeek regularly "hangs" like this: it writes
 // "Now update README to mention …", "Let me run the tests", "I'll check now"

package/dist/browser.js

@@ -646,15 +646,23 @@ export class DeepSeekBrowser {
             .evaluate(() => document.body.innerText.length)
             .catch(() => 0);
         let started = false;
+        // A stale/echo answer (the model repeats the previous text, or the answer
+        // legitimately equals it) does NOT change `cur`. In that case the old loop
+        // either threw "did not start" after 15s or hung until the full timeout —
+        // the operator saw the agent "stop after a tool call". We now also accept
+        // the answer when the generation has clearly SETTLED: no Stop button and
+        // the text has been stable for a couple of ticks.
+        let settledTicks = 0;
+        let lastStartCur = '';
         while (Date.now() < startDeadline) {
             if (this._abort)
                 return '(прервано пользователем)';
             const pageText = await this._readPageText();
             if (isRateLimitText(pageText)) {
                 throw new RateLimitError(pageText.slice(0, 300));
-                if (isServerBusyText(pageText)) {
-                    throw new ServerBusyError(pageText.slice(0, 300));
-                }
+            }
+            if (isServerBusyText(pageText)) {
+                throw new ServerBusyError(pageText.slice(0, 300));
             }
             const cur = await this._readLastAnswerTextClean().catch(() => '');
             const bodyLen = await this.page
@@ -678,9 +686,30 @@ export class DeepSeekBrowser {
                 started = true;
                 break;
             }
+            // Fallback for an echo/stale answer: the send happened (lastSentAt was
+            // just updated), the Stop button is gone and the text stopped changing.
+            // Two stable ticks in a row mean the turn is over even if it equals the
+            // previous text — return it instead of hanging/throwing.
+            const notGenerating = !(await this._isGenerating());
+            if (cur && cur === lastStartCur && notGenerating && cur.trim()) {
+                settledTicks++;
+                if (settledTicks >= 2) {
+                    return cur;
+                }
+            }
+            else {
+                settledTicks = 0;
+            }
+            lastStartCur = cur;
             await this.page.waitForTimeout(300);
         }
         if (!started) {
+            // Last chance: the answer may have arrived and settled exactly at the
+            // deadline. Return the current text instead of a hard error.
+            const cur = await this._readLastAnswerTextClean().catch(() => '');
+            if (cur && cur.trim() && !(await this._isGenerating())) {
+                return cur;
+            }
             // We NO LONGER check the limit over the whole page text — that caused
             // false positives and 5-minute waits. We just report that
             // generation did not start.
@@ -718,21 +747,32 @@ export class DeepSeekBrowser {
             // belongs to the CURRENT send even if the DOM still shows the old text.
             const isNew = !!cur &&
                 (netFresh || normText(cur) !== normText(beforeText));
-            if (isNew && cur === last) {
+            // An echo/stale answer equals beforeText, so isNew stays false and the
+            // old loop waited until the full timeout — the "agent stopped after a
+            // tool call" hang. If generation has clearly ENDED (no Stop button) and
+            // the text is stable, accept it (even when it repeats the previous one).
+            const sameAsBefore = !!cur && !isNew && normText(cur) === normText(beforeText);
+            if ((isNew || sameAsBefore) && cur === last) {
                 stable++;
-                if (stable >= 2)
-                    return cur;
+                if (stable >= 2) {
+                    if (isNew || !(await this._isGenerating()))
+                        return cur;
+                }
             }
             else {
                 stable = 0;
             }
-            if (isNew)
+            if (isNew || sameAsBefore)
                 last = cur;
             await this.page.waitForTimeout(800);
         }
         if (last && (normText(last) !== normText(beforeText) || this._netCapture)) {
             return last;
         }
+        // Fallback: the turn settled on a text identical to the previous answer.
+        if (last && !(await this._isGenerating())) {
+            return last;
+        }
         if (this._netCapture && this._netCaptureAt >= this._lastSentAt) {
             return this._netCapture;
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zames_pro",
-  "version": "2.10.2",
+  "version": "2.10.4",
   "description": "Terminal coding agent over chat.deepseek.com via Playwright",
   "type": "module",
   "bin": {