@skrr-ai/cli 0.1.42 → 0.1.44

package/dist/commands/tasks/complete.js

@@ -1,4 +1,37 @@
 "use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.describePendingReview = describePendingReview;
 const core_1 = require("@oclif/core");
@@ -97,6 +130,11 @@ class TasksComplete extends base_command_1.BaseCommand {
             description: 'Exact Task Resource link id produced by `tasks deliverable add`',
         }),
         evidence: core_1.Flags.string({ multiple: true, description: 'Evidence as <kind>:<id-or-uri>' }),
+        attach: core_1.Flags.string({
+            multiple: true,
+            description: 'Upload a file as a Task asset (role: evidence) and cite it in evidenceRefs. ' +
+                'Repeatable. Re-attaching identical bytes dedupes on the server.',
+        }),
         remaining: core_1.Flags.string({ multiple: true, description: 'Remaining work item' }),
         'source-run': core_1.Flags.string({ multiple: true, description: 'Contributing Task Run id' }),
         'run-id': core_1.Flags.string({ description: 'Expected current Task Run id' }),
@@ -214,6 +252,23 @@ class TasksComplete extends base_command_1.BaseCommand {
                 retryable: false,
             });
         }
+        // --attach: upload once the input is known to be valid — a closeout the CLI
+        // is about to refuse must not leave stored bytes behind — and before the
+        // idempotency key is derived, so a retry sees the same refs.
+        const evidenceRefs = Array.isArray(input.evidenceRefs) ? [...input.evidenceRefs] : [];
+        if (flags.attach?.length) {
+            const { attachTaskAssetFiles } = await Promise.resolve().then(() => __importStar(require('../../lib/task-asset-upload')));
+            try {
+                evidenceRefs.push(...(await attachTaskAssetFiles({
+                    taskId: identity.id,
+                    files: flags.attach,
+                    role: 'evidence',
+                })));
+            }
+            catch (err) {
+                this.error(`--attach failed: ${err.message}`, { exit: 1 });
+            }
+        }
         const payload = {
             status,
             summary,
@@ -223,7 +278,7 @@ class TasksComplete extends base_command_1.BaseCommand {
             resourceLinkIds: Array.isArray(input.resourceLinkIds)
                 ? stringArray(input.resourceLinkIds)
                 : [],
-            evidenceRefs: Array.isArray(input.evidenceRefs) ? input.evidenceRefs : [],
+            evidenceRefs,
             remainingWork: Array.isArray(input.remainingWork) ? input.remainingWork : [],
             contributors: Array.isArray(input.contributors)
                 ? input.contributors

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

@@ -71,6 +71,12 @@ export default class TasksCreate extends BaseCommand {
         'lifecycle-pack': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
     };
     run(): Promise<void>;
+    /**
+     * A create refusal, translated. `TASK_GOAL_NOT_FOUND` names the goal id this
+     * command sent and the flag that gets past it, where the server's sentence
+     * names API fields (OSK-10219); everything else is the uniform translation.
+     */
+    private failTaskCreate;
     private createTaskOrRecover;
     /**
      * Predict the create route's `ASSIGNEE_NOT_IN_SPACE` degradation for a

package/dist/commands/tasks/create.js

@@ -333,7 +333,7 @@ class TasksCreate extends base_command_1.BaseCommand {
                 instantiated = (await data_provider_1.dataService.createTask(payload));
             }
             catch (err) {
-                this.handleApiError(err);
+                this.failTaskCreate(err, payload, flags.goal !== undefined);
             }
             const root = instantiated.rootTask ||
                 (instantiated.rootTasks || [])[0];
@@ -475,7 +475,7 @@ class TasksCreate extends base_command_1.BaseCommand {
                 }
             }
             if (!(defaultAgentResolution.applied && failureCode === 'ASSIGNEE_NOT_IN_SPACE')) {
-                this.handleApiError(err);
+                this.failTaskCreate(err, payload, flags.goal !== undefined);
             }
             delete payload.assigneeAgentIds;
             delete payload.assigneeUserIds;
@@ -502,7 +502,7 @@ class TasksCreate extends base_command_1.BaseCommand {
                 createRecovery = result.recovery;
             }
             catch (retryErr) {
-                this.handleApiError(retryErr);
+                this.failTaskCreate(retryErr, payload, flags.goal !== undefined);
             }
         }
         const url = (0, web_url_1.taskUrlFrom)(this.cliConfig.baseURL, created);
@@ -537,6 +537,29 @@ class TasksCreate extends base_command_1.BaseCommand {
             this.log(`View it at ${url}`);
         await (0, task_instruction_offer_1.maybeOfferTaskManagementInstructions)((message) => this.log(message));
     }
+    /**
+     * A create refusal, translated. `TASK_GOAL_NOT_FOUND` names the goal id this
+     * command sent and the flag that gets past it, where the server's sentence
+     * names API fields (OSK-10219); everything else is the uniform translation.
+     */
+    failTaskCreate(err, payload, goalFromFlag) {
+        const goalId = payload.goalId;
+        if ((0, base_command_1.apiRefusal)(err).code === tasks_1.TASK_GOAL_NOT_FOUND && typeof goalId === 'string' && goalId) {
+            this.failWithCliError({
+                message: (0, tasks_1.describeTaskGoalNotFound)({
+                    goalId,
+                    command: 'create',
+                    bin: this.config.bin,
+                    goalSource: goalFromFlag ? 'flag' : 'json',
+                }),
+                code: tasks_1.TASK_GOAL_NOT_FOUND,
+                status: 400,
+                exit: 1,
+                retryable: false,
+            });
+        }
+        this.handleApiError(err);
+    }
     async createTaskOrRecover(payload) {
         try {
             return {

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

@@ -28,6 +28,10 @@ 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({
+            // `--label` is what `spaces labels add` takes and what the managed
+            // task-management instructions tell every agent to type; without the
+            // alias that documented command died at parse.
+            aliases: ['label'],
             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,

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

@@ -58,6 +58,7 @@ export default class TasksList 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>;
         q: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        view: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         limit: import("@oclif/core/lib/interfaces").OptionFlag<number, import("@oclif/core/lib/interfaces").CustomOptions>;
         cursor: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         calendar: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;

package/dist/commands/tasks/list.js

@@ -48,6 +48,7 @@ const FILTER_FLAGS = new Set([
     'tag',
     'label-id',
     'q',
+    'view',
 ]);
 /**
  * What the zero-row case is allowed to say. The bare `No tasks.` was step 3
@@ -182,6 +183,7 @@ function buildListParams(flags, assigneeUserId) {
             ? { labelIds: flag('label-id') }
             : {}),
         q: flag('q'),
+        ...(flag('view') ? { taskViewId: flag('view') } : {}),
         limit: flag('limit'),
         cursor: flag('cursor'),
     };
@@ -205,6 +207,7 @@ class TasksList extends base_command_1.BaseCommand {
         '<%= config.bin %> tasks list --actionable-by agent --actionable-for-agent <agent-id>',
         '<%= config.bin %> tasks list --needs-me',
         '<%= config.bin %> tasks list --dri me',
+        '<%= config.bin %> tasks list --view <task-view-id>',
         '<%= config.bin %> tasks list --calendar 2026-08-01..2026-08-31 --json',
     ];
     static flags = {
@@ -314,6 +317,12 @@ class TasksList extends base_command_1.BaseCommand {
             multiple: true,
         }),
         q: core_1.Flags.string({ description: 'Free-text search query' }),
+        // A saved Task View is a query; this is the one place outside the browser
+        // that runs it (OSK-4651). Resolved server-side by the same evaluator the
+        // view subscriptions use, over the tasks this caller may read.
+        view: core_1.Flags.string({
+            description: 'Only tasks a saved Task View selects (task_view_…; see `views list`). Combines with other filters',
+        }),
         limit: core_1.Flags.integer({ description: 'Max results', default: 50 }),
         cursor: core_1.Flags.string({ description: 'Pagination cursor from a previous page' }),
         calendar: core_1.Flags.string({

package/dist/commands/tasks/result/submit.d.ts

@@ -15,6 +15,7 @@ export default class TasksResultSubmit extends BaseCommand {
         'structured-json': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         deliverable: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         evidence: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        attach: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         remaining: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'run-id': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'source-run': import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
@@ -23,4 +24,6 @@ export default class TasksResultSubmit extends BaseCommand {
         json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
+    /** The two ways forward from a closed Task, named where the commands are known. */
+    private reopenRequired;
 }

package/dist/commands/tasks/result/submit.js

@@ -1,4 +1,37 @@
 "use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Object.defineProperty(exports, "__esModule", { value: true });
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
@@ -43,6 +76,11 @@ class TasksResultSubmit extends base_command_1.BaseCommand {
             multiple: true,
             description: 'Evidence reference as <kind>:<id-or-uri>',
         }),
+        attach: core_1.Flags.string({
+            multiple: true,
+            description: 'Upload a file as a Task asset (role: evidence) and cite it in evidenceRefs. ' +
+                'Repeatable. Re-attaching identical bytes dedupes on the server.',
+        }),
         remaining: core_1.Flags.string({ multiple: true, description: 'Remaining work item (repeatable)' }),
         'run-id': core_1.Flags.string({ description: 'Canonical Task Run that produced this Result' }),
         'source-run': core_1.Flags.string({ multiple: true, description: 'Contributing Run id (repeatable)' }),
@@ -85,6 +123,36 @@ class TasksResultSubmit extends base_command_1.BaseCommand {
             const value = raw.slice(split + 1);
             return /^https?:|^file:/.test(value) ? { kind, uri: value } : { kind, id: value };
         });
+        // --attach: upload first so the Result cites real asset ids. Uploads land
+        // before the idempotency key is derived, so a retry sees the same refs.
+        if (flags.attach?.length) {
+            const { attachTaskAssetFiles, readTaskForUpload } = await Promise.resolve().then(() => __importStar(require('../../../lib/task-asset-upload')));
+            // A closed Task refuses the Result, and the files would already be
+            // stored — indexed as unreferenced uploads nobody asked for. Ask first.
+            // The server's refusal below stays the authority; this only keeps a
+            // refusal we can already see from costing an upload.
+            let target;
+            try {
+                target = await readTaskForUpload(task.id);
+            }
+            catch (err) {
+                this.error(`--attach failed: ${err.message}`, { exit: 1 });
+            }
+            if ((0, task_closure_1.taskIsSettled)(target)) {
+                this.reopenRequired(task.id, 'This Task is closed, and a Result can only be submitted while it is open. ' +
+                    'Nothing was uploaded.');
+            }
+            try {
+                evidenceRefs.push(...(await attachTaskAssetFiles({
+                    taskId: task.id,
+                    files: flags.attach,
+                    role: 'evidence',
+                })));
+            }
+            catch (err) {
+                this.error(`--attach failed: ${err.message}`, { exit: 1 });
+            }
+        }
         const payload = {
             summary: flags.summary,
             body: flags.body,
@@ -127,14 +195,7 @@ class TasksResultSubmit extends base_command_1.BaseCommand {
             // (OSK-10031).
             const refusal = (0, base_command_1.apiRefusal)(error);
             if (refusal.code === 'TASK_REOPEN_REQUIRED') {
-                this.error([
-                    refusal.message ?? 'This Task is closed.',
-                    ...(0, task_closure_1.reopenRequiredLines)({
-                        bin: this.config.bin,
-                        task: task.id,
-                        command: 'result submit',
-                    }),
-                ].join('\n'), { exit: 1, code: 'TASK_REOPEN_REQUIRED' });
+                this.reopenRequired(task.id, refusal.message ?? 'This Task is closed.');
             }
             this.handleApiError(error);
         }
@@ -154,5 +215,12 @@ class TasksResultSubmit extends base_command_1.BaseCommand {
         if (url)
             this.log(`Task: ${url}`);
     }
+    /** The two ways forward from a closed Task, named where the commands are known. */
+    reopenRequired(taskId, message) {
+        this.error([
+            message,
+            ...(0, task_closure_1.reopenRequiredLines)({ bin: this.config.bin, task: taskId, command: 'result submit' }),
+        ].join('\n'), { exit: 1, code: 'TASK_REOPEN_REQUIRED' });
+    }
 }
 exports.default = TasksResultSubmit;

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

@@ -10,6 +10,29 @@ import { BaseCommand } from '../../base-command';
  * exists for them to show.
  */
 export declare function taskPointerLines(t: Record<string, unknown>, ref: string, bin?: string): string[];
+/**
+ * The `Labels:` value.
+ *
+ * `labels attach` writes ids, and without this line the only way to see
+ * whether one landed was to attach it again and read "No change — labels
+ * already attached" (OSK-10313). Names, because an id is not something a
+ * person recognises — but an id beats nothing, so a vocabulary that could not
+ * be read (or a label missing from it) degrades per entry rather than failing
+ * the command or hiding the label.
+ */
+export declare function labelsLine(labelIds?: string[], nameById?: Map<string, string>): string;
+/**
+ * The `Executor:` value — who does this task, and whether anyone said so.
+ *
+ * It printed `legacy / inferred` whenever `executionMode` was absent, which is
+ * the normal result of `tasks create` without `--execution-mode`. A task made
+ * ten seconds earlier by the most ordinary invocation there is read as old data
+ * made the deprecated way (OSK-4629). The executor is now the one the server
+ * acts on (`resolveTaskExecutor`, shared with `TaskActionability`), and the
+ * value says when it is implied by the assignees rather than declared, because
+ * reassigning the task changes only the implied kind.
+ */
+export declare function taskExecutorLabel(t: Record<string, unknown>, bin?: string): string;
 /**
  * The failure rows of `tasks show`, when there are any. (S3 dogfood finding)
  *

package/dist/commands/tasks/show.js

@@ -1,6 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.taskPointerLines = taskPointerLines;
+exports.labelsLine = labelsLine;
+exports.taskExecutorLabel = taskExecutorLabel;
 exports.taskFailureLines = taskFailureLines;
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
@@ -11,6 +13,7 @@ const task_closure_1 = require("../../lib/task-closure");
 const task_resolver_1 = require("../../lib/task-resolver");
 const web_url_1 = require("../../lib/web-url");
 const html_text_1 = require("../../lib/html-text");
+const label_ref_1 = require("../../lib/label-ref");
 const format_1 = require("../../lib/format");
 /**
  * Where to look next — as COMMANDS, not as facts about the task.
@@ -39,6 +42,46 @@ function taskPointerLines(t, ref, bin = 'skrr') {
         ...pointers.map(([command, what]) => `  ${command.padEnd(width)}  ${what}`),
     ];
 }
+/**
+ * The `Labels:` value.
+ *
+ * `labels attach` writes ids, and without this line the only way to see
+ * whether one landed was to attach it again and read "No change — labels
+ * already attached" (OSK-10313). Names, because an id is not something a
+ * person recognises — but an id beats nothing, so a vocabulary that could not
+ * be read (or a label missing from it) degrades per entry rather than failing
+ * the command or hiding the label.
+ */
+function labelsLine(labelIds, nameById) {
+    const ids = (labelIds ?? []).filter((id) => typeof id === 'string' && id.length > 0);
+    if (ids.length === 0)
+        return '-';
+    return ids.map((id) => nameById?.get(id) ?? id).join(', ');
+}
+/**
+ * The `Executor:` value — who does this task, and whether anyone said so.
+ *
+ * It printed `legacy / inferred` whenever `executionMode` was absent, which is
+ * the normal result of `tasks create` without `--execution-mode`. A task made
+ * ten seconds earlier by the most ordinary invocation there is read as old data
+ * made the deprecated way (OSK-4629). The executor is now the one the server
+ * acts on (`resolveTaskExecutor`, shared with `TaskActionability`), and the
+ * value says when it is implied by the assignees rather than declared, because
+ * reassigning the task changes only the implied kind.
+ */
+function taskExecutorLabel(t, bin = 'skrr') {
+    const executor = (0, data_provider_1.resolveTaskExecutor)(t);
+    switch (executor.source) {
+        case 'declared':
+            return executor.mode;
+        case 'assignees':
+            return `${executor.mode} (not declared; implied by the ${executor.mode} assignees)`;
+        case 'mixed-assignees':
+            return `not chosen — Human and Agent assignees; pick one with \`${bin} tasks update --execution-mode\``;
+        case 'unassigned':
+            return '-';
+    }
+}
 /**
  * The failure rows of `tasks show`, when there are any. (S3 dogfood finding)
  *
@@ -284,7 +327,7 @@ class TasksShow extends base_command_1.BaseCommand {
             }
             this.log(`Result:      ${priorLine}`);
         }
-        this.log(`Executor:    ${t.executionMode ?? 'legacy / inferred'}`);
+        this.log(`Executor:    ${taskExecutorLabel(t, this.config.bin)}`);
         this.log(`Priority:    ${t.priority ?? '-'}`);
         this.log(`Parent:      ${t.parentId ?? '-'}`);
         this.log(`Space:       ${t.spaceId ?? '-'}`);
@@ -293,6 +336,23 @@ class TasksShow extends base_command_1.BaseCommand {
         // was no way to tell an ignored flag from an honoured one.
         this.log(`Project:     ${t.projectId ?? '-'}`);
         this.log(`Goal:        ${t.goalId ?? '-'}`);
+        // Printed for the same reason Project is: `labels attach` writes ids, and
+        // without this line the only way to see whether one landed was to attach
+        // it again and read "No change — labels already attached" (OSK-10313).
+        // Names, because an id is not something a person recognises; a vocabulary
+        // read that fails prints the ids rather than failing the whole command.
+        const labelIds = Array.isArray(t.labelIds) ? t.labelIds : [];
+        let labelNames;
+        if (labelIds.length > 0) {
+            try {
+                const vocabulary = await (0, label_ref_1.collectLabelVocabulary)({ applicableTo: 'task' });
+                labelNames = new Map(vocabulary.map((row) => [row.id, row.name]));
+            }
+            catch {
+                // A vocabulary that cannot be read prints ids; see labelsLine.
+            }
+        }
+        this.log(`Labels:      ${labelsLine(labelIds, labelNames)}`);
         this.log(`Author:      ${t.author ?? '-'}`);
         const dri = t.dri;
         if (dri?.resolved) {

package/dist/commands/tasks/update.js

@@ -97,7 +97,9 @@ class TasksUpdate extends base_command_1.BaseCommand {
         // set of "move this task somewhere" options — which is how a reader
         // concludes the CLI cannot change a task's project at all (dogfood
         // OSK-4652). It can: `tasks move <id> --project <id|none>`.
-        goal: core_1.Flags.string({ description: 'Move task to a different goal' }),
+        goal: core_1.Flags.string({
+            description: 'Move task to a different goal (goal id), or "none" to detach it from its goal',
+        }),
         cycle: core_1.Flags.string({
             description: 'Set planning cycle by ID, or "current" / "next" in the task Space',
         }),
@@ -324,7 +326,7 @@ class TasksUpdate extends base_command_1.BaseCommand {
             // sees it and answers with the sentence that names `tasks move`.
             status: flags.status,
             duplicateOfId: flags['duplicate-of'],
-            goalId: flags.goal,
+            goalId: (0, tasks_2.resolveTaskGoalFlag)(flags.goal),
             cycleId: flags.cycle,
             spaceId: flags.space,
             assigneeAgentIds: flags['assignee-agent'],
@@ -401,6 +403,18 @@ class TasksUpdate extends base_command_1.BaseCommand {
             });
         }
         catch (err) {
+            // The server's refusal says "detach the goal with goalId: null", which is
+            // not something a CLI user can type. Name the id sent and `--goal none`.
+            const goalId = payload.goalId;
+            if ((0, base_command_1.apiRefusal)(err).code === tasks_2.TASK_GOAL_NOT_FOUND && typeof goalId === 'string' && goalId) {
+                this.failWithCliError({
+                    message: (0, tasks_2.describeTaskGoalNotFound)({ goalId, command: 'update', bin: this.config.bin }),
+                    code: tasks_2.TASK_GOAL_NOT_FOUND,
+                    status: 400,
+                    exit: 1,
+                    retryable: false,
+                });
+            }
             this.handleApiError(err);
         }
         // Postcondition, not a prediction — see `droppedEstimateWarning`. On stderr

package/dist/commands/tasks/updates/add.d.ts

@@ -15,6 +15,7 @@ export default class TasksUpdatesAdd extends BaseCommand {
         'spec-version': import("@oclif/core/lib/interfaces").OptionFlag<number | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         supersedes: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         reference: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        attach: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'idempotency-key': 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/updates/add.js

@@ -1,4 +1,37 @@
 "use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Object.defineProperty(exports, "__esModule", { value: true });
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
@@ -64,6 +97,12 @@ class TasksUpdatesAdd extends base_command_1.BaseCommand {
         }),
         supersedes: core_1.Flags.string({ description: 'Update id this correction supersedes' }),
         reference: core_1.Flags.string({ multiple: true, description: 'Reference as <kind>:<id-or-uri>' }),
+        attach: core_1.Flags.string({
+            multiple: true,
+            description: 'Upload a file as a Task asset (role: evidence) and cite it in this Update — appended to ' +
+                'data.evidenceRefs for finding/verification payloads, to the Update references otherwise. ' +
+                'Repeatable. Re-attaching identical bytes dedupes on the server.',
+        }),
         'idempotency-key': core_1.Flags.string({ description: 'Stable retry key for this exact Update' }),
         json: core_1.Flags.boolean({ description: 'Output JSON' }),
     };
@@ -88,6 +127,30 @@ class TasksUpdatesAdd extends base_command_1.BaseCommand {
             const value = raw.slice(split + 1);
             return /^https?:|^file:/.test(value) ? { kind, uri: value } : { kind, id: value };
         });
+        // --attach: upload first so the Update can cite the asset ids. For
+        // finding/verification the citation lands in data.evidenceRefs where the
+        // evidence renderer reads it; for the other Update kinds it becomes a
+        // task_attachment reference on the Update itself.
+        if (flags.attach?.length) {
+            const { attachTaskAssetFiles } = await Promise.resolve().then(() => __importStar(require('../../../lib/task-asset-upload')));
+            let refs;
+            try {
+                refs = await attachTaskAssetFiles({
+                    taskId: task.id,
+                    files: flags.attach,
+                    role: 'evidence',
+                });
+            }
+            catch (err) {
+                this.error(`--attach failed: ${err.message}`, { exit: 1 });
+            }
+            if (Array.isArray(data.evidenceRefs)) {
+                data = { ...data, evidenceRefs: [...data.evidenceRefs, ...refs] };
+            }
+            else {
+                references.push(...refs);
+            }
+        }
         const idempotencyKey = flags['idempotency-key'] ||
             (0, outbox_1.idempotencyKeyFor)('task-update', task.id, JSON.stringify({ type: flags.type, summary: flags.summary, data, resume }));
         let result;

package/dist/commands/views/create.js

@@ -19,7 +19,10 @@ class ViewsCreate extends base_command_1.BaseCommand {
         'from-json': core_1.Flags.string({ description: 'JSON Task View body' }),
         name: core_1.Flags.string({ description: 'View name' }),
         description: core_1.Flags.string({ description: 'Short explanation shown in the modal' }),
-        type: core_1.Flags.string({ options: [...vocabulary_1.VIEW_TYPES], description: 'Layout' }),
+        type: core_1.Flags.string({
+            options: [...vocabulary_1.VIEW_TYPES],
+            description: `Layout (default: ${vocabulary_1.TASK_VIEW_DEFAULT_TYPE})`,
+        }),
         'task-id': core_1.Flags.string({ multiple: true, description: 'Include this exact task ID' }),
         space: core_1.Flags.string({ multiple: true, description: 'Limit to this Space ID' }),
         // Status labels are per-Space. The API bounds and normalizes values but
@@ -41,7 +44,7 @@ class ViewsCreate extends base_command_1.BaseCommand {
         }),
         'group-by': core_1.Flags.string({
             options: [...vocabulary_1.VIEW_GROUP_BY_VALUES],
-            description: 'Grouping dimension',
+            description: `Grouping dimension (default: ${vocabulary_1.TASK_VIEW_DEFAULT_GROUP_BY})`,
         }),
         sort: core_1.Flags.string({
             options: [...vocabulary_1.VIEW_SORT_KEYS],
@@ -107,6 +110,13 @@ class ViewsCreate extends base_command_1.BaseCommand {
             return void this.log(JSON.stringify(result, null, 2));
         }
         this.log(`Created Task View ${view.id}`);
+        // What the SERVER saved, read back from the response — not an echo of the
+        // flags. Creation is the one moment the caller still knows what they meant
+        // to type, so it is where a filter that silently did not take has to show
+        // (dogfood OSK-4646). Same renderer as `views show`.
+        for (const line of (0, task_view_render_1.describeTaskView)(view)) {
+            this.log(line);
+        }
         this.log(result.url);
         if (warning)
             this.warn(warning);