@skrr-ai/cli 0.1.94 → 0.1.96

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. */
@@ -632,14 +642,6 @@ export type EvidenceHealthView = {
  * is noise in a summary someone reads to find what is wrong.
  */
 export declare function describeEvidenceHealth(health?: EvidenceHealthView | null, criteriaCount?: number): string | undefined;
-/**
- * `stalled` is a verdict, not a reason. Printed bare it reads like `unknown`
- * with a harsher word, and the operator's next move is to go and read run
- * history to find what the reconciler had already decided. Returns undefined
- * for every other health value, including `unknown` — "nobody has judged" has
- * no diagnosis to name.
- */
-export declare function describeStalledHealth(commitment: CommitmentView): string | undefined;
 /** Render an interval back as a compact human string. */
 export declare function describeIntervalMs(ms: number | undefined): string;
 /**