@skrr-ai/cli 0.1.109 → 0.1.111

package/dist/commands/tasks/show.js

@@ -6,6 +6,7 @@ exports.completionLines = completionLines;
 exports.reviewPolicyLine = reviewPolicyLine;
 exports.labelsLine = labelsLine;
 exports.taskExecutorLabel = taskExecutorLabel;
+exports.resolutionLines = resolutionLines;
 exports.recurrenceArmFailureLines = recurrenceArmFailureLines;
 exports.taskFailureLines = taskFailureLines;
 const core_1 = require("@oclif/core");
@@ -151,6 +152,43 @@ function taskExecutorLabel(t, bin = 'skrr') {
             return '-';
     }
 }
+function resolverLabel(by, names) {
+    if (!by || typeof by !== 'object' || !by.kind)
+        return null;
+    const kind = String(by.kind);
+    if (kind === 'human')
+        return `person ${(0, principal_display_1.displayPrincipal)(by.userId, names)}`;
+    if (kind === 'agent')
+        return `agent ${(0, principal_display_1.displayPrincipal)(by.agentId, names)}`;
+    if (kind === 'system')
+        return `system${by.service ? ` (${String(by.service)})` : ''}`;
+    if (kind === 'integration')
+        return `integration${by.provider ? ` (${String(by.provider)})` : ''}`;
+    return kind;
+}
+/**
+ * The `Resolution:` lines for any resolved Task: how it ended, the reason the
+ * closer recorded, who closed it, and what the close stood on. The reason used to
+ * be JSON-only, so text showed "Task workflow type is cancelled" for every
+ * cancel, and a Task its Workflow step cancelled could not be told from one a
+ * person cancelled by hand (OSK-16475). Silent for an open Task.
+ */
+function resolutionLines(t, names) {
+    const resolution = t.resolution;
+    if (!resolution || typeof resolution !== 'object' || !resolution.kind)
+        return [];
+    const reason = typeof resolution.reason === 'string' ? resolution.reason.trim() : '';
+    const closure = resolution.closure
+        ? (0, task_closure_1.closureSnapshotLines)(resolution.closure)
+        : [];
+    const lines = [`Resolution:  ${String(resolution.kind)}${reason ? ` — ${reason}` : ''}`];
+    if (closure.length > 0)
+        lines.push(`             ${closure.join(' ')}`);
+    const by = resolverLabel(resolution.resolvedBy, names);
+    if (by)
+        lines.push(`Resolved by: ${by}`);
+    return lines;
+}
 /**
  * The `Recurrence:` line for a series whose next occurrence was never armed.
  * Silent otherwise: a healthy series has nothing to say here.
@@ -163,10 +201,14 @@ function recurrenceArmFailureLines(t, bin = 'skrr') {
     const ref = (typeof t.identifier === 'string' && t.identifier) ||
         (typeof t.id === 'string' && t.id) ||
         '<task-id>';
+    // A finished occurrence is re-armed (`tasks schedule` refuses it, OSK-15338); a LIVE series the
+    // scheduler stopped on an unrunnable pattern is rescheduled with a valid one, which clears it.
+    const finished = ['completed', 'cancelled'].includes(String(t.statusType ?? ''));
     return [
         `Recurrence:  ${(0, data_provider_1.describeRecurrenceArmFailure)(code)} (${code})`,
-        // Not `tasks schedule`: that refuses the finished occurrence carrying this failure (OSK-15338).
-        `             ${(0, task_recurrence_outcome_1.recurrenceRearmHint)(code, ref, bin)}`,
+        finished || t.statusType === undefined
+            ? `             ${(0, task_recurrence_outcome_1.recurrenceRearmHint)(code, ref, bin)}`
+            : `             Reschedule it with \`${bin} tasks schedule ${ref} --at <ISO time> --cron "<minute hour day month weekday>"\`.`,
     ];
 }
 /**
@@ -203,25 +245,31 @@ function taskFailureLines(t, bin = 'skrr') {
     if (!error && !failedAt && !isFailed)
         return [];
     const lines = [];
-    const attempts = typeof t.executionAttempts === 'number' ? t.executionAttempts : null;
-    const runId = typeof t.lastFinalizedRunId === 'string' ? t.lastFinalizedRunId : null;
-    const facts = [
-        failedAt,
-        attempts != null ? `attempt ${attempts}` : null,
-        runId ? `run ${runId}` : null,
-    ].filter(Boolean);
-    if (facts.length > 0)
-        lines.push(`Failed:      ${facts.join(' · ')}`);
+    // Task failure stamps survive later successful Runs. Neither the lifetime
+    // attempt counter nor lastFinalizedRunId identifies the Run that failed.
+    if (failedAt)
+        lines.push(`Failure recorded: ${failedAt}`);
     if (error) {
         // Labelled with the JSON key, because a reader who needs this in a script
         // needs the key more than they need the prose.
-        lines.push(`Error:       ${error}`);
+        lines.push(`Recorded error: ${error}`);
         lines.push('             (`executionError` in --json)');
     }
     else if (isFailed) {
-        lines.push('Error:       none recorded on the task.');
+        lines.push('Recorded error: none recorded on the task.');
+    }
+    const failure = t.executionFailure;
+    if (failure?.recovery && typeof failure.recovery === 'string' && !error.includes(failure.recovery)) {
+        lines.push(`Recovery:    ${failure.recovery}`);
+    }
+    if (failure?.headline && typeof failure.headline === 'string' && !error.includes(failure.headline)) {
+        lines.push(`Cause:       ${failure.headline}`);
+    }
+    if (failure?.runId) {
+        lines.push(`Diagnostic:  executionFailure in --json (run ${failure.runId})`);
     }
     const ref = String(t.identifier ?? t.id ?? t._id ?? '<task-id>');
+    lines.push(`Latest runs: ${bin} tasks runs ${ref}`);
     lines.push(`Per-run why: ${bin} tasks activity ${ref} --json   (details.error, keyed by runId)`);
     return lines;
 }
@@ -352,6 +400,21 @@ class TasksShow extends base_command_1.BaseCommand {
             this.log(line);
         const agents = Array.isArray(t.assigneeAgentIds) ? t.assigneeAgentIds : [];
         const users = Array.isArray(t.assigneeUserIds) ? t.assigneeUserIds : [];
+        // Principals print as names, not ObjectIds — same rule as labels: a
+        // roster that cannot be read prints ids rather than failing the command.
+        const resolvedBy = t.resolution?.resolvedBy;
+        const principalIds = [
+            t.author,
+            t.dri?.resolved?.id,
+            ...(Array.isArray(t.assigneeAgentIds) ? t.assigneeAgentIds : []),
+            ...(Array.isArray(t.assigneeUserIds) ? t.assigneeUserIds : []),
+            t.economics?.confirmedBy,
+            resolvedBy?.userId,
+            resolvedBy?.agentId,
+        ];
+        const principalNames = await (0, principal_display_1.resolvePrincipalNames)(principalIds, {
+            spaceId: t.spaceId ?? null,
+        });
         this.log(`ID:          ${t.id ?? t._id ?? '-'}`);
         this.log(`Ref:         ${t.identifier ?? (t.taskNumber ? `#${t.taskNumber}` : '-')}`);
         this.log(`Title:       ${t.title ?? '-'}`);
@@ -368,25 +431,12 @@ class TasksShow extends base_command_1.BaseCommand {
          * Closing is the actor's call — a Task may be `done` with work still
          * outstanding, and `resolution.closure` is where the server recorded what
          * it stood on at close. Show it so a bare "done" never reads as "clean."
+         * The closure sentences share the `tasks move` receipt's renderer, so the
+         * two cannot disagree; the reason and the closer are printed for every
+         * resolved Task (OSK-16475).
          */
