hippo-memory 1.12.4 → 1.12.6

package/dist/src/cli.js

@@ -67,9 +67,11 @@ import { multihopSearch } from './multihop.js';
 import { getReranker } from './rerankers/index.js';
 import { computeSalience } from './salience.js';
 import { computeAmbientState, renderAmbientSummary } from './ambient.js';
+import { validateOwner, isStrictOwnerEnv } from './owner-validation.js';
 import { listDlq, replayDlqEntry } from './connectors/slack/dlq.js';
 import { backfillChannel } from './connectors/slack/backfill.js';
 import { slackHistoryFetcher } from './connectors/slack/web-client.js';
+import { addWorkspace as addSlackWorkspace, listWorkspaces as listSlackWorkspaces, removeWorkspace as removeSlackWorkspace, } from './connectors/slack/workspaces.js';
 import { cmdGithub } from './connectors/github/cli-impl.js';
 // ---------------------------------------------------------------------------
 // Helpers
@@ -529,7 +531,15 @@ async function cmdRemember(hippoRoot, text, flags) {
         console.error(`(kind='raw' is reserved for ingestion connectors; kind='archived' is internal.)`);
         process.exit(1);
     }
-    const ownerFlag = typeof flags['owner'] === 'string' ? flags['owner'] : null;
+    const ownerRaw = typeof flags['owner'] === 'string' ? flags['owner'] : null;
+    const ownerCheck = validateOwner(ownerRaw, { strict: isStrictOwnerEnv() });
+    if (!ownerCheck.ok) {
+        console.error(ownerCheck.message);
+        process.exit(1);
+    }
+    if (ownerCheck.message)
+        console.error(ownerCheck.message);
+    const ownerFlag = ownerCheck.value ?? null;
     const artifactRefFlag = typeof flags['artifact-ref'] === 'string' ? flags['artifact-ref'] : null;
     const scopeForEnvelope = typeof flags['scope'] === 'string' ? flags['scope'].trim() || null : null;
     // A5 stub auth: stamp tenant_id from env (HIPPO_TENANT) so recall isolation
@@ -4567,6 +4577,71 @@ function cmdSlackDlqReplay(hippoRoot, args, flags) {
     }
     console.log(`replay ok: status=${result.status} memory_id=${result.memoryId ?? '(none)'} retry_count=${result.retryCount}`);
 }
+function printSlackWorkspacesUsage() {
+    console.log('hippo slack workspaces <add|list|remove> [options]');
+    console.log('  add --team <T> --tenant <t>   Register a workspace (upserts on existing team-id)');
+    console.log('  list                          List all registered workspaces');
+    console.log('  remove --team <T>             Remove a workspace registration');
+}
+function cmdSlackWorkspacesAdd(hippoRoot, flags) {
+    if (flags['help']) {
+        printSlackWorkspacesUsage();
+        return;
+    }
+    const teamId = typeof flags['team'] === 'string' ? flags['team'].trim() : '';
+    const tenantId = typeof flags['tenant'] === 'string' ? flags['tenant'].trim() : '';
+    if (!teamId || !tenantId) {
+        console.error('Usage: hippo slack workspaces add --team <T> --tenant <t>');
+        process.exit(1);
+    }
+    const db = openHippoDb(hippoRoot);
+    try {
+        const ws = addSlackWorkspace(db, { teamId, tenantId });
+        console.log(`added: ${ws.teamId} -> ${ws.tenantId} (${ws.addedAt})`);
+    }
+    finally {
+        closeHippoDb(db);
+    }
+}
+function cmdSlackWorkspacesList(hippoRoot) {
+    const db = openHippoDb(hippoRoot);
+    try {
+        const items = listSlackWorkspaces(db);
+        if (items.length === 0) {
+            console.log('(no registered workspaces; routing via HIPPO_TENANT fallback)');
+            return;
+        }
+        for (const ws of items) {
+            console.log(`${ws.teamId}\t${ws.tenantId}\t${ws.addedAt}`);
+        }
+    }
+    finally {
+        closeHippoDb(db);
+    }
+}
+function cmdSlackWorkspacesRemove(hippoRoot, flags) {
+    if (flags['help']) {
+        printSlackWorkspacesUsage();
+        return;
+    }
+    const teamId = typeof flags['team'] === 'string' ? flags['team'].trim() : '';
+    if (!teamId) {
+        console.error('Usage: hippo slack workspaces remove --team <T>');
+        process.exit(1);
+    }
+    const db = openHippoDb(hippoRoot);
+    try {
+        const removed = removeSlackWorkspace(db, teamId);
+        if (!removed) {
+            console.error(`no workspace registered for team ${teamId}`);
+            process.exit(1);
+        }
+        console.log(`removed: ${teamId}`);
+    }
+    finally {
+        closeHippoDb(db);
+    }
+}
 function cmdSlack(hippoRoot, args, flags) {
     const sub = args[0];
     if (sub === 'backfill') {
@@ -4581,7 +4656,24 @@ function cmdSlack(hippoRoot, args, flags) {
         cmdSlackDlqReplay(hippoRoot, args, flags);
         return;
     }
-    console.error('Usage: hippo slack <backfill|dlq list|dlq replay <id> [--force]> [...]');
+    if (sub === 'workspaces') {
+        const action = args[1];
+        if (action === 'add') {
+            cmdSlackWorkspacesAdd(hippoRoot, flags);
+            return;
+        }
+        if (action === 'list') {
+            cmdSlackWorkspacesList(hippoRoot);
+            return;
+        }
+        if (action === 'remove') {
+            cmdSlackWorkspacesRemove(hippoRoot, flags);
+            return;
+        }
+        printSlackWorkspacesUsage();
+        process.exit(1);
+    }
+    console.error('Usage: hippo slack <backfill|dlq list|dlq replay <id> [--force]|workspaces add|workspaces list|workspaces remove> [...]');
     process.exit(1);
 }
 function printUsage() {
@@ -4983,12 +5075,23 @@ async function main() {
                     const tags = Array.isArray(tagsRaw)
                         ? tagsRaw.map(String)
                         : typeof tagsRaw === 'string' ? [tagsRaw] : undefined;
+                    // B2 v1.12.6 — validate --owner on the thin-client path too.
+                    // Failure on this path exits early so the user gets the same
+                    // validation experience whether or not a server is up.
+                    const thinOwnerRaw = typeof flags['owner'] === 'string' ? flags['owner'] : undefined;
+                    const thinOwnerCheck = validateOwner(thinOwnerRaw, { strict: isStrictOwnerEnv() });
+                    if (!thinOwnerCheck.ok) {
+                        console.error(thinOwnerCheck.message);
+                        process.exit(1);
+                    }
+                    if (thinOwnerCheck.message)
+                        console.error(thinOwnerCheck.message);
                     const remembered = await runViaServerIfAvailable(hippoRoot, async (info, apiKey) => {
                         const result = await client.remember(info.url, apiKey, {
                             content: text,
                             kind: rememberKindRaw,
                             scope: typeof flags['scope'] === 'string' ? flags['scope'] : undefined,
-                            owner: typeof flags['owner'] === 'string' ? flags['owner'] : undefined,
+                            owner: thinOwnerCheck.value,
                             artifactRef: typeof flags['artifact-ref'] === 'string' ? flags['artifact-ref'] : undefined,
                             tags,
                         });