@kahitsan/plugin-sdk 0.5.0-staging.29 → 0.5.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kahitsan/plugin-sdk",
-  "version": "0.5.0-staging.29",
+  "version": "0.5.0",
   "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,21 +182,6 @@ 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
@@ -213,15 +198,7 @@ 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)) {
-    // 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 (RETURNS_ROWS.test(sql)) 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;