@phnx-labs/agents-cli 1.20.30 → 1.20.32

package/dist/lib/session/render.js

@@ -11,6 +11,7 @@ import { summarizeToolUse } from './parse.js';
 import { cleanSessionPrompt, extractSessionTopic } from './prompt.js';
 import { renderMarkdown } from '../markdown.js';
 import { redactSecrets } from '../redact.js';
+import { classifyFileChanges, changeCounts, toolHistogram, detectTestResult } from './digest.js';
 // ── Path helpers ──────────────────────────────────────────────────────────────
 /**
  * Return absPath relative to cwd; fall back to ~/… then absolute.
@@ -153,6 +154,7 @@ export function collapseRetries(commands) {
 /** Compute aggregate statistics (turns, tools, tokens, duration) from session events. */
 export function computeSummaryStats(events) {
     const modelSet = new Set();
+    const toolCounts = {};
     let userTurns = 0;
     let assistantTurns = 0;
     let toolCount = 0;
@@ -177,6 +179,8 @@ export function computeSummaryStats(events) {
         }
         else if (e.type === 'tool_use' && !e._local) {
             toolCount++;
+            if (e.tool)
+                toolCounts[e.tool] = (toolCounts[e.tool] ?? 0) + 1;
         }
         else if (e.type === 'error') {
             errorCount++;
@@ -193,6 +197,7 @@ export function computeSummaryStats(events) {
         userTurns,
         assistantTurns,
         toolCount,
+        toolCounts,
         errorCount,
         outputTokens,
         cacheReadTokens,
@@ -426,6 +431,76 @@ function renderActivityLine(item) {
             return chalk.green('Msg  ') + ' ' + chalk.gray('"' + trim(item.label) + '"');
     }
 }
+// ── Catch-up digest sections ──────────────────────────────────────────────────
+const OP_GLYPH = {
+    created: (s) => chalk.green(s),
+    modified: (s) => chalk.yellow(s),
+    deleted: (s) => chalk.red(s),
+};
+const OP_MARK = { created: '+', modified: '~', deleted: '−' };
+/**
+ * Render the Changes section: files grouped by directory, each tagged with its
+ * create/modify/delete lifecycle, plus a `+N ~N −N` summary. Replaces the old
+ * flat "Modified" list. Returns true if anything was rendered.
+ */
+function renderChangesSection(lines, events, cwd) {
+    // In-project changes only; edits outside cwd (e.g. /tmp) keep their own
+    // "External edits" section so they don't clutter the project's changeset.
+    const inCwd = (p) => !cwd || !p.startsWith('/') || p.startsWith(cwd + '/');
+    const changes = classifyFileChanges(events).filter(ch => inCwd(ch.path));
+    if (changes.length === 0)
+        return false;
+    const c = changeCounts(changes);
+    const opByRel = new Map();
+    for (const ch of changes)
+        opByRel.set(relativeToCwd(ch.path, cwd), ch.op);
+    const summary = [
+        c.created ? chalk.green(`+${c.created}`) : '',
+        c.modified ? chalk.yellow(`~${c.modified}`) : '',
+        c.deleted ? chalk.red(`−${c.deleted}`) : '',
+    ].filter(Boolean).join(' ');
+    lines.push(chalk.bold('Changes') + chalk.gray(` (${changes.length})  `) + summary);
+    const groups = groupByParentDir(changes.map(ch => ch.path), cwd);
+    const single = groups.size === 1;
+    for (const [dir, files] of groups) {
+        // Single dir: show the full relative path per file (dir/base). Multiple
+        // dirs: a dir header, then bare filenames under it.
+        if (!single)
+            lines.push('  ' + chalk.dim(dir + '/'));
+        for (const f of files.sort()) {
+            const rel = dir === '.' ? f : `${dir}/${f}`;
+            const op = opByRel.get(rel) ?? 'modified';
+            const shown = single ? rel : f;
+            const name = op === 'deleted' ? chalk.strikethrough(chalk.gray(shown)) : shown;
+            lines.push((single ? '  ' : '    ') + OP_GLYPH[op](OP_MARK[op]) + ' ' + name);
+        }
+    }
+    lines.push('');
+    return true;
+}
+/** Render the tool histogram: `Edit 61 · Bash 48 · Read 35 …`. */
+function renderToolsSection(lines, stats) {
+    const hist = toolHistogram(stats.toolCounts, 8);
+    if (hist.length === 0)
+        return;
+    const parts = hist.map(h => `${chalk.white(h.tool)} ${chalk.gray(String(h.count))}`);
+    lines.push(chalk.bold('Tools') + '  ' + parts.join(chalk.gray(' · ')));
+    lines.push('');
+}
+/** Render the last test/build verdict, e.g. `Tests  tests: 294 pass · 4 fail`. */
+function renderTestsLine(lines, events) {
+    const r = detectTestResult(events);
+    if (!r || !r.ok)
+        return;
+    const bits = [];
+    if (r.passed !== undefined)
+        bits.push(chalk.green(`${r.passed} pass`));
+    if (r.failed !== undefined)
+        bits.push(r.failed > 0 ? chalk.red(`${r.failed} fail`) : chalk.gray('0 fail'));
+    const verdict = r.failed && r.failed > 0 ? chalk.red('✗') : chalk.green('✓');
+    lines.push(chalk.bold('Tests') + `  ${verdict} ${chalk.cyan(r.runner)} ${bits.join(chalk.gray(' · '))}`);
+    lines.push('');
+}
 // ── Main summary renderer ─────────────────────────────────────────────────────
 /**
  * Render session as an activity summary.
@@ -560,7 +635,6 @@ export function renderSummary(events, cwd) {
         }
         return m;
     };
-    const modifiedAbsMap = buildAbsMap(filesModifiedAbs);
     const readAbsMap = buildAbsMap(filesReadAbs);
     // ── Render sections ───────────────────────────────────────────────────────
     const lines = [''];
@@ -640,15 +714,14 @@ export function renderSummary(events, cwd) {
             chalk.gray(`: ${errors.length} failure${errors.length !== 1 ? 's' : ''} — first: ${firstDesc}`));
         lines.push('');
     }
-    // 6. Modified files
-    if (filesModifiedAbs.size > 0) {
-        lines.push(chalk.bold('Modified') + chalk.gray(` (${filesModifiedAbs.size})`));
-        const groups = groupByParentDir(filesModifiedAbs, cwd);
-        renderFileGroup(lines, groups, modifiedAbsMap);
-        lines.push('');
-    }
-    // 6b. External edits (files edited outside the project root — typically /tmp)
-    // Filter out plan files (already shown in Plan section)
+    // 6. Changes — files grouped by directory with create/modify/delete lifecycle
+    // (replaces the old flat "Modified" + "External edits" lists).
+    renderChangesSection(lines, events, cwd);
+    // 6b. Catch-up signals: last test/build verdict, then the tool histogram.
+    renderTestsLine(lines, events);
+    renderToolsSection(lines, computeSummaryStats(events));
+    // 6c. External edits (files edited outside the project root — typically /tmp).
+    // Filter out plan files (already shown in Plan section).
     const externalNonPlan = [...filesModifiedExternal].filter(p => !(p.includes('.claude/plans/') && p.endsWith('.md')));
     if (externalNonPlan.length > 0) {
         const externalList = externalNonPlan.sort();

package/dist/lib/state.d.ts

@@ -156,6 +156,10 @@ export declare function getTeamsAgentsDir(): string;
 export declare function getTeamsRegistryPath(): string;
 /** Path to the device registry — SSH device profiles with platform/auth metadata. Durable runtime, per-machine (host list + addresses are NOT pulled by `agents repo push`). */
 export declare function getDevicesRegistryPath(): string;
+/** Path to the device ignore-list — tailscale node names the user dismissed, so auto-discovery never re-suggests them. Per-machine, same dir as the registry. */
+export declare function getDevicesIgnoredPath(): string;
+/** Dir of "pending device" sentinels (~/.agents/.cache/state/devices-pending/) — one empty-ish file per newly-discovered, not-yet-approved tailnet node. Written by the daemon probe, read by the menu-bar helper (mirrors the attention sentinel dir). */
+export declare function getDevicesPendingDir(): string;
 /** Path to cloud dispatch cache (~/.agents/.cache/cloud/). */
 export declare function getCloudDir(): string;
 /** Path to terminal session metadata (~/.agents/.cache/terminals/). */

package/dist/lib/state.js

@@ -352,6 +352,10 @@ export function getTeamsAgentsDir() { return TEAMS_AGENTS_DIR; }
 export function getTeamsRegistryPath() { return path.join(HISTORY_DIR, 'teams', 'registry.json'); }
 /** Path to the device registry — SSH device profiles with platform/auth metadata. Durable runtime, per-machine (host list + addresses are NOT pulled by `agents repo push`). */
 export function getDevicesRegistryPath() { return path.join(HISTORY_DIR, 'devices', 'registry.json'); }
+/** Path to the device ignore-list — tailscale node names the user dismissed, so auto-discovery never re-suggests them. Per-machine, same dir as the registry. */
+export function getDevicesIgnoredPath() { return path.join(HISTORY_DIR, 'devices', 'ignored.json'); }
+/** Dir of "pending device" sentinels (~/.agents/.cache/state/devices-pending/) — one empty-ish file per newly-discovered, not-yet-approved tailnet node. Written by the daemon probe, read by the menu-bar helper (mirrors the attention sentinel dir). */
+export function getDevicesPendingDir() { return path.join(RUNTIME_STATE_DIR, 'devices-pending'); }
 /** Path to cloud dispatch cache (~/.agents/.cache/cloud/). */
 export function getCloudDir() { return CLOUD_DIR; }
 /** Path to terminal session metadata (~/.agents/.cache/terminals/). */

package/dist/lib/sync-umbrella.d.ts

@@ -57,6 +57,11 @@ export interface UmbrellaResult {
         pulled: number;
         merged: number;
     };
+    devices?: {
+        synced: number;
+        pending: number;
+        skipped: boolean;
+    };
     reconciled: boolean;
 }
 export interface RunUmbrellaArgs {

package/dist/lib/sync-umbrella.js

@@ -120,6 +120,20 @@ export async function runUmbrellaSync(args) {
         const { refresh } = await import('./refresh.js');
         await refresh({ skipPrompts: yes });
         result.reconciled = true;
+        // Keep already-registered devices' reachability current, and surface newly
+        // appeared tailnet nodes as "pending" for the menu-bar Register/Ignore gate
+        // rather than silently adding them (refresh mode). Soft: a machine without
+        // tailscale is a clean no-op, never a sync failure. First-run population is
+        // `agents setup` / manual `agents devices sync` (bootstrap).
+        const { runDeviceSync } = await import('./devices/sync.js');
+        const { reconcilePendingSentinels } = await import('./devices/pending.js');
+        const dev = await runDeviceSync({ soft: true, mode: 'refresh' });
+        if (dev.ok)
+            reconcilePendingSentinels(dev.pending);
+        result.devices = { synced: dev.synced, pending: dev.pending.length, skipped: !dev.ok };
+        if (dev.ok) {
+            log(`devices: ${dev.synced} refreshed${dev.pending.length ? `, ${dev.pending.length} new pending` : ''}`);
+        }
     }
     return result;
 }

package/dist/lib/versions.d.ts

@@ -197,6 +197,17 @@ export declare function installVersion(agent: AgentId, version: string, onProgre
  * when nothing was resolved or no stale dir is present, so it is safe to call
  * on every script-based install. Returns the action taken (for tests/logging).
  */
+/**
+ * Proactively fold a stale `latest` version-home into its concrete version,
+ * WITHOUT needing a fresh install (RUSH-1320). `reconcileStaleLatestDir` only
+ * fires at install time, so a `latest` dir left by an old probe-failed install
+ * lingers in `agents view` indefinitely. This resolves the live CLI version and
+ * reconciles — cheap no-op when there's no `latest` dir (the common case, so
+ * `agents view` pays a `--version` shell-out only the once, until it's folded).
+ * Skipped when the active config symlink still points at `latest`, since
+ * renaming that dir would dangle the live symlink.
+ */
+export declare function reconcileStaleLatestForAgent(agent: AgentId): Promise<void>;
 export declare function reconcileStaleLatestDir(agent: AgentId, installedVersion: string): Promise<'none' | 'renamed' | 'trashed'>;
 /**
  * Soft-delete a version directory by moving it to ~/.agents/.system/trash/versions/.

package/dist/lib/versions.js

@@ -1209,6 +1209,26 @@ function removeInstallArtifacts(versionDir) {
  * when nothing was resolved or no stale dir is present, so it is safe to call
  * on every script-based install. Returns the action taken (for tests/logging).
  */
+/**
+ * Proactively fold a stale `latest` version-home into its concrete version,
+ * WITHOUT needing a fresh install (RUSH-1320). `reconcileStaleLatestDir` only
+ * fires at install time, so a `latest` dir left by an old probe-failed install
+ * lingers in `agents view` indefinitely. This resolves the live CLI version and
+ * reconciles — cheap no-op when there's no `latest` dir (the common case, so
+ * `agents view` pays a `--version` shell-out only the once, until it's folded).
+ * Skipped when the active config symlink still points at `latest`, since
+ * renaming that dir would dangle the live symlink.
+ */
+export async function reconcileStaleLatestForAgent(agent) {
+    if (!fs.existsSync(getVersionDir(agent, 'latest')))
+        return;
+    if (getConfigSymlinkVersion(agent) === 'latest')
+        return;
+    const concrete = await getCliVersionFromPath(agent);
+    if (concrete && concrete !== 'latest') {
+        await reconcileStaleLatestDir(agent, concrete);
+    }
+}
 export async function reconcileStaleLatestDir(agent, installedVersion) {
     if (installedVersion === 'latest')
         return 'none';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@phnx-labs/agents-cli",
-  "version": "1.20.30",
+  "version": "1.20.32",
   "description": "One CLI for all your AI coding agents - versions, config, cloud dispatch, sessions, and teams (now with first-class Grok Build CLI support)",
   "type": "module",
   "main": "dist/index.js",