@skrr-ai/cli 0.1.95 → 0.1.96

package/dist/commands/tasks/list.js

@@ -272,7 +272,9 @@ function buildListParams(flags, assigneeUserId) {
         // `--compact` asks the server for its own card projection (no activityLog,
         // narrative or ledgers) so the bytes are never sent, then the CLI trims the
         // rest to the navigation set. Only the JSON path reads it.
-        ...(flag('compact') && !flag('fields') ? { view: 'summary' } : {}),
+        ...(flag('compact') && !flag('fields') && !flag('rank')
+            ? { view: 'summary' }
+            : {}),
         ...(flag('view') ? { taskViewId: flag('view') } : {}),
         limit: flag('limit'),
         cursor: flag('cursor'),
@@ -374,6 +376,10 @@ class TasksList extends base_command_1.BaseCommand {
         'needs-me': core_1.Flags.boolean({
             description: 'Filter to Human-actionable work addressed to you',
         }),
+        rank: core_1.Flags.string({
+            options: ['economics'],
+            description: 'Rank only this fetched page by annualized USD / max(effort hours,0.5) × risk discount; unknown last, never an action policy',
+        }),
         sort: core_1.Flags.string({
             description: 'Order by a shared saved-view field',
             options: [...data_provider_1.VIEW_SORT_KEYS],
@@ -527,6 +533,19 @@ class TasksList extends base_command_1.BaseCommand {
         catch (err) {
             this.handleApiError(err);
         }
+        if (flags.rank && calendarRange)
+            this.error('--rank applies to fetched task pages, not calendar occurrences.', { exit: 2 });
+        if (flags.rank &&
+            response &&
+            typeof response === 'object' &&
+            'data' in response &&
+            Array.isArray(response.data)) {
+            response = {
+                ...response,
+                data: (0, data_provider_1.rankEconomicItems)(response.data),
+                rank_applied: { key: 'economics', scope: 'fetched_page' },
+            };
+        }
         const list = response;
         const statusWarnings = calendarRange
             ? []
@@ -604,6 +623,12 @@ class TasksList extends base_command_1.BaseCommand {
             return;
         }
         const rows = items.map((t) => {
+            const economics = t.economics;
+            const basis = {
+                agent_estimate: 'Agent estimate',
+                human_confirmed: 'Human confirmed',
+                measured: 'Measured',
+            };
             const agents = Array.isArray(t.assigneeAgentIds) ? t.assigneeAgentIds.length : 0;
             const users = Array.isArray(t.assigneeUserIds) ? t.assigneeUserIds.length : 0;
             const assignees = agents + users > 0 ? `${agents}a/${users}u` : '-';
@@ -617,6 +642,14 @@ class TasksList extends base_command_1.BaseCommand {
                 assignees,
                 next: nextActionText(t),
                 match: String(t._matchField ?? '-'),
+                savings: economics?.savings
+                    ? `${economics.savings.currency} ${economics.savings.amount}/${economics.savings.per}`
+                    : 'Unknown',
+                effort: economics?.effortHours !== undefined ? `${economics.effortHours}h` : 'Unknown',
+                risk: economics?.risk ?? 'Unknown',
+                basis: economics?.basis && Object.prototype.hasOwnProperty.call(basis, economics.basis)
+                    ? basis[economics.basis]
+                    : 'Unknown',
             };
         });
         (0, format_1.renderTable)(rows, [
@@ -624,6 +657,14 @@ class TasksList extends base_command_1.BaseCommand {
             { key: 'title', header: 'TITLE', maxWidth: 50 },
             { key: 'status', header: 'STATUS' },
             { key: 'priority', header: 'PRIORITY' },
+            ...(flags.rank
+                ? [
+                    { key: 'savings', header: 'SAVINGS' },
+                    { key: 'effort', header: 'EFFORT' },
+                    { key: 'risk', header: 'RISK' },
+                    { key: 'basis', header: 'BASIS' },
+                ]
+                : []),
             { key: 'estimate', header: 'ESTIMATE' },
             { key: 'assignees', header: 'ASSIGNEES' },
             // Bounded: the reason is a sentence, and an unbounded column wraps the
@@ -634,6 +675,8 @@ class TasksList extends base_command_1.BaseCommand {
             ...(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.');
         if (r?.next_cursor) {
             this.log('');
             this.log(`Next page: --cursor ${r.next_cursor}`);

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

@@ -63,6 +63,11 @@ 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 `Recurrence:` line for a series whose next occurrence was never armed.
+ * Silent otherwise: a healthy series has nothing to say here.
+ */
+export declare function recurrenceArmFailureLines(t: Record<string, unknown>, bin?: string): string[];
 /**
  * The failure rows of `tasks show`, when there are any. (S3 dogfood finding)
  *

package/dist/commands/tasks/show.js

@@ -5,6 +5,7 @@ exports.blockerDetailLines = blockerDetailLines;
 exports.completionLines = completionLines;
 exports.labelsLine = labelsLine;
 exports.taskExecutorLabel = taskExecutorLabel;
+exports.recurrenceArmFailureLines = recurrenceArmFailureLines;
 exports.taskFailureLines = taskFailureLines;
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
@@ -137,6 +138,20 @@ function taskExecutorLabel(t, bin = 'skrr') {
             return '-';
     }
 }
+/**
+ * The `Recurrence:` line for a series whose next occurrence was never armed.
+ * Silent otherwise: a healthy series has nothing to say here.
+ */
+function recurrenceArmFailureLines(t, bin = 'skrr') {
+    const recurrence = t.recurrence;
+    const code = recurrence?.armFailure?.code;
+    if (typeof code !== 'string' || !code)
+        return [];
+    return [
+        `Recurrence:  ${(0, data_provider_1.describeRecurrenceArmFailure)(code)} (${code})`,
+        `             Reschedule with \`${bin} tasks schedule\` to start the series again.`,
+    ];
+}
 /**
  * The failure rows of `tasks show`, when there are any. (S3 dogfood finding)
  *
@@ -321,6 +336,8 @@ class TasksShow extends base_command_1.BaseCommand {
         // question and this is the answer to it.
         for (const line of taskFailureLines(t, this.config.bin))
             this.log(line);
+        for (const line of recurrenceArmFailureLines(t, this.config.bin))
+            this.log(line);
         /*
          * Closing is the actor's call — a Task may be `done` with work still
          * outstanding, and `resolution.closure` is where the server recorded what

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

@@ -17,6 +17,10 @@ export default class TasksUpdate extends BaseCommand {
         priority: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'start-date': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'due-date': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        savings: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        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>;
         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

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 const core_1 = require("@oclif/core");
+const task_economics_1 = require("../../lib/task-economics");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
 const web_url_1 = require("../../lib/web-url");
@@ -62,6 +63,13 @@ class TasksUpdate extends base_command_1.BaseCommand {
         }),
         'start-date': core_1.Flags.string({ description: 'New planned start date (ISO string)' }),
         'due-date': core_1.Flags.string({ description: 'New due date (ISO string)' }),
+        savings: core_1.Flags.string({ description: 'Expected USD savings (requires --per)' }),
+        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' }),
         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 ' +
@@ -356,6 +364,7 @@ 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),
             // 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`.

package/dist/commands/tasks/workflow-findings.js

@@ -88,8 +88,8 @@ class TasksWorkflowFindings extends base_command_1.BaseCommand {
         }
         this.log(`Reported ${response.reported} finding(s) on ${response.nodeKey}.`);
         this.log(`Open blocking: ${response.openBlocking} of ${response.total} total.`);
-        this.log(response.converged
-            ? 'No blocking findings remain open — this review has converged.'
+        this.log(response.openBlocking === 0
+            ? 'No blocking findings are currently open. Panel availability is evaluated by the Workflow engine.'
             : 'Each open blocking finding must be fixed, or rebutted with evidence, before this converges.');
     }
 }

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

@@ -21,6 +21,8 @@ export default class TriggersCreate extends BaseCommand {
         'effect-contract': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'effect-write-mode': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'effect-options': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'workflow-event-key': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'workflow-outcome': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         dispatch: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         reference: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         references: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;

package/dist/commands/triggers/create.js

@@ -61,6 +61,14 @@ class TriggersCreate extends base_command_1.BaseCommand {
             dependsOn: ['effect-contract'],
         }),
         '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',
+        }),
+        'workflow-outcome': core_1.Flags.string({
+            description: 'How the resolved step settles (default success)',
+            options: ['success', 'failure'],
+            dependsOn: ['workflow-event-key'],
+        }),
         dispatch: core_1.Flags.string({
             description: 'auto_start fires the work immediately; manual parks it as todo',
             options: ['auto_start', 'manual'],
@@ -218,6 +226,13 @@ class TriggersCreate extends base_command_1.BaseCommand {
                 builtinAction,
                 conversationId: flags.conversation,
                 effectPolicy,
+                // Built only when a key is named, for the same reason as effectPolicy.
+                workflowSignal: flags['workflow-event-key']
+                    ? {
+                        eventKey: flags['workflow-event-key'],
+                        ...(flags['workflow-outcome'] ? { outcome: flags['workflow-outcome'] } : {}),
+                    }
+                    : undefined,
                 dispatch: flags.dispatch,
                 references: (0, triggers_1.parseExecutionReferences)(flags.reference, flags.references),
                 workingHours: (0, triggers_1.parseJsonObject)(flags['working-hours'], '--working-hours'),

package/dist/commands/workspaces/grants.d.ts

@@ -0,0 +1,12 @@
+import { BaseCommand } from '../../base-command';
+export default class WorkspacesGrants extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static args: {
+        workspaceId: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+    };
+    static flags: {
+        json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+    };
+    run(): Promise<void>;
+}

package/dist/commands/workspaces/grants.js

@@ -0,0 +1,31 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const base_command_1 = require("../../base-command");
+const workspaces_1 = require("../../lib/workspaces");
+const stewardship_output_1 = require("../../lib/stewardship-output");
+class WorkspacesGrants extends base_command_1.BaseCommand {
+    static description = 'Read workspace Grant metadata and impact (person owner/admin only)';
+    static examples = ['<%= config.bin %> workspaces grants <workspace-id> --json'];
+    static args = {
+        workspaceId: core_1.Args.string({ required: true, ignoreStdin: true, description: 'Workspace ID' }),
+    };
+    static flags = {
+        json: core_1.Flags.boolean({ description: 'Output metadata and coverage as JSON' }),
+    };
+    async run() {
+        this.requireAuth();
+        const { args, flags } = await this.parse(WorkspacesGrants);
+        try {
+            const report = await (0, workspaces_1.listWorkspaceGrants)(args.workspaceId);
+            if (flags.json)
+                this.log(JSON.stringify(report, null, 2));
+            else
+                (0, stewardship_output_1.printWorkspaceGrants)(report, (line) => this.log(line));
+        }
+        catch (error) {
+            this.handleApiError(error);
+        }
+    }
+}
+exports.default = WorkspacesGrants;

package/dist/commands/workspaces/stewardship.d.ts

@@ -0,0 +1,13 @@
+import { BaseCommand } from '../../base-command';
+export default class WorkspacesStewardship extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static args: {
+        workspaceId: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+        userId: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+    };
+    static flags: {
+        json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+    };
+    run(): Promise<void>;
+}

package/dist/commands/workspaces/stewardship.js

@@ -0,0 +1,34 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const base_command_1 = require("../../base-command");
+const workspaces_1 = require("../../lib/workspaces");
+const stewardship_output_1 = require("../../lib/stewardship-output");
+class WorkspacesStewardship extends base_command_1.BaseCommand {
+    static description = 'Read a member’s workspace stewardship metadata (person owner/admin only)';
+    static examples = [
+        '<%= config.bin %> workspaces stewardship <workspace-id> <user-id> --json',
+    ];
+    static args = {
+        workspaceId: core_1.Args.string({ required: true, ignoreStdin: true, description: 'Workspace ID' }),
+        userId: core_1.Args.string({ required: true, ignoreStdin: true, description: 'Member user ID' }),
+    };
+    static flags = {
+        json: core_1.Flags.boolean({ description: 'Output metadata and coverage as JSON' }),
+    };
+    async run() {
+        this.requireAuth();
+        const { args, flags } = await this.parse(WorkspacesStewardship);
+        try {
+            const report = await (0, workspaces_1.getWorkspaceStewardship)(args.workspaceId, args.userId);
+            if (flags.json)
+                this.log(JSON.stringify(report, null, 2));
+            else
+                (0, stewardship_output_1.printWorkspaceStewardship)(report, (line) => this.log(line));
+        }
+        catch (error) {
+            this.handleApiError(error);
+        }
+    }
+}
+exports.default = WorkspacesStewardship;

package/dist/lib/agent-config.d.ts

@@ -1,15 +1,3 @@
-export declare const COMMITMENT_RESPONSIBILITY_LEVELS: readonly ["reasoning_only", "full_ownership"];
-export declare const REASONING_TOOL_ESCAPES: readonly ["bash", "browser"];
-/** Warning printed by the three retired Commitment permission flags (OSK-11975). */
-export declare const COMMITMENT_PERMISSION_FLAG_RETIRED = "This setting no longer has any effect: Commitment runs use the agent's own permissions.";
-/** The three retired flags themselves: accepted at parse, warned on, never sent. */
-export declare const RETIRED_COMMITMENT_PERMISSION_FLAGS: readonly ["reasoning-tool-escape", "commitment-responsibility", "commitment-responsibility-ceiling"];
-/**
- * The deprecation warning ONE retired flag prints. oclif prints a
- * `deprecated.message` verbatim, so a shared message printed the same line
- * once per flag with nothing saying which flag each one was about (OSK-12068).
- */
-export declare function retiredCommitmentPermissionFlagMessage(flag: (typeof RETIRED_COMMITMENT_PERMISSION_FLAGS)[number]): string;
 /**
  * The fields the server ignored and why, from a write's `deprecatedFields`
  * notice, one line each. Empty when the response carries none.

package/dist/lib/agent-config.js

@@ -1,31 +1,12 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.RELEASE_INSTALL_CHANNELS = exports.RETIRED_COMMITMENT_PERMISSION_FLAGS = exports.COMMITMENT_PERMISSION_FLAG_RETIRED = exports.REASONING_TOOL_ESCAPES = exports.COMMITMENT_RESPONSIBILITY_LEVELS = void 0;
-exports.retiredCommitmentPermissionFlagMessage = retiredCommitmentPermissionFlagMessage;
+exports.RELEASE_INSTALL_CHANNELS = void 0;
 exports.deprecatedFieldsWarnings = deprecatedFieldsWarnings;
 exports.deprecatedFieldNames = deprecatedFieldNames;
 exports.parseWorkingHoursFlag = parseWorkingHoursFlag;
 exports.mergeAgentConfigFlags = mergeAgentConfigFlags;
 const prompt_1 = require("./prompt");
 const agent_visual_refs_1 = require("./agent-visual-refs");
-exports.COMMITMENT_RESPONSIBILITY_LEVELS = ['reasoning_only', 'full_ownership'];
-exports.REASONING_TOOL_ESCAPES = ['bash', 'browser'];
-/** Warning printed by the three retired Commitment permission flags (OSK-11975). */
-exports.COMMITMENT_PERMISSION_FLAG_RETIRED = "This setting no longer has any effect: Commitment runs use the agent's own permissions.";
-/** The three retired flags themselves: accepted at parse, warned on, never sent. */
-exports.RETIRED_COMMITMENT_PERMISSION_FLAGS = [
-    'reasoning-tool-escape',
-    'commitment-responsibility',
-    'commitment-responsibility-ceiling',
-];
-/**
- * The deprecation warning ONE retired flag prints. oclif prints a
- * `deprecated.message` verbatim, so a shared message printed the same line
- * once per flag with nothing saying which flag each one was about (OSK-12068).
- */
-function retiredCommitmentPermissionFlagMessage(flag) {
-    return `--${flag} is retired and was not sent. ${exports.COMMITMENT_PERMISSION_FLAG_RETIRED}`;
-}
 /**
  * The fields the server ignored and why, from a write's `deprecatedFields`
  * notice, one line each. Empty when the response carries none.
@@ -141,13 +122,6 @@ function mergeAgentConfigFlags(body, flags) {
     if (flags['todo-working-hours'] !== undefined) {
         parsed.todoWorkingHours = parseWorkingHoursFlag(String(flags['todo-working-hours']));
     }
-    // `--reasoning-tool-escape`, `--commitment-responsibility` and
-    // `--commitment-responsibility-ceiling` are retired (OSK-11975) and are
-    // deliberately NOT forwarded. The server still accepts and echoes the stored
-    // fields, so sending them wrote a value nothing reads and handed it back as
-    // if it were live — and "a ceiling a commitment may not exceed" is a safety
-    // claim a person then believes (OSK-12068). The flags stay parseable so old
-    // scripts do not break; oclif prints COMMITMENT_PERMISSION_FLAG_RETIRED.
     if (flags['tool-resources'] !== undefined) {
         parsed.tool_resources = JSON.parse(String(flags['tool-resources']));
     }

package/dist/lib/agent-mailbox.d.ts

@@ -0,0 +1,35 @@
+/** Human operator projections; never expose mail content or provider/storage identifiers. */
+export interface AgentMailboxMetadata {
+    id: string;
+    agentId: string;
+    workspaceId: string;
+    handle: string;
+    address: string;
+    status: string;
+    sending: string;
+    sendingPausedReason?: string | null;
+    version: number;
+}
+export interface AgentMailMessageMetadata {
+    id: string;
+    messageId: string;
+    receivedAt: string;
+    classification: string;
+    status: string;
+    expiresAt?: string | null;
+}
+export interface AgentMailboxStatus {
+    mailbox: AgentMailboxMetadata | null;
+    receivingRuntime?: {
+        status: 'running' | 'starting' | 'unavailable';
+    };
+}
+export interface AgentMailboxMessages {
+    data: AgentMailMessageMetadata[];
+    nextCursor: string | null;
+}
+export declare function getAgentMailboxMetadata(agentId: string): Promise<AgentMailboxStatus>;
+export declare function getAgentMailboxMessageMetadata(agentId: string, options?: {
+    limit?: number;
+    cursor?: string;
+}): Promise<AgentMailboxMessages>;

package/dist/lib/agent-mailbox.js

@@ -0,0 +1,55 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.getAgentMailboxMetadata = getAgentMailboxMetadata;
+exports.getAgentMailboxMessageMetadata = getAgentMailboxMessageMetadata;
+const api_fetch_1 = require("./api-fetch");
+function withMailboxQuery(path, query) {
+    return query.size ? `${path}?${query.toString()}` : path;
+}
+async function getAgentMailboxMetadata(agentId) {
+    const result = await (0, api_fetch_1.apiFetch)(`/api/agents/${encodeURIComponent(agentId)}/mailbox/metadata`);
+    const mailbox = result.mailbox;
+    // Also project at the output boundary. A future server field must not
+    // accidentally turn the CLI's JSON into a content or credential export.
+    return {
+        ...(['running', 'starting', 'unavailable'].includes(result.receivingRuntime?.status || '')
+            ? { receivingRuntime: { status: result.receivingRuntime.status } }
+            : {}),
+        mailbox: mailbox
+            ? {
+                id: mailbox.id,
+                agentId: mailbox.agentId,
+                workspaceId: mailbox.workspaceId,
+                handle: mailbox.handle,
+                address: mailbox.address,
+                status: mailbox.status,
+                sending: mailbox.sending,
+                sendingPausedReason: mailbox.sendingPausedReason,
+                version: mailbox.version,
+            }
+            : null,
+    };
+}
+async function getAgentMailboxMessageMetadata(agentId, options = {}) {
+    if (options.limit !== undefined &&
+        (!Number.isInteger(options.limit) || options.limit < 1 || options.limit > 100)) {
+        throw new Error('Mailbox message limit must be an integer from 1 to 100.');
+    }
+    const query = new URLSearchParams();
+    if (options.limit !== undefined)
+        query.set('limit', String(options.limit));
+    if (options.cursor)
+        query.set('cursor', options.cursor);
+    const result = await (0, api_fetch_1.apiFetch)(withMailboxQuery(`/api/agents/${encodeURIComponent(agentId)}/mailbox/messages/metadata`, query));
+    return {
+        data: result.data.map((message) => ({
+            id: message.id,
+            messageId: message.messageId,
+            receivedAt: message.receivedAt,
+            classification: message.classification,
+            status: message.status,
+            expiresAt: message.expiresAt,
+        })),
+        nextCursor: result.nextCursor,
+    };
+}

package/dist/lib/commitment-endpoints.d.ts

@@ -1,6 +1,7 @@
 import { withQuery } from './triggers';
 export declare const ENDPOINT_CAPABILITIES: readonly ["push", "reply", "structuredAsk"];
-export declare const COMMITMENT_ENDPOINT_DESTINATIONS: readonly ["slack_channel", "slack_owner_dm", "app_owner_dm"];
+/** Derived from the one destination registry; never a second list. */
+export declare const COMMITMENT_ENDPOINT_DESTINATIONS: ("email" | "pinned" | "app_owner_dm" | "slack_owner_dm" | "slack_channel" | "main" | "discord_owner_dm" | "user_notify")[];
 export type EndpointCapability = (typeof ENDPOINT_CAPABILITIES)[number];
 export interface CommitmentEndpoint {
     id: string;

package/dist/lib/commitment-endpoints.js

@@ -21,11 +21,8 @@ const data_provider_1 = require("@skrr-ai/data-provider");
 const format_1 = require("./format");
 const triggers_1 = require("./triggers");
 exports.ENDPOINT_CAPABILITIES = ['push', 'reply', 'structuredAsk'];
-exports.COMMITMENT_ENDPOINT_DESTINATIONS = [
-    'slack_channel',
-    'slack_owner_dm',
-    'app_owner_dm',
-];
+/** Derived from the one destination registry; never a second list. */
+exports.COMMITMENT_ENDPOINT_DESTINATIONS = data_provider_1.COMMITMENT_REGISTRABLE_REPORT_DESTINATIONS;
 const MOUNT = '/api/commitment-endpoints';
 const base = (id) => `${MOUNT}/${encodeURIComponent(id)}`;
 exports.commitmentEndpointApi = {

package/dist/lib/commitment-product.d.ts

@@ -1,9 +1,11 @@
-import type { CommitmentDeliveryIntent, CommitmentExecutionReceipt, CommitmentExecutionReceiptQuarterlyMetricsResponse } from '@skrr-ai/data-provider';
+import type { CommitmentDeliveryIntent, TaskReviewPolicy, CommitmentExecutionReceipt, CommitmentExecutionReceiptQuarterlyMetricsResponse } from '@skrr-ai/data-provider';
 export declare const COMMITMENT_DELIVERY_CHOICES: readonly ["report-only", "local-patch", "draft-pr", "push-branch", "direct-branch"];
 export declare function deliveryLabel(mode?: string): string;
+export declare function normalizeTaskReviewPolicy(value: unknown): TaskReviewPolicy;
 export declare function deliveryInput(mode: string, options?: {
     repo?: string;
     base?: string;
+    taskReviewPolicy?: TaskReviewPolicy;
 }): CommitmentDeliveryIntent;
 export declare function renderExecutionReceipt(receipt: CommitmentExecutionReceipt, log: (line: string) => void, verbose?: boolean): void;
 /**

package/dist/lib/commitment-product.js

@@ -2,9 +2,11 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.COMMITMENT_DELIVERY_CHOICES = void 0;
 exports.deliveryLabel = deliveryLabel;
+exports.normalizeTaskReviewPolicy = normalizeTaskReviewPolicy;
 exports.deliveryInput = deliveryInput;
 exports.renderExecutionReceipt = renderExecutionReceipt;
 exports.renderExecutionReceiptQuarterlyMetrics = renderExecutionReceiptQuarterlyMetrics;
+const data_provider_1 = require("@skrr-ai/data-provider");
 exports.COMMITMENT_DELIVERY_CHOICES = [
     'report-only',
     'local-patch',
@@ -21,6 +23,11 @@ function deliveryLabel(mode) {
         direct_branch: 'Direct branch',
     }[mode || ''] || 'Not declared');
 }
+function normalizeTaskReviewPolicy(value) {
+    if (typeof value !== 'string' || !data_provider_1.TASK_REVIEW_POLICIES.includes(value))
+        throw new Error('Task review policy must be none or required.');
+    return value;
+}
 function deliveryInput(mode, options = {}) {
     const normalized = mode.replace(/-/g, '_');
     if (!exports.COMMITMENT_DELIVERY_CHOICES.some((choice) => choice.replace(/-/g, '_') === normalized)) {
@@ -31,6 +38,9 @@ function deliveryInput(mode, options = {}) {
         mode: normalized,
         ...(options.repo ? { repo: options.repo } : {}),
         ...(options.base ? { baseBranch: options.base } : {}),
+        ...(options.taskReviewPolicy !== undefined
+            ? { reviewPolicy: normalizeTaskReviewPolicy(options.taskReviewPolicy) }
+            : {}),
     };
 }
 function renderExecutionReceipt(receipt, log, verbose = false) {

package/dist/lib/commitment-terms.js

@@ -94,7 +94,7 @@ function renderCommitmentTerms(commitment, log) {
     const reports = commitment.reports ?? [];
     log(`Reports:   ${reports.length
         ? reports
-            .map((r) => `${r.destination}${r.destinationTarget ? `:${r.destinationTarget}` : ''} on ${r.on ?? 'material_change'}`)
+            .map((r) => `${r.destination}${r.destinationTarget ? `:${r.destinationTarget}` : ''} on ${r.on ?? 'material_change'}${r.format ? ` format ${r.format}` : ''}${r.review ? ` · reviewed by ${r.review}` : ''}`)
             .join(', ')
         : 'default surface only'}`);
 }

package/dist/lib/commitments.d.ts

@@ -1,3 +1,4 @@
+import { type CommitmentObservationsQuery, type CommitmentObservationsResponse } from '@skrr-ai/data-provider';
 /**
  * commitments.ts — request-based client for the Commitments subsystem
  * (`/api/commitments/*`).
@@ -79,6 +80,8 @@ export type CommitmentPreflightReport = {
         label?: string;
         available: boolean;
         reason?: string;
+        /** Why a destination cannot deliver, in words (e.g. a stored-only destination). */
+        detail?: string;
         capabilities: string[];
     }>;
     attention: {
@@ -201,6 +204,12 @@ export declare const commitmentPackApi: {
     preflight: (id: string, nextRunCount?: number) => Promise<CommitmentPackPreflightResponse>;
     activate: (id: string) => Promise<CommitmentPackLifecycleResponse>;
 };
+export type ObservationWindowQuery = {
+    key: string;
+    from: string;
+    to: string;
+    limit?: number;
+};
 export declare const commitmentApi: {
     /**
      * The lib owns every URL in this domain, and the parity spec depends on that:
@@ -287,6 +296,8 @@ export declare const commitmentApi: {
     decideActionProposal: (id: string, proposalId: string, decision: "approve" | "reject", snapshot?: {
         snapshotDigest?: string;
         bundleVersion?: number;
+        reason?: string;
+        reasonCode?: import("@skrr-ai/data-provider").ProposalDiscardReasonCode;
     }) => Promise<Record<string, unknown>>;
     executeActionProposal: (id: string, proposalId: string) => Promise<Record<string, unknown>>;
     preflight: (id: string, nextRunCount?: number) => Promise<CommitmentPreflightReport>;
@@ -434,13 +445,12 @@ export declare const commitmentApi: {
     }) => Promise<CommitmentListPage>;
     createEvaluation: (id: string, body: Record<string, unknown>) => Promise<Record<string, unknown>>;
     retryEvaluationLearning: (id: string, scheduleId: string) => Promise<Record<string, unknown>>;
-    observations: (id: string, query?: {
-        limit?: number;
-        declaredOnly?: boolean;
-        changedOnly?: boolean;
-        since?: string;
-        cursor?: string;
-    }) => Promise<CommitmentListPage>;
+    observations: (id: string, query?: CommitmentObservationsQuery) => Promise<CommitmentObservationsResponse>;
+    observationSeries: (id: string, query: ObservationWindowQuery) => Promise<Record<string, unknown>>;
+    observationCompare: (id: string, query: ObservationWindowQuery & {
+        baselineFrom: string;
+        baselineTo: string;
+    }) => Promise<Record<string, unknown>>;
     /** One-tap feedback on a check — the signal Reflection retunes cadence from. */
     checkFeedback: (id: string, checkId: string, body: Record<string, unknown>) => Promise<Record<string, unknown>>;
     /** Bounded, named corrections — never a free-form contract edit. */