@skrr-ai/cli 0.1.40 → 0.1.42

package/dist/commands/harnesses/installers.d.ts

@@ -11,7 +11,7 @@ import { BaseCommand } from '../../base-command';
  *
  * That gap is not cosmetic, because `skrr harnesses list` answers a DIFFERENT
  * question and looks like it answers this one. Its `status` is the daemon's
- * socket, so on this account `skrr-code` read `online` with no binary installed
+ * socket, so on this account the first-party harness read `online` with no binary installed
  * at all, and `harnesses ping` answered it in 78ms — the daemon was there, the
  * harness was not. A user could not discover that from inside the product; it
  * took `command -v` in a shell.

package/dist/commands/harnesses/installers.js

@@ -4,7 +4,6 @@ const core_1 = require("@oclif/core");
 const base_command_1 = require("../../base-command");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const cli_installers_1 = require("../../lib/cli-installers");
-const daemon_ref_1 = require("../../lib/daemon-ref");
 const daemon_target_1 = require("../../lib/daemon-target");
 const format_1 = require("../../lib/format");
 /**
@@ -19,7 +18,7 @@ const format_1 = require("../../lib/format");
  *
  * That gap is not cosmetic, because `skrr harnesses list` answers a DIFFERENT
  * question and looks like it answers this one. Its `status` is the daemon's
- * socket, so on this account `skrr-code` read `online` with no binary installed
+ * socket, so on this account the first-party harness read `online` with no binary installed
  * at all, and `harnesses ping` answered it in 78ms — the daemon was there, the
  * harness was not. A user could not discover that from inside the product; it
  * took `command -v` in a shell.
@@ -44,7 +43,7 @@ class HarnessesInstallers extends base_command_1.BaseCommand {
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output as JSON' }),
         daemon: core_1.Flags.string({
-            description: 'Which machine to ask. Defaults to your only active daemon; required when several are active.',
+            description: 'Which machine to ask. Defaults to your only connected daemon; required when several are connected.',
         }),
     };
     async run() {
@@ -53,7 +52,7 @@ class HarnessesInstallers extends base_command_1.BaseCommand {
         const target = await (0, daemon_target_1.resolveDaemonTarget)({
             explicit: flags.daemon,
             local: (0, daemon_target_1.localMachine)(),
-            listActiveDaemons: () => (0, daemon_target_1.listActiveDaemons)(this.resolvedCredential),
+            listActiveDaemons: () => (0, daemon_target_1.listConnectedDaemons)(this.resolvedCredential),
         });
         const daemonId = this.daemonIdFor(target);
         let res;
@@ -65,8 +64,21 @@ class HarnessesInstallers extends base_command_1.BaseCommand {
             return;
         }
         const providers = res.providers ?? [];
+        // Computed before the output mode splits: a JSON reader needs the same
+        // "this daemon is older than that provider" signal the table prints, and
+        // the daemon version the header shows (OSK-9537 verifier).
+        const notOffered = providers.length > 0 ? (0, cli_installers_1.providersNotOffered)(providers, data_provider_1.MANAGED_CLI_PROVIDER_IDS) : [];
         if (flags.json) {
-            this.log(JSON.stringify({ daemonId, platform: res.platform, providers }, null, 2));
+            this.log(JSON.stringify({
+                daemonId,
+                platform: res.platform,
+                daemonVersion: res.daemonVersion ?? null,
+                providers,
+                notOffered: notOffered.map((provider) => ({
+                    provider,
+                    addedInDaemon: cli_installers_1.MANAGED_CLI_PROVIDER_SINCE[(0, cli_installers_1.readinessKey)(provider)] ?? null,
+                })),
+            }, null, 2));
             return;
         }
         if (providers.length === 0) {
@@ -116,12 +128,12 @@ class HarnessesInstallers extends base_command_1.BaseCommand {
         }
         // A provider the platform knows and this daemon did not mention. Without
         // this it simply is not on the list, which reads as dropped support rather
-        // than as an older daemon — `devin` arrived in daemon 0.8.43, so 0.8.35
+        // than as an older daemon — `devin` arrived in the 0.8.45 daemon release, so 0.8.35
         // answers with six providers and says nothing about the seventh.
-        const missing = (0, cli_installers_1.providersNotOffered)(providers, data_provider_1.MANAGED_CLI_PROVIDER_IDS);
+        const missing = notOffered;
         if (missing.length > 0) {
             this.log('');
-            this.log(`  Not offered by this daemon: ${missing.join(', ')}. ` +
+            this.log(`  Not offered by this daemon: ${(0, cli_installers_1.describeProvidersNotOffered)(missing)}. ` +
                 'That usually means the daemon is older than the provider — ' +
                 'check `skrr daemon status` for its version and how to update it.');
         }
@@ -142,14 +154,26 @@ class HarnessesInstallers extends base_command_1.BaseCommand {
             this.error('This machine is a Dedicated Runtime whose daemon has not published an id this process can read. Name a machine with --daemon.', { exit: 1 });
         }
         if (target.kind === 'none') {
-            this.error(`No active daemon is registered for this account, so nothing can report on it. Start the local runtime with \`${this.config.bin} daemon start\`.`, { exit: 1 });
+            this.failWithCliError({
+                message: `No daemon is connected for this account, so nothing can report on it. Start the background service on this computer with \`${this.config.bin} daemon start\`.`,
+                code: 'NO_DAEMON',
+                exit: 1,
+                retryable: false,
+            });
         }
         const choices = target.daemons
             .slice(0, 10)
-            .map((d) => `  ${(0, daemon_ref_1.formatDaemonRef)(d.id)}  ${d.name || '—'}`)
+            // Full ids: they are what `--daemon` takes (OSK-9993).
+            .map((d) => `  ${d.id}  ${d.name || '—'}`)
             .join('\n');
-        this.error(`${target.daemons.length} active daemons — say which machine to report on:\n${choices}\n` +
-            `Pass one with --daemon, or see them all with \`${this.config.bin} list-daemons\`.`, { exit: 1 });
+        // The same refusal, and the same code, as \`code install\` (OSK-9993 verifier).
+        this.failWithCliError({
+            message: `${target.daemons.length} connected daemons — say which machine to report on:\n${choices}\n` +
+                `Pass one with --daemon, or see each daemon id in the DAEMON ID column of \`${this.config.bin} harnesses list\`.`,
+            code: 'AMBIGUOUS_DAEMON',
+            exit: 1,
+            retryable: false,
+        });
     }
 }
 exports.default = HarnessesInstallers;

package/dist/commands/harnesses/list.d.ts

@@ -1,4 +1,5 @@
 import { BaseCommand } from '../../base-command';
+import { type ProviderReadiness } from '../../lib/cli-installers';
 /**
  * Minimal shape of a provider Harness row as returned by
  * `GET /api/harnesses` (`{ harnesses: WireHarness[] }`). Only the
@@ -71,6 +72,25 @@ export declare function shortMachineLabel(machineUuid?: string): string;
 export declare function emptyListMessage(input: {
     provider: string | null;
     local: boolean;
+    /** `--all` was passed, so suggesting it is noise (OSK-9285). */
+    all?: boolean;
+    /**
+     * Every provider this account has a row for, whatever its status, when the
+     * caller could read it. Absent or null: unknown, and nothing is claimed.
+     */
+    accountProviders?: readonly string[] | null;
+}): string;
+/**
+ * The NAME cell. `show` and `--json` said when YOUR alias covers the shared
+ * name; this table printed the alias as if it were the name everyone sees
+ * (OSK-10004). The marker leads because the column truncates from the right,
+ * and a trailing one is the first thing a long name would cut.
+ */
+export declare const ALIAS_MARKER = "*";
+export declare function harnessNameCell(row: {
+    name?: string;
+    sharedName?: string;
+    aliased?: boolean;
 }): string;
 /**
  * What the DEFAULT column says for one row.
@@ -89,9 +109,33 @@ export type HarnessSelection = {
     hiddenOffline: number;
     /** Rows dropped by `--limit` alone. */
     hiddenByLimit: number;
