@skrr-ai/cli 0.1.127 → 0.1.128

package/dist/commands/machines/update.js

@@ -1,22 +1,29 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 const core_1 = require("@oclif/core");
+const promises_1 = require("node:fs/promises");
+const node_path_1 = require("node:path");
+const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
+const agent_visual_refs_1 = require("../../lib/agent-visual-refs");
 const machines_1 = require("../../lib/machines");
 /**
- * Rename / describe / tag a machine.
+ * Rename / describe / tag a machine, and choose its icon.
  *
  * `preferences` is the ONLY user-editable part of a machine — identities,
  * capabilities and lifecycle stay authoritative in the daemon, harness and
- * lease records this aggregate reads. The name set here is yours: it does not
- * rename the machine for anyone else.
+ * lease records this aggregate reads. The name and icon set here are yours: they
+ * do not rename or re-draw the machine for anyone else.
  */
 class MachinesUpdate extends base_command_1.BaseCommand {
-    static description = 'Rename a machine, or set its description and tags';
+    static description = 'Rename a machine, set its description, tags and icon';
     static examples = [
         '<%= config.bin %> machines update 03548116 --name "Work laptop"',
         '<%= config.bin %> machines update 03548116 --tag personal --tag macbook',
         '<%= config.bin %> machines update 03548116 --clear-tags',
+        '<%= config.bin %> machines update 03548116 --icon laptop',
+        '<%= config.bin %> machines update 03548116 --icon none',
+        '<%= config.bin %> machines update 03548116 --icon-file ./mac.png',
     ];
     static args = {
         machine: core_1.Args.string({
@@ -37,6 +44,15 @@ class MachinesUpdate extends base_command_1.BaseCommand {
             description: 'Remove all tags',
             exclusive: ['tag'],
         }),
+        icon: core_1.Flags.string({
+            description: `Icon for this machine: ${data_provider_1.MACHINE_ICON_PRESET_IDS.join(', ')}. ` +
+                '"none" goes back to the default.',
+            exclusive: ['icon-file'],
+        }),
+        'icon-file': core_1.Flags.string({
+            description: 'Use your own image as the icon (.png, .jpg, .jpeg, .webp or .gif). It is resized to a square.',
+            exclusive: ['icon'],
+        }),
     };
     async run() {
         this.requireAuth();
@@ -50,17 +66,61 @@ class MachinesUpdate extends base_command_1.BaseCommand {
             patch.tags = [];
         else if (flags.tag)
             patch.tags = flags.tag;
-        if (Object.keys(patch).length === 0) {
+        if (flags.icon !== undefined) {
+            try {
+                patch.icon = (0, machines_1.parseMachineIconFlag)(flags.icon);
+            }
+            catch (err) {
+                this.error(err.message, { exit: 1 });
+            }
+        }
+        // Everything that can be refused without a request is refused before one is
+        // sent, so a typo'd path never leaves the other changes half applied.
+        let upload;
+        const iconFile = flags['icon-file'];
+        if (iconFile !== undefined) {
+            const type = (0, agent_visual_refs_1.imageUploadMimeType)(iconFile);
+            if (!type) {
+                this.error(`--icon-file must be a .png, .jpg, .jpeg, .webp or .gif image (got ${(0, node_path_1.basename)(iconFile)}).`, { exit: 1 });
+            }
+            let bytes;
+            try {
+                bytes = await (0, promises_1.readFile)(iconFile);
+            }
+            catch (err) {
+                this.error(`--icon-file: cannot read ${iconFile} (${err.message}).`, {
+                    exit: 1,
+                });
+            }
+            upload = { bytes, type, filename: (0, node_path_1.basename)(iconFile) };
+        }
+        const hasPatch = Object.keys(patch).length > 0;
+        if (!hasPatch && !upload) {
             // Sending an empty patch would return the machine unchanged and read as
             // success, which is the wrong answer to "I asked you to change something".
-            this.error('Nothing to update. Pass --name, --description, --tag, or --clear-tags.', {
-                exit: 1,
-            });
+            this.error('Nothing to update. Pass --name, --description, --tag, --clear-tags, --icon, or --icon-file.', { exit: 1 });
         }
         let machine;
         try {
             const resolved = (0, machines_1.resolveMachineRef)(await (0, machines_1.listMachines)(), args.machine);
-            machine = await (0, machines_1.updateMachine)(resolved.id, patch);
+            if (hasPatch)
+                machine = await (0, machines_1.updateMachine)(resolved.id, patch);
+            if (upload) {
+                const form = new FormData();
+                // Typed, or the route refuses it as application/octet-stream (OSK-9841).
+                form.append('file', new Blob([new Uint8Array(upload.bytes)], { type: upload.type }), upload.filename);
+                try {
+                    await data_provider_1.dataService.uploadMachineIcon(resolved.id, form);
+                }
+                catch (err) {
+                    if (hasPatch)
+                        this.warn('The other changes were saved, but the icon was not uploaded.');
+                    throw err;
+                }
+                // The upload answers only the icon; read the machine back so what is
+                // printed is what is stored.
+                machine = await (0, machines_1.getMachine)(resolved.id);
+            }
         }
         catch (err) {
             this.handleApiError(err);
@@ -77,6 +137,7 @@ class MachinesUpdate extends base_command_1.BaseCommand {
             this.log(`  description  ${machine.preferences.description}`);
         }
         this.log(`  tags         ${machine.preferences?.tags?.join(', ') || '-'}`);
+        this.log(`  icon         ${(0, machines_1.describeMachineIcon)(machine.preferences?.icon)}`);
     }
 }
 exports.default = MachinesUpdate;

package/dist/commands/northstars/create.js

@@ -31,7 +31,7 @@ class NorthstarsCreate extends base_command_1.BaseCommand {
             min: 0,
         }),
         'max-active': core_1.Flags.integer({
-            description: 'Soft cap on simultaneously active commitments',
+            description: 'Soft cap on simultaneously active commitments (0 = no cap)',
             min: 0,
         }),
         goal: core_1.Flags.string({
@@ -98,6 +98,8 @@ class NorthstarsCreate extends base_command_1.BaseCommand {
         }
         this.log(`Created ${created.id} — ${created.title} (${created.status})`);
         this.log(`URL: ${created.url ?? '-'}`);
+        if (flags['max-active'] === 0)
+            this.log('Attention   max active 0 (no cap)');
         this.log(flags['review-interval-hours'] !== undefined
             ? `Review runs every ${flags['review-interval-hours']}h.`
             : 'Review is manual — set --review-interval-hours or run `skrr northstars review-now`.');

package/dist/commands/northstars/governance/configure.d.ts

@@ -22,6 +22,7 @@ export default class NorthstarsGovernanceConfigure extends BaseCommand {
         allow: import("@oclif/core/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/interfaces").CustomOptions>;
         'require-approval-for': import("@oclif/core/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/interfaces").CustomOptions>;
         protect: import("@oclif/core/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        'clear-protected': import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'max-intents-per-review': import("@oclif/core/interfaces").OptionFlag<number | undefined, import("@oclif/core/interfaces").CustomOptions>;
         'max-auto-per-review': import("@oclif/core/interfaces").OptionFlag<number | undefined, import("@oclif/core/interfaces").CustomOptions>;
         'max-auto-per-day': import("@oclif/core/interfaces").OptionFlag<number | undefined, import("@oclif/core/interfaces").CustomOptions>;
@@ -30,8 +31,11 @@ export default class NorthstarsGovernanceConfigure extends BaseCommand {
         'max-interval-hours': import("@oclif/core/interfaces").OptionFlag<number | undefined, import("@oclif/core/interfaces").CustomOptions>;
         'max-relative-change': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         'max-observation-runs-per-day': import("@oclif/core/interfaces").OptionFlag<number | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        'clear-max-observation-runs-per-day': import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'max-runs-per-day': import("@oclif/core/interfaces").OptionFlag<number | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        'clear-max-runs-per-day': import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'max-concurrent-runs': import("@oclif/core/interfaces").OptionFlag<number | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        'clear-max-concurrent-runs': import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'critical-reserve': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
     };
     run(): Promise<void>;

package/dist/commands/northstars/governance/configure.js

@@ -19,6 +19,8 @@ class NorthstarsGovernanceConfigure extends base_command_1.BaseCommand {
         '<%= config.bin %> northstars governance configure nst_abc123 --execution approval',
         '<%= config.bin %> northstars governance configure nst_abc123 --execution bounded_auto --allow request_commitment_check --max-auto-per-day 3',
         '<%= config.bin %> northstars governance configure nst_abc123 --protect cmt_1 --protect cmt_2',
+        '<%= config.bin %> northstars governance configure nst_abc123 --clear-protected',
+        '<%= config.bin %> northstars governance configure nst_abc123 --clear-max-runs-per-day',
     ];
     static args = {
         northstarId: core_1.Args.string({ ignoreStdin: true, description: 'Northstar ID', required: true }),
@@ -30,7 +32,7 @@ class NorthstarsGovernanceConfigure extends base_command_1.BaseCommand {
             options: [...northstars_1.NORTHSTAR_ANALYSIS_MODES],
         }),
         execution: core_1.Flags.string({
-            description: 'How much write authority a review outcome carries',
+            description: "What happens to management intents: advisory records them (none is applied), approval asks first, bounded_auto applies a narrow set. In every mode a review still writes each linked active commitment's allocation (cadence band, run share); it never pauses, resumes, reprioritizes or checks in advisory",
             options: [...northstars_1.NORTHSTAR_EXECUTION_MODES],
         }),
         'reviewer-agent': core_1.Flags.string({
@@ -48,8 +50,13 @@ class NorthstarsGovernanceConfigure extends base_command_1.BaseCommand {
             multiple: true,
         }),
         protect: core_1.Flags.string({
-            description: 'Commitment the governor may never touch (repeatable; replaces the current list)',
+            description: 'Commitment the governor may never touch (repeatable; replaces the current list; --clear-protected empties it)',
             multiple: true,
+            exclusive: ['clear-protected'],
+        }),
+        'clear-protected': core_1.Flags.boolean({
+            description: 'Remove every protected commitment (the list becomes empty)',
+            exclusive: ['protect'],
         }),
         'max-intents-per-review': core_1.Flags.integer({
             description: 'Cap on intents per review',
@@ -88,16 +95,31 @@ class NorthstarsGovernanceConfigure extends base_command_1.BaseCommand {
             description: 'Opt-in portfolio observation runs per UTC day',
             min: 0,
             max: 10000,
+            exclusive: ['clear-max-observation-runs-per-day'],
+        }),
+        'clear-max-observation-runs-per-day': core_1.Flags.boolean({
+            description: 'Un-set the observation ceiling (0 means never look; un-set means no ceiling)',
+            exclusive: ['max-observation-runs-per-day'],
         }),
         'max-runs-per-day': core_1.Flags.integer({
             description: 'Portfolio-wide action run budget (execution, NOT attention)',
             min: 0,
             max: 10000,
+            exclusive: ['clear-max-runs-per-day'],
+        }),
+        'clear-max-runs-per-day': core_1.Flags.boolean({
+            description: 'Un-set the action run budget (0 is a setting, not un-set)',
+            exclusive: ['max-runs-per-day'],
         }),
         'max-concurrent-runs': core_1.Flags.integer({
-            description: 'Portfolio concurrency cap',
+            description: 'Portfolio concurrency cap (recorded, not yet enforced: nothing limits concurrent runs yet)',
             min: 0,
             max: 1000,
+            exclusive: ['clear-max-concurrent-runs'],
+        }),
+        'clear-max-concurrent-runs': core_1.Flags.boolean({
+            description: 'Un-set the concurrency cap',
+            exclusive: ['max-concurrent-runs'],
         }),
         'critical-reserve': core_1.Flags.string({
             description: 'Fraction of the execution budget held for critical commitments (0..1)',
@@ -119,8 +141,17 @@ class NorthstarsGovernanceConfigure extends base_command_1.BaseCommand {
             body.allowedIntentKinds = flags.allow;
         if (flags['require-approval-for'])
             body.requireApprovalFor = flags['require-approval-for'];
-        if (flags.protect)
+        if (flags['clear-protected']) {
+            body.protectedCommitmentIds = [];
+        }
+        else if (flags.protect) {
+            // An empty value used to reach the server and come back as
+            // `protectedCommitmentIds.0: Too small`, a REST field name nobody typed.
+            if (flags.protect.some((id) => id.trim() === '')) {
+                this.error('--protect needs a commitment id. To remove every protected commitment, pass --clear-protected.');
+            }
             body.protectedCommitmentIds = flags.protect;
+        }
         if (flags['max-intents-per-review'] !== undefined) {
             body.maxIntentsPerReview = flags['max-intents-per-review'];
         }
@@ -150,12 +181,21 @@ class NorthstarsGovernanceConfigure extends base_command_1.BaseCommand {
         if (Object.keys(cadencePolicy).length)
             body.cadencePolicy = cadencePolicy;
         const executionBudget = {};
-        if (flags['max-observation-runs-per-day'] !== undefined)
+        // `null` is how the API un-sets a budget; 0 is a setting and is never read as "un-set".
+        if (flags['clear-max-observation-runs-per-day'])
+            executionBudget.maxObservationRunsPerDay = null;
+        else if (flags['max-observation-runs-per-day'] !== undefined)
             executionBudget.maxObservationRunsPerDay = flags['max-observation-runs-per-day'];
-        if (flags['max-runs-per-day'] !== undefined) {
+        if (flags['clear-max-runs-per-day']) {
+            executionBudget.maxRunsPerDay = null;
+        }
+        else if (flags['max-runs-per-day'] !== undefined) {
             executionBudget.maxRunsPerDay = flags['max-runs-per-day'];
         }
-        if (flags['max-concurrent-runs'] !== undefined) {
+        if (flags['clear-max-concurrent-runs']) {
+            executionBudget.maxConcurrentRuns = null;
+        }
+        else if (flags['max-concurrent-runs'] !== undefined) {
             executionBudget.maxConcurrentRuns = flags['max-concurrent-runs'];
         }
         if (flags['critical-reserve'] !== undefined) {
@@ -170,12 +210,16 @@ class NorthstarsGovernanceConfigure extends base_command_1.BaseCommand {
         if (Object.keys(body).length === 0) {
             this.error('Nothing to configure — pass at least one flag.');
         }
+        let written;
         try {
-            await northstars_1.northstarApi.configureGovernance(args.northstarId, body);
+            written = await northstars_1.northstarApi.configureGovernance(args.northstarId, body);
         }
         catch (err) {
             this.handleApiError(err);
         }
+        // The write response says what the server did to the linked commitments; the status
+        // re-read below does not (OSK-21245).
+        const refresh = (0, northstars_1.readAllocationRefresh)(written);
         // Re-read the STATUS rather than echoing the write response: the effective
         // policy is what the server resolved, and readiness (is the reviewer even
         // online?) is the part a configure command must not leave the user guessing.
@@ -187,10 +231,32 @@ class NorthstarsGovernanceConfigure extends base_command_1.BaseCommand {
             this.handleApiError(err);
         }
         if (flags.json) {
-            this.log(JSON.stringify(status, null, 2));
+            this.log(JSON.stringify(refresh
+                ? {
+                    ...status,
+                    allocationsRefreshed: refresh.refreshed,
+                    allocationsPending: refresh.pending,
+                    allocationsRefreshFailed: refresh.failed,
+                }
+                : status, null, 2));
             return;
         }
         (0, northstars_1.renderGovernance)(status, (line) => this.log(line));
+        const summary = (0, northstars_1.describeAllocationRefresh)(refresh, {
+            northstarId: args.northstarId,
+            inputsSent: northstars_1.ALLOCATION_INPUT_FLAGS.some((flag) => flags[flag] !== undefined && flags[flag] !== false),
+            bin: this.config?.bin,
+        });
+        if (summary.lines.length)
+            this.log('');
+        for (const line of summary.lines)
+            this.log(line);
+        for (const warning of summary.warnings)
+            this.warn(warning);
+        if (flags['max-concurrent-runs'] !== undefined) {
+            this.log('');
+            this.log('Note: --max-concurrent-runs is recorded but not yet enforced. Nothing limits concurrent runs across this northstar; each commitment keeps its own concurrency limit.');
+        }
     }
 }
 exports.default = NorthstarsGovernanceConfigure;

package/dist/commands/northstars/intents/decide.d.ts

@@ -7,6 +7,12 @@ import { BaseCommand } from '../../../base-command';
  * hours later against a portfolio that has since changed is invalidated rather
  * than forced through. That is the point of approving an INTENT rather than
  * queueing a command.
+ *
+ * ONE exit rule: the command exits 0 only when the thing asked for now holds
+ * (an approval left the intent `applied`, a rejection left it `rejected`).
+ * Retrying a decision that already took effect is a replay and also exits 0;
+ * everything else exits 1 with a sentence, and `--json` keeps the raw code plus
+ * `reasonText`.
  */
 export default class NorthstarsIntentsDecide extends BaseCommand {
     static description: string;

package/dist/commands/northstars/intents/decide.js

@@ -11,6 +11,12 @@ const northstars_1 = require("../../../lib/northstars");
  * hours later against a portfolio that has since changed is invalidated rather
  * than forced through. That is the point of approving an INTENT rather than
  * queueing a command.
+ *
+ * ONE exit rule: the command exits 0 only when the thing asked for now holds
+ * (an approval left the intent `applied`, a rejection left it `rejected`).
+ * Retrying a decision that already took effect is a replay and also exits 0;
+ * everything else exits 1 with a sentence, and `--json` keeps the raw code plus
+ * `reasonText`.
  */
 class NorthstarsIntentsDecide extends base_command_1.BaseCommand {
     static description = 'Approve or reject a pending northstar management intent';
@@ -40,32 +46,80 @@ class NorthstarsIntentsDecide extends base_command_1.BaseCommand {
         if (!flags.approve && !flags.reject) {
             this.error('Pass --approve or --reject.');
         }
+        const ctx = {
+            northstarId: args.northstarId,
+            intentId: args.intentId,
+            decision: flags.approve ? 'approve' : 'reject',
+            bin: this.config.bin,
+        };
         let result;
         try {
             result = await northstars_1.northstarApi.decideIntent(args.northstarId, args.intentId, {
-                decision: flags.approve ? 'approve' : 'reject',
+                decision: ctx.decision,
                 ...(flags.review ? { reviewId: flags.review } : {}),
             });
         }
         catch (err) {
+            // The server refuses a decision with a bare code (`northstar_version_drift`,
+            // `intent_not_pending`). Say what it means, and keep the code.
+            const status = err?.status;
+            const { body } = (0, base_command_1.apiRefusal)(err);
+            if ((status === 404 || status === 409) && body && typeof body.reason === 'string') {
+                const { code, text } = (0, northstars_1.describeDecisionRefusal)(body, ctx);
+                // A retry of a decision that already took effect (the first response was
+                // lost, or a hedge raced itself) leaves the state that was asked for.
+                const verdict = typeof body.verdict === 'string' ? body.verdict : undefined;
+                const alreadyHolds = code === 'intent_not_pending' &&
+                    ((ctx.decision === 'approve' && verdict === 'applied') ||
+                        (ctx.decision === 'reject' && verdict === 'rejected'));
+                if (alreadyHolds) {
+                    if (flags.json) {
+                        this.log(JSON.stringify({
+                            ok: true,
+                            verdict,
+                            replayed: true,
+                            ...(body.decidedAt ? { decidedAt: body.decidedAt } : {}),
+                            ...(body.decidedByYou !== undefined ? { decidedByYou: body.decidedByYou } : {}),
+                            reasonText: text,
+                        }, null, 2));
+                    }
+                    else {
+                        this.log(`${args.intentId} → ${verdict}`);
+                        this.log(text);
+                    }
+                    return;
+                }
+                this.failWithCliError({
+                    message: text,
+                    code,
+                    status,
+                    exit: 1,
+                    retryable: false,
+                    details: { ...body, reasonText: text },
+                });
+            }
             this.handleApiError(err);
         }
-        if (flags.json) {
-            this.log(JSON.stringify(result, null, 2));
-            return;
-        }
-        const verdict = String(result.verdict ?? 'unknown');
-        if (result.proposalOnly) {
-            // A proposal is a decision to act BY HAND. Saying so avoids the reasonable
-            // assumption that approving it made something happen.
-            this.log(`Recorded: ${args.intentId} approved as a proposal — no mutation was performed.`);
+        const outcome = (0, northstars_1.describeDecisionOutcome)(result, ctx);
+        if (outcome.exitCode === 0) {
+            if (flags.json) {
+                this.log(JSON.stringify({ ...result, ...(outcome.text ? { reasonText: outcome.text } : {}) }, null, 2));
+                return;
+            }
+            this.log(outcome.headline);
+            if (outcome.text)
+                this.log(outcome.text);
             return;
         }
-        this.log(`${args.intentId} → ${verdict}`);
-        const applyResult = result.applyResult;
-        if (applyResult && applyResult.applied !== true && applyResult.reason) {
-            this.log(`Not applied: ${String(applyResult.reason)}`);
-        }
+        if (!flags.json)
+            this.log(outcome.headline);
+        this.failWithCliError({
+            message: outcome.text ?? outcome.headline,
+            code: outcome.code ?? 'intent_not_applied',
+            exit: 1,
+            retryable: false,
+            details: { ...result, reasonText: outcome.text },
+        });
     }
 }
 exports.default = NorthstarsIntentsDecide;

package/dist/commands/northstars/intents/list.d.ts

@@ -1,6 +1,6 @@
 import { BaseCommand } from '../../../base-command';
 /**
- * `skrr northstars intents <northstar-id>` — management intents across recent
+ * `skrr northstars intents list <northstar-id>` — management intents across recent
  * reviews, newest first.
  *
  * Flattened across reviews on purpose: the question a person asks is "what is
@@ -17,6 +17,7 @@ export default class NorthstarsIntentsList extends BaseCommand {
         json: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         status: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         limit: import("@oclif/core/interfaces").OptionFlag<number | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        wide: import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
 }

package/dist/commands/northstars/intents/list.js

@@ -5,7 +5,7 @@ const core_1 = require("@oclif/core");
 const base_command_1 = require("../../../base-command");
 const northstars_1 = require("../../../lib/northstars");
 /**
- * `skrr northstars intents <northstar-id>` — management intents across recent
+ * `skrr northstars intents list <northstar-id>` — management intents across recent
  * reviews, newest first.
  *
  * Flattened across reviews on purpose: the question a person asks is "what is
@@ -15,9 +15,9 @@ const northstars_1 = require("../../../lib/northstars");
 class NorthstarsIntentsList extends base_command_1.BaseCommand {
     static description = 'List northstar management intents (default: everything recent)';
     static examples = [
-        '<%= config.bin %> northstars intents nst_abc123',
-        '<%= config.bin %> northstars intents nst_abc123 --status awaiting_approval',
-        '<%= config.bin %> northstars intents nst_abc123 --status applied --json',
+        '<%= config.bin %> northstars intents list nst_abc123',
+        '<%= config.bin %> northstars intents list nst_abc123 --status awaiting_approval',
+        '<%= config.bin %> northstars intents list nst_abc123 --status applied --json',
     ];
     static args = {
         northstarId: core_1.Args.string({ ignoreStdin: true, description: 'Northstar ID', required: true }),
@@ -29,6 +29,7 @@ class NorthstarsIntentsList extends base_command_1.BaseCommand {
             options: [...northstars_1.NORTHSTAR_INTENT_VERDICTS],
         }),
         limit: core_1.Flags.integer({ description: 'Max intents (1..500)', min: 1, max: 500 }),
+        wide: northstars_1.northstarWideFlag,
     };
     async run() {
         this.requireAuth();
@@ -47,7 +48,16 @@ class NorthstarsIntentsList extends base_command_1.BaseCommand {
             this.printJsonResult((0, list_output_1.listResult)(response));
             return;
         }
-        (0, northstars_1.renderIntentList)(response.data ?? [], (line) => this.log(line));
+        const rows = response.data ?? [];
+        (0, northstars_1.renderIntentList)(rows, (line) => this.log(line), { wide: flags.wide });
+        const footer = (0, northstars_1.listFooter)({
+            shown: rows.length,
+            hasMore: response.has_more === true,
+            limit: flags.limit ?? 100,
+            max: 500,
+        });
+        if (footer)
+            this.log(footer);
     }
 }
 exports.default = NorthstarsIntentsList;

package/dist/commands/northstars/intents.d.ts

@@ -0,0 +1,26 @@
+import NorthstarsIntentsList from './intents/list';
+/**
+ * `skrr northstars intents <northstar-id>` — the topic's default: list the intents.
+ *
+ * `intents` is a topic (`list`, `decide`), and every sibling read command
+ * (`show`, `reviews`) works with just a northstar id. Without this command
+ * `northstars intents nst_abc123` was answered with "is not a command" and a
+ * did-you-mean that listed `create` and `delete` first, while the help examples
+ * printed exactly that form. Listing is read-only, so it is the safe default; the
+ * subcommands are untouched (oclif resolves the longest command id first, so
+ * `intents decide …` and `intents list …` still reach their own commands).
+ *
+ * With no northstar id it prints the topic help instead of a bare missing-argument error.
+ */
+export default class NorthstarsIntents extends NorthstarsIntentsList {
+    static description: string;
+    static examples: string[];
+    static args: typeof NorthstarsIntentsList.args;
+    static flags: {
+        json: import("@oclif/core/interfaces").BooleanFlag<boolean>;
+        status: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        limit: import("@oclif/core/interfaces").OptionFlag<number | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        wide: import("@oclif/core/interfaces").BooleanFlag<boolean>;
+    };
+    run(): Promise<void>;
+}

package/dist/commands/northstars/intents.js

@@ -0,0 +1,49 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const list_1 = __importDefault(require("./intents/list"));
+/**
+ * `skrr northstars intents <northstar-id>` — the topic's default: list the intents.
+ *
+ * `intents` is a topic (`list`, `decide`), and every sibling read command
+ * (`show`, `reviews`) works with just a northstar id. Without this command
+ * `northstars intents nst_abc123` was answered with "is not a command" and a
+ * did-you-mean that listed `create` and `delete` first, while the help examples
+ * printed exactly that form. Listing is read-only, so it is the safe default; the
+ * subcommands are untouched (oclif resolves the longest command id first, so
+ * `intents decide …` and `intents list …` still reach their own commands).
+ *
+ * With no northstar id it prints the topic help instead of a bare missing-argument error.
+ */
+class NorthstarsIntents extends list_1.default {
+    static description = 'List northstar management intents; subcommands: list, decide (`intents <id>` is `intents list <id>`)';
+    static examples = [
+        '<%= config.bin %> northstars intents nst_abc123',
+        '<%= config.bin %> northstars intents nst_abc123 --status awaiting_approval',
+        '<%= config.bin %> northstars intents list nst_abc123',
+        '<%= config.bin %> northstars intents decide nst_abc123 nsi_xyz --approve',
+    ];
+    // Own, not inherited: oclif spreads the command class into the parser input, and a static a
+    // subclass only inherits is dropped (OSK-15346). The argument is optional so a bare
+    // `skrr northstars intents` can show the subcommands rather than fail on a missing id.
+    static args = {
+        northstarId: core_1.Args.string({
+            ignoreStdin: true,
+            description: 'Northstar ID (omit to see the subcommands)',
+            required: false,
+        }),
+    };
+    static flags = list_1.default.flags;
+    async run() {
+        const { args } = await this.parse(NorthstarsIntents);
+        if (!args.northstarId) {
+            await this.config.runCommand('help', ['northstars', 'intents']);
+            return;
+        }
+        await super.run();
+    }
+}
+exports.default = NorthstarsIntents;

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

@@ -13,6 +13,7 @@ export default class NorthstarsList extends BaseCommand {
         json: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         status: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         limit: import("@oclif/core/interfaces").OptionFlag<number | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        cursor: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
     };
     run(): Promise<void>;
 }

package/dist/commands/northstars/list.js

@@ -21,14 +21,24 @@ class NorthstarsList extends base_command_1.BaseCommand {
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output as JSON' }),
         status: core_1.Flags.string({ description: 'Filter by status', options: [...northstars_1.NORTHSTAR_STATUSES] }),
-        limit: core_1.Flags.integer({ description: 'Max results (1..100)', min: 1, max: 100 }),
+        limit: core_1.Flags.integer({ description: 'Max results (1..100, default 50)', min: 1, max: 100 }),
+        cursor: core_1.Flags.string({
+            description: 'Continue from the next_cursor a previous page returned',
+        }),
     };
     async run() {
         this.requireAuth();
         const { flags } = await this.parse(NorthstarsList);
         let response;
         try {
-            response = await northstars_1.northstarApi.list({ status: flags.status, limit: flags.limit });
+            // Cursor paging is always on: it is the only list path that returns a cursor, and the
+            // default page is sent explicitly so it stays the 50 this command always listed.
+            response = await northstars_1.northstarApi.list({
+                status: flags.status,
+                limit: flags.limit ?? northstars_1.NORTHSTAR_LIST_DEFAULT_LIMIT,
+                pagination: 'cursor',
+                cursor: flags.cursor,
+            });
         }
         catch (err) {
             this.handleApiError(err);
@@ -37,7 +47,18 @@ class NorthstarsList extends base_command_1.BaseCommand {
             this.printJsonResult((0, list_output_1.listResult)(response));
             return;
         }
-        (0, northstars_1.renderNorthstarList)(response.data ?? [], (line) => this.log(line));
+        const rows = response.data ?? [];
+        (0, northstars_1.renderNorthstarList)(rows, (line) => this.log(line));
+        const footer = (0, northstars_1.listFooter)({
+            shown: rows.length,
+            hasMore: response.has_more === true,
+            limit: flags.limit ?? northstars_1.NORTHSTAR_LIST_DEFAULT_LIMIT,
+            max: 100,
+            nextCursor: response.next_cursor,
+            cursor: true,
+        });
+        if (footer)
+            this.log(footer);
     }
 }
 exports.default = NorthstarsList;