@skrr-ai/cli 0.1.13 → 0.1.15

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

@@ -5,11 +5,21 @@ 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");
+/**
+ * `0 = unlimited` is the persisted convention for both task limits. A flag
+ * description that omits it leaves the most useful setting undiscoverable —
+ * a user who wants "no cap" has no reason to guess that zero means it.
+ */
+const UNLIMITED_HINT = '0 = unlimited';
 class SpacesAutoExecutorUpdate extends base_command_1.BaseCommand {
     static description = 'Update a space auto-executor configuration';
     static examples = [
         '<%= config.bin %> spaces auto-executor update <space-id> --max-active-tasks 6',
         '<%= config.bin %> spaces auto-executor update <space-id> --enabled --interval-ms 600000',
+        '<%= config.bin %> spaces auto-executor update <space-id> --mode cadenced',
+        '<%= config.bin %> spaces auto-executor update <space-id> --daily-time 09:00 --daily-time 17:00 --timezone America/Vancouver',
+        '<%= config.bin %> spaces auto-executor update <space-id> --cron "0 */2 * * *"',
+        '<%= config.bin %> spaces auto-executor update <space-id> --max-starts-per-run 0   # unlimited',
         '<%= config.bin %> spaces auto-executor update <space-id> --no-enabled --json',
     ];
     static args = {
@@ -21,9 +31,28 @@ class SpacesAutoExecutorUpdate extends base_command_1.BaseCommand {
             description: 'Path to a JSON file (or "-" for stdin) to use as the patch body',
         }),
         enabled: core_1.Flags.boolean({ description: 'Enable or disable auto-execution', allowNo: true }),
-        'interval-ms': core_1.Flags.integer({ description: 'Auto-executor interval in milliseconds' }),
-        'max-starts-per-run': core_1.Flags.integer({ description: 'Max tasks to start per auto run' }),
-        'max-active-tasks': core_1.Flags.integer({ description: 'Max active task executions in this space' }),
+        mode: core_1.Flags.string({
+            description: 'continuous = a finished task promptly wakes the Leader to refill capacity; ' +
+                'cadenced = only run on the schedule, leaving a review window between batches',
+            options: [...data_provider_1.SPACE_AUTO_EXECUTION_MODES],
+        }),
+        'interval-ms': core_1.Flags.integer({
+            description: `Run every N milliseconds (min ${data_provider_1.SPACE_AUTO_EXECUTION_LIMITS.intervalMs.min}, max ${data_provider_1.SPACE_AUTO_EXECUTION_LIMITS.intervalMs.max})`,
+        }),
+        cron: core_1.Flags.string({ description: 'Run on a cron expression, e.g. "0 */2 * * *"' }),
+        'daily-time': core_1.Flags.string({
+            description: 'Run daily at HH:MM (24h). Repeatable for several times a day.',
+            multiple: true,
+        }),
+        timezone: core_1.Flags.string({
+            description: 'IANA timezone for --cron / --daily-time (default UTC)',
+        }),
+        'max-starts-per-run': core_1.Flags.integer({
+            description: `Max tasks to start per auto run (${UNLIMITED_HINT}, max ${data_provider_1.SPACE_AUTO_EXECUTION_LIMITS.maxStartsPerRun.max})`,
+        }),
+        'max-active-tasks': core_1.Flags.integer({
+            description: `Max concurrently active task executions in this space (${UNLIMITED_HINT}, max ${data_provider_1.SPACE_AUTO_EXECUTION_LIMITS.maxActiveTasks.max})`,
+        }),
     };
     async run() {
         this.requireAuth();
@@ -35,9 +64,37 @@ class SpacesAutoExecutorUpdate extends base_command_1.BaseCommand {
         catch (err) {
             this.error(err.message, { exit: 1 });
         }
+        // The API accepts exactly one recurrence shape. Rejecting here — by name,
+        // with the alternatives listed — beats a 400 that echoes the Zod refine.
+        const scheduleFlags = [
+            flags.cron !== undefined ? '--cron' : null,
+            flags['daily-time']?.length ? '--daily-time' : null,
+            flags['interval-ms'] !== undefined ? '--interval-ms' : null,
+        ].filter(Boolean);
+        if (scheduleFlags.length > 1) {
+            this.error(`Pick one schedule shape: ${scheduleFlags.join(', ')} cannot be combined.`, {
+                exit: 1,
+            });
+        }
+        const timezone = flags.timezone ? { timezone: flags.timezone } : {};
+        let schedule;
+        if (flags.cron !== undefined) {
+            schedule = { cronExpression: flags.cron, ...timezone };
+        }
+        else if (flags['daily-time']?.length) {
+            schedule = { times: flags['daily-time'], ...timezone };
+        }
+        if (flags.timezone && !schedule) {
+            this.error('--timezone applies to --cron or --daily-time; pass one of them too.', {
+                exit: 1,
+            });
+        }
         const payload = (0, prompt_1.mergeDefined)(body, {
             enabled: flags.enabled,
-            intervalMs: flags['interval-ms'],
+            mode: flags.mode,
+            // `intervalMs` and `schedule` are mutually exclusive at the API.
+            intervalMs: schedule ? undefined : flags['interval-ms'],
+            schedule,
             maxStartsPerRun: flags['max-starts-per-run'],
             maxActiveTasks: flags['max-active-tasks'],
         });
@@ -59,11 +116,25 @@ class SpacesAutoExecutorUpdate extends base_command_1.BaseCommand {
             return;
         }
         const space = response?.space ?? {};
+        // A stored `0` means unlimited. Printing the raw number told the user the
+        // opposite of what the setting does.
+        const limit = (value) => typeof value === 'number' ? (0, data_provider_1.formatLimit)(value === 0 ? null : value) : '-';
         this.log(`Updated auto-executor for ${space.id ?? args.id}`);
         this.log(`Enabled:          ${space.autoExecuteEnabled ? 'yes' : 'no'}`);
-        this.log(`Interval:         ${space.autoExecuteIntervalMs ?? '-'}`);
-        this.log(`Max starts/run:   ${space.autoExecuteMaxStartsPerRun ?? '-'}`);
-        this.log(`Max active tasks: ${space.autoExecuteMaxActiveTasks ?? '-'}`);
+        this.log(`Mode:             ${space.autoExecuteMode ?? '-'}`);
+        this.log(`Interval:         ${formatIntervalMs(space.autoExecuteIntervalMs)}`);
+        this.log(`Max starts/run:   ${limit(space.autoExecuteMaxStartsPerRun)}`);
+        this.log(`Max active tasks: ${limit(space.autoExecuteMaxActiveTasks)}`);
     }
 }
 exports.default = SpacesAutoExecutorUpdate;