+    /**
+     * Superseded rows a status filter dropped — the ones `--status offline`
+     * leaves to `--status superseded`. Only superseded rows are counted: online
+     * rows the same filter drops were never offline, so they are not "hidden".
+     */
+    hiddenSuperseded: number;
     /** Rows that survived every filter, before the online-only default and `--limit`. */
     matched: number;
 };
+/**
+ * What `--status` accepts: exactly what the STATUS column prints (OSK-10005).
+ *
+ * The column says `superseded` for a row a later registration on the same
+ * machine replaced; the filter accepted only online|offline, so the one value
+ * that says "never coming back" could be read and never selected.
+ */
+export declare const HARNESS_STATUS_FILTERS: readonly ["online", "offline", "superseded"];
+/**
+ * The status to ask the SERVER for. Only `online` narrows there.
+ *
+ * Supersession is judged among the rows in one response — a row is superseded
+ * when a newer row for the same provider on the same machine exists. Asked for
+ * `offline` only, the server cannot see the live row that replaced an offline
+ * one, so it labelled that row `offline` while `--all` called it `superseded`.
+ * Online rows are never superseded, so that one filter is safe to push down.
+ */
+export declare function serverStatusParam(status: string | undefined): string | undefined;
 /**
  * Harness rows created before `harnessLocation` was introduced do not carry that
  * field.  A daemon id is the durable discriminator for those legacy rows:
@@ -145,6 +189,8 @@ export declare function selectHarnessRows(rows: HarnessRow[], options: {
     local: boolean;
     daemon?: string;
     limit?: number;
+    /** Keep only rows whose STATUS column reads this (`harnessStatusLabel`). */
+    status?: string;
     /** Hide offline rows (the human-readable default). */
     onlineOnly: boolean;
 }): HarnessSelection;
