@skrr-ai/cli 0.1.15 → 0.1.18

package/dist/lib/commitments.js

@@ -1,6 +1,10 @@
 "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_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;
 exports.buildWatchedSources = buildWatchedSources;
@@ -45,8 +49,15 @@ Object.defineProperty(exports, "COMMITMENT_TEMPLATE_IDS", { enumerable: true, ge
 Object.defineProperty(exports, "COMMITMENT_CADENCE_MODES", { enumerable: true, get: function () { return data_provider_1.COMMITMENT_CADENCE_MODES; } });
 Object.defineProperty(exports, "COMMITMENT_CADENCE_TIME_PATTERN", { enumerable: true, get: function () { return data_provider_1.COMMITMENT_CADENCE_TIME_PATTERN; } });
 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
@@ -63,27 +74,43 @@ exports.COMMITMENT_STATUSES = [
 ];
 const _statusDriftGuard = true;
 void _statusDriftGuard;
-/** Autonomy ladder — how much the commitment may act without asking. */
-exports.COMMITMENT_AUTONOMY_MODES = [
-    'off',
-    'suggestions_only',
-    'ask_first',
-    'auto_run_limited',
-    'auto_run',
-];
-const _autonomyDriftGuard = true;
-void _autonomyDriftGuard;
+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)}`;
 }
@@ -116,6 +143,7 @@ exports.commitmentApi = {
     get: (id) => data_provider_1.request.get(base(id)),
     create: (body) => data_provider_1.request.post('/api/commitments', body),
     update: (id, body) => data_provider_1.request.patch(base(id), body),
+    draftPreflight: (body) => data_provider_1.request.post('/api/commitments/preflight', body),
     workLinks: (id, includeInactive = false) => data_provider_1.request.get((0, triggers_1.withQuery)(`${base(id)}/links`, { ...(includeInactive ? { includeInactive: true } : {}) })),
     createWorkLink: (id, body) => data_provider_1.request.post(`${base(id)}/links`, body),
     updateWorkLink: (id, linkId, body) => data_provider_1.request.patch(`${base(id)}/links/${encodeURIComponent(linkId)}`, body),
@@ -124,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 } : {}),
     }),
@@ -132,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`, {
@@ -219,6 +257,7 @@ exports.COMMITMENT_REMEDIATION_ACTIONS = [
 ];
 /** The only keys `createBodySchema` (a `z.strictObject`) accepts. */
 exports.COMMITMENT_CREATE_KEYS = [
+    'mode',
     'agentId',
     'kind',
     'compassId',
@@ -231,6 +270,7 @@ exports.COMMITMENT_CREATE_KEYS = [
     'target',
     'watchedSources',
     'policy',
+    'delivery',
     'cadence',
     'subject',
     'sensors',
@@ -238,6 +278,20 @@ exports.COMMITMENT_CREATE_KEYS = [
     'reports',
     'createdVia',
 ];
+/**
+ * Read the per-Commitment domain policy from an inline flag or a text file.
+ *
+ * This deliberately uses a separate pair of names from Agent-level instruction
+ * commands: the result configures one Commitment run, not the Agent's global
+ * prompt. The explicit error also keeps shell users from accidentally masking
+ * a reviewed file with a short inline experiment.
+ */
+async function readCommitmentAgentDirective({ directive, file, }) {
+    if (directive !== undefined && file !== undefined) {
+        throw new Error('Use either --agent-directive or --agent-directive-file, not both.');
+    }
+    return (0, instruction_input_1.readInstructionBody)({ body: directive, file });
+}
 /**
  * Normalize a `--from-json` body into something the create endpoint accepts.
  *
@@ -248,8 +302,9 @@ exports.COMMITMENT_CREATE_KEYS = [
  *     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;
@@ -261,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;
@@ -314,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;
 }
@@ -499,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
@@ -518,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.
  */
@@ -589,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, 
 /**
@@ -621,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);
@@ -645,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) {
@@ -658,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 || [])

package/dist/lib/tasks.js

@@ -170,8 +170,34 @@ function parseDeliverableCoordinate(spec) {
 }
 /** Lifecycle values a caller may assert, for flag validation. */
 exports.DELIVERABLE_LIFECYCLE_VALUES = data_provider_1.DELIVERABLE_LIFECYCLES;
+/**
+ * `--from-json` takes stored API field names, while the interactive flags use
+ * shorter names such as `--project` and `--spawned-by`. Normalize the obvious
+ * camel-case forms before a permissive API route can silently drop them.
+ */
+function normalizeTaskCreateJsonAliases(body) {
+    const normalized = { ...body };
+    const aliases = [
+        ['parent', 'parentId'],
+        ['space', 'spaceId'],
+        ['goal', 'goalId'],
+        ['commitment', 'commitmentId'],
+        ['project', 'projectId'],
+        ['cycle', 'cycleId'],
+        ['spawnedBy', 'spawnedByTaskId'],
+    ];
+    for (const [alias, canonical] of aliases) {
+        if (Object.prototype.hasOwnProperty.call(normalized, canonical))
+            continue;
+        if (!Object.prototype.hasOwnProperty.call(normalized, alias))
+            continue;
+        normalized[canonical] = normalized[alias];
+        delete normalized[alias];
+    }
+    return normalized;
+}
 function buildTaskCreatePayload(body, flags = {}) {
-    const payload = (0, prompt_1.mergeDefined)(body, {
+    const payload = (0, prompt_1.mergeDefined)(normalizeTaskCreateJsonAliases(body), {
         templateId: flags.template,
         title: flags.title,
         description: flags.description,