@skrr-ai/cli 0.1.15 → 0.1.18

package/dist/commands/commitments/remediate.js

@@ -15,7 +15,6 @@ const commitments_1 = require("../../lib/commitments");
  *   pause_commitment        stop the loop entirely
  *   mute_user_nudges        keep watching, stop interrupting (max nudges → 0)
  *   reduce_nudge_frequency  at most one nudge per day
- *   tighten_policy          require approval for the risky action classes
  *   mark_noop               record that this fired for no reason
  */
 class CommitmentsRemediate extends base_command_1.BaseCommand {
@@ -30,8 +29,11 @@ class CommitmentsRemediate extends base_command_1.BaseCommand {
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output as JSON' }),
         action: core_1.Flags.string({
-            description: 'Remediation action',
-            options: [...commitments_1.COMMITMENT_REMEDIATION_ACTIONS],
+            description: 'Remediation action. Change work authority with commitments mode; Workspace approval rules belong in Workspace governance.',
+            // `tighten_policy` remains a REST compatibility value for legacy rows. It
+            // is deliberately absent from the product CLI: v2 authority changes are
+            // explicit Mode changes, never a hidden risk/approval tier.
+            options: commitments_1.COMMITMENT_REMEDIATION_ACTIONS.filter((action) => action !== 'tighten_policy'),
             required: true,
         }),
         reason: core_1.Flags.string({ description: 'Why (recorded on the boundary-change ledger)' }),

package/dist/commands/commitments/reports.js

@@ -6,14 +6,24 @@ const base_command_1 = require("../../base-command");
 const commitments_1 = require("../../lib/commitments");
 const commitment_terms_1 = require("../../lib/commitment-terms");
 /** Destinations that address one specific place and therefore need a target. */
-const DESTINATIONS_NEEDING_TARGET = new Set(['slack_channel', 'email']);
+const DESTINATIONS_NEEDING_TARGET = new Set(['slack_channel']);
+/**
+ * Keep the CLI safe across a rolling data-provider build. Older installed
+ * packages expose the compatibility destination list but not the narrowed
+ * authoring list; derive the same restriction instead of crashing command
+ * discovery or accidentally offering the inert email transport.
+ */
+const AUTHORABLE_REPORT_DESTINATIONS = Array.isArray(data_provider_1.COMMITMENT_AUTHORABLE_REPORT_DESTINATIONS)
+    ? data_provider_1.COMMITMENT_AUTHORABLE_REPORT_DESTINATIONS
+    : data_provider_1.COMMITMENT_REPORT_DESTINATIONS.filter((destination) => destination !== 'email');
 /**
  * Identity of a report target = destination + its target id. The same channel
  * may legitimately appear twice with different `on` conditions (a digest on
  * completion, a ping on escalation), so `on` is NOT part of the key.
  */
 function reportKey(r) {
-    return r.destinationTarget ? `${r.destination}:${r.destinationTarget}` : r.destination;
+    const target = r.destinationTarget ? `${r.destination}:${r.destinationTarget}` : r.destination;
+    return `${target}:${r.on ?? 'material_change'}`;
 }
 /**
  * `skrr commitments reports <id>` — where this commitment tells you things.
@@ -40,10 +50,10 @@ class CommitmentsReports extends base_command_1.BaseCommand {
         json: core_1.Flags.boolean({ description: 'Output as JSON' }),
         add: core_1.Flags.string({
             description: 'Destination to add',
-            options: [...data_provider_1.COMMITMENT_REPORT_DESTINATIONS],
+            options: [...AUTHORABLE_REPORT_DESTINATIONS],
         }),
         target: core_1.Flags.string({
-            description: 'Channel id or address, for destinations that address one (slack_channel, email)',
+            description: 'Channel id for slack_channel. App and owner-DM destinations need no target.',
         }),
         on: core_1.Flags.string({
             description: 'When to report (default: material_change)',
@@ -98,8 +108,20 @@ class CommitmentsReports extends base_command_1.BaseCommand {
                 payload = { reports: (0, commitment_terms_1.appendUnique)(reports, next, reportKey, 'Report target') };
             }
             else {
+                const base = String(flags.remove);
+                const matches = reports.filter((report) => {
+                    const target = report.destinationTarget
+                        ? `${report.destination}:${report.destinationTarget}`
+                        : report.destination;
+                    return target === base && (!flags.on || (report.on ?? 'material_change') === flags.on);
+                });
+                if (matches.length !== 1) {
+                    throw new Error(matches.length > 1
+                        ? `Multiple report targets match ${base}. Pass --on to select one condition.`
+                        : `Report target ${base} is not attached to this commitment.`);
+                }
                 payload = {
-                    reports: (0, commitment_terms_1.removeByKey)(reports, String(flags.remove), reportKey, 'Report target'),
+                    reports: (0, commitment_terms_1.removeByKey)(reports, reportKey(matches[0]), reportKey, 'Report target'),
                 };
             }
         }

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

@@ -15,6 +15,7 @@ export default class CommitmentsShow extends BaseCommand {
     };
     static flags: {
         json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        verbose: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
 }

package/dist/commands/commitments/show.js

@@ -4,6 +4,7 @@ const core_1 = require("@oclif/core");
 const base_command_1 = require("../../base-command");
 const commitments_1 = require("../../lib/commitments");
 const web_url_1 = require("../../lib/web-url");
+const commitment_product_1 = require("../../lib/commitment-product");
 /**
  * `skrr commitments show <id>` — the commitment's contract + live control-loop
  * state (health, last check, next check, unresolved-check streak), plus the most
@@ -23,6 +24,9 @@ class CommitmentsShow extends base_command_1.BaseCommand {
     };
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output as JSON' }),
+        verbose: core_1.Flags.boolean({
+            description: 'Include raw compatibility policy and check diagnostics',
+        }),
     };
     async run() {
         this.requireAuth();
@@ -46,13 +50,30 @@ class CommitmentsShow extends base_command_1.BaseCommand {
             return;
         }
         (0, commitments_1.renderCommitmentSummary)(response.commitment, (line) => this.log(line), response.latestCheck);
-        if (response.latestCheck) {
+        try {
+            const receipts = await commitments_1.commitmentApi.receipts(args.id, 1);
+            if (receipts.receipts[0]) {
+                this.log('');
+                (0, commitment_product_1.renderExecutionReceipt)(receipts.receipts[0], (line) => this.log(line), flags.verbose);
+            }
+            else
+                this.log('No wake has been observed yet.');
+        }
+        catch {
+            this.log('Execution receipt is unavailable. Retry with commitments receipt <id> --latest.');
+        }
+        if (flags.verbose && response.latestCheck) {
             this.log('');
             this.log('Latest check:');
             (0, commitments_1.renderCheckList)([response.latestCheck], (line) => this.log(line));
             this.log('');
             this.log(`Full history:  ${this.config.bin} commitments checks ${args.id}`);
         }
+        if (flags.verbose)
+            this.log(`Compatibility policy: ${JSON.stringify(response.commitment.policy)}`);
+        const url = (0, web_url_1.commitmentUrlFrom)(this.cliConfig.baseURL, response.commitment);
+        if (url)
+            this.log(`URL: ${url}`);
     }
 }
 exports.default = CommitmentsShow;

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

@@ -13,8 +13,11 @@ import { BaseCommand } from '../../base-command';
  * `freshnessExpectation` thresholds the loop uses to detect stale open loops.
  * `--criterion` still REPLACES the criteria list (that is its purpose).
  *
- * Autonomy is deliberately not settable here — `policy` is likewise replaced
- * wholesale. Use `skrr commitments autonomy <id>`.
+ * Autonomy is deliberately not settable here: raising it wants a recorded
+ * reason, which `skrr commitments autonomy <id>` requires and this command has
+ * no place for. Other policy fields ARE patchable — the API deep-merges a
+ * `policy` patch (`mergePolicyPatch`), so a single-key patch leaves the rest of
+ * the contract untouched.
  */
 export default class CommitmentsUpdate extends BaseCommand {
     static description: string;
@@ -25,8 +28,11 @@ export default class CommitmentsUpdate extends BaseCommand {
     static flags: {
         json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         'from-json': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'if-version': import("@oclif/core/lib/interfaces").OptionFlag<number | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         title: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         description: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'agent-directive': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'agent-directive-file': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         status: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         outcome: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         deadline: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
@@ -40,6 +46,7 @@ export default class CommitmentsUpdate extends BaseCommand {
         cadence: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'cadence-mode': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         timezone: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        isolation: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
     };
     run(): Promise<void>;
 }

package/dist/commands/commitments/update.js

@@ -20,8 +20,11 @@ const commitments_1 = require("../../lib/commitments");
  * `freshnessExpectation` thresholds the loop uses to detect stale open loops.
  * `--criterion` still REPLACES the criteria list (that is its purpose).
  *
- * Autonomy is deliberately not settable here — `policy` is likewise replaced
- * wholesale. Use `skrr commitments autonomy <id>`.
+ * Autonomy is deliberately not settable here: raising it wants a recorded
+ * reason, which `skrr commitments autonomy <id>` requires and this command has
+ * no place for. Other policy fields ARE patchable — the API deep-merges a
+ * `policy` patch (`mergePolicyPatch`), so a single-key patch leaves the rest of
+ * the contract untouched.
  */
 class CommitmentsUpdate extends base_command_1.BaseCommand {
     static description = "Update a commitment's contract (title, target, cadence, sources)";
@@ -30,6 +33,7 @@ class CommitmentsUpdate extends base_command_1.BaseCommand {
         '<%= config.bin %> commitments update <commitment-id> --title "Ship v2 (Q3)" --outcome "v2 is live for all users"',
         '<%= config.bin %> commitments update <commitment-id> --watch tasks --watch github',
         '<%= config.bin %> commitments update <commitment-id> --criterion "signups confirmed" --evaluator metric --measure <measure-id>',
+        '<%= config.bin %> commitments update <commitment-id> --agent-directive-file ./review-policy.md',
     ];
     static args = {
         id: core_1.Args.string({ description: 'Commitment ID', required: true, ignoreStdin: true }),
@@ -39,8 +43,17 @@ class CommitmentsUpdate extends base_command_1.BaseCommand {
         'from-json': core_1.Flags.string({
             description: 'Path to a JSON file (or "-" for stdin) for fields not exposed as flags',
         }),
+        'if-version': core_1.Flags.integer({
+            description: 'Only update this observed contract version; returns a conflict if it changed',
+        }),
         title: core_1.Flags.string({ description: 'New title' }),
         description: core_1.Flags.string({ description: 'New description' }),
+        'agent-directive': core_1.Flags.string({
+            description: 'Replace the per-Commitment domain policy (max 8,000 characters). Mutually exclusive with --agent-directive-file.',
+        }),
+        'agent-directive-file': core_1.Flags.string({
+            description: 'Read replacement domain policy from a text file, or - for stdin. Mutually exclusive with --agent-directive.',
+        }),
         status: core_1.Flags.string({
             description: 'New status. Invalid lifecycle jumps (e.g. completed → active) are rejected by the API.',
             options: [...commitments_1.COMMITMENT_STATUSES],
@@ -76,12 +89,22 @@ class CommitmentsUpdate extends base_command_1.BaseCommand {
             description: 'Signal Monitor id to watch (repeatable). Implies --watch monitor.',
             multiple: true,
         }),
-        cadence: core_1.Flags.string({ description: 'New check interval, e.g. "15m", "6h", "1d" (1m..30d)' }),
+        cadence: core_1.Flags.string({
+            description: 'New check interval, e.g. "15m", "6h", "30d" (1m..365d)',
+        }),
         'cadence-mode': core_1.Flags.string({
             description: 'Cadence mode',
-            options: [...commitments_1.COMMITMENT_CADENCE_MODES],
+            options: [...commitments_1.COMMITMENT_AUTHORABLE_CADENCE_MODES],
         }),
         timezone: core_1.Flags.string({ description: 'IANA timezone for the cadence' }),
+        isolation: core_1.Flags.string({
+            // Sent as a single-key `policy` patch. The API deep-merges it, so this
+            // cannot blank an autonomy mode or an attention budget set elsewhere.
+            description: 'Where autonomous runs do their work: "ephemeral" (one-shot git worktree per run) or ' +
+                '"shared" (the daemon working tree, in place). Cannot be combined with a remediation ' +
+                'capability.',
+            options: [...commitments_1.COMMITMENT_EXECUTION_ISOLATION_MODES],
+        }),
     };
     async run() {
         this.requireAuth();
@@ -115,10 +138,18 @@ class CommitmentsUpdate extends base_command_1.BaseCommand {
         let payload;
         try {
             payload = await (0, prompt_1.readJsonBody)(flags['from-json']);
+            if (flags['if-version'] !== undefined)
+                payload.expectedVersion = flags['if-version'];
             if (flags.title)
                 payload.title = flags.title;
             if (flags.description !== undefined)
                 payload.description = flags.description;
+            const agentDirective = await (0, commitments_1.readCommitmentAgentDirective)({
+                directive: flags['agent-directive'],
+                file: flags['agent-directive-file'],
+            });
+            if (agentDirective !== undefined)
+                payload.agentDirective = agentDirective;
             if (flags.status)
                 payload.status = flags.status;
             if (wantsTarget) {
@@ -189,6 +220,11 @@ class CommitmentsUpdate extends base_command_1.BaseCommand {
                     cadence.timezone = flags.timezone;
                 payload.cadence = cadence;
             }
+            // A single key, deliberately. `mergePolicyPatch` on the server merges it
+            // into the stored policy, so naming only this field is what keeps the
+            // rest of the contract intact.
+            if (flags.isolation)
+                payload.policy = { executionIsolation: flags.isolation };
         }
         catch (err) {
             this.error(err.message, { exit: 1 });

package/dist/commands/commitments/wake-capabilities.d.ts

@@ -1,4 +1,7 @@
 import { BaseCommand } from '../../base-command';
+export declare function wakePolicyHeadline(result: {
+    workPromotionModel?: string;
+}): string;
 /**
  * `skrr commitments wake-capabilities <id>` — which wakes an Agent may pick.
  *
@@ -10,6 +13,7 @@ import { BaseCommand } from '../../base-command';
  * misconfiguration. Showing only one list would make those indistinguishable.
  */
 export default class CommitmentsWakeCapabilities extends BaseCommand {
+    static hidden: boolean;
     static description: string;
     static examples: string[];
     static args: {
@@ -17,6 +21,7 @@ export default class CommitmentsWakeCapabilities extends BaseCommand {
     };
     static flags: {
         json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        verbose: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
 }

package/dist/commands/commitments/wake-capabilities.js

@@ -1,9 +1,15 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.wakePolicyHeadline = wakePolicyHeadline;
 const core_1 = require("@oclif/core");
 const base_command_1 = require("../../base-command");
 const commitments_1 = require("../../lib/commitments");
 const format_1 = require("../../lib/format");
+function wakePolicyHeadline(result) {
+    return result.workPromotionModel === 'monitor_prepare_execute_v1'
+        ? 'Wake behavior: configured Commitment sources may wake it automatically; there is no separate permission level.'
+        : 'Compatibility wake policy is in use. Migrate the Commitment by choosing Monitor, Prepare, or Execute.';
+}
 /**
  * `skrr commitments wake-capabilities <id>` — which wakes an Agent may pick.
  *
@@ -15,7 +21,10 @@ const format_1 = require("../../lib/format");
  * misconfiguration. Showing only one list would make those indistinguishable.
  */
 class CommitmentsWakeCapabilities extends base_command_1.BaseCommand {
-    static description = 'List the wake capabilities an agent may select for a commitment';
+    // Compatibility/operator diagnostic. Product users configure sources and
+    // cadence directly; v2 has no separate wake-permission level.
+    static hidden = true;
+    static description = 'Inspect configured wake-source diagnostics for a commitment';
     static examples = [
         '<%= config.bin %> commitments wake-capabilities obj_abc123',
         '<%= config.bin %> commitments wake-capabilities obj_abc123 --json',
@@ -23,7 +32,10 @@ class CommitmentsWakeCapabilities extends base_command_1.BaseCommand {
     static args = {
         commitmentId: core_1.Args.string({ ignoreStdin: true, description: 'Commitment ID', required: true }),
     };
-    static flags = { json: core_1.Flags.boolean({ description: 'Output as JSON' }) };
+    static flags = {
+        json: core_1.Flags.boolean({ description: 'Output as JSON' }),
+        verbose: core_1.Flags.boolean({ description: 'Include raw legacy wake-policy values' }),
+    };
     async run() {
         this.requireAuth();
         const { args, flags } = await this.parse(CommitmentsWakeCapabilities);
@@ -38,11 +50,15 @@ class CommitmentsWakeCapabilities extends base_command_1.BaseCommand {
             this.log(JSON.stringify(result, null, 2));
             return;
         }
-        this.log(`Wake permission mode: ${result.mode}`);
+        const productModel = result.workPromotionModel === 'monitor_prepare_execute_v1';
+        this.log(wakePolicyHeadline(result));
         const selected = result.selectedCapabilityIds;
-        this.log(selected === undefined
-            ? 'Allowlist: not configured (pre-picker commitment — every visible capability is offered)'
-            : `Allowlist: ${selected.length ? selected.join(', ') : '(empty — no agent-selected wake may materialize)'}`);
+        if (flags.verbose) {
+            this.log(`Raw compatibility mode: ${result.mode}`);
+            this.log(selected === undefined
+                ? 'Raw allowlist: not configured'
+                : `Raw allowlist: ${selected.length ? selected.join(', ') : '(empty)'}`);
+        }
         this.log('');
         const visible = new Set(result.visibleCapabilities.map((item) => item.id));
         (0, format_1.renderTable)(result.eligibleCapabilities.map((item) => ({
@@ -54,7 +70,7 @@ class CommitmentsWakeCapabilities extends base_command_1.BaseCommand {
             { key: 'id', header: 'CAPABILITY' },
             { key: 'kind', header: 'KIND' },
             { key: 'source', header: 'SOURCE' },
-            { key: 'offered', header: 'OFFERED' },
+            { key: 'offered', header: productModel ? 'ACTIVE' : 'OFFERED' },
         ], (line) => this.log(line));
     }
 }

package/dist/commands/spaces/bootstrap.js

@@ -60,7 +60,7 @@ class SpacesBootstrap extends base_command_1.BaseCommand {
     // one. It names the scope now, and the command prints the server's own
     // contract so the boundary is visible at the point of use rather than buried
     // in a route comment.
-    static description = 'Fetch the Space first-paint payload (space, summary, permissions, and Live slices). ' +
+    static description = 'Fetch the Space first-paint payload (space, summary, and permissions). ' +
         'NOT a complete Space export — tasks, members, Room, files and wiki have their own commands; ' +
         'the printed contract names each one.';
     static examples = [

package/dist/commands/tasks/complete.js

@@ -22,6 +22,38 @@ const task_transcript_1 = require("../../lib/task-transcript");
 const web_url_1 = require("../../lib/web-url");
 const tasks_1 = require("../../lib/tasks");
 const git_task_1 = require("../../lib/git-task");
+/**
+ * Forward any `--deliverable` value that is really a ledger coordinate.
+ *
+ * FAIL-OPEN, and quietly. This is a convenience bridge on the completion path:
+ * the completion itself is the operation the caller asked for, and a ledger
+ * write that fails must not take it down. A caller who needed the row and did
+ * not get it meets the gate's refusal, which names the command that writes it.
+ *
+ * `local`, never anything stronger — this is the executor talking about its own
+ * work, which is exactly what the authority ladder is built to weigh.
+ */
+async function recordCoordinateDeliverables({ taskId, values, warn, }) {
+    const seen = new Set();
+    for (const value of values) {
+        const raw = (value ?? '').trim();
+        if (!raw || raw === 'none' || seen.has(raw)) {
+            continue;
+        }
+        seen.add(raw);
+        const parsed = (0, tasks_1.parseDeliverableCoordinate)(raw);
+        if (!parsed.ok) {
+            continue;
+        }
+        try {
+            await data_provider_1.dataService.addTaskDeliverable(taskId, parsed.value);
+        }
+        catch {
+            warn(`Recorded "${raw}" in the report but not in the deliverables ledger. ` +
+                `If the task refuses to close, run \`tasks deliverable add\` for it.`);
+        }
+    }
+}
 /**
  * Mutation endpoints require the task's canonical UUID even though the CLI
  * accepts human-readable refs and titles. Resolve the display reference before
@@ -483,6 +515,31 @@ class TasksComplete extends base_command_1.BaseCommand {
                 });
             }
         }
+        // THE THIRD SPELLING, BRIDGED.
+        //
+        // `--deliverable` here is PROSE for the TASK_OUTPUT block. The ledger the
+        // completion gate reads is written by `tasks deliverable add`, and nothing
+        // connected the two — so an agent that ran
+        // `tasks complete --deliverable code:abc@origin/main` had, from its own
+        // point of view, recorded its deliverable, and was then refused by a gate
+        // whose message told it to record its deliverable.
+        //
+        // Three surfaces in one CLI answer to the word (`tasks deliverable add`,
+        // this flag, and `tasks report --deliverable`), and the project's own
+        // operating instructions teach the two that do NOT write the ledger. That
+        // is a documentation problem only if you believe people read the docs
+        // before the error; the cheaper fix is to make the wrong spelling do the
+        // right thing when what was typed is unambiguously a coordinate.
+        //
+        // Deliberately narrow: only values that PARSE as `<kind>:<identity>` are
+        // forwarded. A prose reference like `report.pdf` is left exactly as it was —
+        // guessing a coordinate out of prose is how the ledger fills with rows
+        // nobody can look up again.
+        await recordCoordinateDeliverables({
+            taskId: completionTaskId,
+            values: [...(flags.deliverable ?? []), ...(flags.deliverables ?? [])],
+            warn: (message) => this.warn(message),
+        });
         let result;
         try {
             result = await completeTaskDurably({

package/dist/commands/tasks/create.js

@@ -71,7 +71,8 @@ class TasksCreate extends base_command_1.BaseCommand {
             description: 'Validate and print the create payload without writing a task',
         }),
         'from-json': core_1.Flags.string({
-            description: 'Path to a JSON file (or "-" for stdin) to use as the request body',
+            description: 'Path to a JSON file (or "-" for stdin) to use as the request body. ' +
+                'Accepts stored names (projectId, goalId) and create-flag aliases (project, goal, spawnedBy).',
         }),
         template: core_1.Flags.string({
             description: 'Saved task template ID, or an exact template name with --space. Creates its whole task tree; supplied task flags override the root.',

package/dist/commands/tasks/deliverable/confirm.d.ts

@@ -0,0 +1,35 @@
+import { BaseCommand } from '../../../base-command';
+/**
+ * `skrr tasks deliverable confirm <task> <kind>:<identity>[@<location>] --reason "<what you checked>"`
+ *
+ * Somebody looked, and it is there.
+ *
+ * WHY THIS IS A SEPARATE COMMAND from `deliverable add`. Recording and vouching
+ * are different claims. `add` says "this exists, and I am the one who made it";
+ * this says "I checked, independently of whoever made it". Five of the six
+ * outcome kinds have no automatic observer — nothing in the platform watches a
+ * Drive folder or a Slack channel — so without this rung an expectation for a
+ * PDF or a posted message could only ever be closed by waiving it, which
+ * records "we decided we did not need this" about work that was actually done.
+ *
+ * The reason is required, for the same purpose a waiver's and a judgement's
+ * are: the entire value of a confirmation is that a person actually looked, and
+ * a confirmation with nothing written down is indistinguishable from a click.
+ *
+ * It UPSERTS. Confirming something the executor never recorded is legitimate —
+ * the thing is there either way — and making a person file the row first before
+ * they may vouch for it is ceremony that buys nothing.
+ */
+export default class TasksDeliverableConfirm extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static args: {
+        task: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+        coordinate: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+    };
+    static flags: {
+        reason: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
+        json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+    };
+    run(): Promise<void>;
+}

package/dist/commands/tasks/deliverable/confirm.js

@@ -0,0 +1,77 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const data_provider_1 = require("@skrr-ai/data-provider");
+const base_command_1 = require("../../../base-command");
+const tasks_1 = require("../../../lib/tasks");
+/**
+ * `skrr tasks deliverable confirm <task> <kind>:<identity>[@<location>] --reason "<what you checked>"`
+ *
+ * Somebody looked, and it is there.
+ *
+ * WHY THIS IS A SEPARATE COMMAND from `deliverable add`. Recording and vouching
+ * are different claims. `add` says "this exists, and I am the one who made it";
+ * this says "I checked, independently of whoever made it". Five of the six
+ * outcome kinds have no automatic observer — nothing in the platform watches a
+ * Drive folder or a Slack channel — so without this rung an expectation for a
+ * PDF or a posted message could only ever be closed by waiving it, which
+ * records "we decided we did not need this" about work that was actually done.
+ *
+ * The reason is required, for the same purpose a waiver's and a judgement's
+ * are: the entire value of a confirmation is that a person actually looked, and
+ * a confirmation with nothing written down is indistinguishable from a click.
+ *
+ * It UPSERTS. Confirming something the executor never recorded is legitimate —
+ * the thing is there either way — and making a person file the row first before
+ * they may vouch for it is ceremony that buys nothing.
+ */
+class TasksDeliverableConfirm extends base_command_1.BaseCommand {
+    static description = 'Confirm you checked a deliverable and it is there — the evidence an expectation needs when nothing observes it automatically';
+    static examples = [
+        '<%= config.bin %> tasks deliverable confirm <task-id> file/pdf:report_92@Drive/Reports --reason "Opened it; all four sections are there."',
+        '<%= config.bin %> tasks deliverable confirm <task-id> message:m_51@#general --reason "Read the thread; it went out at 09:04."',
+    ];
+    static args = {
+        task: core_1.Args.string({ description: 'Task id or ref', required: true, ignoreStdin: true }),
+        coordinate: core_1.Args.string({
+            description: '<kind>[/<format>]:<identity>[@<location>] — the thing you checked',
+            required: true,
+            ignoreStdin: true,
+        }),
+    };
+    static flags = {
+        reason: core_1.Flags.string({
+            required: true,
+            description: 'What you looked at. A confirmation without one is not a confirmation.',
+        }),
+        json: core_1.Flags.boolean({ description: 'Output as JSON' }),
+    };
+    async run() {
+        this.requireAuth();
+        const { args, flags } = await this.parse(TasksDeliverableConfirm);
+        const parsed = (0, tasks_1.parseDeliverableCoordinate)(args.coordinate);
+        if (!parsed.ok) {
+            // For an agent the refusal IS the documentation — it hits the error and
+            // corrects — so it carries the whole form and vocabulary.
+            this.error(parsed.error, { exit: 1 });
+            return;
+        }
+        let result;
+        try {
+            result = await data_provider_1.dataService.confirmTaskDeliverable(args.task, {
+                ...parsed.value,
+                reason: flags.reason,
+            });
+        }
+        catch (err) {
+            this.handleApiError(err);
+            return;
+        }
+        if (flags.json) {
+            this.log(JSON.stringify(result, null, 2));
+            return;
+        }
+        this.log(`Confirmed ${parsed.value.kind}:${parsed.value.identity}: ${flags.reason}`);
+    }
+}
+exports.default = TasksDeliverableConfirm;

package/dist/commands/tasks/expectations.js

@@ -15,6 +15,10 @@ const STATE_LABEL = {
     met: 'done',
     waived: 'dropped',
     blocked: 'stuck',
+    // Its own word, because its next action is its own: somebody looks, rather
+    // than somebody works. Reading it as 'not yet' sends people to redo finished
+    // work, which is precisely the confusion the state was added to remove.
+    unverified: 'needs a look',
     underway: 'on its way',
     owed: 'not yet',
     unjudged: 'needs a person',
@@ -26,7 +30,13 @@ function line(expectation) {
     // trusted.
     const gate = expectation.gating ? '' : '  (does not block completion)';
     const from = expectation.source === 'space' ? '  [space default]' : '';
-    return `  ${expectation.id.padEnd(18)} ${label.padEnd(14)} ${expectation.sentence}${gate}${from}`;
+    // Only shown when the work exists and the evidence is short — stating the
+    // required authority on every row would be noise on the rows where it is
+    // already satisfied or not yet relevant.
+    const evidence = expectation.state === 'unverified'
+        ? `  (has ${expectation.evidenceAuthority}, needs ${expectation.requiredAuthority})`
+        : '';
+    return `  ${expectation.id.padEnd(18)} ${label.padEnd(14)} ${expectation.sentence}${evidence}${gate}${from}`;
 }
 class TasksExpectations extends base_command_1.BaseCommand {
     static description = 'Show what a task must produce and how far each expectation got ("Done when")';
@@ -83,6 +93,15 @@ class TasksExpectations extends base_command_1.BaseCommand {
         if (settleable > 0) {
             this.log(`${settleable} awaiting a person — settle with \`${this.config.bin} tasks judge\`.`);
         }
+        // Named separately from `unjudged`, and with a different command, because
+        // the two look alike on the board and are not: one needs a verdict on prose,
+        // the other needs somebody to look at a thing that already exists.
+        const unverified = acceptance.expectations.filter((e) => e.state === 'unverified').length;
+        if (unverified > 0) {
+            this.log(`${unverified} produced but not vouched for — confirm with ` +
+                `\`${this.config.bin} tasks deliverable confirm\`, or lower the bar with ` +
+                `\`${this.config.bin} tasks trust\`.`);
+        }
         const inherited = acceptance.unjudged - settleable;
         if (inherited > 0) {
             this.log(`${inherited} inherited from the space default — declare them on the task to settle them.`);