@swmansion/argent 0.22.1-next.2 → 0.22.1-next.4

package/dist/installer.mjs

@@ -10916,9 +10916,7 @@ var require_min_release_age = __commonJS({
     var PROBE_TIMEOUT_MS2 = 3e3;
     var OVERRIDE_ENV = "ARGENT_MIN_RELEASE_AGE_DAYS";
     var PM_PROBES = {
-      // npm flattens `min-release-age` to an effective `before` cutoff, and some
-      // npm 11.x builds report `min-release-age=null` even while the policy is
-      // active. Probe `before`, which is what the resolver actually uses.
+      // Probe `before`: that is the effective cutoff npm's resolver applies.
       npm: { command: "npm config get before", parse: config_parse_1.parseBeforeAgeMs },
       pnpm: {
         command: "pnpm config get minimumReleaseAge",
@@ -15272,17 +15270,16 @@ function createExporter(config) {
       // The exporter bounds a request with `req.setTimeout()`, which Node only
       // arms once the socket is CONNECTED — so a collector whose address drops
       // packets (corporate egress filter, dead host behind a firewall) leaves a
-      // socket stuck in the connecting state that no export deadline can reach,
-      // holding the process open for the OS connect timeout (~75s on macOS)
-      // after shutdown() has already resolved. The agent's socket timeout is
-      // armed when the socket is CREATED, so it also covers connect: it fires,
-      // the request emits 'timeout', and the exporter's handler destroys it.
+      // socket stuck connecting that no export deadline can reach, holding the
+      // process open for the OS connect timeout after shutdown() has resolved.
+      // The agent's socket timeout is armed when the socket is CREATED, so it
+      // covers connect too: it fires, the request emits 'timeout', and the
+      // exporter's handler destroys it.
       //
       // keepAlive is restated because supplying httpAgentOptions at all
-      // replaces the agent the SDK would otherwise build, and its default is
-      // keepAlive: true. Without it the long-lived tool-server pays a fresh
-      // TCP+TLS handshake for every 10s batch — measured as one socket per
-      // request against a loopback collector, versus one socket shared.
+      // replaces the agent the SDK would otherwise build, whose default is
+      // keepAlive: true; without it the long-lived tool-server pays a fresh
+      // TCP+TLS handshake for every batch.
       httpAgentOptions: { timeout: EXPORT_TIMEOUT_MS, keepAlive: true }
     });
   } finally {
@@ -15564,15 +15561,12 @@ var FAILURE_CODES = {
   NATIVE_PROFILER_SESSION_TORN_DOWN: "NATIVE_PROFILER_SESSION_TORN_DOWN",
   NATIVE_PROFILER_APP_PROCESS_NOT_FOUND: "NATIVE_PROFILER_APP_PROCESS_NOT_FOUND",
   NATIVE_PROFILER_NO_EXPORTED_TRACE: "NATIVE_PROFILER_NO_EXPORTED_TRACE",
-  // Android perfetto start-failure modes — mirror the iOS xctrace set so a
-  // failed recording start is classified rather than falling through to the
-  // generic tool-execution bucket.
+  // Android perfetto start failures, mirroring the iOS xctrace set above.
   NATIVE_PROFILER_PERFETTO_PROCESS_ERROR: "NATIVE_PROFILER_PERFETTO_PROCESS_ERROR",
   NATIVE_PROFILER_PERFETTO_READY_TIMEOUT: "NATIVE_PROFILER_PERFETTO_READY_TIMEOUT",
   NATIVE_PROFILER_PERFETTO_READY_EXITED: "NATIVE_PROFILER_PERFETTO_READY_EXITED",
-  // screen-recording-start / screen-recording-stop. One capture path for every
-  // platform (simulator-server's frame stream into ffmpeg), so the stages name
-  // the step that failed rather than the device family.
+  // iOS and Android share one capture path (simulator-server frames into
+  // ffmpeg), so these name the failing stage, not the device family.
   SCREEN_RECORDING_FACTORY_OPTIONS_MISSING: "SCREEN_RECORDING_FACTORY_OPTIONS_MISSING",
   SCREEN_RECORDING_WRONG_PLATFORM: "SCREEN_RECORDING_WRONG_PLATFORM",
   SCREEN_RECORDING_ALREADY_ACTIVE: "SCREEN_RECORDING_ALREADY_ACTIVE",
@@ -15823,7 +15817,6 @@ var ALLOWED = {
     install_mode: INSTALL_MODE
   },
   "installation:global_install_decision": {
-    // `from_tar` is intentionally absent; the installer skips that dev path.
     decision: oneOf(["install", "cancel", "already_installed"])
   },
   "installation:update_decision": {
@@ -15902,9 +15895,8 @@ var ALLOWED = {
     tool_invocation_id: UUID,
     platform: PLATFORM,
     duration_ms: DURATION_MS,
-    // Schema-declared parameter names only (emit side filters against the tool's
-    // own zod shape and caps at 16 before this gate); the array validator also
-    // voids anything longer or with a non-identifier element.
+    // Emit side sends only names declared in the tool's zod shape, capped at 16
+    // because arrayOf voids the whole array once it is longer.
     invalid_params: arrayOf(matches(/^[a-z][a-z0-9_]{0,63}$/i, 64), 16),
     ...FAILURE_SIGNAL,
     ...AI_TELEMETRY
@@ -16436,7 +16428,7 @@ var _CI_VENDOR_COUNT_FOR_TEST = vendors_default.length;
 var SESSION_ID = randomUUID4();
 function readCliVersion() {
   if (true) {
-    return "0.22.1-next.2";
+    return "0.22.1-next.4";
   }
   return "0.0.0";
 }
@@ -16749,15 +16741,11 @@ function resolveHostFingerprint() {
     const out = execFileSync(simulatorServerBinaryPath(), ["fingerprint"], {
       encoding: "utf8",
       timeout: FINGERPRINT_TIMEOUT_MS,
-      // Reap with SIGKILL, not the default SIGTERM: a binary that ignores SIGTERM
-      // would otherwise block the (synchronous) event loop past the cap forever.
-      // SIGKILL can't be trapped, so the timeout is a genuine bound here.
+      // SIGKILL, not the default SIGTERM: a binary that ignores SIGTERM would
+      // otherwise block the (synchronous) event loop past the cap forever.
       killSignal: "SIGKILL",
-      // Cap captured stdout (same limit as the async variant): a binary streaming
-      // output is SIGKILL'd at this cap rather than filling Node's 1 MiB default.
       maxBuffer: FINGERPRINT_MAX_BUFFER,
-      // Ignore stderr so a binary that logs diagnostics doesn't pollute the
-      // caller's stderr; stdout (index 1) is captured as the return value.
+      // Ignore stderr so the binary's diagnostics don't pollute the caller's.
       stdio: ["ignore", "pipe", "ignore"]
     }).trim();
     return out.length > 0 ? out : null;
@@ -20336,12 +20324,8 @@ var cursorAdapter = {
   globalPath() {
     return path16.join(homedir4(), ".cursor", "mcp.json");
   },
-  // Cursor is a VS Code fork: .cursor/mcp.json is JSONC (line/block comments,
-  // trailing commas). Routing write/remove/hasArgentEntry through readJsonc /
-  // editJsoncFile applies path-targeted text edits that preserve comments and
-  // foreign servers. The old readJson → writeJson path ran commented files
-  // through readJson's `catch { return {} }`, then persisted only the argent
-  // entry — destroying every pre-existing server and comment. Matches VS Code.
+  // Read and written as JSONC through readJsonc / editJsoncFile: path-targeted
+  // text edits that preserve user comments and foreign servers.
   write(configPath, entry) {
     editJsoncFile(configPath, ["mcpServers", MCP_SERVER_KEY], {
       command: entry.command,
@@ -20366,14 +20350,8 @@ var cursorAdapter = {
   hasArgentEntry(configPath) {
     return this.getArgentEntry(configPath) !== null;
   },
-  // Cursor's allowlist lives in a separate ~/.cursor/permissions.json, which —
-  // like every Cursor config — is JSONC (comments, trailing commas) and may
-  // carry the user's own unrelated rules. Route through readJsonc / editJsoncFile
-  // so a hand-commented or multi-rule permissions.json survives: the old readJson
-  // path ran it through `catch { return {} }` and rewrote the whole file with
-  // only { mcpAllowlist }, dropping every foreign rule and comment. Newly matters
-  // because `hasArgentEntry` now reads mcp.json with readJsonc, so `update`
-  // detects a commented config as configured and calls this.
+  // The allowlist lives in a separate ~/.cursor/permissions.json that may carry
+  // the user's own rules, so edit that one key instead of rewriting the file.
   addAllowlist() {
     const permPath = path16.join(homedir4(), ".cursor", "permissions.json");
     const config = readJsonc(permPath);
@@ -20411,10 +20389,9 @@ function claudeDisabledListFinding(settingsPath, label, projectConfined) {
     location: label,
     reason: projectConfined ? `a recorded "reject" in disabledMcpjsonServers blocks the .mcp.json entry from loading` : `a machine-wide "reject" in disabledMcpjsonServers blocks .mcp.json argent entries in every project; if that is not deliberate, remove "argent" from the list`,
     entry: null,
-    // Removing a PROJECT-confined rejection only lets Claude Code prompt for
-    // approval again — running `argent init` is that consent. The user-global
-    // list (~/.claude/settings.json) reaches every project on the machine, so
-    // it is never auto-removed; the shared policy warns instead.
+    // Removing a PROJECT-confined rejection only re-enables the approval prompt
+    // — running `argent init` is that consent. The user-global list reaches
+    // every project on the machine, so it is never auto-removed.
     autoRemove: projectConfined,
     remove: () => {
       const config = readJson(settingsPath);
@@ -20441,9 +20418,8 @@ var claudeAdapter = {
   globalPath() {
     return path16.join(homedir4(), ".claude.json");
   },
-  // JSONC is a superset of JSON, so routing through readJsonc / editJsoncFile is
-  // safe for this strict-JSON config and keeps every MCP-entry write on the one
-  // comment- and foreign-server-preserving path (see the Cursor adapter).
+  // JSONC is a superset of JSON, so this config shares the one comment- and
+  // foreign-server-preserving write path (see the Cursor adapter).
   write(configPath, entry) {
     editJsoncFile(configPath, ["mcpServers", MCP_SERVER_KEY], {
       type: "stdio",
@@ -20475,12 +20451,9 @@ var claudeAdapter = {
   hasArgentEntry(configPath) {
     return this.getArgentEntry(configPath) !== null;
   },
-  // Claude Code's scope precedence is local > project (.mcp.json) > user
-  // (~/.claude.json top-level). "Local scope" lives outside the
-  // projectPath/globalPath pair: projects["<abs path>"].mcpServers in
-  // ~/.claude.json (the default target of `claude mcp add`). A stale entry
-  // there outranks BOTH scopes init can write — no tools, no error. Also
-  // reports recorded .mcp.json rejections (see claudeDisabledListFinding).
+  // Scans the scope outside the projectPath/globalPath pair —
+  // projects["<abs path>"].mcpServers in ~/.claude.json — plus recorded
+  // .mcp.json rejections (see claudeDisabledListFinding).
   findShadowingConfigs(root, writtenScope) {
     const findings = [];
     const claudeJsonPath = path16.join(homedir4(), ".claude.json");
@@ -20549,13 +20522,8 @@ var vscodeAdapter = {
   globalPath() {
     return null;
   },
-  // .vscode/mcp.json is JSONC — VS Code allows line/block comments and trailing
-  // commas. The previous JSON.parse → mutate → JSON.stringify path ran through
-  // readJson, whose `catch { return {} }` turned any commented file into {} and
-  // then persisted only { servers: { argent } }, destroying every pre-existing
-  // user server (and their comments). All four entry points now go through
-  // readJsonc / editJsoncFile — path-targeted text edits that preserve comments
-  // and foreign servers — matching the Zed and opencode adapters.
+  // Read and written as JSONC through readJsonc / editJsoncFile so user comments
+  // and foreign servers survive (see the Cursor adapter).
   write(configPath, entry) {
     editJsoncFile(configPath, ["servers", MCP_SERVER_KEY], {
       type: "stdio",
@@ -20581,11 +20549,8 @@ var vscodeAdapter = {
   hasArgentEntry(configPath) {
     return this.getArgentEntry(configPath) !== null;
   },
-  // The user-profile mcp.json is a scope the projectPath/globalPath pair
-  // doesn't cover. User-vs-workspace precedence is undocumented (observed:
-  // silent single-winner), so a stale entry there can shadow a fresh
-  // .vscode/mcp.json entry. Report it; the shared policy removes it only when
-  // provably dead.
+  // The user-profile mcp.json is a scope the projectPath/globalPath pair doesn't
+  // cover. Report it; the shared policy removes it only when provably dead.
   findShadowingConfigs(_root, _writtenScope) {
     const findings = [];
     for (const userDir of vscodeUserDirs()) {
@@ -20635,8 +20600,7 @@ var windsurfAdapter = {
   globalPath() {
     return path16.join(homedir4(), ".codeium", "windsurf", "mcp_config.json");
   },
-  // JSONC-safe MCP-entry writes (see the Cursor adapter): editJsoncFile
-  // preserves comments and pre-existing foreign servers on this JSON config.
+  // JSONC-safe writes (see the Cursor adapter).
   write(configPath, entry) {
     editJsoncFile(configPath, ["mcpServers", MCP_SERVER_KEY], {
       command: entry.command,
@@ -20661,11 +20625,8 @@ var windsurfAdapter = {
   hasArgentEntry(configPath) {
     return this.getArgentEntry(configPath) !== null;
   },
-  // JSONC-safe allowlist edits (see the Cursor adapter): the argent entry lives
-  // in this same mcp_config.json, and `init` runs write() (comment-preserving)
-  // before addAllowlist(). The old readJson path choked on any user comment and
-  // silently skipped the toggle; editJsoncFile targets just the argent entry's
-  // alwaysAllow key so comments and foreign servers survive.
+  // Targets just the argent entry's alwaysAllow key, so comments and foreign
+  // servers survive.
   addAllowlist() {
     const configPath = path16.join(homedir4(), ".codeium", "windsurf", "mcp_config.json");
     const config = readJsonc(configPath);
@@ -20695,11 +20656,8 @@ var zedAdapter = {
   globalPath() {
     return path16.join(homedir4(), ".config", "zed", "settings.json");
   },
-  // Zed's settings.json is JSONC (line + block comments, trailing commas).
-  // The previous JSON.parse → mutate → JSON.stringify path silently stripped
-  // every comment in the user's hand-edited file. All four entry points now
-  // go through editJsoncFile, which applies path-targeted text edits via
-  // jsonc-parser so comments and formatting outside the touched key survive.
+  // Read and written as JSONC through readJsonc / editJsoncFile so comments and
+  // formatting outside the touched key survive (see the Cursor adapter).
   write(configPath, entry) {
     editJsoncFile(configPath, ["context_servers", MCP_SERVER_KEY], {
       source: "custom",
@@ -20725,10 +20683,8 @@ var zedAdapter = {
   hasArgentEntry(configPath) {
     return this.getArgentEntry(configPath) !== null;
   },
-  // Zed doesn't support server-level wildcards for MCP tools — each tool
-  // would need its own entry.  Setting the global default to "allow" is the
-  // documented opt-in; built-in security rules still protect against
-  // destructive operations.
+  // Zed has no server-level wildcard for MCP tools — each tool would need its
+  // own entry — so the global default is set to "allow" instead.
   addAllowlist(root, scope) {
     const settingsPath = scope === "global" ? path16.join(homedir4(), ".config", "zed", "settings.json") : path16.join(root, ".zed", "settings.json");
     editJsoncFile(settingsPath, ["agent", "tool_permissions", "default"], "allow");
@@ -20753,8 +20709,7 @@ var geminiAdapter = {
   globalPath() {
     return path16.join(homedir4(), ".gemini", "settings.json");
   },
-  // JSONC-safe MCP-entry writes (see the Cursor adapter): editJsoncFile
-  // preserves comments and pre-existing foreign servers on this JSON config.
+  // JSONC-safe writes (see the Cursor adapter).
   write(configPath, entry) {
     editJsoncFile(configPath, ["mcpServers", MCP_SERVER_KEY], {
       command: entry.command,
@@ -20779,11 +20734,8 @@ var geminiAdapter = {
   hasArgentEntry(configPath) {
     return this.getArgentEntry(configPath) !== null;
   },
-  // JSONC-safe allowlist edits (see the Cursor adapter): the argent entry lives
-  // in this same settings.json, and `init` runs write() (comment-preserving)
-  // before addAllowlist(). The old readJson path choked on any user comment and
-  // silently skipped the toggle; editJsoncFile targets just the argent entry's
-  // trust key so comments and foreign servers survive.
+  // Targets just the argent entry's trust key, so comments and foreign servers
+  // survive.
   addAllowlist(root, scope) {
     const configPath = scope === "global" ? this.globalPath() : this.projectPath(root);
     if (!configPath) {
@@ -21020,10 +20972,8 @@ var kiroAdapter = {
   globalPath() {
     return path16.join(homedir4(), ".kiro", "settings", "mcp.json");
   },
-  // Kiro is a VS Code fork: .kiro/settings/mcp.json is JSONC. As with Cursor,
-  // route write/remove/hasArgentEntry through readJsonc / editJsoncFile so
-  // comments and foreign servers survive instead of being flattened away by the
-  // old readJson → writeJson path.
+  // Read and written as JSONC through readJsonc / editJsoncFile so comments and
+  // foreign servers survive (see the Cursor adapter).
   write(configPath, entry) {
     editJsoncFile(configPath, ["mcpServers", MCP_SERVER_KEY], {
       command: entry.command,
@@ -21048,11 +20998,8 @@ var kiroAdapter = {
   hasArgentEntry(configPath) {
     return this.getArgentEntry(configPath) !== null;
   },
-  // JSONC-safe allowlist edits (see the Cursor adapter): .kiro/settings/mcp.json
-  // is JSONC, the argent entry lives in it, and `init` runs write() (comment-
-  // preserving) before addAllowlist(). The old readJson path choked on any user
-  // comment and silently skipped the toggle; editJsoncFile targets just the
-  // argent entry's autoApprove key so comments and foreign servers survive.
+  // Targets just the argent entry's autoApprove key, so comments and foreign
+  // servers survive.
   addAllowlist(root, scope) {
     const configPath = scope === "global" ? this.globalPath() : this.projectPath(root);
     if (!configPath) return;
@@ -21468,8 +21415,7 @@ var InitTelemetry = class {
   editorsConfiguredCount = 0;
   initSucceeded = false;
   finalized = false;
-  // Drains buffered events and records the single terminal cli_init_complete.
-  // Idempotent — only the first call emits.
+  // Records the single terminal cli_init_complete, then drains the event queue.
   async finalize(failureSignal2) {
     if (this.finalized) return;
     this.finalized = true;
@@ -22547,19 +22493,16 @@ async function init2(args) {
     note(mcpLines.join("\n"), "MCP Configuration");
     const staleCleanup = await cleanupStaleMcpConfigs({
       writtenAdapters,
-      // Sweep EVERY adapter, not just the detected set: the sweep is fully
-      // gated on an existing argent entry (getArgentEntry), and the stale
-      // entries most worth pruning live in argent-only dirs (~/.cursor,
-      // ~/.codex) that detection now deliberately ignores.
+      // Every adapter, not just the detected set: the sweep only acts where an
+      // argent entry already exists, and detection ignores dirs holding nothing
+      // but argent's own artifacts (~/.cursor, ~/.codex).
       detectedAdapters: ALL_ADAPTERS,
       installMode: tel.installMode,
       scope: normalizedScope,
       effectiveRoot,
-      // Removals reaching beyond this project (dead entries in global config
-      // files) get one confirmation: the "dead" verdict is a PATH probe in
-      // THIS shell, which an nvm-style split can fool, so a human always gets
-      // the last word. Non-interactive runs pass no confirmer, making the
-      // sweep report those entries instead of removing them.
+      // Removals in global config files get one confirmation: the "dead" verdict
+      // is a PATH probe in THIS shell, which an nvm-style split can fool. No
+      // confirmer non-interactively, so the sweep reports them instead.
       confirmCrossProjectRemovals: parsed.nonInteractive ? void 0 : async (items) => {
         log.warn(
           `Dead argent entries from a previous global install were found in
@@ -23204,9 +23147,9 @@ async function update(args) {
       const { landed: reachedTarget, exitError: installError } = await runTrustingDisk(
         () => {
           execShellCommandSync(cmd, {
-            // Older versions still ship a postinstall script that honors this
-            // (current ones have none); keep it so downgrades and pinned
-            // installs of those versions stay quiet and don't double-kill.
+            // Current versions ship no postinstall, but downgrades and pinned
+            // installs fetch older ones whose postinstall honors this — keep it
+            // so they stay quiet and don't double-kill the tool server.
             env: { ...process.env, ARGENT_SKIP_POSTINSTALL: "1" },
             // Local installs must rewrite the project's manifest/lockfile.
             ...mode === "local" ? { cwd: projectRoot } : {}
@@ -23214,8 +23157,8 @@ async function update(args) {
         },
         () => {
           landedVersion = mode === "local" ? (
-            // Cache-free read: the pre-install probe memoized the old version's
-            // realpath, so getLocallyInstalledVersion would report it stale here.
+            // Cache-free: the pre-install probe memoized the old version's
+            // realpath, so getLocallyInstalledVersion would read it stale.
             readLocalPackageVersionUncached(projectRoot) ?? getLocallyInstalledVersion(projectRoot)
           ) : getGloballyInstalledVersion();
           return landedVersion !== null && !isNewerVersion(target, landedVersion);
@@ -23418,9 +23361,9 @@ async function update(args) {
         scope: localAdapters.length > 0 ? "local" : "global",
         effectiveRoot: projectRoot,
         // Same one-shot confirmation init uses for cross-project removals.
-        // --yes passes no confirmer (sweep goes report-only there) — the agent-
-        // triggered `update --yes` must never delete cross-project state on a
-        // fallible PATH probe.
+        // --yes passes no confirmer (the sweep goes report-only) — the
+        // agent-triggered `update --yes` must never delete cross-project state
+        // on a fallible PATH probe.
         confirmCrossProjectRemovals: nonInteractive ? void 0 : async (items) => {
           log.warn(
             `Dead argent entries from a previous global install were found in

package/dist/mcp-server.mjs

@@ -16589,15 +16589,12 @@ var FAILURE_CODES = {
   NATIVE_PROFILER_SESSION_TORN_DOWN: "NATIVE_PROFILER_SESSION_TORN_DOWN",
   NATIVE_PROFILER_APP_PROCESS_NOT_FOUND: "NATIVE_PROFILER_APP_PROCESS_NOT_FOUND",
   NATIVE_PROFILER_NO_EXPORTED_TRACE: "NATIVE_PROFILER_NO_EXPORTED_TRACE",
-  // Android perfetto start-failure modes — mirror the iOS xctrace set so a
-  // failed recording start is classified rather than falling through to the
-  // generic tool-execution bucket.
+  // Android perfetto start failures, mirroring the iOS xctrace set above.
   NATIVE_PROFILER_PERFETTO_PROCESS_ERROR: "NATIVE_PROFILER_PERFETTO_PROCESS_ERROR",
   NATIVE_PROFILER_PERFETTO_READY_TIMEOUT: "NATIVE_PROFILER_PERFETTO_READY_TIMEOUT",
   NATIVE_PROFILER_PERFETTO_READY_EXITED: "NATIVE_PROFILER_PERFETTO_READY_EXITED",
-  // screen-recording-start / screen-recording-stop. One capture path for every
-  // platform (simulator-server's frame stream into ffmpeg), so the stages name
-  // the step that failed rather than the device family.
+  // iOS and Android share one capture path (simulator-server frames into
+  // ffmpeg), so these name the failing stage, not the device family.
   SCREEN_RECORDING_FACTORY_OPTIONS_MISSING: "SCREEN_RECORDING_FACTORY_OPTIONS_MISSING",
   SCREEN_RECORDING_WRONG_PLATFORM: "SCREEN_RECORDING_WRONG_PLATFORM",
   SCREEN_RECORDING_ALREADY_ACTIVE: "SCREEN_RECORDING_ALREADY_ACTIVE",
@@ -17142,14 +17139,12 @@ var CONFIG_SCHEMA = [
     description: "Whether anonymous opt-out telemetry is enabled (on by default; environment opt-outs like DO_NOT_TRACK are not reflected here \u2014 `argent telemetry status` shows effective consent).",
     scopes: ["global"],
     parse: asBoolean,
-    // A committed project file must never re-enable telemetry a user disabled
-    // globally, so the more-restrictive (opt-out) value always wins.
     merge: "prioritize-restrictive",
-    // Telemetry is opt-out: with nothing stored, consent.ts treats it as
-    // enabled, and the config surface must report the same instead of "(unset)".
+    // Opt-out: consent.ts reads an unstored value as enabled, so the config
+    // surface must show the same rather than "(unset)".
     default: true,
-    // Read-only under `argent config`: opt-in/out goes through the dedicated
-    // command so the live client is drained/reset, not just the file rewritten.
+    // Opt-in/out goes through the dedicated command so the live client is
+    // drained/reset, not just the file rewritten.
     manageCommand: "argent telemetry"
   },
   {
@@ -17157,8 +17152,6 @@ var CONFIG_SCHEMA = [
     description: "Coding-agent id remembered by `argent lens` to skip the picker.",
     scopes: ["project", "global"],
     parse: asString,
-    // A repo can pin the agent its screenshots should use; falls back to the
-    // user's global remembered choice.
     merge: "prioritize-local",
     example: "claude"
   },
@@ -17167,11 +17160,9 @@ var CONFIG_SCHEMA = [
     description: "Additional CoreSimulator device-set directories whose simulators argent should see alongside the default set. Absolute paths (or ~/\u2026); relative entries resolve against the project root (project scope) or home (global scope).",
     scopes: ["project", "global"],
     parse: asStringArray,
-    // Additive: the scopes extend each other rather than shadow — a repo's
-    // committed device sets are appended to the user's global ones (global
-    // baseline first, project extras after, deduplicated). Note that
-    // `getAdditionalIosDeviceSets` re-implements this union (path resolution
-    // must precede dedup) and guards on the preset staying "union".
+    // Additive rather than shadowing: global baseline first, project extras
+    // after, deduplicated. `getAdditionalIosDeviceSets` re-implements this union
+    // (path resolution must precede dedup) and guards on the preset staying "union".
     merge: "union",
     example: '["~/DeviceSets/ci"]'
   },
@@ -17180,10 +17171,8 @@ var CONFIG_SCHEMA = [
     description: "Directory where finished screen recordings (mp4) are saved on the client host. Absolute, `~`-prefixed, or relative to the project root (home dir when not in a project). Unset \u21D2 `.argent/recordings` under the project root.",
     scopes: ["project", "global"],
     parse: asString,
-    // A repo can pin where its recordings land; falls back to the user's global
-    // preference. Resolution happens on the client (the machine the mp4 is
-    // persisted to), so with a remote `argent link` tool-server it is the
-    // *client's* config that decides.
+    // Resolved on the client (the machine the mp4 is persisted to), so with a
+    // remote `argent link` tool-server it is the *client's* config that decides.
     merge: "prioritize-local",
     example: "~/Movies/argent"
   }
@@ -17646,7 +17635,6 @@ var ALLOWED = {
     install_mode: INSTALL_MODE
   },
   "installation:global_install_decision": {
-    // `from_tar` is intentionally absent; the installer skips that dev path.
     decision: oneOf(["install", "cancel", "already_installed"])
   },
   "installation:update_decision": {
@@ -17725,9 +17713,8 @@ var ALLOWED = {
     tool_invocation_id: UUID,
     platform: PLATFORM,
     duration_ms: DURATION_MS,
-    // Schema-declared parameter names only (emit side filters against the tool's
-    // own zod shape and caps at 16 before this gate); the array validator also
-    // voids anything longer or with a non-identifier element.
+    // Emit side sends only names declared in the tool's zod shape, capped at 16
+    // because arrayOf voids the whole array once it is longer.
     invalid_params: arrayOf(matches(/^[a-z][a-z0-9_]{0,63}$/i, 64), 16),
     ...FAILURE_SIGNAL,
     ...AI_TELEMETRY
@@ -18684,9 +18671,8 @@ async function startMcpServer(options) {
     let finalArgs = args;
     if (meta2?.fileInputs?.length) {
       finalArgs = await prepareFileInputs(meta2.fileInputs, args ?? {}, {
-        // `resolved` is the startup routing decision that picked TOOLS_URL —
-        // an external target means this process may not share the server's
-        // filesystem, so file bytes must ride along.
+        // An external target may not share this process's filesystem, so the
+        // file bytes have to ride along.
         includeContent: resolved.url !== null
       });
     }

package/dist/preview-window/main.cjs

@@ -107,22 +107,18 @@ async function createWindow() {
     width: TARGET_WIDTH,
     height: TARGET_HEIGHT,
     title: "Argent Lens",
-    // Native window controls (the macOS traffic lights) instead of a
-    // handcrafted close button: `titleBarStyle: "hidden"` keeps the real OS
-    // buttons in the top-left while hiding the rest of the title bar, so the
-    // window still reads as frameless. (`frame: false` would suppress the
-    // buttons entirely — that's what we used to do.)
+    // Keeps the real macOS traffic lights in the top-left while hiding the rest
+    // of the title bar, so the window still reads as frameless. `frame: false`
+    // would suppress the buttons entirely.
     titleBarStyle: "hidden",
-    // Inset the traffic lights to share the same 14px padding the in-window
-    // corner panels use (theme toggle, status pill), so the native buttons and
-    // the handcrafted chrome read as one consistent margin. Keep x and y equal —
-    // an asymmetric inset (e.g. 14/18) reads as visibly off against the square
-    // corner panels.
+    // Same 14px inset the in-window corner panels use, so native buttons and
+    // handcrafted chrome read as one margin. Keep x and y equal — an asymmetric
+    // inset reads as visibly off against the square corner panels.
     trafficLightPosition: { x: 14, y: 14 },
     show: false,
-    // Transparent + no native shadow: the OS window is just a passthrough
-    // for the renderer. The visible "card" is whatever <html> + <body>
-    // paints, which is what we squeeze with the CSS transform.
+    // The OS window is just a passthrough for the renderer: the visible "card"
+    // is whatever <html> + <body> paints, and that is what the CSS transform
+    // squeezes.
     transparent: true,
     hasShadow: false,
     backgroundColor: "#00000000",