graphile-llm 1.11.1 → 1.11.3

package/esm/plugins/agent-discovery-plugin.d.ts

@@ -7,6 +7,11 @@
  *
  * Results are cached per-database with a TTL so the REST middleware
  * doesn't hit the database on every request.
+ *
+ * Discovery is keyed by `database_id`, as every other module lookup is: one
+ * serving database holds several tenants' schemas, so an unkeyed lookup does
+ * not fail — it resolves a neighbouring tenant's agent tables, and the cache
+ * then serves that for its whole TTL.
  */
 import { Pool } from 'pg';
 export interface AgentTableInfo {
@@ -24,6 +29,6 @@ export interface AgentDiscovery {
 export declare function clearAgentDiscoveryCache(): void;
 /**
  * Look up agent table info for a database, querying the module config table.
- * Results are cached per-database with a 60s TTL.
+ * Results are cached per database id with a 60s TTL.
  */
-export declare function getAgentDiscovery(pool: Pool, dbname: string): Promise<AgentDiscovery | null>;
+export declare function getAgentDiscovery(pool: Pool, databaseId: string): Promise<AgentDiscovery | null>;

package/esm/plugins/agent-discovery-plugin.js

@@ -7,6 +7,11 @@
  *
  * Results are cached per-database with a TTL so the REST middleware
  * doesn't hit the database on every request.
+ *
+ * Discovery is keyed by `database_id`, as every other module lookup is: one
+ * serving database holds several tenants' schemas, so an unkeyed lookup does
+ * not fail — it resolves a neighbouring tenant's agent tables, and the cache
+ * then serves that for its whole TTL.
  */
 import { ModuleConfigCache } from 'graphile-cache';
 // ─── Cache ──────────────────────────────────────────────────────────────────
@@ -27,20 +32,34 @@ const DISCOVERY_SQL = `
     acm.task_table_name
   FROM metaschema_modules_public.agent_chat_module acm
   JOIN metaschema_public.schema s ON s.id = acm.schema_id
+  WHERE acm.database_id = $1
   LIMIT 1
 `;
+/** The module (or the whole metaschema) is simply absent from this database. */
+const NOT_PROVISIONED = new Set([
+    '42P01', // undefined_table
+    '3F000' // invalid_schema_name
+]);
+const isNotProvisioned = (err) => typeof err === 'object' &&
+    err !== null &&
+    'code' in err &&
+    typeof err.code === 'string' &&
+    NOT_PROVISIONED.has(err.code);
 /**
  * Look up agent table info for a database, querying the module config table.
- * Results are cached per-database with a 60s TTL.
+ * Results are cached per database id with a 60s TTL.
  */
-export async function getAgentDiscovery(pool, dbname) {
-    const cached = agentDiscoveryCache.get(dbname);
+export async function getAgentDiscovery(pool, databaseId) {
+    if (!databaseId) {
+        throw new Error('getAgentDiscovery: databaseId is required');
+    }
+    const cached = agentDiscoveryCache.get(databaseId);
     if (cached !== undefined) {
         return cached;
     }
     let discovery = null;
     try {
-        const { rows } = await pool.query(DISCOVERY_SQL);
+        const { rows } = await pool.query(DISCOVERY_SQL, [databaseId]);
         if (rows.length > 0) {
             const row = rows[0];
             const schemaName = row.schema_name;
@@ -57,9 +76,12 @@ export async function getAgentDiscovery(pool, dbname) {
             };
         }
     }
-    catch {
-        // Module table doesn't exist in this database — not provisioned
+    catch (err) {
+        // Only the absence being probed for is swallowed. A dead pool or a bad
+        // databaseId reported as "not provisioned" is a silently agent-less API.
+        if (!isNotProvisioned(err))
+            throw err;
     }
-    agentDiscoveryCache.set(dbname, discovery);
+    agentDiscoveryCache.set(databaseId, discovery);
     return discovery;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "graphile-llm",
-  "version": "1.11.1",
+  "version": "1.11.3",
   "description": "LLM integration plugin for PostGraphile v5 — server-side text-to-vector embedding and text companion fields for pgvector columns",
   "author": "Constructive <developers@constructive.io>",
   "homepage": "https://github.com/constructive-io/constructive",
@@ -30,9 +30,9 @@
   },
   "dependencies": {
     "@agentic-kit/ollama": "2.13.0",
-    "@constructive-io/express-context": "^0.24.1",
+    "@constructive-io/express-context": "^0.24.2",
     "@constructive-io/llm-env": "^0.13.0",
-    "graphile-cache": "^4.10.1"
+    "graphile-cache": "^4.10.2"
   },
   "peerDependencies": {
     "@dataplan/pg": "1.0.3",
@@ -56,11 +56,11 @@
   },
   "devDependencies": {
     "@types/node": "^22.19.11",
-    "graphile-connection-filter": "^2.11.1",
-    "graphile-search": "^2.11.1",
-    "graphile-test": "^5.11.1",
+    "graphile-connection-filter": "^2.11.3",
+    "graphile-search": "^2.11.3",
+    "graphile-test": "^5.11.3",
     "makage": "^0.3.0",
-    "pgsql-test": "^5.10.1"
+    "pgsql-test": "^5.10.3"
   },
   "keywords": [
     "postgraphile",
@@ -75,5 +75,5 @@
     "ollama",
     "openai"
   ],
-  "gitHead": "6da0705f8aa49d62dc2defb8b135b71674611483"
+  "gitHead": "43121004a8bedd2c5fcf7bf33208079d55bad01f"
 }

package/plugins/agent-discovery-plugin.d.ts

@@ -7,6 +7,11 @@
  *
  * Results are cached per-database with a TTL so the REST middleware
  * doesn't hit the database on every request.
+ *
+ * Discovery is keyed by `database_id`, as every other module lookup is: one
+ * serving database holds several tenants' schemas, so an unkeyed lookup does
+ * not fail — it resolves a neighbouring tenant's agent tables, and the cache
+ * then serves that for its whole TTL.
  */
 import { Pool } from 'pg';
 export interface AgentTableInfo {
@@ -24,6 +29,6 @@ export interface AgentDiscovery {
 export declare function clearAgentDiscoveryCache(): void;
 /**
  * Look up agent table info for a database, querying the module config table.
- * Results are cached per-database with a 60s TTL.
+ * Results are cached per database id with a 60s TTL.
  */
-export declare function getAgentDiscovery(pool: Pool, dbname: string): Promise<AgentDiscovery | null>;
+export declare function getAgentDiscovery(pool: Pool, databaseId: string): Promise<AgentDiscovery | null>;

package/plugins/agent-discovery-plugin.js

@@ -8,6 +8,11 @@
  *
  * Results are cached per-database with a TTL so the REST middleware
  * doesn't hit the database on every request.
+ *
+ * Discovery is keyed by `database_id`, as every other module lookup is: one
+ * serving database holds several tenants' schemas, so an unkeyed lookup does
+ * not fail — it resolves a neighbouring tenant's agent tables, and the cache
+ * then serves that for its whole TTL.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.clearAgentDiscoveryCache = clearAgentDiscoveryCache;
@@ -31,20 +36,34 @@ const DISCOVERY_SQL = `
     acm.task_table_name
   FROM metaschema_modules_public.agent_chat_module acm
   JOIN metaschema_public.schema s ON s.id = acm.schema_id
+  WHERE acm.database_id = $1
   LIMIT 1
 `;
+/** The module (or the whole metaschema) is simply absent from this database. */
+const NOT_PROVISIONED = new Set([
+    '42P01', // undefined_table
+    '3F000' // invalid_schema_name
+]);
+const isNotProvisioned = (err) => typeof err === 'object' &&
+    err !== null &&
+    'code' in err &&
+    typeof err.code === 'string' &&
+    NOT_PROVISIONED.has(err.code);
 /**
  * Look up agent table info for a database, querying the module config table.
- * Results are cached per-database with a 60s TTL.
+ * Results are cached per database id with a 60s TTL.
  */
-async function getAgentDiscovery(pool, dbname) {
-    const cached = agentDiscoveryCache.get(dbname);
+async function getAgentDiscovery(pool, databaseId) {
+    if (!databaseId) {
+        throw new Error('getAgentDiscovery: databaseId is required');
+    }
+    const cached = agentDiscoveryCache.get(databaseId);
     if (cached !== undefined) {
         return cached;
     }
     let discovery = null;
     try {
-        const { rows } = await pool.query(DISCOVERY_SQL);
+        const { rows } = await pool.query(DISCOVERY_SQL, [databaseId]);
         if (rows.length > 0) {
             const row = rows[0];
             const schemaName = row.schema_name;
@@ -61,9 +80,12 @@ async function getAgentDiscovery(pool, dbname) {
             };
         }
     }
-    catch {
-        // Module table doesn't exist in this database — not provisioned
+    catch (err) {
+        // Only the absence being probed for is swallowed. A dead pool or a bad
+        // databaseId reported as "not provisioned" is a silently agent-less API.
+        if (!isNotProvisioned(err))
+            throw err;
     }
-    agentDiscoveryCache.set(dbname, discovery);
+    agentDiscoveryCache.set(databaseId, discovery);
     return discovery;
 }