zames_pro 2.82.136 → 2.82.138

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 = [];
@@ -80,7 +74,7 @@ export function renderChangelogDraft(groups) {
 }
 /**
  * Replace the body between `## [Unreleased]` and the next `## [` heading with
- * a freshly drafted body (D2). PURE so a test can pin the exact behavior.
+ * a freshly drafted body. PURE so a test can pin the exact behavior.
  *
  * When `body` is empty the existing Unreleased section is CLEARED to just the
  * heading (no stray blank content). A changelog without an Unreleased heading

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'];
 }
@@ -238,7 +238,7 @@ export function renderCost(stats, transcriptFile, tokenUsage = null, t = transla
         for (const [name, n] of top)
             lines.push(' ' + nameLine(name) + ': ' + n);
     }
-    // T-D3: where the time went. Sort by total ms, show the top tools with the
+    // Where the time went. Sort by total ms, show the top tools with the
     // count and a human duration, so Read→Edit cycles and slow Bash calls are
     // visible at a glance.
     const byTime = Object.entries(stats.toolTime || {})
@@ -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/compact.js

@@ -43,7 +43,7 @@ export async function performCompact(opts) {
     try {
         if (!quiet)
             log(t('compact.start'));
-        // PreCompact hook (N32): runs BEFORE the chat is summarized/rewritten, so
+        // PreCompact hook: runs BEFORE the chat is summarized/rewritten, so
         // a hook can persist state or notify. Best-effort.
         try {
             const out = await runLifecycleHooks(loadHooks(workdir), 'PreCompact', workdir, { chatId: currentChatId });

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,

package/dist/config.js

@@ -135,7 +135,7 @@ export const DEFAULTS = {
         contextLimit: 1_000_000,
         // 0 = auto: terminal width, capped at 100 columns (the historical width).
         answerWidth: 0,
-        // N37 statusLine hook. Off by default; edit config.json to set a command.
+        // statusLine hook. Off by default; edit config.json to set a command.
         statusLineCommand: '',
         statusLineIntervalMs: 10_000,
         // Use the Ink/React TUI for the input area (the rich editor). Set to false
@@ -611,14 +611,11 @@ export function getConfigField(path) {
 }
 export function getByPath(obj, path) {
     const parts = path.split('.');
-    // Same guard as setByPath: a dotted path must never reach the PROTOTYPE
-    // chain. `__proto__` used to return Object.prototype (an object), and
-    // `constructor` / `toString` the corresponding FUNCTION — the /config list
-    // then rendered `{}` for a field whose path did not exist instead of the
-    // default marker. Reading those keys is never meaningful for a config path.
-    // `toString`, `valueOf`, `hasOwnProperty`, ... are inherited METHODS, not
-    // config values: reading them through a dotted path returned a FUNCTION.
-    // Only an OWN property counts.
+    // Two guards, both load-bearing. UNSAFE_PATH_KEYS keeps a dotted path off
+    // the PROTOTYPE chain (`__proto__` answered Object.prototype, `constructor`
+    // a function, and the /config list rendered `{}` instead of the default
+    // marker). The hasOwnProperty check keeps inherited METHODS out: `toString`
+    // / `valueOf` are not config values, and only an OWN property counts.
     if (parts.some((p) => UNSAFE_PATH_KEYS.has(p)))
         return undefined;
     let cur = obj;

package/dist/deepseek-ui.js

@@ -45,9 +45,9 @@ export const SEND_SELECTORS = [
     'button[aria-label*="отправ" i]',
 ];
 // DeepSeek's own button classes. Used as a coarse "is this a DeepSeek button?"
-// filter in the raw-DOM fallbacks for Stop and for the send/stop icon
-// detection. Kept as a source string because the fallback runs INSIDE
-// page.evaluate, which cannot receive a RegExp object.
+// filter in the raw-DOM fallbacks for Stop and the send/stop icon detection.
+// Passed to page.evaluate as a source string (`.source`), because the in-page
+// callback cannot receive a RegExp object.
 export const BUTTON_CLASS_RE = /ds-button--(circle|primary|filled)/i;
 // ---------- stop ----------
 export const STOP_SELECTORS = [
@@ -108,10 +108,10 @@ export const MESSAGE_CONTAINER_SELECTORS = [
 export const ASSISTANT_CLASS_RE = /ds-assistant-message|assistant-message/i;
 export const ASSISTANT_SELECTOR = '[class*="ds-assistant-message"]';
 export const ANSWER_MARKDOWN_SELECTOR = '[class*="ds-markdown"]';
-// A cheap "did the page grow?" signal (_chatSignal): count message nodes and
-// measure the LAST answer node's text length. The old code read
-// document.body.innerText.length every tick, which serializes the WHOLE page
-// (sidebar, history, menus) and forces a layout — expensive on a long chat.
+// The node/markdown selectors for the cheap "did the page grow?" signal
+// (chatSignalInDom): counting message nodes and measuring the LAST answer's
+// length instead of document.body.innerText.length, which serializes the WHOLE
+// page (sidebar, history, menus) and forces a layout on every tick.
 export const CHAT_SIGNAL_NODES_SELECTOR = '[class*="ds-message"], [class*="chat-message"], .ds-markdown';
 export const MARKDOWN_SELECTOR = '.ds-markdown';
 // ---------- new chat ----------

package/dist/diagnostics.js

@@ -24,13 +24,11 @@ export function summarizeDiagnostics(raw, maxChars) {
     if (!text)
         return null;
     // tsc prints nothing on success, but with `--watch` it prints a banner
-    // ("Found 0 errors. Watching for file changes.") that is longer than the
-    // old 40-char cutoff — so every edit in watch mode appended a pointless
-    // diagnostics block to the tool result. Treat a ZERO-count banner as clean
-    // however long it is; a non-zero count still falls through to the lines.
-    // Strip ANSI colour codes and a leading `[12:00:00]`-style timestamp before
-    // the clean checks: a watch-mode banner is usually coloured and/or
-    // prefixed, and the anchored regexes missed it — every edit then
+    // ("Found 0 errors. Watching for file changes."). Treat a ZERO-count banner
+    // as clean however long it is; a non-zero count still falls through to the
+    // lines. The banner is usually coloured and/or prefixed with a
+    // `[12:00:00]`-style timestamp, so both are stripped before the clean
+    // checks — with them in place the anchored regexes missed it and every edit
     // appended a pointless diagnostics block.
     const plain = text.replace(/\x1b\[[0-9;?]*[A-Za-z]/g, '');
     const cleanCheck = plain.replace(/^\[[^\]]*\]\s*/, '');

