@gscdump/engine-duckdb-wasm 0.37.3 → 0.37.4

package/dist/index.mjs

@@ -2,7 +2,7 @@ import { arrowToRows } from "@gscdump/engine/arrow";
 import { DefaultLogger, NoopLogger, entityKind, sql } from "drizzle-orm";
 import { PgAsyncDatabase, PgAsyncPreparedQuery, PgAsyncSession, PgDialect } from "drizzle-orm/pg-core";
 import { buildRelations } from "drizzle-orm/relations";
-import { toIsoDate } from "gscdump";
+import { toIsoDate } from "gscdump/dates";
 import { countries, dates, drizzleSchema as schema, hourly_pages, page_queries, pages, queries } from "@gscdump/engine/schema";
 import { createScopedHelpers } from "@gscdump/engine/scope";
 import { runAnalyzerFromSource } from "@gscdump/engine/analyzer";
@@ -505,6 +505,9 @@ async function strikingMomentum(runner, opts = {}) {
 function createOpfsHandleRegistry(backend) {
 	const entries = /* @__PURE__ */ new Map();
 	const pending = /* @__PURE__ */ new Map();
+	function reportCleanupFailure(operation, error) {
+		console.warn(`[gscdump/engine-duckdb-wasm] ${operation} failed`, error);
+	}
 	async function acquire(name, openHandle) {
 		const existing = entries.get(name);
 		if (existing) {
@@ -513,7 +516,7 @@ function createOpfsHandleRegistry(backend) {
 		}
 		const inflight = pending.get(name);
 		if (inflight) {
-			await inflight.catch(() => {});
+			await inflight.then(() => void 0, () => void 0);
 			const settled = entries.get(name);
 			if (settled) {
 				settled.refs++;
@@ -543,7 +546,9 @@ function createOpfsHandleRegistry(backend) {
 			entry.refs--;
 			if (entry.refs > 0) continue;
 			entries.delete(name);
-			await backend.drop(name).catch(() => {});
+			await backend.drop(name).catch((error) => {
+				reportCleanupFailure(`dropping OPFS handle ${name}`, error);
+			});
 		}
 	}
 	const viewRefs = /* @__PURE__ */ new Map();
@@ -570,7 +575,9 @@ function createOpfsHandleRegistry(backend) {
 			return;
 		}
 		viewRefs.delete(key);
-		await drop().catch(() => {});
+		await drop().catch((error) => {
+			reportCleanupFailure(`dropping view ${key}`, error);
+		});
 	}
 	return {
 		acquire,
@@ -609,10 +616,25 @@ const IDENT_RE = /^[A-Z_][\w$]*$/i;
 function nowMs() {
 	return globalThis.performance?.now?.() ?? Date.now();
 }
+function isNotFoundError(error) {
+	return typeof error === "object" && error !== null && error.name === "NotFoundError";
+}
+function reportBestEffortFailure(operation, error) {
+	console.warn(`[gscdump/engine-duckdb-wasm] ${operation} failed`, error);
+}
+async function attemptCleanup(operation, cleanup) {
+	try {
+		await cleanup();
+	} catch (error) {
+		reportBestEffortFailure(operation, error);
+	}
+}
 function emitTiming(onTiming, info) {
 	try {
 		onTiming?.(info);
-	} catch {}
+	} catch (error) {
+		reportBestEffortFailure("timing callback", error);
+	}
 }
 async function timed(onTiming, stage, meta, fn) {
 	const started = nowMs();
@@ -694,7 +716,11 @@ async function sweepStaleEntries(root, registry, expectedByTable) {
 	for await (const name of dir.keys()) {
 		const m = matchers.find((m) => m.re.test(name));
 		if (!m || m.expected.has(name) || registry.refs(name) > 0) continue;
-		await root.removeEntry(name).catch(() => {});
+		try {
+			await root.removeEntry(name);
+		} catch (error) {
+			if (!isNotFoundError(error)) reportBestEffortFailure(`removing stale OPFS entry ${name}`, error);
+		}
 	}
 }
 async function requestPersistentStorage() {
@@ -745,7 +771,8 @@ async function readOpfsSnapshotFile(table, contentHash, index, expectedBytes) {
 		const file = await (await root.getFileHandle(name)).getFile();
 		if (file.size !== expectedBytes) return null;
 		return new Uint8Array(await file.arrayBuffer());
-	} catch {
+	} catch (error) {
+		if (!isNotFoundError(error)) reportBestEffortFailure(`reading OPFS snapshot ${table}`, error);
 		return null;
 	}
 }
@@ -758,7 +785,9 @@ async function materialiseFile(root, name, file, fetchImpl, fetchInit, signal) {
 			handle,
 			outcome: "cache-hit"
 		};
-	} catch {}
+	} catch (error) {
+		if (!isNotFoundError(error)) throw error;
+	}
 	signal?.throwIfAborted();
 	const deadline = AbortSignal.timeout(DOWNLOAD_DEADLINE_MS);
 	const fetchSignal = signal ? AbortSignal.any([signal, deadline]) : deadline;
@@ -775,8 +804,8 @@ async function materialiseFile(root, name, file, fetchImpl, fetchInit, signal) {
 		await writable.write(buf);
 		await writable.close();
 	} catch (err) {
-		await writable.abort?.().catch(() => {});
-		await root.removeEntry(name).catch(() => {});
+		if (writable.abort) await attemptCleanup(`aborting partial OPFS write ${name}`, () => writable.abort());
+		await attemptCleanup(`removing partial OPFS file ${name}`, () => root.removeEntry(name));
 		throw err;
 	}
 	return {
@@ -886,7 +915,9 @@ async function attachOpfsParquetTables(options) {
 	await timed(onTiming, "sweep", {
 		files: total,
 		tables: tables.length
-	}, () => sweepStaleEntries(root, registry, expectedByTable)).catch(() => {});
+	}, () => sweepStaleEntries(root, registry, expectedByTable)).catch((error) => {
+		reportBestEffortFailure("stale OPFS cache sweep", error);
+	});
 	const tableFiles = /* @__PURE__ */ new Map();
 	const degraded = /* @__PURE__ */ new Set();
 	const degradeReason = /* @__PURE__ */ new Map();
@@ -981,7 +1012,7 @@ async function attachOpfsParquetTables(options) {
 			tables: tables.length
 		}, runDownloads);
 	} catch (err) {
-		await withDb(() => registry.release([...acquiredNames])).catch(() => {});
+		await attemptCleanup("releasing handles after download failure", () => withDb(() => registry.release([...acquiredNames])));
 		throw err;
 	}
 	const attached = [];
@@ -1009,17 +1040,17 @@ async function attachOpfsParquetTables(options) {
 			}, () => withDb(() => ensureView(registry, conn, schema, t.table, signature, overlayName ? readParquetViewWithOverlaySql(schema, t.table, lakeNames, overlayName) : readParquetViewSql$1(schema, t.table, lakeNames))));
 		} catch (err) {
 			if (isAbortError$1(err)) {
-				await withDb(() => detachOpfs(registry, conn, schema, attached, [...acquiredNames])).catch(() => {});
+				await attemptCleanup("detaching OPFS resources after abort", () => withDb(() => detachOpfs(registry, conn, schema, attached, [...acquiredNames])));
 				throw err;
 			}
 			if (isOpfsAccessHandleConflict(err)) {
-				if (registry.viewRefs(viewKey(schema, t.table)) === 0) await withDb(() => conn.query(`DROP VIEW IF EXISTS ${schema}.${t.table}`)).catch(() => {});
+				if (registry.viewRefs(viewKey(schema, t.table)) === 0) await attemptCleanup(`dropping incomplete view ${schema}.${t.table}`, () => withDb(() => conn.query(`DROP VIEW IF EXISTS ${schema}.${t.table}`)));
 				degraded.add(t.table);
 				degradeReason.set(t.table, "contention");
 				await releaseTable(t.table);
 				continue;
 			}
-			await withDb(() => detachOpfs(registry, conn, schema, attached, [...acquiredNames])).catch(() => {});
+			await attemptCleanup("detaching OPFS resources after view failure", () => withDb(() => detachOpfs(registry, conn, schema, attached, [...acquiredNames])));
 			throw err;
 		}
 		attached.push(t.table);
