@skrr-ai/cli 0.1.104 → 0.1.105

package/dist/commands/proposals/show.js

@@ -1,24 +1,51 @@
 "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 proposals_1 = require("../../lib/proposals");
 class ProposalsShow extends base_command_1.BaseCommand {
     static description = 'Show one viewer-scoped proposal and its frozen revision';
+    static examples = [
+        '<%= config.bin %> proposals show request <proposal-id>',
+        '<%= config.bin %> proposals show request:<proposal-id>',
+    ];
+    // The kind is validated in `resolveProposalRef`, not by oclif, so the KIND:ID that `proposals list`
+    // prints can be pasted whole (OSK-15371).
     static args = {
-        kind: core_1.Args.string({ required: true, options: [...data_provider_1.PROPOSAL_KINDS], ignoreStdin: true }),
-        id: core_1.Args.string({ required: true, ignoreStdin: true }),
+        kind: core_1.Args.string({
+            required: true,
+            description: 'Proposal kind, or the KIND:ID that `proposals list` prints',
+            ignoreStdin: true,
+        }),
+        id: core_1.Args.string({ required: false, description: 'Proposal id', ignoreStdin: true }),
     };
     static flags = { json: core_1.Flags.boolean() };
     async run() {
         this.requireAuth();
         const { args } = await this.parse(ProposalsShow);
+        const ref = (0, proposals_1.resolveProposalRef)(args.kind, args.id, this.config?.bin);
+        if ('error' in ref) {
+            this.failWithCliError({
+                message: ref.error,
+                code: 'PROPOSAL_REF_INVALID',
+                exit: 2,
+                retryable: false,
+            });
+        }
         let result;
         try {
-            result = await proposals_1.proposalApi.show(args.kind, args.id);
+            result = await proposals_1.proposalApi.show(ref.kind, ref.id);
         }
         catch (error) {
+            if (error?.status === 404) {
+                this.failWithCliError({
+                    message: (0, proposals_1.describeProposalNotFound)(ref.kind, ref.id, this.config?.bin),
+                    code: 'PROPOSAL_NOT_FOUND',
+                    status: 404,
+                    exit: 4,
+                    retryable: false,
+                });
+            }
             this.handleApiError(error);
         }
         this.log(JSON.stringify(result, null, 2));

package/dist/commands/proposals/snooze.d.ts

@@ -3,7 +3,7 @@ export default class ProposalsSnooze extends ProposalDecisionCommand {
     static description: string;
     static args: {
         kind: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
-        id: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+        id: import("@oclif/core/lib/interfaces").Arg<string | undefined, Record<string, unknown>>;
     };
     static decision: "snooze";
 }

package/dist/commands/requests/create.js

@@ -10,10 +10,15 @@ function hasRequestTarget(payload) {
 }
 class RequestsCreate extends base_command_1.BaseCommand {
     static description = 'Create a new request';
+    // Every example names a recipient: a request without one is refused, so an example that omits it
+    // is a command that cannot run (OSK-15375).
     static examples = [
-        '<%= config.bin %> requests create --title "Approve budget" --type approval',
-        '<%= config.bin %> requests create --title "Q1 review" --type task --description "Review Q1 goals"',
-        '<%= config.bin %> requests create --from-json request.json',
+        '<%= config.bin %> requests create --title "Approve budget" --type approval --target-user <user-id>',
+        '<%= config.bin %> requests create --title "Q1 review" --type task --description "Review Q1 goals" --target-agent <agent-id>',
+        `cat > request.json <<'JSON'
+{ "title": "Approve budget", "type": "approval", "targetUserId": "<user-id>" }
+JSON
+<%= config.bin %> requests create --from-json request.json`,
     ];
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output the created request as JSON' }),

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

@@ -29,15 +29,15 @@ class SpacesWorkflowsCreate extends base_command_1.BaseCommand {
         title: core_1.Flags.string({ required: true, description: 'Workflow title' }),
         description: core_1.Flags.string({ description: 'Optional description' }),
         graph: core_1.Flags.string({
-            description: 'Required exactly once with --draft/--starter: path to graph JSON ({ nodes, edges }); inspect config with spaces workflows node-types',
+            description: 'One of --graph, --draft or --starter (exactly one): path to graph JSON ({ nodes, edges }); inspect config with spaces workflows node-types',
             exactlyOne: ['graph', 'draft', 'starter'],
         }),
         draft: core_1.Flags.string({
-            description: 'Required exactly once with --graph/--starter: path to complete Canvas draft JSON ({ semantic, presentation })',
+            description: 'One of --graph, --draft or --starter (exactly one): path to complete Canvas draft JSON ({ semantic, presentation })',
             exactlyOne: ['graph', 'draft', 'starter'],
         }),
         starter: core_1.Flags.boolean({
-            description: 'Required exactly once with --graph/--draft: create a valid one-node starter graph',
+            description: 'One of --graph, --draft or --starter (exactly one): create a valid one-node starter graph',
             exactlyOne: ['graph', 'draft', 'starter'],
         }),
         json: core_1.Flags.boolean({ description: 'Output JSON' }),

package/dist/commands/tasks/complete.js

@@ -38,6 +38,7 @@ const core_1 = require("@oclif/core");
 const base_command_1 = require("../../base-command");
 const task_closure_1 = require("../../lib/task-closure");
 const api_fetch_1 = require("../../lib/api-fetch");
+const task_recurrence_outcome_1 = require("../../lib/task-recurrence-outcome");
 const outbox_1 = require("../../lib/outbox");
 const prompt_1 = require("../../lib/prompt");
 const session_task_write_1 = require("../../lib/session-task-write");
@@ -356,6 +357,9 @@ class TasksComplete extends base_command_1.BaseCommand {
         for (const sentence of (0, task_closure_1.closureSnapshotLines)(result.task?.resolution?.closure)) {
             this.log(sentence);
         }
+        const recurrenceLine = (0, task_recurrence_outcome_1.recurrenceOutcomeLine)(result.recurrence);
+        if (recurrenceLine)
+            this.log(recurrenceLine);
         for (const advisory of result.advisories || [])
             this.warn(advisory.message);
         if (url)

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

@@ -71,6 +71,8 @@ export default class TasksCreate extends BaseCommand {
         per: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         effort: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         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>;
         estimate: import("@oclif/core/lib/interfaces").OptionFlag<number | 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>;

package/dist/commands/tasks/create.js

@@ -226,13 +226,27 @@ class TasksCreate extends base_command_1.BaseCommand {
         }),
         'start-date': core_1.Flags.string({ description: 'Planned start date (ISO string)' }),
         'due-date': core_1.Flags.string({ description: 'Due date (ISO string)' }),
-        savings: core_1.Flags.string({ description: 'Expected USD savings (requires --per)' }),
+        savings: core_1.Flags.string({
+            description: 'Expected USD savings (requires --per). An estimate you write as a person is recorded as Human confirmed under your name, and an Agent edit is then refused with a 409; an Agent writes an Agent estimate. Re-sending the same values changes nothing.',
+        }),
         per: core_1.Flags.string({
             options: ['month', 'year', 'once'],
             description: 'Savings period (requires --savings)',
         }),
-        effort: core_1.Flags.string({ description: 'Expected effort in hours, nonnegative' }),
-        risk: core_1.Flags.string({ options: ['low', 'medium', 'high'], description: 'Expected risk' }),
+        effort: core_1.Flags.string({
+            description: 'Expected effort in hours, nonnegative (part of the estimate; see --savings)',
+        }),
+        risk: core_1.Flags.string({
+            options: ['low', 'medium', 'high'],
+            description: 'Expected risk (part of the estimate; see --savings)',
+        }),
+        confidence: core_1.Flags.string({
+            options: ['low', 'medium', 'high'],
+            description: 'How sure the estimate is (part of the estimate; see --savings). An earlier confidence stays on the claim until you replace it.',
+        }),
+        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.',
+        }),
         estimate: core_1.Flags.integer({
             // Names the prerequisite AND how to satisfy it. The first version of this
             // help said the scale was reachable only from the web UI, which was true

package/dist/commands/tasks/deliverable/add.d.ts

@@ -46,3 +46,5 @@ export default class TasksDeliverableAdd extends BaseCommand {
      */
     private archive;
 }
+/** The sentence for a `--status` an archived copy overrides, or null when nothing was overridden. */
+export declare function statusNotApplied(requestedStatus: string | undefined): string | null;

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

@@ -34,6 +34,7 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.defaultDeliverableIdempotencyKey = defaultDeliverableIdempotencyKey;
+exports.statusNotApplied = statusNotApplied;
 const fs = __importStar(require("node:fs"));
 const path = __importStar(require("node:path"));
 const core_1 = require("@oclif/core");
@@ -277,7 +278,15 @@ class TasksDeliverableAdd extends base_command_1.BaseCommand {
                 `Run skrr tasks deliverable list ${opts.taskId} --json to find the exact version before submitting a Result.`);
         }
         if (opts.json) {
-            this.log(JSON.stringify({ ...resourceReceipt, archived, identity: opts.identity }, null, 2));
+            // `--status` is overridden for an archived copy, and --json used to say nothing of it: the
+            // receipt's observation read `available` and the requested status appeared nowhere (OSK-15437).
+            const notes = statusNotApplied(opts.requestedStatus);
+            this.log(JSON.stringify({
+                ...resourceReceipt,
+                archived,
+                identity: opts.identity,
+                ...(notes ? { requestedStatus: opts.requestedStatus, notes: [notes] } : {}),
+            }, null, 2));
             return true;
         }
         const recorded = resourceReceipt?.link.id
@@ -286,16 +295,22 @@ class TasksDeliverableAdd extends base_command_1.BaseCommand {
         const version = resourceReceipt?.version.id ? `version ${resourceReceipt.version.id}; ` : '';
         this.log(`${recorded} — archived copy ${asset.id} ` +
             `(${version}sha256 pinned; observed available; self reported).`);
-        if (opts.requestedStatus && opts.requestedStatus !== 'available') {
-            // Not silent: the platform WATCHED these bytes land, which is a stronger
-            // and verified statement than the one that was asked for. Who vouched
-            // is unchanged — still `self_reported`, still nobody else's word.
-            this.log(`--status ${opts.requestedStatus} did not apply: an archived copy is observed available.`);
-        }
+        const notApplied = statusNotApplied(opts.requestedStatus);
+        // Not silent: the platform WATCHED these bytes land, which is a stronger
+        // and verified statement than the one that was asked for. Who vouched
+        // is unchanged — still `self_reported`, still nobody else's word.
+        if (notApplied)
+            this.log(notApplied);
         return true;
     }
 }
 exports.default = TasksDeliverableAdd;
+/** The sentence for a `--status` an archived copy overrides, or null when nothing was overridden. */
+function statusNotApplied(requestedStatus) {
+    return requestedStatus && requestedStatus !== 'available'
+        ? `--status ${requestedStatus} did not apply: an archived copy is observed available.`
+        : null;
+}
 /**
  * The local file a coordinate already names, if it names one. `location` is
  * checked first because it is where the thing IS; `identity` is the fallback

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

@@ -1,4 +1,17 @@
 import { BaseCommand } from '../../../base-command';
+/**
+ * Which listed Deliverables are an EARLIER version of one that is also listed.
+ *
+ * v1 and v2 of the same coordinate share a Resource and both read `available`, so nothing said which
+ * was current (OSK-15437). The newest observation of a Resource is its current version; a tie, or a
+ * row with no observation time, is left unmarked rather than guessed. Returns link id -> the link id
+ * of the newer version.
+ */
+export declare function earlierVersions(items: Array<{
+    linkId?: string;
+    resourceId?: string;
+    observedAt?: string;
+}>): Map<string, string>;
 export default class TasksDeliverableList extends BaseCommand {
     static description: string;
     static args: {

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

@@ -1,5 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.earlierVersions = earlierVersions;
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../../base-command");
@@ -37,6 +38,35 @@ function versionLabel(item) {
     const id = (item.resourceVersionId || '').replace(/^wrv_/, '');
     return id ? `version ${id.slice(0, 12)}` : 'version unpinned';
 }
+/**
+ * Which listed Deliverables are an EARLIER version of one that is also listed.
+ *
+ * v1 and v2 of the same coordinate share a Resource and both read `available`, so nothing said which
+ * was current (OSK-15437). The newest observation of a Resource is its current version; a tie, or a
+ * row with no observation time, is left unmarked rather than guessed. Returns link id -> the link id
+ * of the newer version.
+ */
+function earlierVersions(items) {
+    const newest = new Map();
+    const times = (item) => Date.parse(item.observedAt ?? '');
+    for (const item of items) {
+        const at = times(item);
+        if (!item.linkId || !item.resourceId || Number.isNaN(at))
+            continue;
+        const current = newest.get(item.resourceId);
+        if (!current || at > current.at)
+            newest.set(item.resourceId, { linkId: item.linkId, at });
+    }
+    const marks = new Map();
+    for (const item of items) {
+        const at = times(item);
+        const latest = item.resourceId ? newest.get(item.resourceId) : undefined;
+        if (item.linkId && latest && !Number.isNaN(at) && at < latest.at) {
+            marks.set(item.linkId, latest.linkId);
+        }
+    }
+    return marks;
+}
 class TasksDeliverableList extends base_command_1.BaseCommand {
     static description = 'List Deliverables with exact version, location, and evidence state';
     static args = {
@@ -61,11 +91,14 @@ class TasksDeliverableList extends base_command_1.BaseCommand {
             return this.log(JSON.stringify(result, null, 2));
         if (!result.data.length)
             return this.log('No Deliverables recorded.');
+        const earlier = earlierVersions(result.data);
         for (const item of result.data) {
+            const newer = item.linkId ? earlier.get(item.linkId) : undefined;
             this.log(`${item.linkId} · ${item.kind}${item.format ? `/${item.format}` : ''} · ` +
                 `${item.title || item.identity || '(untitled)'}\n` +
                 `  ${item.location || 'no locator'} · ${fidelityLabel(item.versionFidelity)} · ` +
-                `${versionLabel(item)} · ${item.status} · ${item.evidenceLevel}`);
+                `${versionLabel(item)} · ${item.status} · ${item.evidenceLevel}` +
+                (newer ? `\n  an earlier version of this Deliverable; the newer one is ${newer}` : ''));
         }
     }
 }

package/dist/commands/tasks/economics-compare.js

@@ -3,6 +3,7 @@ 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 task_economics_1 = require("../../lib/task-economics");
 class TasksEconomicsCompare extends base_command_1.BaseCommand {
     static description = 'Show a Task economics estimate beside retained platform readings (no measurement binding is created)';
     static examples = [
@@ -28,6 +29,11 @@ class TasksEconomicsCompare extends base_command_1.BaseCommand {
     async run() {
         this.requireAuth();
         const { args, flags } = await this.parse(TasksEconomicsCompare);
+        // Named by flag here; the server names the same faults by parameter (OSK-15368).
+        if (flags.fire.length > 2)
+            this.error('Give --fire once or twice: at most two readings can be compared.', { exit: 2 });
+        if (new Set(flags.fire).size !== flags.fire.length)
+            this.error('The two --fire ids must differ.', { exit: 2 });
         const taskRef = await this.resolveEntityRef('task', args.id);
         let response;
         try {
@@ -45,7 +51,7 @@ class TasksEconomicsCompare extends base_command_1.BaseCommand {
             return;
         }
         const readings = response.platformReadings;
-        this.log(`Estimate (revision ${response.economicsRevision}): ${JSON.stringify(response.economics)}`);
+        this.log(`Estimate (revision ${response.economicsRevision}): ${(0, task_economics_1.describeEstimate)(response.economics)}`);
         if (readings.status === 'unavailable') {
             this.log(`Platform readings unavailable: ${readings.reason}`);
         }

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

@@ -1,4 +1,14 @@
+import type { JudgedExpectation } from '@skrr-ai/data-provider';
 import { BaseCommand } from '../../base-command';
+export declare function evidenceGap(expectation: JudgedExpectation): string;
+/**
+ * A judgement made about a Result that is no longer the current one (OSK-15431).
+ *
+ * An assessment stands for the specification, not for one Result, so submitting a superseding Result
+ * does not retire it, and the count after the v2 submit included judgements of v1 with nothing to say
+ * so. Only true when the assessment named a Result and the Task has a different current one.
+ */
+export declare function judgedOnEarlierResult(expectation: JudgedExpectation, currentResultId: string | null | undefined): boolean;
 export default class TasksExpectations extends BaseCommand {
     static description: string;
     static examples: string[];

package/dist/commands/tasks/expectations.js

@@ -1,5 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.evidenceGap = evidenceGap;
+exports.judgedOnEarlierResult = judgedOnEarlierResult;
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const data_provider_2 = require("@skrr-ai/data-provider");
@@ -59,6 +61,11 @@ function evidenceGap(expectation) {
         const { provided, required } = expectation.evidenceShortfall;
         return `  (vouched at ${provided}, asked for ${required})`;
     }
+    // `owed` reads "not produced yet"; when a deliverable of the right kind is recorded but does not
+    // qualify, say why instead of implying nothing was produced (OSK-15457).
+    if (expectation.state === 'owed' && expectation.nearMiss) {
+        return `  (one was recorded, but ${expectation.nearMiss})`;
+    }
     if (!PINNED_STATES.has(expectation.state))
         return '';
     if (!expectation.evidenceLevel || !expectation.requiredEvidence)
@@ -85,7 +92,19 @@ function assessorSuffix(expectation) {
         return '';
     return ` (${(0, task_closure_1.describeAssessor)(expectation.assessedByKind)})`;
 }
-function line(expectation) {
+/**
+ * A judgement made about a Result that is no longer the current one (OSK-15431).
+ *
+ * An assessment stands for the specification, not for one Result, so submitting a superseding Result
+ * does not retire it, and the count after the v2 submit included judgements of v1 with nothing to say
+ * so. Only true when the assessment named a Result and the Task has a different current one.
+ */
+function judgedOnEarlierResult(expectation, currentResultId) {
+    return Boolean(expectation.assessedResultId &&
+        currentResultId &&
+        expectation.assessedResultId !== currentResultId);
+}
+function line(expectation, currentResultId) {
     const label = (STATE_LABEL[expectation.state] || expectation.state) + assessorSuffix(expectation);
     const from = expectation.source === 'space' ? '  [space default]' : '';
     /*
@@ -98,7 +117,10 @@ function line(expectation) {
      * thing is owed. The fallback is for a row that predates the resolved field.
      */
     const sentence = expectation.sentence || expectation.text;
-    return `  ${expectation.id.padEnd(18)} ${label.padEnd(22)} ${sentence}${evidenceGap(expectation)}${from}`;
+    const earlier = judgedOnEarlierResult(expectation, currentResultId)
+        ? '  (judged on an earlier Result)'
+        : '';
+    return `  ${expectation.id.padEnd(18)} ${label.padEnd(22)} ${sentence}${evidenceGap(expectation)}${earlier}${from}`;
 }
 class TasksExpectations extends base_command_1.BaseCommand {
     static description = 'Show what must be true before the Task can close and the evidence-backed assessment state';
@@ -142,7 +164,13 @@ class TasksExpectations extends base_command_1.BaseCommand {
         }
         this.log('Expectations:');
         for (const expectation of acceptance.expectations) {
-            this.log(line(expectation));
+            this.log(line(expectation, acceptance.currentResultId));
+        }
+        const carried = acceptance.expectations.filter((e) => judgedOnEarlierResult(e, acceptance.currentResultId)).length;
+        if (carried > 0) {
+            this.log(`${carried} judgement${carried === 1 ? ' was' : 's were'} made on an earlier Result. ` +
+                `${carried === 1 ? 'It still counts' : 'They still count'}: an assessment stands for the ` +
+                'Task specification, not for one Result. Re-assess if the current Result changes the answer.');
         }
         const gating = acceptance.expectations.filter((e) => e.gating);
         // `met: true` is the GATE's answer — a waiver unblocks closure exactly as a

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

@@ -35,6 +35,15 @@ export declare function emptyTasksMessage(flags: ListFlags, bin?: string): strin
  * naming the stage sat unread in the same payload.
  */
 export declare function nextActionText(t: Record<string, unknown>): string;
+/**
+ * The NEXT column's width. Bounded, because a sentence in an unbounded column wraps the table into
+ * unreadability on a narrow terminal, but widened when a row says what a Task was closed over: the
+ * two things it names come last in the sentence, and a column cut before them hid exactly the
+ * disclosure the row exists to make (OSK-15429).
+ */
+export declare function nextColumnWidth(rows: Array<{
+    next: string;
+}>): number;
 /**
  * The REF column's value, aliased to `ref` under `--json` because `identifier`
  * is the one name an agent scanning 43 keys does not guess (OSK-10692).

package/dist/commands/tasks/list.js

@@ -3,6 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.agentSessionOpenDefault = agentSessionOpenDefault;
 exports.emptyTasksMessage = emptyTasksMessage;
 exports.nextActionText = nextActionText;
+exports.nextColumnWidth = nextColumnWidth;
 exports.taskRef = taskRef;
 exports.buildListParams = buildListParams;
 exports.statusFilterWarnings = statusFilterWarnings;
@@ -13,6 +14,7 @@ const web_url_1 = require("../../lib/web-url");
 const format_1 = require("../../lib/format");
 const list_output_1 = require("../../lib/list-output");
 const tasks_1 = require("../../lib/tasks");
+const task_economics_1 = require("../../lib/task-economics");
 /**
  * Names of `tasks list` flags that narrow the result set. Used to distinguish
  * "no tasks anywhere" from "no tasks matching your filters" — the first case
@@ -174,9 +176,38 @@ function statusTypeList(flags) {
  */
 function nextActionText(t) {
     const actionability = t.actionability;
-    return actionability
-        ? `${String(actionability.nextActor ?? '-')}: ${String(actionability.reason || actionability.reasonCode || actionability.state || '-')}`
-        : String(t.executionMode ?? '-');
+    if (!actionability)
+        return String(t.executionMode ?? '-');
+    // A finished Task has no next action, which is all this column said about one closed over unmet
+    // Expectations, a missing required Review, or past a Review by explicit force ("none: Task workflow
+    // type is completed"). The list row already carries the persisted closure snapshot, so say what it
+    // was closed over with the shared renderer (OSK-15429). Only when something is owed or overridden:
+    // a clean close reads as before.
+    if (actionability.state === 'terminal') {
+        const closed = closedOverText(t);
+        if (closed)
+            return `${String(actionability.nextActor ?? '-')}: closed. ${closed}`;
+    }
+    return `${String(actionability.nextActor ?? '-')}: ${String(actionability.reason || actionability.reasonCode || actionability.state || '-')}`;
+}
+/** What a finished Task's persisted closure snapshot says was overridden or is still owed, or null. */
+function closedOverText(t) {
+    const closure = t.resolution?.closure;
+    const outstanding = (0, data_provider_1.describeClosureOutstanding)(closure);
+    const parts = [
+        ...(0, data_provider_1.describeClosureOverrides)(closure),
+        ...(outstanding ? [`Outstanding at close: ${outstanding}.`] : []),
+    ];
+    return parts.length > 0 ? parts.join(' ') : null;
+}
+/**
+ * The NEXT column's width. Bounded, because a sentence in an unbounded column wraps the table into
+ * unreadability on a narrow terminal, but widened when a row says what a Task was closed over: the
+ * two things it names come last in the sentence, and a column cut before them hid exactly the
+ * disclosure the row exists to make (OSK-15429).
+ */
+function nextColumnWidth(rows) {
+    return rows.some((row) => row.next.includes(': closed. ')) ? 140 : 46;
 }
 /**
  * The REF column's value, aliased to `ref` under `--json` because `identifier`
@@ -647,6 +678,7 @@ 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),
                 basis: economics?.basis && Object.prototype.hasOwnProperty.call(basis, economics.basis)
                     ? basis[economics.basis]
                     : 'Unknown',
@@ -662,6 +694,7 @@ class TasksList extends base_command_1.BaseCommand {
                     { key: 'savings', header: 'SAVINGS' },
                     { key: 'effort', header: 'EFFORT' },
                     { key: 'risk', header: 'RISK' },
+                    { key: 'score', header: 'SCORE' },
                     { key: 'basis', header: 'BASIS' },
                 ]
                 : []),
@@ -669,14 +702,16 @@ class TasksList extends base_command_1.BaseCommand {
             { key: 'assignees', header: 'ASSIGNEES' },
             // Bounded: the reason is a sentence, and an unbounded column wraps the
             // table into unreadability on a narrow terminal.
-            { key: 'next', header: 'NEXT', maxWidth: 46 },
+            { key: 'next', header: 'NEXT', maxWidth: nextColumnWidth(rows) },
             // Only meaningful on a search — the server computes it from the same
             // relevance rank that orders the rows, and a plain listing has none.
             ...(flags.q ? [{ key: 'match', header: 'MATCH' }] : []),
             { key: 'id', header: 'ID' },
         ], (line) => this.log(line));
         if (flags.rank)
-            this.log('\nRanked only within this fetched page; estimates are claims, not measured savings.');
+            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.');
         if (r?.next_cursor) {
             this.log('');
             this.log(`Next page: --cursor ${r.next_cursor}`);

package/dist/commands/tasks/report.js

@@ -171,7 +171,7 @@ class TasksReport extends base_command_1.BaseCommand {
             this.log(JSON.stringify({ ...result, url }, null, 2));
             return;
         }
-        this.log(`Submitted Result v${result.result.version} on task ${task.id}.${url ? ` ${url}` : ''}`);
+        this.log(`Submitted Result v${result.result.version} (${result.result.id}) on task ${task.id}.${url ? ` ${url}` : ''}`);
     }
 }
 exports.default = TasksReport;

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

@@ -1,4 +1,11 @@
+import type { TaskResultRecord } from '@skrr-ai/data-provider';
 import { BaseCommand } from '../../../base-command';
+/**
+ * What a reviewer needs to accept ONE exact Result (OSK-15430). `result show` printed the summary and
+ * the conclusion, and kept the Result id (the value `tasks review --result-id` requires), the linked
+ * deliverables, the evidence, the remaining work and the Result it supersedes for `--json`.
+ */
+export declare function describeResultDetail(current: TaskResultRecord, all?: TaskResultRecord[]): string[];
 export default class TasksResultShow extends BaseCommand {
     static description: string;
     static args: {

package/dist/commands/tasks/result/show.js

@@ -1,10 +1,39 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.describeResultDetail = describeResultDetail;
 const core_1 = require("@oclif/core");
 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 task_legacy_1 = require("../../../lib/task-legacy");
+/**
+ * What a reviewer needs to accept ONE exact Result (OSK-15430). `result show` printed the summary and
+ * the conclusion, and kept the Result id (the value `tasks review --result-id` requires), the linked
+ * deliverables, the evidence, the remaining work and the Result it supersedes for `--json`.
+ */
+function describeResultDetail(current, all = []) {
+    const lines = [];
+    if (current.supersedesResultId) {
+        const previous = all.find((row) => row.id === current.supersedesResultId);
+        lines.push(`Supersedes: ${current.supersedesResultId}${previous ? ` (v${previous.version})` : ''}`);
+    }
+    const links = current.resourceLinkIds ?? [];
+    lines.push(`Deliverables: ${links.length > 0 ? links.join(', ') : 'none linked'}`);
+    const evidence = current.evidenceRefs ?? [];
+    lines.push(`Evidence: ${evidence.length > 0
+        ? evidence.map((ref) => `${ref.kind}:${ref.id}`).join(', ')
+        : 'none attached'}`);
+    const remaining = current.remainingWork ?? [];
+    if (remaining.length > 0) {
+        lines.push('Remaining work:');
+        for (const item of remaining)
+            lines.push(`  - ${item.summary}`);
+    }
+    else {
+        lines.push('Remaining work: none declared');
+    }
+    return lines;
+}
 class TasksResultShow extends base_command_1.BaseCommand {
     static description = 'Show the current Task Result and its exact Review';
     static args = {
@@ -32,9 +61,11 @@ class TasksResultShow extends base_command_1.BaseCommand {
         if (!current)
             return this.log(await (0, task_legacy_1.emptyCanonicalView)(task.id, 'Result'));
         const review = result.reviews.find((row) => row.resultId === current.id);
-        this.log(`Result v${current.version}\n${current.summary}`);
+        this.log(`Result v${current.version}  ${current.id}\n${current.summary}`);
         if (current.conclusion)
             this.log(`Conclusion (${current.conclusion.type}): ${JSON.stringify(current.conclusion.value)}`);
+        for (const line of describeResultDetail(current, result.results))
+            this.log(line);
         // Only "awaiting" while a review can still be recorded. A closed Task
         // refuses one outright (`TASK_REOPEN_REQUIRED`), so the old wording named a
         // state nothing could ever leave — and a reader could not tell it from a

package/dist/commands/tasks/result/submit.js

@@ -211,7 +211,7 @@ class TasksResultSubmit extends base_command_1.BaseCommand {
         // managed instructions told the agent to read the status the call returns,
         // which the call did not carry. Submitting a Result is not closing a Task;
         // the response now reports where it stands and this prints exactly that.
-        this.log(`Submitted Result v${result.result.version} for ${task.id}.`);
+        this.log(`Submitted Result v${result.result.version} (${result.result.id}) for ${task.id}.`);
         for (const line of (0, task_closure_1.closureReport)(result.closure, { closing: false }))
             this.log(line);
         if (url)

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

@@ -50,6 +50,12 @@ export declare function completionLines(projection: {
 } | null | undefined, { read }?: {
     read?: boolean;
 }): string[];
+/**
+ * The review-policy line, or null for the default. Only a non-default policy is printed: `none` is
+ * every Task that never asked, and printing it would add a line to all of them. Without this the
+ * policy was readable in --json alone (OSK-15434).
+ */
+export declare function reviewPolicyLine(policy: unknown): string | null;
 export declare function labelsLine(labelIds?: string[], nameById?: Map<string, string>): string;
 /**
  * The `Executor:` value — who does this task, and whether anyone said so.