zames_pro 2.82.137 → 2.82.139

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;

package/dist/cli-args.js

@@ -1,4 +1,4 @@
-// PURE CLI argument parsing . index.ts used to hold `args` as a
+// PURE CLI argument parsing. index.ts used to hold `args` as a
 // module-level constant plus three helpers closing over it, which made the
 // parsing impossible to unit-test in isolation (index.ts is the entry point and
 // is not covered). `createCliArgs(argv)` binds the argv ONCE and returns the
@@ -23,11 +23,9 @@ export function createCliArgs(argv) {
     const args = Array.isArray(argv) ? argv : [];
     const getArg = (flag, fallback = null) => {
         const i = args.indexOf(flag);
-        // `args[i + 1]` truthiness means an empty string or a missing value falls
-        // back — the historical behavior (an empty `--task ''` is not a task).
-        // NOTE: this does NOT special-case a following FLAG, so `--task --debug`
-        // returns `--debug` (the original inline helper did the same); we preserve
-        // it rather than "fix" a long-standing quirk in a refactor.
+        // Truthiness, not `!== undefined`: an EMPTY value (`--task ''`) must
+        // fall back too - an empty task is not a task. A following FLAG is not
+        // special-cased either (see the CliArgs doc above).
         return i !== -1 && args[i + 1] ? args[i + 1] : fallback;
     };
     const hasFlag = (flag) => args.includes(flag);

package/dist/code-blocks.js

@@ -1,9 +1,6 @@
-// PURE extraction of fenced code blocks from a Markdown answer.
-//
-// The TUI cannot offer a real "copy button", so copying a code block is bound
-// to a key / command. That needs the RAW fenced text (the highlighted render is
-// full of ANSI and must NOT be copied), which is what this module returns.
-// Kept pure so it is unit-tested without a terminal.
+// PURE extraction of fenced code blocks from a Markdown answer. It returns the
+// RAW fenced text: the highlighted render is full of ANSI and must never be
+// what gets copied out of the TUI.
 /**
  * Extract every fenced code block (``` or ~~~) from `md`, in order. Handles a
  * longer closing fence than the opening one and an unterminated final block

package/dist/commands.js

@@ -3,9 +3,9 @@ import path from 'path';
 import os from 'os';
 import { isImageName } from './attachments.js';
 import { translate } from './i18n.js';
-// Helpers for the extra slash commands (/diff, /cost, /export, /doctor,
-// /review, /add-dir). Pure functions, unit-tested without a
-// live agent/browser. index.ts only renders their output.
+// Helpers for the slash commands (/diff, /cost, /export, /doctor, /review,
+// /add-dir, /queue, /goal, custom commands). Pure functions, unit-tested
+// without a live agent/browser; index.ts only renders their output.
 const NL = String.fromCharCode(10);
 const CR = String.fromCharCode(13);
 // ---------- /diff ----------
@@ -27,8 +27,8 @@ export function formatDiff(diffText, opts = {}, t = translate('en')) {
     const rest = lines.length - maxLines;
     return head + NL + t('diff.more_lines', { n: String(rest) });
 }
-// Git arguments for the `/diff` command. An ARRAY (no shell): the caller runs
-// them through runGitArgs(), so nothing is interpreted by a shell. Pure.
+// Git arguments for the `/diff` command. An ARRAY, not a shell string: the
+// caller runs them through runGitArgs(), so nothing is interpreted by a shell.
 export function diffGitArgs(staged = false) {
     return staged ? ['diff', '--staged'] : ['diff'];
 }
@@ -1038,10 +1038,6 @@ export function mergeMessages(msgs) {
         list.length +
         ' messages while you were working. They are listed below in order — ' +
         'treat them as one instruction.';
-    // The parts are OPERATOR text: a line that reads `---` (a Markdown rule, a
-    // pasted diff) would spoof a message boundary and the model would see an
-    // extra message. Neutralize such a line inside a part; the separator below
-    // stays the only one.
     // The message text is OPERATOR input: a line that reads `---` (a Markdown
     // rule, a pasted diff) or repeats the header sentence spoofs a message
     // BOUNDARY / count — the model then sees an extra message that never

package/dist/config-command.js

@@ -1,4 +1,4 @@
-// The /config subcommand DISPATCH . The PURE text rendering lives
+// The /config subcommand DISPATCH. The PURE text rendering lives
 // in config-commands.ts; this module owns the parsing/routing of the text
 // subcommands (list/get/set/reset/lang/path) and prints through injected
 // callbacks. Everything that touches live state (the runtime config object,
@@ -26,7 +26,7 @@ export async function handleConfigCommand(input, deps) {
         ? all
         : ['/config', ...all];
     const sub = (parts[1] || '').toLowerCase();
-    // With no subcommand - the menu (or a text list in non-TTY).
+    // Bare `/config` opens the menu (a text list in non-TTY, via openMenu).
     if (!sub || sub === 'menu') {
         await openMenu();
         return;
@@ -39,7 +39,8 @@ export async function handleConfigCommand(input, deps) {
         print(theme.system(t('cfg.paths', { global: homeConfigPath, project: projectConfigPath })));
         return;
     }
-    // /config lang <ru|en> - quick access to ui.locale
+    // `/config lang <ru|en>` is a shortcut for `set ui.locale`; `язык` is an
+    // accepted alias because the operator types the command in their own language.
     if (sub === 'lang' || sub === 'language' || sub === 'язык') {
         const val = parts[2];
         if (!val) {
@@ -123,7 +124,8 @@ export async function handleConfigCommand(input, deps) {
         print(theme.assistant(t('cfg.reset', { v: key })));
         return;
     }
-    // Unknown subcommand - show the list.
+    // An unknown subcommand reports the key AND shows the list, so the operator
+    // sees the valid keys without a second command.
     printErr(theme.error(t('cfg.unknown_key', { v: sub })));
     showList();
 }

package/dist/config-commands.js

@@ -1,4 +1,4 @@
-// PURE rendering for the /config command's TEXT output . These
+// PURE rendering for the /config command's TEXT output. These
 // helpers format the schema into lines; they touch no browser/terminal state,
 // so they are unit-tested without a live agent (the /config dispatch in
 // index.ts only prints what these return). The interactive menu lives in
@@ -6,8 +6,7 @@
 import { theme } from './theme.js';
 /**
  * The value shown for a field in the LIST: the default marker, on/off for a
- * boolean, a fixed mask for a password, otherwise the JSON form. Mirrors the
- * long-used inline logic so the output is byte-for-byte identical.
+ * boolean, a fixed mask for a password, otherwise the JSON form.
  */
 export function configListValue(field, value, t) {
     if (value === undefined)

package/dist/config-live.js

@@ -4,9 +4,9 @@
 // construction, so a change must be pushed onto the live instance too or it
 // is silently a no-op until a restart. The path->target mapping is PURE and
 // unit-tested here; index.ts wires it to the live instances.
-/** config path -> field name on the browser instance. Both backends
- *  share these names; a field that is absent on the active backend
- *  (e.g. DeepSeek-only thinkingExtraMs on Qwen) is skipped, not thrown. */
+/** config path -> field name on the browser instance. All three backends
+ *  share these names; a field that is absent on the active backend is skipped
+ *  rather than throwing, so one map serves every provider. */
 export const BROWSER_LIVE_CONFIG = {
     'browser.answerTimeoutMs': 'answerTimeoutMs',
     'browser.askRetries': 'askRetries',
@@ -30,11 +30,9 @@ export function applyBrowserLiveConfig(path, value, browser) {
     const field = BROWSER_LIVE_CONFIG[path];
     if (!field || !(field in browser))
         return false;
-    // A timing value must be a FINITE NON-NEGATIVE number. `applyBrowserLiveConfig`
-    // wrote anything it was given, so a NaN/negative value reached a live field
-    // (a timer with NaN fires immediately), and `answerTimeoutMs = 0` SKIPPED
-    // the derived `askDeadlineMs` update, leaving a deadline that no longer
-    // matches the timeout. Reject the bad value instead of corrupting the field.
+    // A timing value must be a FINITE NON-NEGATIVE number: a NaN reached a live
+    // field (a timer with NaN fires immediately) and a negative one is nonsense.
+    // Reject the value instead of corrupting the field.
     if (typeof value !== 'number' || !Number.isFinite(value) || value < 0) {
         return false;
     }
@@ -45,11 +43,7 @@ export function applyBrowserLiveConfig(path, value, browser) {
     // give up before the browser's own timeout ever fired. Re-derive it the same
     // way the constructor does.
     if (field === 'answerTimeoutMs' && 'askDeadlineMs' in browser) {
-        // ANY accepted number re-derives the deadline; the old `value > 0` guard
-        // left the deadline at its stale value when the timeout was set to 0.
-        if (typeof value === 'number' && Number.isFinite(value)) {
-            browser.askDeadlineMs = value + 60_000;
-        }
+        browser.askDeadlineMs = value + 60_000;
     }
     return true;
 }

package/dist/config-menu.js

@@ -11,9 +11,9 @@
 import { theme } from './theme.js';
 import { visLen } from './input.js';
 const ESC = String.fromCharCode(27);
-// Strip ANSI sequences to compute the visible length of a line. (A module-
-// level name distinct from input.ts's internal helper; the old regex had a
-// stray backtick and never matched a real CSI, so visCols was over-counted.)
+// Strip ANSI sequences to compute the visible length of a line. A local copy
+// (input.ts keeps its own private helper): the regex must match a real CSI,
+// otherwise visCols over-counts and the erase moves to the wrong row.
 function stripAnsiLocal(s) {
     return s.replace(/\x1b\[[0-9;?]*[A-Za-z]/g, '');
 }
@@ -197,7 +197,8 @@ export function runConfigMenu(opts) {
         lines.push(...tail);
         // Erase the previous block and print the new one. We count VISUAL lines
         // (accounting for wrapping by terminal width), otherwise on narrow
-        // terminals you get "garbage" from unerased tails.
+        // terminals the unerased tails leave garbage behind.
+        //
         // Reserve the last column. A line whose visible width reaches exactly the
         // terminal width hits the autowrap edge: depending on the terminal it
         // either consumes two rows for the following `\n` or keeps a pending wrap,