@skrr-ai/cli 0.1.17 → 0.1.19

package/dist/lib/commitments.js

@@ -1,6 +1,9 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.COMMITMENT_CLI_EVALUATORS = exports.COMMITMENT_CREATE_KEYS = exports.COMMITMENT_REMEDIATION_ACTIONS = exports.COMMITMENT_FEEDBACK_KINDS = exports.commitmentApi = exports.commitmentPackApi = exports.MAX_COMMITMENT_INTERVAL_MS = exports.MIN_COMMITMENT_INTERVAL_MS = exports.COMMITMENT_EXECUTION_ISOLATION_MODES = exports.COMMITMENT_AUTONOMY_MODES = exports.COMMITMENT_STATUSES = exports.COMMITMENT_CADENCE_MAX_TIMES = exports.COMMITMENT_CADENCE_TIME_PATTERN = exports.COMMITMENT_CADENCE_MODES = exports.COMMITMENT_TEMPLATE_IDS = exports.COMMITMENT_WATCHED_SOURCES = void 0;
+exports.COMMITMENT_CLI_EVALUATORS = exports.COMMITMENT_CREATE_KEYS = exports.COMMITMENT_REMEDIATION_ACTIONS = exports.COMMITMENT_FEEDBACK_KINDS = exports.commitmentApi = exports.commitmentPackApi = exports.MAX_COMMITMENT_INTERVAL_MS = exports.MIN_COMMITMENT_INTERVAL_MS = exports.COMMITMENT_EXECUTION_ISOLATION_MODES = exports.COMMITMENT_AUTHORABLE_AUTONOMY_MODES = exports.COMMITMENT_AUTONOMY_MODES = exports.COMMITMENT_STATUSES = exports.COMMITMENT_AUTHORABLE_CADENCE_MODES = exports.COMMITMENT_CADENCE_MAX_TIMES = exports.COMMITMENT_CADENCE_TIME_PATTERN = exports.COMMITMENT_CADENCE_MODES = exports.COMMITMENT_TEMPLATE_IDS = exports.COMMITMENT_WATCHED_SOURCES = void 0;
+exports.storedAutonomyModeForAuthoring = storedAutonomyModeForAuthoring;
+exports.authoringAutonomyModeForStored = authoringAutonomyModeForStored;
+exports.commitmentAutonomyLabel = commitmentAutonomyLabel;
 exports.readCommitmentAgentDirective = readCommitmentAgentDirective;
 exports.normalizeCreateBody = normalizeCreateBody;
 exports.parseIntervalToMs = parseIntervalToMs;
