@skrr-ai/cli 0.1.36 → 0.1.37

package/dist/commands/spaces/labels/remove.js

@@ -3,6 +3,7 @@ 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 label_ref_1 = require("../../../lib/label-ref");
 const web_url_1 = require("../../../lib/web-url");
 class SpacesLabelsRemove extends base_command_1.BaseCommand {
     static description = 'Detach a label from a space';
@@ -15,16 +16,35 @@ class SpacesLabelsRemove extends base_command_1.BaseCommand {
     };
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output the updated space as JSON' }),
-        label: core_1.Flags.string({ description: 'Label ID to detach', required: true }),
+        label: core_1.Flags.string({
+            description: 'Label — id, or a name from `skrr labels list --applicable-to space`. An unknown name is refused, never created.',
+            required: true,
+        }),
     };
     async run() {
         this.requireAuth();
         const { args, flags } = await this.parse(SpacesLabelsRemove);
         // Detach is intentionally permissive server-side: you can remove a label
         // you can no longer read, so a revoked label never gets stuck on a space.
+        // A name is resolved against the DECLARED vocabulary; an unknown one is
+        // refused with the near misses named, never created. See lib/label-ref.
+        let labelId = flags.label;
+        try {
+            [labelId] = await (0, label_ref_1.resolveLabelRefs)([flags.label], {
+                applicableTo: 'space',
+                ...(process.env.OVERSKY_WORKSPACE_ID
+                    ? { workspaceId: process.env.OVERSKY_WORKSPACE_ID }
+                    : {}),
+            });
+        }
+        catch (err) {
+            if (err instanceof label_ref_1.LabelRefError)
+                this.error(err.message, { exit: 1 });
+            this.handleApiError(err);
+        }
         let result;
         try {
-            result = await data_provider_1.dataService.removeLabelFromSpace({ spaceId: args.id, labelId: flags.label });
+            result = await data_provider_1.dataService.removeLabelFromSpace({ spaceId: args.id, labelId });
         }
         catch (err) {
             this.handleApiError(err);

package/dist/commands/spaces/show.js

@@ -52,6 +52,8 @@ class SpacesShow extends base_command_1.BaseCommand {
             this.log(`URL:         ${url}`);
         this.log(`Status:      ${s.status ?? '-'}`);
         this.log(`Color:       ${s.color ?? '-'}`);
+        this.log(`Icon:        ${s.icon ?? '-'}`);
+        this.log(`Icon Color:  ${s.iconColor ?? '-'}`);
         this.log(`Leader:      ${s.leadAgentId ?? '-'}`);
         this.log(`Target Date: ${s.targetDate ?? '-'}`);
         this.log(`Goals:       ${Array.isArray(s.goalIds) ? s.goalIds.length : 0}`);

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

@@ -12,6 +12,8 @@ export default class SpacesUpdate extends BaseCommand {
         description: 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>;
         color: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        icon: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'icon-color': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'lead-agent': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'target-date': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         goal: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
@@ -25,6 +27,9 @@ export default class SpacesUpdate extends BaseCommand {
         'integration-branch': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'auto-merge-tier': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'worktree-isolation': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'auto-start-on-backlog-exit': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        'triage-external-intake': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        'default-task-template': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
     };
     run(): Promise<void>;
 }

package/dist/commands/spaces/update.js

@@ -6,11 +6,19 @@ const tasks_1 = require("../../lib/tasks");
 const base_command_1 = require("../../base-command");
 const web_url_1 = require("../../lib/web-url");
 const prompt_1 = require("../../lib/prompt");
+/**
+ * Nullable PATCH fields take `none`/`null` as the clear sentinel — the same
+ * spelling `goals update` uses for --target-date/--project. `undefined` means
+ * the flag was not passed and mergeDefined leaves the field untouched.
+ */
+const clearable = (value) => value === undefined ? undefined : ['none', 'null'].includes(value.toLowerCase()) ? null : value;
 class SpacesUpdate extends base_command_1.BaseCommand {
     static description = 'Update an existing space (partial patch)';
     static examples = [
         '<%= config.bin %> spaces update <space-id> --title "New title"',
         '<%= config.bin %> spaces update <space-id> --status in_progress --color "#4f46e5"',
+        '<%= config.bin %> spaces update <space-id> --icon Rocket --icon-color "#2DD4BF"',
+        '<%= config.bin %> spaces update <space-id> --icon none',
         '<%= config.bin %> spaces update <space-id> --lead-agent <agent-id>',
         '<%= config.bin %> spaces update <space-id> --no-inherit-access',
         '<%= config.bin %> spaces update <space-id> --from-json patch.json',
@@ -28,8 +36,18 @@ class SpacesUpdate extends base_command_1.BaseCommand {
         description: core_1.Flags.string({ description: 'New description' }),
         status: core_1.Flags.string({ description: 'New status', options: [...data_provider_1.SPACE_STATUSES] }),
         color: core_1.Flags.string({ description: 'New color (hex)' }),
-        'lead-agent': core_1.Flags.string({ description: 'New Space Leader agent ID' }),
-        'target-date': core_1.Flags.string({ description: 'New target date (ISO string)' }),
+        icon: core_1.Flags.string({
+            description: 'New space icon — a curated icon name or an emoji. Pass "none" to restore the default folder.',
+        }),
+        'icon-color': core_1.Flags.string({
+            description: 'New color applied to the space icon (hex). Pass "none" to restore the neutral color.',
+        }),
+        'lead-agent': core_1.Flags.string({
+            description: 'New Space Leader agent ID. Pass "none" to clear the Leader.',
+        }),
+        'target-date': core_1.Flags.string({
+            description: 'New target date (ISO string). Pass "none" to clear it.',
+        }),
         goal: core_1.Flags.string({
             description: 'Link a goal to this space by ID (repeatable; additive). Routed through the ' +
                 'atomic link endpoint — re-linking an already-linked goal is a no-op.',
@@ -103,6 +121,20 @@ class SpacesUpdate extends base_command_1.BaseCommand {
                 'tree/branch. A per-task --worktree override wins. Merged into deliveryPolicy without ' +
                 'clobbering its other fields.',
         }),
+        'auto-start-on-backlog-exit': core_1.Flags.boolean({
+            allowNo: true,
+            description: 'Emit a nudge event when a task is dragged out of backlog, so the client can offer ' +
+                'a "Run now" toast. No silent execution.',
+        }),
+        'triage-external-intake': core_1.Flags.boolean({
+            allowNo: true,
+            description: 'Route externally sourced tasks (bots/channels) through the triage workflow state ' +
+                'instead of backlog. Enabling materializes the triage state server-side.',
+        }),
+        'default-task-template': core_1.Flags.string({
+            description: 'Default task template for new tasks in this space — a template id, reference, or ' +
+                'exact title (the server resolves it to the template). Pass "none" to clear.',
+        }),
     };
     async run() {
         this.requireAuth();
@@ -119,9 +151,14 @@ class SpacesUpdate extends base_command_1.BaseCommand {
             description: flags.description,
             status: flags.status,
             color: flags.color,
-            leadAgentId: flags['lead-agent'],
-            targetDate: flags['target-date'],
+            icon: clearable(flags.icon),
+            iconColor: clearable(flags['icon-color']),
+            leadAgentId: clearable(flags['lead-agent']),
+            targetDate: clearable(flags['target-date']),
             inheritAccess: flags['inherit-access'],
+            autoStartOnBacklogExit: flags['auto-start-on-backlog-exit'],
+            triageExternalIntake: flags['triage-external-intake'],
+            defaultTaskTemplateId: clearable(flags['default-task-template']),
             estimateScale: flags['estimate-scale'],
             estimateScaleMigration: flags['estimate-migration'],
             defaultEstimate: flags['default-estimate'],

package/dist/commands/spaces/workflows/lifecycle-diagnostics.d.ts

@@ -0,0 +1,11 @@
+import { BaseCommand } from '../../../base-command';
+export default class SpacesWorkflowsLifecycleDiagnostics extends BaseCommand {
+    static description: string;
+    static args: {
+        spaceId: 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/spaces/workflows/lifecycle-diagnostics.js

@@ -0,0 +1,45 @@
+"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");
+class SpacesWorkflowsLifecycleDiagnostics extends base_command_1.BaseCommand {
+    static description = 'Bounded lifecycle operational counters for a Space — pending effect age, repairs, transition conflicts, orphan stages';
+    static args = {
+        spaceId: core_1.Args.string({ required: true, ignoreStdin: true, description: 'Owning Space ID' }),
+    };
+    static flags = { json: core_1.Flags.boolean({ description: 'Output JSON' }) };
+    async run() {
+        this.requireAuth();
+        const { args, flags } = await this.parse(SpacesWorkflowsLifecycleDiagnostics);
+        let response;
+        try {
+            response = await data_provider_1.dataService.getSpaceLifecycleDiagnostics(args.spaceId);
+        }
+        catch (error) {
+            this.handleApiError(error);
+        }
+        if (flags.json)
+            return this.log(JSON.stringify(response, null, 2));
+        this.log(`Lifecycle diagnostics for space ${response.spaceId}:`);
+        this.log(`  bindings: ${response.bindings.bound} bound, ${response.bindings.active} active` +
+            `  (sampled ${response.sampledRuns} runs${response.truncated ? ', truncated' : ''})`);
+        const runStates = Object.entries(response.runs)
+            .map(([status, count]) => `${status}: ${count}`)
+            .join(', ');
+        this.log(`  runs: ${runStates || 'none'}`);
+        this.log(`  stages: ${response.stages.waiting} waiting` +
+            ` (oldest ${Math.round(response.stages.oldestWaitingMs / 1000)}s)` +
+            `, ${response.stages.completed} completed` +
+            ` (median ${Math.round(response.stages.durationMs.median / 1000)}s` +
+            `, max ${Math.round(response.stages.durationMs.max / 1000)}s)`);
+        this.log(`  effects: ${response.effects.pending} pending` +
+            ` (oldest ${Math.round(response.effects.oldestPendingMs / 1000)}s)` +
+            `, ${response.effects.repairs} repaired`);
+        this.log(`  transitions: ${response.transitions.applied} applied` +
+            `, ${response.transitions.gated} gated` +
+            `, ${response.transitions.superseded} superseded`);
+        this.log(`  orphan stages: ${response.orphans}`);
+    }
+}
+exports.default = SpacesWorkflowsLifecycleDiagnostics;

package/dist/commands/tasks/complete.d.ts

@@ -31,6 +31,7 @@ export default class TasksComplete extends BaseCommand {
         'run-id': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'spec-version': import("@oclif/core/lib/interfaces").OptionFlag<number | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'idempotency-key': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        force: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         'from-json': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
     };

package/dist/commands/tasks/complete.js

@@ -60,7 +60,9 @@ function describePendingReview({ statusType, taskId, resultId, bin, advisories,
     }
     return ('The Result was submitted, but the Task is not done yet. It is waiting on an independent ' +
         `accepted Review — someone OTHER than the submitter runs: ${bin} tasks review ${taskId} ` +
-        `--result-id ${resultId || '<result-id>'} --decision accepted`);
+        `--result-id ${resultId || '<result-id>'} --decision accepted. ` +
+        `(The Task owner, or a non-owner with ${bin} tasks move ${taskId} --status done --force, ` +
+        'may close without one — an override is recorded on the resolution.)');
 }
 class TasksComplete extends base_command_1.BaseCommand {
     static description = 'Submit one versioned Result and advance the Task workflow';
@@ -101,6 +103,9 @@ class TasksComplete extends base_command_1.BaseCommand {
             description: 'Task specification version this Result answers',
         }),
         'idempotency-key': core_1.Flags.string({ description: 'Stable retry key for this exact closeout' }),
+        force: core_1.Flags.boolean({
+            description: "Close past an owner-declared required Review as a non-owner; the override is recorded on the task's resolution",
+        }),
         'from-json': core_1.Flags.string({
             description: 'JSON file containing the same canonical Result fields; "-" for stdin',
         }),
@@ -188,6 +193,7 @@ class TasksComplete extends base_command_1.BaseCommand {
             expectedRunId: flags['run-id'],
             expectedSpecVersion: flags['spec-version'],
             idempotencyKey: flags['idempotency-key'],
+            force: flags.force === true ? true : undefined,
         });
         let status;
         let summary;
@@ -224,6 +230,7 @@ class TasksComplete extends base_command_1.BaseCommand {
             sourceRunIds: stringArray(input.sourceRunIds),
             expectedRunId: optionalString(input.expectedRunId),
             expectedSpecVersion: typeof input.expectedSpecVersion === 'number' ? input.expectedSpecVersion : undefined,
+            ...(input.force === true ? { force: true } : {}),
         };
         const idempotencyKey = optionalString(input.idempotencyKey) ||
             (0, outbox_1.idempotencyKeyFor)('task-complete', identity.id, JSON.stringify(payload));

package/dist/commands/tasks/create.d.ts

@@ -66,6 +66,9 @@ export default class TasksCreate extends BaseCommand {
         tag: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'label-id': import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         worktree: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        'lifecycle-definition': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'lifecycle-version': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'lifecycle-pack': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
     };
     run(): Promise<void>;
     private createTaskOrRecover;

package/dist/commands/tasks/create.js

@@ -6,6 +6,7 @@ exports.taskMatchesCreateIdentity = taskMatchesCreateIdentity;
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
+const label_ref_1 = require("../../lib/label-ref");
 const prompt_1 = require("../../lib/prompt");
 const tasks_1 = require("../../lib/tasks");
 const task_instruction_offer_1 = require("../../lib/task-instruction-offer");
@@ -220,8 +221,8 @@ class TasksCreate extends base_command_1.BaseCommand {
         }),
         tag: core_1.Flags.string({ description: 'Tag (repeatable)', multiple: true }),
         'label-id': core_1.Flags.string({
-            description: 'Attach a task label by id (repeatable). Manage labels with `skrr tasks labels`. ' +
-                'Every id must be an attachable label or the server rejects the create.',
+            description: 'Attach a task label (repeatable) — id, or a name from `skrr labels list --applicable-to task`. ' +
+                'An unknown name is refused, never created. Manage labels with `skrr labels`.',
             multiple: true,
         }),
         worktree: core_1.Flags.boolean({
@@ -230,6 +231,18 @@ class TasksCreate extends base_command_1.BaseCommand {
                 'or force the shared daemon tree (--no-worktree). Omit to inherit the space ' +
                 'policy / global default (shared).',
         }),
+        'lifecycle-definition': core_1.Flags.string({
+            description: 'Bind the created task to a task_lifecycle Workflow definition in its space. ' +
+                'Validated before the write; attach failure rolls the task back.',
+        }),
+        'lifecycle-version': core_1.Flags.string({
+            description: 'Exact published version to bind (requires --lifecycle-definition). ' +
+                'Omit to pin the definition’s current published version at create time.',
+        }),
+        'lifecycle-pack': core_1.Flags.string({
+            description: 'Bind the created task to a built-in lifecycle pack (e.g. finding, doc-to-task). ' +
+                'Alternative to --lifecycle-definition.',
+        }),
     };
     async run() {
         this.requireAuth();
@@ -260,8 +273,27 @@ class TasksCreate extends base_command_1.BaseCommand {
                 retryable: false,
             });
         }
+        // Resolve `--label-id` names before the create so a typo fails here, with
+        // the near misses named, rather than reaching the server as an id it will
+        // reject with no idea what was meant. Ids pass through untouched.
+        let resolvedLabelIds;
+        try {
+            const ids = await (0, label_ref_1.resolveLabelRefs)(flags['label-id'], {
+                applicableTo: 'task',
+                ...(process.env.OVERSKY_WORKSPACE_ID
+                    ? { workspaceId: process.env.OVERSKY_WORKSPACE_ID }
+                    : {}),
+            });
+            resolvedLabelIds = ids.length > 0 ? ids : undefined;
+        }
+        catch (err) {
+            if (err instanceof label_ref_1.LabelRefError)
+                this.error(err.message, { exit: 1 });
+            this.handleApiError(err);
+        }
         const payload = (0, tasks_1.buildTaskCreatePayload)(body, {
             ...flags,
+            'label-id': resolvedLabelIds,
             expectations: parsedExpectations.ok && parsedExpectations.value.length > 0
                 ? parsedExpectations.value
                 : undefined,

package/dist/commands/tasks/expectations/assess.d.ts

@@ -15,7 +15,18 @@ export default class TasksExpectationsAssess extends BaseCommand {
         evidence: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'spec-version': import("@oclif/core/lib/interfaces").OptionFlag<number | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'idempotency-key': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        all: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
     };
+    /**
+     * Which Expectations `--all` covers.
+     *
+     * ONLY the ones nothing has settled. An already-assessed Expectation carries
+     * somebody's judgement, and a bulk pass that overwrote it would silently
+     * replace a considered `unmet` with a blanket `met` — which is the one thing a
+     * batch verb must never do. Overriding an existing assessment stays a
+     * deliberate, single, named act.
+     */
+    private awaitingIds;
     run(): Promise<void>;
 }

package/dist/commands/tasks/expectations/assess.js

@@ -41,21 +41,50 @@ class TasksExpectationsAssess extends base_command_1.BaseCommand {
         }),
         'spec-version': core_1.Flags.integer({ description: 'Task specification version being assessed' }),
         'idempotency-key': core_1.Flags.string({ description: 'Stable retry key' }),
+        all: core_1.Flags.boolean({
+            description: 'Assess every Expectation still awaiting one on this Task, with the same decision ' +
+                'and reason. Already-settled Expectations are left alone.',
+        }),
         json: core_1.Flags.boolean({ description: 'Output JSON' }),
     };
+    /**
+     * Which Expectations `--all` covers.
+     *
+     * ONLY the ones nothing has settled. An already-assessed Expectation carries
+     * somebody's judgement, and a bulk pass that overwrote it would silently
+     * replace a considered `unmet` with a blanket `met` — which is the one thing a
+     * batch verb must never do. Overriding an existing assessment stays a
+     * deliberate, single, named act.
+     */
+    async awaitingIds(taskId) {
+        const acceptance = await data_provider_1.dataService.getTaskAcceptance(taskId);
+        return acceptance.expectations
+            .filter((expectation) => expectation.gating && !expectation.met && !expectation.assessmentId)
+            .map((expectation) => expectation.id);
+    }
     async run() {
         this.requireAuth();
         const { args, flags } = await this.parse(TasksExpectationsAssess);
         // `assess <task> <expectation>` gives both; `assess <expectation>` inside a
         // task runtime gives one, and it is the Expectation.
         const expectationId = args.expectation ?? args.task;
-        if (!expectationId) {
-            this.error('An Expectation id is required: `tasks expectations assess <task> <expectation>`', {
+        if (!expectationId && !flags.all) {
+            this.error('An Expectation id is required: `tasks expectations assess <task> <expectation>`, ' +
+                'or pass --all to settle every Expectation still awaiting one.', {
                 exit: 1,
             });
         }
+        /*
+         * Which positional is the TASK.
+         *
+         * `assess <task> <expectation>` gives both, so the first is the Task.
+         * `assess <expectation>` inside a task runtime gives one, and it is the
+         * Expectation — there is no Task positional to read. With `--all` there is
+         * no Expectation to name at all, so a lone positional is the Task again.
+         */
+        const taskPositional = flags.all || args.expectation ? args.task : undefined;
         const task = (0, task_resolver_1.resolveTaskIdentity)({
-            positionalId: args.expectation ? args.task : undefined,
+            positionalId: taskPositional,
             flagTask: flags.task,
         });
         const evidenceRefs = (flags.evidence || []).map((raw) => {
@@ -75,22 +104,64 @@ class TasksExpectationsAssess extends base_command_1.BaseCommand {
             evidenceRefs,
             expectedSpecVersion: flags['spec-version'],
         };
-        let result;
-        try {
+        const reuseCallerKey = Boolean(flags['idempotency-key']) && !flags.all;
+        const submit = async (id) => {
             const data = {
                 ...payload,
-                expectationId,
-                idempotencyKey: flags['idempotency-key'] ||
-                    (0, outbox_1.idempotencyKeyFor)('task-expectation-assessment', task.id, JSON.stringify({ expectationId, ...payload })),
+                expectationId: id,
+                // A per-expectation key under `--all`: one caller-supplied key reused
+                // across the set would make the second write a duplicate of the first,
+                // settling exactly one Expectation while reporting several.
+                idempotencyKey: reuseCallerKey
+                    ? flags['idempotency-key']
+                    : (0, outbox_1.idempotencyKeyFor)('task-expectation-assessment', task.id, JSON.stringify({ expectationId: id, ...payload })),
             };
             const sessionPath = await (0, session_task_endpoints_1.sessionScopedTaskPath)('assessment');
-            result = sessionPath
+            return sessionPath
                 ? await (0, api_fetch_1.apiFetch)(sessionPath, { method: 'POST', body: data })
-                : await data_provider_1.dataService.assessTaskExpectation({
-                    taskId: task.id,
-                    expectationId,
-                    data,
-                });
+                : await data_provider_1.dataService.assessTaskExpectation({ taskId: task.id, expectationId: id, data });
+        };
+        if (flags.all) {
+            let ids;
+            try {
+                ids = await this.awaitingIds(task.id);
+            }
+            catch (error) {
+                this.handleApiError(error);
+                return;
+            }
+            if (ids.length === 0) {
+                this.log('Nothing is awaiting assessment on this Task.');
+                return;
+            }
+            // Each write is reported on its own. A partial failure must leave the
+            // caller knowing exactly which Expectations were settled, because the
+            // remedy is to re-run for the rest — a single rolled-up "failed" would
+            // make that impossible to work out.
+            const settled = [];
+            const failed = [];
+            for (const id of ids) {
+                try {
+                    await submit(id);
+                    settled.push(id);
+                }
+                catch (error) {
+                    failed.push({ id, message: error?.message || 'unknown error' });
+                }
+            }
+            if (flags.json) {
+                return this.log(JSON.stringify({ decision: flags.decision, settled, failed }, null, 2));
+            }
+            this.log(`Recorded ${settled.length} Expectation(s) as ${flags.decision}: ${settled.join(', ')}`);
+            for (const failure of failed)
+                this.log(`  FAILED ${failure.id}: ${failure.message}`);
+            if (failed.length > 0)
+                this.exit(1);
+            return;
+        }
+        let result;
+        try {
+            result = await submit(expectationId);
         }
         catch (error) {
             this.handleApiError(error);

package/dist/commands/tasks/expectations.js

@@ -4,6 +4,7 @@ const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const data_provider_2 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
+const task_closure_1 = require("../../lib/task-closure");
 /**
  * `skrr tasks expectations <task>` — what this task owes, and how far each got.
  *
@@ -62,8 +63,24 @@ function evidenceGap(expectation) {
         return '';
     return `  (has ${expectation.evidenceLevel}, needs ${expectation.requiredEvidence})`;
 }
+/**
+ * `done (agent)` rather than `done`.
+ *
+ * An agent may assess any Expectation, so the control that keeps this honest is
+ * not permission — it is that a reader can always see whose judgement closed a
+ * row and decide how much to trust it. Printed only on rows an assessment
+ * actually settled; a state derived from the deliverable ledger has no assessor
+ * and must not be given a fake one.
+ */
+function assessorSuffix(expectation) {
+    if (!expectation.assessedByKind)
+        return '';
+    if (!['met', 'waived'].includes(expectation.state))
+        return '';
+    return ` (${(0, task_closure_1.describeAssessor)(expectation.assessedByKind)})`;
+}
 function line(expectation) {
-    const label = STATE_LABEL[expectation.state] || expectation.state;
+    const label = (STATE_LABEL[expectation.state] || expectation.state) + assessorSuffix(expectation);
     const from = expectation.source === 'space' ? '  [space default]' : '';
     /*
      * `sentence` before `text`.
@@ -75,7 +92,7 @@ function line(expectation) {
      * thing is owed. The fallback is for a row that predates the resolved field.
      */
     const sentence = expectation.sentence || expectation.text;
-    return `  ${expectation.id.padEnd(18)} ${label.padEnd(18)} ${sentence}${evidenceGap(expectation)}${from}`;
+    return `  ${expectation.id.padEnd(18)} ${label.padEnd(22)} ${sentence}${evidenceGap(expectation)}${from}`;
 }
 class TasksExpectations extends base_command_1.BaseCommand {
     static description = 'Show what must be true before the Task can close and the evidence-backed assessment state';

package/dist/commands/tasks/labels/attach.js

@@ -3,6 +3,7 @@ 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 label_ref_1 = require("../../../lib/label-ref");
 /**
  * Incremental label attach. `dataService.updateTask` only accepts a whole-list
  * `labelIds: string[]`, so we fetch the current task, append the new label
@@ -27,7 +28,7 @@ class TasksLabelsAttach extends base_command_1.BaseCommand {
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output the updated task as JSON' }),
         'label-id': core_1.Flags.string({
-            description: 'Label id to attach (repeatable). Create labels with `skrr tasks labels create`.',
+            description: 'Label to attach — id, or a name from `skrr labels list --applicable-to task`. An unknown name is refused, never created (repeatable).',
             multiple: true,
             required: true,
         }),
@@ -35,10 +36,26 @@ class TasksLabelsAttach extends base_command_1.BaseCommand {
     async run() {
         this.requireAuth();
         const { args, flags } = await this.parse(TasksLabelsAttach);
-        const incoming = (flags['label-id'] ?? []).map((id) => id.trim()).filter((id) => id.length > 0);
-        if (incoming.length === 0) {
+        const refs = (flags['label-id'] ?? []).map((id) => id.trim()).filter((id) => id.length > 0);
+        if (refs.length === 0) {
             this.error('At least one --label-id is required.', { exit: 1 });
         }
+        // A name is resolved against the DECLARED vocabulary; an unknown one is
+        // refused with the near misses named, never created. See lib/label-ref.
+        let incoming = [];
+        try {
+            incoming = await (0, label_ref_1.resolveLabelRefs)(refs, {
+                applicableTo: 'task',
+                ...(process.env.OVERSKY_WORKSPACE_ID
+                    ? { workspaceId: process.env.OVERSKY_WORKSPACE_ID }
+                    : {}),
+            });
+        }
+        catch (err) {
+            if (err instanceof label_ref_1.LabelRefError)
+                this.error(err.message, { exit: 1 });
+            this.handleApiError(err);
+        }
         let current;
         try {
             current = await data_provider_1.dataService.getTask(args.id, { enrich: false });

package/dist/commands/tasks/labels/detach.js

@@ -3,6 +3,7 @@ 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 label_ref_1 = require("../../../lib/label-ref");
 /**
  * Incremental label detach. Mirror of `labels attach` — fetch, filter, write
  * back the whole `labelIds` list. Matches label ids exactly (they are opaque
@@ -20,7 +21,7 @@ class TasksLabelsDetach extends base_command_1.BaseCommand {
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output the updated task as JSON' }),
         'label-id': core_1.Flags.string({
-            description: 'Label id to detach (repeatable)',
+            description: 'Label to detach — id, or a name from `skrr labels list --applicable-to task` (repeatable)',
             multiple: true,
             required: true,
         }),
@@ -28,10 +29,27 @@ class TasksLabelsDetach extends base_command_1.BaseCommand {
     async run() {
         this.requireAuth();
         const { args, flags } = await this.parse(TasksLabelsDetach);
-        const targets = new Set((flags['label-id'] ?? []).map((id) => id.trim()).filter((id) => id.length > 0));
-        if (targets.size === 0) {
+        const refs = (flags['label-id'] ?? []).map((id) => id.trim()).filter((id) => id.length > 0);
+        if (refs.length === 0) {
             this.error('At least one --label-id is required.', { exit: 1 });
         }
+        // A name is resolved against the DECLARED vocabulary; an unknown one is
+        // refused with the near misses named, never created. See lib/label-ref.
+        let resolvedTargets = [];
+        try {
+            resolvedTargets = await (0, label_ref_1.resolveLabelRefs)(refs, {
+                applicableTo: 'task',
+                ...(process.env.OVERSKY_WORKSPACE_ID
+                    ? { workspaceId: process.env.OVERSKY_WORKSPACE_ID }
+                    : {}),
+            });
+        }
+        catch (err) {
+            if (err instanceof label_ref_1.LabelRefError)
+                this.error(err.message, { exit: 1 });
+            this.handleApiError(err);
+        }
+        const targets = new Set(resolvedTargets);
         let current;
         try {
             current = await data_provider_1.dataService.getTask(args.id, { enrich: false });

package/dist/commands/tasks/lifecycle/attach.d.ts

@@ -0,0 +1,14 @@
+import { BaseCommand } from '../../../base-command';
+export default class TasksLifecycleAttach extends BaseCommand {
+    static description: string;
+    static args: {
+        id: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+    };
+    static flags: {
+        pack: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        definition: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        version: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+    };
+    run(): Promise<void>;
+}