-        const resolution = t.resolution;
-        /*
-         * Same renderer as the `tasks move` receipt, so the two cannot disagree
-         * about a Task they are both describing.
-         *
-         * This copy showed an owner's ordinary close as an override:
-         * `TASK_RESULT_REVIEW_REQUIRED` lands in `blockers` whenever the policy is
-         * unsatisfied — including when the owner, who is not bound by the Review
-         * they declared, closes it themselves — and this rendered it identically to
-         * a genuine forced close. So the only read surface for "was this gate
-         * overridden?" could not answer the question.
-         */
-        if (resolution?.closure) {
-            const sentences = (0, task_closure_1.closureSnapshotLines)(resolution.closure);
-            if (sentences.length > 0) {
-                this.log(`Resolution:  ${resolution.kind ?? '-'} — ${sentences.join(' ')}`);
-            }
-        }
+        for (const line of resolutionLines(t, principalNames))
+            this.log(line);
         /*
          * A Result orphaned by a spec edit is not the same as no Result.
          *
@@ -519,18 +569,6 @@ class TasksShow extends base_command_1.BaseCommand {
             }
         }
         this.log(`Labels:      ${labelsLine(labelIds, labelNames)}`);
-        // Principals print as names, not ObjectIds — same rule as labels above: a
-        // roster that cannot be read prints ids rather than failing the command.
-        const principalIds = [
-            t.author,
-            t.dri?.resolved?.id,
-            ...(Array.isArray(t.assigneeAgentIds) ? t.assigneeAgentIds : []),
-            ...(Array.isArray(t.assigneeUserIds) ? t.assigneeUserIds : []),
-            t.economics?.confirmedBy,
-        ];
-        const principalNames = await (0, principal_display_1.resolvePrincipalNames)(principalIds, {
-            spaceId: t.spaceId ?? null,
-        });
         this.log(`Author:      ${t.authorDisplayName || (0, principal_display_1.displayPrincipal)(t.author, principalNames)}`);
         const dri = t.dri;
         if (dri?.resolved) {

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

@@ -8,6 +8,7 @@ export default class TasksStart extends BaseCommand {
     };
     static flags: {
         json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        'dry-run': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         task: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         skill: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         branch: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;

package/dist/commands/tasks/start.js

@@ -7,6 +7,7 @@ const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
 const task_resolver_1 = require("../../lib/task-resolver");
 const git_task_1 = require("../../lib/git-task");
+const task_admission_1 = require("../../lib/task-admission");
 class TasksStart extends base_command_1.BaseCommand {
     static description = 'Start execution of a task: its assignee agent runs it in a new conversation + session. ' +
         'Alias: tasks execute';
@@ -20,6 +21,7 @@ class TasksStart extends base_command_1.BaseCommand {
         '<%= config.bin %> tasks start <task-id> --skill <skill-id>',
         '<%= config.bin %> tasks start OSK-123 --branch',
         '<%= config.bin %> tasks start <task-id> --json',
+        '<%= config.bin %> tasks start <task-id> --dry-run --json',
     ];
     static args = {
         id: core_1.Args.string({
@@ -30,6 +32,7 @@ class TasksStart extends base_command_1.BaseCommand {
     };
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output as JSON' }),
+        'dry-run': core_1.Flags.boolean({ description: 'Preview effective unattended execution prerequisites without creating a Run or checking out a branch', exclusive: ['branch'] }),
         task: core_1.Flags.string({
             description: 'Task ID; overrides daemon runtime context',
         }),
@@ -57,6 +60,36 @@ class TasksStart extends base_command_1.BaseCommand {
                 details: { suggestion: failure.suggestion, context: failure.details },
             });
         }
+        if (flags['dry-run']) {
+            if (flags.branch)
+                this.error('--dry-run cannot check out a branch; omit --branch.', { exit: 2 });
+            let preview;
+            try {
+                // A separate route is intentional: an older /start silently ignores
+                // unknown request fields. A dryRun flag there could start real work.
+                preview = await data_provider_1.dataService.previewTaskStart(taskIdentity.id, flags.skill ? { skillId: flags.skill } : undefined);
+            }
+            catch (err) {
+                this.handleApiError(err, { subject: `Task ${taskIdentity.id} execution preview` });
+            }
+            if (!(0, task_admission_1.isAdmissionPreview)(preview)) {
+                this.failWithCliError({
+                    message: 'The server did not return a confirmed execution preview. Update the server before using --dry-run.',
+                    code: 'TASK_START_PREVIEW_UNSUPPORTED', exit: 2, retryable: false,
+                });
+            }
+            if (flags.json)
+                this.printJsonResult(preview, { ok: preview.executionAdmission.state !== 'blocked' });
+            else {
+                this.log(`Execution preview for ${preview.taskId}:`);
+                for (const line of (0, task_admission_1.admissionLines)(preview.executionAdmission))
+                    this.log(line);
+                this.log('No Run was created. Starting later rechecks current prerequisites.');
+            }
+            if (preview.executionAdmission.state === 'blocked')
+                this.exit(1);
+            return;
+        }
         // Resolve once before asking the server to start. It makes a title or
         // human-readable ref usable for branch naming, while UUID callers retain
         // their existing one-request behavior when --branch is omitted.
@@ -137,6 +170,8 @@ class TasksStart extends base_command_1.BaseCommand {
         this.log(`Started task ${taskId}${flags.skill ? ` with skill ${flags.skill}` : ''}`);
         if (branch)
             this.log(`${branch.created ? 'Created' : 'Checked out'} branch: ${branch.branch}`);
+        for (const line of (0, task_admission_1.admissionLines)(result.executionAdmission))
+            this.log(line);
         this.log(`Conversation: ${result.conversationId}`);
         this.log(`Session:      ${result.sessionId}`);
         for (const line of startFollowUpLines(this.config.bin, taskIdentity.id))

package/dist/commands/tasks/templates/list.js

@@ -1,5 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+const list_output_1 = require("../../../lib/list-output");
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../../base-command");
@@ -25,7 +26,7 @@ class TasksTemplatesList extends base_command_1.BaseCommand {
             this.handleApiError(err);
         }
         if (flags.json) {
-            this.log(JSON.stringify(response, null, 2));
+            this.printJsonResult((0, list_output_1.listResult)(response));
             return;
         }
         const items = response?.data ?? [];

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

@@ -6,6 +6,11 @@ import { BaseCommand } from '../../base-command';
  * renders `details` with, so the two surfaces describe one entry the same way.
  */
 export declare function withDetails(item: Record<string, unknown>): string;
