zames_pro 2.4.7 → 2.5.1

package/README.md

@@ -9,14 +9,21 @@ directory, reads and edits files, runs commands, and commits to git.
 ## Requirements
 
 - Node.js >= 18
-- Google Chrome or Chromium (used through Playwright)
 - A DeepSeek account (you log in manually in the browser window that opens on first launch)
 
 ## Installation
 
 ```bash
 npm install -g zames_pro
+```
+
+Chromium for Playwright is downloaded automatically on install. On Linux/WSL
+the required system libraries are installed too when passwordless `sudo` is
+available; otherwise run once by hand:
+
+```bash
 npx playwright install chromium
+sudo npx playwright install-deps chromium
 ```
 
 ## Usage

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) — пользовательский ввод: без паузы.
@@ -79,7 +86,14 @@ export async function runAgentLoop({ browser, tools, task, workdir, maxIteration
             }
         }
         if (!parsed) {
-            const looksLikeToolCall = /("tool"\s*:|\btool_calls?\b|\binvoke\b|\bparameter\b|DSML|function_call)/i.test(rawResponse) || /\{\s*"?(tool|name)"?\s*:/.test(rawResponse);
+            // Ответ похож на (возможно, обрезанный) вызов инструмента. Ловим не
+            // только явный JSON, но и XML/DSML-формы, «грязные» варианты и
+            // незакрытые фрагменты: если такой ответ молча принять за финальный,
+            // агент встанет, хотя модель пыталась позвать инструмент.
+            const looksLikeToolCall = /("tool"\s*:|\btool_calls?\b|\binvoke\b|\bparameter\b|DSML|function_call)/i.test(rawResponse) ||
+                /\{\s*"?(tool|name|args)"?\s*:/.test(rawResponse) ||
+                /<\s*\|?\s*(DSML|invoke|parameter)/i.test(rawResponse) ||
+                /^\s*\[?\s*\{[^}]*$/.test(rawResponse.trim());
             if (looksLikeToolCall && malformedRetries < MAX_MALFORMED_RETRIES) {
                 malformedRetries++;
                 transcript?.log('malformed_toolcall', {
@@ -101,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', {
@@ -124,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;
@@ -134,6 +180,25 @@ export async function runAgentLoop({ browser, tools, task, workdir, maxIteration
             const msg = typeof respondCall.args.message === 'string'
                 ? respondCall.args.message
                 : String(respondCall.args.message ?? '');
+            // Пустой respond — не финал: модель позвала respond, но не написала
+            // итог. Если так завершить, оператор не увидит ничего, а задача
+            // «зависнет». Просим продолжить (в пределах stallRetries).
+            if (!msg.trim() && stallRetries < MAX_STALL_RETRIES) {
+                stallRetries++;
+                transcript?.log('empty_respond', { attempt: stallRetries });
+                if (debugLog) {
+                    console.error('внимание: пустой respond, прошу продолжить (попытка ' +
+                        stallRetries +
+                        '/' +
+                        MAX_STALL_RETRIES +
+                        ')');
+                }
+                message =
+                    'Ты вызвал respond с пустым message. Если задача выполнена — ' +
+                        'вызови respond с итоговым сообщением оператору. Если нет — ' +
+                        'продолжи работу вызовом инструмента.';
+                continue;
+            }
             onAssistantMessage(msg);
             transcript?.log('assistant_final', { message: msg });
             return msg;
@@ -180,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/tools.js

@@ -115,7 +115,7 @@ export function createTools(workdir, { undo } = {}) {
                     await undo.backup(file);
                 await fs.mkdir(path.dirname(file), { recursive: true });
                 await fs.writeFile(file, text, 'utf-8');
-                return `Файл записан: {p}`;
+                return `Файл записан: ${p}`;
             },
         },
         {

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.4.7",
+  "version": "2.5.1",
   "description": "Terminal coding agent over chat.deepseek.com via Playwright",
   "type": "module",
   "bin": {
@@ -8,6 +8,7 @@
   },
   "files": [
     "dist",
+    "scripts",
     "README.md",
     "LICENSE",
     "logo.png"
@@ -20,6 +21,7 @@
     "prepublishOnly": "npm run build",
     "test": "tsx --test test/*.test.ts",
     "test:watch": "tsx --test --watch test/*.test.ts",
+    "postinstall": "node scripts/postinstall.mjs",
     "prepare": "husky"
   },
   "keywords": [

package/scripts/postinstall.mjs

@@ -0,0 +1,67 @@
+// Automatically install the Chromium build and system dependencies
+// that Playwright needs.
+//
+// Runs when the zames package is installed so the user does not have to run
+// "npx playwright install chromium" / "install-deps chromium" by hand.
+//
+// On Linux (including WSL) the browser binary also needs system libraries.
+// They are installed via "install-deps", which requires root (sudo).
+// If we are not root and sudo is not passwordless, we skip it and tell the
+// user the exact command to run.
+//
+// Install failure is NOT fatal: without network access (or in CI) the
+// package install must still succeed.
+
+import { spawnSync } from 'node:child_process'
+
+const isWin = process.platform === 'win32'
+
+// npm puts node_modules/.bin on PATH for lifecycle scripts, so the local
+// playwright CLI is resolvable. On Windows npm runs scripts through cmd.exe,
+// so shell: true is needed for the .cmd shim to be found.
+function run(cmd, args) {
+  return spawnSync(cmd, args, {
+    stdio: 'inherit',
+    shell: isWin,
+  })
+}
+
+function canSudo() {
+  // Already root? No sudo needed.
+  if (typeof process.getuid === 'function' && process.getuid() === 0) return true
+  // Passwordless sudo (common in WSL / CI images).
+  const r = run('sudo', ['-n', 'true'])
+  return !r.error && r.status === 0
+}
+
+// 1) Browser binary for this Playwright version.
+const browser = run('playwright', ['install', 'chromium'])
+if (browser.error || browser.status !== 0) {
+  console.warn(
+    '\nzames: could not install Chromium for Playwright automatically.\n' +
+      '   Run it manually: npx playwright install chromium\n',
+  )
+  process.exit(0)
+}
+
+// 2) System libraries (Linux / WSL only). Without them Chromium fails
+// to launch with "error while loading shared libraries".
+if (!isWin) {
+  if (canSudo()) {
+    const deps = run('playwright', ['install-deps', 'chromium'])
+    if (deps.error || deps.status !== 0) {
+      console.warn(
+        '\nzames: could not install Chromium system dependencies automatically.\n' +
+          '   Run it manually: sudo npx playwright install-deps chromium\n',
+      )
+    }
+  } else {
+    console.warn(
+      '\nzames: on Linux/WSL Chromium needs system libraries (sudo required).\n' +
+        '   Run it once manually: sudo npx playwright install-deps chromium\n',
+    )
+  }
+}
+
+// Never fail the package install because of a browser download.
+process.exit(0)