zames_pro 2.82.136 → 2.82.138

package/dist/goal.js

@@ -69,13 +69,10 @@ export function buildContinuationDirective(reason) {
         // Collapse whitespace in the model-written reason: a NEWLINE started a new
         // line of the directive, letting the evaluator's text inject an extra
         // instruction into the next turn.
-        (argumentReason(reason)
+        (String(reason ?? '').trim()
             ? 'Evaluator note: ' + String(reason).trim().replace(/\s+/g, ' ')
             : ''));
 }
-function argumentReason(reason) {
-    return String(reason ?? '').trim().length > 0;
-}
 /**
  * The text a verdict may be read from. A plain answer is returned as-is; a
  * TOOL CALL (JSON or DSML/XML) yields only its `respond` message, because the

package/dist/i18n/common.js

@@ -199,7 +199,7 @@ export const commonMessages = {
         ru: '📎 Вложен файл {marker} {name} ({size}) — {path}',
         en: '📎 Attached file {marker} {name} ({size}) — {path}',
     },
-    // B5: shown when `@path` references in a task were inlined as file contents.
+    // Shown when `@path` references in a task were inlined as file contents.
     'msg.at_refs': {
         ru: '📄 Встроено файлов по @-ссылкам: {n} —',
         en: '📄 Inlined @-referenced files: {n} —',
@@ -303,7 +303,7 @@ export const commonMessages = {
     },
     'cost.auto_compacts': { ru: ' авто-сжатий: {n}', en: ' auto-compacts: {n}' },
     'cost.by_tool': { ru: ' по инструментам:', en: ' by tool:' },
-    // T-D3: total time spent inside tools, then a top-N per-tool breakdown.
+    // Total time spent inside tools, then a top-N per-tool breakdown.
     'cost.by_time': {
         ru: ' время в инструментах: {dur}; топ по времени:',
         en: ' time in tools: {dur}; top by time:',

package/dist/i18n.js

@@ -2,8 +2,7 @@
 //
 // Goals:
 //   * change the interface language (help, service messages);
-//   * the agent's answer language — via the system-prompt;
-//   * the agent's answer language for a plain language.
+//   * the agent's answer language — via the system-prompt.
 //
 // Strategy: a single key->translation map. New keys are easy to add.
 // Russian texts (ru) are the default, English (en) is the alternative.

package/dist/index.js

@@ -76,7 +76,6 @@ function authMarkerExists(provider = config.provider) {
         return false;
     }
 }
-// ---------- CLI parsing ----------
 // The pure helpers live in src/cli-args.ts and are bound to this process's
 // argv ONCE here. Call sites keep using getArg/hasFlag/getPositional.
 const { getArg, hasFlag, getPositional } = createCliArgs(process.argv.slice(2));
@@ -182,7 +181,6 @@ if (jsonMode) {
         console.error(...a);
     });
 }
-// ---------- session persistence ----------
 // We store sessions (DeepSeek chats) in ~/.zames/.sessions so they survive
 // a restart and are available for explicit restore (--resume-last,
 // --chat <id>, /resume-id <id>). They are NOT brought up automatically on launch.
@@ -231,7 +229,6 @@ async function saveGoal(workdir, goal) {
         // best-effort: a failed write must not break the command
     }
 }
-// ---------- hot reload ----------
 // We keep references to the logic modules in the mod object. The /reload
 // command re-reads them via a dynamic import with a timestamp query — Node
 // caches ESM by URL, so such an import returns a fresh version of the module.
@@ -257,7 +254,7 @@ const RELOADABLE = [
     'config',
     'fsutil',
     'backlog',
-    // C3 modules index.ts now calls through the mod bag (G2): without this a
+    // Modules index.ts now calls through the mod bag: without this a
     // /reload silently kept the OLD commands/run-task/attach-refs until a restart.
     // 'input/layout' is NOT listed: its only live consumers are input.ts (the
     // long-lived LineEditor, itself not reloadable) and spinner.ts, which already
@@ -406,7 +403,6 @@ function warnIfBrowserChanged() {
     }
     catch { }
 }
-// ---------- helpers ----------
 function printHelp() {
     // Self-development commands (backlog / self-review) are shown ONLY in dev
     // mode. A regular package install must not advertise them to the user — and
@@ -594,7 +590,7 @@ async function refreshDynamicCommands(workdir) {
         }
         const cmds = await loadCommands(workdir);
         for (const c of cmds) {
-            // B7: carry the declared argument hint as a separate field so the hint is
+            // Carry the declared argument hint as a separate field so the hint is
             // SHOWN in the suggest list but never inserted into the input line.
             out.push({
                 name: '/' + c.name,
@@ -619,7 +615,7 @@ async function expandSlashTarget(workdir, name, rest) {
         const commands = await loadCommands(workdir);
         const cmd = commands.find((c) => c.name.toLowerCase() === name.toLowerCase());
         if (cmd) {
-            // B7: refuse to run when declared positional arguments are missing.
+            // Refuse to run when declared positional arguments are missing.
             const missing = mod.commands.missingCommandArgs(cmd.arguments, rest);
             if (missing.length)
                 return { missing };
@@ -689,7 +685,6 @@ async function cleanTmpDir() {
             console.error('tmp: cleanup failed:', e.message);
     }
 }
-// ---------- workdir resolution ----------
 // The agent works in the directory it was launched from (process.cwd()).
 // This is the sandbox root: tools cannot go above it.
 async function resolveWorkdir() {
@@ -701,7 +696,6 @@ async function resolveWorkdir() {
     }
     return dir;
 }
-// ---------- restored dialogue ----------
 /**
  * The answer timeout for a one-off provider call (compact summary, etc.).
  * Qwen's cold guest turns are far slower than DeepSeek's, so using
@@ -1403,14 +1397,13 @@ async function main() {
     // Timestamp of the last Ctrl+C while the agent was busy. Two presses
     // within 2s escalate from "abort the running tool" to "stop the whole run".
     let lastCtrlCAt = 0;
-    // ---------- review mode state ----------
     // null — normal mode.
     // { snapDir, snapName, originalWorkdir } — we're inside a snapshot, the chat
     // is already initialized with the review prompt, the user can just type "fix...".
     let reviewMode = null;
     process.on('SIGINT', async () => {
         running = false;
-        // Kill any background processes (N33) so no dev-server outlives the agent.
+        // Kill any background processes so no dev-server outlives the agent.
         killAllBackground();
         await browser.close().catch(() => { });
         // Close the lazy headless browser from web.js, otherwise it would stay
@@ -1458,7 +1451,6 @@ async function main() {
             console.error(theme.error(t('msg.open_chat_error', { v: e.message })));
         }
     }
-    // ---------- input: permanent line below + status above ----------
     // In TTY we use LineEditor: it owns the input all the time, shows the
     // status above the input line and prints the agent's answers ABOVE it, so
     // the typed text is never overwritten by output. In non-TTY (pipe) —
@@ -1468,7 +1460,7 @@ async function main() {
     // swap. Assigned where the editor is built (see the login wiring below);
     // a no-op until then so /provider before setup is harmless.
     let wireLoginCallbacks = () => { };
-    // N37 statusLine hook timer (cleared on exit).
+    // statusLine hook timer (cleared on exit).
     let statusLineTimer = null;
     let waiter = null;
     const takeInput = () => {
@@ -1619,7 +1611,7 @@ async function main() {
             };
         };
         wireLoginCallbacks(browser);
-        // N37 statusLine hook: run the configured command on its own timer (never
+        // statusLine hook: run the configured command on its own timer (never
         // per-render — a spawn would block typing) and show its first stdout line
         // in the status line. Off when ui.statusLineCommand is empty.
         if (config.ui.statusLineCommand) {
@@ -1878,7 +1870,6 @@ async function main() {
         edAny._origLog = origLog;
         edAny._origErr = origErr;
     }
-    // ---------- /config ----------
     // View and edit settings without a restart. Values are validated against
     // CONFIG_SCHEMA (src/config.ts) and written to the project .zamesrc.json.
     // The language (ui.locale) is applied immediately: we change currentLocale,
@@ -2407,7 +2398,6 @@ async function main() {
             }
             continue;
         }
-        // ---------- Self-review ----------
         if (lower === '/self-review' || lower.startsWith('/self-review ')) {
             const focus = trimmed.slice('/self-review'.length).trim();
             // Remember where to return
@@ -2571,7 +2561,6 @@ async function main() {
             }
             continue;
         }
-        // ---------- Regular commands ----------
         if (lower === '/chats' || lower.startsWith('/chats ')) {
             const filter = trimmed.slice('/chats'.length).trim().toLowerCase();
             const spin = editor || mod.createSpinner(currentLocale);
@@ -4065,7 +4054,7 @@ async function main() {
                 }
             }
         }
-        // B7: a custom command with mandatory `arguments:` refuses to run until the
+        // A custom command with mandatory `arguments:` refuses to run until the
         // operator supplies them (by position).
         if (missingArgs) {
             console.error(theme.error(t('msg.missing_args', { v: missingArgs.join(', ') })));
@@ -4081,7 +4070,7 @@ async function main() {
             : expandedTask !== null
                 ? expandedTask
                 : trimmed;
-        // B5: inline any `@path` references into the task so the agent does not
+        // Inline any `@path` references into the task so the agent does not
         // spend a round-trip reading a file the operator already pointed at. Only
         // existing files are inlined; a bare `@name` in prose is left untouched.
         let finalTaskText = taskText;

package/dist/input/suggest.js

@@ -107,13 +107,10 @@ export function completeSlashCommand(buf, sugg, selected) {
             prefix = prefix.slice(0, -1);
         }
     }
-    // Stop shrinking at the TYPED text: with an empty buffer the list matches
-    // everything, and the shared prefix used to shrink down to a bare "/" (or a
-    // common first letter like "/h") — Tab then inserted something the operator
-    // never typed. A completion must EXTEND what is already there.
-    // With an EMPTY buffer everything matches and the shared prefix collapses
-    // to a bare "/" (or a common first letter like "/h"): Tab must not guess a
-    // command the operator has not started typing.
+    // A completion must EXTEND what is already typed: with an empty buffer the
+    // list matches everything and the shared prefix used to collapse to a bare
+    // "/" (or a common first letter like "/h"), so Tab inserted something the
+    // operator never typed.
     if (typed.length < 2)
         return null;
     if (!prefix.toLowerCase().startsWith(typed))

package/dist/input.js

@@ -96,7 +96,7 @@ export class LineEditor {
     // is stored so it can be removed in dispose().
     _onResize;
     _resizeTimer;
-    // ESC disambiguation (N29): a read may end on a bare ESC that is really the
+    // ESC disambiguation: a read may end on a bare ESC that is really the
     // head of a control sequence split across two reads. We hold it briefly and
     // fire onEscape only if no tail arrives.
     _escPending;
@@ -173,7 +173,7 @@ export class LineEditor {
     // A callback the editor calls before every render to read the agent's
     // current task-list summary ("tasks: 2/5"), empty when there is no list.
     onTasksQuery;
-    // Extra status text produced by an external command (N37 statusLine hook).
+    // Extra status text produced by an external command (the statusLine hook).
     // Set via setStatusLine(); shown before the queue/tasks badges. Kept as a
     // plain field because the CALLER runs the command on its own timer (a
     // per-render spawn would block typing).
@@ -361,7 +361,7 @@ export class LineEditor {
             return '';
         }
     }
-    // The statusLine hook text (N37), shown before the queue/tasks badges.
+    // The statusLine hook text, shown before the queue/tasks badges.
     // A trailing space separates it from whatever follows; empty hides it.
     _statusLineBadge() {
         return this.statusLine ? theme.system(this.statusLine) + ' ' : '';
@@ -605,7 +605,7 @@ export class LineEditor {
     }
     // Update the context window size (from config.ui.contextLimit). The next
     // render recomputes the percentage and its color with the new limit.
-    // Set the statusLine hook text (N37). The caller runs the command on its
+    // Set the statusLine hook text. The caller runs the command on its
     // own timer; this only stores + repaints. An empty string hides the badge.
     setStatusLine(text) {
         const t = String(text || '').trim();
@@ -644,7 +644,6 @@ export class LineEditor {
         this.pastes = [];
         this._render();
     }
-    // ---------- rendering ----------
     _eraseBlock() {
         if (!this.rendered)
             return;
@@ -936,7 +935,6 @@ export class LineEditor {
         process.stdout.write(String(text) + NL);
         this._writeBlock();
     }
-    // ---------- spinner-compatible interface ----------
     setStatus(text) {
         this.statusText = text || '';
         this._render();
@@ -1124,7 +1122,6 @@ export class LineEditor {
         const wrapped = wrapToWidth('⚠ ' + msg, maxW);
         this.printAbove(NL + theme.warn(wrapped));
     }
-    // ---------- input ----------
     _submit() {
         if (this.locked) {
             this._render();
@@ -1367,7 +1364,7 @@ export class LineEditor {
     }
     // Bare ESC: wait a moment for a possible control-sequence tail before firing
     // onEscape. Without this a sequence split across two reads (ESC, then "[A")
-    // looked like a lone Escape and aborted the current generation (N29).
+    // looked like a lone Escape and aborted the current generation.
     _scheduleEscape() {
         this._escPending = true;
         if (this._escTimer)
@@ -1460,7 +1457,6 @@ export class LineEditor {
         chars.splice(this.cursor, i - this.cursor);
         this.buf = chars.join('');
     }
-    // ---------- history ----------
     _historyUp() {
         if (!this.history.length)
             return;
@@ -1485,7 +1481,6 @@ export class LineEditor {
         }
         this.cursor = Array.from(this.buf).length;
     }
-    // ---------- reverse-search (Ctrl+R) ----------
     // Start an incremental reverse search, keeping the current buffer aside so
     // Esc restores it (bash behavior). An empty history leaves the editor
     // untouched.
@@ -1596,7 +1591,7 @@ export class LineEditor {
         // A previous read ended on a bare ESC. That is ambiguous: the Escape key
         // or the head of a control sequence split across reads (arrows, Ctrl+Del).
         // Reattach the ESC and parse the whole sequence, so a split arrow does not
-        // abort the generation (N29).
+        // abort the generation.
         if (this._escPending) {
             this._escPending = false;
             if (this._escTimer) {

package/dist/markdown.js

@@ -15,7 +15,6 @@ const theme = {
     tableHeader: { color: '#9fb8d8', bold: true },
     tableCell: { color: '#d0d0d0' },
 };
-// Code highlighting via cli-highlight (already present transitively).
 function highlighter(code, lang) {
     try {
         return highlight(code, {

package/dist/mcp.js

@@ -477,11 +477,8 @@ export async function createMcpPool(opts) {
         }
     }
     // Distinct servers can produce the SAME qualified name (server 'a' + tool
-    // 'b__c' vs server 'a__b' + tool 'c'), and agent-loop resolves a call with
-    // tools.find(name) — it takes the FIRST. A duplicate would make the second
-    // tool silently unreachable (the model calls "the right name", a DIFFERENT
-    // tool runs). Rename the later one with a numeric suffix so every name is
-    // unique.
+    // 'b__c' vs server 'a__b' + tool 'c'); dedupeToolNames below explains why
+    // that has to be fixed rather than left to tools.find().
     dedupeToolNames(tools);
     let closed = false;
     return {

package/dist/net-capture.js

@@ -259,7 +259,7 @@ export function isFinishedWithAnswer(body) {
         return false;
     // A truncated RESPONSE head (a lone `{`) is NOT a complete answer: the
     // finish loop would return it as the model's text and the tool-call parser
-    // would fail on it. Require real content (see MIN_FINISHED_ANSWER_CHARS).
+    // would fail on it. Require real content (see responseTextLength).
     return responseContentLength(b) >= MIN_REAL_ANSWER_CHARS;
 }
 // The CONTEXT size (in tokens) DeepSeek reports for the current answer.

package/dist/path-token.js

@@ -118,15 +118,11 @@ export function extractAtFileRefs(text) {
         }
         let j = i + 1;
         let tok = '';
-        // The token is rejected WHOLE when it contains an at-sign that is NOT a
-        // list separator — `@Component` / `user@host` are a decorator and an
-        // email. Trimming the prefix at that at-sign would otherwise inline a
-        // bogus path.
-        let hadInnerAt = false;
         // A token that contains an at-sign (`@Component`, `user@host`) is a
-        // decorator / an email, NOT a file reference. The scan used to stop at
-        // that at-sign and hand the PREFIX (`Component` … no) or, worse, the part
-        // BEFORE it to the inliner. Now the whole token is rejected instead.
+        // decorator or an email, NOT a file reference, and the WHOLE token is
+        // rejected: the scan used to stop at that at-sign and hand the part
+        // BEFORE it to the inliner, which inlined a bogus path.
+        let hadInnerAt = false;
         const isRef = (t) => t.length > 0 && !t.includes('@');
         while (j < n && !isRefEnd(chars[j])) {
             // A COMMA directly followed by another `@` starts the NEXT reference:
@@ -154,12 +150,9 @@ export function extractAtFileRefs(text) {
             seen.add(cleaned);
             out.push(cleaned);
         }
-        // The break above leaves `j` ON the comma; step past it so the next `@`
-        // is seen as the start of a reference. When the scan ended on a
-        // terminator instead, continue AFTER it.
-        // The scan stopped ON a separator that split the token (`,`, `;` or a
-        // second at-sign): resume AT it so the next part is scanned as its own
-        // reference. Continuing past it dropped the second file of a list.
+        // The scan may have stopped ON a separator that split the token (`,`, `;`
+        // or a second at-sign): resume AT it so the next part is scanned as its
+        // own reference. Continuing past it dropped the second file of a list.
         const sep = j < n ? chars[j] : '';
         if (j > i + 1 && (sep === ',' || sep === ';' || sep === '@'))
             i = j;

package/dist/prompt.js

@@ -31,7 +31,8 @@ export async function promptOnce(question) {
     const wasRaw = stdin.isRaw;
     stdin.setRawMode(true);
     stdin.resume();
-    // We enable/disable bracketed paste in pairs.
+    // Bracketed paste is enabled/disabled in pairs, so the terminal is left in
+    // normal mode however the prompt exits.
     stdout.write('\x1b[?2004h');
     stdout.write(question);
     let line = '';
@@ -40,11 +41,11 @@ export async function promptOnce(question) {
     const PASTE_START = '\x1b[200~';
     const PASTE_END = '\x1b[201~';
     const redraw = () => {
-        // Return to the start of the line, erase and print again.
+        // Redraw in place: CR + erase-to-end, then the prompt and the line again.
         stdout.write(String.fromCharCode(13));
         stdout.write('\x1b[K');
         stdout.write(question + line);
-        // Put the cursor in the right position.
+        // Step back to the cursor, measured in UTF-16 units.
         const back = line.length - cursor;
         if (back > 0)
             stdout.write('\x1b[' + back + 'D');

package/dist/qwen-browser.js

@@ -6,7 +6,7 @@
 // and the chat-id scheme. DeepSeekBrowser is already large; folding a second
 // site into it would make both harder to audit. The shared surface is the
 // ChatProvider interface (types.ts); the agent loop only needs BrowserLike and
-// never sees either concrete class.
+// never sees a concrete class.
 //
 // Verified against a live session (headless Chromium, guest + logged in):
 //   input  -> textarea.message-input-textarea
@@ -233,7 +233,6 @@ export class QwenBrowser {
     sendIntervalMs() {
         return this.minSendIntervalMs;
     }
-    // ---------- lifecycle ----------
     async launch() {
         if (await profileLooksLocked(QWEN_USER_DATA_DIR)) {
             if (this.debug)
@@ -487,7 +486,6 @@ export class QwenBrowser {
         }
         catch { }
     }
-    // ---------- login ----------
     // Are we signed in? Detected from the PAGE, the same way a human sees it: a
     // signed-out visitor has the header "Log in" / "Sign up" buttons, a signed-in
     // one shows the account menu instead. The `auths/refresh` API was the first
@@ -1005,7 +1003,6 @@ export class QwenBrowser {
         }
         return false;
     }
-    // ---------- chat management ----------
     async newChat() {
         const before = await this.getCurrentChatId();
         const btn = this.page.locator(NEW_CHAT_SELECTOR).first();
@@ -1272,7 +1269,6 @@ export class QwenBrowser {
     getLastTokenUsage() {
         return this._lastTokenUsage;
     }
-    // ---------- answer reading ----------
     async _readLastAnswerText() {
         if (this._netCapture && this._netCaptureAt >= this._lastSentAt) {
             return this._netCapture;
@@ -1346,7 +1342,6 @@ export class QwenBrowser {
         catch { }
         return false;
     }
-    // ---------- toggles (model thinking / search) ----------
     getToggleStatesSync() {
         return this._toggles;
     }
@@ -1523,7 +1518,6 @@ export class QwenBrowser {
         }
         catch { }
     }
-    // ---------- diagnostics ----------
     async dumpDom(filePath) {
         if (!this.page)
             throw new Error('qwen: browser not launched');
@@ -1566,7 +1560,6 @@ export class QwenBrowser {
     chatUrl(id) {
         return chatUrl(id);
     }
-    // ---------- ask ----------
     async _waitForSendSlot(agent) {
         if (!agent)
             return;

package/dist/qwen-chats.js

@@ -84,7 +84,6 @@ export function scrapeQwenChatMessages(opts) {
     }
     return out;
 }
-// ---------- API payload parsers (pure, unit-tested) ----------
 /**
  * Parse the chat list from `GET /api/v2/chats/?page=1&exclude_project=true`.
  * The shape has drifted between builds, so accept several:

package/dist/qwen-net-capture.js

@@ -223,13 +223,10 @@ export function isFinishedWithoutAnswer(body) {
     return extractFromSse(b) === '';
 }
 // True when the body carries Qwen's rate-limit / busy notice, so ask() can
-// back off instead of waiting out the deadline. Matched against the RAW body.
-// The classifier runs against the RAW body, so a phrase like `please try again
-// later` would also match a NORMAL answer (the model writing about retry
-// logic) and make ask() back off on a good turn. A real notice arrives in an
-// ERROR payload (`event: error` / `"type":"error"`) or as a code; answer text
-// lives in `choices[].delta.content`. We therefore only inspect the body when
-// it is NOT a plain answer stream, or when it carries no answer content.
+// back off instead of waiting out the deadline. The classifier runs against the
+// RAW body, where a phrase like `please try again later` also matches a NORMAL
+// answer (the model writing about retry logic) and made ask() back off on a
+// good turn. So it only counts an ERROR payload or a body with NO answer text.
 export function isRateLimitText(body) {
     const b = String(body || '');
     if (!b)

package/dist/qwen-ui.js

@@ -174,9 +174,6 @@ export const STATUS_RE = /^(thinking|searching|generating|thinking completed|о
 // Qwen's rate-limit / busy notices (best-effort; the toast may be short-lived).
 export const RATE_LIMIT_RE = /too many requests|rate limit|too frequent|please try again later|слишком часто/i;
 export const SERVER_BUSY_RE = /server is busy|service unavailable|try again later/i;
-// ---------- toggles ----------
-// The "Auto" thinking/search toggle and the mode selector. Read-only probing
-// support; the model's thinking mode is set via config when supported.
 // ---------- toggles (thinking mode / web search) ----------
 // Qwen exposes thinking as a MODE dropdown (`.qwen-thinking-selector`, label
 // "Auto" by default) with options Auto / Thinking / Fast — NOT a boolean

package/dist/run-task.js

@@ -5,7 +5,7 @@ import { formatCompactTokens, pasteReplacement, expandPastes } from './input.js'
 import { DEFAULT_LOCALE } from './i18n.js';
 import { providerAskDeadlineMs } from './config.js';
 // Ask the operator to approve a tool call flagged by an `ask` permission rule
-// (C1). While the LineEditor owns the terminal it must be paused first, or the
+// While the LineEditor owns the terminal it must be paused first, or the
 // confirmation prompt and the input line fight for the same keys. A non-TTY run
 // has no way to answer, so it DENIES (never silently allows a guarded call).
 export async function askOperatorConfirm(editor, question, t, 
@@ -330,9 +330,9 @@ export async function runTask(deps, browser, tools, taskText, workdir, opts, att
         });
     try {
         // Set when the loop ends without a real answer (iteration limit / watchdog)
-        // so the one-shot mode can exit non-zero. See N5.
+        // so the one-shot mode can exit non-zero.
         let failure = null;
-        // N31: the machine-readable reason for the last task's outcome.
+        // The machine-readable reason for the last task's outcome.
         let outcomeKind = 'ok';
         // How many auto-continue turns a completion goal has already spent.
         let goalTurns = 0;
@@ -351,7 +351,7 @@ export async function runTask(deps, browser, tools, taskText, workdir, opts, att
         // Calling it here made the spinner run for the whole pre-send phase (chat
         // creation, throttle wait) with no generation in flight.
         while (true) {
-            // T7: per-task summary (duration + number of tool calls + tokens).
+            // Per-task summary: duration, number of tool calls and tokens spent.
             const taskStart = Date.now();
             // Token delta for THIS task: accumulated_token_usage is cumulative for
             // the whole chat, so the per-task spend is after - before. Null when the
@@ -389,7 +389,7 @@ export async function runTask(deps, browser, tools, taskText, workdir, opts, att
                 },
                 onWarning: (msg) => ui.warning(msg),
                 onAskPermission: async (info) => {
-                    // C1: a rule with action "ask". Show the reason and wait for the
+                    // A rule with action "ask". Show the reason and wait for the
                     // operator. In non-TTY this denies.
                     ui.warning(t('perm.ask', { tool: info.tool, reason: info.reason }));
                     const ok = await askOperatorConfirm(editor || null, t('perm.confirm'), t, locale);
@@ -440,7 +440,7 @@ export async function runTask(deps, browser, tools, taskText, workdir, opts, att
                         }
                     }
                     : onAskUser,
-                // N38: the check command appended to a tool result after an edit
+                // The check command appended to a tool result after an edit
                 // (off by default; config.diagnostics).
                 diagnostics: deps.config.diagnostics,
             });
@@ -452,7 +452,7 @@ export async function runTask(deps, browser, tools, taskText, workdir, opts, att
                 ui.warning(outcome);
                 transcript?.log('agent_no_answer', { outcome });
                 failure = outcome;
-                // N31: a machine-readable reason so `--output-format jsonl` consumers
+                // A machine-readable reason so `--output-format jsonl` consumers
                 // (CI) can tell a real error from an exhausted limit / watchdog.
                 outcomeKind = outcome.startsWith('Iteration limit reached')
                     ? 'iteration_limit'
@@ -461,8 +461,8 @@ export async function runTask(deps, browser, tools, taskText, workdir, opts, att
             else {
                 outcomeKind = 'ok';
             }
-            // T7: one summary line per task (duration + tool calls + tokens spent).
-            // Printed even on an abort, so the operator sees what happened.
+            // One summary line per task, printed even on an abort, so the operator
+            // sees what happened.
             const tokensAfter = getTokenUsage ? getTokenUsage() : null;
             const tokenDelta = typeof tokensBefore === 'number' && typeof tokensAfter === 'number'
                 ? Math.max(0, tokensAfter - tokensBefore)
@@ -570,7 +570,7 @@ export async function runTask(deps, browser, tools, taskText, workdir, opts, att
                 sendSystemPrompt: false,
             };
         }
-        // Stop hook (N32): the whole task (including drained queue messages) is
+        // Stop hook: the whole task (including drained queue messages) is
         // done. A cleanup/notification hook runs here. Best-effort.
         try {
             // LOAD the config: `undefined` here made every Stop hook a no-op — the
@@ -584,7 +584,7 @@ export async function runTask(deps, browser, tools, taskText, workdir, opts, att
                 ui.warning(out);
         }
         catch { }
-        // N31: a single terminal event carrying the reason, so a jsonl consumer
+        // A single terminal event carrying the reason, so a jsonl consumer
         // does not have to guess from the exit code alone.
         transcript?.log('task_result', {
             ok: !failure,

package/dist/self-review.js

@@ -122,7 +122,6 @@ async function copyContextFiles(srcDir, snapDir) {
         }
     }
 }
-// ---------- /self-review ----------
 export async function selfReview({ browser, focus, transcript, locale, }) {
     const t = translator(locale);
     const stamp = new Date().toISOString().replace(/[:.]/g, '-');
@@ -190,7 +189,6 @@ export async function selfReview({ browser, focus, transcript, locale, }) {
     console.log();
     return { snapDir, reportPath, changed };
 }
-// ---------- /self-diff ----------
 export async function selfDiff({ name, locale, }) {
     const t = translator(locale);
     const snapDir = path.join(SNAP_ROOT, name);
@@ -218,7 +216,6 @@ export async function selfDiff({ name, locale, }) {
         console.log(theme.system(t('self.no_diff')));
     }
 }
-// ---------- /self-apply ----------
 export async function selfApply({ name, locale, }) {
     const t = translator(locale);
     const snapDir = path.join(SNAP_ROOT, name);
@@ -247,7 +244,6 @@ export async function selfApply({ name, locale, }) {
     console.log(theme.system(t('self.backup', { dir: backupDir })));
     console.log(theme.warn('\n' + t('self.restart_hint') + '\n'));
 }
-// ---------- /self-list ----------
 export async function selfList({ locale, } = {}) {
     const t = translator(locale);
     const snapRoot = path.join(SNAP_ROOT);
@@ -285,7 +281,6 @@ export async function selfList({ locale, } = {}) {
     console.log();
     console.log(theme.system(t('self.list_commands') + '\n'));
 }
-// ---------- helpers ----------
 async function diffFiles(origDir, newDir) {
     const files = await listTsFiles(newDir).catch(() => []);
     const changed = [];