@phnx-labs/agents-cli 1.22.120 → 1.22.121

package/dist/commands/repo.js

@@ -13,12 +13,6 @@ import { itemPicker } from '../lib/picker.js';
 import { addHostOption } from '../lib/hosts/option.js';
 import { inspectRepo, resolveRepoTarget, } from './inspect.js';
 const HOME = os.homedir();
-/**
- * Resolve a target argument to an absolute path.
- * - No arg: ~/.agents/
- * - Looks like a path (starts with /, ~, .): resolve as path
- * - Otherwise: ~/.agents-{name}/
- */
 function resolveRepoPath(target) {
     if (!target)
         return path.join(HOME, '.agents');
@@ -36,12 +30,6 @@ import { getGlobalDefault, getVersionHomePath, listInstalledVersions } from '../
 import { syncAllMarketplaces } from '../lib/plugins/plugin-marketplace.js';
 import { gatherRemoteAgentsJson } from '../lib/remote-agents-json.js';
 import { machineId, normalizeHost } from '../lib/machine-id.js';
-/**
- * After a repo add/remove/enable/disable, reconcile each plugins-capable
- * agent's default version against the new marketplace set. Re-synthesizes
- * catalogs and known_marketplaces.json entries. Source-copy of plugins is
- * out of scope here — full sync still goes through `agents sync`.
- */
 function syncMarketplacesForDefaults() {
     for (const agent of capableAgents('plugins')) {
         const def = getGlobalDefault(agent);
@@ -52,15 +40,9 @@ function syncMarketplacesForDefaults() {
         try {
             syncAllMarketplaces(agent, getVersionHomePath(agent, def));
         }
-        catch { /* best-effort */ }
+        catch { }
     }
 }
-/**
- * Push the reserved auth bundle to peers whose last-received verdict says
- * `missing`. Daemon state (usage, verdicts, session digests) no longer rides the
- * user repo — it moves over SSH on the usage-sync tick (PHNX-4116) — so a repo
- * push/sync of human-authored resources only runs this credential half.
- */
 async function syncUserRepoAuthBundle() {
     const { syncReservedAuthBundle } = await import('../lib/secrets-policy.js');
     const auth = await syncReservedAuthBundle();
@@ -70,7 +52,6 @@ async function syncUserRepoAuthBundle() {
         console.error(chalk.yellow(`Auth bundle: ${err.device}: ${err.message}`));
 }
 const ALIAS_PATTERN = /^[a-zA-Z0-9][a-zA-Z0-9_-]{0,63}$/;
-/** Derive a default alias from a source URL (e.g. gh:foo/.agents-work -> agents-work). */
 function deriveAlias(source) {
     const parsed = parseSource(source);
     let base;
@@ -81,18 +62,14 @@ function deriveAlias(source) {
         const match = parsed.url.match(/github\.com\/([^/]+)\/([^/]+?)(?:\.git)?$/);
         base = match ? match[2] : parsed.url;
     }
-    // Strip leading dots and any "agents-" prefix so the alias is short and
-    // becomes ~/.agents-<alias>/ on disk (e.g. ".agents-work" -> "work").
     return base.replace(/^\.+/, '').replace(/^agents-/, '') || 'repo';
 }
-/** Top-level dir -> resource kind. Directory-based resources collapse to one unit. */
 const RESOURCE_DIRS = {
     skills: 'skill', commands: 'command', prompts: 'command', plugins: 'plugin',
     hooks: 'hook', mcp: 'mcp', subagents: 'subagent', rules: 'rule',
     workflows: 'workflow', routines: 'routine', profiles: 'profile',
     permissions: 'permission', clis: 'clis',
 };
-/** [singular, plural] display labels per kind. */
 const RESOURCE_LABELS = {
     skill: ['skill', 'skills'], command: ['command', 'commands'],
     plugin: ['plugin', 'plugins'], hook: ['hook', 'hooks'], mcp: ['MCP', 'MCPs'],
@@ -102,16 +79,10 @@ const RESOURCE_LABELS = {
     clis: ['CLI', 'CLIs'], config: ['config file', 'config files'],
     other: ['other file', 'other files'],
 };
-/** Display order — the resources a user cares about most come first. */
 const RESOURCE_ORDER = [
     'skill', 'command', 'plugin', 'hook', 'mcp', 'subagent', 'rule',
     'workflow', 'routine', 'profile', 'permission', 'clis', 'config', 'other',
 ];
-/**
- * Map a repo-relative path to the resource unit it belongs to. Directory-based
- * resources (skills/foo/SKILL.md) collapse to `skills/foo` so all their files
- * count as one unit; flat config files (agents.yaml, hooks.yaml) count alone.
- */
 export function resourceUnit(file) {
     const parts = file.split('/');
     const top = parts[0];
@@ -122,14 +93,7 @@ export function resourceUnit(file) {
         return { kind, unit: parts.length > 1 ? `${top}/${parts[1]}` : file };
     return { kind: 'other', unit: file };
 }
-/**
- * Collapse a changed-file set to distinct resource units, then count them by
- * (action, kind). A skill whose three files all changed counts as one changed
- * skill; a unit with mixed add+modify reads as a single change. This is the
- * shared structured core behind every SYNC / CHANGES rendering.
- */
 export function resourceDelta(entries) {
-    // Gather every action seen across a unit's files, then collapse to one action.
     const units = new Map();
     for (const { action, file } of entries) {
         const { kind, unit } = resourceUnit(file);
@@ -138,7 +102,7 @@ export function resourceDelta(entries) {
         cur.actions.add(action);
         units.set(key, cur);
     }
-    const counts = new Map(); // `${action} ${kind}` -> count
+    const counts = new Map();
     for (const { kind, actions } of units.values()) {
         let action;
         if (actions.size === 1)
@@ -148,7 +112,7 @@ export function resourceDelta(entries) {
         else if ([...actions].every((a) => a === 'removed'))
             action = 'removed';
         else
-            action = 'changed'; // mixed add+modify within one unit reads as a change
+            action = 'changed';
         const key = `${action} ${kind}`;
         counts.set(key, (counts.get(key) ?? 0) + 1);
     }
@@ -165,14 +129,12 @@ export function resourceDelta(entries) {
 const ACTION_COLOR = {
     new: chalk.green, changed: chalk.yellow, removed: chalk.red,
 };
-/** One colored phrase per counted resource, e.g. `2 new skills`. */
 function deltaPhrases(delta) {
     return delta.counts.map(({ action, kind, count }) => {
         const [singular, plural] = RESOURCE_LABELS[kind];
         return ACTION_COLOR[action](`${count} ${action} ${count === 1 ? singular : plural}`);
     });
 }
-/** Join phrases with `, `, capping at `maxParts` and appending `+N more`. */
 function joinPhrases(parts, maxParts) {
     if (parts.length > maxParts) {
         const shown = parts.slice(0, maxParts);
@@ -181,12 +143,6 @@ function joinPhrases(parts, maxParts) {
     }
     return parts.join(', ');
 }
-/**
- * Kind-only brief for the compact (wide-terminal) layout: `(24 skills, 9 commands, +20)`
- * — the top `maxKinds` kinds by display order, with a `+N` remainder. Drops the
- * new/changed verb (the arrow already carries direction) to stay short. Returns
- * '' when there is nothing to summarize.
- */
 export function deltaBrief(delta, maxKinds = 2) {
     if (delta.total === 0)
         return '';
@@ -201,24 +157,16 @@ export function deltaBrief(delta, maxKinds = 2) {
         kinds.push(`+${rest}`);
     return chalk.gray(`(${kinds.join(', ')})`);
 }
-/**
- * Render a set of changed files as a resource-level summary, e.g.
- * `2 new skills, 1 changed hook`. Counts distinct resource units (a skill whose
- * three files all changed is "1 changed skill"), grouped by action then kind,
- * and colors each phrase green/yellow/red for new/changed/removed. Caps at
- * `maxParts` phrases, appending `+N more` so a big diff stays scannable.
- */
 export function formatResourceDelta(entries, maxParts = 5) {
     return joinPhrases(deltaPhrases(resourceDelta(entries)), maxParts);
 }
-/** Parse `git diff --name-status <range>` into resource-delta entries. */
 async function diffResourceEntries(git, range) {
     let raw;
     try {
         raw = await git.diff(['--name-status', range]);
     }
     catch {
-        return []; // no upstream resolvable / range invalid — caller falls back to counts
+        return [];
     }
     const out = [];
     for (const line of raw.split('\n')) {
@@ -226,7 +174,7 @@ async function diffResourceEntries(git, range) {
             continue;
         const cols = line.split('\t');
         const code = cols[0] ?? '';
-        const file = cols[cols.length - 1] ?? ''; // renames put the new path last
+        const file = cols[cols.length - 1] ?? '';
         if (!file)
             continue;
         const c = code[0];
@@ -235,12 +183,6 @@ async function diffResourceEntries(git, range) {
     }
     return out;
 }
-/**
- * Read one repo's status into structured data: branch, resource-level sync (what
- * a pull/push would move), resource-level local edits, and remote URL + commit.
- * Formatting is deferred to the layout renderers so the same data can drive both
- * the wide table and the narrow cards.
- */
 async function renderRepoRow(t) {
     if (!fs.existsSync(t.dir)) {
         return { alias: t.alias, raw: `${chalk.red('missing')} ${chalk.gray(t.dir)}` };
@@ -251,23 +193,18 @@ async function renderRepoRow(t) {
     try {
         const git = simpleGit(t.dir);
         const status = await git.status();
-        // Show the local branch name; the upstream remote is already implied by URL.
         const branch = status.current || (status.tracking ? status.tracking.replace(/^origin\//, '') : '(detached)');
         const row = { alias: t.alias, branch, tracking: !!status.tracking };
-        // SYNC: what a pull brings in / a push sends out, described by resource.
         const ahead = status.ahead ?? 0;
         const behind = status.behind ?? 0;
         if (status.tracking) {
             if (behind > 0) {
-                // Three-dot isolates upstream's side via the merge-base, so a diverged
-                // branch reports exactly what a pull adds (not the inverse of local commits).
                 row.pull = { delta: resourceDelta(await diffResourceEntries(git, 'HEAD...@{upstream}')), commits: behind };
             }
             if (ahead > 0) {
                 row.push = { delta: resourceDelta(await diffResourceEntries(git, '@{upstream}...HEAD')), commits: ahead };
             }
         }
-        // CHANGES: uncommitted working-tree edits, described by resource.
         const localEntries = [
             ...status.created.map((f) => ({ action: 'new', file: f })),
             ...status.not_added.map((f) => ({ action: 'new', file: f })),
@@ -289,15 +226,12 @@ async function renderRepoRow(t) {
         return { alias: t.alias, raw: `${chalk.red('error')} ${err.message}` };
     }
 }
-/** Terminal width at or above which `list` uses the wide table instead of cards. */
 const WIDE_COLS = 120;
 const commitsWord = (n) => chalk.yellow(`${n} commit${n > 1 ? 's' : ''}`);
-/** `owner/repo` from a git URL, for the compact card header. Falls back to the raw URL. */
 export function repoSlug(url) {
     const m = url.match(/github\.com[:/]([^/]+\/[^/]+?)(?:\.git)?$/);
     return m ? m[1] : url;
 }
-/** Full SYNC string: `24 new skills, ... to pull  ·  ... to push` (verbose table). */
 function syncFull(row) {
     if (!row.tracking)
         return chalk.gray('no upstream');
@@ -314,7 +248,6 @@ function syncFull(row) {
     }
     return pieces.join(chalk.gray('  ·  '));
 }
-/** Compact SYNC for the wide table: `↓53 (24 skills, 9 commands, +20)  ↑13`. */
 function syncCompact(row) {
     if (!row.tracking)
         return chalk.gray('no upstream');
@@ -332,24 +265,17 @@ function syncCompact(row) {
     }
     return pieces.join('  ');
 }
-/** Full CHANGES string for the verbose table. */
 function changesFull(row) {
     if (row.clean)
         return chalk.green('clean');
     return joinPhrases(deltaPhrases(row.local), 5);
 }
-/** Compact CHANGES for the wide table: `clean` or `~1 edit`. */
 function changesCompact(row) {
     if (row.clean)
         return chalk.green('clean');
     const n = row.local.total;
     return chalk.yellow(`~${n} edit${n === 1 ? '' : 's'}`);
 }
-/**
- * Print the aligned table. `compact` picks short (arrow-count) SYNC/CHANGES cells
- * that keep the table bounded; verbose passes `compact: false` for full detail.
- * Only the trailing REMOTE column is allowed to run long.
- */
 function renderTable(rows, compact) {
     const cells = new Map();
     for (const r of rows) {
@@ -383,7 +309,6 @@ function renderTable(rows, compact) {
         }
     }
 }
-/** Pack colored phrases into `, `-joined lines no wider than `width`. */
 export function wrapPhrases(parts, width) {
     const lines = [];
     let cur = '';
@@ -401,14 +326,9 @@ export function wrapPhrases(parts, width) {
         lines.push(cur);
     return lines;
 }
-/**
- * Print one block per repo, width-independent. The category list wraps under an
- * indented `↓ pull` / `↑ push` / `local` label instead of running off the edge,
- * so a narrow terminal stays readable.
- */
 function renderCards(rows, cols) {
-    const LABEL = '      '; // indent for detail labels
-    const detailWidth = Math.max(20, cols - LABEL.length - 9); // 9 ≈ "↓ pull   "
+    const LABEL = '      ';
+    const detailWidth = Math.max(20, cols - LABEL.length - 9);
     let first = true;
     for (const r of rows) {
         if (!first)
@@ -446,19 +366,7 @@ function renderCards(rows, cols) {
         }
     }
 }
-/**
- * Recursion guard for the fleet fan-out, passed as an env var (not a CLI flag)
- * so an OLDER remote `agents` that predates this feature ignores it harmlessly
- * instead of erroring on an unknown option. A peer new enough to fan out reads
- * it and answers only for itself. Mirrors `NO_FANOUT_ENV` in remote-active.ts.
- */
 export const NO_REPO_FANOUT_ENV = 'AGENTS_REPO_LOCAL';
-/**
- * Parse one peer's `repo status --json` stdout (a RepoRow[]) into a single
- * device entry tagged with `machine`. Defensive against version skew: non-JSON
- * or a non-array yields no entry, and non-object rows are dropped rather than
- * throwing. Exported for unit testing without a live tailnet.
- */
 export function parseRemoteRepoRows(stdout, machine) {
     let parsed;
     try {
@@ -472,12 +380,6 @@ export function parseRemoteRepoRows(stdout, machine) {
     const rows = parsed.filter((x) => !!x && typeof x === 'object' && !Array.isArray(x));
     return [{ device: machine, reachable: true, rows }];
 }
-/**
- * Query repo status on the fleet. With an explicit `hosts` list, fan out to
- * exactly those; otherwise sweep every registered, online peer (excluding this
- * machine). Each peer runs `repo status [alias] --json` locally — never its own
- * fan-out — and unreachable peers are skipped with a gray note, never fatal.
- */
 async function gatherRemoteRepoStatus(alias, hosts) {
     const args = ['repo', 'status', ...(alias ? [alias] : []), '--json'];
     const { items } = await gatherRemoteAgentsJson({
@@ -488,13 +390,6 @@ async function gatherRemoteRepoStatus(alias, hosts) {
     });
     return items;
 }
-/**
- * Render the fleet repo-status table: one row per (device, repo), with the
- * device name shown once per group. SYNC and CHANGES reuse the same compact
- * cells as the local wide table (`↓N (…)  ↑N` / `clean` / `~N edits`), and an
- * unreachable device collapses to a single `unreachable` marker row. Pure — no
- * I/O — so the aggregation/formatting is unit-testable without live devices.
- */
 export function renderDeviceStatusRows(results) {
     const cells = [];
     for (const r of results) {
@@ -511,7 +406,6 @@ export function renderDeviceStatusRows(results) {
             const device = firstOfDevice ? r.device : '';
             firstOfDevice = false;
             if (row.raw) {
-                // missing / no-git-remote / error — the free-form trailer says it all.
                 cells.push({ device, repo: row.alias, sync: row.raw, changes: '' });
             }
             else {
@@ -532,12 +426,6 @@ export function renderDeviceStatusRows(results) {
     }
     return lines;
 }
-/**
- * Read the device-scope flags into an intent, or null for the default local-only
- * run. `--devices-all` (and `--devices all`) sweeps every reachable peer;
- * `--devices box1,box2` targets an explicit list. A peer carrying
- * {@link NO_REPO_FANOUT_ENV} never fans out again (recursion guard).
- */
 export function resolveDeviceIntent(opts) {
     if (process.env[NO_REPO_FANOUT_ENV] === '1')
         return null;
@@ -552,12 +440,6 @@ export function resolveDeviceIntent(opts) {
     const hosts = v.split(',').map((s) => s.trim()).filter(Boolean);
     return hosts.length ? { hosts } : { all: true };
 }
-/**
- * `repo status`/`list` across the fleet: read THIS machine's rows locally, fan
- * out to peers, and render one aggregated table (device · repo · sync · changes).
- * The local box is always shown first; unreachable peers were already noted by
- * the fan-out and are simply absent from the table.
- */
 async function listReposAcrossDevices(alias, opts, intent) {
     const localTargets = collectRepoTargets(alias);
     if (!localTargets) {
@@ -568,7 +450,6 @@ async function listReposAcrossDevices(alias, opts, intent) {
     const self = machineId();
     const hosts = 'hosts' in intent ? intent.hosts : undefined;
     const remote = (await gatherRemoteRepoStatus(alias, hosts))
-        // An explicit host list may name this box; the local rows already cover it.
         .filter((r) => normalizeHost(r.device) !== self);
     const combined = [
         { device: self, reachable: true, rows: localRows },
@@ -588,13 +469,6 @@ async function listReposAcrossDevices(alias, opts, intent) {
         console.log(line);
     console.log('');
 }
-/**
- * Shared action body for `agents repo list` and the hidden `agents repo status`
- * alias. Responsive: a wide terminal gets an aligned table with compact SYNC /
- * CHANGES cells; a narrow one gets one card per repo with wrapping detail;
- * `--verbose` forces the full-detail table at any width. A device-scope flag
- * (`--devices-all` / `--devices <list>`) instead fans out across the fleet.
- */
 async function listRepos(alias, opts = {}) {
     const intent = resolveDeviceIntent(opts);
     if (intent) {
@@ -616,14 +490,10 @@ async function listRepos(alias, opts = {}) {
     }
     const rows = await Promise.all(targets.map(renderRepoRow));
     if (opts.json) {
-        // Structured dump of the same rows so agents can enumerate repos + sync state.
-        // `raw` (the missing / no-git-remote human label) is the only colored field —
-        // strip ANSI so the JSON stays clean; every other field is already structured.
         const clean = rows.map((r) => (r.raw !== undefined ? { ...r, raw: stripAnsi(r.raw) } : r));
         console.log(JSON.stringify(clean, null, 2));
         return;
     }
-    // TTY width when interactive; fall back to $COLUMNS (honored when piped) then 80.
     const cols = process.stdout.columns || Number(process.env.COLUMNS) || 80;
     console.log('');
     if (opts.verbose) {
@@ -641,30 +511,16 @@ async function listRepos(alias, opts = {}) {
     }
     console.log('');
 }
-/**
- * Label for push/pull spinners and results: alias + resolved dir + tracking ref.
- * e.g. `user (~/.agents → origin/main)`.
- */
 function formatRepoTarget(alias, dir, branch) {
     const ref = branch ? `origin/${branch}` : 'origin';
     return `${alias} (${displayHomePath(dir)} → ${ref})`;
 }
-/**
- * Register the fleet device-scope flags shared by `repo list` / `repo status`.
- * `--devices-all` sweeps every reachable device; `--devices <who>` takes `all`
- * or a comma-separated list. A single `--device` is handled upstream by
- * maybeRunOnHost, which streams that one box's `agents repo status` — these are
- * the aggregating forms.
- */
 function addDeviceStatusOptions(cmd) {
     return cmd
         .option('--devices-all', 'Report repo sync state across ALL reachable fleet devices.')
         .option('--devices <who>', 'Fleet devices to report on: "all", or a comma-separated device list.');
 }
-/** Register the `agents repos` command tree (`repo` is a convenience alias). */
 export function registerRepoCommands(program) {
-    // addHostOption on the group so --help documents --device; remote
-    // routing is handled pre-parse by maybeRunOnHost (passthrough.ts).
     const repoCmd = addHostOption(program
         .command('repos')
         .alias('repo')
@@ -781,7 +637,6 @@ export function registerRepoCommands(program) {
                 fs.rmSync(targetDir, { recursive: true, force: true });
             }
             catch {
-                /* best-effort cleanup */
             }
             process.exitCode = 1;
         }
@@ -828,12 +683,6 @@ export function registerRepoCommands(program) {
         ensureAgentsDir();
         const targetDir = getExtraRepoDir(alias);
         if (fs.existsSync(targetDir)) {
-            // Adopt an existing checkout instead of hard-erroring. The common case:
-            // the user already cloned ~/.agents-<alias> by hand, then ran `repos add`
-            // and hit a dead end — which forced a second, inconsistent install method.
-            // If it's already a git repo whose origin matches the requested source,
-            // register it in place (no re-clone); adopt a mismatched/remoteless repo
-            // only with an explicit --adopt.
             if (isGitRepo(targetDir)) {
                 const existingUrl = await getRemoteUrl(targetDir);
                 const matches = sameGitRemote(existingUrl, parsed.url);
@@ -879,7 +728,6 @@ export function registerRepoCommands(program) {
                 fs.rmSync(targetDir, { recursive: true, force: true });
             }
             catch {
-                /* best-effort cleanup */
             }
             process.exitCode = 1;
             return;
@@ -947,8 +795,6 @@ export function registerRepoCommands(program) {
             return;
         }
         const dir = resolveExtraRepoDir(alias, extras[alias]);
-        // Managed clones live at the default ~/.agents-<alias>/ — those we own
-        // and should delete on remove. Anything else is user-owned, leave alone.
         const isManagedClone = path.resolve(dir) === path.resolve(getExtraRepoDir(alias));
         try {
             if (isManagedClone && fs.existsSync(dir)) {
@@ -993,12 +839,6 @@ export function registerRepoCommands(program) {
         let userPulled = false;
         for (const t of targets) {
             if (!fs.existsSync(t.dir) || !isGitRepo(t.dir)) {
-                // A plain (never-cloned) user repo — setup makes ~/.agents a bare dir and
-                // never git-clones it (state.ts ensureAgentsDir), so a fresh/Windows box
-                // silently falls out of config sync. If a URL was passed, git-back it in
-                // place (keeping local files, backing up any that differ); else guide the
-                // user. Only the USER repo gets this — system is cloned by setup, extras
-                // by `repo add`.
                 if (t.alias === 'user' && url) {
                     const spinner = interruptibleSpinner(`Git-backing ${t.dir} from ${url}...`).start();
                     const res = await adoptRepo(url, t.dir);
@@ -1012,7 +852,6 @@ export function registerRepoCommands(program) {
                         console.log(chalk.yellow(`  backed up ${res.backedUp.length} locally-modified file(s) to ${res.backupDir}`));
                         console.log(chalk.gray(`  (${res.backedUp.slice(0, 5).join(', ')}${res.backedUp.length > 5 ? ', …' : ''})`));
                     }
-                    // Now a git repo — fall through to the normal pull below.
                 }
                 else {
                     console.log(chalk.yellow(`  ${t.alias}: not a git repo, skipping`));
@@ -1023,7 +862,6 @@ export function registerRepoCommands(program) {
                 }
             }
             if (t.alias === 'system' && alias !== 'system') {
-                // Skip system repo unless explicitly requested
                 continue;
             }
             const spinner = interruptibleSpinner(`Pulling ${formatRepoTarget(t.alias, t.dir)}...`).start();
@@ -1036,18 +874,9 @@ export function registerRepoCommands(program) {
             }
             else {
                 spinner.fail(`${formatRepoTarget(t.alias, t.dir)}: ${result.error}`);
-                // A failed repo must fail the command. Without this, `agents fleet run
-                // "agents repo pull user"` reported 11 ok across a fleet that pulled
-                // nothing — the silence that hid RUSH-2056. Matches commands/sync.ts.
                 process.exitCode = 1;
             }
         }
-        // RUSH-1980: a pull rewrites the routine YAML on disk, but the daemon's
-        // scheduler froze its JobConfigs (device pins included) at load. Without a
-        // reload it keeps firing the pre-pull pins — a routine re-pinned to another
-        // host still fires here, double-firing across the fleet. SIGHUP the daemon
-        // so scheduler.reloadAll() re-reads the synced YAML and device pins refresh.
-        // No-op when the daemon isn't running (or on Windows, which has no SIGHUP).
         if (anyPulled) {
             const { isDaemonRunning, signalDaemonReload } = await import('../lib/daemon/daemon.js');
             if (isDaemonRunning() && signalDaemonReload()) {
@@ -1077,7 +906,6 @@ export function registerRepoCommands(program) {
             process.exitCode = 1;
             return;
         }
-        // Drop system-repo targets — read-only by design.
         const pushable = [];
         for (const t of targets) {
             if (t.alias === 'system') {
@@ -1088,7 +916,6 @@ export function registerRepoCommands(program) {
                 console.log(chalk.yellow(`  ${t.alias}: not a git repo, skipping`));
                 continue;
             }
-            // Defense in depth: refuse if origin happens to be the system upstream.
             if (await isSystemRepoOrigin(t.dir)) {
                 console.log(chalk.red(`  ${t.alias}: origin tracks the system repo — refusing to push.`));
                 continue;
@@ -1109,16 +936,10 @@ export function registerRepoCommands(program) {
             }
             else {
                 spinner.fail(`${formatRepoTarget(t.alias, t.dir)}: ${result.error}`);
-                // A failed repo must fail the command. Without this, `agents fleet run
-                // "agents repo push user"` reported ok across a fleet that pushed
-                // nothing — the silence that hid RUSH-2056. Matches commands/sync.ts.
                 process.exitCode = 1;
             }
         }
     });
-    // agents repo sync <name> — git-level sync for a single DotAgents repo.
-    // Replaces the git portion of the old `agents sync <repo>` verb.
-    // Semantics: system is pull-only (push: false); user and enabled extras push.
     repoCmd
         .command('sync <alias>')
         .description('Git-sync a repo: pull (and push for user/extras). Aliases: "system", "user", or a registered extra.')
@@ -1141,10 +962,6 @@ Examples:
         }
         for (const t of targets) {
             if (!fs.existsSync(t.dir) || !isGitRepo(t.dir)) {
-                // The user repo self-heals in place instead of being skipped: adopt a
-                // non-git / partial checkout against its remote, preserving runtime
-                // state, then fall through to the normal sync (PHNX-3301). System is
-                // cloned by setup and extras by `repo add`, so those still skip.
                 if (t.alias === 'user' && fs.existsSync(t.dir)) {
                     const spinner = interruptibleSpinner(`Adopting ${formatRepoTarget(t.alias, t.dir)} in place...`).start();
                     const adopted = await adoptUserRepoIfNeeded(t.dir);
@@ -1162,7 +979,6 @@ Examples:
                     if (adopted.agentsYamlBackup) {
                         console.log(chalk.gray(`  saved the previous agents.yaml to ${adopted.agentsYamlBackup}`));
                     }
-                    // Now a git repo — fall through to the normal sync below.
                 }
                 else {
                     console.log(chalk.yellow(`  ${t.alias}: not a git repo, skipping`));
@@ -1170,8 +986,6 @@ Examples:
                 }
             }
             const push = t.alias !== 'system';
-            // A plain pull/rebase (+ push) of human-authored resources for every
-            // alias, the user repo included: daemon state left git (PHNX-4116).
             const spinner = interruptibleSpinner(`Syncing ${formatRepoTarget(t.alias, t.dir)}...`).start();
             const result = await syncRepoGit(t.dir, { push });
             if (result.success) {
@@ -1201,10 +1015,6 @@ Examples:
         await listRepos(alias, options);
     });
 }
-/**
- * Resolve an alias (or undefined for "all") to a list of repo targets.
- * Returns null when a named alias isn't found (callers should set exit code).
- */
 function collectRepoTargets(alias) {
     const meta = readMeta();
     const extras = meta.extraRepos || {};
@@ -1226,12 +1036,6 @@ function collectRepoTargets(alias) {
     }
     return [found];
 }
-/**
- * Keep already-installed versions' selectors in sync with an extra-repo change:
- * add `<alias>:*` when the repo is registered/enabled, strip it when removed.
- * Newly-installed versions inherit it from `defaultPatterns()` at scaffold time,
- * so without this a repo added after install is invisible to existing versions.
- */
 function syncExtraAliasAcrossVersions(alias, add) {
     const n = applyExtraAliasToVersions(alias, add);
     if (n > 0) {
@@ -1253,8 +1057,6 @@ async function toggle(alias, enabled) {
     }
     extras[alias] = { ...extras[alias], enabled };
     updateMeta({ extraRepos: extras });
-    // Re-enabling backfills the alias into existing versions; disabling leaves the
-    // selectors (resolution skips disabled extras) so a later enable is a no-op.
     if (enabled)
         syncExtraAliasAcrossVersions(alias, true);
     syncMarketplacesForDefaults();

package/dist/commands/resource-view.d.ts

@@ -1,11 +1,3 @@
-/**
- * Shared resource list and detail view.
- *
- * Provides a reusable picker (TTY) / table (piped) presentation layer
- * for resource-type commands (plugins, subagents, skills, etc.). Each
- * resource supplies rows with sync targets; this module handles layout,
- * filtering, and paging.
- */
 import type { AgentId } from '../lib/types.js';
 export type SyncStatus = 'synced' | 'stale' | 'missing';
 export interface SyncTarget {
@@ -30,30 +22,13 @@ interface ResourceViewOptions {
     rows: ResourceRow[];
     emptyMessage: string;
     centralPath?: string;
-    /** When the user specified agent or agent@version, we scope per-agent. */
     filterAgent?: AgentId;
     filterVersion?: string;
-    /** Emit machine-readable JSON instead of the picker/table (for agents/scripts). */
     json?: boolean;
-    /**
-     * Show the "Synced" column. Default true — the central-storage commands all
-     * answer "which agent versions have this?". `agents inspect <repo>` does not:
-     * there the repo IS the source, and empty `targets` would render the
-     * misleading "no installed versions". Set false to drop the column entirely.
-     */
     showSync?: boolean;
-    /**
-     * Cap for the Name column. Defaults to NAME_CAP (22), which suits the
-     * central-storage commands. Kinds whose rows have no description — hooks are
-     * all script names and no prose — want a wider cap, otherwise four pairs of
-     * `00-agent-verify-work-complete…` truncate to the same string while the
-     * empty Description column wastes the rest of the terminal.
-     */
     nameCap?: number;
-    /** Per-row OSC-8 link target, keyed by row name; makes names clickable. */
     linkFor?: (row: ResourceRow) => string | undefined;
 }
-/** Display a resource list: interactive picker in TTY mode, plain table otherwise. */
 export declare function showResourceList(opts: ResourceViewOptions): Promise<void>;
 interface ResourceLayout {
     mode: 'table' | 'cards';
@@ -63,19 +38,11 @@ interface ResourceLayout {
     descW: number;
     syncW: number;
 }
-/**
- * Pure column arithmetic. Decides table vs. cards for the effective terminal
- * width and sizes the flexible Description column from whatever is left after the
- * fixed columns and a capped Sync column — so the table fits `cols` instead of
- * the old fixed 22+10+16+42+∞ layout that overflowed every narrow terminal.
- */
 export declare function resourceLayout(cols: number, o: {
     hasExtra: boolean;
     hasExtra2: boolean;
     nameW: number;
     syncW: number;
 }): ResourceLayout;
-/** Build the sync targets section showing version pills grouped by agent. */
 export declare function buildTargetsSection(targets: SyncTarget[]): string;
 export {};
-/** Pad a string to a fixed width, accounting for ANSI escape codes in length calculation. */