@skrr-ai/cli 0.1.106 → 0.1.108

package/dist/commands/agents/operating-plan.js

@@ -44,7 +44,7 @@ class AgentOperatingPlan extends base_command_1.BaseCommand {
                     this.log(`  Evaluation: ${w.status} — ${w.reason}${w.nextAt ? `; next ${w.nextAt}` : ''}`);
                 this.log(`  Next intervention: ${r.lifecycle !== 'active' ? 'inactive; requires activation' : r.nextIntervention?.nextAt || r.nextIntervention?.status || 'not recorded'}`);
                 if (r.blocker)
-                    this.log(`  Blocker: ${r.blocker.code} — ${r.blocker.remediation || ''}`);
+                    this.log(`  Blocker: ${r.blocker.code} — ${r.blocker.remediation || r.blocker.detail || ''}`);
             }
             for (const line of (0, operating_plan_loops_1.planningLoopLines)(plan.planning.loops, plan.planning.todos.length))
                 this.log(line);

package/dist/commands/commitments/doctor.js

@@ -16,7 +16,8 @@ function doctorVerdict(input) {
         const lifecycle = /^status_([a-z_]+)$/.exec(String(blocker.code));
         return {
             verdict: lifecycle ? lifecycle[1] : blocker.code === 'deleted' ? 'deleted' : 'blocked',
-            next: String(blocker.remediation || 'Inspect the Commitment'),
+            // A typed blocker carries no remedy, only what happened (OSK-16230).
+            next: String(blocker.remediation || blocker.detail || 'Inspect the Commitment'),
         };
     }
     if (operatingState.status === 'waiting_approval') {

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

@@ -1,3 +1,4 @@
+import type { ProposalView } from '@skrr-ai/data-provider';
 import { BaseCommand } from '../../base-command';
 /**
  * What an empty page means. With a filter AND a next cursor it does NOT mean "none": the inbox reads a
@@ -6,6 +7,12 @@ import { BaseCommand } from '../../base-command';
  * Without a cursor the inbox is exhausted and "none" is true.
  */
 export declare function emptyPageLines(narrowed: boolean, nextCursor: string | null | undefined): string[];
+/**
+ * What the row is ABOUT: the Commitment and the Agent behind it, in the order the web inbox shows them. 51 distinct titles covered 747 decidable
+ * initiatives, so the title alone could not tell two rows apart (OSK-15378). A dash is "not named" (an id the
+ * server could not read for this viewer is in `--json`), never "none".
+ */
+export declare function aboutFor(row: ProposalView): string;
 export default class ProposalsList extends BaseCommand {
     static description: string;
     static flags: {

package/dist/commands/proposals/list.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.emptyPageLines = emptyPageLines;
+exports.aboutFor = aboutFor;
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
@@ -23,6 +24,14 @@ function emptyPageLines(narrowed, nextCursor) {
         ]
         : ['No proposals on this page; the cursor below continues.'];
 }
+/**
+ * What the row is ABOUT: the Commitment and the Agent behind it, in the order the web inbox shows them. 51 distinct titles covered 747 decidable
+ * initiatives, so the title alone could not tell two rows apart (OSK-15378). A dash is "not named" (an id the
+ * server could not read for this viewer is in `--json`), never "none".
+ */
+function aboutFor(row) {
+    return [row.commitmentTitle, row.agent?.name].filter(Boolean).join(' · ') || '—';
+}
 /** A snoozed proposal reads `proposed` in its disposition; the wake time is what tells them apart. */
 function noteFor(row) {
     if (row.sourceStatus === 'snoozed' && row.snoozedUntil)
@@ -77,6 +86,7 @@ class ProposalsList extends base_command_1.BaseCommand {
         (0, format_1.renderTable)(result.data.map((row) => ({
             id: row.id,
             title: row.title,
+            about: aboutFor(row),
             disposition: row.disposition,
             // Most proposals carry no estimate; "unknown" read as a failure to determine one (OSK-15377).
             basis: row.economics?.basis || '—',
@@ -85,6 +95,7 @@ class ProposalsList extends base_command_1.BaseCommand {
         })), [
             { key: 'id', header: 'PROPOSAL', verbatim: true },
             { key: 'title', header: 'TITLE' },
+            { key: 'about', header: 'ABOUT', maxWidth: 48 },
             { key: 'disposition', header: 'DISPOSITION' },
             { key: 'basis', header: 'BASIS' },
             { key: 'deadline', header: 'DEADLINE' },

package/dist/lib/commitment-product.js

@@ -58,13 +58,31 @@ function renderExecutionReceipt(receipt, log, verbose = false) {
                 log(`    ${evidence.label}: ${location}`);
         }
     }
+    // "Next" names an action only when the receipt has one. A reason that only
+    // restates what happened (a typed blocker's summary, which servers before
+    // OSK-16429 sent as the reason) is narrative, not something to do: the actor
+    // is printed without it and the narrative stays with the blocker.
+    const narrative = [receipt.summary, receipt.blocker?.summary].filter(Boolean);
+    const action = receipt.nextActor?.reason && !narrative.includes(receipt.nextActor.reason)
+        ? receipt.nextActor.reason
+        : '';
     if (receipt.nextActor)
-        log(`Next: ${receipt.nextActor.label || receipt.nextActor.kind}${receipt.nextActor.reason ? ` — ${receipt.nextActor.reason}` : ''}`);
+        log(`Next: ${receipt.nextActor.label || receipt.nextActor.kind}${action ? ` — ${action}` : ''}`);
     if (receipt.blocker) {
         log(`Blocker: ${receipt.blocker.code}`);
+        // The headline already carries a typed blocker's summary; say it again only
+        // when it differs.
+        if (receipt.blocker.summary && receipt.blocker.summary !== receipt.summary)
+            log(`  Detail: ${receipt.blocker.summary}`);
         for (const evidenceRef of receipt.blocker.evidenceRefs || [])
             log(`  Evidence: ${evidenceRef}`);
     }
+    // A source gap beside this episode, not the reason it stopped (OSK-16230).
+    if (receipt.standingBlocker) {
+        log(`Standing: ${receipt.standingBlocker.code} — ${receipt.standingBlocker.summary} (${receipt.standingBlocker.owner.label})`);
+        for (const evidenceRef of receipt.standingBlocker.evidenceRefs || [])
+            log(`  Evidence: ${evidenceRef}`);
+    }
     if (verbose)
         log(`Diagnostics:\n${JSON.stringify({ id: receipt.id, revision: receipt.revision, evidence: receipt.evidence, diagnostics: receipt.diagnostics }, null, 2)}`);
 }

package/dist/lib/commitments.d.ts

@@ -818,8 +818,16 @@ export declare function describeOperation(operation: string | null | undefined,
  * already composed sat only in `--json` and `doctor` (OSK-16212). Nothing is
  * printed while the operation is ready or working, where a fix would contradict
  * the line above it.
- */
-export declare function describeOperationFix(commitment: CommitmentView, operation: string | null | undefined): string | null;
+ *
+ * Returns one or two lines. `Fix:` only when the blocker carries an action; a
+ * typed blocker an agent returned carries only what happened, which printed
+ * under "Fix:" read as a remedy nobody wrote (OSK-16230). That narrative is
+ * labelled `Blocker:` instead, and no remedy is made up for it.
+ */
+export declare function describeOperationFix(commitment: CommitmentView, operation: string | null | undefined): Array<{
+    label: 'Fix' | 'Blocker';
+    text: string;
+}>;
 /**
  * Who stopped a Commitment, and when and why, in one line. The server stores it as `stoppedBy`;
  * `show` printed neither (OSK-15423), so a Commitment you paused did not say you had.

package/dist/lib/commitments.js

@@ -955,20 +955,32 @@ function describeOperation(operation, reason) {
  * already composed sat only in `--json` and `doctor` (OSK-16212). Nothing is
  * printed while the operation is ready or working, where a fix would contradict
  * the line above it.
+ *
+ * Returns one or two lines. `Fix:` only when the blocker carries an action; a
+ * typed blocker an agent returned carries only what happened, which printed
+ * under "Fix:" read as a remedy nobody wrote (OSK-16230). That narrative is
+ * labelled `Blocker:` instead, and no remedy is made up for it.
  */
 function describeOperationFix(commitment, operation) {
     if (operation === 'ready' || operation === 'working')
-        return null;
+        return [];
     const blocker = commitment.firstActionableBlocker;
-    const remediation = blocker && typeof blocker.remediation === 'string' ? blocker.remediation.trim() : '';
-    if (!remediation)
-        return null;
-    const who = typeof blocker?.ownerLabel === 'string' && blocker.ownerLabel
+    if (!blocker)
+        return [];
+    const remediation = typeof blocker.remediation === 'string' ? blocker.remediation.trim() : '';
+    const detail = typeof blocker.detail === 'string' ? blocker.detail.trim() : '';
+    const who = typeof blocker.ownerLabel === 'string' && blocker.ownerLabel
         ? blocker.ownerLabel
-        : typeof blocker?.owner === 'string' && blocker.owner
+        : typeof blocker.owner === 'string' && blocker.owner
             ? blocker.owner
             : '';
-    return `${remediation}${who ? ` (${who})` : ''}`;
+    const lines = [];
+    // The detail is shown only when the fix does not already say it.
+    if (detail && detail !== remediation)
+        lines.push({ label: 'Blocker', text: `${detail}${!remediation && who ? ` (${who})` : ''}` });
+    if (remediation)
+        lines.push({ label: 'Fix', text: `${remediation}${who ? ` (${who})` : ''}` });
+    return lines;
 }
 /**
  * Who stopped a Commitment, and when and why, in one line. The server stores it as `stoppedBy`;
@@ -1082,9 +1094,8 @@ scheduleIssues) {
     log(`Assessment: ${assessment?.label ?? 'Unknown'}`);
     log(`Evidence:   ${assessment?.evidence ?? 'unavailable'}`);
     log(`Operation:  ${describeOperation(assessment?.operation, assessment?.operationReason)}`);
-    const fix = describeOperationFix(commitment, assessment?.operation);
-    if (fix)
-        log(`            Fix: ${fix}`);
+    for (const line of describeOperationFix(commitment, assessment?.operation))
+        log(`            ${line.label}: ${line.text}`);
     if (commitment.state?.lastCheckedAt)
         log(`Checked:   ${commitment.state.lastCheckedAt}`);
     if (typeof commitment.state?.consecutiveUnresolvedChecks === 'number') {