zames_pro 2.5.0 → 2.5.1

package/dist/agent-loop.js

@@ -51,6 +51,13 @@ export async function runAgentLoop({ browser, tools, task, workdir, maxIteration
     const MAX_MALFORMED_RETRIES = 3;
     let stallRetries = 0;
     const MAX_STALL_RETRIES = 5;
+    // Защита от «агент встал»: DeepSeek иногда присылает финальный текст,
+    // который лишь ОПИСЫВАЕТ следующий вызов инструмента (или рвёт ответ на
+    // полуслове), и агент молча завершает задачу, хотя работа не сделана.
+    // Если финальный ответ похож на «сейчас вызову …» — переспрашиваем, а не
+    // останавливаемся. Счётчик общий, чтобы не зациклиться на болтливой модели.
+    let looksDoneRetries = 0;
+    const MAX_LOOKSDONE_RETRIES = 3;
     for (let i = 0; i < maxIterations; i++) {
         onThinking();
         // Первое сообщение (task) — пользовательский ввод: без паузы.
@@ -108,10 +115,16 @@ export async function runAgentLoop({ browser, tools, task, workdir, maxIteration
                 continue;
             }
             const trimmed = (rawResponse || '').trim();
+            // Служебный ответ — это КОРОТКАЯ заглушка DeepSeek («Reading…») или
+            // короткое уведомление о лимите. Слова про rate limit в ДЛИННОМ
+            // ответе — это, как правило, сам агент цитирует код/логи (в транскрипте
+            // был ровно такой случай: ответ на 1365 символов про ask() и лимиты),
+            // и принимать его за «служебный» нельзя, иначе агент зря переспрашивает.
             const looksService = !trimmed ||
                 trimmed.length < 2 ||
                 /^(reading|thinking|searching|analyzing|generating|stop|остановить|читаю|думаю|поиск|анализ)[\s.…]*$/i.test(trimmed) ||
-                /(messages? too frequent|too many requests|rate limit|слишком часто|try again later)/i.test(trimmed);
+                (trimmed.length <= 200 &&
+                    /(messages? too frequent|too many requests|rate limit|слишком часто|try again later)/i.test(trimmed));
             if (looksService && stallRetries < MAX_STALL_RETRIES) {
                 stallRetries++;
                 transcript?.log('stall_retry', {
@@ -131,6 +144,32 @@ export async function runAgentLoop({ browser, tools, task, workdir, maxIteration
                         'Если задача выполнена — вызови инструмент respond с итоговым сообщением.';
                 continue;
             }
+            // Ответ похож на «сейчас вызову инструмент», но вызова в нём нет.
+            // DeepSeek иногда так обрывает ход: пишет «Now update README…» или
+            // «Let me run the tests…» и замолкает. Если принять это за финал,
+            // агент встаёт, не сделав работу. Просим продолжить и на этот раз
+            // обязательно вызвать инструмент (или respond, если правда готово).
+            if (looksLikeUnfinishedWork(trimmed) && looksDoneRetries < MAX_LOOKSDONE_RETRIES) {
+                looksDoneRetries++;
+                transcript?.log('unfinished_retry', {
+                    attempt: looksDoneRetries,
+                    response: rawResponse,
+                });
+                if (debugLog) {
+                    console.error('внимание: ответ похож на незавершённую работу, прошу продолжить (попытка ' +
+                        looksDoneRetries +
+                        '/' +
+                        MAX_LOOKSDONE_RETRIES +
+                        ')');
+                }
+                message =
+                    'Похоже, ты собирался вызвать инструмент, но не вызвал. ' +
+                        'Если задача ещё не выполнена — ответь РОВНО одним JSON-объектом ' +
+                        'вызова инструмента, без текста до и после. ' +
+                        'Если задача действительно выполнена — вызови respond с итоговым ' +
+                        'сообщением оператору.';
+                continue;
+            }
             onAssistantMessage(rawResponse);
             transcript?.log('assistant_final', { message: rawResponse });
             return rawResponse;
@@ -206,6 +245,27 @@ export async function runAgentLoop({ browser, tools, task, workdir, maxIteration
     }
     return 'Достигнут лимит итераций.';
 }
+// Текст, который обещает вызов инструмента в будущем времени, но самого
+// вызова не содержит. DeepSeek регулярно так «зависает»: пишет
+// «Now update README to mention …», «Let me run the tests», «Сейчас проверю»
+// и останавливается. Такие ответы нельзя принимать за финальные — иначе
+// агент встаёт, не выполнив работу. Держим эвристику узкой (будущее время /
+// намерение), чтобы не ловить обычные отчёты о выполненной работе.
+function looksLikeUnfinishedWork(text) {
+    const t = (text || '').trim();
+    if (!t)
+        return false;
+    // Длинные ответы (отчёты) не трогаем — там может быть что угодно.
+    if (t.length > 600)
+        return false;
+    // Уже есть финальный маркер — считаем ответ завершённым.
+    if (/\b(done|finished|completed|готово|выполнено|завершено)\b/i.test(t)) {
+        return false;
+    }
+    const en = /\b(now|next|then|let me|let's|i will|i'll|i am going to|i'm going to|going to|about to|will now|time to)\b[^.!?\n]{0,120}\b(update|write|edit|read|run|check|add|fix|create|remove|delete|apply|test|commit|push|install|open|search|look|verify|change|modify|implement|review)\b/i;
+    const ru = /(^|[^а-яё])(проверю|обновлю|исправлю|добавлю|запущу|выполню|посмотрю|прочитаю|изменю|попробую|сделаю)([^а-яё]|$)/i;
+    return en.test(t) || ru.test(t);
+}
 // ============ JSON parsing ============
 function repairRawControlChars(str) {
     let out = '';

package/dist/xml-toolcall.js

@@ -13,11 +13,37 @@ function readAttr(attrs, name) {
     const m = attrs.match(re);
     return m ? m[2] : null;
 }
+// Достаёт параметры из тела <invoke>. Тег открытия и закрытия может нести
+// произвольный префикс (DSML и т.п.), а некоторые модели путают пары тегов
+// (<parameter …> … </|DSML| parameter>). Регулярка ниже матчит любую пару
+// <…parameter…> … </…parameter…>, поэтому такие «смешанные» теги тоже
+// разбираются. Значения дедуплицируются: если модель повторила parameter
+// несколько раз, берём ПЕРВОЕ значение, а не последнее (повторы обычно
+// дублируют корректный параметр).
+function parseParameters(body) {
+    const args = {};
+    const paramRe = /<[^>]*?parameter([^>]*)>([\s\S]*?)<\/[^>]*?parameter[^>]*>/gi;
+    let p;
+    while ((p = paramRe.exec(body)) !== null) {
+        const pname = readAttr(p[1] || '', 'name');
+        if (!pname)
+            continue;
+        const isString = /string[ ]*=[ ]*"?true"?/i.test(p[1] || '');
+        let value = unescapeXml(p[2]);
+        if (!isString) {
+            try {
+                value = JSON.parse(value);
+            }
+            catch { }
+        }
+        if (!(pname in args))
+            args[pname] = value;
+    }
+    return args;
+}
 export function parseXmlToolCalls(text) {
     if (!text || typeof text !== 'string')
         return null;
-    if (!/<[^>]*invoke/i.test(text))
-        return null;
     const invokeRe = /<[^>]*?invoke([^>]*)>/gi;
     const calls = [];
     let m;
@@ -30,23 +56,7 @@ export function parseXmlToolCalls(text) {
         const closeRe = /<\/[^>]*?invoke[^>]*>/i;
         const closeMatch = closeRe.exec(rest);
         const body = closeMatch ? rest.slice(0, closeMatch.index) : rest;
-        const args = {};
-        const paramRe = /<[^>]*?parameter([^>]*)>([\s\S]*?)<\/[^>]*?parameter[^>]*>/gi;
-        let p;
-        while ((p = paramRe.exec(body)) !== null) {
-            const pname = readAttr(p[1] || '', 'name');
-            if (!pname)
-                continue;
-            const isString = /string[ ]*=[ ]*"?true"?/i.test(p[1] || '');
-            let value = unescapeXml(p[2]);
-            if (!isString) {
-                try {
-                    value = JSON.parse(value);
-                }
-                catch { }
-            }
-            args[pname] = value;
-        }
+        const args = parseParameters(body);
         // Некоторые модели кладут весь JSON-объект аргументов в один
         // parameter с именем args. Разворачиваем, чтобы не было
         // {args: {args: {...}}}.
@@ -61,7 +71,12 @@ export function parseXmlToolCalls(text) {
         }
         calls.push({ tool: name, args: finalArgs });
     }
-    if (!calls.length)
-        return null;
-    return calls.length === 1 ? calls[0] : calls;
+    if (calls.length)
+        return calls.length === 1 ? calls[0] : calls;
+    // Модель иногда опускает тег <invoke> и оставляет только DSML-обёртку
+    // (<|DSML| calls> … <|DSML| parameter name=…>…). Имя инструмента в таком
+    // ответе отсутствует, поэтому восстановить вызов нельзя — но и молча
+    // принимать его за финальный ответ нельзя: agent-loop попросит повторить
+    // (looksLikeToolCall срабатывает по слову parameter/DSML).
+    return null;
 }

package/package.json

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