@phnx-labs/agents-cli 1.22.35 → 1.22.38

package/dist/commands/resource-view.js

@@ -7,11 +7,15 @@
  * filtering, and paging.
  */
 import chalk from 'chalk';
-import { truncate, padVisible } from '../lib/format.js';
+import { truncate, padVisible, termLink } from '../lib/format.js';
 import { agentLabel } from '../lib/agents.js';
 import { itemPicker } from '../lib/picker.js';
 import { isInteractiveTerminal, isPromptCancelled, printWithPager } from './utils.js';
 import { terminalWidth, truncateToWidth, padToWidth, stringWidth, stripAnsi } from '../lib/session/width.js';
+/** Whether the Synced column renders for this view. */
+function syncEnabled(opts) {
+    return opts.showSync !== false;
+}
 /** Display a resource list: interactive picker in TTY mode, plain table otherwise. */
 export async function showResourceList(opts) {
     if (opts.json) {
@@ -74,7 +78,12 @@ function filterRows(rows, query) {
 }
 /** Row label rendered inside the picker list. */
 function formatPickerRow(row, opts) {
-    const name = chalk.cyan(padVisible(row.name, 22));
+    // Link here, not in the wide table: that renders only when stdout is NOT a
+    // TTY, and termLink is a no-op off-TTY — so a link there could never appear.
+    // The picker is the TTY path, which is where the pre-picker list had one.
+    const linkTarget = opts.linkFor?.(row);
+    const padded = chalk.cyan(padVisible(row.name, opts.nameCap ?? 22));
+    const name = linkTarget ? termLink(padded, linkTarget) : padded;
     const extra = opts.extraLabel
         ? chalk.gray(padVisible(row.extra ?? '-', 10))
         : '';
@@ -82,6 +91,10 @@ function formatPickerRow(row, opts) {
         ? chalk.gray(padVisible(row.extra2 ?? '-', 16))
         : '';
     const descRaw = row.description ? truncate(row.description, 40) : '';
+    if (!syncEnabled(opts)) {
+        // No sync column: give the reclaimed width back to the description.
+        return `${name} ${extra}${extra2}${chalk.gray(row.description ? truncate(row.description, 84) : '')}`;
+    }
     const desc = padVisible(chalk.gray(descRaw), 42);
     const sync = formatSyncSummary(row.targets, opts);
     return `${name} ${extra}${extra2}${desc} ${sync}`;
@@ -100,7 +113,9 @@ export function resourceLayout(cols, o) {
     const extraW = o.hasExtra ? 10 : 0;
     const extra2W = o.hasExtra2 ? 16 : 0;
     // Cap Sync so a long "missing on …" tail can't starve the description.
-    const syncW = Math.min(o.syncW, Math.max(14, Math.floor(cols * 0.32)));
+    // syncW 0 means the caller dropped the column — keep it 0 rather than
+    // floor it to 14, which would reserve width for a column we never draw.
+    const syncW = o.syncW === 0 ? 0 : Math.min(o.syncW, Math.max(14, Math.floor(cols * 0.32)));
     const fixed = o.nameW + 1 + (extraW ? extraW + 1 : 0) + (extra2W ? extra2W + 1 : 0) + 1 + syncW;
     const descW = cols - fixed;
     return {
@@ -116,8 +131,10 @@ export function resourceLayout(cols, o) {
 function printResourceTable(opts) {
     const cols = terminalWidth();
     const syncStrings = opts.rows.map((r) => formatSyncSummary(r.targets, opts));
-    const nameW = Math.min(NAME_CAP, Math.max('Name'.length, ...opts.rows.map((r) => stringWidth(r.name))));
-    const syncMax = Math.max('Synced'.length, ...syncStrings.map((s) => stringWidth(s)));
+    const nameW = Math.min(opts.nameCap ?? NAME_CAP, Math.max('Name'.length, ...opts.rows.map((r) => stringWidth(r.name))));
+    const syncMax = syncEnabled(opts)
+        ? Math.max('Synced'.length, ...syncStrings.map((s) => stringWidth(s)))
+        : 0;
     const layout = resourceLayout(cols, {
         hasExtra: Boolean(opts.extraLabel),
         hasExtra2: Boolean(opts.extra2Label),
@@ -153,28 +170,33 @@ function renderResourceWideTable(opts, L, syncStrings) {
     if (L.extra2W)
         headerParts.push(cell(opts.extra2Label ?? '', L.extra2W, chalk.bold));
     headerParts.push(cell('Description', L.descW, chalk.bold));
-    headerParts.push(chalk.bold('Synced'));
-    console.log(headerParts.join(' '));
-    const contentW = L.nameW + 1 + (L.extraW ? L.extraW + 1 : 0) + (L.extra2W ? L.extra2W + 1 : 0) + L.descW + 1 + L.syncW;
+    if (L.syncW)
+        headerParts.push(chalk.bold('Synced'));
+    console.log(headerParts.join(' ').trimEnd());
+    const contentW = L.nameW + 1 + (L.extraW ? L.extraW + 1 : 0) + (L.extra2W ? L.extra2W + 1 : 0) + L.descW + (L.syncW ? 1 + L.syncW : 0);
     console.log(chalk.gray('─'.repeat(Math.min(contentW, terminalWidth()))));
     opts.rows.forEach((row, i) => {
+        // No link here: this path renders only when stdout is not a TTY, where
+        // termLink is a no-op. The picker (formatPickerRow) carries the link.
         const parts = [cell(row.name, L.nameW, chalk.cyan)];
         if (L.extraW)
             parts.push(cell(row.extra ?? '-', L.extraW));
         if (L.extra2W)
             parts.push(cell(row.extra2 ?? '-', L.extra2W));
         parts.push(cell(row.description ?? '-', L.descW, chalk.gray));
-        let sync = syncStrings[i] ?? '';
-        if (stringWidth(sync) > L.syncW)
-            sync = chalk.gray(truncateToWidth(sync, L.syncW));
-        parts.push(sync);
-        console.log(parts.join(' '));
+        if (L.syncW) {
+            let sync = syncStrings[i] ?? '';
+            if (stringWidth(sync) > L.syncW)
+                sync = chalk.gray(truncateToWidth(sync, L.syncW));
+            parts.push(sync);
+        }
+        console.log(parts.join(' ').trimEnd());
     });
 }
 /** Stacked cards for narrow terminals: name + meta on one line, description below. */
 function renderResourceCards(opts, cols, syncStrings) {
     opts.rows.forEach((row, i) => {
-        const meta = [row.extra, row.extra2, stripAnsi(syncStrings[i] ?? '')]
+        const meta = [row.extra, row.extra2, syncEnabled(opts) ? stripAnsi(syncStrings[i] ?? '') : '']
             .filter((s) => Boolean(s && s.trim()))
             .join(' · ');
         const metaBudget = Math.max(8, cols - stringWidth(row.name) - 2);

package/dist/commands/resume.js

@@ -59,8 +59,15 @@ export function registerResumeCommand(program) {
         const pinnedHere = consumeResumePinned() || !!options.here;
         const outcome = await resolveSessionMetadataValue(sessionId.trim());
         if (outcome.kind === 'partial') {
-            console.error(chalk.red(`Could not resolve session while these devices were unavailable: ${outcome.failedPeers.join(', ')}`));
-            process.exitCode = 2;
+            // RUSH-2492: an unreachable peer is a warning, not a hard failure. The
+            // resolver already resolves an id found on the reachable fleet (SES-9a),
+            // so reaching here means the session was not found on any device we COULD
+            // reach — it may live on an unreachable peer, which we could not check.
+            const offline = outcome.failedPeers;
+            console.error(chalk.yellow(`Warning: ${offline.length} device(s) unreachable, not checked: ${offline.join(', ')}`));
+            console.error(chalk.red(`No session matching "${sessionId}" on any reachable device (${offline.length} unreachable, not checked).`));
+            console.error(chalk.gray('  If it lives on an offline box, wake it (agents devices) or run there: agents ssh <device>'));
+            process.exitCode = 1;
             return;
         }
         if (outcome.kind === 'not-found') {

package/dist/commands/routines.d.ts

@@ -46,8 +46,6 @@ export declare function groupRoutineJobsByDevice(jobs: JobConfig[], registry: De
  */
 export declare function groupRoutineJobsByProject(jobs: JobConfig[], knownProjectNames: Set<string>): RoutineListGroup[];
 export declare function buildRunsJson(runs: RunMeta[]): Record<string, unknown>[];
-/** Build the exact structured routine rows shared by `routines list --json`
- * and the one-process AGI Menu snapshot. */
 export declare function buildRoutineListJson(): Record<string, unknown>[];
 /** Register the `agents routines` command tree. */
 export declare function registerRoutinesCommands(program: Command): void;