@skrr-ai/cli 0.1.95 → 0.1.97

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. */