zames_pro 2.82.137 → 2.82.138

package/CHANGELOG.md

@@ -7,6 +7,76 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [2.82.138] - 2026-10-25
+
+### Changed
+
+- docs(types): fix a stale provider count
+- docs(tui/widgets): prune a section divider
+- docs(tui): prune section dividers
+- docs(system-prompt): drop a backlog-id reference
+- docs(statusline): prune and fix stale comments
+- docs(status-command): prune and fix stale comments
+- docs(spinner): prune and fix stale comments
+- docs(shell): prune and fix stale comments
+- docs(sessions): prune and fix stale comments
+- docs(send-throttle): prune and fix stale comments
+- docs(send-telemetry): prune and fix stale comments
+- docs(send-runtime): prune and fix stale comments
+- docs(self-review): prune section dividers
+- docs(run-task): prune and fix stale comments
+- docs(qwen-ui): prune and fix stale comments
+- docs(qwen-net-capture): prune and fix stale comments
+- docs(qwen-chats): prune and fix stale comments
+- docs(qwen-browser): prune section dividers
+- docs(prompt): prune and fix stale comments
+- docs(permissions): prune and fix stale comments
+- docs(path-token): prune and fix stale comments
+- docs(net-capture): prune and fix stale comments
+- docs(mcp): prune and fix stale comments
+- docs(markdown): prune and fix stale comments
+- docs(limits-command): prune and fix stale comments
+- docs(input/suggest): prune and fix stale comments
+- docs(input/layout): drop a backlog-id reference
+- docs(input): prune backlog ids and section dividers
+- docs(index): prune section dividers
+- docs(i18n): prune and fix stale comments
+- docs(i18n): drop backlog-id references from common.ts comments
+- docs(goal): prune and fix stale comments
+- docs(glm-ui): prune and fix stale comments
+- docs(glm-net-capture): prune and fix stale comments
+- docs(glm-chats): prune and fix stale comments
+- docs(glm-browser): prune and fix stale comments
+- docs(glm-browser): prune and fix stale comments
+- docs(gitTools): prune and fix stale comments
+- docs(fsutil): prune and fix stale comments
+- docs(extraTools): prune and fix stale comments
+- docs(diff): prune and fix stale comments
+- docs(diagnostics): prune and fix stale comments
+- docs(deepseek-ui): prune and fix stale comments
+- docs(config): prune and fix stale comments
+- docs(config-menu): prune and fix stale comments
+- docs(config-live): prune and fix stale comments
+- docs(config-commands): prune and fix stale comments
+- docs(config-command): prune and fix stale comments
+- docs(commands): prune and fix stale comments
+- docs(code-blocks): prune and fix stale comments
+- docs(cli-args): prune and fix stale comments
+- docs(checkpoint): prune and fix stale comments
+- docs(changelog): prune and fix stale comments
+- docs(browser): prune and fix stale comments
+- docs(browser-pure): prune and fix stale comments
+- docs(browser-login): prune and fix stale comments
+- docs(browser-dom): prune and fix stale comments
+- docs(browser-chats): prune and fix stale comments
+- docs(banner): prune and fix stale comments
+- docs(backlog): prune and fix stale comments
+- docs(attachments): prune and fix stale comments
+- docs(attach-refs): prune and fix stale comments
+- docs(attach-flow): prune and fix stale comments
+- docs(answer-parse): prune and fix stale comments
+- docs(agent-loop): prune and fix stale comments
+
 ## [2.82.137] - 2026-10-24
 
 ### Changed

package/dist/agent-loop.js