+/**
+ * Who, not just which kind: `human:<id> (via CLI credential)` (OSK-16621). A
+ * row written before channels existed renders as it always did.
+ */
+export declare function timelineActor(item: Record<string, unknown>): string;
 export default class TasksTimeline extends BaseCommand {
     static description: string;
     static args: {

package/dist/commands/tasks/timeline.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.withDetails = withDetails;
+exports.timelineActor = timelineActor;
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
@@ -21,6 +22,13 @@ function withDetails(item) {
     const details = (0, format_1.formatCell)(item.data ?? item.details, '');
     return details ? `${preview} (${details})` : preview;
 }
+/**
+ * Who, not just which kind: `human:<id> (via CLI credential)` (OSK-16621). A
+ * row written before channels existed renders as it always did.
+ */
+function timelineActor(item) {
+    return item.actor ? (0, data_provider_1.formatTaskPrincipal)(item.actor) : '-';
+}
 class TasksTimeline extends base_command_1.BaseCommand {
     static description = 'Show the immutable multi-actor Task Activity timeline';
     static args = {
@@ -67,7 +75,7 @@ class TasksTimeline extends base_command_1.BaseCommand {
                 .replace('T', ' ')
                 .slice(0, 19),
             source: String(item.type ?? '-'),
-            actor: String(item.actor?.kind ?? '-'),
+            actor: timelineActor(item),
             // A lifecycle row's preview is the bare action, so `status_changed`
             // appeared three times in a row without ever naming a status. The
             // answer is in `details` (`status=blocked`), which `tasks activity`

package/dist/commands/tasks/unschedule.js

@@ -5,7 +5,11 @@ const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
 const web_url_1 = require("../../lib/web-url");
 class TasksUnschedule extends base_command_1.BaseCommand {
-    static description = 'Clear the schedule on a task (no future fires)';
+    // Decision (OSK-15321, round 3): unschedule ENDS the series at this occurrence; it clears the
+    // time and the recurrence. Stopping without ending is `pause-schedule` (keeps the series,
+    // `resume-schedule` continues it). A series ended here by mistake is restored, settings and
+    // all, with `rearm-recurrence <the previous occurrence>`.
+    static description = 'Clear the schedule on a task and end its series here (no future fires). To stop without ending the series use pause-schedule; to restore an ended series run rearm-recurrence on the previous occurrence';
     static examples = [
         '<%= config.bin %> tasks unschedule <task-id>',
         '<%= config.bin %> tasks unschedule <task-id> --json',

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

@@ -50,6 +50,7 @@ export default class TasksUpdate extends BaseCommand {
         goal: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         cycle: 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>;
+        unassigned: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         'assignee-agent': import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         '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>;

package/dist/commands/tasks/update.js

@@ -138,6 +138,9 @@ class TasksUpdate extends base_command_1.BaseCommand {
         space: core_1.Flags.string({
             description: 'Move task to a different space (for a project, use `tasks move --project`)',
         }),
+        unassigned: core_1.Flags.boolean({
+            description: 'Clear all Agent and Human assignees, retaining executor intent unless explicitly changed. Overrides JSON assignment; does not start a Run.',
+        }),
         'assignee-agent': core_1.Flags.string({
             description: 'Replace assignee agents — id or name (repeatable)',
             multiple: true,
@@ -153,7 +156,7 @@ class TasksUpdate extends base_command_1.BaseCommand {
             description: 'Set explicit accountable human DRI; use --from-json null to clear',
         }),
         'execution-mode': core_1.Flags.string({
-            description: 'Set the default executor class for mixed Human/Agent assignees',
+            description: 'Set executor intent independently of assignment; required when assigning both Humans and Agents',
             options: ['agent', 'human'],
         }),
         'add-assignee-agent': core_1.Flags.string({
@@ -219,6 +222,15 @@ class TasksUpdate extends base_command_1.BaseCommand {
     async run() {
         this.requireAuth();
         const { args, flags } = await this.parse(TasksUpdate);
+        this.setJsonOperation({ command: 'tasks:update', mode: 'execute' });
+        const assignmentError = (0, tasks_2.taskUnassignmentFlagError)(flags);
+        if (assignmentError)
+            this.failWithCliError({
+                message: assignmentError,
+                code: 'TASK_ASSIGNMENT_FLAG_CONFLICT',
+                exit: 2,
+                retryable: false,
+            });
         // Default to {} so TS flow-analysis is satisfied — the catch calls this.error(), which throws.
         let body = {};
         try {
@@ -451,6 +463,7 @@ class TasksUpdate extends base_command_1.BaseCommand {
             // for, so it never counts toward the "nothing to update" check below.
             repairReleasePins: flags['repair-release-pins'] ? true : undefined,
         });
+        (0, tasks_2.applyExplicitTaskUnassignment)(payload, flags.unassigned === true);
         const payloadKeys = Object.keys(payload);
         if (payload.driAgentId && payload.driUserId) {
             this.failWithCliError({
@@ -494,6 +507,7 @@ class TasksUpdate extends base_command_1.BaseCommand {
         // to the task's current space. Fetching the task is skipped entirely when
         // every assignee entry is already a raw id — the overwhelmingly common
         // scripted case — so this adds no round-trip to existing usage.
+        await this.resolveTaskResourceReferences(payload, { taskId: taskRef });
         await this.resolveAssigneesInPayload(payload, {
             resolveSpaceId: async () => {
                 try {
@@ -538,12 +552,31 @@ class TasksUpdate extends base_command_1.BaseCommand {
         if (tagWarning)
             this.warn(tagWarning);
         if (flags.json) {
-            this.log(JSON.stringify((0, web_url_1.withTaskUrl)(this.cliConfig.baseURL, updated), null, 2));
+            const row = updated;
+            this.printJsonOperation({
+                ...(0, web_url_1.withTaskUrl)(this.cliConfig.baseURL, row),
+                assignmentPreview: (0, tasks_2.taskAssignmentPreview)(row),
+            }, {
+                resource: row.id ? { kind: 'task', id: String(row.id) } : null,
+                task: row.id ? { id: String(row.id) } : null,
+            });
             return;
         }
         const t = updated;
         const ref = t.identifier ?? (t.taskNumber ? `#${t.taskNumber}` : '');
-        this.log(`Updated task ${t.id ?? t._id ?? taskRef}${ref ? ` (${ref})` : ''} — ${t.title ?? '(no title)'}`);
+        const label = `${t.id ?? t._id ?? taskRef}${ref ? ` (${ref})` : ''}`;
+        // The server's own answer about the estimate (OSK-16476): a no-op says so, never "Updated".
+        const unchanged = (0, task_economics_1.estimateUnchangedLine)(payload.economics, t);
+        const onlyEstimate = payloadKeys.every((key) => key === 'economics' || payload[key] === undefined);
+        if (unchanged && onlyEstimate) {
+            this.log(`Task ${label}: ${unchanged}`);
+            return;
+        }
+        this.log(`Updated task ${label} — ${t.title ?? '(no title)'}`);
+        if (flags.unassigned)
+            this.log(`Executor intent: ${(0, tasks_2.taskAssignmentPreview)(t).executor.mode || 'unspecified'}; assignment: unassigned. No Run requested.`);
+        if (unchanged)
+            this.log(unchanged);
         // `--yes` asserts the revocation; it does not erase that it happened.
         // Report the cost after the write too, or the evidence is silently gone.
         if (changesSpec && flags.yes && (settledSpecAssessments > 0 || acceptanceUnreadable)) {

package/dist/commands/tasks/updates/add.js

@@ -118,6 +118,7 @@ class TasksUpdatesAdd extends base_command_1.BaseCommand {
         json: core_1.Flags.boolean({ description: 'Output JSON' }),
     };
     async run() {
+        this.setJsonOperation({ command: 'tasks:updates:add', mode: 'execute' });
         this.requireAuth();
         const { args, flags } = await this.parse(TasksUpdatesAdd);
         const task = (0, task_resolver_1.resolveTaskIdentity)({ positionalId: args.task, flagTask: flags.task });
@@ -190,7 +191,11 @@ class TasksUpdatesAdd extends base_command_1.BaseCommand {
             this.handleApiError(error);
         }
         if (flags.json)
-            return this.log(JSON.stringify(result, null, 2));
+            return this.printJsonOperation(result, {
+                resource: { kind: 'task_update', id: result?.update?.id },
+                task: { id: result?.update?.taskId },
+                ...(typeof result?.idempotent === 'boolean' ? { replayed: result.idempotent } : {}),
+            });
         // A replayed idempotency key answers the SAME stored record — say so,
         // otherwise a retried write reads as a second write.
         this.log(result?.idempotent

package/dist/commands/tasks/updates/list.js

@@ -1,5 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+const list_output_1 = require("../../../lib/list-output");
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../../base-command");
@@ -44,7 +45,7 @@ class TasksUpdatesList extends base_command_1.BaseCommand {
             this.handleApiError(error);
         }
         if (flags.json)
-            return this.log(JSON.stringify(result, null, 2));
+            return this.printJsonResult((0, list_output_1.listResult)(result));
         if (!result?.data.length)
             return this.log('No Updates recorded.');
         for (const update of result.data) {

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

@@ -7,6 +7,7 @@ export default class TasksUpsert extends BaseCommand {
         'from-json': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'external-key': import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
         'default-agent': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        unassigned: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         title: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         description: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'description-format': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;

package/dist/commands/tasks/upsert.js

@@ -25,6 +25,7 @@ class TasksUpsert extends base_command_1.BaseCommand {
         'default-agent': core_1.Flags.string({
             description: 'Override the automatic assignee for creates, or fill an unassigned existing match. Use "auto" to inherit the space lead agent.',
         }),
+        unassigned: core_1.Flags.boolean({ description: 'Explicitly clear both assignee kinds, suppressing automatic defaults while retaining executor intent.', exclusive: ['default-agent', 'assignee-agent', 'assignee-user'] }),
         title: core_1.Flags.string({ description: 'Task title (required when creating)' }),
         description: core_1.Flags.string({ description: 'Task description' }),
         'description-format': core_1.Flags.string({
@@ -66,6 +67,10 @@ class TasksUpsert extends base_command_1.BaseCommand {
     async run() {
         this.requireAuth();
         const { flags } = await this.parse(TasksUpsert);
+        this.setJsonOperation({ command: 'tasks:upsert', mode: 'execute' });
+        const unassignmentError = (0, tasks_1.taskUnassignmentFlagError)(flags);
+        if (unassignmentError)
+            this.failWithCliError({ message: unassignmentError, code: 'TASK_UNASSIGNED_CONFLICT', exit: 2, retryable: false });
         let body = {};
         try {
             body = await (0, prompt_1.readJsonBody)(flags['from-json']);
@@ -92,6 +97,7 @@ class TasksUpsert extends base_command_1.BaseCommand {
             });
         }
         let scope;
+        await this.resolveTaskResourceReferences(payload);
         try {
             scope = await (0, tasks_1.resolveTaskCreateScope)(payload);
         }
@@ -219,10 +225,11 @@ class TasksUpsert extends base_command_1.BaseCommand {
             externalKey,
             defaultAgent: defaultAgentResolution,
             task,
+            assignmentPreview: (0, tasks_1.taskAssignmentPreview)(task),
             url: (0, web_url_1.taskUrlFrom)(this.cliConfig.baseURL, task),
         };
         if (flags.json) {
-            this.log(JSON.stringify(result, null, 2));
+            this.printJsonOperation(result, { resource: { kind: 'task', id: task?.id }, task: { id: task?.id } });
             return;
         }
         this.log(`${capitalize(action)} task ${(0, tasks_1.formatTaskLabel)(task)}`);

package/dist/commands/token/list.js

@@ -1,5 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+const list_output_1 = require("../../lib/list-output");
 const core_1 = require("@oclif/core");
 const base_command_1 = require("../../base-command");
 const api_fetch_1 = require("../../lib/api-fetch");
@@ -102,7 +103,7 @@ class TokenList extends base_command_1.BaseCommand {
         // web session as if it were one.
         const sessions = allSessions.filter((session) => !flags.kind || session.kind === flags.kind);
         if (flags.json) {
-            return this.log(JSON.stringify({ sessions, count: sessions.length, hasMore }, null, 2));
+            return this.printJsonResult((0, list_output_1.listResult)({ sessions, count: sessions.length, hasMore }, 'sessions'));
         }
         if (!sessions.length && !hasMore) {
             return this.log(flags.kind ? 'No matching active sessions.' : 'No active sessions.');

package/dist/commands/tokens/list.js

@@ -1,5 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+const list_output_1 = require("../../lib/list-output");
 const core_1 = require("@oclif/core");
 const base_command_1 = require("../../base-command");
 const access_tokens_1 = require("../../lib/access-tokens");
@@ -28,7 +29,7 @@ class TokensList extends base_command_1.BaseCommand {
         }
         const shown = flags.all ? tokens : tokens.filter((t) => t.state === 'active');
         if (flags.json) {
-            this.log(JSON.stringify({ accessTokens: shown }, null, 2));
+            this.printJsonResult((0, list_output_1.listResult)({ accessTokens: shown }, 'accessTokens'));
             return;
         }
         if (shown.length === 0) {

package/dist/commands/tones/list.js

@@ -1,5 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+const list_output_1 = require("../../lib/list-output");
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
@@ -41,11 +42,12 @@ class TonesList extends base_command_1.BaseCommand {
             : library.presets.filter((preset) => !flags.family || preset.family === flags.family);
         const custom = flags.presets || flags.family ? [] : library.custom;
         if (flags.json) {
-            this.log(JSON.stringify({
+            this.printJsonResult((0, list_output_1.listResult)({
+                data: [...presets, ...custom].map(tone_input_1.summarizeToneForList),
                 presets: presets.map(tone_input_1.summarizeToneForList),
                 custom: custom.map(tone_input_1.summarizeToneForList),
                 defaultToneId: library.defaultToneId ?? null,
-            }, null, 2));
+            }));
             return;
         }
         const rows = [...presets, ...custom].map((tone) => ({

package/dist/commands/triggers/audit.js

@@ -25,17 +25,25 @@ class TriggersAudit extends base_command_1.BaseCommand {
         agent: core_1.Flags.string({ description: 'Agent ID filter for owner-wide audit history' }),
         scope: core_1.Flags.string({ description: 'Scope kind filter for owner-wide audit history' }),
         outcome: core_1.Flags.string({
-            description: 'Outcome filter for owner-wide audit history (succeeded, failed, skipped, incomplete, capped, ...); an unknown value is refused with the full list',
+            description: 'Outcome filter (succeeded, failed, skipped, incomplete, capped, ...); an unknown value is refused with the full list',
         }),
         source: core_1.Flags.string({
-            description: 'Source filter for owner-wide audit history (schedule, once, event, manual, webhook, replay, ...); an unknown value is refused with the full list',
+            description: 'Source filter (schedule, once, event, manual, webhook, replay, ...); an unknown value is refused with the full list',
         }),
     };
     async run() {
         this.requireAuth();
         const { args, flags } = await this.parse(TriggersAudit);
         const triggerId = args.id ?? flags.trigger;
-        const path = triggerId
+        // The per-trigger route takes no filters. A filter passed with a trigger id used to be
+        // dropped without a word, so `--outcome failed` listed succeeded rows and exited 0 (OSK-16474).
+        // A filter now always reaches the filtered history, narrowed to that trigger; that history
+        // covers triggers you own, which is said out loud rather than discovered.
+        const filtered = [flags.outcome, flags.source, flags.scope, flags.agent].some((value) => value !== undefined);
+        if (triggerId && filtered && !flags.json) {
+            this.warn('Filters read the fire history of triggers you own: a trigger shared with you but owned by someone else lists no rows here (omit the filters to read its own audit).');
+        }
+        const path = triggerId && !filtered
             ? (0, triggers_1.withQuery)(`/api/triggers/${encodeURIComponent(triggerId)}/audit`, {
                 limit: flags.limit,
                 before: flags.before,
@@ -47,6 +55,7 @@ class TriggersAudit extends base_command_1.BaseCommand {
                 agentId: flags.agent,
                 outcome: flags.outcome,
                 source: flags.source,
+                ...(triggerId ? { triggerId } : {}),
             });
         let response;
         try {

package/dist/commands/triggers/create.js

@@ -99,7 +99,7 @@ class TriggersCreate extends base_command_1.BaseCommand {
             description: 'ISO-8601 datetime after which a schedule stops firing',
         }),
         'stale-catchup-ms': core_1.Flags.integer({
-            description: 'How late a missed occurrence may still run, in ms (a positive number). After downtime, an occurrence more than this overdue is skipped rather than run late, and the schedule moves to its next future slot. Unset: 3600000 (1 hour).',
+            description: 'How late a missed occurrence may still run, in ms, from 1000 (1 second) to 31622400000 (366 days). After downtime, an occurrence more than this overdue is skipped rather than run late, and the schedule moves to its next future slot. Unset: 3600000 (1 hour).',
         }),
         at: core_1.Flags.string({
             description: 'ISO-8601 datetime for once triggers. --next-run-at is also accepted with --kind once for backwards compatibility.',

package/dist/commands/triggers/fire-detail.d.ts

@@ -1,4 +1,11 @@
 import { BaseCommand } from '../../base-command';
+/**
+ * The text rendering of one fire. `reason` is the typed code and `errorMessage` the sentence a
+ * person reads; the view printed whichever came first, so a fire that resolved nothing showed
+ * only `Reason: no_waiting_step` and its sentence ("No step in this Space was waiting on that
+ * event key.") was JSON-only (OSK-16474). Both are shown now when both exist.
+ */
+export declare function describeFireAuditDetail(detail: Record<string, unknown>, fireId: string, triggerId: string): string[];
 export default class TriggersFireDetail extends BaseCommand {
     static description: string;
     static examples: string[];