+/** Milliseconds are how the field is stored, not how anyone reads a cadence. */
+function formatIntervalMs(value) {
+    if (typeof value !== 'number' || !Number.isFinite(value))
+        return '-';
+    const minutes = value / 60_000;
+    if (minutes < 60)
+        return `${minutes % 1 === 0 ? minutes : minutes.toFixed(1)}m (${value}ms)`;
+    const hours = minutes / 60;
+    return `${hours % 1 === 0 ? hours : hours.toFixed(1)}h (${value}ms)`;
+}

package/dist/commands/spaces/capacity.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 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_execution_1 = require("../../lib/task-execution");
@@ -33,11 +34,22 @@ class SpacesCapacity extends base_command_1.BaseCommand {
             return;
         }
         const space = capacity.space;
+        // `maxActiveTasks` / `availableSlots` are null when the space has no cap.
+        // Interpolating them directly is what produced `0/null active` (OSK-7087);
+        // the shared formatters in @skrr-ai/data-provider are the one place that
+        // decides how an absent ceiling reads, for the web and the CLI alike.
         this.log(`Space:      ${space.title ?? args.id}`);
         this.log(`ID:         ${space.id ?? args.id}`);
         this.log(`Auto exec:  ${space.autoExecuteEnabled ? 'enabled' : 'disabled'}`);
