fix(db,tests): export table name variables; fix PG omnibus cleanup

Export vectors_table, waypoints_table, users_table from db.ts following
the same pattern as memories_table and embed_log_table. All four are
set to SQLite bare names by default and overwritten with schema-qualified
PG names in the PG init block.

Update test_omnibus.ts cleanup() to use all exported table name variables
instead of hardcoded strings. Removes the redundant openmemory_vectors
fallback that existed only to paper over the missing variable.

Verified: clean tsc build + all omnibus tests pass against both SQLite
and PostgreSQL (postgres.itsa.house/openmemory_test).

Co-Authored-By: Claude <noreply@anthropic.com>
AI-Generated: true

Author: rbrenner

packages/openmemory-js/src/core/db.ts

@@ -60,6 +60,9 @@ let q: q_type;
 let vector_store: VectorStore;
 let memories_table: string;
 let embed_log_table = "embed_logs"; // default for SQLite; overwritten by PG init
+let vectors_table = "vectors";     // default for SQLite; overwritten by PG init
+let waypoints_table = "waypoints"; // default for SQLite; overwritten by PG init
+let users_table = "users";         // default for SQLite; overwritten by PG init
 let make_transaction: () => {
     begin(): Promise<void>;
     commit(): Promise<void>;
@@ -118,6 +121,9 @@ if (is_pg) {
     const w = `"${sc}"."openmemory_waypoints"`;
     const l = `"${sc}"."openmemory_embed_logs"`;
     embed_log_table = l;
+    vectors_table = v;
+    waypoints_table = w;
+    users_table = `"${sc}"."openmemory_users"`;
     const f = `"${sc}"."openmemory_memories_fts"`;
     const exec = async (sql: string, p: any[] = []) => {
         const c = cli || pg;
@@ -1108,4 +1114,4 @@ export const log_maint_op = async (
     }
 };
 
-export { q, transaction, make_transaction, all_async, get_async, run_async, memories_table, embed_log_table, vector_store };
+export { q, transaction, make_transaction, all_async, get_async, run_async, memories_table, embed_log_table, vectors_table, waypoints_table, users_table, vector_store };

packages/openmemory-js/tests/test_omnibus.ts

@@ -1,6 +1,6 @@
 
 import { Memory } from "../src/core/memory";
-import { run_async, q, embed_log_table, make_transaction } from "../src/core/db";
+import { run_async, q, embed_log_table, make_transaction, memories_table, vectors_table, waypoints_table, users_table } from "../src/core/db";
 
 // Mock time for evolutionary stability
 let mockTime: number | null = null;
@@ -10,11 +10,10 @@ Date.now = () => (mockTime !== null ? mockTime : originalNow());
 const sleep = (ms: number) => new Promise((resolve) => setTimeout(resolve, ms));
 
 async function cleanup(user_id: string) {
-    await run_async(`DELETE FROM memories`);
-    try { await run_async(`DELETE FROM vectors`); } catch { }
-    try { await run_async(`DELETE FROM openmemory_vectors`); } catch { }
-    try { await run_async(`DELETE FROM waypoints`); } catch { }
-    try { await run_async(`DELETE FROM users`); } catch { }
+    await run_async(`DELETE FROM ${memories_table}`);
+    try { await run_async(`DELETE FROM ${vectors_table}`); } catch { }
+    try { await run_async(`DELETE FROM ${waypoints_table}`); } catch { }
+    try { await run_async(`DELETE FROM ${users_table}`); } catch { }
     if (global.gc) global.gc();
 }