@@ -1076,7 +1107,7 @@ async function attachOpfsParquetTables(options) {
 		const before = bufferFiles.length;
 		if (await recoverTableToBuffers(t).catch(() => false)) degraded.delete(t.table);
 		else {
-			await withDb(() => bufferRegistry.release(bufferFiles.slice(before))).catch(() => {});
+			await attemptCleanup(`releasing recovery buffers for ${t.table}`, () => withDb(() => bufferRegistry.release(bufferFiles.slice(before))));
 			bufferFiles.length = before;
 		}
 	}
@@ -1109,14 +1140,21 @@ async function detachOpfs(registry, conn, schema, tables, files) {
 	await registry.release(files);
 }
 async function clearOpfsSnapshotCache() {
-	const root = await getOpfsRoot().catch(() => null);
+	const root = await getOpfsRoot().catch((error) => {
+		reportBestEffortFailure("opening OPFS cache for clearing", error);
+		return null;
+	});
 	if (!root) return;
 	const removable = [];
 	const dir = root;
 	if (dir.keys) {
 		for await (const name of dir.keys()) if (name.startsWith(OPFS_PREFIX)) removable.push(name);
 	}
-	for (const name of removable) await root.removeEntry(name).catch(() => {});
+	for (const name of removable) try {
+		await root.removeEntry(name);
+	} catch (error) {
+		if (!isNotFoundError(error)) reportBestEffortFailure(`clearing OPFS cache entry ${name}`, error);
+	}
 }
 async function createInsightRunner(opts) {
 	const client = await createClient(opts.db, opts.conn);
@@ -1476,14 +1514,16 @@ function createBrowserAnalysisRuntime(boot, options = {}) {
 	}
 	function runExclusive(signal, work) {
 		const next = chain.then(work, work);
-		chain = next.catch(() => {});
+		chain = next.then(() => void 0, () => void 0);
 		return raceSignal(next, signal);
 	}
 	async function cancelOnAbort(signal, work) {
 		if (!signal) return work;
 		if (signal.aborted) throw abortError(signal);
 		const onAbort = () => {
-			conn.cancelSent().catch(() => {});
+			conn.cancelSent().catch((error) => {
+				console.warn("[gscdump/engine-duckdb-wasm] failed to cancel aborted query", error);
+			});
 		};
 		signal.addEventListener("abort", onAbort, { once: true });
 		try {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@gscdump/engine-duckdb-wasm",
   "type": "module",
-  "version": "0.37.3",
+  "version": "0.37.4",
   "description": "DuckDB-WASM engine adapter for @gscdump/analysis — typed browser analytics against parquet via R2.",
   "author": {
     "name": "Harlan Wilton",
@@ -45,9 +45,9 @@
   },
   "dependencies": {
     "drizzle-orm": "1.0.0-rc.3",
-    "@gscdump/contracts": "0.37.3",
-    "@gscdump/engine": "0.37.3",
-    "gscdump": "0.37.3"
+    "@gscdump/engine": "0.37.4",
+    "gscdump": "0.37.4",
+    "@gscdump/contracts": "0.37.4"
   },
   "devDependencies": {
     "@duckdb/duckdb-wasm": "^1.32.0",