@skrr-ai/cli 0.1.36 → 0.1.37

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

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

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

@@ -1,11 +1,34 @@
 import { BaseCommand } from '../../base-command';
+/**
+ * `labels list` — every label you can actually USE, by default.
+ *
+ * This used to default to `--scope user` and show one scope at a time, which
+ * is a faithful rendering of `GET /api/labels` and a poor answer to the
+ * question people ask. `scope` is an OWNERSHIP boundary, not a filter anyone
+ * browses by: a workspace-scoped label is attachable by every member, so
+ * leaving it out of the default view means the vocabulary you are told about
+ * is smaller than the one you have.
+ *
+ * It had a measurable cost. Three workspace-scoped labels existed and were
+ * invisible to the bare command, so nobody knew they were there — which is
+ * part of why the label system shipped complete and went unused, and why a
+ * second `dogfood` nearly got created beside one that already existed.
+ *
+ * `--scope` still narrows to exactly one scope, so every explicit invocation
+ * behaves as it always did. Only the bare command changed, and it changed to
+ * a superset.
+ *
+ * The merge itself is `lib/label-ref.collectLabelVocabulary` — the same
+ * function `--label <name>` resolves through. A separate copy here is how the
+ * name you can type and the name you can see start to disagree.
+ */
 export default class LabelsList extends BaseCommand {
     static description: string;
     static examples: string[];
     static flags: {
+        scope: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'applicable-to': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'include-archived': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
-        scope: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
         'scope-id': 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/labels/list.js

@@ -5,17 +5,48 @@ const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
 const format_1 = require("../../lib/format");
 const label_scope_1 = require("../../lib/label-scope");
+const label_ref_1 = require("../../lib/label-ref");
+/**
+ * `labels list` — every label you can actually USE, by default.
+ *
+ * This used to default to `--scope user` and show one scope at a time, which
+ * is a faithful rendering of `GET /api/labels` and a poor answer to the
+ * question people ask. `scope` is an OWNERSHIP boundary, not a filter anyone
+ * browses by: a workspace-scoped label is attachable by every member, so
+ * leaving it out of the default view means the vocabulary you are told about
+ * is smaller than the one you have.
+ *
+ * It had a measurable cost. Three workspace-scoped labels existed and were
+ * invisible to the bare command, so nobody knew they were there — which is
+ * part of why the label system shipped complete and went unused, and why a
+ * second `dogfood` nearly got created beside one that already existed.
+ *
+ * `--scope` still narrows to exactly one scope, so every explicit invocation
+ * behaves as it always did. Only the bare command changed, and it changed to
+ * a superset.
+ *
+ * The merge itself is `lib/label-ref.collectLabelVocabulary` — the same
+ * function `--label <name>` resolves through. A separate copy here is how the
+ * name you can type and the name you can see start to disagree.
+ */
 class LabelsList extends base_command_1.BaseCommand {
-    static description = 'List labels in a scope';
+    static description = 'List labels — every scope you can use, unless --scope narrows it';
     static examples = [
         '<%= config.bin %> labels list',
         '<%= config.bin %> labels list --applicable-to space',
+        '<%= config.bin %> labels list --scope user',
         '<%= config.bin %> labels list --scope workspace --scope-id <workspace-id>',
         '<%= config.bin %> labels list --include-archived --json',
     ];
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output as JSON' }),
         ...label_scope_1.labelScopeFlags,
+        // No default, unlike the shared flag: its absence is what selects the
+        // merged view, so "not passed" and "passed user" must stay distinguishable.
+        scope: core_1.Flags.string({
+            description: 'Narrow to ONE scope. Omit to list every scope you can use.',
+            options: ['user', 'workspace'],
+        }),
         'applicable-to': core_1.Flags.string({
             description: 'Only labels applicable to this entity type',
             options: [...label_scope_1.LABEL_APPLICABLE],
@@ -25,38 +56,49 @@ class LabelsList extends base_command_1.BaseCommand {
     async run() {
         this.requireAuth();
         const { flags } = await this.parse(LabelsList);
-        let scope;
-        try {
-            scope = (0, label_scope_1.resolveLabelScope)(flags);
-        }
-        catch (err) {
-            this.error(err.message, { exit: 1 });
-        }
-        let response;
+        const applicableTo = flags['applicable-to'];
+        let items = [];
         try {
-            response = await data_provider_1.dataService.getLabels({
-                ...scope,
-                ...(flags['applicable-to']
-                    ? { applicableTo: flags['applicable-to'] }
-                    : {}),
-                ...(flags['include-archived'] ? { includeArchived: true } : {}),
-            });
+            if (flags.scope) {
+                const scope = (0, label_scope_1.resolveLabelScope)(flags);
+                const response = (await data_provider_1.dataService.getLabels({
+                    ...scope,
+                    ...(applicableTo ? { applicableTo } : {}),
+                    ...(flags['include-archived'] ? { includeArchived: true } : {}),
+                }));
+                items = response?.data ?? [];
+            }
+            else {
+                items = await (0, label_ref_1.collectLabelVocabulary)({
+                    ...(applicableTo ? { applicableTo } : {}),
+                    ...(process.env.OVERSKY_WORKSPACE_ID
+                        ? { workspaceId: process.env.OVERSKY_WORKSPACE_ID }
+                        : {}),
+                    ...(flags['include-archived'] ? { includeArchived: true } : {}),
+                });
+            }
         }
         catch (err) {
+            if (err instanceof Error && !('status' in err))
+                this.error(err.message, { exit: 1 });
             this.handleApiError(err);
         }
         if (flags.json) {
-            this.log(JSON.stringify(response, null, 2));
+            this.log(JSON.stringify({ data: items }, null, 2));
             return;
         }
-        const items = response?.data ?? [];
         if (items.length === 0) {
-            this.log('No labels.');
+            this.log(flags.scope
+                ? 'No labels in this scope.'
+                : 'No labels you can use. Create one with `labels create --name <name> --applicable-to <entity>`.');
             return;
         }
         const rows = items.map((l) => ({
             id: String(l.id ?? '-'),
             name: String(l.name ?? '-'),
+            // Where it lives decides who else sees it, and it is the one column the
+            // old single-scope view never had to show.
+            scope: String(l.scope ?? '-'),
             color: String(l.color ?? '-'),
             appliesTo: Array.isArray(l.applicableTo) ? l.applicableTo.join(',') : '-',
             used: String(l.usageCount ?? 0),
@@ -65,6 +107,7 @@ class LabelsList extends base_command_1.BaseCommand {
         (0, format_1.renderTable)(rows, [
             { key: 'id', header: 'ID' },
             { key: 'name', header: 'NAME', maxWidth: 32 },
+            { key: 'scope', header: 'SCOPE' },
             { key: 'color', header: 'COLOR' },
             { key: 'appliesTo', header: 'APPLIES TO' },
             { key: 'used', header: 'USED' },

package/dist/commands/memory/governance/index.d.ts

@@ -0,0 +1,21 @@
+import { BaseCommand } from '../../../base-command';
+/**
+ * An agent's memory POLICY: which pools its runtime may read and write, how long
+ * retained facts are governed, and what deletion means.
+ *
+ * Its own route rather than a corner of `PATCH /api/agents/:id`, and the CLI
+ * uses the same door the web does. The distinction the route draws — and this
+ * command surfaces — is between a policy an owner has DELIBERATELY saved and the
+ * defaults an unconfigured agent reads as. Mongoose materializes the leaf
+ * defaults on every new agent row, so "has a policy" cannot be answered by the
+ * field being present; only `updatedAt` says a human decided something.
+ */
+export default class MemoryGovernanceShow extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static flags: {
+        agent: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+    };
+    run(): Promise<void>;
+}

package/dist/commands/memory/governance/index.js

@@ -0,0 +1,64 @@
+"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 agent_resolver_1 = require("../../../lib/agent-resolver");
+/**
+ * An agent's memory POLICY: which pools its runtime may read and write, how long
+ * retained facts are governed, and what deletion means.
+ *
+ * Its own route rather than a corner of `PATCH /api/agents/:id`, and the CLI
+ * uses the same door the web does. The distinction the route draws — and this
+ * command surfaces — is between a policy an owner has DELIBERATELY saved and the
+ * defaults an unconfigured agent reads as. Mongoose materializes the leaf
+ * defaults on every new agent row, so "has a policy" cannot be answered by the
+ * field being present; only `updatedAt` says a human decided something.
+ */
+class MemoryGovernanceShow extends base_command_1.BaseCommand {
+    static description = "Show an agent's memory governance policy";
+    static examples = [
+        '<%= config.bin %> memory governance --agent <agent-id>',
+        '<%= config.bin %> memory governance --agent <agent-id> --json',
+    ];
+    static flags = {
+        agent: core_1.Flags.string({
+            description: 'Agent ID (falls back to OVERSKY_AGENT_ID, then your primary agent)',
+        }),
+        json: core_1.Flags.boolean({ description: 'Output as JSON' }),
+    };
+    async run() {
+        this.requireAuth();
+        const { flags } = await this.parse(MemoryGovernanceShow);
+        let agentId;
+        try {
+            agentId = await (0, agent_resolver_1.resolveAgentId)(flags.agent);
+        }
+        catch (err) {
+            this.error(err.message, { exit: 1 });
+        }
+        let policy;
+        try {
+            policy = await data_provider_1.dataService.getAgentMemoryGovernance(agentId);
+        }
+        catch (err) {
+            this.handleApiError(err);
+        }
+        if (flags.json) {
+            this.log(this.agentJson(agentId, policy));
+            return;
+        }
+        const p = policy;
+        this.log(`Memory governance for agent ${agentId}`);
+        this.log(`  Read scopes:        ${(p.allowedReadScopes ?? []).join(', ') || '-'}`);
+        this.log(`  Write scopes:       ${(p.allowedWriteScopes ?? []).join(', ') || '-'}`);
+        this.log(`  Retention:          ${p.retentionDays ?? '-'} days`);
+        this.log(`  On deletion:        ${p.deletionMode ?? '-'}`);
+        this.log(`  Shared memory:      ${p.requireEvidenceForSharedMemory ? 'requires evidence' : 'no evidence required'}`);
+        this.log('');
+        this.log(p.updatedAt
+            ? `Configured ${String(p.updatedAt).slice(0, 10)}.`
+            : 'Never configured — these are the defaults a run reads, not a saved policy.');
+    }
+}
+exports.default = MemoryGovernanceShow;

package/dist/commands/memory/governance/set.d.ts

@@ -0,0 +1,24 @@
+import { BaseCommand } from '../../../base-command';
+/**
+ * Change an agent's memory policy. Requires EDIT on the agent.
+ *
+ * PARTIAL by design and worth being explicit about: the route merges over the
+ * leaves an owner has actually set, so omitting a flag leaves that leaf unset
+ * rather than freezing today's default into the row. That is why `--read-scope`
+ * REPLACES the list instead of adding to it — a list flag that appended would
+ * make "only agent memory" unreachable once anything wider had been granted.
+ */
+export default class MemoryGovernanceSet extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static flags: {
+        agent: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'read-scope': import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'write-scope': import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'retention-days': import("@oclif/core/lib/interfaces").OptionFlag<number | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'deletion-mode': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'require-evidence': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+    };
+    run(): Promise<void>;
+}

package/dist/commands/memory/governance/set.js

@@ -0,0 +1,95 @@
+"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 agent_resolver_1 = require("../../../lib/agent-resolver");
+const MEMORY_SCOPES = ['agent', 'user', 'workspace'];
+/**
+ * Change an agent's memory policy. Requires EDIT on the agent.
+ *
+ * PARTIAL by design and worth being explicit about: the route merges over the
+ * leaves an owner has actually set, so omitting a flag leaves that leaf unset
+ * rather than freezing today's default into the row. That is why `--read-scope`
+ * REPLACES the list instead of adding to it — a list flag that appended would
+ * make "only agent memory" unreachable once anything wider had been granted.
+ */
+class MemoryGovernanceSet extends base_command_1.BaseCommand {
+    static description = "Change an agent's memory governance policy (requires EDIT)";
+    static examples = [
+        '<%= config.bin %> memory governance set --agent <agent-id> --retention-days 90',
+        '<%= config.bin %> memory governance set --agent <agent-id> --read-scope agent --read-scope workspace',
+        '<%= config.bin %> memory governance set --agent <agent-id> --deletion-mode delete --no-require-evidence',
+    ];
+    static flags = {
+        agent: core_1.Flags.string({ description: 'Agent ID' }),
+        'read-scope': core_1.Flags.string({
+            description: 'Memory pool the runtime may READ (repeatable; replaces the list)',
+            options: [...MEMORY_SCOPES],
+            multiple: true,
+        }),
+        'write-scope': core_1.Flags.string({
+            description: 'Memory pool the runtime may WRITE (repeatable; replaces the list)',
+            options: [...MEMORY_SCOPES],
+            multiple: true,
+        }),
+        'retention-days': core_1.Flags.integer({
+            description: 'How long retained facts are governed (1-3650)',
+        }),
+        'deletion-mode': core_1.Flags.string({
+            description: 'What deletion means for this agent',
+            options: ['archive', 'delete'],
+        }),
+        'require-evidence': core_1.Flags.boolean({
+            description: 'Require evidence before a fact enters shared memory',
+            allowNo: true,
+        }),
+        json: core_1.Flags.boolean({ description: 'Output as JSON' }),
+    };
+    async run() {
+        this.requireAuth();
+        const { flags } = await this.parse(MemoryGovernanceSet);
+        const patch = {
+            ...(flags['read-scope'] === undefined
+                ? {}
+                : { allowedReadScopes: flags['read-scope'] }),
+            ...(flags['write-scope'] === undefined
+                ? {}
+                : { allowedWriteScopes: flags['write-scope'] }),
+            ...(flags['retention-days'] === undefined ? {} : { retentionDays: flags['retention-days'] }),
+            ...(flags['deletion-mode'] === undefined
+                ? {}
+                : { deletionMode: flags['deletion-mode'] }),
+            ...(flags['require-evidence'] === undefined
+                ? {}
+                : { requireEvidenceForSharedMemory: flags['require-evidence'] }),
+        };
+        if (Object.keys(patch).length === 0) {
+            this.error('Nothing to change. Pass at least one of --read-scope, --write-scope, ' +
+                '--retention-days, --deletion-mode, --require-evidence.', { exit: 1 });
+        }
+        let agentId;
+        try {
+            agentId = await (0, agent_resolver_1.resolveAgentId)(flags.agent);
+        }
+        catch (err) {
+            this.error(err.message, { exit: 1 });
+        }
+        let policy;
+        try {
+            policy = await data_provider_1.dataService.updateAgentMemoryGovernance(agentId, patch);
+        }
+        catch (err) {
+            this.handleApiError(err);
+        }
+        if (flags.json) {
+            this.log(this.agentJson(agentId, policy));
+            return;
+        }
+        this.log(`Updated memory governance for agent ${agentId}.`);
+        this.log(`  Read scopes:  ${(policy.allowedReadScopes ?? []).join(', ') || '-'}`);
+        this.log(`  Write scopes: ${(policy.allowedWriteScopes ?? []).join(', ') || '-'}`);
+        this.log(`  Retention:    ${policy.retentionDays ?? '-'} days`);
+    }
+}
+exports.default = MemoryGovernanceSet;

package/dist/commands/memory/provenance.d.ts

@@ -0,0 +1,18 @@
+import { BaseCommand } from '../../base-command';
+/**
+ * Where an agent's stored memories came from.
+ *
+ * The other half of governance: a policy says what a runtime MAY remember, and
+ * this says what it actually did and on whose evidence. Read-only, VIEW on the
+ * agent.
+ */
+export default class MemoryProvenance extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static flags: {
+        agent: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        limit: 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/memory/provenance.js

@@ -0,0 +1,66 @@
+"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 agent_resolver_1 = require("../../lib/agent-resolver");
+const format_1 = require("../../lib/format");
+/**
+ * Where an agent's stored memories came from.
+ *
+ * The other half of governance: a policy says what a runtime MAY remember, and
+ * this says what it actually did and on whose evidence. Read-only, VIEW on the
+ * agent.
+ */
+class MemoryProvenance extends base_command_1.BaseCommand {
+    static description = "Show where an agent's stored memories came from";
+    static examples = [
+        '<%= config.bin %> memory provenance --agent <agent-id>',
+        '<%= config.bin %> memory provenance --agent <agent-id> --limit 50 --json',
+    ];
+    static flags = {
+        agent: core_1.Flags.string({ description: 'Agent ID' }),
+        limit: core_1.Flags.integer({ description: 'How many entries to return' }),
+        json: core_1.Flags.boolean({ description: 'Output as JSON' }),
+    };
+    async run() {
+        this.requireAuth();
+        const { flags } = await this.parse(MemoryProvenance);
+        let agentId;
+        try {
+            agentId = await (0, agent_resolver_1.resolveAgentId)(flags.agent);
+        }
+        catch (err) {
+            this.error(err.message, { exit: 1 });
+        }
+        let response;
+        try {
+            response = await data_provider_1.dataService.getAgentMemoryProvenance(agentId, flags.limit === undefined ? undefined : { limit: flags.limit });
+        }
+        catch (err) {
+            this.handleApiError(err);
+        }
+        if (flags.json) {
+            this.log(this.agentJson(agentId, response));
+            return;
+        }
+        const payload = response;
+        const entries = Array.isArray(response)
+            ? response
+            : (payload?.entries ?? []);
+        if (entries.length === 0) {
+            this.log(`No memory provenance recorded for agent ${agentId}.`);
+            return;
+        }
+        (0, format_1.renderTable)(entries.map((entry) => ({
+            key: String(entry.key ?? '-'),
+            source: String(entry.source ?? entry.origin ?? '-'),
+            at: entry.updated_at ? String(entry.updated_at).slice(0, 10) : '-',
+        })), [
+            { key: 'key', header: 'KEY', verbatim: true },
+            { key: 'source', header: 'SOURCE', maxWidth: 30 },
+            { key: 'at', header: 'UPDATED' },
+        ], (line) => this.log(line));
+    }
+}
+exports.default = MemoryProvenance;

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

@@ -9,7 +9,10 @@ export default class SpacesCreate extends BaseCommand {
         description: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         status: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         color: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        icon: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'icon-color': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'lead-agent': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        team: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         'target-date': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         goal: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         label: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;

package/dist/commands/spaces/create.js

@@ -5,6 +5,7 @@ const data_provider_1 = require("@skrr-ai/data-provider");
 const outbox_1 = require("../../lib/outbox");
 const api_fetch_1 = require("../../lib/api-fetch");
 const base_command_1 = require("../../base-command");
+const label_ref_1 = require("../../lib/label-ref");
 const web_url_1 = require("../../lib/web-url");
 const prompt_1 = require("../../lib/prompt");
 const created_line_1 = require("../../lib/created-line");
@@ -13,6 +14,7 @@ class SpacesCreate extends base_command_1.BaseCommand {
     static examples = [
         '<%= config.bin %> spaces create --title "Q2 Launch"',
         '<%= config.bin %> spaces create --title "Support" --status in_progress --color "#4f46e5"',
+        '<%= config.bin %> spaces create --title "Launch" --icon Rocket --icon-color "#2DD4BF"',
         '<%= config.bin %> spaces create --title "Research" --lead-agent <agent-id>',
         '<%= config.bin %> spaces create --title "Frontend" --parent <space-id>',
         '<%= config.bin %> spaces create --title "Private notes" --parent <space-id> --no-inherit-access',
@@ -27,11 +29,18 @@ class SpacesCreate extends base_command_1.BaseCommand {
         description: core_1.Flags.string({ description: 'Space description' }),
         status: core_1.Flags.string({ description: 'Space status', options: [...data_provider_1.SPACE_STATUSES] }),
         color: core_1.Flags.string({ description: 'Space color (hex, e.g. #4f46e5)' }),
+        icon: core_1.Flags.string({
+            description: 'Space icon — a curated icon name (e.g. Rocket, Star) or an emoji',
+        }),
+        'icon-color': core_1.Flags.string({
+            description: 'Color applied to the space icon (hex, e.g. #2DD4BF)',
+        }),
         'lead-agent': core_1.Flags.string({ description: 'Space Leader agent ID' }),
+        team: core_1.Flags.string({ description: 'Team ID to attach this space to' }),
         'target-date': core_1.Flags.string({ description: 'Target date (ISO string)' }),
         goal: core_1.Flags.string({ description: 'Goal ID to link (repeatable)', multiple: true }),
         label: core_1.Flags.string({
-            description: 'Label ID to attach (repeatable, max 50). Must be applicable to spaces. After creation use `spaces labels add/remove`.',
+            description: 'Label to attach (repeatable, max 50) — id, or a name from `skrr labels list --applicable-to space`. An unknown name is refused, never created. After creation use `spaces labels add/remove`.',
             multiple: true,
         }),
         'group-chat': core_1.Flags.string({ description: 'Group chat ID to link this space to' }),
@@ -53,15 +62,36 @@ class SpacesCreate extends base_command_1.BaseCommand {
         catch (err) {
             this.error(err.message, { exit: 1 });
         }
+        // A name is resolved against the DECLARED vocabulary before the create, so
+        // a typo fails here — cheaply, with the near misses named — rather than
+        // minting a second spelling of a label somebody already has.
+        let resolvedLabelIds;
+        try {
+            const ids = await (0, label_ref_1.resolveLabelRefs)(flags.label, {
+                applicableTo: 'space',
+                ...(process.env.OVERSKY_WORKSPACE_ID
+                    ? { workspaceId: process.env.OVERSKY_WORKSPACE_ID }
+                    : {}),
+            });
+            resolvedLabelIds = ids.length > 0 ? ids : undefined;
+        }
+        catch (err) {
+            if (err instanceof label_ref_1.LabelRefError)
+                this.error(err.message, { exit: 1 });
+            this.handleApiError(err);
+        }
         const payload = (0, prompt_1.mergeDefined)(body, {
             title: flags.title,
             description: flags.description,
             status: flags.status,
             color: flags.color,
+            icon: flags.icon,
+            iconColor: flags['icon-color'],
             leadAgentId: flags['lead-agent'],
+            teamId: flags.team,
             targetDate: flags['target-date'],
             goalIds: flags.goal,
-            labelIds: flags.label,
+            labelIds: resolvedLabelIds,
             groupChatId: flags['group-chat'],
             parentSpaceId: flags.parent,
             inheritAccess: flags['inherit-access'],

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

@@ -3,6 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../../base-command");
+const label_ref_1 = require("../../../lib/label-ref");
 const web_url_1 = require("../../../lib/web-url");
 class SpacesLabelsAdd extends base_command_1.BaseCommand {
     static description = 'Attach a label to a space';
@@ -16,7 +17,7 @@ class SpacesLabelsAdd extends base_command_1.BaseCommand {
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output the updated space as JSON' }),
         label: core_1.Flags.string({
-            description: 'Label ID — must be applicable to spaces (see `skrr labels list --applicable-to space`)',
+            description: 'Label — id, or a name from `skrr labels list --applicable-to space`. An unknown name is refused, never created.',
             required: true,
         }),
     };
@@ -26,9 +27,25 @@ class SpacesLabelsAdd extends base_command_1.BaseCommand {
         // `spaces update` deliberately refuses labelIds — the server routes label
         // changes through this atomic $addToSet endpoint so concurrent edits can't
         // clobber each other. Re-attaching an already-attached label is a no-op.
+        // A name is resolved against the DECLARED vocabulary; an unknown one is
+        // refused with the near misses named, never created. See lib/label-ref.
+        let labelId = flags.label;
+        try {
+            [labelId] = await (0, label_ref_1.resolveLabelRefs)([flags.label], {
+                applicableTo: 'space',
+                ...(process.env.OVERSKY_WORKSPACE_ID
+                    ? { workspaceId: process.env.OVERSKY_WORKSPACE_ID }
+                    : {}),
+            });
+        }
+        catch (err) {
+            if (err instanceof label_ref_1.LabelRefError)
+                this.error(err.message, { exit: 1 });
+            this.handleApiError(err);
+        }
         let result;
         try {
-            result = await data_provider_1.dataService.addLabelToSpace({ spaceId: args.id, labelId: flags.label });
+            result = await data_provider_1.dataService.addLabelToSpace({ spaceId: args.id, labelId });
         }
         catch (err) {
             this.handleApiError(err);