@phnx-labs/agents-cli 1.22.120 → 1.22.121

package/dist/commands/inspect.js

@@ -1,16 +1,3 @@
-/**
- * `agents inspect <target>` — detail view for one agent+version or one DotAgents repo.
- *
- * Agent targets (`claude`, `claude@2.1.170`) show the per-version header (paths,
- * shim, capabilities, resource counts, sessions). Repo targets (`user`, `system`,
- * `project`, a registered extra-repo alias, or a filesystem path to a repo with a
- * `.agents/` dir or to a DotAgents root itself) show the repo root, git state, and
- * per-kind resource counts. Drill-down flags (`--skills`, `--hooks`, `--mcp`, ...)
- * list one resource kind for either target form; passing a positional query to the
- * same flag fuzzy-searches for a single resource and prints its detail. Resource
- * names render as OSC-8 hyperlinks to the marker file (SKILL.md / WORKFLOW.md /
- * AGENT.md / the file itself) so users can click straight to the source.
- */
 import { execFileSync } from 'child_process';
 import { addHostOption } from '../lib/hosts/option.js';
 import * as fs from 'fs';
@@ -36,13 +23,12 @@ import { showResourceList } from './resource-view.js';
 import { PLUGIN_GROUP_COLORS } from './plugins.js';
 import { isInteractiveTerminal } from './utils.js';
 import { countSessionsInScope } from '../lib/session/discover.js';
-import { isSessionTrackedAgent } from '../lib/session/types.js';
+import { isSessionTrackedAgent } from '@phnx-labs/sessions-cli/reader';
 import { damerauLevenshtein } from '../lib/fuzzy.js';
 import { terminalWidth, truncateToWidth, stringWidth, stripAnsi } from '../lib/session/width.js';
 import { readJobFileResult, getJobRunsDir, readRunMeta, isOneShotRoutine, isPastOneShotRoutine, listJobs, } from '../lib/scheduling/routines.js';
 import { routineDeviceIndex, currentRoutineDevice, routineEnabledOnThisDevice, } from '../lib/routine-activation.js';
 import { humanizeCron } from '../lib/routines-format.js';
