@skrr-ai/cli 0.1.107 → 0.1.109

package/dist/commands/commitments/operating-review.js

@@ -1,8 +1,10 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.noQuestionsSentence = noQuestionsSentence;
+exports.formatOperatingReview = formatOperatingReview;
 const core_1 = require("@oclif/core");
 const base_command_1 = require("../../base-command");
+const commitment_loop_status_1 = require("../../lib/commitment-loop-status");
 const commitments_1 = require("../../lib/commitments");
 /** What each silent loop is for, in the owner's terms rather than the role key. */
 const ROLE_PURPOSE = {
@@ -39,6 +41,79 @@ function noQuestionsSentence(alwaysOn, roles) {
         ? 'None. It has not told you anything is missing.'
         : 'None recorded yet. None of its loops has checked, so this says nothing about whether anything is missing.';
 }
+/** The review as text. Pure, so the whole output can be pinned. */
+function formatOperatingReview(review, loops) {
+    const lines = [];
+    const roles = review.systemCommitments || [];
+    const alwaysOn = review.alwaysOn === true;
+    const questions = review.openQuestions || [];
+    const portfolio = review.portfolio || {};
+    const metrics = review.metrics || {};
+    lines.push(`Agent ${review.agentId}`);
+    lines.push('');
+    // A loop's own status says `active`, but a system-managed loop is only
+    // given a scheduling trigger while the Agent is Always-On. Printing
+    // `active` beside "last checked never" without saying that read as
+    // "nothing has happened yet" rather than "this cannot happen", and nothing
+    // led the owner to the switch that decides it.
+    if (!alwaysOn) {
+        lines.push('Always-On is OFF — these loops are provisioned but will not run.');
+        lines.push(`  skrr agents always-on enable ${review.agentId}`);
+        lines.push('');
+    }
+    lines.push('Background loops');
+    if (roles.length === 0) {
+        lines.push('  (none provisioned)');
+    }
+    if ('error' in loops) {
+        // "Cannot see" must not read as "nothing to report": say what is missing and what it hides.
+        lines.push(`  Loop detail unavailable (${loops.error}). The states below are the stored lifecycle only; ` +
+            'they do not say whether a loop is armed, what it can read, or why it last stopped.');
+    }
+    for (const role of roles) {
+        const key = role.systemRole || '?';
+        const purpose = ROLE_PURPOSE[key] ? ` — ${ROLE_PURPOSE[key]}` : '';
+        const contact = role.maxUserNudgesPerDay && role.maxUserNudgesPerDay > 0
+            ? `may contact you ${role.maxUserNudgesPerDay}/day`
+            : 'never contacts you';
+        // `idle` replaces `active` only. A paused loop is paused for its own
+        // reason — Relationship Pulse ships paused as a separate consent
+        // decision — and collapsing that into `idle` would hide a distinction
+        // the owner needs when they do turn Always-On on.
+        const state = !alwaysOn && role.status === 'active' ? 'idle' : role.status || '?';
+        lines.push(`  ${state.padEnd(7)} ${key}${purpose}`);
+        lines.push(`          last checked ${ago(role.lastCheckedAt)} · ${contact}`);
+        if ('rows' in loops) {
+            const row = role.id ? loops.rows.get(role.id) : undefined;
+            if (row) {
+                for (const line of (0, commitment_loop_status_1.formatLoopSummary)(row))
+                    lines.push(`          ${line}`);
+            }
+            else {
+                lines.push(`          loop detail: not in the roster read${loops.truncated ? ' (the roster is bounded and this Agent holds more)' : ''}; run \`skrr commitments status ${role.id ?? '<id>'}\``);
+            }
+        }
+    }
+    lines.push('');
+    lines.push(`Open questions from this agent (${questions.length})`);
+    if (questions.length === 0) {
+        lines.push(`  ${noQuestionsSentence(alwaysOn, roles)}`);
+    }
+    for (const q of questions) {
+        lines.push(`  • ${q.question}`);
+        if (q.whyItMatters)
+            lines.push(`    why: ${q.whyItMatters}`);
+        lines.push(`    asked ${ago(q.askedAt)}`);
+    }
+    lines.push('');
+    const drafts = portfolio.draftCommitmentCount ?? 0;
+    lines.push(`Business commitments: ${portfolio.activeCommitmentCount ?? 0} active, ${portfolio.pausedCommitmentCount ?? 0} paused${drafts > 0 ? `, ${drafts} draft` : ''}`);
+    if (typeof metrics.awaitingApproval === 'number') {
+        lines.push(`Proposals awaiting your approval: ${metrics.awaitingApproval}`);
+    }
+    lines.push(`Every commitment it holds, loop by loop: skrr commitments roster --agent ${review.agentId}`);
+    return lines;
+}
 class CommitmentsOperatingReview extends base_command_1.BaseCommand {
     static description = 'Read an agent’s system-managed commitment operating review';
     static args = {
@@ -60,61 +135,39 @@ class CommitmentsOperatingReview extends base_command_1.BaseCommand {
         catch (error) {
             this.handleApiError(error);
         }
+        // The roster is a second read; its failure must not take the review down with it, and must not
+        // pass for "nothing to report" either.
+        let loops;
+        try {
+            const roster = await commitments_1.commitmentApi.roster(args.agentId);
+            loops = {
+                rows: new Map((Array.isArray(roster.commitments) ? roster.commitments : []).map((row) => [
+                    row.commitmentId,
+                    row,
+                ])),
+                truncated: roster.truncated === true,
+            };
+        }
+        catch (error) {
+            loops = { error: error instanceof Error ? error.message : String(error) };
+        }
         if (flags.json) {
-            this.log(JSON.stringify(result, null, 2));
+            const review = result;
+            const roles = review.systemCommitments || [];
+            this.log(JSON.stringify('error' in loops
+                ? { ...review, loopStatusError: loops.error }
+                : {
+                    ...review,
+                    loopStatus: roles
+                        .map((role) => (role.id ? loops.rows.get(role.id) : undefined))
+                        .filter((row) => row !== undefined),
+                }, null, 2));
             return;
         }
         // Without this the command printed raw JSON in both modes, which is why the
         // meta loops read as machinery rather than as something an owner supervises.
-        const review = result;
-        const roles = review.systemCommitments || [];
-        const alwaysOn = review.alwaysOn === true;
-        const questions = review.openQuestions || [];
-        const portfolio = review.portfolio || {};
-        this.log(`Agent ${review.agentId}`);
-        this.log('');
-        // A loop's own status says `active`, but a system-managed loop is only
-        // given a scheduling trigger while the Agent is Always-On. Printing
-        // `active` beside "last checked never" without saying that read as
-        // "nothing has happened yet" rather than "this cannot happen", and nothing
-        // led the owner to the switch that decides it.
-        if (!alwaysOn) {
-            this.log('Always-On is OFF — these loops are provisioned but will not run.');
-            this.log(`  skrr agents always-on enable ${review.agentId}`);
-            this.log('');
-        }
-        this.log('Background loops');
-        if (roles.length === 0) {
-            this.log('  (none provisioned)');
-        }
-        for (const role of roles) {
-            const key = role.systemRole || '?';
-            const purpose = ROLE_PURPOSE[key] ? ` — ${ROLE_PURPOSE[key]}` : '';
-            const contact = role.maxUserNudgesPerDay && role.maxUserNudgesPerDay > 0
-                ? `may contact you ${role.maxUserNudgesPerDay}/day`
-                : 'never contacts you';
-            // `idle` replaces `active` only. A paused loop is paused for its own
-            // reason — Relationship Pulse ships paused as a separate consent
-            // decision — and collapsing that into `idle` would hide a distinction
-            // the owner needs when they do turn Always-On on.
-            const state = !alwaysOn && role.status === 'active' ? 'idle' : role.status || '?';
-            this.log(`  ${state.padEnd(7)} ${key}${purpose}`);
-            this.log(`          last checked ${ago(role.lastCheckedAt)} · ${contact}`);
-        }
-        this.log('');
-        this.log(`Open questions from this agent (${questions.length})`);
-        if (questions.length === 0) {
-            this.log(`  ${noQuestionsSentence(alwaysOn, roles)}`);
-        }
-        for (const q of questions) {
-            this.log(`  • ${q.question}`);
-            if (q.whyItMatters)
-                this.log(`    why: ${q.whyItMatters}`);
-            this.log(`    asked ${ago(q.askedAt)}`);
-        }
-        this.log('');
-        const drafts = portfolio.draftCommitmentCount ?? 0;
-        this.log(`Business commitments: ${portfolio.activeCommitmentCount ?? 0} active, ${portfolio.pausedCommitmentCount ?? 0} paused${drafts > 0 ? `, ${drafts} draft` : ''}`);
+        for (const line of formatOperatingReview(result, loops))
+            this.log(line);
     }
 }
 exports.default = CommitmentsOperatingReview;

package/dist/commands/commitments/update.js

@@ -31,7 +31,8 @@ const commitments_1 = require("../../lib/commitments");
  * the contract untouched.
  */
 class CommitmentsUpdate extends base_command_1.BaseCommand {
-    static description = "Update a commitment's contract (title, target, cadence, sources)";
+    static description = "Update a commitment's contract (title, target, cadence, sources). A cadence is manual, an interval or daily clock times; " +
+        'it has no weekday, holiday, business-day or trading-calendar fields';
     static examples = [
         '<%= config.bin %> commitments update <commitment-id> --cadence 12h',
         '<%= config.bin %> commitments update <commitment-id> --title "Ship v2 (Q3)" --outcome "v2 is live for all users"',
@@ -275,9 +276,13 @@ JSON
                 // only for a binding it still names; explicit JSON is the authored base.
                 const declarations = Array.isArray(payload.watchedSources)
                     ? (0, commitments_1.normalizeWatchedSourceDeclarations)(payload.watchedSources)
-                    : current?.watchedSources ?? [];
+                    : (current?.watchedSources ?? []);
                 payload.watchedSources = built.map((next) => {
-                    const scopeKey = next.source === 'space' ? 'spaceId' : next.source === 'monitor' ? 'monitorId' : undefined;
+                    const scopeKey = next.source === 'space'
+                        ? 'spaceId'
+                        : next.source === 'monitor'
+                            ? 'monitorId'
+                            : undefined;
                     const nextConfig = next.config;
                     const prior = declarations.find((binding) => binding?.source === next.source &&
                         (!scopeKey || binding.config?.[scopeKey] === nextConfig?.[scopeKey]));
@@ -301,7 +306,9 @@ JSON
                         storedCadence[key] = existing[key];
                 }
                 if (payload.cadence !== undefined) {
-                    if (!payload.cadence || typeof payload.cadence !== 'object' || Array.isArray(payload.cadence)) {
+                    if (!payload.cadence ||
+                        typeof payload.cadence !== 'object' ||
+                        Array.isArray(payload.cadence)) {
                         this.error('cadence must be a JSON object when using cadence flags.', { exit: 1 });
                     }
                 }
@@ -403,7 +410,7 @@ JSON
             this.log(JSON.stringify((0, web_url_1.withCommitmentUrl)(this.cliConfig.baseURL, commitment), null, 2));
             return;
         }
-        this.log('Updated commitment:');
+        this.log((0, commitments_1.describeCommitmentRevision)(commitment?.revision));
         (0, commitments_1.renderCommitmentSummary)(commitment, (line) => this.log(line));
     }
 }

package/dist/commands/doctor.js

@@ -49,6 +49,7 @@ const cli_runtime_1 = require("../lib/cli-runtime");
 const command_manifest_1 = require("../lib/command-manifest");
 const completion_index_1 = require("../lib/completion-index");
 const runtime_context_1 = require("../lib/runtime-context");
+const deployed_build_1 = require("../lib/deployed-build");
 const outbox_1 = require("../lib/outbox");
 /**
  * What each check actually verifies, in the words a reader thinks in.
@@ -230,6 +231,9 @@ class Doctor extends base_command_1.BaseCommand {
                 }
             }
         }
+        // OSK-15497 — which code the server is running, API and web separately.
+        // Public reads, no credential; a failure is reported, never an issue.
+        const deployed = await (0, deployed_build_1.readDeployedBuild)(this.cliConfig.baseURL);
         const now = Date.now();
         const jwtExp = this.resolvedCredential?.token
             ? (0, auth_core_1.decodeJwtExpiry)(this.resolvedCredential.token)
@@ -287,6 +291,9 @@ class Doctor extends base_command_1.BaseCommand {
                 command: process.argv.slice(1).join(' '),
             },
             server: this.cliConfig.baseURL,
+            // The commit the server's API image and its published web shell were
+            // each built from. They differ whenever a release changed only one.
+            deployed,
             profile: (0, keychain_1.getActiveProfile)(),
             config: {
                 path: (0, config_1.getConfigPath)(),
@@ -351,6 +358,8 @@ class Doctor extends base_command_1.BaseCommand {
         }
         this.log(`skrr CLI ${payload.cli.version} at ${root}`);
         this.log(`Server: ${payload.server}`);
+        for (const line of (0, deployed_build_1.deployedBuildLines)(deployed))
+            this.log(line);
         this.log(`Auth:   ${payload.auth.signedIn
             ? `${account?.email ?? account?.id}`
             : serverReachability === 'unavailable'

package/dist/commands/goals/list.js

@@ -5,6 +5,7 @@ const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
 const web_url_1 = require("../../lib/web-url");
 const format_1 = require("../../lib/format");
+const list_output_1 = require("../../lib/list-output");
 class GoalsList extends base_command_1.BaseCommand {
     static description = 'List goals';
     static examples = [
@@ -72,7 +73,11 @@ class GoalsList extends base_command_1.BaseCommand {
             this.handleApiError(err);
         }
         if (flags.json) {
-            this.log(JSON.stringify((0, web_url_1.withDataUrls)(this.cliConfig.baseURL, response, web_url_1.goalUrlFrom), null, 2));
+            // The one list envelope (`{object:'list', data, has_more, next_cursor}`), so
+            // a script reads `.data` here exactly as it does for tasks and projects. The
+            // space-scoped endpoint is not paginated and answers a bare `{data}`; filling
+            // the missing fields is additive, so nothing that read `.data` changes.
+            this.log(JSON.stringify((0, web_url_1.withDataUrls)(this.cliConfig.baseURL, (0, list_output_1.toListEnvelope)(response) ?? response, web_url_1.goalUrlFrom), null, 2));
             return;
         }
         // Both listGoals and listSpaceGoals return { data: Goal[] }. The general

package/dist/commands/grants/rotate.js

@@ -51,6 +51,10 @@ class GrantsRotate extends base_command_1.BaseCommand {
         }
         this.log(`Rotated grant ${grant.id}.`);
         this.log(`  variables: ${grant.credential?.envKeys.join(', ') || '—'}`);
+        // When the server recorded the new value: the proof the rotation landed,
+        // without anything derived from the value itself (OSK-15358).
+        if (grant.credential?.setAt)
+            this.log(`  value set: ${grant.credential.setAt}`);
         if (!grant.access) {
             this.log('  access: not declared (rotation clears the declared class; use --access to re-declare)');
         }

package/dist/commands/login.js

@@ -11,6 +11,8 @@ const config_1 = require("../lib/config");
 const cli_id_1 = require("../lib/cli-id");
 const keychain_1 = require("../lib/keychain");
 const profile_machine_identity_guard_1 = require("../lib/profile-machine-identity-guard");
+const daemon_command_hint_1 = require("../lib/daemon-command-hint");
+const local_daemon_readiness_1 = require("../lib/local-daemon-readiness");
 const login_1 = require("../lib/login");
 const device_code_1 = require("../lib/device-code");
 const api_fetch_1 = require("../lib/api-fetch");
@@ -121,9 +123,7 @@ class Login extends base_command_1.BaseCommand {
     async run() {
         const { flags } = await this.parse(Login);
         if ((0, profile_machine_identity_guard_1.legacyProfileSharesDefaultMachine)()) {
-            this.error(`Profile '${(0, keychain_1.getActiveProfile)()}' was created before profiles had separate machine identities. ` +
-                "Signing in could replace the default machine's device-key pin. " +
-                'Create a new profile name and sign in there; the existing profile was not changed.', { exit: 78 });
+            this.error((0, profile_machine_identity_guard_1.describeLegacySharedProfile)((0, keychain_1.getActiveProfile)()).join('\n'), { exit: 78 });
         }
         if (flags['base-url'] && flags.server && flags['base-url'] !== flags.server) {
             this.error('--base-url and --server were both provided with different values.', { exit: 2 });
@@ -272,8 +272,21 @@ class Login extends base_command_1.BaseCommand {
                         this.log(`This machine is not set up — ${setup.detail}.`);
                     }
                 }
+                // What the daemon is, once the handoff is over: verified connected, or
+                // handed a credential it is not using. "Delivered" only means the
+                // credential was written; saying "signed in" on that, and discovering on
+                // the next line that nobody could verify it, was a login that
+                // contradicted itself (OSK-16437).
+                let verifiedLocalDaemonId;
+                let localDaemonNotReady = false;
                 if (handoff.status === 'delivered') {
-                    this.log(`The local daemon was signed in too — ${handoff.detail}.`);
+                    const reported = await (0, local_daemon_readiness_1.reportLocalDaemonAfterHandoff)({
+                        deliveredDetail: handoff.detail,
+                        bin: this.config.bin,
+                        log: (line) => this.log(line),
+                    });
+                    verifiedLocalDaemonId = reported.daemonId;
+                    localDaemonNotReady = reported.notReady;
                 }
                 else if (handoff.reason === 'platform-managed') {
                     // A Dedicated guest: its daemon is the platform's, so there is
@@ -291,20 +304,35 @@ class Login extends base_command_1.BaseCommand {
                     // people to re-login a healthy daemon (OSK-12189).
                     this.log(`This login was not handed to the local daemon (${handoff.detail}). ` +
                         `The daemon keeps whatever sign-in it already had — check it with ` +
-                        `\`${this.config.bin} daemon status\`, and run \`${this.config.bin} daemon login\` ` +
+                        `\`${(0, daemon_command_hint_1.daemonCommandHint)('status', { bin: this.config.bin })}\`, and run ` +
+                        `\`${(0, daemon_command_hint_1.daemonCommandHint)('login', { bin: this.config.bin })}\` ` +
                         `only if that says it is signed out.`);
                 }
-                else if (handoff.status === 'failed') {
+                else if (handoff.status === 'failed' || handoff.reason === 'unsupported-daemon') {
+                    // `unsupported-daemon` is a skip, not a failure, and printed nothing: the
+                    // person was left believing the machine was signed in.
                     this.log(`The local daemon was not signed in (${handoff.detail}). ` +
-                        `Run \`${this.config.bin} daemon login\` to finish.`);
+                        `Run \`${(0, daemon_command_hint_1.daemonCommandHint)('login', { bin: this.config.bin })}\` to finish.`);
+                }
+                if (localDaemonNotReady) {
+                    // The daemon has been described above; asking the same question of it
+                    // again would print a second, vaguer version of the same answer.
+                    if (!(0, managed_instruction_setup_1.shouldSkipInstructionPrompt)()) {
+                        this.log(`Run \`${this.config.bin} instructions setup\` once it is connected.`);
+                    }
+                }
+                else {
+                    // Authentication above is complete. The common onboarding helper verifies
+                    // the daemon's live account/server/profile and handles failures nonfatally.
+                    // The wait for a daemon that was just handed a credential has already
+                    // happened (`confirmLocalDaemonSignedIn`), and its answer is passed on.
+                    await (0, managed_instruction_setup_1.maybeOfferManagedInstructionSetup)({
+                        trigger: 'login',
+                        log: (line) => this.log(line),
+                        ...(verifiedLocalDaemonId ? { daemonId: verifiedLocalDaemonId } : {}),
+                        waitForLocalReady: false,
+                    });
                 }
-                // Authentication above is complete. The common onboarding helper verifies
-                // the daemon's live account/server/profile and handles failures nonfatally.
-                await (0, managed_instruction_setup_1.maybeOfferManagedInstructionSetup)({
-                    trigger: 'login',
-                    log: (line) => this.log(line),
-                    waitForLocalReady: handoff.status === 'delivered',
-                });
             }
             catch (err) {
                 // Ctrl-C while waiting for device-code approval: a person's decision,

package/dist/commands/machines/list.js

@@ -1,7 +1,9 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 const core_1 = require("@oclif/core");
+const machine_uuid_1 = require("@skrr-ai/auth-core/machine-uuid");
 const base_command_1 = require("../../base-command");
+const keychain_1 = require("../../lib/keychain");
 const format_1 = require("../../lib/format");
 const machines_1 = require("../../lib/machines");
 /**
@@ -23,8 +25,10 @@ class MachinesList extends base_command_1.BaseCommand {
         // with the one live machine at the top and a wall of dead ones after it
         // (OSK-7639). A list exists so someone can pick a row.
         all: core_1.Flags.boolean({
-            description: 'Include machines that are offline, paused, stopped or terminated. Off by default — dead sandboxes ' +
-                'accumulate forever and bury the ones you can use.',
+            description: 'Include every machine, including the offline, paused, stopped or terminated hosted ones, ' +
+                'and the older harness registrations a machine keeps. Off by default — dead sandboxes and ' +
+                'replaced registrations accumulate forever and bury the ones you can use. Your own ' +
+                'computers are listed while offline either way, with when they were last seen.',
         }),
     };
     async run() {
@@ -38,9 +42,18 @@ class MachinesList extends base_command_1.BaseCommand {
             this.handleApiError(err);
         }
         const all = machines ?? [];
-        const shown = flags.all ? all : all.filter((m) => (0, machines_1.isReachableMachine)(m));
+        const reachable = flags.all ? all : all.filter((m) => (0, machines_1.isListedByDefault)(m));
+        // A machine row keeps every harness registration it ever had, so one daemon
+        // coming online made a list of four rows read as ninety-eight (OSK-16440).
+        // The default shows what still stands; `--all` shows the history.
+        const views = reachable.map((m) => (0, machines_1.machineListView)(m, {
+            all: flags.all === true,
+            hardwareMachineUuid: (0, machine_uuid_1.getHardwareMachineUuid)(),
+            activeProfile: (0, keychain_1.getActiveProfile)(),
+        }));
+        const shown = reachable;
         if (flags.json) {
-            this.log(JSON.stringify(shown, null, 2));
+            this.log(JSON.stringify(views, null, 2));
             return;
         }
         if (all.length === 0) {
@@ -51,14 +64,17 @@ class MachinesList extends base_command_1.BaseCommand {
             this.log(`No machine is reachable right now (${(0, machines_1.hiddenMachinesSummary)(all)}) — see them with --all.`);
             return;
         }
-        (0, format_1.renderTable)(shown.map((m) => ({
+        (0, format_1.renderTable)(shown.map((m, index) => ({
             // Printed in full. A hard 20-character slice with no ellipsis made
             // `machine:03548116-d36` read like a complete id and it is not — and
             // two different hosted machines rendered as byte-identical rows,
             // so the one column a list exists to be picked from could not
             // distinguish them (OSK-7621).
             id: (0, format_1.formatCell)(m.id),
-            name: (0, format_1.formatCell)(m.preferences?.name || m.name),
+            // Which row is the computer in front of you, said on the row: two machines
+            // can carry the same hostname, and "the one I am on" is what a person is
+            // usually picking (OSK-16440).
+            name: (0, format_1.formatCell)(`${m.preferences?.name || m.name}${views[index].thisComputer ? ' (this computer)' : ''}`),
             status: (0, format_1.formatCell)((0, machines_1.machineStateLabel)(m)),
             // Both numbers matter: "2/7" is a machine that is up with most of its
             // harnesses signed out, which reads nothing like "7/7". The
@@ -88,6 +104,12 @@ class MachinesList extends base_command_1.BaseCommand {
                 this.log(`  ${m.preferences?.name || m.name}: ${m.statusReason}`);
             }
         }
+        const foldedHistory = views.reduce((sum, view) => sum + (view.hiddenHarnessCount ?? 0), 0);
+        if (foldedHistory > 0) {
+            this.log('');
+            this.log(`${foldedHistory} older harness registration(s) that a newer one replaced are hidden — ` +
+                'pass --all to include them, or --json for the daemons each machine has registered.');
+        }
         if (shown.length < all.length) {
             const hidden = all.filter((m) => !shown.includes(m));
             this.log('');

package/dist/commands/projects/show.js

@@ -52,7 +52,7 @@ class ProjectsShow extends base_command_1.BaseCommand {
             ...project,
         });
         if (flags.json) {
-            this.log(JSON.stringify({ ...project, url }, null, 2));
+            this.printJsonResult({ ...project, url });
             return;
         }
         for (const line of (0, projects_1.renderProjectShow)(project)) {

package/dist/commands/revoke-daemon.js

@@ -4,6 +4,7 @@ const core_1 = require("@oclif/core");
 const base_command_1 = require("../base-command");
 const api_fetch_1 = require("../lib/api-fetch");
 const daemon_ref_1 = require("../lib/daemon-ref");
+const daemon_command_hint_1 = require("../lib/daemon-command-hint");
 /**
  * `skrr revoke-daemon <id-or-prefix>`
  *
@@ -105,7 +106,7 @@ class RevokeDaemon extends base_command_1.BaseCommand {
                 // Said now, not discovered later as a refresh failure: the sign-in this
                 // command used was brokered by that daemon and died with it.
                 this.warn(`This also signed out ${this.config.bin} on this machine: its sign-in was brokered ` +
-                    `by that daemon and was revoked with it. Run \`${this.config.bin} daemon login\` to ` +
+                    `by that daemon and was revoked with it. Run \`${(0, daemon_command_hint_1.daemonCommandHint)('login', { bin: this.config.bin })}\` to ` +
                     `re-enroll the daemon, then \`${this.config.bin} login\` to sign the CLI in again.`);
             }
             // exit 0 (default)

package/dist/commands/spaces/workflows/archive.d.ts

@@ -1,5 +1,6 @@
 import { BaseCommand } from '../../../base-command';
 export default class SpacesWorkflowsArchive extends BaseCommand {
+    static aliases: string[];
     static description: string;
     static args: {
         spaceId: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;

package/dist/commands/spaces/workflows/archive.js

@@ -5,6 +5,7 @@ const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../../base-command");
 const prompt_1 = require("../../../lib/prompt");
 class SpacesWorkflowsArchive extends base_command_1.BaseCommand {
+    static aliases = ['workflows:archive'];
     static description = 'Archive a workflow while retaining immutable versions and run history';
     static args = {
         spaceId: core_1.Args.string({ required: true, ignoreStdin: true }),

package/dist/commands/spaces/workflows/artifact.d.ts

@@ -1,5 +1,6 @@
 import { BaseCommand } from '../../../base-command';
 export default class SpacesWorkflowsArtifact extends BaseCommand {
+    static aliases: string[];
     static description: string;
     static args: {
         spaceId: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;

package/dist/commands/spaces/workflows/artifact.js

@@ -5,6 +5,7 @@ const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../../base-command");
 const space_output_1 = require("../../../lib/space-output");
 class SpacesWorkflowsArtifact extends base_command_1.BaseCommand {
+    static aliases = ['workflows:artifact'];
     static description = 'Read a persisted input or output artifact for a workflow node run';
     static args = {
         spaceId: core_1.Args.string({ description: 'Owning Space ID', required: true, ignoreStdin: true }),

package/dist/commands/spaces/workflows/cancel-node.d.ts

@@ -1,5 +1,6 @@
 import { BaseCommand } from '../../../base-command';
 export default class SpacesWorkflowsCancelNode extends BaseCommand {
+    static aliases: string[];
     static description: string;
     static examples: string[];
     static args: {

package/dist/commands/spaces/workflows/cancel-node.js

@@ -6,6 +6,7 @@ const base_command_1 = require("../../../base-command");
 const prompt_1 = require("../../../lib/prompt");
 const workflow_node_control_1 = require("../../../lib/workflow-node-control");
 class SpacesWorkflowsCancelNode extends base_command_1.BaseCommand {
+    static aliases = ['workflows:cancel-node'];
     static description = 'Request cancellation of one exact workflow node attempt and show the returned execution state';
     static examples = [
         '<%= config.bin %> spaces workflows run <space> <run> --json > reviewed-run.json',

package/dist/commands/spaces/workflows/cancel.d.ts

@@ -39,6 +39,7 @@ export declare function describePreservedChildren(spaceId: string, parentRunId:
     reviewedScope?: boolean;
 }): string[];
 export default class SpacesWorkflowsCancel extends BaseCommand {
+    static aliases: string[];
     static description: string;
     static args: {
         spaceId: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;

package/dist/commands/spaces/workflows/cancel.js

@@ -80,6 +80,7 @@ function describePreservedChildren(spaceId, parentRunId, children, { reviewedSco
     return lines;
 }
 class SpacesWorkflowsCancel extends base_command_1.BaseCommand {
+    static aliases = ['workflows:cancel'];
     static description = 'Cancel a workflow run and stop further graph coordination';
     static args = {
         spaceId: core_1.Args.string({ description: 'Owning Space ID', required: true, ignoreStdin: true }),
@@ -87,7 +88,7 @@ class SpacesWorkflowsCancel extends base_command_1.BaseCommand {
     };
     static flags = {
         json: core_1.Flags.boolean({
-            description: 'Output the cancelled run as JSON (cancellation.scope and the waitingFor.workflowRunId of the cancelled nodes show which child runs were left running)',
+            description: 'Output the cancelled run as JSON (cancelReceipt.outcome says whether this request stopped it or found it already cancelled; cancellation.scope and the waitingFor.workflowRunId of the cancelled nodes show which child runs were left running)',
         }),
         scope: core_1.Flags.string({
             description: 'What happens to child workflow runs this run started: preserve_children (default) leaves them running, subtree stops every descendant run too',
@@ -128,7 +129,10 @@ class SpacesWorkflowsCancel extends base_command_1.BaseCommand {
             this.log(JSON.stringify((0, space_output_1.withSpacePerspectiveUrl)(this.cliConfig.baseURL, args.spaceId, 'workflows', run), null, 2));
             return;
         }
-        this.log(`Cancelled workflow run ${run.id} (${run.status}).`);
+        // The receipt says whether THIS request stopped the run or found it stopped (and why): a
+        // repeat used to print "Cancelled" as if new, hiding a stop its deadline made earlier
+        // (OSK-15336).
+        this.log((0, data_provider_1.describeWorkflowRunCancelReceipt)(run.id, run.cancelReceipt));
         for (const line of describeCancellationProgress(args.spaceId, run))
             this.log(line);
         if ((run.cancellation?.scope ?? run.cancellationScope ?? scope) === 'subtree')

package/dist/commands/spaces/workflows/create.d.ts

@@ -3,6 +3,7 @@ import { BaseCommand } from '../../../base-command';
 /** A headless-safe first draft: valid on its own, then ready for template insertion or editing. */
 export declare function starterWorkflowGraph(): WorkflowGraph;
 export default class SpacesWorkflowsCreate extends BaseCommand {
+    static aliases: string[];
     static description: string;
     static args: {
         spaceId: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;

package/dist/commands/spaces/workflows/create.js

@@ -6,6 +6,7 @@ const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../../base-command");
 const prompt_1 = require("../../../lib/prompt");
 const space_output_1 = require("../../../lib/space-output");
+const workflow_issues_1 = require("../../../lib/workflow-issues");
 /** A headless-safe first draft: valid on its own, then ready for template insertion or editing. */
 function starterWorkflowGraph() {
     return {
@@ -23,6 +24,7 @@ function starterWorkflowGraph() {
     };
 }
 class SpacesWorkflowsCreate extends base_command_1.BaseCommand {
+    static aliases = ['workflows:create'];
     static description = 'Create a workflow draft from graph or Canvas draft JSON';
     static args = { spaceId: core_1.Args.string({ required: true, ignoreStdin: true }) };
     static flags = {
@@ -94,7 +96,11 @@ class SpacesWorkflowsCreate extends base_command_1.BaseCommand {
         this.log(`Created draft ${workflow.id}: ${workflow.title}`);
         if (decorated.url)
             this.log(`URL:  ${decorated.url}`);
-        this.log(`Next: skrr spaces workflows publish ${args.spaceId} ${workflow.id}`);
+        for (const line of (0, workflow_issues_1.describeDraftIssues)(workflow.issues))
+            this.log(line);
+        this.log(workflow.issues?.some((issue) => issue.severity === 'error')
+            ? `Next: fix the draft, then skrr workflows update ${args.spaceId} ${workflow.id} --graph <file>`
+            : `Next: skrr workflows publish ${args.spaceId} ${workflow.id}`);
     }
 }
 exports.default = SpacesWorkflowsCreate;

package/dist/commands/spaces/workflows/events.d.ts

@@ -2,6 +2,7 @@ import { BaseCommand } from '../../../base-command';
 export declare function workflowEventStreamHeaders(token: string | null | undefined, env?: Record<string, string | undefined>): Record<string, string>;
 /** Follow the server's metadata-only SSE invalidation stream until Ctrl-C. */
 export default class SpacesWorkflowsEvents extends BaseCommand {
+    static aliases: string[];
     static description: string;
     static args: {
         spaceId: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;

package/dist/commands/spaces/workflows/events.js

@@ -14,6 +14,7 @@ function workflowEventStreamHeaders(token, env = process.env) {
 }
 /** Follow the server's metadata-only SSE invalidation stream until Ctrl-C. */
 class SpacesWorkflowsEvents extends base_command_1.BaseCommand {
+    static aliases = ['workflows:events'];
     static description = 'Follow durable workflow run events (Ctrl-C to stop; fetch run details for canonical state)';
     static args = {
         spaceId: core_1.Args.string({ required: true, ignoreStdin: true }),