@kahitsan/plugin-sdk 0.5.0 → 0.5.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kahitsan/plugin-sdk",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "THE single public author surface for Hilinga plugins: createPlugin, auth guards, the workspace-scoped data surface (makeDataSurface) + migrations + withTenantContext, cross-plugin RPC, and the wire-protocol verify surface. One npm package, no -composite/-protocol companions. Self-contained, no kernel internals, no signing capability.",
   "type": "module",
   "main": "./dist/index.js",

package/src/dev/sqlite-pool.ts

@@ -182,6 +182,21 @@ function quote(name: string): string {
   return `"${name.replace(/"/g, '""')}"`;
 }
 
+/** For a degraded bare-aggregate read (no GROUP BY), return a single zero row keyed by
+ *  the projected column name so `rows[0].count` resolves to 0 instead of throwing.
+ *  Returns null when the SQL isn't a bare aggregate (caller then degrades to []). */
+function aggregateZeroRow<R>(sql: string): QueryResult<R> | null {
+  if (/\bGROUP\s+BY\b/i.test(sql)) return null;
+  const m = /^\s*SELECT\s+(.+?)\s+FROM\b/is.exec(sql);
+  if (!m) return null;
+  const proj = m[1].trim();
+  const fn = /^(count|sum|avg|min|max|coalesce)\s*\(/i.exec(proj);
+  if (!fn) return null;
+  const aliasM = /\bAS\s+"?([A-Za-z_][A-Za-z0-9_]*)"?\s*$/i.exec(proj);
+  const alias = aliasM ? aliasM[1] : fn[1].toLowerCase();
+  return { rows: [{ [alias]: 0 } as unknown as R], rowCount: 1 };
+}
+
 /** Classify a SQLite execution error into a tolerable empty result (Vision §5.1
  *  resilience), or null to rethrow. Tolerated: an idempotent re-`ADD COLUMN`; any
  *  statement during the boot migration window or any DDL (best-effort — the plugin
@@ -198,7 +213,15 @@ function tolerateError<R>(sql: string, err: Error): QueryResult<R> | null {
   if (/duplicate column name/i.test(msg) && /\bADD\s+COLUMN\b/i.test(sql)) return empty;
   const migrating = (globalThis as { __KSERP_DEV_MIGRATING__?: boolean }).__KSERP_DEV_MIGRATING__;
   if (migrating || IS_DDL.test(sql)) return (debug("skipped a Postgres-only statement"), empty);
-  if (RETURNS_ROWS.test(sql)) return (debug("degraded a Postgres-only read (empty result)"), empty);
+  if (RETURNS_ROWS.test(sql)) {
+    // A bare aggregate SELECT (COUNT/SUM/… with no GROUP BY) ALWAYS returns exactly one
+    // row in Postgres (0/NULL over an empty set). Degrading it to [] crashes the common
+    // list-count handler `rows[0].count`, so hand back a single zero row keyed by the
+    // projected column (explicit `AS alias`, else pg's default = the function name).
+    const agg = aggregateZeroRow<R>(sql);
+    if (agg) return (debug("degraded a Postgres-only aggregate (zero row)"), agg);
+    return (debug("degraded a Postgres-only read (empty result)"), empty);
+  }
   if (process.env.KSERP_DEV_SQL_DEBUG)
     console.error(`[dev-stub sqlite] FAILED SQL:\n${sql}\n→ ${msg}`);
   return null;