@skrr-ai/cli 0.1.108 → 0.1.110

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

@@ -105,6 +105,8 @@ export default class TasksList extends BaseCommand {
         subscribed: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         'include-snoozed': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         'external-key': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'external-key-prefix': 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>;
         tag: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'label-id': import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         q: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;

package/dist/commands/tasks/list.js

@@ -48,6 +48,7 @@ const FILTER_FLAGS = new Set([
     'subscribed',
     'include-snoozed',
     'external-key',
+    'external-key-prefix',
     'blocked-by',
     'blocks',
     'due-before',
@@ -286,6 +287,13 @@ function buildListParams(flags, assigneeUserId) {
         ...(flag('external-key')
             ? { externalKey: flag('external-key') }
             : {}),
+        ...(flag('external-key-prefix')
+            ? { externalKeyPrefix: flag('external-key-prefix') }
+            : {}),
+        // A presentation of each row, not a filter: it is deliberately not in FILTER_FLAGS.
+        ...(flag('description-format') === 'markdown'
+            ? { descriptionFormat: 'markdown' }
+            : {}),
         ...(flag('blocked-by') ? { blockedBy: flag('blocked-by') } : {}),
         ...(flag('blocks') ? { blocks: flag('blocks') } : {}),
         ...(flag('due-before') ? { dueBefore: flag('due-before') } : {}),
@@ -469,6 +477,13 @@ class TasksList extends base_command_1.BaseCommand {
             description: 'Include tasks you have personally snoozed (hidden by default)',
         }),
         'external-key': core_1.Flags.string({ description: 'Filter by external key' }),
+        'external-key-prefix': core_1.Flags.string({
+            description: 'Filter to tasks whose external key starts with this literal prefix, e.g. every task of one tree written under --external-key-prefix. Combine with --space; see also `tasks create-tree --verify`.',
+        }),
+        'description-format': core_1.Flags.string({
+            description: 'JSON/text: how to read each description. html (default) is what is stored; markdown returns it as it was written, so a description you wrote compares equal to the one you read back.',
+            options: ['html', 'markdown'],
+        }),
         tag: core_1.Flags.string({
             description: 'Filter to tasks carrying EVERY listed tag (repeatable, AND-intersection, ' +
                 'case-insensitive)',
@@ -573,7 +588,9 @@ class TasksList extends base_command_1.BaseCommand {
             Array.isArray(response.data)) {
             response = {
                 ...response,
-                data: (0, data_provider_1.rankEconomicItems)(response.data),
+                // Each row carries the `economicsRank` it was sorted by, so `--json` shows the same SCORE
+                // as the table, computed once in the shared ranking (OSK-16476).
+                data: (0, data_provider_1.annotateEconomicRank)(response.data),
                 rank_applied: { key: 'economics', scope: 'fetched_page' },
             };
         }
@@ -678,7 +695,9 @@ class TasksList extends base_command_1.BaseCommand {
                     : 'Unknown',
                 effort: economics?.effortHours !== undefined ? `${economics.effortHours}h` : 'Unknown',
                 risk: economics?.risk ?? 'Unknown',
-                score: (0, task_economics_1.economicsScoreText)(economics),
+                score: t.economicsRank
+                    ? (0, task_economics_1.scoreText)(t.economicsRank.score)
+                    : (0, task_economics_1.economicsScoreText)(economics),
                 basis: economics?.basis && Object.prototype.hasOwnProperty.call(basis, economics.basis)
                     ? basis[economics.basis]
                     : 'Unknown',
@@ -711,7 +730,8 @@ class TasksList extends base_command_1.BaseCommand {
         if (flags.rank)
             this.log('\nRanked only within this fetched page; estimates are claims, not measured savings.\n' +
                 'SCORE = annualized USD savings / max(effort hours, 0.5) x risk factor (low 1, medium 0.6, high 0.25); ' +
-                '— means savings, effort or risk is missing, so the row is not scored and sorts last.');
+                '— means savings, effort or risk is missing (or outside the plausible range), so the row is not scored ' +
+                'and sorts after every scored row: larger stated savings first, then rows with no estimate.');
         if (r?.next_cursor) {
             this.log('');
             this.log(`Next page: --cursor ${r.next_cursor}`);
@@ -719,6 +739,8 @@ class TasksList extends base_command_1.BaseCommand {
         const note = (0, list_output_1.limitNote)(flags.limit, items.length, Boolean(r?.has_more));
         for (const line of (0, list_output_1.warningLines)(note.warning ? [note.warning] : []))
             this.log(line);
+        for (const line of (0, list_output_1.textModeProjectionNotes)(flags))
+            this.log(line);
     }
 }
 exports.default = TasksList;

package/dist/commands/tasks/rearm-recurrence.d.ts

@@ -0,0 +1,22 @@
+import { BaseCommand } from '../../base-command';
+/**
+ * Continue a recurring series whose next occurrence was never armed (OSK-15338).
+ *
+ * The failure (`recurrence.armFailure`) is recorded on the FINISHED occurrence, and `tasks schedule`
+ * refuses a finished Task, so this is the command every surface names for the way back. It arms the
+ * occurrence the series is owed and clears the failure; `--cron` / `--timezone` correct the series
+ * first, which is the only cure for an `invalid_pattern` failure.
+ */
+export default class TasksRearmRecurrence extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static args: {
+        id: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+    };
+    static flags: {
+        json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        cron: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        timezone: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+    };
+    run(): Promise<void>;
+}

package/dist/commands/tasks/rearm-recurrence.js

@@ -0,0 +1,87 @@
+"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 web_url_1 = require("../../lib/web-url");
+/**
+ * Continue a recurring series whose next occurrence was never armed (OSK-15338).
+ *
+ * The failure (`recurrence.armFailure`) is recorded on the FINISHED occurrence, and `tasks schedule`
+ * refuses a finished Task, so this is the command every surface names for the way back. It arms the
+ * occurrence the series is owed and clears the failure; `--cron` / `--timezone` correct the series
+ * first, which is the only cure for an `invalid_pattern` failure.
+ */
+class TasksRearmRecurrence extends base_command_1.BaseCommand {
+    static description = 'Re-arm a recurring series from the finished occurrence whose next occurrence was never scheduled';
+    static examples = [
+        '<%= config.bin %> tasks rearm-recurrence OSK-42',
+        '<%= config.bin %> tasks rearm-recurrence OSK-42 --cron "0 9 * * 1-5" --timezone America/New_York',
+    ];
+    static args = {
+        id: core_1.Args.string({
+            description: 'The finished occurrence (Task ID or identifier) that carries the arm failure',
+            required: true,
+            ignoreStdin: true,
+        }),
+    };
+    static flags = {
+        json: core_1.Flags.boolean({ description: 'Output as JSON' }),
+        cron: core_1.Flags.string({
+            description: 'Correct the series pattern first: five cron fields "minute hour day-of-month month day-of-week", e.g. "0 9 * * 1-5"',
+        }),
+        timezone: core_1.Flags.string({
+            description: 'Correct the series timezone first (IANA zone, e.g. America/New_York)',
+        }),
+    };
+    async run() {
+        this.requireAuth();
+        const { args, flags } = await this.parse(TasksRearmRecurrence);
+        const taskRef = await this.resolveEntityRef('task', args.id);
+        let result;
+        try {
+            result = await data_provider_1.dataService.rearmTaskRecurrence({
+                taskId: taskRef,
+                pattern: flags.cron,
+                timezone: flags.timezone,
+            });
+        }
+        catch (err) {
+            this.handleApiError(err);
+        }
+        if (flags.json) {
+            this.log(JSON.stringify({
+                ...result,
+                task: (0, web_url_1.withTaskUrl)(this.cliConfig.baseURL, result.task),
+            }, null, 2));
+            return;
+        }
+        const finished = result.task;
+        const from = finished.identifier || finished.id || taskRef;
+        const next = result.successor.identifier || result.successor.id;
+        const when = result.successor.scheduledFor ? ` for ${result.successor.scheduledFor}` : '';
+        // The server's sentence IS the answer: it is derived with the JSON fields, so the two can
+        // never disagree (round 3 of OSK-15321 found them disagreeing). The fallback below is only
+        // for an older server that sends no `message`.
+        if (typeof result.message === 'string' && result.message) {
+            this.log(result.message);
+            return;
+        }
+        // Never call a successor with no schedule "armed": it will not fire (#4262 verification).
+        let line;
+        if (!result.successor.scheduledFor) {
+            line = `The next occurrence ${next} of the series from ${from} exists but has no schedule, so nothing will fire it on its own; start it by hand or give it a time with \`${this.config.bin} tasks schedule ${next} --at <ISO time>\`.`;
+        }
+        else if (result.rescheduled) {
+            line = `The next occurrence ${next} existed but had no schedule; it is now scheduled${when}. The failure marker is cleared.`;
+        }
+        else if (result.alreadyArmed) {
+            line = `The series from ${from} was already armed: next occurrence ${next}${when}. The failure marker is cleared.`;
+        }
+        else {
+            line = `Re-armed the series from ${from}: next occurrence ${next} is scheduled${when}.`;
+        }
+        this.log(line);
+    }
+}
+exports.default = TasksRearmRecurrence;

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

@@ -69,6 +69,14 @@ export declare function labelsLine(labelIds?: string[], nameById?: Map<string, s
  * reassigning the task changes only the implied kind.
  */
 export declare function taskExecutorLabel(t: Record<string, unknown>, bin?: string): string;
+/**
+ * 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.
+ */
+export declare function resolutionLines(t: Record<string, unknown>, names?: Map<string, string>): string[];
 /**
  * The `Recurrence:` line for a series whose next occurrence was never armed.
  * Silent otherwise: a healthy series has nothing to say here.
@@ -112,6 +120,7 @@ export default class TasksShow extends BaseCommand {
         json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         task: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'no-enrich': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        'description-format': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
     };
     run(): Promise<void>;
     /**

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");
@@ -21,6 +22,7 @@ const label_ref_1 = require("../../lib/label-ref");
 const principal_display_1 = require("../../lib/principal-display");
 const task_economics_line_1 = require("../../lib/task-economics-line");
 const format_1 = require("../../lib/format");
+const task_recurrence_outcome_1 = require("../../lib/task-recurrence-outcome");
 /**
  * Where to look next — as COMMANDS, not as facts about the task.
  *
@@ -150,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.
@@ -159,9 +198,17 @@ function recurrenceArmFailureLines(t, bin = 'skrr') {
     const code = recurrence?.armFailure?.code;
     if (typeof code !== 'string' || !code)
         return [];
+    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})`,
-        `             Reschedule with \`${bin} tasks schedule\` to start the series again.`,
+        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>"\`.`,
     ];
 }
 /**
@@ -246,6 +293,10 @@ class TasksShow extends base_command_1.BaseCommand {
         'no-enrich': core_1.Flags.boolean({
             description: 'Skip server-side enrichment (assignee names, parent titles)',
         }),
+        'description-format': core_1.Flags.string({
+            description: 'How to read the description. html (default) is what is stored; markdown returns it as it was written.',
+            options: ['html', 'markdown'],
+        }),
     };
     async run() {
         this.requireAuth();
@@ -304,7 +355,12 @@ class TasksShow extends base_command_1.BaseCommand {
                 task = bound;
             }
             else {
-                task = await data_provider_1.dataService.getTask(taskId, { enrich: !flags['no-enrich'] });
+                task = await data_provider_1.dataService.getTask(taskId, {
+                    enrich: !flags['no-enrich'],
+                    ...(flags['description-format'] === 'markdown'
+                        ? { descriptionFormat: 'markdown' }
+                        : {}),
+                });
             }
         }
         catch (err) {
@@ -331,13 +387,28 @@ class TasksShow extends base_command_1.BaseCommand {
             completionRead = false;
         }
         if (flags.json) {
-            this.log(JSON.stringify({ ...t, url, completion }, null, 2));
+            this.printJsonResult({ ...t, url, completion });
             return;
         }
         for (const line of completionLines(completion, { read: completionRead }))
             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 ?? '-'}`);
@@ -354,25 +425,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.
          *
@@ -505,18 +563,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/timeline.js

@@ -5,6 +5,7 @@ const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
 const format_1 = require("../../lib/format");
+const task_economics_1 = require("../../lib/task-economics");
 const task_legacy_1 = require("../../lib/task-legacy");
 /**
  * `<action> (<k=v ...>)`, or just the preview when there is nothing to add.
@@ -13,6 +14,9 @@ const task_legacy_1 = require("../../lib/task-legacy");
  * renders `details` with, so the two surfaces describe one entry the same way.
  */
 function withDetails(item) {
+    // An estimate change reads as from-what-to-what, the same line `tasks activity` prints.
+    if (typeof item.type === 'string' && item.type.startsWith(data_provider_1.TASK_ECONOMICS_ACTIVITY_TYPE_PREFIX))
+        return (0, task_economics_1.describeEconomicsChange)(item.data);
     const preview = String(item.preview ?? '-');
     const details = (0, format_1.formatCell)(item.data ?? item.details, '');
     return details ? `${preview} (${details})` : preview;

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

@@ -23,6 +23,7 @@ export default class TasksUpdate extends BaseCommand {
         risk: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         confidence: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         rationale: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'retract-estimate': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         estimate: import("@oclif/core/lib/interfaces").OptionFlag<number | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         /**
          * Declared, hidden, and refused — on purpose. (S4 dogfood finding)

package/dist/commands/tasks/update.js

@@ -84,6 +84,10 @@ class TasksUpdate extends base_command_1.BaseCommand {
         rationale: core_1.Flags.string({
             description: 'Why the estimate is what it is, up to 500 characters (part of the estimate). An earlier rationale stays on the claim, under its new basis, until you replace it.',
         }),
+        'retract-estimate': core_1.Flags.boolean({
+            description: "Remove the Task's economic estimate (savings, effort, risk, confidence and rationale). Recorded in `tasks activity` as task.economics.retracted under your name; an Agent cannot retract an estimate a person confirmed (409).",
+            exclusive: ['savings', 'per', 'effort', 'risk', 'confidence', 'rationale'],
+        }),
         estimate: core_1.Flags.integer({
             description: 'New estimate value from the Space estimate scale. Requires a scale on the Space (or an ' +
                 'ancestor it inherits from); with none set, the server drops the value. Set the scale ' +
@@ -410,7 +414,8 @@ class TasksUpdate extends base_command_1.BaseCommand {
             startDate: flags['start-date'],
             dueDate: flags['due-date'],
             estimate: flags.estimate,
-            economics: (0, task_economics_1.economicsFromFlags)(flags, body.economics),
+            // `null` is the retraction the server records as its own Activity event (OSK-15370).
+            economics: flags['retract-estimate'] ? null : (0, task_economics_1.economicsFromFlags)(flags, body.economics),
             // Never sent: `taskUpdateMutationGuard` refuses any payload carrying
             // `status` before the request is built. It is merged in only so the guard
             // sees it and answers with the sentence that names `tasks move`.
@@ -538,7 +543,17 @@ class TasksUpdate extends base_command_1.BaseCommand {
         }
         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 (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/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

@@ -7,7 +7,7 @@ const builtinActions_1 = require("../../lib/builtinActions");
 const triggers_1 = require("../../lib/triggers");
 const web_url_1 = require("../../lib/web-url");
 class TriggersCreate extends base_command_1.BaseCommand {
-    static description = 'Create a universal trigger';
+    static description = 'Create a universal trigger. A schedule trigger skips an occurrence that is overdue by more than its catch-up window (--stale-catchup-ms)';
     static examples = [
         '<%= config.bin %> triggers create --scope agent --ref agentId=<agent-id> --kind schedule --mode send_message --cron "0 9 * * *" --prompt "Prepare the daily brief"',
         '<%= config.bin %> triggers create --scope agent --ref agentId=<agent-id> --kind once --mode send_message --at 2026-06-01T15:30:00Z --prompt "Send the launch recap"',
@@ -65,7 +65,7 @@ class TriggersCreate extends base_command_1.BaseCommand {
         }),
         'effect-options': core_1.Flags.string({ description: 'Effect policy options JSON object' }),
         'workflow-event-key': core_1.Flags.string({
-            description: 'Exact event key the parked workflow step waits on (required for mode=workflow_signal). Resolves steps in the scope Space; the trigger owner must hold Space edit access at every fire',
+            description: 'Exact event key the parked workflow step waits on (required for mode=workflow_signal). Resolves steps in the scope Space; the trigger owner must hold Space edit access at every fire. A person resolves one wait by hand with `skrr workflows signal`',
         }),
         'workflow-outcome': core_1.Flags.string({
             description: 'How the resolved step settles (default success)',
@@ -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.',
@@ -290,6 +290,8 @@ class TriggersCreate extends base_command_1.BaseCommand {
                 dryRun: flags['dry-run'],
             });
             (0, triggers_1.mergeObjectField)(payload, 'execution', execution);
+            const mergedExecution = payload.execution;
+            (0, triggers_1.assertModeRequiredFlags)(mergedExecution?.mode, flags, mergedExecution);
             (0, triggers_1.mergeTriggerKindConfig)(payload, flags.kind ?? payload.kind, {
                 schedule,
                 once,
@@ -304,6 +306,10 @@ class TriggersCreate extends base_command_1.BaseCommand {
             this.error(err.message, { exit: 1 });
         }
         this.validatePayload(payload);
+        // A body assembled from flags: the server's field paths are read back as those flags. A
+        // body the person wrote (--from-json) keeps the paths, which are then what they typed.
+        if (!flags['from-json'])
+            this.requestFieldFlags = triggers_1.TRIGGER_CREATE_FIELD_FLAGS;
         let trigger;
         try {
             trigger = await triggers_1.triggerApi.create(payload);

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[];

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

@@ -1,8 +1,36 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.describeFireAuditDetail = describeFireAuditDetail;
 const core_1 = require("@oclif/core");
 const base_command_1 = require("../../base-command");
 const triggers_1 = require("../../lib/triggers");
+/**
+ * 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.
+ */
+function describeFireAuditDetail(detail, fireId, triggerId) {
+    const text = (value) => (typeof value === 'string' && value.trim() ? value.trim() : '');
+    const lines = [
+        `Fire:         ${String(detail._id ?? fireId)}`,
+        `Trigger:      ${String(detail.triggerId ?? triggerId)}`,
+        `Fired at:     ${String(detail.firedAt ?? '-')}`,
+        `Outcome:      ${String(detail.outcome ?? '-')}`,
+        `Source:       ${String(detail.source ?? '-')}`,
+    ];
+    const reason = text(detail.reason);
+    const message = text(detail.errorMessage);
+    if (reason)
+        lines.push(`Reason:       ${reason}`);
+    if (message && message !== reason)
+        lines.push(`${reason ? 'Detail:' : 'Reason:'}       ${message}`);
+    if (detail.durationMs !== undefined)
+        lines.push(`Duration:     ${String(detail.durationMs)}ms`);
+    if (detail.conversationId)
+        lines.push(`Conversation: ${String(detail.conversationId)}`);
+    return lines;
+}
 class TriggersFireDetail extends base_command_1.BaseCommand {
     static description = 'Show full detail for one trigger fire record';
     static examples = [
@@ -34,18 +62,8 @@ class TriggersFireDetail extends base_command_1.BaseCommand {
             this.log(JSON.stringify(detail, null, 2));
             return;
         }
-        this.log(`Fire:         ${detail._id ?? args.fireId}`);
-        this.log(`Trigger:      ${detail.triggerId ?? args.id}`);
-        this.log(`Fired at:     ${detail.firedAt ?? '-'}`);
-        this.log(`Outcome:      ${detail.outcome ?? '-'}`);
-        this.log(`Source:       ${detail.source ?? '-'}`);
-        if (detail.reason || detail.errorMessage) {
-            this.log(`Reason:       ${detail.reason ?? detail.errorMessage}`);
-        }
-        if (detail.durationMs !== undefined)
-            this.log(`Duration:     ${detail.durationMs}ms`);
-        if (detail.conversationId)
-            this.log(`Conversation: ${detail.conversationId}`);
+        for (const line of describeFireAuditDetail(detail, args.fireId, args.id))
+            this.log(line);
     }
 }
 exports.default = TriggersFireDetail;