@@ -177,7 +223,9 @@ export type HarnessDefaults = {
     workspaceDefaultHarnessId?: string | null;
 };
 export type HarnessesJson = {
-    harnesses: HarnessRow[];
+    harnesses: Array<HarnessRow & {
+        readiness?: ProviderReadiness | null;
+    }>;
     filters: HarnessesFilters;
     defaults: HarnessDefaults;
     counts: {
@@ -189,6 +237,8 @@ export type HarnessesJson = {
         hiddenByLocal: number;
         /** Rows `--limit` dropped — raise `--limit` to get them back. */
         hiddenByLimit: number;
+        /** Superseded rows `--status` dropped — `--status superseded` lists them. */
+        hiddenSuperseded: number;
         /**
          * OSK-3385 — how many of the returned rows are `status: 'online'`.
          *
@@ -213,7 +263,22 @@ export type HarnessesJson = {
  * (`compareHarnessRows`), not in server order. Round 1 introduced that sort;
  * it is deliberate, so `.harnesses[0]` is the row you almost always want.
  */
-export declare function buildHarnessesJson(selection: HarnessSelection, filters: HarnessesFilters, defaults?: HarnessDefaults): HarnessesJson;
+export declare function buildHarnessesJson(selection: HarnessSelection, filters: HarnessesFilters, defaults?: HarnessDefaults, readiness?: (row: HarnessRow) => ProviderReadiness | undefined): HarnessesJson;
+/**
+ * What the READINESS column's non-answers mean, counted from the rows shown.
+ *
+ * Each footnote names a different next step, so they are not merged: a `-` on
+ * an offline row wants `--all`/a started daemon, a `-` on a provider the daemon
+ * does not manage (hermes, gemini) wants nothing, `unknown` is installed but
+ * unverifiable, and only a definite not-ready label (interactive-required,
+ * not-installed, …) has something to fix. Counting `unknown` as not ready
+ * would send the reader to fix eleven rows that are fine as far as anyone can
+ * tell.
+ */
+export declare function readinessFootnotes(rows: readonly HarnessRow[], readiness: {
+    lookup: (row: HarnessRow) => ProviderReadiness | undefined;
+    daemonAnswered: (row: HarnessRow) => boolean;
+}): string[];
 /**
  * Rows with the provider in its CANONICAL spelling.
  *
@@ -246,6 +311,13 @@ export default class HarnessesList extends BaseCommand {
         daemon: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         limit: import("@oclif/core/lib/interfaces").OptionFlag<number | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         local: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        readiness: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
+    /**
+     * The canonical providers this account has any row for, or null when that
+     * cannot be read. Asked only when a --provider filter came back empty, so
+     * the ordinary path pays nothing for it.
+     */
+    private accountProviders;
 }

package/dist/commands/harnesses/list.js

@@ -1,18 +1,25 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.HARNESS_STATUS_FILTERS = exports.ALIAS_MARKER = void 0;
 exports.shortMachineLabel = shortMachineLabel;
 exports.emptyListMessage = emptyListMessage;
+exports.harnessNameCell = harnessNameCell;
 exports.defaultMarker = defaultMarker;
+exports.serverStatusParam = serverStatusParam;
 exports.isLocalHarnessRow = isLocalHarnessRow;
 exports.selectHarnessRows = selectHarnessRows;
 exports.buildHarnessesJson = buildHarnessesJson;
+exports.readinessFootnotes = readinessFootnotes;
 exports.canonicalHarnessRows = canonicalHarnessRows;
 const core_1 = require("@oclif/core");
 const auth_core_1 = require("@skrr-ai/auth-core");
 const base_command_1 = require("../../base-command");
 const api_fetch_1 = require("../../lib/api-fetch");
+const cli_installers_1 = require("../../lib/cli-installers");
 const format_1 = require("../../lib/format");
 const harness_provider_input_1 = require("../../lib/harness-provider-input");
+const data_provider_1 = require("@skrr-ai/data-provider");
+const harness_labels_1 = require("../../lib/harness-labels");
 /**
  * Products skrr provisions and owns. A row that says it is one of these is
  * cloud, whatever `harnessLocation` reads.
@@ -50,17 +57,48 @@ function shortMachineLabel(machineUuid) {
  * pointing at the one command that shows which providers actually exist.
  */
 function emptyListMessage(input) {
+    if (input.provider) {
+        const known = input.accountProviders;
+        // A provider that appears on no row of the account — a typo, or a harness
+        // never registered — is a different fact from one whose rows are merely
+        // hidden by a filter. Both used to print the same sentence (OSK-9285).
+        if (known && !known.includes(input.provider)) {
+            return known.length > 0
+                ? `No harness on this account has provider "${input.provider}". Providers here: ${known.join(', ')}.`
+                : `No harness on this account has provider "${input.provider}", and the account has no harnesses yet.`;
+        }
+        const lifts = [];
+        if (!input.all)
+            lifts.push('offline rows are hidden by default (--all shows them)');
+        if (input.local)
+            lifts.push('hosted rows are excluded (--no-local includes them)');
+        const liftText = lifts.join('; ');
+        return (`No harnesses with provider "${input.provider}" match these filters.` +
+            (liftText ? ` ${liftText.charAt(0).toUpperCase()}${liftText.slice(1)}.` : '') +
+            (known
+                ? ''
+                : ' Run `skrr harnesses list --all` without --provider to see which providers this account has registered.'));
+    }
     const hiddenByDefault = input.local
         ? ' Offline rows are hidden by default (--all shows them) and hosted rows are excluded (--no-local includes them).'
         : '';
-    if (input.provider) {
-        return (`No harnesses with provider "${input.provider}".${hiddenByDefault}` +
-            ' Run `skrr harnesses list --all` without --provider to see which providers this account has registered.');
-    }
     return input.local
         ? `No harnesses found. Is your daemon connected?${hiddenByDefault}`
         : 'No harnesses found.';
 }
+/**
+ * The NAME cell. `show` and `--json` said when YOUR alias covers the shared
+ * name; this table printed the alias as if it were the name everyone sees
+ * (OSK-10004). The marker leads because the column truncates from the right,
+ * and a trailing one is the first thing a long name would cut.
+ */
+exports.ALIAS_MARKER = '*';
+function harnessNameCell(row) {
+    // The same rule the web sidebar renders (OSK-9290): a name stored under an
+    // older spelling or `oversky …` name is not a second product.
+    const name = row.name ? (0, data_provider_1.canonicalizeHarnessLabel)(row.name) : (0, format_1.formatCell)(row.name);
+    return (0, harness_labels_1.isAliasCoveringSharedName)(row) ? `${exports.ALIAS_MARKER} ${name}` : name;
+}
 /**
  * What the DEFAULT column says for one row.
  *
@@ -84,6 +122,26 @@ const MANAGED_REGISTRATION_SOURCES = new Set([
     'hosted-job-machine',
     'cloud-coding',
 ]);
+/**
+ * What `--status` accepts: exactly what the STATUS column prints (OSK-10005).
+ *
+ * The column says `superseded` for a row a later registration on the same
+ * machine replaced; the filter accepted only online|offline, so the one value
+ * that says "never coming back" could be read and never selected.
+ */
+exports.HARNESS_STATUS_FILTERS = ['online', 'offline', 'superseded'];
+/**
+ * The status to ask the SERVER for. Only `online` narrows there.
+ *
+ * Supersession is judged among the rows in one response — a row is superseded
+ * when a newer row for the same provider on the same machine exists. Asked for
+ * `offline` only, the server cannot see the live row that replaced an offline
+ * one, so it labelled that row `offline` while `--all` called it `superseded`.
+ * Online rows are never superseded, so that one filter is safe to push down.
+ */
+function serverStatusParam(status) {
+    return status === 'online' ? 'online' : undefined;
+}
 /**
  * Harness rows created before `harnessLocation` was introduced do not carry that
  * field.  A daemon id is the durable discriminator for those legacy rows:
@@ -155,6 +213,16 @@ function selectHarnessRows(rows, options) {
         const needle = options.daemon.toLowerCase();
         filtered = filtered.filter((r) => (r.daemonId ?? '').toLowerCase().includes(needle));
     }
+    let hiddenSuperseded = 0;
+    if (options.status) {
+        const wanted = options.status;
+        filtered = filtered.filter((r) => {
+            const label = (0, harness_labels_1.harnessStatusLabel)(r);
+            if (label !== wanted && label === 'superseded')
+                hiddenSuperseded += 1;
+            return label === wanted;
+        });
+    }
     const sorted = [...filtered].sort(compareHarnessRows);
     const shown = options.onlineOnly ? sorted.filter((r) => r.status === 'online') : sorted;
     const limited = options.limit != null && options.limit > 0 ? shown.slice(0, options.limit) : shown;
@@ -163,6 +231,7 @@ function selectHarnessRows(rows, options) {
         hiddenByLocal,
         hiddenOffline: sorted.length - shown.length,
         hiddenByLimit: shown.length - limited.length,
+        hiddenSuperseded,
         matched: sorted.length,
     };
 }
@@ -178,9 +247,13 @@ function selectHarnessRows(rows, options) {
  * (`compareHarnessRows`), not in server order. Round 1 introduced that sort;
  * it is deliberate, so `.harnesses[0]` is the row you almost always want.
  */
-function buildHarnessesJson(selection, filters, defaults = {}) {
+function buildHarnessesJson(selection, filters, defaults = {}, readiness) {
     return {
-        harnesses: selection.visible,
+        // `readiness` is the per-provider answer the READINESS column prints —
+        // `null` where no daemon answered for that row, never a guess (OSK-10100).
+        harnesses: readiness
+            ? selection.visible.map((row) => ({ ...row, readiness: readiness(row) ?? null }))
+            : selection.visible,
         filters,
         defaults: {
             effectiveDefaultHarnessId: defaults.effectiveDefaultHarnessId ?? null,
@@ -193,10 +266,61 @@ function buildHarnessesJson(selection, filters, defaults = {}) {
             matched: selection.matched,
             hiddenByLocal: selection.hiddenByLocal,
             hiddenByLimit: selection.hiddenByLimit,
+            hiddenSuperseded: selection.hiddenSuperseded,
             online: selection.visible.filter((r) => r.status === 'online').length,
         },
     };
 }
+/**
+ * What the READINESS column's non-answers mean, counted from the rows shown.
+ *
+ * Each footnote names a different next step, so they are not merged: a `-` on
+ * an offline row wants `--all`/a started daemon, a `-` on a provider the daemon
+ * does not manage (hermes, gemini) wants nothing, `unknown` is installed but
+ * unverifiable, and only a definite not-ready label (interactive-required,
+ * not-installed, …) has something to fix. Counting `unknown` as not ready
+ * would send the reader to fix eleven rows that are fine as far as anyone can
+ * tell.
+ */
+function readinessFootnotes(rows, readiness) {
+    const lines = [];
+    let notLive = 0;
+    let unmanaged = 0;
+    let noAnswer = 0;
+    let unknown = 0;
+    let notReady = 0;
+    for (const row of rows) {
+        const verdict = readiness.lookup(row);
+        if (verdict) {
+            if (verdict.label === 'unknown')
+                unknown += 1;
+            else if (!verdict.ok)
+                notReady += 1;
+        }
+        else if (row.status !== 'online' || row.superseded)
+            notLive += 1;
+        else if (readiness.daemonAnswered(row))
+            unmanaged += 1;
+        else
+            noAnswer += 1;
+    }
+    const dash = [];
+    if (notLive > 0)
+        dash.push(`${notLive} offline or superseded`);
+    if (unmanaged > 0)
+        dash.push(`${unmanaged} a provider its daemon does not report on`);
+    if (noAnswer > 0)
+        dash.push(`${noAnswer} whose daemon did not answer in time`);
+    if (dash.length > 0)
+        lines.push(`(READINESS -: ${dash.join('; ')}.)`);
+    if (unknown > 0) {
+        lines.push(`(READINESS unknown: ${unknown} installed, but the daemon cannot read that provider's sign-in state.)`);
+    }
+    if (notReady > 0) {
+        lines.push(`(${notReady} not ready — \`skrr harnesses show <id>\` says what each needs.)`);
+    }
+    return lines;
+}
 /**
  * Rows with the provider in its CANONICAL spelling.
  *
@@ -253,8 +377,8 @@ class HarnessesList extends base_command_1.BaseCommand {
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output as JSON' }),
         status: core_1.Flags.string({
-            description: 'Filter by harness status.',
-            options: ['online', 'offline'],
+            description: 'Filter by the STATUS column. `offline` leaves out superseded rows — a later registration on the same machine replaced them — and `superseded` lists only those.',
+            options: [...exports.HARNESS_STATUS_FILTERS],
             exclusive: ['online', 'all'],
         }),
         online: core_1.Flags.boolean({
@@ -284,6 +408,11 @@ class HarnessesList extends base_command_1.BaseCommand {
             default: true,
             allowNo: true,
         }),
+        readiness: core_1.Flags.boolean({
+            description: 'Ask each connected daemon whether its coding-agent CLIs are installed and signed in (the READINESS column; `readiness` in --json). One request per daemon. On by default — pass --no-readiness to skip it.',
+            default: true,
+            allowNo: true,
+        }),
     };
     async run() {
         this.requireAuth();
@@ -296,8 +425,9 @@ class HarnessesList extends base_command_1.BaseCommand {
         // asked for the canonical one, which it matches against every stored spelling.
         const provider = flags.provider ? (0, harness_provider_input_1.canonicalHarnessInput)(flags.provider) : undefined;
         const params = new URLSearchParams();
-        if (status)
-            params.set('status', status);
+        const serverStatus = serverStatusParam(status);
+        if (serverStatus)
+            params.set('status', serverStatus);
         if (provider)
             params.set('provider', provider);
         const query = params.toString() ? `?${params.toString()}` : '';
@@ -312,16 +442,36 @@ class HarnessesList extends base_command_1.BaseCommand {
             local: flags.local,
             daemon: flags.daemon,
             limit: flags.limit,
+            status,
             onlineOnly: false,
         });
+        // STATUS is the daemon's socket, not the coding agent: a Devin that needs
+        // sign-in read `online` exactly like a ready one, and `list` is where a
+        // user picks a harness (OSK-10100). One read per connected daemon, bounded;
+        // a daemon that cannot answer leaves its rows unknown rather than failing
+        // the list.
+        const readiness = flags.readiness
+            ? await (0, cli_installers_1.readinessForRows)(selection.visible, this.resolvedCredential)
+            : null;
         if (flags.json) {
+            // A provider no row on the account has is a typo (or a harness never
+            // registered), and a script that got an empty list and exit 0 could not
+            // tell it from a filter that hid real rows — the silent failure OSK-9285
+            // describes. Refused exactly as the table refuses it; a real provider
+            // whose rows a filter hid still gets its empty list.
+            if (provider && selection.visible.length === 0) {
+                const accountProviders = await this.accountProviders();
+                if (accountProviders && !accountProviders.includes(provider)) {
+                    this.error(emptyListMessage({ provider, local: flags.local, all: flags.all, accountProviders }), { exit: 1 });
+                }
+            }
             const payload = buildHarnessesJson(selection, {
                 local: flags.local,
                 status: status ?? null,
                 provider: provider ?? null,
                 daemon: flags.daemon ?? null,
                 limit: flags.limit ?? null,
-            }, res);
+            }, res, readiness ? readiness.lookup : () => undefined);
             this.log(JSON.stringify(payload, null, 2));
             return;
         }
@@ -338,18 +488,39 @@ class HarnessesList extends base_command_1.BaseCommand {
             if (selection.hiddenByLocal > 0) {
                 hidden.push(`${selection.hiddenByLocal} hosted (--no-local)`);
             }
+            if (selection.hiddenSuperseded > 0) {
+                hidden.push(`${selection.hiddenSuperseded} superseded (--status superseded)`);
+            }
             if (hidden.length > 0) {
                 this.log(`No harnesses match the default filters. Hidden: ${hidden.join(', ')}.`);
                 return;
             }
-            this.log(emptyListMessage({ provider: provider ?? null, local: flags.local }));
+            if (!provider) {
+                this.log(emptyListMessage({ provider: null, local: flags.local, all: flags.all }));
+                return;
+            }
+            const accountProviders = await this.accountProviders();
+            const message = emptyListMessage({
+                provider,
+                local: flags.local,
+                all: flags.all,
+                accountProviders,
+            });
+            if (accountProviders && !accountProviders.includes(provider)) {
+                // Nothing on the account answers to this name: exit non-zero so a script
+                // cannot mistake a typo for an empty-but-valid filter. `--json` refuses it
+                // the same way (above).
+                this.error(message, { exit: 1 });
+            }
+            this.log(message);
             return;
         }
         const tableRows = selection.visible.map((r) => ({
             id: (0, format_1.formatCell)(r._id),
-            name: (0, format_1.formatCell)(r.name),
-            provider: (0, format_1.formatCell)(r.provider),
-            status: r.superseded ? 'superseded' : (0, format_1.formatCell)(r.status),
+            name: harnessNameCell(r),
+            provider: (0, harness_labels_1.harnessProviderLabel)(r.provider),
+            status: (0, harness_labels_1.harnessStatusLabel)(r),
+            readiness: readiness?.lookup(r)?.label ?? '-',
             default: defaultMarker(r._id, res),
             daemon: (0, format_1.formatCell)(r.daemonId),
             model: (0, format_1.formatCell)(r.model),
@@ -363,6 +534,7 @@ class HarnessesList extends base_command_1.BaseCommand {
         // only inside a third command (`machines show`). Now that the server labels
         // supersession here too, say it where the rows are.
         const supersededShown = selection.visible.filter((r) => r.superseded).length;
+        const aliasedShown = selection.visible.filter((r) => (0, harness_labels_1.isAliasCoveringSharedName)(r)).length;
         (0, format_1.renderTable)(tableRows, [
             { key: 'id', header: 'HARNESS ID' },
             // `harnesses alias` exists to make a row recognisable in a LIST. Without
@@ -371,6 +543,11 @@ class HarnessesList extends base_command_1.BaseCommand {
             { key: 'name', header: 'NAME', maxWidth: 22 },
             { key: 'provider', header: 'PROVIDER' },
             { key: 'status', header: 'STATUS' },
+            // Whether the coding agent itself can run — installed, signed in — as
+            // opposed to STATUS, which is only whether its daemon is connected.
+            ...(readiness && readiness.answered > 0
+                ? [{ key: 'readiness', header: 'READINESS' }]
+                : []),
             // `set-default` had no reader anywhere in the CLI. This is it.
             { key: 'default', header: 'DEFAULT' },
             { key: 'daemon', header: 'DAEMON ID', verbatim: true },
@@ -378,11 +555,28 @@ class HarnessesList extends base_command_1.BaseCommand {
             { key: 'machine', header: 'MACHINE' },
             { key: 'seen', header: `LAST SEEN (${(0, format_1.localZone)()})` },
         ], (line) => this.log(line));
+        if (readiness && readiness.answered > 0) {
+            for (const line of readinessFootnotes(selection.visible, readiness))
+                this.log(line);
+        }
+        else if (readiness && readiness.asked > 0) {
+            this.log('(Readiness not reported — no connected daemon answered. `skrr harnesses installers --daemon <DAEMON ID>` asks one directly.)');
+        }
+        const noCodingAgentShown = selection.visible.filter((r) => (0, harness_labels_1.isNoCodingAgentProvider)(r.provider)).length;
+        if (noCodingAgentShown > 0) {
+            this.log(`(provider ${harness_labels_1.NO_CODING_AGENT_PROVIDER_LABEL}: ${noCodingAgentShown} shown are machines with no coding agent installed — the row is how the machine is listed, not a harness named for a provider.)`);
+        }
+        if (aliasedShown > 0) {
+            this.log(`(${exports.ALIAS_MARKER} your alias — everyone else this harness is shared with sees its shared name. \`skrr harnesses show <id>\` prints both.)`);
+        }
         if (supersededShown > 0) {
             // This is the sentence that makes `machines list`'s fraction derivable
             // from here: live = printed − superseded.
             this.log(`(${supersededShown} of ${selection.visible.length} shown are superseded — a later registration on the same machine replaced them, so they will not come back. ${selection.visible.length - supersededShown} live.)`);
         }
+        if (selection.hiddenSuperseded > 0) {
+            this.log(`(${selection.hiddenSuperseded} superseded hidden — \`--status superseded\` lists them.)`);
+        }
         if (selection.hiddenByLocal > 0) {
             // Same reason as the JSON `counts` block: the `--local` default is the
             // single biggest source of "the list is shorter than I expected".
@@ -397,5 +591,27 @@ class HarnessesList extends base_command_1.BaseCommand {
             this.log(`(showing ${selection.visible.length} of ${selection.visible.length + selection.hiddenByLimit} — raise --limit for the rest)`);
         }
     }
+    /**
+     * The canonical providers this account has any row for, or null when that
+     * cannot be read. Asked only when a --provider filter came back empty, so
+     * the ordinary path pays nothing for it.
+     */
+    async accountProviders() {
+        try {
+            const all = await (0, api_fetch_1.apiFetch)('/api/harnesses', {
+                credential: this.resolvedCredential,
+            });
+            const providers = new Set();
+            for (const row of canonicalHarnessRows(all.harnesses ?? [])) {
+                if (typeof row.provider === 'string' && !(0, harness_labels_1.isNoCodingAgentProvider)(row.provider)) {
+                    providers.add(row.provider);
+                }
+            }
+            return [...providers].sort();
+        }
+        catch {
+            return null;
+        }
+    }
 }
 exports.default = HarnessesList;

package/dist/commands/harnesses/ping.d.ts

@@ -19,4 +19,6 @@ export default class HarnessesPing extends BaseCommand {
         timeout: import("@oclif/core/lib/interfaces").OptionFlag<number, import("@oclif/core/lib/interfaces").CustomOptions>;
     };
     run(): Promise<void>;
+    /** Best-effort: never turns a successful ping into a failure. */
+    private logReadiness;
 }