@skrr-ai/cli 0.1.107 → 0.1.109

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

@@ -105,6 +105,8 @@ export default class TasksList extends BaseCommand {
         subscribed: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         'include-snoozed': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         'external-key': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'external-key-prefix': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'description-format': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         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>;

package/dist/commands/tasks/list.js

@@ -48,6 +48,7 @@ const FILTER_FLAGS = new Set([
     'subscribed',
     'include-snoozed',
     'external-key',
+    'external-key-prefix',
     'blocked-by',
     'blocks',
     'due-before',
@@ -286,6 +287,13 @@ function buildListParams(flags, assigneeUserId) {
         ...(flag('external-key')
             ? { externalKey: flag('external-key') }
             : {}),
+        ...(flag('external-key-prefix')
+            ? { externalKeyPrefix: flag('external-key-prefix') }
+            : {}),
+        // A presentation of each row, not a filter: it is deliberately not in FILTER_FLAGS.
+        ...(flag('description-format') === 'markdown'
+            ? { descriptionFormat: 'markdown' }
+            : {}),
         ...(flag('blocked-by') ? { blockedBy: flag('blocked-by') } : {}),
         ...(flag('blocks') ? { blocks: flag('blocks') } : {}),
         ...(flag('due-before') ? { dueBefore: flag('due-before') } : {}),
@@ -469,6 +477,13 @@ class TasksList extends base_command_1.BaseCommand {
             description: 'Include tasks you have personally snoozed (hidden by default)',
         }),
         'external-key': core_1.Flags.string({ description: 'Filter by external key' }),
+        'external-key-prefix': core_1.Flags.string({
+            description: 'Filter to tasks whose external key starts with this literal prefix, e.g. every task of one tree written under --external-key-prefix. Combine with --space; see also `tasks create-tree --verify`.',
+        }),
+        'description-format': core_1.Flags.string({
+            description: 'JSON/text: how to read each description. html (default) is what is stored; markdown returns it as it was written, so a description you wrote compares equal to the one you read back.',
+            options: ['html', 'markdown'],
+        }),
         tag: core_1.Flags.string({
             description: 'Filter to tasks carrying EVERY listed tag (repeatable, AND-intersection, ' +
                 'case-insensitive)',
@@ -711,7 +726,8 @@ class TasksList extends base_command_1.BaseCommand {
         if (flags.rank)
             this.log('\nRanked only within this fetched page; estimates are claims, not measured savings.\n' +
                 'SCORE = annualized USD savings / max(effort hours, 0.5) x risk factor (low 1, medium 0.6, high 0.25); ' +
-                '— means savings, effort or risk is missing, so the row is not scored and sorts last.');
+                '— means savings, effort or risk is missing (or outside the plausible range), so the row is not scored ' +
+                'and sorts after every scored row: larger stated savings first, then rows with no estimate.');
         if (r?.next_cursor) {
             this.log('');
             this.log(`Next page: --cursor ${r.next_cursor}`);
@@ -719,6 +735,8 @@ class TasksList extends base_command_1.BaseCommand {
         const note = (0, list_output_1.limitNote)(flags.limit, items.length, Boolean(r?.has_more));
         for (const line of (0, list_output_1.warningLines)(note.warning ? [note.warning] : []))
             this.log(line);
+        for (const line of (0, list_output_1.textModeProjectionNotes)(flags))
+            this.log(line);
     }
 }
 exports.default = TasksList;

package/dist/commands/tasks/rearm-recurrence.d.ts

@@ -0,0 +1,22 @@
+import { BaseCommand } from '../../base-command';
+/**
+ * Continue a recurring series whose next occurrence was never armed (OSK-15338).
+ *
+ * The failure (`recurrence.armFailure`) is recorded on the FINISHED occurrence, and `tasks schedule`
+ * refuses a finished Task, so this is the command every surface names for the way back. It arms the
+ * occurrence the series is owed and clears the failure; `--cron` / `--timezone` correct the series
+ * first, which is the only cure for an `invalid_pattern` failure.
+ */
+export default class TasksRearmRecurrence extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static args: {
+        id: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+    };
+    static flags: {
+        json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        cron: 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>;
+    };
+    run(): Promise<void>;
+}

package/dist/commands/tasks/rearm-recurrence.js

@@ -0,0 +1,68 @@
+"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 web_url_1 = require("../../lib/web-url");
+/**
+ * Continue a recurring series whose next occurrence was never armed (OSK-15338).
+ *
+ * The failure (`recurrence.armFailure`) is recorded on the FINISHED occurrence, and `tasks schedule`
+ * refuses a finished Task, so this is the command every surface names for the way back. It arms the
+ * occurrence the series is owed and clears the failure; `--cron` / `--timezone` correct the series
+ * first, which is the only cure for an `invalid_pattern` failure.
+ */
+class TasksRearmRecurrence extends base_command_1.BaseCommand {
+    static description = 'Re-arm a recurring series from the finished occurrence whose next occurrence was never scheduled';
+    static examples = [
+        '<%= config.bin %> tasks rearm-recurrence OSK-42',
+        '<%= config.bin %> tasks rearm-recurrence OSK-42 --cron "0 9 * * 1-5" --timezone America/New_York',
+    ];
+    static args = {
+        id: core_1.Args.string({
+            description: 'The finished occurrence (Task ID or identifier) that carries the arm failure',
+            required: true,
+            ignoreStdin: true,
+        }),
+    };
+    static flags = {
+        json: core_1.Flags.boolean({ description: 'Output as JSON' }),
+        cron: core_1.Flags.string({
+            description: 'Correct the series pattern first: five cron fields "minute hour day-of-month month day-of-week", e.g. "0 9 * * 1-5"',
+        }),
+        timezone: core_1.Flags.string({
+            description: 'Correct the series timezone first (IANA zone, e.g. America/New_York)',
+        }),
+    };
+    async run() {
+        this.requireAuth();
+        const { args, flags } = await this.parse(TasksRearmRecurrence);
+        const taskRef = await this.resolveEntityRef('task', args.id);
+        let result;
+        try {
+            result = await data_provider_1.dataService.rearmTaskRecurrence({
+                taskId: taskRef,
+                pattern: flags.cron,
+                timezone: flags.timezone,
+            });
+        }
+        catch (err) {
+            this.handleApiError(err);
+        }
+        if (flags.json) {
+            this.log(JSON.stringify({
+                ...result,
+                task: (0, web_url_1.withTaskUrl)(this.cliConfig.baseURL, result.task),
+            }, null, 2));
+            return;
+        }
+        const finished = result.task;
+        const from = finished.identifier || finished.id || taskRef;
+        const next = result.successor.identifier || result.successor.id;
+        const when = result.successor.scheduledFor ? ` for ${result.successor.scheduledFor}` : '';
+        this.log(result.alreadyArmed
+            ? `The series from ${from} was already armed: next occurrence ${next}${when}. The failure marker is cleared.`
+            : `Re-armed the series from ${from}: next occurrence ${next} is scheduled${when}.`);
+    }
+}
+exports.default = TasksRearmRecurrence;

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

@@ -112,6 +112,7 @@ export default class TasksShow extends BaseCommand {
         json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         task: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'no-enrich': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        'description-format': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
     };
     run(): Promise<void>;
     /**

package/dist/commands/tasks/show.js

@@ -21,6 +21,7 @@ const label_ref_1 = require("../../lib/label-ref");
 const principal_display_1 = require("../../lib/principal-display");
 const task_economics_line_1 = require("../../lib/task-economics-line");
 const format_1 = require("../../lib/format");
+const task_recurrence_outcome_1 = require("../../lib/task-recurrence-outcome");
 /**
  * Where to look next — as COMMANDS, not as facts about the task.
  *
@@ -159,9 +160,13 @@ function recurrenceArmFailureLines(t, bin = 'skrr') {
     const code = recurrence?.armFailure?.code;
     if (typeof code !== 'string' || !code)
         return [];
+    const ref = (typeof t.identifier === 'string' && t.identifier) ||
+        (typeof t.id === 'string' && t.id) ||
+        '<task-id>';
     return [
         `Recurrence:  ${(0, data_provider_1.describeRecurrenceArmFailure)(code)} (${code})`,
-        `             Reschedule with \`${bin} tasks schedule\` to start the series again.`,
+        // Not `tasks schedule`: that refuses the finished occurrence carrying this failure (OSK-15338).
+        `             ${(0, task_recurrence_outcome_1.recurrenceRearmHint)(code, ref, bin)}`,
     ];
 }
 /**
@@ -246,6 +251,10 @@ class TasksShow extends base_command_1.BaseCommand {
         'no-enrich': core_1.Flags.boolean({
             description: 'Skip server-side enrichment (assignee names, parent titles)',
         }),
+        'description-format': core_1.Flags.string({
+            description: 'How to read the description. html (default) is what is stored; markdown returns it as it was written.',
+            options: ['html', 'markdown'],
+        }),
     };
     async run() {
         this.requireAuth();
@@ -304,7 +313,12 @@ class TasksShow extends base_command_1.BaseCommand {
                 task = bound;
             }
             else {
-                task = await data_provider_1.dataService.getTask(taskId, { enrich: !flags['no-enrich'] });
+                task = await data_provider_1.dataService.getTask(taskId, {
+                    enrich: !flags['no-enrich'],
+                    ...(flags['description-format'] === 'markdown'
+                        ? { descriptionFormat: 'markdown' }
+                        : {}),
+                });
             }
         }
         catch (err) {
@@ -331,7 +345,7 @@ class TasksShow extends base_command_1.BaseCommand {
             completionRead = false;
         }
         if (flags.json) {
-            this.log(JSON.stringify({ ...t, url, completion }, null, 2));
+            this.printJsonResult({ ...t, url, completion });
             return;
         }
         for (const line of completionLines(completion, { read: completionRead }))

package/dist/commands/tasks/timeline.js

@@ -5,6 +5,7 @@ const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
 const format_1 = require("../../lib/format");
+const task_economics_1 = require("../../lib/task-economics");
 const task_legacy_1 = require("../../lib/task-legacy");
 /**
  * `<action> (<k=v ...>)`, or just the preview when there is nothing to add.
@@ -13,6 +14,9 @@ const task_legacy_1 = require("../../lib/task-legacy");
  * renders `details` with, so the two surfaces describe one entry the same way.
  */
 function withDetails(item) {
+    // An estimate change reads as from-what-to-what, the same line `tasks activity` prints.
+    if (typeof item.type === 'string' && item.type.startsWith(data_provider_1.TASK_ECONOMICS_ACTIVITY_TYPE_PREFIX))
+        return (0, task_economics_1.describeEconomicsChange)(item.data);
     const preview = String(item.preview ?? '-');
     const details = (0, format_1.formatCell)(item.data ?? item.details, '');
     return details ? `${preview} (${details})` : preview;

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

@@ -23,6 +23,7 @@ export default class TasksUpdate extends BaseCommand {
         risk: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         confidence: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         rationale: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'retract-estimate': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         estimate: import("@oclif/core/lib/interfaces").OptionFlag<number | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         /**
          * Declared, hidden, and refused — on purpose. (S4 dogfood finding)

package/dist/commands/tasks/update.js

@@ -84,6 +84,10 @@ class TasksUpdate extends base_command_1.BaseCommand {
         rationale: core_1.Flags.string({
             description: 'Why the estimate is what it is, up to 500 characters (part of the estimate). An earlier rationale stays on the claim, under its new basis, until you replace it.',
         }),
+        'retract-estimate': core_1.Flags.boolean({
+            description: "Remove the Task's economic estimate (savings, effort, risk, confidence and rationale). Recorded in `tasks activity` as task.economics.retracted under your name; an Agent cannot retract an estimate a person confirmed (409).",
+            exclusive: ['savings', 'per', 'effort', 'risk', 'confidence', 'rationale'],
+        }),
         estimate: core_1.Flags.integer({
             description: 'New estimate value from the Space estimate scale. Requires a scale on the Space (or an ' +
                 'ancestor it inherits from); with none set, the server drops the value. Set the scale ' +
@@ -410,7 +414,8 @@ class TasksUpdate extends base_command_1.BaseCommand {
             startDate: flags['start-date'],
             dueDate: flags['due-date'],
             estimate: flags.estimate,
-            economics: (0, task_economics_1.economicsFromFlags)(flags, body.economics),
+            // `null` is the retraction the server records as its own Activity event (OSK-15370).
+            economics: flags['retract-estimate'] ? null : (0, task_economics_1.economicsFromFlags)(flags, body.economics),
             // Never sent: `taskUpdateMutationGuard` refuses any payload carrying
             // `status` before the request is built. It is merged in only so the guard
             // sees it and answers with the sentence that names `tasks move`.

package/dist/commands/triggers/create.js

@@ -7,7 +7,7 @@ const builtinActions_1 = require("../../lib/builtinActions");
 const triggers_1 = require("../../lib/triggers");
 const web_url_1 = require("../../lib/web-url");
 class TriggersCreate extends base_command_1.BaseCommand {
-    static description = 'Create a universal trigger';
+    static description = 'Create a universal trigger. A schedule trigger skips an occurrence that is overdue by more than its catch-up window (--stale-catchup-ms)';
     static examples = [
         '<%= config.bin %> triggers create --scope agent --ref agentId=<agent-id> --kind schedule --mode send_message --cron "0 9 * * *" --prompt "Prepare the daily brief"',
         '<%= config.bin %> triggers create --scope agent --ref agentId=<agent-id> --kind once --mode send_message --at 2026-06-01T15:30:00Z --prompt "Send the launch recap"',
@@ -65,7 +65,7 @@ class TriggersCreate extends base_command_1.BaseCommand {
         }),
         'effect-options': core_1.Flags.string({ description: 'Effect policy options JSON object' }),
         'workflow-event-key': core_1.Flags.string({
-            description: 'Exact event key the parked workflow step waits on (required for mode=workflow_signal). Resolves steps in the scope Space; the trigger owner must hold Space edit access at every fire',
+            description: 'Exact event key the parked workflow step waits on (required for mode=workflow_signal). Resolves steps in the scope Space; the trigger owner must hold Space edit access at every fire. A person resolves one wait by hand with `skrr workflows signal`',
         }),
         'workflow-outcome': core_1.Flags.string({
             description: 'How the resolved step settles (default success)',
@@ -290,6 +290,8 @@ class TriggersCreate extends base_command_1.BaseCommand {
                 dryRun: flags['dry-run'],
             });
             (0, triggers_1.mergeObjectField)(payload, 'execution', execution);
+            const mergedExecution = payload.execution;
+            (0, triggers_1.assertModeRequiredFlags)(mergedExecution?.mode, flags, mergedExecution);
             (0, triggers_1.mergeTriggerKindConfig)(payload, flags.kind ?? payload.kind, {
                 schedule,
                 once,
@@ -304,6 +306,10 @@ class TriggersCreate extends base_command_1.BaseCommand {
             this.error(err.message, { exit: 1 });
         }
         this.validatePayload(payload);
+        // A body assembled from flags: the server's field paths are read back as those flags. A
+        // body the person wrote (--from-json) keeps the paths, which are then what they typed.
+        if (!flags['from-json'])
+            this.requestFieldFlags = triggers_1.TRIGGER_CREATE_FIELD_FLAGS;
         let trigger;
         try {
             trigger = await triggers_1.triggerApi.create(payload);

package/dist/commands/triggers/parse-intent.d.ts

@@ -1,4 +1,5 @@
 import { BaseCommand } from '../../base-command';
+import { type TriggerParseIntentResponse } from '../../lib/triggers';
 export default class TriggersParseIntent extends BaseCommand {
     static description: string;
     static examples: string[];
@@ -14,3 +15,9 @@ export default class TriggersParseIntent extends BaseCommand {
     };
     run(): Promise<void>;
 }
+/**
+ * The human rendering of a parse. Every unsupported condition is printed on its own line, never
+ * folded into the proposal (OSK-15402: "skip market holidays" was dropped while the summary claimed
+ * "every NYSE trading day"), and a clarifying question names the supported way to answer it.
+ */
+export declare function parseIntentLines(result: TriggerParseIntentResponse, bin?: string): string[];

package/dist/commands/triggers/parse-intent.js

@@ -1,5 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.parseIntentLines = parseIntentLines;
 const core_1 = require("@oclif/core");
 const base_command_1 = require("../../base-command");
 const prompt_1 = require("../../lib/prompt");
@@ -21,11 +22,11 @@ class TriggersParseIntent extends base_command_1.BaseCommand {
             multiple: true,
         }),
         message: core_1.Flags.string({
-            description: 'User message to parse (repeatable; messages are appended as user turns)',
+            description: 'User message to parse (repeatable; appended as user turns, after --history-json when both are given)',
             multiple: true,
         }),
         'history-json': core_1.Flags.string({
-            description: 'Full parser history JSON array of {role:"user"|"assistant",content:string} objects',
+            description: 'Parser history JSON array of {role:"user"|"assistant",content:string} objects. To answer a clarifying question, pass the `history` a previous --json run returned and add your reply with --message',
         }),
         timezone: core_1.Flags.string({ description: 'IANA timezone used for relative times' }),
         now: core_1.Flags.string({ description: 'Override current ISO timestamp used for relative times' }),
@@ -51,7 +52,10 @@ class TriggersParseIntent extends base_command_1.BaseCommand {
             if (history)
                 body.history = history;
             if (flags.message && flags.message.length > 0) {
-                body.history = flags.message.map((content) => ({ role: 'user', content }));
+                // Appended, never replacing: a reply to a clarifying question only means something after
+                // the question it answers (OSK-15402). It used to discard --history-json entirely.
+                const prior = Array.isArray(body.history) ? body.history : [];
+                body.history = [...prior, ...flags.message.map((content) => ({ role: 'user', content }))];
             }
             if (flags.timezone)
                 body.timezone = flags.timezone;
@@ -83,12 +87,30 @@ class TriggersParseIntent extends base_command_1.BaseCommand {
             this.log(JSON.stringify(result, null, 2));
             return;
         }
-        this.log(result.message ?? '(no parser message)');
-        this.log(`Needs clarification: ${result.needsClarification ? 'yes' : 'no'}`);
-        if (result.proposedTrigger) {
-            this.log('');
-            this.log(JSON.stringify(result.proposedTrigger, null, 2));
-        }
+        for (const line of parseIntentLines(result, this.config.bin))
+            this.log(line);
     }
 }
 exports.default = TriggersParseIntent;
+/**
+ * The human rendering of a parse. Every unsupported condition is printed on its own line, never
+ * folded into the proposal (OSK-15402: "skip market holidays" was dropped while the summary claimed
+ * "every NYSE trading day"), and a clarifying question names the supported way to answer it.
+ */
+function parseIntentLines(result, bin = 'skrr') {
+    const lines = [result.message ?? '(no parser message)'];
+    lines.push(`Needs clarification: ${result.needsClarification ? 'yes' : 'no'}`);
+    const unsupported = Array.isArray(result.unsupported) ? result.unsupported : [];
+    if (unsupported.length > 0) {
+        lines.push('', 'Not applied by this trigger:');
+        for (const item of unsupported)
+            lines.push(`  - "${item.text}" (${item.kind}): ${item.reason}`);
+    }
+    if (result.proposedTrigger) {
+        lines.push('', JSON.stringify(result.proposedTrigger, null, 2));
+    }
+    else if (result.needsClarification) {
+        lines.push('', 'To answer, run it again with your reply after the earlier messages, e.g.', `  ${bin} triggers parse-intent --scope <kind> --ref <key=value> --message "<first message>" --message "<your answer>"`, 'or pass the returned `history` (see --json) with --history-json and add --message "<your answer>".');
+    }
+    return lines;
+}

package/dist/commands/workspaces/grants.js

@@ -18,10 +18,11 @@ class WorkspacesGrants extends base_command_1.BaseCommand {
         const { args, flags } = await this.parse(WorkspacesGrants);
         try {
             const report = await (0, workspaces_1.listWorkspaceGrants)(args.workspaceId);
-            if (flags.json)
+            if (flags.json) {
                 this.log(JSON.stringify(report, null, 2));
-            else
-                (0, stewardship_output_1.printWorkspaceGrants)(report, (line) => this.log(line));
+                return;
+            }
+            (0, stewardship_output_1.printWorkspaceGrants)(report, (line) => this.log(line), await memberNames(args.workspaceId));
         }
         catch (error) {
             this.handleApiError(error);
@@ -29,3 +30,22 @@ class WorkspacesGrants extends base_command_1.BaseCommand {
     }
 }
 exports.default = WorkspacesGrants;
+/**
+ * Owner ids to names, for the table's OWNER column (OSK-15357). Best effort: the
+ * report is already authoritative, and a members read that fails only leaves
+ * the column showing ids, as it did before.
+ */
+async function memberNames(workspaceId) {
+    try {
+        const members = await (0, workspaces_1.listWorkspaceMembers)(workspaceId);
+        return new Map(members
+            .map((member) => [
+            String(member.userId),
+            member.user?.name || member.user?.username || member.user?.email || '',
+        ])
+            .filter(([, name]) => name));
+    }
+    catch {
+        return new Map();
+    }
+}

package/dist/commands/workspaces/members.d.ts

@@ -0,0 +1,20 @@
+import { BaseCommand } from '../../base-command';
+/**
+ * `skrr workspaces members` — who is in a workspace, with the user id the
+ * other workspace commands take (OSK-15356).
+ *
+ * `workspaces stewardship` needs a member's user id, and nothing in this
+ * command family printed one but `skrr whoami` (your own). An owner handing
+ * over a departing member's work had no way to find theirs.
+ */
+export default class WorkspacesMembers extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static args: {
+        workspaceId: 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/workspaces/members.js

@@ -0,0 +1,64 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const base_command_1 = require("../../base-command");
+const format_1 = require("../../lib/format");
+const workspaces_1 = require("../../lib/workspaces");
+/**
+ * `skrr workspaces members` — who is in a workspace, with the user id the
+ * other workspace commands take (OSK-15356).
+ *
+ * `workspaces stewardship` needs a member's user id, and nothing in this
+ * command family printed one but `skrr whoami` (your own). An owner handing
+ * over a departing member's work had no way to find theirs.
+ */
+class WorkspacesMembers extends base_command_1.BaseCommand {
+    static description = 'List the current members of a workspace with their user ids and roles';
+    static examples = [
+        '<%= config.bin %> workspaces members <workspace-id>',
+        '<%= config.bin %> workspaces members <workspace-id> --json',
+    ];
+    static args = {
+        workspaceId: core_1.Args.string({ required: true, ignoreStdin: true, description: 'Workspace ID' }),
+    };
+    static flags = {
+        json: core_1.Flags.boolean({ description: 'Output as JSON' }),
+    };
+    async run() {
+        this.requireAuth();
+        const { args, flags } = await this.parse(WorkspacesMembers);
+        let members;
+        try {
+            members = await (0, workspaces_1.listWorkspaceMembers)(args.workspaceId);
+        }
+        catch (error) {
+            this.handleApiError(error);
+            return;
+        }
+        if (flags.json) {
+            this.log(JSON.stringify(members, null, 2));
+            return;
+        }
+        if (members.length === 0) {
+            this.log('No current members.');
+            return;
+        }
+        (0, format_1.renderTable)(members.map((member) => ({
+            userId: String(member.userId),
+            name: member.user?.name || member.user?.username || '—',
+            email: member.user?.email || '—',
+            role: member.role || '—',
+            joined: member.joinedAt || '—',
+        })), [
+            { key: 'userId', header: 'USER ID', verbatim: true },
+            { key: 'name', header: 'NAME', maxWidth: 28 },
+            { key: 'email', header: 'EMAIL', maxWidth: 36 },
+            { key: 'role', header: 'ROLE' },
+            { key: 'joined', header: 'JOINED' },
+        ], (line) => this.log(line));
+        this.log('');
+        this.log('Current members only: someone who has left is not listed. ' +
+            `\`skrr workspaces stewardship ${args.workspaceId} <user-id|email>\` reads one member's handover.`);
+    }
+}
+exports.default = WorkspacesMembers;

package/dist/commands/workspaces/stewardship.js

@@ -8,13 +8,15 @@ class WorkspacesStewardship extends base_command_1.BaseCommand {
     static description = 'Read a member’s workspace stewardship metadata (person owner/admin only)';
     static examples = [
         '<%= config.bin %> workspaces stewardship <workspace-id> <user-id> --json',
+        '<%= config.bin %> workspaces stewardship <workspace-id> person@example.com',
     ];
     static args = {
         workspaceId: core_1.Args.string({ required: true, ignoreStdin: true, description: 'Workspace ID' }),
         userId: core_1.Args.string({
             required: true,
             ignoreStdin: true,
-            description: 'Member user ID: 24 hexadecimal characters, not an email (`skrr whoami` prints your own)',
+            description: "Member user ID (24 hexadecimal characters) or a current member's email. " +
+                '`skrr workspaces members <workspace-id>` lists them; `skrr whoami` prints your own',
         }),
     };
     static flags = {
@@ -23,8 +25,18 @@ class WorkspacesStewardship extends base_command_1.BaseCommand {
     async run() {
         this.requireAuth();
         const { args, flags } = await this.parse(WorkspacesStewardship);
+        let userId;
         try {
-            const report = await (0, workspaces_1.getWorkspaceStewardship)(args.workspaceId, args.userId);
+            userId = await (0, workspaces_1.resolveMemberUserId)(args.workspaceId, args.userId);
+        }
+        catch (error) {
+            if (error instanceof workspaces_1.MemberReferenceError)
+                this.error(error.message, { exit: 1 });
+            this.handleApiError(error);
+            return;
+        }
+        try {
+            const report = await (0, workspaces_1.getWorkspaceStewardship)(args.workspaceId, userId);
             if (flags.json)
                 this.log(JSON.stringify(report, null, 2));
             else

package/dist/help.js

@@ -61,6 +61,18 @@ class SkrrHelp extends core_1.Help {
         // so it must fall through — only an id that is neither a command nor a
         // topic is a miss.
         if (id && !this.config.findCommand(id) && !this.config.findTopic(id)) {
+            // `skrr tasks get --help` is asked about the command `skrr tasks get` runs
+            // as: the same synonym the hook resolves when it is run (OSK-16434).
+            const flagStart = argv.findIndex((token) => token.startsWith('-'));
+            const words = flagStart === -1 ? argv : argv.slice(0, flagStart);
+            const synonym = (0, command_miss_1.resolveVerbSynonym)(lookup, words);
+            if (synonym) {
+                return super.showHelp([
+                    ...synonym.id.split(':'),
+                    ...synonym.rest,
+                    ...(flagStart === -1 ? [] : argv.slice(flagStart)),
+                ]);
+            }
             throw new Error((0, command_miss_1.describeCommandMiss)(lookup, id));
         }
         return super.showHelp(argv);

package/dist/hooks/command-not-found.js

@@ -1,8 +1,10 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
 const command_manifest_1 = require("../lib/command-manifest");
 const usage_discovery_1 = require("../lib/usage-discovery");
 const command_miss_1 = require("../lib/command-miss");
+const json_mode_1 = require("../lib/json-mode");
 /**
  * Turn "command not found" back into the truth when the command DOES exist.
  *
@@ -65,6 +67,30 @@ const hook = async function (opts) {
                 throw err;
         }
     }
+    const lookup = (0, command_miss_1.lookupFromConfig)(this.config);
+    const rawArgv = process.argv.slice(2);
+    const words = (0, command_miss_1.typedCommandWords)(id, rawArgv);
+    // A read verb spelled the other way (`tasks get`, `machines ls`) means the
+    // command that exists. Run it rather than refuse: refusing sent a reader to a
+    // suggestion list that did not contain the command they meant (OSK-16434).
+    // Only `list`/`ls` and `show`/`get`, only when the typed spelling is not a
+    // command of that topic, and only when the other one is.
+    const synonym = (0, command_miss_1.resolveVerbSynonym)(lookup, words);
+    if (synonym) {
+        // Said to a person, never to a pipe: a script gets its output undisturbed.
+        if (process.stderr.isTTY && !(0, json_mode_1.wantsJsonFromArgv)(rawArgv)) {
+            process.stderr.write(`\`${synonym.typed}\` is \`${synonym.resolved}\`.\n`);
+        }
+        return this.config.runCommand(synonym.id, [...synonym.rest, ...(opts.argv ?? [])]);
+    }
+    // `--json` asked for a document. A miss is a refusal like any other, and every
+    // other refusal prints `{ok:false, error}`; this one printed prose, so the
+    // pipe into `jq` failed on the prose and hid the refusal (OSK-16434). Exit
+    // status 1, as before.
+    if ((0, json_mode_1.wantsJsonFromArgv)([...rawArgv, ...(opts.argv ?? [])])) {
+        process.stdout.write(`${JSON.stringify((0, command_miss_1.commandMissEnvelope)(lookup, id, words), null, 2)}\n`);
+        core_1.Errors.exit(1);
+    }
     // The message itself lives in `lib/command-miss`, shared with the help
     // class — `skrr daemon reload` reaches this hook and `skrr daemon reload
     // --help` does not, so the two surfaces have to read from one description or
@@ -76,7 +102,7 @@ const hook = async function (opts) {
     // — a typo would silently look like success, which is far worse than the
     // message this hook was written to improve. (I shipped exactly that for one
     // build; `skrr shortcuts lst` said nothing and exited 0.)
-    throw new Error((0, command_miss_1.describeCommandMiss)((0, command_miss_1.lookupFromConfig)(this.config), id));
+    throw new Error((0, command_miss_1.describeCommandMiss)(lookup, id));
 };
 exports.default = hook;
 class VirtualUsageFlagError extends Error {