@skrr-ai/cli 0.1.108 → 0.1.110

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

@@ -1,5 +1,6 @@
 import { BaseCommand } from '../../../base-command';
 export default class SpacesWorkflowsTest 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/test.js

@@ -6,6 +6,7 @@ const base_command_1 = require("../../../base-command");
 const prompt_1 = require("../../../lib/prompt");
 const space_output_1 = require("../../../lib/space-output");
 class SpacesWorkflowsTest extends base_command_1.BaseCommand {
+    static aliases = ['workflows:test'];
     static description = 'Start a test workflow run, optionally using recorded node fixtures';
     static args = {
         spaceId: core_1.Args.string({ description: 'Owning Space ID', required: true, ignoreStdin: true }),

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

@@ -2,6 +2,7 @@ import { type WorkflowTrigger } from '@skrr-ai/data-provider';
 import { BaseCommand } from '../../../../base-command';
 declare function readTriggerConfig(inline: string | undefined, file: string | undefined): Promise<WorkflowTrigger['config']>;
 export default class SpacesWorkflowsTriggersCreate 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/triggers/create.js

@@ -15,6 +15,7 @@ async function readTriggerConfig(inline, file) {
     return value;
 }
 class SpacesWorkflowsTriggersCreate extends base_command_1.BaseCommand {
+    static aliases = ['workflows:triggers:create'];
     static description = 'Create a schedule, webhook, or space-event workflow trigger';
     static args = {
         spaceId: core_1.Args.string({ required: true, ignoreStdin: true }),

package/dist/commands/spaces/workflows/triggers/fire.d.ts

@@ -1,5 +1,6 @@
 import { BaseCommand } from '../../../../base-command';
 export default class SpacesWorkflowsTriggersFire 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/triggers/fire.js

@@ -6,6 +6,7 @@ const base_command_1 = require("../../../../base-command");
 const prompt_1 = require("../../../../lib/prompt");
 const space_output_1 = require("../../../../lib/space-output");
 class SpacesWorkflowsTriggersFire extends base_command_1.BaseCommand {
+    static aliases = ['workflows:triggers:fire'];
     static description = 'Manually fire a workflow trigger with an idempotent delivery identity';
     static args = {
         spaceId: core_1.Args.string({ required: true, ignoreStdin: true }),

package/dist/commands/spaces/workflows/triggers/list.d.ts

@@ -1,5 +1,6 @@
 import { BaseCommand } from '../../../../base-command';
 export default class SpacesWorkflowsTriggersList 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/triggers/list.js

@@ -6,6 +6,7 @@ const base_command_1 = require("../../../../base-command");
 const format_1 = require("../../../../lib/format");
 const space_output_1 = require("../../../../lib/space-output");
 class SpacesWorkflowsTriggersList extends base_command_1.BaseCommand {
+    static aliases = ['workflows:triggers:list'];
     static description = 'List workflow triggers';
     static args = {
         spaceId: core_1.Args.string({ required: true, ignoreStdin: true }),

package/dist/commands/spaces/workflows/triggers/update.d.ts

@@ -1,5 +1,6 @@
 import { BaseCommand } from '../../../../base-command';
 export default class SpacesWorkflowsTriggersUpdate 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/triggers/update.js

@@ -6,6 +6,7 @@ const base_command_1 = require("../../../../base-command");
 const create_1 = require("./create");
 const space_output_1 = require("../../../../lib/space-output");
 class SpacesWorkflowsTriggersUpdate extends base_command_1.BaseCommand {
+    static aliases = ['workflows:triggers:update'];
     static description = 'Update a workflow trigger’s type, state, or configuration';
     static args = {
         spaceId: core_1.Args.string({ required: true, ignoreStdin: true }),

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

@@ -16,6 +16,7 @@ export declare function buildWorkflowUpdatePayload({ body, title, description, c
 }): WorkflowPatch;
 /** Update metadata or the mutable draft; executable changes use optimistic concurrency. */
 export default class SpacesWorkflowsUpdate extends BaseCommand {
+    static aliases: string[];
     static description: string;
     static examples: string[];
     static args: {

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

@@ -7,6 +7,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");
 /** `90m`, `4h`, `2d` to milliseconds; null when malformed or outside the shared bounds. */
 function parseWorkflowDuration(text) {
     const match = /^(\d+)\s*([mhd])$/i.exec(text.trim());
@@ -29,6 +30,7 @@ function buildWorkflowUpdatePayload({ body, title, description, clearDescription
 }
 /** Update metadata or the mutable draft; executable changes use optimistic concurrency. */
 class SpacesWorkflowsUpdate extends base_command_1.BaseCommand {
+    static aliases = ['workflows:update'];
     static description = 'Update workflow metadata or its mutable graph draft';
     static examples = [
         '<%= config.bin %> spaces workflows update <space-id> <workflow-id> --title "Release train"',
@@ -159,6 +161,8 @@ class SpacesWorkflowsUpdate extends base_command_1.BaseCommand {
         }
         this.log(`Updated workflow ${workflow.id}: ${workflow.title}`);
         this.log(`Draft revisions: semantic ${workflow.draft.semanticRevision}, presentation ${workflow.draft.presentationRevision}`);
+        for (const line of (0, workflow_issues_1.describeDraftIssues)(workflow.issues))
+            this.log(line);
     }
 }
 exports.default = SpacesWorkflowsUpdate;

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

@@ -1,5 +1,6 @@
 import { BaseCommand } from '../../../base-command';
 export default class SpacesWorkflowsValidate 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/validate.js

@@ -4,7 +4,9 @@ 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");
+const workflow_issues_1 = require("../../../lib/workflow-issues");
 class SpacesWorkflowsValidate extends base_command_1.BaseCommand {
+    static aliases = ['workflows:validate'];
     static description = 'Validate a workflow draft before publishing';
     static args = {
         spaceId: core_1.Args.string({ description: 'Owning Space ID', required: true, ignoreStdin: true }),
@@ -28,15 +30,38 @@ class SpacesWorkflowsValidate extends base_command_1.BaseCommand {
             this.handleApiError(error);
             return;
         }
+        // The one rule (OSK-16493): an error-severity issue means invalid, whatever `valid` says. A
+        // server that predates the shared gate answered `valid: true` beside such an issue, and the
+        // text view then called the draft ready to publish and hid the issue.
+        const blocking = (0, data_provider_1.blockingWorkflowIssues)(result.issues ?? []);
+        const valid = Boolean(result.valid) && blocking.length === 0;
         if (flags.json) {
             this.log(JSON.stringify((0, space_output_1.withSpacePerspectiveUrl)(this.cliConfig.baseURL, args.spaceId, 'workflows', result), null, 2));
+            // One answer, two renderings: an invalid draft fails the command in either mode, so a
+            // script reading --json can trust the exit status exactly as a person reading text can
+            // (OSK-16473).
+            if (!valid)
+                this.exit(1);
             return;
         }
-        if (result.valid) {
+        const advisories = (result.issues ?? []).filter((issue) => issue.severity !== 'error');
+        if (valid) {
             this.log('Workflow draft is valid and ready to publish.');
+            if (advisories.length) {
+                this.log(`${advisories.length} warning${advisories.length === 1 ? '' : 's'} (publishing is allowed):`);
+                for (const line of (0, workflow_issues_1.formatWorkflowIssues)(advisories))
+                    this.log(line);
+            }
             return;
         }
-        this.log(`Workflow draft is invalid${result.code ? ` (${result.code})` : ''}: ${result.message ?? 'Unknown validation error'}`);
+        this.log(`Workflow draft is invalid${result.code ? ` (${result.code})` : ''}: ${result.message ?? blocking[0]?.message ?? 'Unknown validation error'}`);
+        // Every problem, not only the first one publish would stop at.
+        const all = result.issues ?? [];
+        if (all.length) {
+            this.log('All problems found:');
+            for (const line of (0, workflow_issues_1.formatWorkflowIssues)(all))
+                this.log(line);
+        }
         this.exit(1);
     }
 }

package/dist/commands/tasks/activity.js

@@ -6,7 +6,18 @@ 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 task_economics_1 = require("../../lib/task-economics");
 const task_legacy_1 = require("../../lib/task-legacy");
+/**
+ * A system-written embedded entry names no person (OSK-16475): it carries
+ * `details.actorKind: 'system'` instead of a `userId`, so the actor is the service.
+ */
+function systemEntryActor(details) {
+    const value = details;
+    if (value?.actorKind !== 'system')
+        return undefined;
+    return `system:${value.actorService ?? '-'}`;
+}
 function activityRow(entry) {
     const actor = entry.actor;
     const actorText = actor
@@ -17,12 +28,15 @@ function activityRow(entry) {
                 : actor.kind === 'integration'
                     ? `integration:${actor.provider}`
                     : `system:${actor.service}`
-        : (entry.userId ?? entry.actorId);
+        : (entry.userId ?? entry.actorId ?? systemEntryActor(entry.details));
+    const type = entry.action ?? entry.event ?? entry.type;
     return {
         time: (0, format_1.localTime)(entry.occurredAt ?? entry.timestamp ?? entry.createdAt),
-        event: (0, format_1.formatCell)(entry.action ?? entry.event ?? entry.type),
+        event: (0, format_1.formatCell)(type),
         actor: (0, format_1.formatCell)(actorText),
-        details: (0, format_1.formatCell)(entry.data ?? entry.details ?? entry.metadata),
+        details: typeof type === 'string' && type.startsWith(data_provider_1.TASK_ECONOMICS_ACTIVITY_TYPE_PREFIX)
+            ? (0, task_economics_1.describeEconomicsChange)(entry.data)
+            : (0, format_1.formatCell)(entry.data ?? entry.details ?? entry.metadata),
     };
 }
 /**

package/dist/commands/tasks/bulk-update.d.ts

@@ -3,6 +3,7 @@ export default class TasksBulkUpdate extends BaseCommand {
     static description: string;
     static examples: string[];
     static flags: {
+        'expect-target': import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         'from-json': import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
         'dry-run': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;

package/dist/commands/tasks/bulk-update.js

@@ -7,6 +7,7 @@ const web_url_1 = require("../../lib/web-url");
 const prompt_1 = require("../../lib/prompt");
 const dry_run_plans_1 = require("../../lib/dry-run-plans");
 const task_batch_1 = require("../../lib/task-batch");
+const write_target_1 = require("../../lib/write-target");
 const tasks_1 = require("../../lib/tasks");
 class TasksBulkUpdate extends base_command_1.BaseCommand {
     static description = 'Apply partial updates to many tasks from JSON';
@@ -14,6 +15,7 @@ class TasksBulkUpdate extends base_command_1.BaseCommand {
         '<%= config.bin %> tasks bulk-update --from-json updates.json --json',
         '<%= config.bin %> tasks bulk-update --from-json updates.json --continue-on-error',
         '<%= config.bin %> tasks bulk-update --from-json updates.json --dry-run --json',
+        '<%= config.bin %> tasks bulk-update --from-json updates.json --expect-target skrr.ai --json',
     ];
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output bulk result as JSON' }),
@@ -27,6 +29,7 @@ class TasksBulkUpdate extends base_command_1.BaseCommand {
         'continue-on-error': core_1.Flags.boolean({
             description: 'Continue applying later updates after one task fails',
         }),
+        ...write_target_1.expectTargetFlag,
     };
     async run() {
         this.requireAuth();
@@ -80,6 +83,9 @@ class TasksBulkUpdate extends base_command_1.BaseCommand {
                 this.exit(1);
             return;
         }
+        // Everything above only read. Say where the writes land, and honour
+        // `--expect-target`, before the first one (OSK-S-440 #10).
+        const target = await this.assertWriteTarget(flags['expect-target']);
         const results = [];
         const errors = [];
         const rows = updates.map((update) => {
@@ -180,6 +186,7 @@ class TasksBulkUpdate extends base_command_1.BaseCommand {
             const base = this.cliConfig.baseURL;
             const enriched = {
                 ...summary,
+                target,
                 results: results.map((row) => {
                     const rec = row;
                     // Only link rows that actually updated — a failed row may not exist

package/dist/commands/tasks/complete.js

@@ -357,7 +357,8 @@ class TasksComplete extends base_command_1.BaseCommand {
         for (const sentence of (0, task_closure_1.closureSnapshotLines)(result.task?.resolution?.closure)) {
             this.log(sentence);
         }
-        const recurrenceLine = (0, task_recurrence_outcome_1.recurrenceOutcomeLine)(result.recurrence);
+        const completedRef = result.task?.identifier || taskId;
+        const recurrenceLine = (0, task_recurrence_outcome_1.recurrenceOutcomeLine)(result.recurrence, completedRef, this.config.bin);
         if (recurrenceLine)
             this.log(recurrenceLine);
         for (const advisory of result.advisories || [])

package/dist/commands/tasks/create-tree.d.ts

@@ -6,6 +6,14 @@ export type TreeNode = TaskRecord & {
 export type PlannedNode = {
     key: string;
     parentKey?: string;
+    /**
+     * Keys of other nodes in this file that block this one (`blockedByTempIds`).
+     * The ORDER of `planTree`'s result is parents-first only; `orderForCreation`
+     * makes it blockers-first too.
+     */
+    blockerKeys?: string[];
+    /** True when `externalKey` was filled in from a key prefix rather than written on the node. */
+    keyDerived?: boolean;
     node: TreeNode;
     payload: TaskRecord;
 };
@@ -45,6 +53,12 @@ export type TreeFailure = {
 /**
  * What a half-applied tree reports (E4). The ids already created are the only
  * way to continue or clean up, and the failing call used to print none of them.
+ *
+ * Every confirmed id is in `mapping` however the run ended, and `summary` counts
+ * the three things a failure can leave: confirmed, UNKNOWN (submitted, outcome
+ * not established — it may exist) and not attempted. They are different next
+ * steps, and a count that folded them together was the thing the first report
+ * of OSK-S-440 could not rely on.
  */
 export declare function partialFailureResult(args: {
     results: Array<Record<string, unknown>>;
@@ -55,14 +69,23 @@ export declare function partialFailureResult(args: {
     keyed?: boolean;
     warnings: Array<Record<string, unknown>>;
     failures?: Array<Record<string, unknown>>;
+    /** The command to run next, as it should be typed. */
+    next?: string;
 }): {
     warnings?: Record<string, unknown>[] | undefined;
+    next?: string | undefined;
     rerun: string;
     failures?: Record<string, unknown>[] | undefined;
     object: string;
     ok: boolean;
     count: number;
     total: number;
+    summary: {
+        confirmed: number;
+        unknown: number;
+        notAttempted: number;
+        failed: number;
+    };
     created: {
         tempId: unknown;
         parentTempId: unknown;
@@ -74,22 +97,68 @@ export declare function partialFailureResult(args: {
     mapping: Record<string, string>;
     failedAt: TreeFailure;
 };
+/**
+ * Order nodes so everything a node depends on — its parent and every blocker —
+ * comes BEFORE it, keeping input order otherwise. The batch and the serial path
+ * both create in this order, so a parent id and a blocker id exist by the time a
+ * node that needs them is written.
+ *
+ * Also the place a bad `blockedByTempIds` is caught: before any request, naming
+ * the node, rather than as a per-row refusal after half the tree exists.
+ */
+export declare function orderForCreation(planned: PlannedNode[]): {
+    ordered: PlannedNode[];
+    errors: string[];
+};
 export default class TasksCreateTree extends BaseCommand {
     static description: string;
     static examples: string[];
     static flags: {
+        'expect-target': import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         'from-json': import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
         'dry-run': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        verify: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         'external-key-prefix': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         fresh: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         resume: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        'no-batch': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         'default-agent': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         space: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         goal: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        project: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
     };
     run(): Promise<void>;
+    /**
+     * `--verify`: which nodes of this tree exist, and which do not. Read-only by
+     * construction — it is reached before anything that writes — so it is safe to
+     * run at any time, as often as wanted, against a tree of any size.
+     */
+    private verifyTree;
+    /**
+     * The command that continues this tree, as it should be typed: this
+     * invocation's own input and scope, plus `--resume`. Rebuilt from the parsed
+     * flags rather than echoed from argv, so it is the same whether the person
+     * typed `--space x` or `--space=x`, and so it can say plainly when the input
+     * came from stdin and cannot be replayed from a path.
+     */
+    private nextCommand;
 }
+/**
+ * The note that names an Agent put in charge of nodes by DEFAULT, or undefined
+ * when no node was. Sources a person wrote themselves (`explicit-assignee`,
+ * `provided` — an operator's own `--default-agent`) are not surprises; the
+ * session's agent, a parent's, or the space lead are.
+ */
+export declare function describeDefaultAgentAssignment(resolutions: Record<string, unknown>): {
+    code: string;
+    value: {
+        count: number;
+        agentIds: string[];
+        sources: string[];
+    };
+    hint: string;
+} | undefined;
 export declare function normalizeTreeRoots(input: TaskRecord): TreeNode[];
 export declare function planTree(roots: TreeNode[], opts: {
     defaults: TaskRecord;