@@ -233,14 +233,10 @@ export async function runAgentLoop({ browser, tools, task, workdir, maxIteration
     // call but was not recognized (e.g. a DSML/XML form the strict parser
     // missed), the tool did NOT run and the echo is the only copy of the
     // call — it must be executed, not discarded as stale.
-    // const ranToolLastTurn follows the previous turn's actual effect.
     let lastTurnRanTool = false;
     let justRanTool = false;
     let watchdogRetries = 0;
     const MAX_WATCHDOG_RETRIES = 3;
-    // Retry budget for a browser.ask() TIMEOUT (not for content): the send did
-    // not come back in time. This is separate from unparsedRetries because a
-    // timeout is an infrastructure failure, not a model protocol violation.
     // Loop detection: the model sometimes repeats the SAME tool call with the
     // SAME arguments (e.g. Read the same file) turn after turn. There was no
     // explicit detector for that — the stale-answer guard only catches a
@@ -263,6 +259,9 @@ export async function runAgentLoop({ browser, tools, task, workdir, maxIteration
     // Set when the argument-error series hits the cap; appended to the message
     // AFTER the tool results are formatted (they rebuild `message`).
     let argErrNudge = '';
+    // Retry budget for a browser.ask() TIMEOUT (not for content): the send did
+    // not come back in time. Separate from unparsedRetries because a timeout is
+    // an infrastructure failure, not a model protocol violation.
     let afterToolRetries = 0;
     // toolsRanInTask: how many tools actually ran in THIS task. The protocol
     // guard uses it to tell "started, then slipped into chat mode" (a reasoning
@@ -1065,8 +1064,6 @@ export async function runAgentLoop({ browser, tools, task, workdir, maxIteration
                 const poll = setInterval(syncToolAbort, 100);
                 if (typeof poll.unref === 'function')
                     poll.unref();
-                // Time the tool: the transcript carries durationMs so /cost can
-                // show where the time goes (frequent Read→Edit cycles vs slow Bash).
                 const toolStart = Date.now();
                 try {
                     result = await tool.fn(call.args, { signal: toolAbort.signal });

package/dist/answer-parse.js

@@ -1,12 +1,9 @@
 import { parseXmlToolCalls } from './xml-toolcall.js';
 import { normText } from './browser-pure.js';
-// Pure, state-free helpers extracted from agent-loop.ts : the
-// tool-call PARSER (JSON / permissive / XML-DSML repair) and the small answer
-// heuristics. None of them touch the browser or the loop state, so this
-// module is safe to import on its own. agent-loop.ts imports the ones the
-// loop needs and RE-EXPORTS the public ones, so the existing API (tests import
-// parseToolCall / responseLooksLikeToolCall / truncateToolResult from './agent-loop.js')
-// is unchanged.
+// Pure, state-free helpers split out of agent-loop.ts: the tool-call PARSER
+// (JSON / permissive / XML-DSML repair) and the small answer heuristics. None
+// of them touch the browser or the loop state, so the module is safe to import
+// on its own; tests and agent-loop.ts import the functions from here directly.
 // Cap a tool result for the model, but make the truncation EXPLICIT: a bare
 // slice() silently hid the tail and the model had no idea it was looking at a
 // partial output (it would act on a half-read file/log).
@@ -537,11 +534,9 @@ function parseToolCallPermissive(text) {
     const tool = toolMatch[1];
     // The model occasionally puts the argument keys INLINE with "tool" —
     // {"tool": "Bash", "command": "..."} — with no "args" wrapper at all.
-    // The strict parser rejects it (no obj.args), and the permissive parser
-    // used to bail out too (indexOf('"args"') === -1). Such a call was
-    // reported as malformed and re-asked up to MAX_MALFORMED_RETRIES times;
-    // after the budget ran out the run stopped with the model's text as the
-    // final answer. Flatten the inline keys into args when "args" is absent.
+    // The strict parser rejects such a call (no obj.args); flatten the inline
+    // keys into args instead of letting the loop re-ask until it gives up and
+    // stops with the model's text as the final answer.
     if (text.indexOf('"args"') === -1) {
         const braceIdx = text.indexOf('{');
         if (braceIdx === -1)
@@ -730,16 +725,15 @@ function repairToolCallPreamble(text) {
     const m = t.match(/(?:^|[^A-Za-z0-9_])(?:\*\*)?(tool)(?:\*\*)?["'`\u2018\u2019\u201c\u201d]*\s*:/);
     if (!m || m.index === undefined)
         return null;
-    // We look for the start from the first quote/bracket around the key, otherwise from the word tool.
     let start = m.index;
     const brace = t.indexOf('{', Math.max(0, start - 1));
     if (brace !== -1 && brace < start)
         start = brace;
     let frag = t.slice(start);
-    // If the fragment does not start with `{` — we add it.
+    // The head lost its opening `{`, so rebuild one around the key below.
     if (!frag.startsWith('{')) {
-        // The key may have lost its opening quote: tool": → "tool".
-        // We trim the leading junk up to the word tool and normalize the key quotes.
+        // The key lost its opening quote (`tool":`), so drop the junk before it
+        // and re-add the quote.
         frag = frag.replace(/^[^A-Za-z0-9_]*/, '');
         frag = frag.replace(/^(?:\*\*)?(["'`\u2018\u2019\u201c\u201d]*)(tool)(?:\*\*)?["'`\u2018\u2019\u201c\u201d]*\s*:/, '"$2":');
         frag = '{' + frag;

package/dist/attach-refs.js

@@ -2,7 +2,7 @@ import fs from 'fs/promises';
 import path from 'path';
 import { looksLikeFilePath, isImageName } from './attachments.js';
 import { extractAtFileRefs } from './path-token.js';
-// Path resolution and `@file` reference inlining .
+// Path resolution and `@file` reference inlining.
 // Extracted from src/index.ts so the logic is unit-testable without the whole
 // CLI. This module touches only the filesystem (no browser/terminal).
 const NL = String.fromCharCode(10);
@@ -81,12 +81,12 @@ async function isInside(workdir, target) {
     const rel = path.relative(root, real);
     return !rel.startsWith('..') && !path.isAbsolute(rel);
 }
-// Per-file and total caps for `@path` inlining . A reference should SAVE
+// Per-file and total caps for `@path` inlining. A reference should SAVE
 // round-trips, not flood the context: a huge file is truncated with a note
 // telling the agent to read the rest itself.
 export const AT_REF_FILE_CAP = 60 * 1024;
 export const AT_REF_TOTAL_CAP = 200 * 1024;
-// Inline `@path` references found in a task text . Returns the augmented
+// Inline `@path` references found in a task text. Returns the augmented
 // text plus the list of files actually inlined (for the operator notice).
 // Unknown/unreadable references are left in place untouched -- they may be
 // part of the prose; we never fail the task over a missing file.

package/dist/attachments.js

@@ -284,13 +284,8 @@ export function hasClipboardTool() {
     }
     return false;
 }
-// Read an image from the OS clipboard. Tries, in order, every known tool for
-// the current platform. Returns the raw bytes plus a short note about which
-// tool was used (or the reason nothing was found) — the caller shows that note
-// to the user so a failed paste is never silent.
-// True when running inside WSL. There the X11 clipboard is a SEPARATE
-// clipboard from the Windows one: the user copies an image in Windows, so we
-// must read it through powershell.exe, not xclip.
+// True when the Windows filesystem is visible from here, i.e. a
+// powershell.exe call can reach the Windows clipboard.
 export function windowsReachable() {
     if (process.platform === 'win32')
         return true;
@@ -328,6 +323,11 @@ const PS_GET_FILE_DROP = 'Add-Type -AssemblyName System.Windows.Forms; ' +
     '$d=[System.Windows.Forms.Clipboard]::GetDataObject(); ' +
     'if($d -and $d.GetDataPresent([System.Windows.Forms.DataFormats]::FileDrop)){' +
     '$f=$d.GetData([System.Windows.Forms.DataFormats]::FileDrop); $f -join [Environment]::NewLine}';
+// Read an image from the OS clipboard, trying every known tool for the current
+// platform. The returned `via` names the tool that worked, or why nothing was
+// found — the caller shows it to the operator so a failed paste is never
+// silent. On WSL the Windows clipboard comes FIRST: it is a DIFFERENT clipboard
+// from X11, and the user copies the image in Windows, not in the X session.
 export async function readClipboardImageDetailed() {
     if (process.platform === 'linux' ||
         process.platform === 'freebsd' ||
@@ -410,7 +410,6 @@ export async function readClipboardImageDetailed() {
     }
     return { data: null, via: 'unsupported-platform' };
 }
-// Backwards-compatible wrapper used elsewhere.
 export async function readClipboardImage() {
     return (await readClipboardImageDetailed()).data;
 }

package/dist/backlog.js

@@ -39,11 +39,8 @@ export function parseBacklogNote(note) {
             break;
         }
     }
-    // A note typed on Windows (or pasted from one) carries CRLF line endings;
-    // the title would keep the trailing `\r` and the heading would get a stray
-    // carriage return.
-    // Normalize BOTH line endings first: a lone `\r` (a classic-Mac paste) is
-    // a line break too, and `body.split(NL)` kept it inside the title.
+    // Normalize CRLF AND a lone `\r` (a classic-Mac paste is a line break too):
+    // left in place, the `\r` stayed in the title and reached the heading.
     const bodyLines = body.split(/\r\n?|\n/).map((l) => l.replace(/\r$/, ''));
     const title = bodyLines[0];
     const rest = bodyLines.slice(1).join(NL).trim();

package/dist/banner.js

@@ -13,7 +13,7 @@ function width(s) {
  * widest line (clamped to `maxWidth`), padded by one space each side. Lines
  * longer than the inner width are truncated with an ellipsis.
  */
-function box(lines, maxWidth, border = '#c8b06a') {
+function box(lines, maxWidth) {
     // A caller may pass 0 ("auto") or a tiny width on a narrow terminal. With
     // `maxWidth - 4` going negative the inner width collapsed to the 10-column
     // floor while every content line was clipped to `inner - 2`, so the title,
@@ -25,22 +25,21 @@ function box(lines, maxWidth, border = '#c8b06a') {
     const top = theme.prompt('╭' + '─'.repeat(inner) + '╮');
     const bottom = theme.prompt('╰' + '─'.repeat(inner) + '╯');
     const body = lines.map((l) => {
-        // Truncate to the inner width so a long path never breaks the frame.
         const maxContent = Math.max(0, inner - 2);
         const clipped = clip(l, maxContent);
         const w = width(clipped);
         const pad = Math.max(0, inner - 1 - w);
         return (theme.prompt('│') + ' ' + clipped + ' '.repeat(pad) + theme.prompt('│'));
     });
-    void border;
     return [top, ...body, bottom].join('\n');
 }
 /** Truncate an (ANSI-carrying) string to `max` visible columns. */
 function clip(s, max) {
     if (width(s) <= max)
         return s;
-    // The banner content has at most a couple of ANSI runs; drop the tail by
-    // visible width while keeping escape sequences intact.
+    // The content carries a couple of ANSI runs, so the tail is dropped by
+    // VISIBLE width and escape sequences are copied whole — a plain slice would
+    // cut an escape in half and leak its remainder as text.
     let out = '';
     let visible = 0;
     let i = 0;

package/dist/browser-chats.js

@@ -1,4 +1,4 @@
-// PURE parsing of the DeepSeek history/message payloads . These
+// PURE parsing of the DeepSeek history/message payloads. These
 // helpers are extracted from DeepSeekBrowser.fetchChatMessages: they used to be
 // copy-pasted INLINE twice (the in-page evaluate AND the Playwright-request
 // fallback), so the same role/fragment extraction lived in two places and could
@@ -69,9 +69,9 @@ export function parseHistoryMessages(messages) {
     return { list, usage };
 }
 /**
- * Pull `chat_messages` out of a history_messages JSON response, tolerating the
- * two shapes the endpoint has shipped (`data.biz_data.chat_messages`); returns
- * null when the shape does not match, so the caller can report a clear error.
+ * Pull `chat_messages` out of a history_messages JSON response; returns null
+ * when the shape does not match, so the caller can report a clear error
+ * instead of silently restoring an empty dialogue.
  */
 export function extractChatMessages(json) {
     const j = json;

package/dist/browser-dom.js

@@ -1,4 +1,4 @@
-// Pure in-page DOM helpers extracted from DeepSeekBrowser .
+// Pure in-page DOM helpers extracted from DeepSeekBrowser.
 //
 // Every function here runs INSIDE page.evaluate, so it MUST be fully
 // self-contained: Playwright serializes the function SOURCE and evaluates it in
@@ -79,9 +79,10 @@ export function readMfaAccountInDom() {
     return '';
 }
 /**
- * The answer as RENDERED on the page, ALWAYS from the DOM (never the network
- * capture). Reasoning blocks (think class) are skipped and must never be
- * picked up as the answer.
+ * The answer as RENDERED on the page. The freshness checks in `_askOnce` MUST
+ * compare rendered text, not the network capture, so they can see that the DOM
+ * actually changed. Reasoning blocks (think class) are skipped and must never
+ * be picked up as the answer.
  */
 export function readLastAnswerInDom(opts) {
     const sels = opts.sels;
@@ -159,10 +160,9 @@ export function continueVisibleCheapInDom(nameRe) {
     return false;
 }
 /**
- * Find the Stop button in the DeepSeek UI. We can't rely on the class alone:
- * during generation the send button (the same circle button) changes its icon
- * to a "square" (stop) while keeping the classes, so a square icon (svg rect)
- * also means Stop.
+ * Find the Stop button. The class alone is not enough: during generation the
+ * send button (the same circle button) swaps its icon to a "square" while
+ * keeping its classes, so a square icon (svg rect) also means Stop.
  */
 export function stopButtonVisibleInDom(btnRe) {
     const re = new RegExp(btnRe, 'i');
@@ -186,9 +186,10 @@ export function stopButtonVisibleInDom(btnRe) {
     return false;
 }
 /**
- * Raw-DOM Stop click with the full pointer/mouse sequence (older builds
- * without a proper role, or a click intercepted by an overlay). Returns true
- * when a square-icon button was found and clicked.
+ * Raw-DOM Stop click with the full pointer/mouse sequence. The label check is
+ * skipped here on purpose — the square-icon button may carry no label at all,
+ * and a plain .click() was swallowed by an overlay. Returns true when a
+ * square-icon button was found and clicked.
  */
 export function clickStopInDom(btnRe) {
     const re = new RegExp(btnRe, 'i');
@@ -310,9 +311,8 @@ export function readElementText(el) {
     return el.innerText || el.textContent || '';
 }
 /**
- * Lowercased tagName of an element. Runs bound to the element (Playwright's
- * Locator.evaluate), used to tell a native input/textarea from a
- * contenteditable box.
+ * Lowercased tagName of an element, used to tell a native input/textarea from
+ * a contenteditable box. Runs bound to the element.
  */
 export function elementTagNameInDom(el) {
     return el.tagName.toLowerCase();
@@ -387,7 +387,8 @@ export function submitLoginFormInDom(opts) {
             return true;
         }
     }
-    // Fallback: the primary-looking button in the scope.
+    // No labelled submit control matched, so a React-rendered form is the likely
+    // case: click the primary-looking button instead.
     const primary = btns.find((b) => {
         const cls = (b.className || '').toString();
         return /primary|submit|ds-button--primary/i.test(cls);

package/dist/browser-login.js

@@ -16,7 +16,9 @@
 //   2. autoLogin() with saved creds     -> done (this is the FIRST step after
 //                                          the probe; credentials are permanent)
 //   3. guest is allowed and no login was requested -> work as a guest
-//   4. not interactive                  -> LoginRequiredError (fast, no window)
+//   4. not interactive                  -> LoginRequiredError (fast, no window,
+//                                          so a cold login fails instead of
+//                                          hanging)
 //   5. no display                       -> fall back to the in-TUI forms
 //   6. headless                         -> relaunchSafe(headed), sign in
 //   7. wait until probeLogin() === 'in' (poll), the operator closes the window
@@ -57,8 +59,9 @@ export async function interactiveBrowserLogin(deps) {
     // 4) Non-interactive: fail fast instead of hanging on a window nobody sees.
     if (!deps.interactive)
         throw new LoginRequiredError(deps.provider);
-    // 5) No display: the in-TUI form is the accepted fallback (DESIGN-NOTES
-    //    "Login stays inside the TUI").
+    // 5) No display: the in-TUI form is the accepted fallback — a captcha cannot
+    //    be typed in a terminal, but a password/code form still can. Without one
+    //    there is nothing to offer, so fail like a non-interactive run.
     if (!deps.canShowWindow) {
         if (deps.tuiLogin) {
             const ok = await deps.tuiLogin().catch(() => false);
@@ -109,8 +112,9 @@ async function runHeadedLogin(deps) {
         const r = await deps.probeLogin().catch(() => 'unknown');
         if (r === 'in')
             return true;
-        // A closed window is the operator's "I am done" signal: the browser object
-        // may still hold the session cookies, so verify once more and accept.
+        // A closed window is the operator's "I am done" signal. The session may
+        // have been established while the page was closing, so probe ONCE more —
+        // skipping that probe threw LoginRequiredError on a signed-in operator.
         if (deps.windowClosed()) {
             deps.notice('login.window_closed');
             const again = await deps

package/dist/browser-pure.js

@@ -14,7 +14,7 @@ export function cleanAnswer(raw) {
         return '';
     return raw;
 }
-// Pure, state-free helpers extracted from browser.ts . Nothing
+// Pure, state-free helpers extracted from browser.ts. Nothing
 // here touches the DeepSeekBrowser instance state, so this module is safe to
 // import anywhere (including hot-reloaded modules) without pulling in the
 // live Playwright context. browser.ts re-exports the public names, so the
@@ -318,15 +318,16 @@ export function normText(s) {
         .replace(/\s+/g, ' ')
         .trim();
 }
-// The "too frequent" error: distinct from others so ask() waits a long time
-// (DeepSeek limits reset over minutes) and retries the send itself.
+// Distinct from the other failures so ask() waits a long time (DeepSeek limits
+// reset over minutes) and then retries the send itself.
 export class RateLimitError extends Error {
     constructor(detail) {
         super('Messages too frequent. Try again later. ' + detail);
         this.name = 'RateLimitError';
     }
 }
-// DeepSeek transient server error (Server busy).
+// A transient DeepSeek overload: clears in seconds, so ask() retries it quickly
+// instead of waiting out the rate-limit pause.
 export class ServerBusyError extends Error {
     constructor(detail) {
         super('Server busy. Try again later. ' + detail);
@@ -357,7 +358,6 @@ export class GenerationIncompleteError extends Error {
         this.name = 'GenerationIncompleteError';
     }
 }
-// ---------- profile cleanup ----------
 export async function cleanSingletonFiles(dir = USER_DATA_DIR) {
     const names = ['SingletonLock', 'SingletonCookie', 'SingletonSocket'];
     for (const name of names) {

package/dist/browser.js

@@ -18,7 +18,7 @@ import { translate, DEFAULT_LOCALE } from './i18n.js';
 // here preserves the existing public API (test/*.test.ts import them from
 // browser.ts).
 export { STOP_NAME_RE, CONTINUE_NAME_RE };
-// The pure, state-free helpers live in browser-pure.ts : they are
+// The pure, state-free helpers live in browser-pure.ts: they are
 // imported for the class below and RE-EXPORTED here, so the public API of this
 // module is unchanged (tests and other modules import these names from
 // './browser.js').
@@ -278,12 +278,8 @@ export class DeepSeekBrowser {
             catch { }
         }
     }
-    // A lightweight IN-PAGE marker so the operator, looking at the browser
-    // window, can tell that the agent is generating (the terminal spinner is not
-    // visible there). We prefix the tab title with a "⏳"; the ORIGINAL title is
-    // saved ONCE on the window and restored when generation ends, so DeepSeek's
-    // own chat title is never lost. Best-effort: a page error must never break
-    // the send loop.
+    // Best-effort: a page error while marking the tab title must never break the
+    // send loop.
     async _setBusyTitle(busy) {
         try {
             await this.page.evaluate(setBusyTitleInDom, busy);
@@ -1313,8 +1309,8 @@ export class DeepSeekBrowser {
         const raw = await this._readLastAnswerTextDom().catch(() => '');
         return this._cleanAnswer(raw);
     }
-    // The pure filter lives in browser-pure.ts (cleanAnswer); kept as a method
-    // for the existing `this._cleanAnswer(...)` call sites.
+    // The filter itself is pure (cleanAnswer in browser-pure.ts); the method is
+    // kept so the existing `this._cleanAnswer(...)` call sites stay unchanged.
     _cleanAnswer(raw) {
         return cleanAnswer(raw);
     }
@@ -2752,17 +2748,11 @@ export class DeepSeekBrowser {
     // bottom. Used by /resume and /resume-id so the operator sees the restored
     // context in the terminal instead of just "Chat opened.".
     //
-    // Best-effort DOM scraping: DeepSeek renders every message as a markdown
-    // block (assistant) or a plain bubble (user). Role detection relies on the
-    // ds-assistant-message-* class; anything that is not an assistant block is
-    // treated as a user message. The model's reasoning (.ds-think-content) is
-    // skipped, exactly like in _readLastAnswerText.
-    // Read the WHOLE visible dialogue of the currently open chat, top to
-    // bottom. Used by /resume and /resume-id so the operator sees the restored
-    // context in the terminal instead of just "Chat opened.".
-    //
-    // The in-page scraper (scrapeChatMessages, browser-chats.ts) is a named
-    // export so the class body stays thin; it runs inside the page.
+    // Best-effort DOM scraping (the in-page scrapeChatMessages in
+    // browser-chats.ts, kept as a named export so the class stays thin): role
+    // detection relies on the ds-assistant-message-* class, anything that is not
+    // an assistant block is a user message, and the model's reasoning
+    // (.ds-think-content) is skipped exactly like in _readLastAnswerText.
     async readChatMessages() {
         if (!this.page)
             return [];

package/dist/changelog.js

@@ -1,15 +1,12 @@
-// Conventional-Commits -> CHANGELOG draft. PURE and
-// unit-tested: the draft is built from git log OUTSIDE the release flow
-// (scripts/changelog-draft.mts), so a misparse never touches CHANGELOG.md.
-// We do NOT auto-write the file: a human (or the agent) reviews the draft
-// first -- the changelog must stay a deliberate document, not a byproduct.
-// Parse one commit SUBJECT into a conventional commit, or null when it
-// does not follow the format (a free-form subject is skipped from the draft).
-// Supported forms:
-//   feat: add X            feat(cli): add X
-//   fix!: change Y         feat!: breaking
-// Explicit breaking markers: `!` after the type/scope. A BREAKING CHANGE
-// body footer is beyond a subject-only parser (git log %s only) -- out of scope.
+// Conventional-Commits -> CHANGELOG draft. PURE and unit-tested: the draft is
+// built from git log by scripts/changelog-draft.mts (and the release script),
+// and `syncUnreleased` rewrites the Unreleased section only on the explicit
+// --write flag — the changelog must stay a deliberate document, not a
+// byproduct of whatever landed on the branch.
+// Parse one commit SUBJECT into a conventional commit, or null when it does
+// not follow the format (a free-form subject is skipped from the draft).
+// `feat: add X`, `feat(cli): add X` and `fix!: change Y` are all understood. A
+// `BREAKING CHANGE:` body footer is out of scope: git log %s carries no body.
 export function parseConventionalCommit(subject) {
     const s = String(subject ?? '').trim();
     if (!s)
@@ -37,11 +34,9 @@ const KIND_BY_TYPE = {
 };
 // Types that never belong in a user-facing changelog (build/CI/internal).
 const SKIP_TYPES = new Set(['chore', 'ci', 'test', 'build', 'docs', 'internal']);
-// Group parsed commits by changelog kind. Each entry is a short
-// imperative line; a scope is prefixed as `**scope:** ` and a breaking marker
-// is appended as `(BREAKING)`. Order within a group is preserved (commit
-// order), and duplicate lines are dropped so a rebase that repeats a subject
-// does not flip the draft twice.
+// Group parsed commits by changelog kind. A scope is prefixed as `**scope:** `
+// and a breaking marker appended as `(BREAKING)`. Duplicate lines are dropped
+// so a rebase that repeats a subject does not list it twice.
 export function groupCommits(forCommit) {
     const groups = {
         Added: [],
@@ -63,9 +58,8 @@ export function groupCommits(forCommit) {
     }
     return groups;
 }
-// Render the groups as a changelog body (the parts under a version
-// heading), with the standard Keep-a-Changelog order and empty groups
-// omitted. Returns '' when there is nothing to report.
+// Render the groups as a changelog body, in the standard Keep-a-Changelog
+// order and with empty groups omitted. Returns '' when there is nothing to report.
 export function renderChangelogDraft(groups) {
     const order = ['Added', 'Fixed', 'Changed', 'Removed'];
     const out = [];

package/dist/checkpoint.js

@@ -4,7 +4,7 @@ import os from 'os';
 import { execFile } from 'child_process';
 import { randomBytes } from 'crypto';
 import { writeJsonAtomic } from './fsutil.js';
-// Checkpoints / rewind .
+// Checkpoints / rewind.
 //
 // Undo (src/undo.ts) reverts ONE file write. A checkpoint snapshots the WHOLE
 // working tree at the start of a task, so a risky task can be rolled back with
@@ -21,7 +21,6 @@ const CHECKPOINT_DIR = path.join(os.homedir(), '.zames', 'checkpoints');
 // dir. Everything else is part of the snapshot. Patterns are anchored to the
 // archive root ('./x') so a nested `src/tmp` is NOT accidentally excluded.
 export const CHECKPOINT_EXCLUDES = ['node_modules', '.git', 'dist', 'tmp'];
-// ---------- pure helpers (unit-tested without tar) ----------
 export function tarCreateArgs(archive, _dir) {
     const args = ['-czf', archive];
     for (const e of CHECKPOINT_EXCLUDES)
@@ -40,15 +39,14 @@ export function rotateRecords(records, max) {
         return records.slice();
     return records.slice(records.length - max);
 }
-/**
- * Resolve a `/rewind` selector against records sorted NEWEST FIRST.
- * `1` = most recent; a non-numeric selector is matched as a stamp prefix.
- */
 /** True for a selector that is exactly a full epoch-ms stamp (13 digits). */
 export function isFullStampSelector(selector) {
     return /^\d{13}$/.test(String(selector ?? '').trim());
 }
-const isFullStamp = isFullStampSelector;
+/**
+ * Resolve a `/rewind` selector against records sorted NEWEST FIRST.
+ * `1` = most recent; a longer number is matched as a stamp prefix.
+ */
 export function pickRecord(records, selector, 
 /** The FULL record set, when `records` is only a display window. */
 allRecords) {
@@ -62,7 +60,7 @@ allRecords) {
     // happened to share its first digits with a NEWER one was resolved to that
     // WRONG record. A full stamp is therefore matched against the full set
     // FIRST, and only then treated as an index.
-    if (isFullStamp(sel)) {
+    if (isFullStampSelector(sel)) {
         const pool = allRecords && allRecords.length ? allRecords : records;
         const exact = pool.find((r) => String(r.stamp) === sel);
         if (exact)
@@ -85,7 +83,6 @@ allRecords) {
         pool.find((r) => String(r.stamp).startsWith(sel)) ||
         null);
 }
-// ---------- tar / fs plumbing ----------
 function runTar(args, cwd, timeout = 60_000) {
     return new Promise((resolve) => {
         // `execFile` throws SYNCHRONOUSLY on a spawn error (ENOENT for the shell,
@@ -137,7 +134,6 @@ async function copyTree(from, to) {
             await fs.copyFile(src, dst);
     }
 }
-// ---------- store ----------
 export class CheckpointStore {
     enabled;
     maxBackups;