package/dist/diff.js

@@ -5,12 +5,11 @@ export function unifiedDiff(a, b, { label = '' } = {}) {
     const out = [];
     if (label)
         out.push('... ' + label);
-    // A LINE diff, not a positional compare: an insertion at the top of a
-    // file shifts every following line, and comparing line i with line i
-    // reported the WHOLE file as removed AND added (a 20-line file with one
-    // new first line produced 41 diff lines). The undo/`/diff` preview became
-    // unreadable and the model was told everything changed. A longest-common-
-    // subsequence walk emits only the lines that really differ.
+    // A LINE diff, not a positional compare: an insertion at the top shifts
+    // every following line, and comparing line i with line i reported the WHOLE
+    // file as removed AND added (a 20-line file with one new first line produced
+    // 41 diff lines), which made the undo preview unreadable and told the model
+    // that everything changed.
     for (const op of diffOps(la, lb)) {
         if (op.kind === 'same')
             continue;
@@ -19,10 +18,9 @@ export function unifiedDiff(a, b, { label = '' } = {}) {
     return out.join(String.fromCharCode(10));
 }
 /**
- * Longest-common-subsequence line diff (Hunt–Szymanski style table, O(n*m)
- * memory). Files here are a handful of lines (a single edit preview), so the
- * simple dynamic-programming table is fine and keeps the module dependency-
- * free. A change pair is emitted as `-old` then `+new`, like a real diff.
+ * Longest-common-subsequence line diff (an O(n*m) dynamic-programming table,
+ * which keeps the module dependency-free). A change pair is emitted as
+ * `-old` then `+new`, like a real diff.
  */
 function diffOps(a, b) {
     const n = a.length;

package/dist/extraTools.js

@@ -1,14 +1,14 @@
 import fs from 'fs/promises';
 import path from 'path';
 import { safePath } from './sandbox.js';
-// Extra tools that bring zames closer to Claude Code / Codex CLI:
+// Extra tools borrowed from Claude Code / Codex CLI:
 //   * LS         - list a directory (Claude Code has LS; Codex has list_dir)
 //   * MultiEdit  - several edits to ONE file applied atomically (Claude Code)
 //   * ApplyPatch - multi-file V4A-style patch (Codex apply_patch)
 //   * TodoWrite  - session task checklist (Claude Code TodoWrite)
 //
-// The core file tools (Read/Write/Edit/Bash/Glob/Grep) live in tools.ts.
-// These are additive: they do not change the existing tool behaviour.
+// The core file tools (Read/Write/Edit/Bash/Glob/Grep) live in tools.ts; these
+// are additive and do not change their behaviour.
 const NL = String.fromCharCode(10);
 function decodeContent(content, contentBase64) {
     if (typeof contentBase64 === 'string' && contentBase64.length) {

package/dist/fsutil.js

@@ -6,10 +6,6 @@ import path from 'path';
 // truncated/broken file behind. The temp name embeds the pid so two
 // concurrent processes never collide on it.
 //
-// Kept in its own module because the pattern used to be copy-pasted in at
-// least three places; drift between the copies is exactly how one of them
-// silently loses its atomicity.
-//
 // The file is created 0o600: ~/.zames/config.json holds the
 // browser password in clear text alongside sessions/undo history, and the
 // default umask (022) would make them readable by other users on a

package/dist/gitTools.js

@@ -89,9 +89,9 @@ export async function getGitContext(workdir) {
     const remoteR = await runGitArgs(['remote'], workdir, 5000);
     const aheadR = await runGitArgs(['rev-list', '--left-right', '--count', '@{upstream}...HEAD'], workdir, 5000);
     const branch = branchR.trim() || '(detached HEAD)';
-    // The shared runner answers an EMPTY successful run with a sentinel. Storing
-    // it verbatim told the model the tree had a modified "file" that is really
-    // an internal string — every task started with a false dirty-tree premise.
+    // The sentinel is an internal string, not a path: kept verbatim it told the
+    // model the tree had a modified "file" by that name, so every task started
+    // with a false dirty-tree premise.
     const statusText = statusR.trim() === NO_OUTPUT_SENTINEL ? '' : statusR;
     const statusLines = statusText
         .split(String.fromCharCode(10))
@@ -286,7 +286,8 @@ export function createGitTools(workdir) {
                 if (!msg.trim()) {
                     return 'Error: message is empty.';
                 }
-                // We stage everything only if the index is empty (as the description promises).
+                // Stage everything only when the index is empty, which is what the tool
+                // description promises the model.
                 const staged = await runGitArgs(['diff', '--cached', '--name-only'], workdir, 10_000);
                 const stagedTrimmed = staged.trim();
                 const hasStaged = stagedTrimmed !== '' && stagedTrimmed !== NO_OUTPUT_SENTINEL;

package/dist/glm-browser.js

@@ -3,8 +3,8 @@
 // WHY a separate class (the same reason QwenBrowser exists): the backends
 // differ in EVERY layer we touch — selectors, the send flow, the SSE format and
 // the chat-id scheme. The shared surface is the ChatProvider interface
-// (types.ts); the agent loop only needs BrowserLike and never sees either
-// concrete class.
+// (types.ts); the agent loop only needs BrowserLike and never sees a concrete
+// class.
 //
 // Verified against a LIVE signed-in session (headless + headed Chromium):
 //   input  -> textarea#chat-input
@@ -233,7 +233,6 @@ export class GlmBrowser {
     sendIntervalMs() {
         return this.sendRuntime.intervalMs();
     }
-    // ---------- lifecycle ----------
     async launch() {
         if (await profileLooksLocked(GLM_USER_DATA_DIR)) {
             if (this.debug)
@@ -398,7 +397,6 @@ export class GlmBrowser {
         }
         throw new Error('glm: the browser profile is still locked');
     }
-    // ---------- login ----------
     /**
      * A FAST login probe with NO long waits (the old isLoggedIn polled up to
      * 20 s, which is a false "logged out" on a slow network). The signal is the
@@ -754,7 +752,6 @@ export class GlmBrowser {
         }
         catch { }
     }
-    // ---------- network capture ----------
     _installNetHook() {
         const pg = this.page;
         if (!pg)
@@ -857,7 +854,6 @@ export class GlmBrowser {
         }
         catch { }
     }
-    // ---------- chat management ----------
     async newChat() {
         const before = await this.getCurrentChatId();
         const btn = this.page.locator(NEW_CHAT_SELECTOR).first();
@@ -1081,7 +1077,6 @@ export class GlmBrowser {
     getLastTokenUsage() {
         return this._lastTokenUsage;
     }
-    // ---------- answer reading ----------
     async _readLastAnswerText() {
         if (this._netCapture && this._netCaptureAt >= this._lastSentAt) {
             return this._netCapture;
@@ -1239,7 +1234,6 @@ export class GlmBrowser {
         }
         catch { }
     }
-    // ---------- toggles ----------
     getToggleStatesSync() {
         return this._toggles;
     }
@@ -1295,7 +1289,6 @@ export class GlmBrowser {
             this._notice(this._t('glm.mode_mismatch', { v: mismatch.join(', ') }));
         }
     }
-    // ---------- diagnostics ----------
     async dumpDom(filePath) {
         if (!this.page)
             throw new Error('glm: browser not launched');
@@ -1332,7 +1325,6 @@ export class GlmBrowser {
     chatUrl(id) {
         return chatUrl(id);
     }
-    // ---------- ask ----------
     async _waitForSendSlot(agent) {
         if (!agent)
             return;

package/dist/glm-chats.js

@@ -78,7 +78,6 @@ export function scrapeGlmChatMessages(opts) {
     }
     return out;
 }
-// ---------- API payload parsers (pure, unit-tested) ----------
 /**
  * Parse `GET /api/v1/chats/?page=1`. The live shape is a BARE ARRAY; older /
  * other builds wrap it in `{data:[...]}` or key it by id. Returns [] for

package/dist/glm-net-capture.js

@@ -280,7 +280,6 @@ function hasAnswerContent(body) {
 // ERROR payload (`type: "error"` / a code) or as a bare notice with NO answer
 // content; answer text lives in `phase: "answer"` chunks.
 const ERROR_PAYLOAD_RE = /"type"\s*:\s*"error"|"error"\s*:|\{\s*"code"\s*:/i;
-/** True when the RAW body is z.ai's rate-limit notice. */
 /**
  * True when z.ai answers that the ACCOUNT quota is exhausted
  * (`insufficient_quota`, HTTP 429). Distinct from a transient rate limit:
@@ -299,6 +298,7 @@ export function isQuotaExhaustedText(body) {
         return false;
     return /insufficient_quota|quota (?:exceeded|exhausted)|exceeded your current quota/i.test(b);
 }
+/** True when the RAW body is z.ai's rate-limit notice. */
 export function isRateLimitText(body) {
     const b = String(body || '');
     if (!b)

package/dist/glm-ui.js

@@ -162,8 +162,8 @@ export const GUEST_LIMIT_RE = /guest.*limit|log ?in to continue|sign ?in to cont
 // The completion body carries the mode flags. `preview_mode` and the
 // `general_agent` flag change the ANSWER FORMAT (they enable z.ai's own agent
 // runtime, which emits its own tool calls), and that would break the zames
-// tool-call protocol. The provider asserts the safe values on every send; the
-// request body is verified per chat (see glm-net-capture.ts).
+// tool-call protocol. The request body is rewritten before the send and then
+// verified (glm-browser's _installRuntimeRewrite/_verifyModes + glm-flags).
 export const SAFE_PREVIEW_MODE = false;
 export const SAFE_ENABLE_THINKING = true;
 // ---------- attachments ----------