-/** Resource kinds the inspect command can drill into. */
 const DRILLABLE_KINDS = [
     'commands',
     'skills',
@@ -54,23 +40,10 @@ const DRILLABLE_KINDS = [
     'subagents',
     'routines',
 ];
-/**
- * Summary-view partition. SIMPLE kinds render as a one-line count + name preview;
- * RICH kinds (hooks/plugins/mcp/routines) get their own expanded section showing
- * each item's key detail (events/predicates, bundle contents, transport/url,
- * schedule + where a routine actually fires). Together they cover every
- * DrillableKind — the type below makes that claim load-bearing.
- */
 const SIMPLE_KINDS = ['commands', 'skills', 'rules', 'subagents', 'workflows'];
 const RICH_KINDS = ['hooks', 'plugins', 'mcp', 'routines'];
 const _kindsPartitioned = true;
 void _kindsPartitioned;
-/**
- * Singular aliases for the plural drill-down flags. `--plugin code` reads as
- * "show the one plugin named code" — a required-value flag that always lands in
- * detail mode, the natural counterpart to `--plugins` (list). `mcp` has no
- * distinct singular, so it is intentionally absent.
- */
 const SINGULAR_DRILL_ALIASES = {
     command: 'commands',
     skill: 'skills',
@@ -84,7 +57,6 @@ const SINGULAR_DRILL_ALIASES = {
 const CAPABILITY_NAMES = [
     'hooks', 'mcp', 'skills', 'commands', 'subagents', 'plugins', 'workflows', 'rules', 'allowlist',
 ];
-/** Wrap a separator-joined value list under `prefix`, preserving a hanging indent. */
 export function wrapJoined(prefix, items, sep, width) {
     if (items.length === 0)
         return [];
@@ -122,7 +94,6 @@ function printWrappedJoined(prefix, items, sep) {
 function truncateValueForPrefix(prefix, value) {
     return truncateToWidth(stripAnsi(value), Math.max(1, terminalWidth() - stringWidth(prefix)));
 }
-// ─── Command registration ────────────────────────────────────────────────────
 export function registerInspectCommand(program) {
     const cmd = addHostOption(program.command('inspect <target>'))
         .description('Inspect one installed agent harness at one version (not a model), or a DotAgents repo — paths, capabilities, resources, and hook capable/on-disk/wired state.')
@@ -138,7 +109,6 @@ export function registerInspectCommand(program) {
         await inspectAction(target, options);
     });
 }
-// ─── Main dispatcher ─────────────────────────────────────────────────────────
 async function inspectAction(target, options) {
     const agentKey = target.split('@')[0].toLowerCase();
     if (!(agentKey in AGENTS)) {
@@ -147,8 +117,6 @@ async function inspectAction(target, options) {
             await inspectRepo(repo, options);
             return;
         }
-        // Repo targets take precedence over typo correction; only fall through to
-        // parseTarget when the key resolves to an agent (alias or single-edit fix).
         if (!resolveAgentName(agentKey)) {
             const extras = getEnabledExtraRepos();
             console.error(chalk.red(`Unknown target: ${target}`));
@@ -180,9 +148,6 @@ async function inspectAction(target, options) {
     await renderSummary(agent, version, versionHome, options);
 }
 function parseTarget(target) {
-    // Route through the agent-spec engine: bare resolves project pin → global
-    // default → sole installed (the meta-only lookup here previously ignored
-    // project pins); @default/@latest/@oldest/@x.y.z all handled uniformly.
     try {
         const { agent, version } = resolveSingleAgentTarget(target);
         return { agent, version };
@@ -202,7 +167,6 @@ function pickDrillKind(options) {
         if (value !== undefined)
             active.push({ flag: `--${kind}`, kind, query: value });
     }
-    // Singular aliases (`--plugin code`) always carry a name → detail mode.
     for (const [singular, plural] of Object.entries(SINGULAR_DRILL_ALIASES)) {
         const value = options[singular];
         if (typeof value === 'string')
@@ -216,14 +180,7 @@ function pickDrillKind(options) {
     }
     return { kind: active[0].kind, query: active[0].query };
 }
-/** Files at a DotAgents root that mark it as one, beyond the per-kind dirs. */
 const REPO_MARKER_FILES = ['agents.yaml', 'hooks.yaml'];
-/**
- * Resolve a non-agent target as a DotAgents repo: the built-in layer names,
- * a registered extra-repo alias, or a filesystem path. Paths accept either a
- * DotAgents root itself or a repo whose `.agents/` dir should be inspected.
- * Returns null when the target is none of these.
- */
 export function resolveRepoTarget(target, cwd) {
     if (target === 'user')
         return { label: 'user', root: getUserAgentsDir() };
@@ -246,21 +203,13 @@ export function resolveRepoTarget(target, cwd) {
     const stat = safeStat(abs);
     if (!stat || !stat.isDirectory())
         return null;
-    // A dir literally named `.agents` is the root itself.
     if (path.basename(abs) === '.agents') {
         return { label: path.basename(path.dirname(abs)), root: abs };
     }
-    // A nested `.agents/` that is a populated DotAgents root wins over `abs` — the
-    // project case (`agents inspect .` from a repo root whose resources live under
-    // `.agents/`, while the repo's own top-level `skills/`, `agents.yaml` pin, etc.
-    // are unrelated source, not a DotAgents tree).
     const nested = path.join(abs, '.agents');
     if (isDotAgentsRoot(nested)) {
         return { label: path.basename(abs), root: nested };
     }
-    // Otherwise treat `abs` itself as the root: standalone clones and extra repos
-    // like ~/.agents-extras keep resources at the top level and use `.agents/`
-    // only for worktrees (so their nested `.agents/` is not a DotAgents root).
     if (isDotAgentsRoot(abs)) {
         return { label: path.basename(abs), root: abs };
     }
@@ -272,11 +221,6 @@ function isDotAgentsRoot(dir) {
             return true;
     }
     for (const kind of DRILLABLE_KINDS) {
-        // `routines/` alone is NOT a marker. Shipping example routine YAML for
-        // `agents routines add` is a documented pattern (this repo's own
-        // cli/routines/), and that directory is the ONLY kind dir there — so
-        // counting it would make `agents inspect cli` resolve a source tree as
-        // a DotAgents repo. Same reasoning as the nested-`.agents/` case above.
         if (kind === 'routines')
             continue;
         if (safeStat(path.join(dir, kind))?.isDirectory())
@@ -288,9 +232,6 @@ export async function inspectRepo(repo, options) {
     const drill = pickDrillKind(options);
     const jsonHead = { repo: repo.label, root: repo.root };
     if (drill) {
-        // Routines carry live state, and an unreadable device document silently
-        // removes that device from every `devices` cell — say so on this path too,
-        // not only in the overview.
         let items;
         if (drill.kind === 'routines') {
             const provider = defaultRoutineLive();
@@ -305,9 +246,6 @@ export async function inspectRepo(repo, options) {
             items = collectRepoKind(repo, drill.kind);
         }
         if (drill.query === true || drill.query === undefined) {
-            // A repo's hooks are wired by its OWN agents.yaml, not by whatever this
-            // machine has installed centrally — the same manifest the repo overview
-            // reads at collectRepoKind.
             await renderItemList(repo.label, jsonHead, drill.kind, items, options, drill.kind === 'hooks'
                 ? hookManifestByScript(hookManifestFromFile(path.join(repo.root, 'agents.yaml')))
                 : undefined);
@@ -319,42 +257,19 @@ export async function inspectRepo(repo, options) {
     }
     renderRepoSummary(repo, options);
 }
-/** List one resource kind from a single repo root — no layering, no overrides. */
 export function collectRepoKind(repo, kind) {
-    // Plugins are bundles with a manifest + nested skills/commands/hooks — read
-    // them through the plugin discoverer so the manifest description and bundled
-    // resources surface, rather than treating each as an opaque directory.
     if (kind === 'plugins') {
         return discoverPluginsInDir(path.join(repo.root, 'plugins'))
             .map(p => pluginToItem(p, repo.label))
             .sort((a, b) => a.name.localeCompare(b.name));
     }
-    // Hooks live nested under event directories (`hooks/pre-tool-use/…`), and a
-    // script pairs with its data sidecar. repoHookItems already reads them that
-    // way for the summary view; a flat readdir here returned the event dirs
-    // themselves plus README/test scaffolding, so `--hooks` and the summary
-    // reported different counts for the same repo.
     if (kind === 'hooks')
         return repoHookItems(repo);
-    // A repo's `rules/` holds the COMPOSED output (AGENTS.md plus its CLAUDE.md /
-    // GEMINI.md symlinks) alongside a `subrules/` dir of the individually named
-    // fragments. The fragments are what `--rule <name>` resolves and what a reader
-    // means by "a rule"; the composed file is a build artifact. Without this,
-    // `subrules` listed as a single opaque leaf and every real rule was unreachable.
-    // Deliberately exclusive: once `subrules/` exists it is the sole home for
-    // rules (composeRules only ever resolves names under it — lib/rules/compose.ts),
-    // so loose top-level `.md` files are not listed. A repo mid-migration to the
-    // subrules convention would see those legacy files disappear from `--rules`;
-    // that is the correct signal, since composeRules would not load them either.
     if (kind === 'rules') {
         const subrulesDir = path.join(repo.root, kind, 'subrules');
         if (fs.existsSync(subrulesDir))
             return readResourceDir(subrulesDir, kind, repo.label);
     }
-    // Routines are YAML job definitions whose real state (does it fire, where, how
-    // did it last run) lives outside the file. Their directory also holds
-    // `<name>/home/` sandbox overlays, which a flat readdir would count as
-    // routines — on this author's box that turned 24 routines into 49.
     if (kind === 'routines')
         return collectRepoRoutines(repo);
     return readResourceDir(path.join(repo.root, kind), kind, repo.label);
@@ -362,25 +277,9 @@ export function collectRepoKind(repo, kind) {
 const NO_LIVE_STATE = {
     devices: [], materialized: false, thisDevice: '', enabledHere: null, lastStatus: null, lastAt: null,
 };
-/**
- * Whether the routine actually runs on this machine.
- *
- * Exactly `applyDeviceActivation` (`lib/routines.ts`): this device's own
- * allowlist answer wins when it exists, and the file's `enabled:` governs only
- * while that answer is `null`. Measured at the same scope as the daemon's, so
- * `agents inspect --routines` and `agents routines list` cannot disagree.
- */
 function routineEnabledHere(config, live) {
     return live.enabledHere === null ? config.enabled : live.enabledHere;
 }
-/**
- * The last run we can read, without paying for the whole history.
- *
- * `getLatestRun` goes through `listRuns`, which parses EVERY meta.json under the
- * routine — one firing every 3 minutes accumulates hundreds. Run ids are ISO-ish
- * and sort lexicographically, so walking from the newest and stopping at the
- * first readable record gives the same answer for a bounded number of reads.
- */
 function latestRunOf(name, maxProbes = 20) {
     let runIds;
     try {
@@ -400,14 +299,6 @@ function latestRunOf(name, maxProbes = 20) {
     }
     return null;
 }
-/**
- * Default live provider: one fleet index per collect, one bounded run read per
- * routine, and this device's own activation answer per routine.
- *
- * Returns the collected `errors` alongside so the caller can surface an
- * unreadable device document instead of letting the affected device silently
- * vanish from every `devices` cell.
- */
 function defaultRoutineLive() {
     let index;
     try {
@@ -423,10 +314,6 @@ function defaultRoutineLive() {
         return '';
     } })();
     const errors = [...index.errors];
-    // This device's own doc is read through readMeta, which THROWS on a corrupt
-    // one. Falling back to the file's `enabled:` silently would be the same
-    // silent-failure class the index's error collection just removed, so record it
-    // once — the answer is identical for every routine on this machine.
     let ownDocError = null;
     const enabledOnThisDevice = (name) => {
         try {
@@ -446,7 +333,6 @@ function defaultRoutineLive() {
             devices: index.byRoutine.get(name) ?? [],
             materialized: index.materialized,
             thisDevice,
-            // The same call applyDeviceActivation makes — per-machine, not fleet-wide.
             enabledHere: enabledOnThisDevice(name),
             lastStatus: run?.status ?? null,
             lastAt: run?.startedAt ?? null,
@@ -454,7 +340,6 @@ function defaultRoutineLive() {
     };
     return { live, errors };
 }
-/** `2h` / `3d` / `just now` — compact enough for a 10-column cell. */
 export function compactAge(iso, now = new Date()) {
     if (!iso)
         return '';
@@ -474,7 +359,6 @@ export function compactAge(iso, now = new Date()) {
         return `${days}d`;
     return `${Math.floor(days / 365)}y`;
 }
-/** What makes this routine fire: a humanized cron, an event trigger, or nothing. */
 export function routineFireLabel(job, now = new Date()) {
     const parts = [];
     if (job.schedule)
@@ -492,7 +376,6 @@ export function routineFireLabel(job, now = new Date()) {
     }
     return parts.join(' · ');
 }
-/** What the routine actually executes. Exactly one of agent/workflow/command is set. */
 export function routineTargetLabel(job) {
     const agent = manifestText(job.agent);
     if (agent)
@@ -504,14 +387,6 @@ export function routineTargetLabel(job) {
         return 'command (shell)';
     return 'nothing configured';
 }
-/**
- * A routine's one-line intent.
- *
- * Routine YAML has no `description:` key, so the generic first-prose-line reader
- * returns `name: growth-metrics` for one file and a real sentence for the next.
- * Prefer the prompt's first sentence, then the file's leading comment block, then
- * the first meaningful line of a shell command.
- */
 export function routineDescription(job, raw) {
     const prompt = job ? manifestText(job.prompt).trim() : '';
     if (prompt)
@@ -526,7 +401,6 @@ export function routineDescription(job, raw) {
                 continue;
         }
         const text = comment[1].trim().replace(/^Routine:\s*/i, '');
-        // Skip the shebang-ish and install-instruction noise that opens many files.
         if (!text || /^(yaml-language-server|Install:|Usage:)/i.test(text))
             continue;
         return summaryLine(text);
@@ -540,15 +414,6 @@ export function routineDescription(job, raw) {
     }
     return '';
 }
-/**
- * Read every routine one DotAgents repo declares.
- *
- * Reads exactly `<root>/routines/`, never the layered `listJobs` — `collectRepoKind`
- * is a single-root view, and returning `~/.agents` routines while inspecting a
- * project repo would be a lie. `live` is injectable because `getUserAgentsDir()`
- * and `getRunsDir()` have no env override, so tests cannot otherwise keep this
- * off the developer's real fleet state.
- */
 export function collectRepoRoutines(repo, live = defaultRoutineLive().live, now = new Date()) {
     const dir = path.join(repo.root, 'routines');
     let entries;
@@ -560,12 +425,6 @@ export function collectRepoRoutines(repo, live = defaultRoutineLive().live, now
     }
     const items = [];
     const seen = new Map();
-    // Files only: `routines/<name>/home/` is a sandbox overlay HOME, not a routine.
-    //
-    // `.yml` before `.yaml` for the same basename, because that is the order
-    // `readJobFromDir` tries — so when both exist we describe the file the daemon
-    // actually loads. A plain alphabetical sort put `.yaml` first and reported the
-    // wrong schedule and command for a routine that fires from its sibling.
     const files = entries
         .filter(e => e.isFile() && /\.ya?ml$/.test(e.name) && !e.name.startsWith('.'))
         .map(e => e.name)
@@ -582,9 +441,6 @@ export function collectRepoRoutines(repo, live = defaultRoutineLive().live, now
         const name = config?.name || basename;
         const existing = seen.get(name);
         if (existing) {
-            // `writeJob` refuses to edit a routine with both extensions. Keep the item
-            // built from the file that loads, and flag the ambiguity on it — in `extra`
-            // for the panes AND in `json` so a --json consumer sees it too.
             const collision = `both .yml and .yaml exist — ${file} is ignored; resolve before editing`;
             existing.extra = [...(existing.extra ?? []), ['problem', collision]];
             if (existing.json)
@@ -638,9 +494,6 @@ export function collectRepoRoutines(repo, live = defaultRoutineLive().live, now
             linkTarget: filePath,
             description: routineDescription(config, raw),
             extra,
-            // Field names match `agents routines list --json` where they overlap, so
-            // one script can consume both. `devices` is what the YAML pins;
-            // `enabledDevices` is what the fleet actually allows — the diagnostic pair.
             json: {
                 path: filePath,
                 schedule: config?.schedule ?? null,
@@ -652,9 +505,7 @@ export function collectRepoRoutines(repo, live = defaultRoutineLive().live, now
                 command: config?.command ?? null,
                 timezone: config?.timezone ?? null,
                 repo: config?.repo ?? null,
-                // Effective on THIS machine, matching `agents routines list`.
                 enabled: config ? routineEnabledHere(config, state) : false,
-                // What the file itself declares, before device activation overrides it.
                 enabledInFile: config?.enabled ?? false,
                 catchup: config ? config.catchup !== false : null,
                 devices: config ? manifestList(config.devices) : [],
@@ -672,15 +523,9 @@ export function collectRepoRoutines(repo, live = defaultRoutineLive().live, now
     }
     return items.sort((a, b) => a.name.localeCompare(b.name));
 }
-/** Read one `extra` row back out — the renderers' only accessor. */
 function extraOf(item, key) {
     return item.extra?.find(([k]) => k === key)?.[1] ?? '';
 }
-/**
- * Enumerate one directory of resources of `kind`, skipping dotfiles, build caches,
- * and directory docs. Shared so the rules `subrules/` branch and the default
- * `<repo>/<kind>/` branch cannot drift apart.
- */
 function readResourceDir(dir, kind, source) {
     let entries;
     try {
@@ -693,13 +538,9 @@ function readResourceDir(dir, kind, source) {
     for (const entry of entries) {
         if (entry.name.startsWith('.'))
             continue;
-        // Build/tooling caches are never resources — they only inflate counts.
         if (entry.name === '__pycache__' || entry.name === 'node_modules')
             continue;
         const name = entry.name.replace(/\.(md|yaml|yml|toml|json)$/, '');
-        // README/AGENTS/CLAUDE/GEMINI describe the directory, not resources of this
-        // kind. CLAUDE.md/GEMINI.md are symlinks to AGENTS.md, so a Dirent reports
-        // isFile() === false — use !isDirectory() to catch them (mirrors resources.ts).
         if (!entry.isDirectory() && isDirectoryDoc(kind, name))
             continue;
         const p = path.join(dir, entry.name);
@@ -713,7 +554,6 @@ function readResourceDir(dir, kind, source) {
     }
     return items.sort((a, b) => a.name.localeCompare(b.name));
 }
-/** A few resource names for the at-a-glance preview, with a `…(+N)` tail. */
 function previewNames(items, n) {
     if (items.length === 0)
         return '';
@@ -721,7 +561,6 @@ function previewNames(items, n) {
     const extra = items.length - shown.length;
     return shown.join(', ') + (extra > 0 ? ` …(+${extra})` : '');
 }
-/** Recursive size + file count of a path; symlinks are not followed. */
 export function pathSize(p) {
     let stat;
     try {
@@ -751,10 +590,6 @@ export function pathSize(p) {
     }
     return { bytes, files };
 }
-/**
- * Weigh only the entries actually listed, for a kind whose directory holds more
- * than its resources (routines share theirs with sandbox overlay HOMEs).
- */
 function itemsSize(items) {
     let bytes = 0, files = 0;
     for (const item of items) {
@@ -764,7 +599,6 @@ function itemsSize(items) {
     }
     return { bytes, files };
 }
-/** Parse the repo's own agents.yaml into the version pins + run strategies it declares. */
 export function repoManifestSummary(root) {
     let parsed;
     try {
@@ -806,8 +640,6 @@ function renderRepoSummary(repo, options) {
     let routineActivationErrors = [];
     if (!options.brief) {
         for (const kind of DRILLABLE_KINDS) {
-            // Routines share one live provider across the collect so the fleet index is
-            // read once, and so its errors survive to be reported below.
             let items;
             if (kind === 'routines') {
                 const provider = defaultRoutineLive();
@@ -817,9 +649,6 @@ function renderRepoSummary(repo, options) {
             else {
                 items = collectRepoKind(repo, kind);
             }
-            // `routines/` also holds `<name>/home/` sandbox overlays — often far larger
-            // than the definitions and not resources at all, so weigh the files listed
-            // rather than the directory tree.
             const size = kind === 'routines' ? itemsSize(items) : pathSize(path.join(repo.root, kind));
             kindData[kind] = { items, size };
             totalBytes += size.bytes;
@@ -873,7 +702,6 @@ function renderRepoSummary(repo, options) {
         return;
     }
     console.log('\n' + chalk.bold(repo.label) + '  ' + chalk.gray('[dotagents repo]') + '\n');
-    // Indent for continuation sub-rows: 2 leading + 10 key column + 1 space.
     const sub = (label, value) => console.log(`  ${''.padEnd(10)} ${chalk.gray(label.padEnd(8))} ${value}`);
     console.log(`  ${'root'.padEnd(10)} ${termLink(repo.root, repo.root)}`);
     if (git) {
@@ -916,8 +744,6 @@ function renderRepoSummary(repo, options) {
             const { items, size } = kindData[kind];
             const count = String(items.length).padStart(4);
             const sz = items.length > 0 ? formatBytes(size.bytes).padStart(8) : ''.padEnd(8);
-            // Width-aware, not a fixed 60: the old cap cut its own "…(+16)" tail off
-            // the rules row identically at 80, 100 and 160 columns.
             const prefix = `  ${kind.padEnd(10)} ${count}  ${sz}  `;
             const preview = items.length > 0
                 ? chalk.gray(truncateToWidth(previewNames(items, 4), Math.max(12, terminalWidth() - stringWidth(prefix))))
@@ -934,8 +760,6 @@ function renderRepoSummary(repo, options) {
             console.log(`  ${chalk.yellow('⚠')}  ${dark}`);
             console.log(chalk.gray(`     Fix:  agents routines devices <name> --set <device>`));
         }
-        // An unreadable device document silently removes that device from every
-        // `devices` cell, which reads as "not enabled there" — say so instead.
         for (const err of routineActivationErrors) {
             console.log(chalk.gray(`  device config unreadable — ${err}`));
         }
@@ -956,8 +780,6 @@ export function repoGitInfo(root) {
     const branch = git(['rev-parse', '--abbrev-ref', 'HEAD']);
     if (branch === null)
         return null;
-    // Read status WITHOUT trimming — trimming would strip the leading
-    // space of the first porcelain line (`XY path`), corrupting the path slice.
     let statusRaw;
     try {
         statusRaw = execFileSync('git', ['-C', root, 'status', '--porcelain'], { stdio: ['ignore', 'pipe', 'ignore'], encoding: 'utf8' });
@@ -984,7 +806,6 @@ export function repoGitInfo(root) {
     }
     return { branch, dirty: dirtyFiles.length, dirtyFiles, url: git(['remote', 'get-url', 'origin']), lastCommit, ahead, behind };
 }
-// ─── Summary mode ────────────────────────────────────────────────────────────
 async function renderSummary(agent, version, versionHome, options) {
     const meta = readMeta();
     const isDefault = meta.agents?.[agent] === version;
@@ -992,18 +813,11 @@ async function renderSummary(agent, version, versionHome, options) {
     const cliState = await getCliState(agent).catch(() => null);
     const configSymlink = path.join(os.homedir(), `.${agent}`);
     const configTarget = readSymlinkSafe(configSymlink);
-    // Report the bare shim only when it is actually there. An isolated install
-    // deliberately has none — that is the promise — so printing the path
-    // unconditionally told the user this copy sits on their PATH when it does not.
-    // Same failure mode as `agents view` claiming an isolated copy was `(global)`.
     const shimPathIfPresent = fs.existsSync(path.join(getShimsDir(), AGENTS[agent].cliCommand))
         ? path.join(getShimsDir(), AGENTS[agent].cliCommand)
         : null;
     const aliasPath = getVersionedAliasPath(agent, version);
     const isolated = isVersionIsolated(agent, version);
-    // `default` means the GLOBAL default, which an isolated copy can never be. Report
-    // the isolated pointer separately rather than leaving it invisible: it is what a
-    // bare `agents run <agent>` resolves to, so "default: false" alone is misleading.
     const isIsolatedDefault = isolated && getIsolatedDefault(agent) === version;
     const capabilities = collectCapabilities(agent, version);
     const itemsByKind = options.brief ? null : collectItemsByKind(agent, versionHome);
@@ -1041,7 +855,6 @@ async function renderSummary(agent, version, versionHome, options) {
         console.log(JSON.stringify(json, null, 2));
         return;
     }
-    // Plain text — model sits right beside the version, same priority (no label).
     const configuredModel = resolveConfiguredModel(agent, version)?.model;
     const modelPart = configuredModel ? '  ' + chalk.yellow(configuredModel) : '';
     const head = `${chalk.bold(agent)} ${chalk.gray('@')} ${chalk.cyan(version)}${modelPart}${isDefault ? '  ' + chalk.green('[default]') : ''}${isolated ? '  ' + chalk.gray(isIsolatedDefault ? '[isolated default]' : '[isolated]') : ''}`;
@@ -1064,9 +877,6 @@ async function renderSummary(agent, version, versionHome, options) {
         const reason = res.ok ? '' : chalk.gray(`(${res.reason}${res.need ? ' ' + res.need : ''})`);
         console.log(`  ${cap.padEnd(10)} ${mark} ${reason}`);
     }
-    // Permission modes are a separate axis from the capability booleans above —
-    // same data as `agents modes <agent>`, kept next to the rest of the inspect
-    // surface so a human/agent does not have to leave this view.
     {
         const modeCat = getAgentModesCatalog(agent, version);
         const modeList = modeCat.modes.map((m) => (m.isDefault ? `${m.mode}*` : m.mode)).join('/');
@@ -1094,7 +904,6 @@ async function renderSummary(agent, version, versionHome, options) {
     console.log(chalk.gray(`Diagnose:   agents doctor ${agent}@${version}`));
     console.log('');
 }
-// ─── List mode ───────────────────────────────────────────────────────────────
 async function renderList(agent, version, versionHome, kind, options) {
     const items = collectKind(agent, versionHome, kind);
     await renderItemList(`${agent}@${version}`, { agent, version }, kind, items, options);
@@ -1122,44 +931,17 @@ async function renderItemList(header, jsonHead, kind, items, options, hookManife
         console.log('');
         return;
     }
-    // Route through the shared resource view: an interactive picker with a live
-    // preview in a TTY, a plain aligned table when piped. This is the same
-    // renderer `agents skills list`, `agents commands` and four others already
-    // use — inspect was the last drill-down printing its own two-line-per-entry
-    // dump with a `[source]` tag repeated on every row.
-    //
-    // Sync targets are deliberately off: for a repo the resources ARE the source,
-    // so the column would read "no installed versions" for every row.
     const sources = new Set(items.map(i => i.source));
-    // A hook has no prose description — it is a script. Leaving the column blank
-    // is honest but useless, so it carries what the Hooks view is actually for:
-    // the events that fire it. Same string the overview prints.
     const hookEvents = kind !== 'hooks' ? null : (hookManifest ?? hookManifestByScript(loadCentralHookManifest()));
-    // A routine's byte size says nothing; whether it last ran, and where it is
-    // allowed to run, is the whole question. Devices needs the room, so it only
-    // takes a column on a wide terminal — below that it folds into the description
-    // rather than being dropped.
     const isRoutines = kind === 'routines';
     const routineDevicesColumn = isRoutines && terminalWidth() >= 100;
     await showResourceList({
         resourcePlural: kind,
         resourceSingular: kind.replace(/s$/, ''),
         extraLabel: isRoutines ? 'Last' : 'Size',
-        // Only carry a source column when the rows actually differ; a uniform
-        // `[.system]` on every row is 13 columns spent on one bit of information.
         extra2Label: routineDevicesColumn ? 'Devices' : (isRoutines ? undefined : (sources.size > 1 ? 'Source' : undefined)),
         showSync: false,
-        // inspect's names run long — hook scripts (`00-agent-verify-work-complete`
-        // and its `_test` sibling) and namespaced plugin entries (`/swarm:orchestrate`)
-        // both blow past the 22-char default and truncate to the same string.
-        //
-        // Gate on terminal width, NOT on "do any rows have a description": that
-        // proxy depended on which path built the items — the repo path hardcodes an
-        // empty description so it widened, while the agent path filled one from the
-        // script's shebang so it never did. Same command, two behaviours.
         nameCap: terminalWidth() >= 120 ? 40 : undefined,
-        // Keep names clickable — the pre-picker list wrapped every name in an OSC-8
-        // link to its file, and losing that would be a silent capability regression.
         linkFor: row => items.find(i => i.name === row.name)?.linkTarget,
         emptyMessage: `  (none installed)`,
         rows: items.map(item => ({
@@ -1172,8 +954,6 @@ async function renderItemList(header, jsonHead, kind, items, options, hookManife
                     const problem = extraOf(item, 'problem');
                     if (problem)
                         return problem;
-                    // Compose at the row, leaving item.description pure prose for JSON.
-                    // Also makes "daily" and "zion" searchable in the picker's filter.
                     return [
                         extraOf(item, 'fires'),
                         routineDevicesColumn ? '' : compactDeviceCell(extraOf(item, 'devices')),
@@ -1187,16 +967,9 @@ async function renderItemList(header, jsonHead, kind, items, options, hookManife
                 ? compactDeviceCell(extraOf(item, 'devices')) || 'none'
                 : (isRoutines ? undefined : (sources.size > 1 ? item.source : undefined)),
             targets: [],
-            // Pass the resolved manifest: the picker rebuilds this on every arrow
-            // key, so re-reading ~11 KB of YAML per keystroke is wasted work on top
-            // of being the wrong manifest for a repo target.
             buildDetail: () => previewFor(kind, item, hookEvents ?? new Map()),
         })),
     });
-    // On a TTY the picker's preview pane carries each plugin's bundled skills and
-    // commands. Piped, there is no pane — and the pre-picker output printed those
-    // lines under every row, so a table alone would silently drop what a plugin
-    // actually ships from `agents inspect . --plugins | grep`.
     if (!isInteractiveTerminal() && items.some(i => i.groups?.length)) {
         for (const item of items) {
             if (!item.groups?.length)
@@ -1205,36 +978,21 @@ async function renderItemList(header, jsonHead, kind, items, options, hookManife
             const width = Math.max(...item.groups.map(g => g.label.length));
             for (const g of item.groups) {
                 const colorFn = PLUGIN_GROUP_COLORS[g.label] ?? chalk.white;
-                // Wrapped, not truncated: routing these through a truncating helper is
-                // what once showed 4 of a 10-command plugin at 80 columns.
                 printWrappedJoined(`  ${chalk.gray(g.label.padEnd(width))}  `, g.items.map(s => colorFn(s)), ', ');
             }
         }
         console.log('');
     }
 }
-/**
- * The scannable half of a description: everything before the trigger clause, and
- * only the first sentence of that. 15 of 20 skills in .system lead with their
- * purpose and then append "Triggers on: …" — in a one-line row that boilerplate
- * is what survives truncation, so the row says nothing. The full text still
- * renders in the preview pane.
- */
 export function summaryLine(description) {
     if (!description)
         return '';
     const cutAtTrigger = description.split(/\s*(?:Triggers on|Use this skill when|Invoke when)\b/i)[0];
-    // A description that OPENS with the trigger clause splits to an empty head.
-    // Returning that would blank the row while --json still carried the text —
-    // showing less than we have is worse than showing boilerplate.
     const trimmed = cutAtTrigger.trim() || description.trim();
-    // First sentence, but only when the split leaves something substantial —
-    // "e.g." and friends would otherwise chop a description to a fragment.
     const firstSentence = trimmed.match(/^.*?[.!?](?=\s+[A-Z(`])/)?.[0];
     const candidate = firstSentence && firstSentence.length >= 40 ? firstSentence : trimmed;
     return candidate.replace(/\s+/g, ' ').trim();
 }
-/** Compact size for a list row: `18 KB` for a bundle, `413 B` for a single file. */
 function itemSizeLabel(p) {
     if (!p)
         return '';
@@ -1246,7 +1004,6 @@ function itemSizeLabel(p) {
     const { bytes } = pathSize(p);
     return formatBytes(bytes);
 }
-// ─── Detail mode (fuzzy) ─────────────────────────────────────────────────────
 async function renderDetail(agent, version, versionHome, kind, query, options) {
     const items = collectKind(agent, versionHome, kind);
     renderItemDetail(`${agent}@${version}`, { agent, version }, kind, query, items, options);
@@ -1283,22 +1040,11 @@ function renderItemDetail(header, jsonHead, kind, query, items, options) {
     const matchTag = best.matchKind === 'exact' ? 'exact' : best.matchKind === 'substring' ? 'substring' : `~${best.distance}`;
     console.log(`  ${chalk.green('✓')}  ${termLink(chalk.bold.cyan(best.item.name), best.item.linkTarget)}  ${chalk.gray(`[${matchTag}, ${best.item.source}]`)}`);
     if (best.item.description) {
-        // Wrap to the real terminal width. This used to be truncate(desc, 100): a
-        // character count blind to the window, so a long description lost its
-        // sentence at 80 columns AND wasted the space at 200.
         for (const line of wrapJoined('     ', best.item.description.split(/\s+/), ' ', terminalWidth())) {
             console.log(chalk.gray(line));
         }
     }
     for (const [k, v] of buildDetailRows(best.item, kind)) {
-        // Wrap, never truncate. These values are `, `-joined lists (commands,
-        // skills, triggers, tools) and cutting them hides real entries — a 9-command
-        // plugin would show 4. Before the detail view wrapped at all, the terminal
-        // soft-wrapped these in full, so truncating here would lose information the
-        // old output had.
-        // String(v) is belt-and-braces: pluginToItem now coerces every manifest field
-        // through manifestText, so nothing non-string should reach here. Kept because
-        // this is the choke point every future row kind flows through.
         printWrappedJoined(`     ${chalk.gray(k.padEnd(10))} `, String(v).split(', '), ', ');
     }
     if (others.length > 0) {
@@ -1310,7 +1056,6 @@ function renderItemDetail(header, jsonHead, kind, query, items, options) {
     }
     console.log('');
 }
-// ─── Data collection ─────────────────────────────────────────────────────────
 function collectCapabilities(agent, version) {
     const out = {};
     for (const cap of CAPABILITY_NAMES) {
@@ -1330,18 +1075,12 @@ function collectItemsByKind(agent, versionHome) {
         out[kind] = collectKind(agent, versionHome, kind);
     return out;
 }
-/** A simple-kind count row: `kind  N   user:30 system:12   name, name …(+K)`. */
 function printSimpleResourceRow(kind, items) {
     const count = String(items.length).padStart(4);
     const breakdown = chalk.gray(scopeBreakdownPlain(countBySource(items.map(i => i.source))).padEnd(18));
     const preview = items.length > 0 ? chalk.gray(truncate(previewNames(items, 3), 48)) : '';
     console.log(`  ${kind.padEnd(10)} ${count}   ${breakdown}  ${preview}`.trimEnd());
 }
-/**
- * Build the `resources` JSON: every kind keeps `total` + `bySource` (back-compat),
- * simple kinds add `names`, and the rich kinds add structured `items` (hook
- * events/predicates, mcp transport/url/command, plugin version + group counts).
- */
 function summaryResourcesJson(itemsByKind, hookByScript, mcpConfigs, hookInventory) {
     const out = {};
     for (const kind of DRILLABLE_KINDS) {
@@ -1417,14 +1156,6 @@ function collectKind(agent, versionHome, kind) {
             return agentRoutineItems(agent);
     }
 }
-/**
- * The routines that dispatch this agent.
- *
- * Version-invariant, like the `rules`/`subagents` cases above which also ignore
- * `versionHome`: a routine names an agent, never an agent@version. `listJobs()`
- * with no cwd is deliberate — it is the daemon's own view (user + system), so an
- * unsynced project routine that can never fire is not listed as if it could.
- */
 function agentRoutineItems(agent) {
     const { live } = defaultRoutineLive();
     const now = new Date();
@@ -1447,8 +1178,6 @@ function agentRoutineItems(agent) {
             description: routineDescription(job, ''),
             extra: [
                 ['fires', routineFireLabel(job, now)],
-                // listJobs already applied device activation, so job.enabled is the
-                // effective answer here — no need to re-derive it.
                 ['enabled', job.enabled ? 'yes' : 'no'],
                 ['devices', state.devices.length > 0
                         ? state.devices.join(', ')
@@ -1464,26 +1193,11 @@ function agentRoutineItems(agent) {
 function pluginItems() {
     return discoverPlugins().map(p => pluginToItem(p, 'user'));
 }
-/**
- * Map a discovered plugin to a resource item, surfacing the manifest description
- * and the bundle's nested resources (skills, commands, hooks, ...) as detail rows.
- *
- * EVERY field read here comes from an uncontrolled `plugin.json`:
- * `loadPluginManifest` casts parsed JSON straight to `PluginManifest` and
- * validates only name/version (`lib/plugins.ts`), so the declared types are a
- * hope, not a guarantee. A non-string reaching a renderer throws on `.split` /
- * `.replace`, and `pluginToItem` runs while BUILDING THE LIST — so one malformed
- * manifest anywhere takes down `inspect .`, `--plugins`, `--json`, and even a
- * query for a different, valid plugin. Coerce every field through `manifestText`;
- * never trust the annotation.
- */
 function pluginToItem(plugin, source) {
     const extra = [];
     const version = manifestText(plugin.manifest.version);
     if (version)
         extra.push(['version', version]);
-    // Which execution surfaces the bundle actually carries. Detection already
-    // exists for the plugin picker; the detail view simply never asked for it.
     const surfaces = pluginCapabilityLabels(inspectPluginCapabilities(plugin.root));
     if (surfaces.length > 0)
         extra.push(['surfaces', surfaces.join(', ')]);
@@ -1491,7 +1205,6 @@ function pluginToItem(plugin, source) {
     const authorLabel = manifestText(typeof author === 'object' && author !== null ? author.name : author);
     if (authorLabel)
         extra.push(['author', authorLabel]);
-    // `.length` is truthy for a bare string too, and a string has no `.join`.
     const deps = plugin.manifest.dependencies;
     const depsLabel = Array.isArray(deps)
         ? deps.map(manifestText).filter(Boolean).join(', ')
@@ -1503,18 +1216,11 @@ function pluginToItem(plugin, source) {
         source,
         path: plugin.root,
         linkTarget: linkTarget(plugin.root),
-        // `?? ''` catches only null/undefined — a numeric or array description used
-        // to reach truncateToWidth/`.split` and kill the render for every plugin.
         description: manifestText(plugin.manifest.description),
         extra,
         groups: pluginResourceGroups(plugin),
     };
 }
-/**
- * Render one uncontrolled manifest value as display text. Objects and arrays
- * carry no sensible one-line form, so they become '' (the row is then dropped)
- * rather than `[object Object]`; everything else stringifies.
- */
 function manifestText(v) {
     if (v === null || v === undefined)
         return '';
@@ -1522,12 +1228,6 @@ function manifestText(v) {
         return '';
     return String(v);
 }
-/**
- * Render one uncontrolled value as a list of display strings. A scalar becomes a
- * one-element list — the case `?? []` and `Array.isArray` both miss, and the one
- * that threw on `.join`. Object entries drop out rather than becoming
- * `[object Object]`.
- */
 function manifestList(v) {
     if (v === null || v === undefined)
         return [];
@@ -1544,12 +1244,6 @@ function entriesFromAgentResources(agent, versionHome, kind) {
         source: e.scope,
         path: e.path,
         linkTarget: linkTarget(e.path),
-        // A hook is a shell/Python script with no human description, so the prose
-        // fallback returned code: all 53 under an agent home read "!/usr/bin/env
-        // bash" (the `#` strip eating the shebang), and skipping that line only
-        // promoted `set -euo pipefail`. repoHookItems already hardcodes '' for the
-        // repo path; this makes the agent path agree instead of guessing. The
-        // Hooks view shows firing events in that column.
         description: kind === 'hooks' ? '' : readDescription(e.path),
     }));
 }
@@ -1573,7 +1267,6 @@ function mcpItems(agent, versionHome) {
         description: m.version ? `version ${m.version}` : '',
     }));
 }
-// ─── Detail field builders ───────────────────────────────────────────────────
 function buildDetail(item, kind) {
     const rows = buildDetailRows(item, kind);
     const out = {
@@ -1586,15 +1279,6 @@ function buildDetail(item, kind) {
         out[k] = v;
     return out;
 }
-/**
- * The preview pane for one row, refreshed as the selection moves.
- *
- * Adaptive on purpose — the sessions picker sets the precedent: a Cursor
- * session shows Dirs/Repos/Artifacts, a Codex one shows a different set, and
- * empty fields simply do not render. A hook's useful metadata (what fires it,
- * whether it is wired) has nothing in common with a skill's (what invokes it,
- * where it is synced), so each kind contributes its own rows and blanks drop out.
- */
 export function previewFor(kind, item, hookManifest) {
     const out = [];
     const label = (k, v) => `  ${chalk.gray(k.padEnd(11))}${v}`;
@@ -1606,16 +1290,7 @@ export function previewFor(kind, item, hookManifest) {
         }
     }
     const rows = [];
-    // A hook's identity is when it fires — the summary view has always shown this
-    // while the drill-down showed only a size.
     if (kind === 'hooks') {
-        // Use the manifest the caller already resolved. Re-resolving the CENTRAL
-        // one here made the preview contradict its own row: a repo hook wired by
-        // that repo's agents.yaml showed `PreToolUse(Bash)` in the table and
-        // "not registered" in the pane below it, and the mirror case credited a
-        // central registration to the repo. No `?? loadCentralHookManifest()`
-        // fallback: it would be dead in production and would silently reinstate
-        // exactly that bug the next time a caller forgot the argument.
         const hook = hookManifest.get(item.name);
         if (hook) {
             rows.push(['fires', chalk.yellow(summarizeHook(hook))]);
@@ -1651,7 +1326,6 @@ export function previewFor(kind, item, hookManifest) {
         for (const [k, v] of rows)
             out.push(label(k, v));
     }
-    // A plugin's bundled resources, each on its own line rather than a comma run.
     if (item.groups?.length) {
         for (const g of item.groups) {
             out.push('');
@@ -1664,7 +1338,6 @@ export function previewFor(kind, item, hookManifest) {
     }
     return out.join('\n');
 }
-/** `18 KB · 3 files` for a bundle, `413 B` for a single file. */
 function itemSizeDetail(p) {
     if (!p)
         return '';
@@ -1680,8 +1353,6 @@ function buildDetailRows(item, kind) {
     const rows = [];
     if (item.path && kind !== 'mcp') {
         const stat = safeStat(item.path);
-        // A bundle reports its real recursive weight. `(bundle)` carried no
-        // information, and pathSize/formatBytes already back the summary view.
         if (stat) {
             if (stat.isDirectory()) {
                 const { bytes, files } = pathSize(item.path);
@@ -1692,17 +1363,12 @@ function buildDetailRows(item, kind) {
             }
         }
     }
-    // Kind-specific fields
     if (kind === 'skills' || kind === 'commands' || kind === 'subagents') {
         const fm = readFrontmatter(item.path);
         if (fm) {
-            // description was already printed by caller; skip if redundant
             if (typeof fm.description === 'string' && fm.description.trim() !== item.description.trim()) {
                 rows.push(['description', truncate(fm.description, 120)]);
             }
-            // Frontmatter is uncontrolled YAML too. These were type-guarded against a
-            // crash but still rendered `[object Object]` for an entry that is a map,
-            // and dropped a scalar `triggers: foo` entirely.
             const triggers = manifestList(fm.triggers).join(', ');
             if (triggers)
                 rows.push(['triggers', triggers]);
@@ -1714,35 +1380,19 @@ function buildDetailRows(item, kind) {
                 rows.push(['tools', tools]);
         }
     }
-    // Plugin bundles surface their nested resources (skills, commands, …) plus
-    // scalar rows (version).
     if (kind === 'plugins') {
         if (item.groups)
             for (const g of item.groups)
                 rows.push([g.label, g.items.join(', ')]);
     }
-    // Every kind's scalar rows. previewFor already renders `extra` unconditionally,
-    // so gating it on plugins here made the detail view and its own preview pane
-    // disagree about the same item — and dropped a routine's schedule/status from
-    // `--routine <name> --json` entirely.
     if (item.extra)
         rows.push(...item.extra);
     return rows;
 }
-/** `system` → `sys`; everything else unchanged. Keeps the tag column narrow. */
 function abbrevSource(s) {
     return s === 'system' ? 'sys' : s;
 }
-/**
- * Compact one-liner for a hook from its manifest entry: the firing events (with
- * the matcher/tool-name in parens), then a `·`-separated predicate summary, then
- * an optional cache tail. Plain text — the caller applies color.
- */
 export function summarizeHook(hook) {
-    // `hook` is an unvalidated YAML cast from agents.yaml, so `??` is not enough:
-    // a scalar `events: PreToolUse` is neither null nor an array, and `.join` threw
-    // — taking down bare `agents inspect <repo>`, and via the central manifest
-    // `agents inspect <agent>` on every box. Same shape as the plugin.json bug.
     const events = manifestList(hook.events).join('/') || '(no event)';
     let matcher = manifestText(hook.matcher);
     if (!matcher && hook.matches?.tool_name) {
@@ -1759,12 +1409,9 @@ export function summarizeHook(hook) {
         line += ` (${ttl} cache)`;
     return line;
 }
-/** `·`-separated predicate summary from a hook's `matches:` block (tool_name omitted — shown in the matcher parens). */
 function summarizeMatches(m) {
     if (!m)
         return '';
-    // Every predicate is raw YAML. `truncate` calls `.slice`, so a numeric
-    // `prompt_contains: 12345` threw here just like the events case above.
     const bits = [];
     if (m.git_dirty)
         bits.push('git_dirty');
@@ -1785,25 +1432,19 @@ function summarizeMatches(m) {
         bits.push(`has ${projectHas}`);
     return bits.join(' · ');
 }
-/** Normalize a hook cache shorthand/object to a display ttl ("5m", "1h"); null when uncached. */
 function hookCacheTtl(cache) {
     if (cache === undefined || cache === null)
         return null;
     if (typeof cache === 'string')
         return cache.replace(/-bg$/, '');
-    // A bare `cache: 5` has no `.ttl`, and `cache: {ttl: {…}}` has a non-scalar
-    // one — String() on either rendered `(undefined cache)` / `([object Object]
-    // cache)`. No ttl to show means no cache tail.
     return manifestText(cache.ttl) || null;
 }
-/** Compact one-liner for an MCP server: padded transport + the url (http) or command line (stdio). */
 export function summarizeMcp(cfg) {
     const target = cfg.transport === 'http'
         ? (cfg.url ?? '')
         : [cfg.command, ...(cfg.args ?? [])].filter(Boolean).join(' ');
     return `${cfg.transport.padEnd(5)}  ${truncate(target, 60)}`.trimEnd();
 }
-/** Print `Title (N)` then up to `max` aligned `[source] name  detail` rows with a `…(+K)` tail. */
 function printExpandedSection(title, rows, max = 6) {
     console.log('\n' + chalk.bold(title) + chalk.gray(` (${rows.length})`));
     if (rows.length === 0) {
@@ -1822,23 +1463,15 @@ function printExpandedSection(title, rows, max = 6) {
     if (rows.length > max)
         console.log(chalk.gray(`  …(+${rows.length - max})`));
 }
-/** Tally a source list into `{user: n, system: m}`. */
 function countBySource(sources) {
     const out = {};
     for (const s of sources)
         out[s] = (out[s] || 0) + 1;
     return out;
 }
-/** Unbracketed scope breakdown for the simple count rows: `user:30 system:12`. */
 function scopeBreakdownPlain(bySource) {
     return Object.entries(bySource).map(([k, v]) => `${k}:${v}`).join(' ');
 }
-/**
- * Index a hook manifest by script basename (no extension). Installed hooks are
- * named after their script file (`04-capture-…`), while the manifest is keyed by
- * logical name (`capture-…`) with the filename in `script:` — so we join on the
- * script basename, not the manifest key.
- */
 export function hookManifestByScript(manifest) {
     const out = new Map();
     for (const hook of Object.values(manifest)) {
@@ -1848,12 +1481,7 @@ export function hookManifestByScript(manifest) {
     }
     return out;
 }
-/** Build hook rows by enriching the installed hook items with manifest events/predicates. */
 function hookRows(items, byScript) {
-    // The section shows only the first handful before a `…(+N)` tail, so order
-    // matters: wired hooks first. Alphabetically `00-…_test` sorts next to the
-    // hook it tests, so half the visible rows were test scaffolding with a blank
-    // event column while real registered hooks hid behind the tail.
     const ordered = [...items].sort((a, b) => {
         const aw = byScript.has(a.name) ? 0 : 1;
         const bw = byScript.has(b.name) ? 0 : 1;
@@ -1865,13 +1493,10 @@ function hookRows(items, byScript) {
             source: item.source,
             name: item.name,
             linkTarget: item.linkTarget,
-            // Hooks are shell scripts with no human description — show events/predicates
-            // from the manifest, or nothing rather than a meaningless shebang line.
             detail: hook ? summarizeHook(hook) : '',
         };
     });
 }
-/** Build plugin rows: `vVERSION  skills:6 commands:5 …` from the bundle's groups. */
 function pluginRows(items) {
     return items.map(item => {
         const version = item.extra?.find(([k]) => k === 'version')?.[1];
@@ -1880,16 +1505,7 @@ function pluginRows(items) {
         return { source: item.source, name: item.name, detail, linkTarget: item.linkTarget };
     });
 }
-/** Statuses that mean the last run did not do its job. */
 const UNHEALTHY_RUN = new Set(['failed', 'timeout', 'missed', 'blocked']);
-/**
- * Health tier for the section ordering. Lower sorts first.
- *
- * `printExpandedSection` shows only the first handful before a `…(+N)` tail, so
- * for a repo with 24 routines the sort IS the section. Same reasoning as
- * `hookRows` putting wired hooks first: a routine that fires nowhere is exactly
- * what the reader opened this for, and must never hide behind the tail.
- */
 export function routineHealthTier(item) {
     if (extraOf(item, 'problem'))
         return 0;
@@ -1906,34 +1522,23 @@ export function routineHealthTier(item) {
         return 3;
     return 4;
 }
-/** Build routine rows: `daily at 9:00 AM   zion   completed · 2h ago`. Broken first. */
 function routineRows(items) {
     const ordered = [...items].sort((a, b) => {
         const at = routineHealthTier(a), bt = routineHealthTier(b);
         return at !== bt ? at - bt : a.name.localeCompare(b.name);
     });
     const nameW = ordered.length > 0 ? Math.max(...ordered.slice(0, 6).map(r => r.name.length)) : 0;
-    // Budget the three cells against what printExpandedSection leaves after the
-    // `[source]` tag and the name column, so the status never truncates to `fai…`.
     const budget = Math.max(30, terminalWidth() - nameW - 14);
     const firesW = Math.min(21, Math.max(12, Math.floor(budget * 0.42)));
     const devicesW = Math.min(14, Math.max(7, Math.floor(budget * 0.22)));
     return ordered.map(item => {
         const problem = extraOf(item, 'problem');
         const fires = extraOf(item, 'fires');
-        // A routine with no `fires` never parsed — the problem IS the whole row. One
-        // that parsed but carries a problem (a duplicate-extension sibling) keeps its
-        // schedule and status, with the problem appended rather than replacing them.
         if (problem && !fires) {
             return { source: item.source, name: item.name, linkTarget: item.linkTarget, detail: chalk.red(problem) };
         }
         const devices = compactDeviceCell(extraOf(item, 'devices'));
         const last = extraOf(item, 'last');
-        // Pad the PLAIN strings, then colour. Padding a chalk-wrapped string counts
-        // the escape codes as width and silently steals columns from the next cell.
-        // The trailing problem gets what the three sized cells leave, or it would
-        // blow the row past the terminal and break the alignment this budget exists
-        // to hold — a 147-column row at COLUMNS=100 in testing.
         const problemW = Math.max(12, budget - firesW - devicesW - stringWidth(last) - 8);
         const detail = [
             chalk.gray(truncateToWidth(fires, firesW).padEnd(firesW)),
@@ -1949,10 +1554,6 @@ function routineRows(items) {
         };
     });
 }
-/**
- * Routine rows for an agent target: `daily at 9:00 AM   completed · 2h ago`.
- * No devices column — an agent view is inherently about this machine.
- */
 function agentRoutineRows(items) {
     const ordered = [...items].sort((a, b) => {
         const at = routineHealthTier(a), bt = routineHealthTier(b);
@@ -1964,13 +1565,6 @@ function agentRoutineRows(items) {
         detail: `${extraOf(item, 'fires').padEnd(20)}  ${colorLast(extraOf(item, 'last'))}`,
     }));
 }
-/**
- * `ok 2h` / `fail 3d` / `never` — the Last column, clamped to the 10-wide cell.
- *
- * The picker path pads without truncating, so an 11-char `blocked 3d` would push
- * the next column; the abbreviations cover only the two commonest statuses, so
- * the clamp is what actually holds the invariant.
- */
 function routineLastCell(item, width = 10) {
     const last = extraOf(item, 'last');
     if (!last || last === 'never run')
@@ -1979,12 +1573,6 @@ function routineLastCell(item, width = 10) {
     const word = status === 'completed' ? 'ok' : status === 'failed' ? 'fail' : status;
     return truncateToWidth(age ? `${word} ${age}` : word, width);
 }
-/**
- * Trim the long-form `devices` phrasing down to a cell.
- *
- * The extra row is written for the detail pane ("no device — will not fire");
- * a 16-column cell needs the same fact in two words.
- */
 function compactDeviceCell(devices) {
     if (devices.startsWith('no device —'))
         return 'none ⚠';
@@ -2003,12 +1591,6 @@ function colorLast(last) {
         return chalk.red(last);
     return chalk.gray(last);
 }
-/**
- * The one-line warning under the Routines section, or null when it would mislead.
- *
- * Silent until some device has materialized a `routines:` list — before that,
- * every routine is legitimately unpinned and "will not fire" would be noise.
- */
 export function routineDarkWarning(items) {
     const materialized = items.some(i => {
         const d = extraOf(i, 'devices');
@@ -2019,21 +1601,17 @@ export function routineDarkWarning(items) {
     const dark = items.filter(i => extraOf(i, 'devices').startsWith('no device —')).length;
     if (dark === 0)
         return null;
-    // The noun pluralizes on the total ("1 of 2 routines"), the verb on the count
-    // that is actually dark ("... is on no device's allowlist").
     const noun = items.length === 1 ? 'routine' : 'routines';
     const verb = dark === 1 ? 'is' : 'are';
     const pronoun = dark === 1 ? 'it' : 'they';
     return `${dark} of ${items.length} ${noun} ${verb} on no device's allowlist — ${pronoun} will not fire.`;
 }
-/** Build MCP rows by joining the installed mcp items with their full configs (transport/url/command). */
 function mcpRows(items, configs) {
     return items.map(item => {
         const cfg = configs.get(item.name);
         return { source: item.source, name: item.name, detail: cfg ? summarizeMcp(cfg) : item.description };
     });
 }
-/** Read a repo/agents.yaml `hooks:` section into a name→ManifestHook map (best-effort). */
 export function hookManifestFromFile(agentsYamlPath) {
     try {
         const meta = yaml.parse(fs.readFileSync(agentsYamlPath, 'utf-8'));
@@ -2043,29 +1621,12 @@ export function hookManifestFromFile(agentsYamlPath) {
         return {};
     }
 }
-/**
- * Merge the system + user `agents.yaml` hook manifests (user wins on key
- * collision). Built directly from the two layer files rather than via
- * `parseHookManifest()` so inspecting never emits the shadow/override warnings
- * that the registrar path prints.
- */
 function loadCentralHookManifest() {
     return {
         ...hookManifestFromFile(path.join(getSystemAgentsDir(), 'agents.yaml')),
         ...hookManifestFromFile(path.join(getUserAgentsDir(), 'agents.yaml')),
     };
 }
-/**
- * Hook items for a repo's Hooks section. Uses the grouped hook reader (script +
- * data file collapsed into one entry, non-hook files like promptcuts.yaml or
- * README.md filtered out) rather than a naive readdir, so names are clean and
- * join cleanly against the manifest by script basename.
- *
- * `description` is intentionally blank: a hook is a script, and the only text a
- * readdir-based reader could scrape from one is its shebang (the old flat path
- * surfaced `!/usr/bin/env bash` as a description). The Hooks section shows firing
- * events via summarizeHook instead, which is the useful signal.
- */
 function repoHookItems(repo) {
     return listHookEntriesFromDir(path.join(repo.root, 'hooks')).map(h => ({
         name: h.name,
@@ -2091,7 +1652,6 @@ function findMatches(items, query) {
         out.sort((a, b) => rankMatch(a) - rankMatch(b));
         return out;
     }
-    // No substring hits — fall back to edit distance.
     const threshold = Math.max(2, Math.floor(q.length * 0.3));
     for (const item of items) {
         const d = damerauLevenshtein(q, item.name.toLowerCase());
@@ -2114,16 +1674,12 @@ function suggestClosest(items, query, n) {
     scored.sort((a, b) => a.d - b.d);
     return scored.slice(0, n).map(s => s.item);
 }
-// ─── Frontmatter + description helpers ───────────────────────────────────────
 function readDescription(p) {
     if (!p)
         return '';
     let filePath = p;
     try {
         if (fs.statSync(p).isDirectory()) {
-            // `rule.md` is the directory form of a subrule (SUBRULE_RULE_FILE in
-            // lib/rules/compose.ts); without it the four dir-form subrules on disk
-            // drill in with a blank preview.
             for (const marker of ['SKILL.md', 'WORKFLOW.md', 'AGENT.md', 'rule.md', 'README.md']) {
                 const c = path.join(p, marker);
                 if (fs.existsSync(c)) {
@@ -2195,7 +1751,6 @@ function readFirstProseLine(p) {
         if (stat.size > 64 * 1024)
             return '';
         const text = fs.readFileSync(p, 'utf-8');
-        // Skip frontmatter
         let body = text;
         if (body.startsWith('---')) {
             const end = body.indexOf('\n---', 3);
@@ -2211,10 +1766,9 @@ function readFirstProseLine(p) {
             return line;
         }
     }
-    catch { /* ignore */ }
+    catch { }
     return '';
 }
-// ─── OSC-8 + path helpers ────────────────────────────────────────────────────
 function linkTarget(p) {
     if (!p)
         return '';