@@ -48,9 +51,13 @@ Object.defineProperty(exports, "COMMITMENT_CADENCE_TIME_PATTERN", { enumerable:
 Object.defineProperty(exports, "COMMITMENT_CADENCE_MAX_TIMES", { enumerable: true, get: function () { return data_provider_1.COMMITMENT_CADENCE_MAX_TIMES; } });
 Object.defineProperty(exports, "COMMITMENT_AUTONOMY_MODES", { enumerable: true, get: function () { return data_provider_1.COMMITMENT_AUTONOMY_MODES; } });
 Object.defineProperty(exports, "COMMITMENT_EXECUTION_ISOLATION_MODES", { enumerable: true, get: function () { return data_provider_1.COMMITMENT_EXECUTION_ISOLATION_MODES; } });
+const commitment_product_1 = require("./commitment-product");
 const instruction_input_1 = require("./instruction-input");
 const format_1 = require("./format");
 const triggers_1 = require("./triggers");
+/** Derived from the installed compatibility vocabulary so the CLI remains
+ * safe even while data-provider and CLI packages roll out separately. */
+exports.COMMITMENT_AUTHORABLE_CADENCE_MODES = data_provider_1.COMMITMENT_CADENCE_MODES.filter((mode) => mode !== 'sampled');
 /**
  * oclif's `options:` needs runtime arrays, but data-provider models these two as
  * union TYPES only. The values live here — and both directions of drift are a
@@ -67,17 +74,43 @@ exports.COMMITMENT_STATUSES = [
 ];
 const _statusDriftGuard = true;
 void _statusDriftGuard;
+exports.COMMITMENT_AUTHORABLE_AUTONOMY_MODES = ['monitor', 'prepare', 'execute'];
+function storedAutonomyModeForAuthoring(mode) {
+    if (mode === 'monitor')
+        return 'suggestions_only';
+    if (mode === 'prepare')
+        return 'ask_first';
+    if (mode === 'execute')
+        return 'auto_run';
+    throw new Error(`Unsupported autonomy mode: ${mode}. Use monitor, prepare, or execute.`);
+}
+function authoringAutonomyModeForStored(mode) {
+    if (mode === 'ask_first')
+        return 'prepare';
+    if (mode === 'auto_run' || mode === 'auto_run_limited')
+        return 'execute';
+    return 'monitor';
+}
+function commitmentAutonomyLabel(mode) {
+    if (mode === 'suggestions_only')
+        return 'Monitor';
+    if (mode === 'ask_first')
+        return 'Prepare';
+    if (mode === 'auto_run' || mode === 'auto_run_limited')
+        return 'Execute';
+    if (mode === 'off')
+        return 'Legacy off (pause the Commitment)';
+    return mode || '-';
+}
 /**
  * Interval bounds for `cadence.intervalMs`.
  *
- * The REST schema permits up to 30d, but the mirrored
- * `TriggerDefinition.schedule.intervalMs` is capped at 24h and
- * `CommitmentTriggerSync` swallows the resulting validation failure — so a cadence
- * over 24h yields a commitment whose control loop NEVER fires while every surface
- * reports success. Cap at 24h so that state is unreachable from the CLI.
+ * The Trigger schedule and Commitment contract now share the same one-year
+ * upper bound. Keep the CLI aligned so a valid long-interval contract does not
+ * get rejected only at this surface.
  */
 exports.MIN_COMMITMENT_INTERVAL_MS = 60_000;
-exports.MAX_COMMITMENT_INTERVAL_MS = 24 * 60 * 60 * 1000;
+exports.MAX_COMMITMENT_INTERVAL_MS = 365 * 24 * 60 * 60 * 1000;
 function base(id) {
     return `/api/commitments/${encodeURIComponent(id)}`;
 }
@@ -119,6 +152,11 @@ exports.commitmentApi = {
     checkNow: (id) => data_provider_1.request.post(`${base(id)}/check-now`, {}),
     checks: (id, limit) => data_provider_1.request.get((0, triggers_1.withQuery)(`${base(id)}/checks`, { limit })),
     actionHistory: (id, limit) => data_provider_1.request.get((0, triggers_1.withQuery)(`${base(id)}/action-history`, { limit })),
+    actionProposals: (id, query = {}) => data_provider_1.request.get((0, triggers_1.withQuery)(`${base(id)}/action-proposals`, query)),
+    decideActionProposal: (id, proposalId, decision) => data_provider_1.request.post(`${base(id)}/action-proposals/${encodeURIComponent(proposalId)}/decision`, {
+        decision,
+    }),
+    executeActionProposal: (id, proposalId) => data_provider_1.request.post(`${base(id)}/action-proposals/${encodeURIComponent(proposalId)}/execute`, {}),
     preflight: (id, nextRunCount) => data_provider_1.request.post(`${base(id)}/preflight`, {
         ...(typeof nextRunCount === 'number' ? { nextRunCount } : {}),
     }),
@@ -127,6 +165,11 @@ exports.commitmentApi = {
     resume: (id) => data_provider_1.request.post(`${base(id)}/resume`, {}),
     complete: (id) => data_provider_1.request.post(`${base(id)}/complete`, {}),
     autonomy: (id, body) => data_provider_1.request.post(`${base(id)}/autonomy`, body),
+    mode: (id, body) => data_provider_1.request.post(`${base(id)}/mode`, body),
+    delivery: (id, body) => data_provider_1.request.post(`${base(id)}/delivery`, body),
+    receipts: (id, limit = 1) => data_provider_1.request.get((0, triggers_1.withQuery)(`${base(id)}/receipts`, { limit })),
+    deliveryPatch: (id, taskId) => data_provider_1.request.get(`${base(id)}/delivery/${encodeURIComponent(taskId)}/patch`),
+    deliveryBundle: (id, taskId) => data_provider_1.request.get(`${base(id)}/delivery/${encodeURIComponent(taskId)}/bundle`),
     operatingReview: (agentId) => data_provider_1.request.get(`/api/commitments/agents/${encodeURIComponent(agentId)}/operating-review`),
     reflection: (id) => data_provider_1.request.get(`${base(id)}/reflection`),
     decideReflectionProposal: (id, proposalKey, decision) => data_provider_1.request.post(`${base(id)}/reflection/proposals/${encodeURIComponent(proposalKey)}/decision`, {
@@ -214,6 +257,7 @@ exports.COMMITMENT_REMEDIATION_ACTIONS = [
 ];
 /** The only keys `createBodySchema` (a `z.strictObject`) accepts. */
 exports.COMMITMENT_CREATE_KEYS = [
+    'mode',
     'agentId',
     'kind',
     'compassId',
@@ -226,6 +270,7 @@ exports.COMMITMENT_CREATE_KEYS = [
     'target',
     'watchedSources',
     'policy',
+    'delivery',
     'cadence',
     'subject',
     'sensors',
@@ -257,8 +302,9 @@ async function readCommitmentAgentDirective({ directive, file, }) {
  *     contract dumped from `list --json` carries id/version/state/ownerUserId/…
  *     — drop everything the schema does not declare.
  *
- * `status` is additionally narrowed: create only permits `draft|active`, but a
- * dumped contract may carry `paused`/`completed`.
+ * `status` is additionally narrowed: create only permits `draft`; activation
+ * is the separate resume + preflight boundary. A dumped contract may carry an
+ * active or terminal state, but importing it must still create a draft.
  */
 function normalizeCreateBody(input) {
     const envelope = input;
@@ -270,7 +316,7 @@ function normalizeCreateBody(input) {
         if (src[key] !== undefined)
             out[key] = src[key];
     }
-    if (out.status !== 'draft' && out.status !== 'active') {
+    if (out.status !== 'draft') {
         delete out.status;
     }
     return out;
@@ -323,7 +369,7 @@ function parseIntervalToMs(input) {
     // Throw rather than clamp: silently turning a requested 90d cadence into 30d
     // (or 0 into 1m) hides a real misconfiguration on a live control loop.
     if (ms < exports.MIN_COMMITMENT_INTERVAL_MS || ms > exports.MAX_COMMITMENT_INTERVAL_MS) {
-        throw new Error(`Interval "${input}" is out of range — must be between 1m and 24h (got ${describeIntervalMs(ms)}).`);
+        throw new Error(`Interval "${input}" is out of range — must be between 1m and 365d (got ${describeIntervalMs(ms)}).`);
     }
     return ms;
 }
@@ -508,14 +554,14 @@ function renderCommitmentList(commitments, log) {
         status: o.status ?? '-',
         health: o.state?.health ?? '-',
         cadence: cadenceLabel(o),
-        autonomy: o.policy?.autonomyMode ?? '-',
+        autonomy: (0, commitment_product_1.productModeLabel)(o.policy?.autonomyMode),
         agent: o.agentId ?? '-',
     })), [
         { key: 'title', header: 'TITLE', maxWidth: 34 },
         { key: 'status', header: 'STATUS', maxWidth: 10 },
         { key: 'health', header: 'HEALTH', maxWidth: 16 },
         { key: 'cadence', header: 'CADENCE', maxWidth: 10 },
-        { key: 'autonomy', header: 'AUTONOMY', maxWidth: 18 },
+        { key: 'autonomy', header: 'MODE', maxWidth: 18 },
         { key: 'agent', header: 'AGENT', verbatim: true },
         // Commitment ids are routinely pasted into `check-now`, so this column
         // is never clipped. Widening the guess (28 → 44) fixed today's
@@ -527,13 +573,11 @@ function renderCommitmentList(commitments, log) {
  * The autonomy mode a check ACTUALLY ran at, read from its policy trace
  * (OSK-4881).
  *
- * A commitment's STATED autonomy (`policy.autonomyMode`) is only a ceiling: the
- * arbiter clamps it to the stricter of the agent's initiative policy and the
- * workspace's governance, then records the mode it settled on in the check's
- * `decision.policyTrace`. The arbiter's own decision carries it as
- * `autonomyMode`; the workspace-governance entry carries the clamped ceiling as
- * `effectiveAutonomyMode`. Either answers "what governed this run". Returns the
- * effective mode when the trace carries one, else undefined.
+ * A check records the Mode it actually used in `decision.policyTrace`. For the
+ * v2 product contract, only an explicit Workspace-admin constraint may lower
+ * the declared Mode; legacy checks may still carry older effective-mode traces.
+ * Either `autonomyMode` or `effectiveAutonomyMode` answers what governed that
+ * particular run. Returns undefined when the trace carries neither.
  *
  * Exported for tests.
  */
@@ -598,14 +642,10 @@ function extractStandstillFromCheck(check) {
 function describeAutonomyLine(commitment, latestCheck) {
     const stated = commitment.policy?.autonomyMode ?? '-';
     const effective = effectiveAutonomyMode(latestCheck);
-    const envelopeEnabled = commitment.policy?.autonomyEnvelope?.enabled;
-    const envelope = typeof envelopeEnabled === 'boolean'
-        ? ` [envelope: ${envelopeEnabled ? 'enabled' : 'disabled'}]`
-        : '';
     if (effective && effective !== stated) {
-        return `${stated} (effective: ${effective} — capped by initiative or workspace policy)${envelope}`;
+        return `${(0, commitment_product_1.productModeLabel)(stated)} (effective: ${(0, commitment_product_1.productModeLabel)(effective)} — constrained by configured policy)`;
     }
-    return `${stated}${envelope}`;
+    return (0, commitment_product_1.productModeLabel)(stated);
 }
 function renderCommitmentSummary(commitment, log, 
 /**
@@ -630,7 +670,8 @@ latestCheck) {
     log(`Cadence:   ${cadenceLabel(commitment)}`);
     if (commitment.cadence?.nextCheckAt)
         log(`Next check:${` ${commitment.cadence.nextCheckAt}`}`);
-    log(`Autonomy:  ${describeAutonomyLine(commitment, latestCheck)}`);
+    log(`Mode:      ${describeAutonomyLine(commitment, latestCheck)}`);
+    log(`Delivery:  ${(0, commitment_product_1.deliveryLabel)(commitment.delivery?.mode)}${commitment.delivery?.repo ? ` · ${commitment.delivery.repo}` : ''}${commitment.delivery?.baseBranch ? ` → ${commitment.delivery.baseBranch}` : ''}`);
     const criteria = commitment.target?.successCriteria || [];
     const evidenceHealth = latestCheck?.inputs?.evidenceHealth;
     const evidenceLine = describeEvidenceHealth(evidenceHealth, criteria.length);
@@ -654,7 +695,7 @@ latestCheck) {
     if (standstill) {
         const streak = commitment.state?.consecutiveUnresolvedChecks;
         const streakClause = typeof streak === 'number' && streak >= 2 ? `${streak} checks in a row: ` : '';
-        log(`Standstill:${` ${streakClause}${standstill.reason}`}`);
+        log(`Standstill:${` ${streakClause}${(0, data_provider_1.describeStandstill)(standstill.reason).title}`}`);
         if (standstill.fix)
             log(`           Fix: ${standstill.fix}`);
         if (standstill.alsoConstrained?.length) {
@@ -667,7 +708,7 @@ latestCheck) {
             })
                 .filter((r) => Boolean(r));
             if (also.length)
-                log(`           Also constrained: ${also.join(', ')}`);
+                log(`           Also constrained: ${also.map((reason) => (0, data_provider_1.describeStandstill)(reason).title).join(', ')}`);
         }
     }
     const sources = (commitment.watchedSources || [])