@skrr-ai/cli 0.1.126 → 0.1.128

package/dist/commands/spaces/auto-executor/preview.js

@@ -1,10 +1,41 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.canStartLine = canStartLine;
+exports.principalLine = principalLine;
 exports.refillLines = refillLines;
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../../base-command");
 const space_output_1 = require("../../../lib/space-output");
+/**
+ * "Can start" is a capacity figure, not a forecast. Printed bare next to a disabled autopilot
+ * it read as "one task will start now" (dogfooded: `Autopilot: disabled` / `Can start: 1 now`).
+ */
+function canStartLine(space, startsAvailable) {
+    const capacity = (0, data_provider_1.formatLimit)(startsAvailable);
+    return space.autoExecuteEnabled
+        ? `Can start:  ${capacity} now`
+        : `Can start:  nothing — autopilot is off (capacity ${capacity})`;
+}
+/**
+ * Whether unattended work may run in the Space at all. Turning the board off does not revoke the
+ * principal and a revoked one keeps a disabled board silent, so the two have to be told apart.
+ * Returns null for a server that predates the field, so an old API prints no claim.
+ */
+function principalLine(space, spaceRef) {
+    const principal = space.executionPrincipal;
+    if (principal === undefined)
+        return null;
+    if (principal === null) {
+        return ('Principal:  none — nothing may run unattended here. A Space owner sets one with ' +
+            `\`skrr spaces auto-executor update ${spaceRef} --enabled\`.`);
+    }
+    if (principal.status === 'active')
+        return `Principal:  active (version ${principal.version})`;
+    return (`Principal:  revoked${principal.revokeReason ? ` (${principal.revokeReason.replace(/_/g, ' ')})` : ''} — ` +
+        `nothing may run unattended here until a Space owner sets it again with ` +
+        `\`skrr spaces auto-executor update ${spaceRef} --enabled\`.`);
+}
 class SpacesAutoExecutorPreview extends base_command_1.BaseCommand {
     static description = 'Preview what a space auto-executor run would do';
     static examples = [
@@ -47,7 +78,10 @@ class SpacesAutoExecutorPreview extends base_command_1.BaseCommand {
         const { capacity } = plan;
         this.log(`Running:    ${(0, data_provider_1.formatRunningCapacity)(capacity.activeCount, capacity.maxActiveTasks)}` +
             (capacity.maxActiveTasks === null ? ' (no concurrency cap)' : ''));
-        this.log(`Can start:  ${(0, data_provider_1.formatLimit)(capacity.startsAvailable)} now`);
+        this.log(canStartLine(space, capacity.startsAvailable));
+        const principal = principalLine(space, spaceRef);
+        if (principal)
+            this.log(principal);
         if (capacity.budget && !capacity.budget.unlimited) {
             this.log(`Budget:     ${capacity.budget.consumedStarts}/${capacity.budget.maxStarts} starts used this round`);
         }

package/dist/commands/spaces/auto-executor/run.d.ts

@@ -1,4 +1,20 @@
 import { BaseCommand } from '../../../base-command';
+/** What `auto-execute-now` answered: `fired: false` carries the reason nothing started. */
+type RunNowAnswer = {
+    fired?: boolean;
+    reason?: string;
+    runId?: string;
+};
+/**
+ * Why a manual run did not start, in words, and whether that is a refusal or just "already busy".
+ * The route answers 200 with `fired: false` and a `reason`; printing "Triggered" for it told an
+ * owner a run had started when nothing had (a Space with no execution principal never runs).
+ * `busy` is the normal "one is already in flight" answer, not a failure.
+ */
+export declare function describeRunNotStarted(answer: RunNowAnswer, spaceRef: string): {
+    message: string;
+    busy: boolean;
+};
 export default class SpacesAutoExecutorRun extends BaseCommand {
     static description: string;
     static args: {
@@ -9,3 +25,4 @@ export default class SpacesAutoExecutorRun extends BaseCommand {
     };
     run(): Promise<void>;
 }
+export {};

package/dist/commands/spaces/auto-executor/run.js

@@ -1,9 +1,52 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.describeRunNotStarted = describeRunNotStarted;
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../../base-command");
 const space_output_1 = require("../../../lib/space-output");
+/**
+ * Why a manual run did not start, in words, and whether that is a refusal or just "already busy".
+ * The route answers 200 with `fired: false` and a `reason`; printing "Triggered" for it told an
+ * owner a run had started when nothing had (a Space with no execution principal never runs).
+ * `busy` is the normal "one is already in flight" answer, not a failure.
+ */
+function describeRunNotStarted(answer, spaceRef) {
+    switch (answer.reason) {
+        case 'principal_missing_or_revoked':
+            return {
+                busy: false,
+                message: `Nothing ran: this Space has no active execution principal, so nothing may run unattended in it. ` +
+                    `A Space owner sets one with \`skrr spaces auto-executor update ${spaceRef} --enabled\`.`,
+            };
+        case 'no_lead_agent':
+            return {
+                busy: false,
+                message: `Nothing ran: this Space has no lead agent to run its board. ` +
+                    `Set one with \`skrr spaces update ${spaceRef} --lead-agent <agent-id>\`.`,
+            };
+        case 'space_not_found':
+            return { busy: false, message: `Nothing ran: Space ${spaceRef} was not found.` };
+        case 'disabled':
+            return {
+                busy: false,
+                message: `Nothing ran: auto-execution is turned off for this Space. ` +
+                    `Turn it on with \`skrr spaces auto-executor update ${spaceRef} --enabled\`.`,
+            };
+        case 'active_orchestration':
+        case 'lock_held':
+            return {
+                busy: true,
+                message: `A run is already in flight for this Space${answer.runId ? ` (${answer.runId})` : ''}; ` +
+                    `it carries this request, so no second one was started.`,
+            };
+        default:
+            return {
+                busy: false,
+                message: `Nothing ran${answer.reason ? ` (${answer.reason})` : ''}: the server did not start a run.`,
+            };
+    }
+}
 class SpacesAutoExecutorRun extends base_command_1.BaseCommand {
     static description = 'Run a space auto-executor immediately';
     static args = {
@@ -21,10 +64,25 @@ class SpacesAutoExecutorRun extends base_command_1.BaseCommand {
         catch (err) {
             this.handleApiError(err);
         }
-        if (flags.json)
+        const answer = (result ?? {});
+        const notStarted = answer.fired === false ? describeRunNotStarted(answer, spaceRef) : undefined;
+        if (flags.json) {
             this.log(JSON.stringify((0, space_output_1.withSpacePerspectiveUrl)(this.cliConfig.baseURL, spaceRef, 'execution', { result }), null, 2));
-        else
+            // The payload above already says `fired: false`; the exit status says it too, so a
+            // script that only checks `$?` is not told a refusal succeeded.
+            if (notStarted && !notStarted.busy)
+                this.exit(1);
+            return;
+        }
+        if (!notStarted) {
             this.log(`Triggered auto-executor for space ${spaceRef}.`);
+            return;
+        }
+        if (notStarted.busy) {
+            this.log(notStarted.message);
+            return;
+        }
+        this.error(notStarted.message, { exit: 1 });
     }
 }
 exports.default = SpacesAutoExecutorRun;

package/dist/commands/spaces/files/upload.js

@@ -35,7 +35,6 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", { value: true });
 const fs = __importStar(require("node:fs"));
 const path = __importStar(require("node:path"));
-const node_buffer_1 = require("node:buffer");
 const node_crypto_1 = require("node:crypto");
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
@@ -86,7 +85,7 @@ class SpacesFilesUpload extends base_command_1.BaseCommand {
         formData.append('spaceId', spaceRef);
         if (flags.folder)
             formData.append('folder_id', flags.folder);
-        formData.append('file', new node_buffer_1.Blob([bytes], { type: (0, file_mime_1.mimeTypeForFilename)(filename) }), filename);
+        formData.append('file', new Blob([bytes], { type: (0, file_mime_1.mimeTypeForFilename)(filename) }), filename);
         let uploaded;
         try {
             uploaded = await data_provider_1.dataService.uploadFile(formData);

package/dist/commands/spaces/index.js

@@ -46,6 +46,29 @@ const PERSPECTIVES = [
         ],
         url: '/spaces/<space-id>/schedule',
     },
+    {
+        perspective: 'loops',
+        purpose: "The Space's standing loops (weekly review, curator, goal advance). No command lists them: read their cycles from the notebook",
+        commands: [
+            'spaces notebook <space-id> --kinds loop_cycle',
+            'commitments pack plan --pack space_review --space <space-id>',
+            'spaces loops migrate <space-id> --from heartbeats',
+            'spaces loops release <space-id> --commitment <id>',
+        ],
+        url: '/spaces/<space-id>/loops',
+    },
+    {
+        perspective: 'notebook',
+        purpose: 'Task results, loop cycles, hypothesis verdicts, journal entries and reports, newest first',
+        commands: ['spaces notebook <space-id>'],
+        url: '/spaces/<space-id>/notebook',
+    },
+    {
+        perspective: 'proposals',
+        purpose: 'Writes held for approval, loop action bundles and Memory proposals; how unattended writes are governed',
+        commands: ['spaces proposals list <space-id>', 'spaces write-policy show <space-id>'],
+        url: '/spaces/<space-id>/proposals',
+    },
     {
         perspective: 'actions',
         purpose: 'Agent actions and space custom actions',

package/dist/commands/spaces/revisions.js

@@ -5,6 +5,7 @@ const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
 const format_1 = require("../../lib/format");
 const space_output_1 = require("../../lib/space-output");
+const revision_actor_1 = require("../../lib/revision-actor");
 class SpacesRevisions extends base_command_1.BaseCommand {
     static description = 'List revisions for a space';
     static examples = [
@@ -40,14 +41,14 @@ class SpacesRevisions extends base_command_1.BaseCommand {
         const rows = items.map((rev) => ({
             version: String(rev.revisionNumber ?? '-'),
             changeType: String(rev.changeType ?? '-'),
-            user: String(rev.userId ?? '-'),
+            user: (0, revision_actor_1.revisionActorLabel)(rev),
             changes: String(Array.isArray(rev.changes) ? rev.changes.length : 0),
             when: rev.createdAt ? String(rev.createdAt).slice(0, 19).replace('T', ' ') : '-',
         }));
         (0, format_1.renderTable)(rows, [
             { key: 'version', header: 'REV' },
             { key: 'changeType', header: 'CHANGE' },
-            { key: 'user', header: 'USER' },
+            { key: 'user', header: 'ACTOR' },
             { key: 'changes', header: 'FIELDS' },
             { key: 'when', header: 'AT' },
         ], (line) => this.log(line));

package/dist/commands/tasks/revisions.js

@@ -5,6 +5,7 @@ const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
 const format_1 = require("../../lib/format");
+const revision_actor_1 = require("../../lib/revision-actor");
 /** " (via CLI credential)" when the revision recorded its credential channel. */
 function revisionChannelSuffix(rev) {
     const via = rev.metadata?.via;
@@ -46,14 +47,14 @@ class TasksRevisions extends base_command_1.BaseCommand {
             version: String(rev.revisionNumber ?? '-'),
             changeType: String(rev.changeType ?? '-'),
             // The channel the person's credential came through, when recorded (OSK-16621).
-            user: `${String(rev.userId ?? '-')}${revisionChannelSuffix(rev)}`,
+            user: `${(0, revision_actor_1.revisionActorLabel)(rev)}${rev.userId != null ? revisionChannelSuffix(rev) : ''}`,
             changes: String(Array.isArray(rev.changes) ? rev.changes.length : 0),
             when: rev.createdAt ? String(rev.createdAt).slice(0, 19).replace('T', ' ') : '-',
         }));
         (0, format_1.renderTable)(rows, [
             { key: 'version', header: 'REV' },
             { key: 'changeType', header: 'CHANGE' },
-            { key: 'user', header: 'USER' },
+            { key: 'user', header: 'ACTOR' },
             { key: 'changes', header: 'FIELDS' },
             { key: 'when', header: 'AT' },
         ], (line) => this.log(line));

package/dist/commands/triggers/fire-detail.js

@@ -29,6 +29,8 @@ function describeFireAuditDetail(detail, fireId, triggerId) {
         lines.push(`Duration:     ${String(detail.durationMs)}ms`);
     if (detail.conversationId)
         lines.push(`Conversation: ${String(detail.conversationId)}`);
+    // Tool calls refused during a turn that was delivered: `triggers show` points here for the facts.
+    lines.push(...(0, triggers_1.fireRefusalLines)(detail.metadata));
     return lines;
 }
 class TriggersFireDetail extends base_command_1.BaseCommand {