@skrr-ai/cli 0.1.110 → 0.1.111

package/dist/commands/tasks/assign.js

@@ -62,6 +62,9 @@ class TasksAssign extends base_command_1.BaseCommand {
             description: 'Agent to assign (repeatable); name or agent id',
             multiple: true,
         }),
+        unassigned: core_1.Flags.boolean({
+            description: 'Clear all Agent and Human assignees while retaining executor intent; does not start a Run.',
+        }),
         replace: core_1.Flags.boolean({
             description: 'Replace existing assignees of the same kind instead of adding',
         }),
@@ -87,6 +90,21 @@ class TasksAssign extends base_command_1.BaseCommand {
     async run() {
         this.requireAuth();
         const { args, flags } = await this.parse(TasksAssign);
+        this.setJsonOperation({ command: 'tasks:assign', mode: 'execute' });
+        if (flags.unassigned &&
+            (flags.replace ||
+                flags.remove ||
+                flags.list ||
+                flags.suggest ||
+                args.person ||
+                flags.person?.length ||
+                flags.agent?.length))
+            this.failWithCliError({
+                message: '--unassigned clears both assignee kinds and cannot be combined with named assignees, --replace, --remove, --list or --suggest.',
+                code: 'TASK_ASSIGNMENT_FLAG_CONFLICT',
+                exit: 2,
+                retryable: false,
+            });
         if (flags.replace && flags.remove) {
             this.failWithCliError({
                 message: '--replace and --remove are mutually exclusive.',
@@ -112,7 +130,11 @@ class TasksAssign extends base_command_1.BaseCommand {
         const taskRef = await this.resolveEntityRef('task', taskIdentity.id);
         const personQueries = [...(args.person ? [args.person] : []), ...(flags.person ?? [])];
         const agentQueries = flags.agent ?? [];
-        if (!flags.list && !flags.suggest && personQueries.length === 0 && agentQueries.length === 0) {
+        if (!flags.unassigned &&
+            !flags.list &&
+            !flags.suggest &&
+            personQueries.length === 0 &&
+            agentQueries.length === 0) {
             // Positional order is `<task-id> <person>`. Inside a task runtime the
             // task id is implicit, so a single positional (`skrr tasks assign frank`)
             // is silently eaten as the TASK id and the user sees a confusing
@@ -174,6 +196,7 @@ class TasksAssign extends base_command_1.BaseCommand {
         }
         if (flags['execution-mode'])
             payload.executionMode = flags['execution-mode'];
+        (0, tasks_1.applyExplicitTaskUnassignment)(payload, flags.unassigned === true);
         const repairGuardFailure = (0, tasks_1.releasePinRepairGuard)(payload);
         if (repairGuardFailure) {
             this.failWithCliError({
@@ -194,17 +217,30 @@ class TasksAssign extends base_command_1.BaseCommand {
             this.handleApiError(err);
         }
         if (flags.json) {
-            this.log(JSON.stringify({
-                ...(0, web_url_1.withTaskUrl)(this.cliConfig.baseURL, updated),
-                // Keep request details separate from the canonical assignee arrays
-                // returned by the server, especially when removing co-assignees.
-                assignmentAction: flags.remove ? 'remove' : flags.replace ? 'replace' : 'add',
+            const row = updated;
+            this.printJsonOperation({
+                ...(0, web_url_1.withTaskUrl)(this.cliConfig.baseURL, row),
+                assignmentAction: flags.unassigned
+                    ? 'unassign_all'
+                    : flags.remove
+                        ? 'remove'
+                        : flags.replace
+                            ? 'replace'
+                            : 'add',
                 resolved: [...people.resolved, ...agents.resolved],
-            }, null, 2));
+                assignmentPreview: (0, tasks_1.taskAssignmentPreview)(row),
+            }, {
+                resource: row.id ? { kind: 'task', id: String(row.id) } : null,
+                task: row.id ? { id: String(row.id) } : null,
+            });
             return;
         }
         const t = updated;
         const ref = t.identifier ?? t.id ?? taskRef;
+        if (flags.unassigned) {
+            this.log(`Unassigned ${ref}; executor intent: ${(0, tasks_1.taskAssignmentPreview)(t).executor.mode || 'unspecified'}. No Run requested.`);
+            return;
+        }
         let verb = 'Assigned';
         if (flags.remove)
             verb = 'Unassigned';

package/dist/commands/tasks/attachments/list.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.attachmentRow = attachmentRow;
+const list_output_1 = require("../../../lib/list-output");
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../../base-command");
@@ -60,7 +61,7 @@ class TasksAttachmentsList extends base_command_1.BaseCommand {
             this.handleApiError(err);
         }
         if (flags.json) {
-            this.log(JSON.stringify(response, null, 2));
+            this.printJsonResult((0, list_output_1.listResult)(response));
             return;
         }
         const r = response;

package/dist/commands/tasks/chat/send.js

@@ -6,6 +6,8 @@ const node_crypto_1 = require("node:crypto");
 const core_1 = require("@oclif/core");
 const base_command_1 = require("../../../base-command");
 const task_extras_1 = require("../../../lib/task-extras");
+const http_error_1 = require("../../../lib/http-error");
+const shell_quote_1 = require("../../../lib/shell-quote");
 /** The server's `appendSchema` bounds on `commandId`. */
 const COMMAND_ID_MIN = 8;
 const COMMAND_ID_MAX = 128;
@@ -100,6 +102,38 @@ class TasksChatSend extends base_command_1.BaseCommand {
             result = await task_extras_1.taskChatApi.sendExchangeMessage(taskId, body);
         }
         catch (err) {
+            const refusal = (0, http_error_1.httpBodyOf)(err);
+            if ((0, http_error_1.httpStatusOf)(err) === 409 && refusal?.code === 'TASK_MANAGED_INPUT_REQUIRED') {
+                const contextArgs = [
+                    ...(typeof flags.profile === 'string' ? ['--profile', flags.profile] : []),
+                    ...(typeof flags.workspace === 'string' ? ['--workspace', flags.workspace] : []),
+                    ...(flags.bare ? ['--bare'] : []),
+                ];
+                const inspect = (0, shell_quote_1.shellQuoteArgv)([
+                    this.config.bin, 'tasks', 'execution-state', taskId, ...contextArgs,
+                ]);
+                const discuss = (0, shell_quote_1.shellQuoteArgv)([
+                    this.config.bin, 'tasks', 'continue', taskId, ...contextArgs, '--purpose', 'discuss',
+                ]);
+                const credentialHint = flags.bare || typeof flags.token === 'string'
+                    ? ' Supply credentials again through your normal secure source; token values are not printed.'
+                    : '';
+                const suggestion = `Inspect ${inspect} first. To discuss, run ${discuss} ` +
+                    '--text "<message>" --command-key "<stable-key>". ' +
+                    'For new work use --purpose continue_work instead. ' +
+                    'Reuse a command key only for an identical retry. Nothing was resent automatically.' +
+                    credentialHint;
+                const requestId = refusal.requestId ?? err?.requestId;
+                this.failWithCliError({
+                    message: `This Task uses managed inputs; this message was not accepted. ${suggestion}`,
+                    code: 'TASK_MANAGED_INPUT_REQUIRED',
+                    status: 409,
+                    exit: 1,
+                    retryable: false,
+                    details: { ...refusal, suggestion },
+                    ...(typeof requestId === 'string' ? { requestId } : {}),
+                });
+            }
             this.handleApiError(err);
         }
         if (flags.json) {

package/dist/commands/tasks/comments/add.js

@@ -97,6 +97,7 @@ class TasksCommentsAdd extends base_command_1.BaseCommand {
         }),
     };
     async run() {
+        this.setJsonOperation({ command: 'tasks:comments:add', mode: 'execute' });
         this.requireAuth();
         const { args, flags } = await this.parse(TasksCommentsAdd);
         let jsonBody = {};
@@ -265,7 +266,11 @@ class TasksCommentsAdd extends base_command_1.BaseCommand {
             });
         }
         if (flags.json) {
-            this.log(JSON.stringify(result, null, 2));
+            const comment = result;
+            this.printJsonOperation(result, {
+                resource: { kind: 'task_comment', id: typeof comment?.id === 'string' ? comment.id : null },
+                task: { id: typeof comment?.taskId === 'string' ? comment.taskId : null },
+            });
             return;
         }
         const c = result;

package/dist/commands/tasks/comments/list.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.threadComments = threadComments;
+const list_output_1 = require("../../../lib/list-output");
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../../base-command");
@@ -95,7 +96,7 @@ class TasksCommentsList extends base_command_1.BaseCommand {
             this.handleApiError(err);
         }
         if (flags.json) {
-            this.log(JSON.stringify(response, null, 2));
+            this.printJsonResult((0, list_output_1.listResult)(response));
             return;
         }
         const r = response;

package/dist/commands/tasks/complete.js

@@ -153,6 +153,7 @@ class TasksComplete extends base_command_1.BaseCommand {
         json: core_1.Flags.boolean({ description: 'Output JSON' }),
     };
     async run() {
+        this.setJsonOperation({ command: 'tasks:complete', mode: 'execute' });
         this.requireAuth();
         const { args, flags } = await this.parse(TasksComplete);
         let jsonBody = {};
@@ -259,7 +260,27 @@ class TasksComplete extends base_command_1.BaseCommand {
         // 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')));
+            const { attachTaskAssetFiles, readTaskForUpload } = await Promise.resolve().then(() => __importStar(require('../../lib/task-asset-upload')));
+            let target;
+            try {
+                target = await readTaskForUpload(identity.id);
+            }
+            catch (err) {
+                this.error(`--attach failed: ${err.message}`, { exit: 1 });
+            }
+            // A Result needs an open Task. Check with the uploader's credential before
+            // storing files for a refusal we can already see; the server still guards
+            // against a Task closing after this read.
+            if ((0, task_closure_1.taskIsSettled)(target)) {
+                this.error([
+                    'This Task is closed, and a Result can only be submitted while it is open. Nothing was uploaded.',
+                    ...(0, task_closure_1.reopenRequiredLines)({
+                        bin: this.config.bin,
+                        task: identity.id,
+                        command: 'complete',
+                    }),
+                ].join('\n'), { exit: 1, code: 'TASK_REOPEN_REQUIRED' });
+            }
             try {
                 evidenceRefs.push(...(await attachTaskAssetFiles({
                     taskId: identity.id,
@@ -332,7 +353,10 @@ class TasksComplete extends base_command_1.BaseCommand {
         }
         const url = (0, web_url_1.taskUrlFrom)(this.cliConfig.baseURL, { id: taskId });
         if (flags.json) {
-            this.log(JSON.stringify({ ...result, url }, null, 2));
+            this.printJsonOperation({ ...result, url }, {
+                resource: { kind: 'task_result', id: result.result.id },
+                task: { id: result.task.id },
+            });
             return;
         }
         this.log(`Submitted Result v${result.result.version}; Task is ${result.task.statusType || result.task.status || 'updated'}.`);

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

@@ -0,0 +1,18 @@
+import { BaseCommand } from '../../base-command';
+export { buildTaskContinuationBody, describeTaskContinuation } from '../../lib/task-work';
+export default class TasksContinue extends BaseCommand {
+    static description: string;
+    static args: {
+        id: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+    };
+    static flags: {
+        text: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        purpose: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'command-key': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        receipt: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        generation: import("@oclif/core/lib/interfaces").OptionFlag<number | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'authorization-generation': import("@oclif/core/lib/interfaces").OptionFlag<number | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+    };
+    run(): Promise<void>;
+}

package/dist/commands/tasks/continue.js

@@ -0,0 +1,54 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.describeTaskContinuation = exports.buildTaskContinuationBody = void 0;
+const core_1 = require("@oclif/core");
+const data_provider_1 = require("@skrr-ai/data-provider");
+const base_command_1 = require("../../base-command");
+const task_work_1 = require("../../lib/task-work");
+var task_work_2 = require("../../lib/task-work");
+Object.defineProperty(exports, "buildTaskContinuationBody", { enumerable: true, get: function () { return task_work_2.buildTaskContinuationBody; } });
+Object.defineProperty(exports, "describeTaskContinuation", { enumerable: true, get: function () { return task_work_2.describeTaskContinuation; } });
+class TasksContinue extends base_command_1.BaseCommand {
+    static description = 'Continue work or discuss the same Task through durable input admission';
+    static args = { id: core_1.Args.string({ required: true, ignoreStdin: true, description: 'Task ID or identifier' }) };
+    static flags = {
+        text: core_1.Flags.string({ description: 'Follow-up message; required unless reading a receipt' }),
+        purpose: core_1.Flags.string({ options: ['auto', 'continue_work', 'discuss'], default: 'continue_work', description: 'Continue work, discuss, or ask the server to classify' }),
+        'command-key': core_1.Flags.string({ description: 'Stable retry identity; reuse only for the identical input and generation' }),
+        receipt: core_1.Flags.boolean({ description: 'Read the current authenticated actor’s receipt for --command-key without sending another input' }),
+        generation: core_1.Flags.integer({ min: 1, description: 'Expected workflow generation; defaults to the current snapshot' }),
+        'authorization-generation': core_1.Flags.integer({ min: 1, description: 'Expected Task chat authorization generation; defaults to current state' }),
+        json: core_1.Flags.boolean({ description: 'Output the durable input receipt as JSON' }),
+    };
+    async run() {
+        this.requireAuth();
+        const { args, flags } = await this.parse(TasksContinue);
+        const taskId = await this.resolveTaskReference(args.id);
+        let receipt;
+        try {
+            if (flags.receipt) {
+                if (!flags['command-key'])
+                    this.error('--receipt requires --command-key.', { exit: 2 });
+                receipt = await data_provider_1.dataService.getTaskConversationInput(taskId, flags['command-key']);
+                this.log(flags.json ? JSON.stringify(receipt, null, 2) : (0, task_work_1.describeTaskContinuation)(receipt));
+                return;
+            }
+            if (!flags.text)
+                this.error('--text is required when sending a follow-up.', { exit: 2 });
+            const chat = await data_provider_1.dataService.getTaskChat(taskId);
+            const body = (0, task_work_1.buildTaskContinuationBody)({
+                text: flags.text,
+                commandKey: flags['command-key'],
+                purpose: flags.purpose,
+                workflowGeneration: flags.generation ?? chat.workflowGeneration,
+                authorizationGeneration: flags['authorization-generation'] ?? chat.authorizationGeneration ?? 1,
+            });
+            receipt = await data_provider_1.dataService.submitTaskConversationInput(taskId, body);
+        }
+        catch (error) {
+            this.handleApiError(error);
+        }
+        this.log(flags.json ? JSON.stringify(receipt, null, 2) : (0, task_work_1.describeTaskContinuation)(receipt));
+    }
+}
+exports.default = TasksContinue;

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

@@ -119,6 +119,9 @@ export default class TasksCreateTree extends BaseCommand {
         'from-json': import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
         'dry-run': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         verify: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        'verify-spec': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        readback: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        unassigned: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         'external-key-prefix': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         fresh: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         resume: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
@@ -129,6 +132,8 @@ export default class TasksCreateTree extends BaseCommand {
         project: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
     };
     run(): Promise<void>;
+    private readbackSpecifications;
+    private printSpecificationSummary;
     /**
      * `--verify`: which nodes of this tree exist, and which do not. Read-only by
      * construction — it is reached before anything that writes — so it is safe to