hippo-memory 1.70.0 → 1.72.0

package/dist/cli/owner-validation.js

@@ -1,30 +1,10 @@
-/**
- * --owner format validation.
- *
- * Documented MEMORY_ENVELOPE.md contract: owner = `user:<id>` | `agent:<id>`
- * with id ∈ `[A-Za-z0-9_-]+`.
- *
- * Default: WARN-ONLY (log + accept) to preserve back-compat with existing
- * scripted callers passing legacy owner strings. Set `HIPPO_STRICT_OWNER=1`
- * to reject + exit. Strict mode will become the default once real auth replaces
- * the stub (see `TODOS.md` for the migration path).
- */
+/** --owner validation per MEMORY_ENVELOPE.md: `user:<id>` | `agent:<id>`, id in `[A-Za-z0-9_-]+`.
+ * Warn-only by default for legacy scripted callers; `HIPPO_STRICT_OWNER=1` rejects and exits. */
 import { processEnv } from '../util/env.js';
 const OWNER_RE = /^(user|agent):[A-Za-z0-9_-]+$/;
 export const OWNER_CONTRACT_HINT = 'Must match ^(user|agent):[A-Za-z0-9_-]+$ (e.g. user:alice, agent:capture-bot).';
-/**
- * Pure validator. Returns `{ ok, value, message }`. Does NOT print or
- * `process.exit`. The CLI wrapper below handles side effects so this is
- * unit-testable.
- *
- * Behaviour:
- *   - owner undefined → ok=true, value=undefined, no message.
- *   - owner matches OWNER_RE → ok=true, value=owner.
- *   - owner doesn't match + strict=false → ok=true, value=owner (accept),
- *     message contains the warning text.
- *   - owner doesn't match + strict=true → ok=false, value=owner (echoed),
- *     message contains the error text.
- */
+/** Pure validator returning `{ ok, value, message }`; never prints or exits, so it is unit-testable.
+ * A non-matching owner is accepted with a warning unless strict, where ok=false with an error message. */
 export function validateOwner(owner, opts = {}) {
     if (owner === undefined || owner === null || owner === '') {
         return { ok: true, value: undefined, message: '' };
@@ -46,10 +26,7 @@ export function validateOwner(owner, opts = {}) {
             `Accepting for back-compat; set HIPPO_STRICT_OWNER=1 to reject.`,
     };
 }
-/**
- * Returns true when strict-owner enforcement is enabled via env var.
- * Centralised here so any future bump to default-strict is one edit.
- */
+/** True when strict-owner enforcement is enabled via env var. */
 export function isStrictOwnerEnv(env = processEnv()) {
     return env.HIPPO_STRICT_OWNER === '1';
 }

package/dist/cli/playbooks.d.ts

@@ -1,5 +1,4 @@
-import { type CliFlags } from './flag-values.js';
-export declare function cmdProcess(hippoRoot: string, args: string[], flags: CliFlags): void;
-export declare function cmdPolicy(hippoRoot: string, args: string[], flags: CliFlags): void;
-export declare function cmdSkill(hippoRoot: string, args: string[], flags: CliFlags): void;
+export declare const handleProcess: (ctx: import("./flag-values.js").CommandContext) => void;
+export declare const handlePolicy: (ctx: import("./flag-values.js").CommandContext) => void;
+export declare const handleSkill: (ctx: import("./flag-values.js").CommandContext) => void;
 //# sourceMappingURL=playbooks.d.ts.map

package/dist/cli/playbooks.js

@@ -1,27 +1,22 @@
 // First-class object verbs for processes, policies and skills.
-import { extractPathTags } from '../search/path-context.js';
 import * as processesModule from '../objects/processes.js';
 import * as policiesModule from '../objects/policies.js';
 import * as skillsModule from '../objects/skills.js';
-import { resolveTenantId } from '../store/tenant.js';
 import { printError } from './output.js';
-import { nonEmptyStringFlag, stringFlag } from './flag-values.js';
-import { requireInit } from './shared.js';
-import { closeObject, foundOrExit, idArgOrExit, listObjects, printLifecycleTail } from './object-verbs.js';
+import { isStringFlag, nonEmptyStringFlag } from './flag-values.js';
 import { errorMessage } from '../util/log.js';
-const PROCESS = { cmd: 'process', noun: 'Process', idLabel: 'process' };
-const POLICY = { cmd: 'policy', noun: 'Policy', idLabel: 'policy' };
-const SKILL = { cmd: 'skill', noun: 'Skill', idLabel: 'skill' };
+import { CliExit } from './exit.js';
+import { requiredText, versionedVerbs } from './versioned-verbs.js';
 // --step is a repeatable flag (collected into an array by parseArgs). A single
 // --step yields a string; normalize both to string[]. A value-less --step errors.
 function collectProcessSteps(stepRaw) {
     if (Array.isArray(stepRaw))
         return stepRaw;
-    if (typeof stepRaw === 'string')
+    if (isStringFlag(stepRaw))
         return [stepRaw];
     if (stepRaw === true) {
         printError('--step requires a value, e.g. hippo process new "<name>" --step "do X".');
-        process.exit(1);
+        throw new CliExit(1);
     }
     return [];
 }
@@ -31,117 +26,60 @@ function printProcessRow(proc) {
     if (proc.changeSummary)
         console.log(`    change: ${proc.changeSummary}`);
 }
-function processList(hippoRoot, tenantId, flags) {
-    listObjects(flags, {
-        plural: 'processes',
-        states: processesModule.VALID_PROCESS_STATES,
-        load: (opts) => processesModule.loadProcesses(hippoRoot, tenantId, opts),
-        printRow: printProcessRow,
-    });
-}
-function processGet(hippoRoot, tenantId, args) {
-    const id = idArgOrExit(args, 'Usage: hippo process get <id>', PROCESS.idLabel);
-    const proc = foundOrExit(processesModule.loadProcessById(hippoRoot, tenantId, id), PROCESS.noun, id);
-    console.log(`Process #${proc.id}`);
-    console.log(`  name: ${proc.processName}`);
-    console.log(`  status: ${proc.status}`);
-    console.log(`  version: ${proc.version}`);
-    if (proc.description)
-        console.log(`  description: ${proc.description}`);
-    if (proc.steps.length > 0) {
-        console.log(`  steps:`);
-        proc.steps.forEach((s, i) => console.log(`    ${i + 1}. ${s}`));
-    }
-    printLifecycleTail(proc);
-}
-function processSupersede(hippoRoot, tenantId, args, flags) {
-    const id = idArgOrExit(args, 'Usage: hippo process supersede <id> --step "<text>" [--step ...] [--change "<summary>"] [--description "<text>"]', PROCESS.idLabel);
-    const steps = collectProcessSteps(flags['step']);
-    if (steps.length === 0) {
-        printError('hippo process supersede requires at least one --step "<text>" for the new version.');
-        process.exit(1);
-    }
-    // A supersession is a new version of the SAME process, so the new row reuses the predecessor's name (stable identity
-    // across versions). loadProcessById gives an early not-found before the write; saveProcess's in-SAVEPOINT preflight
-    // is the authoritative active-state check.
-    const existing = foundOrExit(processesModule.loadProcessById(hippoRoot, tenantId, id), PROCESS.noun, id);
-    const changeSummary = nonEmptyStringFlag(flags, 'change');
-    const description = nonEmptyStringFlag(flags, 'description');
-    const procPathTags = extractPathTags(process.cwd());
-    const created = processesModule.saveProcess(hippoRoot, tenantId, {
-        processName: existing.processName,
-        steps,
-        description,
-        changeSummary,
-        supersedesProcessId: id,
-        extraTags: procPathTags,
-    });
-    console.log(`Process #${created.id} recorded (v${created.version}), superseding #${id}.`);
-    if (created.memoryId)
-        console.log(`  memory: ${created.memoryId}`);
-}
-function processClose(hippoRoot, tenantId, args) {
-    closeObject(args, PROCESS, (id) => processesModule.closeProcess(hippoRoot, tenantId, id));
-}
-export function cmdProcess(hippoRoot, args, flags) {
-    requireInit(hippoRoot);
-    const tenantId = resolveTenantId({});
-    const subcommand = args[0] ?? '';
-    if (subcommand === 'list')
-        return processList(hippoRoot, tenantId, flags);
-    if (subcommand === 'get')
-        return processGet(hippoRoot, tenantId, args);
-    if (subcommand === 'supersede')
-        return processSupersede(hippoRoot, tenantId, args, flags);
-    if (subcommand === 'close')
-        return processClose(hippoRoot, tenantId, args);
-    // Default subcommand: new (create). Accept both the documented
-    // `process new "<name>"` form and the bare `process "<name>"` form: for the
-    // `new` keyword the name is args[1], otherwise args[0] IS the name.
-    processCreate(hippoRoot, tenantId, subcommand === 'new' ? (args[1] ?? '') : subcommand, flags);
-}
-function processCreate(hippoRoot, tenantId, processName, flags) {
-    if (!processName) {
-        printError('Usage: hippo process new "<name>" --step "<text>" [--step ...] [--description "<text>"]');
-        printError('       hippo process list [--status active|superseded|closed|all] [--limit N]');
-        printError('       hippo process get <id>');
-        printError('       hippo process supersede <id> --step "<text>" [--change "<summary>"]');
-        printError('       hippo process close <id>');
-        process.exit(1);
-    }
-    const steps = collectProcessSteps(flags['step']);
-    const description = nonEmptyStringFlag(flags, 'description');
-    const procPathTags = extractPathTags(process.cwd());
-    const created = processesModule.saveProcess(hippoRoot, tenantId, {
-        processName,
-        steps,
-        description,
-        extraTags: procPathTags,
-    });
-    console.log(`Process recorded: #${created.id} (v${created.version}, ${created.steps.length} steps)`);
-    if (created.memoryId)
-        console.log(`  memory: ${created.memoryId}`);
-}
+const processKind = (hippoRoot, tenantId) => ({
+    names: { cmd: 'process', noun: 'Process', idLabel: 'process' },
+    plural: 'processes',
+    states: processesModule.VALID_PROCESS_STATES,
+    usage: [
+        'Usage: hippo process new "<name>" --step "<text>" [--step ...] [--description "<text>"]',
+        '       hippo process list [--status active|superseded|closed|all] [--limit N]',
+        '       hippo process get <id>',
+        '       hippo process supersede <id> --step "<text>" [--change "<summary>"]',
+        '       hippo process close <id>',
+    ],
+    supersedeUsage: 'Usage: hippo process supersede <id> --step "<text>" [--step ...] [--change "<summary>"] [--description "<text>"]',
+    bodyRequired: 'at least one --step "<text>"',
+    body: (flags, verb) => {
+        const steps = collectProcessSteps(flags['step']);
+        // A new process may start with no steps; a new version must say what its steps are.
+        if (verb === 'supersede' && steps.length === 0)
+            return undefined;
+        return { steps, description: nonEmptyStringFlag(flags, 'description') };
+    },
+    keyOf: (proc) => proc.processName,
+    save: (processName, body, extraTags, from) => processesModule.saveProcess(hippoRoot, tenantId, {
+        processName, ...body, extraTags, changeSummary: from?.changeSummary, supersedesProcessId: from?.id,
+    }),
+    recorded: (proc) => `Process recorded: #${proc.id} (v${proc.version}, ${proc.steps.length} steps)`,
+    list: (opts) => processesModule.loadProcesses(hippoRoot, tenantId, opts),
+    loadById: (id) => processesModule.loadProcessById(hippoRoot, tenantId, id),
+    close: (id) => processesModule.closeProcess(hippoRoot, tenantId, id),
+    printRow: printProcessRow,
+    printDetail: (proc) => {
+        console.log(`  name: ${proc.processName}`);
+        console.log(`  status: ${proc.status}`);
+        console.log(`  version: ${proc.version}`);
+        if (proc.description)
+            console.log(`  description: ${proc.description}`);
+        if (proc.steps.length > 0) {
+            console.log(`  steps:`);
+            proc.steps.forEach((s, i) => console.log(`    ${i + 1}. ${s}`));
+        }
+    },
+});
+export const handleProcess = versionedVerbs(processKind);
+const policyRange = (p) => (p.validTo ? `${p.validFrom}..${p.validTo}` : `${p.validFrom}..(open)`);
 function printPolicyRow(p) {
-    const range = p.validTo ? `${p.validFrom}..${p.validTo}` : `${p.validFrom}..(open)`;
-    console.log(`#${p.id} [${p.status}] v${p.version} ${range} memory=${p.memoryId ?? '-'}`);
+    console.log(`#${p.id} [${p.status}] v${p.version} ${policyRange(p)} memory=${p.memoryId ?? '-'}`);
     console.log(`    ${p.policyName}: ${p.policyText}`);
     if (p.changeSummary)
         console.log(`    change: ${p.changeSummary}`);
 }
-function policyList(hippoRoot, tenantId, flags) {
-    listObjects(flags, {
-        plural: 'policies',
-        states: policiesModule.VALID_POLICY_STATES,
-        load: (opts) => policiesModule.loadPolicies(hippoRoot, tenantId, opts),
-        printRow: printPolicyRow,
-    });
-}
 function policyAsOf(hippoRoot, tenantId, args, flags) {
     const dateRaw = args[1];
     if (!dateRaw) {
         printError('Usage: hippo policy asof <iso-date> [--name "<policy>"]');
-        process.exit(1);
+        throw new CliExit(1);
     }
     const name = nonEmptyStringFlag(flags, 'name');
     let results;
@@ -150,7 +88,7 @@ function policyAsOf(hippoRoot, tenantId, args, flags) {
     }
     catch (e) {
         printError(errorMessage(e));
-        process.exit(1);
+        throw new CliExit(1);
     }
     if (results.length === 0) {
         console.log(`No active policies in force at ${dateRaw}${name ? ` for "${name}"` : ''}.`);
@@ -160,97 +98,46 @@ function policyAsOf(hippoRoot, tenantId, args, flags) {
     for (const p of results)
         printPolicyRow(p);
 }
-function policyGet(hippoRoot, tenantId, args) {
-    const id = idArgOrExit(args, 'Usage: hippo policy get <id>', POLICY.idLabel);
-    const p = foundOrExit(policiesModule.loadPolicyById(hippoRoot, tenantId, id), POLICY.noun, id);
-    console.log(`Policy #${p.id}`);
-    console.log(`  name: ${p.policyName}`);
-    console.log(`  text: ${p.policyText}`);
-    console.log(`  status: ${p.status}`);
-    console.log(`  version: ${p.version}`);
-    console.log(`  valid_from: ${p.validFrom}`);
-    console.log(`  valid_to: ${p.validTo ?? '(open-ended)'}`);
-    printLifecycleTail(p);
-}
-function policySupersede(hippoRoot, tenantId, args, flags) {
-    const id = idArgOrExit(args, 'Usage: hippo policy supersede <id> --text "<rule>" [--from <iso>] [--to <iso>] [--change "<summary>"]', POLICY.idLabel);
-    const textRaw = stringFlag(flags, 'text');
-    if (!textRaw?.trim()) {
-        printError('hippo policy supersede requires --text "<rule>" for the new version.');
-        process.exit(1);
-    }
-    const existing = foundOrExit(policiesModule.loadPolicyById(hippoRoot, tenantId, id), POLICY.noun, id);
-    try {
-        const created = policiesModule.savePolicy(hippoRoot, tenantId, {
-            policyName: existing.policyName,
-            policyText: textRaw,
-            validFrom: nonEmptyStringFlag(flags, 'from'),
-            validTo: nonEmptyStringFlag(flags, 'to'),
-            changeSummary: nonEmptyStringFlag(flags, 'change'),
-            supersedesPolicyId: id,
-            extraTags: extractPathTags(process.cwd()),
-        });
-        console.log(`Policy #${created.id} recorded (v${created.version}), superseding #${id}.`);
-        if (created.memoryId)
-            console.log(`  memory: ${created.memoryId}`);
-    }
-    catch (e) {
-        printError(errorMessage(e));
-        process.exit(1);
-    }
-}
-function policyClose(hippoRoot, tenantId, args) {
-    closeObject(args, POLICY, (id) => policiesModule.closePolicy(hippoRoot, tenantId, id));
-}
-export function cmdPolicy(hippoRoot, args, flags) {
-    requireInit(hippoRoot);
-    const tenantId = resolveTenantId({});
-    const subcommand = args[0] ?? '';
-    if (subcommand === 'list')
-        return policyList(hippoRoot, tenantId, flags);
-    if (subcommand === 'asof')
-        return policyAsOf(hippoRoot, tenantId, args, flags);
-    if (subcommand === 'get')
-        return policyGet(hippoRoot, tenantId, args);
-    if (subcommand === 'supersede')
-        return policySupersede(hippoRoot, tenantId, args, flags);
-    if (subcommand === 'close')
-        return policyClose(hippoRoot, tenantId, args);
-    policyCreate(hippoRoot, tenantId, args, flags);
-}
-function policyCreate(hippoRoot, tenantId, args, flags) {
-    const subcommand = args[0] ?? '';
-    // Default subcommand: new (create). Accept both `policy new "<name>"` and the
-    // bare `policy "<name>"` form: for the `new` keyword the name is args[1].
-    const policyName = subcommand === 'new' ? (args[1] ?? '') : subcommand;
-    const textRaw = stringFlag(flags, 'text');
-    if (!policyName || !textRaw?.trim()) {
-        printError('Usage: hippo policy new "<name>" --text "<rule>" [--from <iso>] [--to <iso>]');
-        printError('       hippo policy list [--status active|superseded|closed|all] [--limit N]');
-        printError('       hippo policy get <id>');
-        printError('       hippo policy asof <iso-date> [--name "<policy>"]');
-        printError('       hippo policy supersede <id> --text "<rule>" [--from] [--to] [--change "<summary>"]');
-        printError('       hippo policy close <id>');
-        process.exit(1);
-    }
-    try {
-        const created = policiesModule.savePolicy(hippoRoot, tenantId, {
-            policyName,
-            policyText: textRaw,
-            validFrom: nonEmptyStringFlag(flags, 'from'),
-            validTo: nonEmptyStringFlag(flags, 'to'),
-            extraTags: extractPathTags(process.cwd()),
-        });
-        const range = created.validTo ? `${created.validFrom}..${created.validTo}` : `${created.validFrom}..(open)`;
-        console.log(`Policy recorded: #${created.id} (v${created.version}, effective ${range})`);
-        if (created.memoryId)
-            console.log(`  memory: ${created.memoryId}`);
-    }
-    catch (e) {
-        printError(errorMessage(e));
-        process.exit(1);
-    }
-}
+const policyKind = (hippoRoot, tenantId) => ({
+    names: { cmd: 'policy', noun: 'Policy', idLabel: 'policy' },
+    plural: 'policies',
+    states: policiesModule.VALID_POLICY_STATES,
+    usage: [
+        'Usage: hippo policy new "<name>" --text "<rule>" [--from <iso>] [--to <iso>]',
+        '       hippo policy list [--status active|superseded|closed|all] [--limit N]',
+        '       hippo policy get <id>',
+        '       hippo policy asof <iso-date> [--name "<policy>"]',
+        '       hippo policy supersede <id> --text "<rule>" [--from] [--to] [--change "<summary>"]',
+        '       hippo policy close <id>',
+    ],
+    supersedeUsage: 'Usage: hippo policy supersede <id> --text "<rule>" [--from <iso>] [--to <iso>] [--change "<summary>"]',
+    bodyRequired: '--text "<rule>"',
+    body: (flags) => {
+        const policyText = requiredText(flags, 'text');
+        if (policyText === undefined)
+            return undefined;
+        return { policyText, validFrom: nonEmptyStringFlag(flags, 'from'), validTo: nonEmptyStringFlag(flags, 'to') };
+    },
+    keyOf: (p) => p.policyName,
+    save: (policyName, body, extraTags, from) => policiesModule.savePolicy(hippoRoot, tenantId, {
+        policyName, ...body, extraTags, changeSummary: from?.changeSummary, supersedesPolicyId: from?.id,
+    }),
+    recorded: (p) => `Policy recorded: #${p.id} (v${p.version}, effective ${policyRange(p)})`,
+    list: (opts) => policiesModule.loadPolicies(hippoRoot, tenantId, opts),
+    loadById: (id) => policiesModule.loadPolicyById(hippoRoot, tenantId, id),
+    close: (id) => policiesModule.closePolicy(hippoRoot, tenantId, id),
+    printRow: printPolicyRow,
+    printDetail: (p) => {
+        console.log(`  name: ${p.policyName}`);
+        console.log(`  text: ${p.policyText}`);
+        console.log(`  status: ${p.status}`);
+        console.log(`  version: ${p.version}`);
+        console.log(`  valid_from: ${p.validFrom}`);
+        console.log(`  valid_to: ${p.validTo ?? '(open-ended)'}`);
+    },
+    extra: { sub: 'asof', run: (args, flags) => policyAsOf(hippoRoot, tenantId, args, flags) },
+});
+export const handlePolicy = versionedVerbs(policyKind);
 function printSkillRow(s) {
     const trig = s.trigger ? ` when="${s.trigger}"` : '';
     console.log(`#${s.id} [${s.status}] v${s.version}${trig} memory=${s.memoryId ?? '-'}`);
@@ -258,14 +145,6 @@ function printSkillRow(s) {
     if (s.changeSummary)
         console.log(`    change: ${s.changeSummary}`);
 }
-function skillList(hippoRoot, tenantId, flags) {
-    listObjects(flags, {
-        plural: 'skills',
-        states: skillsModule.VALID_SKILL_STATES,
-        load: (opts) => skillsModule.loadSkills(hippoRoot, tenantId, opts),
-        printRow: printSkillRow,
-    });
-}
 function skillExport(hippoRoot, tenantId) {
     const md = skillsModule.exportSkills(hippoRoot, tenantId);
     if (!md) {
@@ -274,92 +153,44 @@ function skillExport(hippoRoot, tenantId) {
     }
     console.log(md);
 }
-function skillGet(hippoRoot, tenantId, args) {
-    const id = idArgOrExit(args, 'Usage: hippo skill get <id>', SKILL.idLabel);
-    const s = foundOrExit(skillsModule.loadSkillById(hippoRoot, tenantId, id), SKILL.noun, id);
-    console.log(`Skill #${s.id}`);
-    console.log(`  name: ${s.skillName}`);
-    console.log(`  status: ${s.status}`);
-    console.log(`  version: ${s.version}`);
-    if (s.trigger)
-        console.log(`  when: ${s.trigger}`);
-    console.log(`  instructions: ${s.instructions}`);
-    printLifecycleTail(s);
-}
-function skillSupersede(hippoRoot, tenantId, args, flags) {
-    const id = idArgOrExit(args, 'Usage: hippo skill supersede <id> --instructions "<text>" [--trigger "<when>"] [--change "<summary>"]', SKILL.idLabel);
-    const instrRaw = stringFlag(flags, 'instructions');
-    if (!instrRaw?.trim()) {
-        printError('hippo skill supersede requires --instructions "<text>" for the new version.');
-        process.exit(1);
-    }
-    const existing = foundOrExit(skillsModule.loadSkillById(hippoRoot, tenantId, id), SKILL.noun, id);
-    try {
-        const created = skillsModule.saveSkill(hippoRoot, tenantId, {
-            skillName: existing.skillName,
-            instructions: instrRaw,
-            trigger: nonEmptyStringFlag(flags, 'trigger'),
-            changeSummary: nonEmptyStringFlag(flags, 'change'),
-            supersedesSkillId: id,
-            extraTags: extractPathTags(process.cwd()),
-        });
-        console.log(`Skill #${created.id} recorded (v${created.version}), superseding #${id}.`);
-        if (created.memoryId)
-            console.log(`  memory: ${created.memoryId}`);
-    }
-    catch (e) {
-        printError(errorMessage(e));
-        process.exit(1);
-    }
-}
-function skillClose(hippoRoot, tenantId, args) {
-    closeObject(args, SKILL, (id) => skillsModule.closeSkill(hippoRoot, tenantId, id));
-}
-export function cmdSkill(hippoRoot, args, flags) {
-    requireInit(hippoRoot);
-    const tenantId = resolveTenantId({});
-    const subcommand = args[0] ?? '';
-    if (subcommand === 'list')
-        return skillList(hippoRoot, tenantId, flags);
-    if (subcommand === 'export')
-        return skillExport(hippoRoot, tenantId);
-    if (subcommand === 'get')
-        return skillGet(hippoRoot, tenantId, args);
-    if (subcommand === 'supersede')
-        return skillSupersede(hippoRoot, tenantId, args, flags);
-    if (subcommand === 'close')
-        return skillClose(hippoRoot, tenantId, args);
-    skillCreate(hippoRoot, tenantId, args, flags);
-}
-function skillCreate(hippoRoot, tenantId, args, flags) {
-    const subcommand = args[0] ?? '';
-    // Default subcommand: new (create). Accept both `skill new "<name>"` and the
-    // bare `skill "<name>"` form: for the `new` keyword the name is args[1].
-    const skillName = subcommand === 'new' ? (args[1] ?? '') : subcommand;
-    const instrRaw = stringFlag(flags, 'instructions');
-    if (!skillName || !instrRaw?.trim()) {
-        printError('Usage: hippo skill new "<name>" --instructions "<text>" [--trigger "<when>"]');
-        printError('       hippo skill list [--status active|superseded|closed|all] [--limit N]');
-        printError('       hippo skill get <id>');
-        printError('       hippo skill export   (render active skills as an AGENTS.md/CLAUDE.md block)');
-        printError('       hippo skill supersede <id> --instructions "<text>" [--trigger] [--change "<summary>"]');
-        printError('       hippo skill close <id>');
-        process.exit(1);
-    }
-    try {
-        const created = skillsModule.saveSkill(hippoRoot, tenantId, {
-            skillName,
-            instructions: instrRaw,
-            trigger: nonEmptyStringFlag(flags, 'trigger'),
-            extraTags: extractPathTags(process.cwd()),
-        });
-        console.log(`Skill recorded: #${created.id} (v${created.version})`);
-        if (created.memoryId)
-            console.log(`  memory: ${created.memoryId}`);
-    }
-    catch (e) {
-        printError(errorMessage(e));
-        process.exit(1);
-    }
-}
+const skillKind = (hippoRoot, tenantId) => ({
+    names: { cmd: 'skill', noun: 'Skill', idLabel: 'skill' },
+    plural: 'skills',
+    states: skillsModule.VALID_SKILL_STATES,
+    usage: [
+        'Usage: hippo skill new "<name>" --instructions "<text>" [--trigger "<when>"]',
+        '       hippo skill list [--status active|superseded|closed|all] [--limit N]',
+        '       hippo skill get <id>',
+        '       hippo skill export   (render active skills as an AGENTS.md/CLAUDE.md block)',
+        '       hippo skill supersede <id> --instructions "<text>" [--trigger] [--change "<summary>"]',
+        '       hippo skill close <id>',
+    ],
+    supersedeUsage: 'Usage: hippo skill supersede <id> --instructions "<text>" [--trigger "<when>"] [--change "<summary>"]',
+    bodyRequired: '--instructions "<text>"',
+    body: (flags) => {
+        const instructions = requiredText(flags, 'instructions');
+        if (instructions === undefined)
+            return undefined;
+        return { instructions, trigger: nonEmptyStringFlag(flags, 'trigger') };
+    },
+    keyOf: (s) => s.skillName,
+    save: (skillName, body, extraTags, from) => skillsModule.saveSkill(hippoRoot, tenantId, {
+        skillName, ...body, extraTags, changeSummary: from?.changeSummary, supersedesSkillId: from?.id,
+    }),
+    recorded: (s) => `Skill recorded: #${s.id} (v${s.version})`,
+    list: (opts) => skillsModule.loadSkills(hippoRoot, tenantId, opts),
+    loadById: (id) => skillsModule.loadSkillById(hippoRoot, tenantId, id),
+    close: (id) => skillsModule.closeSkill(hippoRoot, tenantId, id),
+    printRow: printSkillRow,
+    printDetail: (s) => {
+        console.log(`  name: ${s.skillName}`);
+        console.log(`  status: ${s.status}`);
+        console.log(`  version: ${s.version}`);
+        if (s.trigger)
+            console.log(`  when: ${s.trigger}`);
+        console.log(`  instructions: ${s.instructions}`);
+    },
+    extra: { sub: 'export', run: () => skillExport(hippoRoot, tenantId) },
+});
+export const handleSkill = versionedVerbs(skillKind);
 //# sourceMappingURL=playbooks.js.map

package/dist/cli/predictions.d.ts

@@ -0,0 +1,3 @@
+import { type CommandContext } from './flag-values.js';
+export declare function handlePredict({ hippoRoot, tenantId, args, flags }: CommandContext): Promise<void>;
+//# sourceMappingURL=predictions.d.ts.map