-        this.log(`Capacity:   ${capacity.activeCount}/${capacity.maxActiveTasks} active`);
-        this.log(`Available:  ${capacity.availableSlots}`);
+        this.log(`Capacity:   ${(0, data_provider_1.formatRunningCapacity)(capacity.activeCount, capacity.maxActiveTasks)} active` +
+            (capacity.maxActiveTasks === null ? ' (no concurrency cap)' : ''));
+        this.log(`Available:  ${(0, data_provider_1.formatLimit)(capacity.availableSlots)}`);
+        if (capacity.budget) {
+            const { consumedStarts, maxStarts, remaining, unlimited } = capacity.budget;
+            this.log(unlimited
+                ? `Starts:     ${consumedStarts} used this round (no per-run budget)`
+                : `Starts:     ${consumedStarts}/${maxStarts} used this round, ${remaining} left`);
+        }
         this.log(`Sampled:    ${capacity.sampledTaskCount} tasks`);
         if (capacity.activeTasks.length > 0) {
             this.log('');

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

@@ -17,6 +17,19 @@ import { BaseCommand } from '../../base-command';
  * convention `describeResourceNotFound` follows in `base-command`.
  */
 export declare function noSpacesMatchedMessage(q: string, bin?: string): string;
+/**
+ * What the unfiltered zero-row case is allowed to say.
+ *
+ * A bare `No spaces.` on a fresh account is technically true and operationally
+ * useless: it names no next step, so the user reading `GETTING_STARTED.md`
+ * step 3 hits three of these in a row and concludes the product is inert.
+ * `commitments/list.ts` learned this the hard way and left the reasoning in a
+ * comment; the fix is to follow the same pattern here — one line naming the
+ * command that fixes it.
+ *
+ * Pure and exported so the wording is testable without an oclif runtime.
+ */
+export declare function noSpacesEmptyMessage(bin?: string): string;
 export default class SpacesList extends BaseCommand {
     static description: string;
     static examples: string[];

package/dist/commands/spaces/list.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.noSpacesMatchedMessage = noSpacesMatchedMessage;
+exports.noSpacesEmptyMessage = noSpacesEmptyMessage;
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
@@ -29,6 +30,22 @@ function noSpacesMatchedMessage(q, bin = 'skrr') {
         'not a tokenized search, so word order matters.\n' +
         `  → retry with one distinctive word, or list them all with \`${bin} spaces list\`.`);
 }
+/**
+ * What the unfiltered zero-row case is allowed to say.
+ *
+ * A bare `No spaces.` on a fresh account is technically true and operationally
+ * useless: it names no next step, so the user reading `GETTING_STARTED.md`
+ * step 3 hits three of these in a row and concludes the product is inert.
+ * `commitments/list.ts` learned this the hard way and left the reasoning in a
+ * comment; the fix is to follow the same pattern here — one line naming the
+ * command that fixes it.
+ *
+ * Pure and exported so the wording is testable without an oclif runtime.
+ */
+function noSpacesEmptyMessage(bin = 'skrr') {
+    return ('No spaces yet.\n' +
+        `  → Create your first one: \`${bin} spaces create --title "My workspace"\``);
+}
 class SpacesList extends base_command_1.BaseCommand {
     static description = 'List spaces';
     static examples = [
@@ -95,7 +112,9 @@ class SpacesList extends base_command_1.BaseCommand {
         const r = response;
         const items = r?.data ?? [];
         if (items.length === 0) {
-            this.log(flags.q ? noSpacesMatchedMessage(flags.q, this.config.bin) : 'No spaces.');
+            this.log(flags.q
+                ? noSpacesMatchedMessage(flags.q, this.config.bin)
+                : noSpacesEmptyMessage(this.config.bin));
             return;
         }
         const rows = items.map((s) => ({

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

@@ -20,6 +20,10 @@ export default class SpacesUpdate extends BaseCommand {
         'estimate-migration': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'default-estimate': import("@oclif/core/lib/interfaces").OptionFlag<number | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'use-default-estimate': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        'default-expect': import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'delivery-mode': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'integration-branch': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'auto-merge-tier': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'worktree-isolation': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
     };
     run(): Promise<void>;

package/dist/commands/spaces/update.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
+const tasks_1 = require("../../lib/tasks");
 const base_command_1 = require("../../base-command");
 const web_url_1 = require("../../lib/web-url");
 const prompt_1 = require("../../lib/prompt");
@@ -70,6 +71,29 @@ class SpacesUpdate extends base_command_1.BaseCommand {
             allowNo: true,
             description: 'Whether rollups treat an unestimated task as --default-estimate',
         }),
+        'default-expect': core_1.Flags.string({
+            multiple: true,
+            description: tasks_1.EXPECT_FLAG_HELP +
+                ' This is the space DEFAULT, and it is where the contract actually gets set: nobody ' +
+                'declares one per task, but a space default is configured once and then holds. A ' +
+                'per-task --expect wins. Merged into deliveryPolicy without clobbering its other fields.',
+        }),
+        'delivery-mode': core_1.Flags.string({
+            options: ['none', 'branch', 'pr', 'pr_automerge', 'direct'],
+            description: "How this space's coding tasks integrate their work. 'pr' opens a pull request; " +
+                "'direct' pushes to the integration branch. Note that auto-merging modes are gated " +
+                'by --auto-merge-tier: naming one here does not by itself authorise landing code.',
+        }),
+        'integration-branch': core_1.Flags.string({
+            description: "The branch this space's coding tasks integrate into, e.g. main or dev. Also the " +
+                'default target for a code expectation that names none.',
+        }),
+        'auto-merge-tier': core_1.Flags.string({
+            options: ['off', 'checks_only', 'trusted'],
+            description: 'How far a task may go without a person. "off" (default) downgrades every auto-merging ' +
+                'mode to a human-approved PR; "checks_only" honours pr_automerge; "trusted" also honours ' +
+                "direct. This is the space owner's sanction and it outranks any per-task override.",
+        }),
         'worktree-isolation': core_1.Flags.string({
             options: ['shared', 'ephemeral'],
             description: "Space default for how this space's autonomous task runs use git worktrees. " +
@@ -105,8 +129,24 @@ class SpacesUpdate extends base_command_1.BaseCommand {
         // which the API replaces wholesale on update. Merge into the policy the user
         // supplied via --from-json, else the space's current policy, so setting just
         // the isolation default never clobbers mode / integrationBranch / trust tier.
+        // Parse before any write: a bad spec must fail here, not reach the server
+        // as a silently-dropped field. A space default nobody is enforcing is worse
+        // than none, because every task under it then shows a contract that isn't real.
+        let defaultExpectations;
+        if (flags['default-expect'] !== undefined && flags['default-expect'].length > 0) {
+            const parsed = (0, tasks_1.parseExpectations)(flags['default-expect']);
+            if (!parsed.ok) {
+                this.error(parsed.error, { exit: 1 });
+            }
+            else {
+                defaultExpectations = parsed.value;
+            }
+        }
         const isolation = flags['worktree-isolation'];
-        if (isolation) {
+        const deliveryMode = flags['delivery-mode'];
+        const integrationBranch = flags['integration-branch'];
+        const autoMergeTier = flags['auto-merge-tier'];
+        if (isolation || defaultExpectations || deliveryMode || integrationBranch || autoMergeTier) {
             let basePolicy = {};
             if (payload.deliveryPolicy && typeof payload.deliveryPolicy === 'object') {
                 basePolicy = { ...payload.deliveryPolicy };
@@ -123,7 +163,14 @@ class SpacesUpdate extends base_command_1.BaseCommand {
                     // Fall through: send a policy carrying only the isolation default.
                 }
             }
-            payload.deliveryPolicy = { ...basePolicy, executionIsolation: isolation };
+            payload.deliveryPolicy = {
+                ...basePolicy,
+                ...(isolation ? { executionIsolation: isolation } : {}),
+                ...(deliveryMode ? { mode: deliveryMode } : {}),
+                ...(integrationBranch ? { integrationBranch } : {}),
+                ...(autoMergeTier ? { autoMergeTrustTier: autoMergeTier } : {}),
+                ...(defaultExpectations ? { defaultExpectations } : {}),
+            };
         }
         // Goal links are NOT part of the PATCH body — the server rejects a PATCH
         // /api/spaces/:id carrying `goalIds` with `space_relations_atomic_only` and

package/dist/commands/tasks/complete.d.ts

@@ -1,6 +1,7 @@
 import { dataService } from '@skrr-ai/data-provider';
 import type { TaskReport } from '@skrr-ai/data-provider';
 import { BaseCommand } from '../../base-command';
+import { type TaskDeliveryReceipt } from '../../lib/git-task';
 type CompletionStatus = 'completed' | 'failed';
 type TaskCommentResult = Awaited<ReturnType<typeof dataService.addTaskComment>>;
 type MovedTaskResult = Awaited<ReturnType<typeof dataService.moveTask>>;
@@ -9,15 +10,18 @@ export interface CompleteEndpointResponse {
     task: MovedTaskResult | null;
     comment: TaskCommentResult | null;
     commentFailed?: boolean;
+    deliveryHeld?: boolean;
+    deliveryReason?: string;
 }
 export interface CompleteTransport {
     /** POST /api/tasks/:taskId/complete — atomic move + TASK_OUTPUT comment. */
     postComplete: (taskId: string, payload: {
         status: CompletionStatus;
         summary: string;
-        artifacts: string;
+        deliverables: string;
         verdict: string;
         report?: TaskReport;
+        deliveryReceipt?: TaskDeliveryReceipt;
     }) => Promise<CompleteEndpointResponse>;
     /** Legacy step 1 — POST /api/tasks/:taskId/comments. */
     addComment: (taskId: string, body: string) => Promise<TaskCommentResult>;
@@ -31,6 +35,8 @@ export interface DurableCompletionResult {
     movedTask: MovedTaskResult | null;
     moved: boolean;
     commentFailed: boolean;
+    deliveryHeld?: boolean;
+    deliveryReason?: string;
     /** true when the atomic /complete endpoint handled the finalization. */
     usedAtomicEndpoint: boolean;
 }
@@ -60,7 +66,7 @@ export declare function resolveCompletionTarget(taskRef: string, lookup: (taskRe
  * The repo contract calls this "the only terminal closeout", and a task has one
  * canonical outcome — but a second run used to succeed silently, post another
  * TASK_OUTPUT block, and leave `tasks output` rendering the throwaway one as
- * authoritative. The real summary and its artifacts link were still on the task
+ * authoritative. The real summary and its deliverables link were still on the task
  * but no longer canonical, which is the worst of the available failures: no
  * error, no warning, exit 0, and a record that now says something different.
  *
@@ -120,9 +126,10 @@ export declare function completeTaskDurably(opts: {
     status: CompletionStatus;
     body: string;
     summary: string;
-    artifacts: string;
+    deliverables: string;
     verdict: string;
     report?: TaskReport;
+    deliveryReceipt?: TaskDeliveryReceipt;
     noMove: boolean;
     transport: CompleteTransport;
 }): Promise<DurableCompletionResult>;
@@ -139,8 +146,8 @@ export default class TasksComplete extends BaseCommand {
         amend: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         status: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
         summary: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
-        artifacts: import("@oclif/core/lib/interfaces").OptionFlag<string[], import("@oclif/core/lib/interfaces").CustomOptions>;
-        artifact: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        deliverables: import("@oclif/core/lib/interfaces").OptionFlag<string[], import("@oclif/core/lib/interfaces").CustomOptions>;
+        deliverable: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         verdict: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
         outcome: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         finding: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;

package/dist/commands/tasks/complete.js

@@ -21,6 +21,7 @@ const task_instruction_offer_1 = require("../../lib/task-instruction-offer");
 const task_transcript_1 = require("../../lib/task-transcript");
 const web_url_1 = require("../../lib/web-url");
 const tasks_1 = require("../../lib/tasks");
+const git_task_1 = require("../../lib/git-task");
 /**
  * Mutation endpoints require the task's canonical UUID even though the CLI
  * accepts human-readable refs and titles. Resolve the display reference before
@@ -55,7 +56,7 @@ async function resolveCompletionTarget(taskRef, lookup) {
  * The repo contract calls this "the only terminal closeout", and a task has one
  * canonical outcome — but a second run used to succeed silently, post another
  * TASK_OUTPUT block, and leave `tasks output` rendering the throwaway one as
- * authoritative. The real summary and its artifacts link were still on the task
+ * authoritative. The real summary and its deliverables link were still on the task
  * but no longer canonical, which is the worst of the available failures: no
  * error, no warning, exit 0, and a record that now says something different.
  *
@@ -128,7 +129,7 @@ exports.TaskCompletionMismatchError = TaskCompletionMismatchError;
  * error handling.
  */
 async function completeTaskDurably(opts) {
-    const { taskId, status, body, summary, artifacts, verdict, report, noMove, transport } = opts;
+    const { taskId, status, body, summary, deliverables, verdict, report, deliveryReceipt, noMove, transport, } = opts;
     if (noMove) {
         const comment = await transport.addComment(taskId, body);
         return {
@@ -143,9 +144,10 @@ async function completeTaskDurably(opts) {
         const response = await transport.postComplete(taskId, {
             status,
             summary,
-            artifacts,
+            deliverables,
             verdict,
             ...(report ? { report } : {}),
+            ...(deliveryReceipt ? { deliveryReceipt } : {}),
         });
         // Reconcile before reporting. On the session-scoped path the server owns
         // the task selection, so a divergence here means the task we were asked to
@@ -160,6 +162,8 @@ async function completeTaskDurably(opts) {
             movedTask: response.task ?? null,
             moved: true,
             commentFailed: response.commentFailed === true,
+            ...(response.deliveryHeld ? { deliveryHeld: true } : {}),
+            ...(response.deliveryReason ? { deliveryReason: response.deliveryReason } : {}),
             usedAtomicEndpoint: true,
         };
     }
@@ -174,7 +178,13 @@ async function completeTaskDurably(opts) {
     // behavior (comment first, then move).
     const comment = await transport.addComment(taskId, body);
     const movedTask = await transport.moveTask(taskId, status === 'completed' ? 'done' : 'failed');
-    return { comment, movedTask, moved: true, commentFailed: false, usedAtomicEndpoint: false };
+    return {
+        comment,
+        movedTask,
+        moved: true,
+        commentFailed: false,
+        usedAtomicEndpoint: false,
+    };
 }
 class TasksComplete extends base_command_1.BaseCommand {
     static description = 'Post a canonical TASK_OUTPUT comment and finish a task';
@@ -199,7 +209,7 @@ class TasksComplete extends base_command_1.BaseCommand {
             description: 'Task ID or identifier; overrides daemon runtime context',
         }),
         'from-json': core_1.Flags.string({
-            description: 'Path to JSON with taskId, status, summary, artifacts, and verdict; "-" for stdin',
+            description: 'Path to JSON with taskId, status, summary, deliverables, and verdict; "-" for stdin',
         }),
         amend: core_1.Flags.boolean({
             description: 'Supersede the canonical outcome of an already-completed task. Without it, a differing report on a closed task is refused',
@@ -210,13 +220,13 @@ class TasksComplete extends base_command_1.BaseCommand {
             default: 'completed',
         }),
         summary: core_1.Flags.string({ description: 'Completion summary' }),
-        artifacts: core_1.Flags.string({
-            description: 'Artifacts produced, or "none". Repeatable, and interchangeable with --artifact. ' +
-                `The whole TASK_OUTPUT block (status + summary + artifacts + verdict) must fit in ${data_provider_1.TASK_OUTPUT_BODY_MAX} ` +
+        deliverables: core_1.Flags.string({
+            description: 'Deliverables produced, or "none". Repeatable, and interchangeable with --deliverable. ' +
+                `The whole TASK_OUTPUT block (status + summary + deliverables + verdict) must fit in ${data_provider_1.TASK_OUTPUT_BODY_MAX} ` +
                 'characters — link to commits, PRs, or file paths rather than pasting content.',
-            // Repeatable, even though `--artifact` already is. The two spellings sit
-            // one letter apart and both read as "the artifacts go here", so the wrong
-            // one gets typed — and oclif answered that with "Flag --artifacts can only
+            // Repeatable, even though `--deliverable` already is. The two spellings sit
+            // one letter apart and both read as "the deliverables go here", so the wrong
+            // one gets typed — and oclif answered that with "Flag --deliverables can only
             // be specified once", which names the flag you used and not the sibling
             // that does what you wanted. Dogfooded (OSK-266) by losing a completion
             // call to exactly that, then re-running with the two references pre-joined
@@ -224,9 +234,9 @@ class TasksComplete extends base_command_1.BaseCommand {
             multiple: true,
             default: ['none'],
         }),
-        artifact: core_1.Flags.string({
+        deliverable: core_1.Flags.string({
             description: 'A single artifact reference (repeatable); joined with commas. Prefer this ' +
-                'over pre-joining into --artifacts.',
+                'over pre-joining into --deliverables.',
             multiple: true,
         }),
         verdict: core_1.Flags.string({ description: 'Final verdict', default: 'approved' }),
@@ -271,13 +281,13 @@ class TasksComplete extends base_command_1.BaseCommand {
             summary: flags.summary,
             // Both spellings, pooled, then normalized to the one-line form the server
             // stores. Pooling rather than letting one win: someone who passes both has
-            // named artifacts in each, and silently dropping half of them is worse
+            // named deliverables in each, and silently dropping half of them is worse
             // than either flag being unavailable.
-            artifacts: (0, data_provider_1.normalizeTaskOutputArtifacts)([
-                ...(flags.artifact ?? []),
+            deliverables: (0, data_provider_1.normalizeTaskOutputDeliverables)([
+                ...(flags.deliverable ?? []),
                 // The literal default is dropped when it is only the default; a user
-                // who typed `--artifacts none` alongside a real one meant the real one.
-                ...(flags.artifacts ?? []).filter((value) => value !== 'none'),
+                // who typed `--deliverables none` alongside a real one meant the real one.
+                ...(flags.deliverables ?? []).filter((value) => value !== 'none'),
             ]) || 'none',
             verdict: flags.verdict,
         });
@@ -312,14 +322,14 @@ class TasksComplete extends base_command_1.BaseCommand {
         }
         let status;
         let summary;
-        let artifacts;
+        let deliverables;
         let verdict;
         let report;
         try {
             status = normalizeStatus(input.status);
             summary = requireOutputField(input.summary, 'summary');
-            artifacts =
-                optionalOutputField((0, data_provider_1.normalizeTaskOutputArtifacts)(input.artifacts)) || 'none';
+            deliverables =
+                optionalOutputField((0, data_provider_1.normalizeTaskOutputDeliverables)(input.deliverables)) || 'none';
             verdict =
                 optionalOutputField(input.verdict) || (status === 'completed' ? 'approved' : 'failed');
             const jsonReport = input.report && typeof input.report === 'object'
@@ -360,7 +370,7 @@ class TasksComplete extends base_command_1.BaseCommand {
         // budget the server enforces. Discovering an undocumented cap by HTTP 400
         // after a run has already finished is the failure being fixed; a caller must
         // be able to see the limit without provoking it.
-        const budget = (0, data_provider_1.measureTaskOutputBudget)({ status, summary, artifacts, verdict });
+        const budget = (0, data_provider_1.measureTaskOutputBudget)({ status, summary, deliverables, verdict });
         if (!budget.fits) {
             this.failWithCliError({
                 message: (0, data_provider_1.describeTaskOutputOverflow)(budget),
@@ -375,14 +385,21 @@ class TasksComplete extends base_command_1.BaseCommand {
                 },
             });
         }
-        const body = formatTaskOutput({ status, summary, artifacts, verdict });
+        const body = formatTaskOutput({ status, summary, deliverables, verdict });
+        // Artifact text such as a commit SHA is a delivery claim. The local probe
+        // both verifies remote-default ancestry and anchors a commit that has no
+        // local branch before the completion request can release this run.
+        const deliveryReceipt = status === 'completed' && !flags['no-move']
+            ? ((0, git_task_1.inspectTaskDeliveryArtifacts)({ deliverables, taskRef: taskIdentity.id }) ?? undefined)
+            : undefined;
         // Computed once, OUTSIDE the try, so the original attempt and the queued
         // replay carry the same key. Must match the payload the outbox stores.
         const completionPayload = {
             summary,
-            artifacts,
+            deliverables,
             verdict,
             ...(report ? { report } : {}),
+            ...(deliveryReceipt ? { deliveryReceipt } : {}),
             noMove: !!flags['no-move'],
         };
         const completionIdempotencyKey = (0, outbox_1.idempotencyKeyFor)('task-complete', taskIdentity.id, JSON.stringify(completionPayload));
@@ -456,7 +473,7 @@ class TasksComplete extends base_command_1.BaseCommand {
             if (decision === 'refuse') {
                 this.failWithCliError({
                     message: `Task ${completionTaskId} is already ${priorStatus}, and this report differs from its canonical ` +
-                        `outcome — completing again would silently replace it (artifacts included).\n` +
+                        `outcome — completing again would silently replace it (deliverables included).\n` +
                         `  → re-run with \`--amend\` to supersede it deliberately, or read what is there now with ` +
                         `\`${this.config.bin} tasks output ${completionTaskId}\`.`,
                     code: 'TASK_ALREADY_COMPLETED',
@@ -473,9 +490,10 @@ class TasksComplete extends base_command_1.BaseCommand {
                 status,
                 body,
                 summary,
-                artifacts,
+                deliverables,
                 verdict,
                 report,
+                deliveryReceipt,
                 noMove: !!flags['no-move'],
                 transport: {
                     postComplete: (taskId, payload) => (0, api_fetch_1.apiFetch)(
@@ -496,7 +514,7 @@ class TasksComplete extends base_command_1.BaseCommand {
             });
         }
         catch (err) {
-            // The completion summary/artifacts/verdict are authored content — the
+            // The completion summary/deliverables/verdict are authored content — the
             // record of what the agent actually did. Losing that to a 503 costs the
             // reasoning, not just a retry, so an OUTAGE queues it durably (OSK-2806)
             // while every 4xx still fails immediately through handleApiError.
@@ -531,11 +549,12 @@ class TasksComplete extends base_command_1.BaseCommand {
             taskOutput: {
                 status,
                 summary,
-                artifacts,
+                deliverables,
                 verdict,
                 ...(report ? { report } : {}),
                 body,
             },
+            ...(result.deliveryHeld ? { deliveryHeld: true, deliveryReason: result.deliveryReason } : {}),
         };
         if (flags.json) {
             const url = movedTask
@@ -558,6 +577,9 @@ class TasksComplete extends base_command_1.BaseCommand {
                 ? `Task ${completedTaskId} was already ${newStatus}; the outcome was amended.`
                 : `Moved task ${completedTaskId} to ${newStatus}`);
         }
+        if (result.deliveryHeld) {
+            this.warn('Delivery is not verified on the default branch; the task was held at in_review instead of done.');
+        }
         if (report) {
             const url = (0, web_url_1.taskUrlFrom)(this.cliConfig.baseURL, movedTask ?? { id: completedTaskId });
             if (url)

package/dist/commands/tasks/continue-ready.js

@@ -70,7 +70,15 @@ class TasksContinueReady extends base_command_1.BaseCommand {
                 this.handleApiError(err);
             }
         }
-        const freeSlots = capacity ? capacity.availableSlots : Number.POSITIVE_INFINITY;
+        // `availableSlots` is null for an UNLIMITED space, which is the default.
+        // `Math.min(n, null)` is 0, so the previous arithmetic silently computed a
+        // start limit of zero and reported every ready task as "skipped for
+        // capacity" on exactly the spaces that had no capacity limit at all. The
+        // platform's own helper warns about this: callers must branch on null
+        // rather than do arithmetic on it (SpaceTaskConcurrency.availableSlotsFor).
+        const freeSlots = capacity && capacity.availableSlots !== null
+            ? capacity.availableSlots
+            : Number.POSITIVE_INFINITY;
         const startLimit = Math.max(0, Math.min(flags['max-starts'], freeSlots));
         const selected = ready.slice(0, startLimit);
         const skippedForCapacity = ready.slice(startLimit);

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

@@ -53,6 +53,7 @@ export default class TasksCreate extends BaseCommand {
         'assignee-user': import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'dri-agent': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'dri-user': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        expect: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'execution-mode': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'start-date': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'due-date': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;