@takosjp/yurucommu-core 4.1.5 → 4.1.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@takosjp/yurucommu-core",
-  "version": "4.1.5",
+  "version": "4.1.7",
   "license": "AGPL-3.0-only",
   "type": "module",
   "workspaces": [

package/packages/api/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@takosjp/yurucommu-api",
-  "version": "4.1.5",
+  "version": "4.1.7",
   "description": "Typed client SDK and public API contract for yurucommu-server clients.",
   "license": "AGPL-3.0-only",
   "type": "module",

package/src/backend/middleware/cache.ts

@@ -319,12 +319,15 @@ async function handleCloudflareCache(
   cacheKey: string,
   config: CacheConfig,
 ): Promise<Response | void> {
-  const cache = caches.default;
   const url = new URL(c.req.url);
   const fullCacheKey = new Request(`${url.origin}/_cache${cacheKey}`);
 
+  let cache: Cache;
   let cachedResponse: Response | undefined;
   try {
+    // Some runtimes expose CacheStorage but have no backing cache configured;
+    // accessing the default getter can fail before match() is reached.
+    cache = caches.default;
     cachedResponse = await cache.match(fullCacheKey);
   } catch (error) {
     if (isDefaultCacheUnavailable(error)) {
@@ -395,7 +398,7 @@ function isDefaultCacheUnavailable(error: unknown): boolean {
       : typeof error === "string"
         ? error
         : "";
-  return /not permitted to access the default cache|default cache.*(?:not|un)available|Cache API.*not available/i.test(
+  return /not permitted to access the default cache|default cache.*(?:not|un)available|Cache API.*not available|^No Cache was configured\.?$/i.test(
     message,
   );
 }

package/src/backend/runtime/edge-facades.ts

@@ -83,7 +83,7 @@ export interface EdgeSqlBinding {
   /** All-or-none. 1..100 statements, ordered, one Host round trip. */
   transaction(
     statements: readonly EdgeSqlStatement[],
-  ): Promise<readonly EdgeSqlResult[]>;
+  ): Promise<{ readonly results: readonly EdgeSqlResult[] }>;
 }
 
 export interface EdgeKvPutOptions {

package/src/backend/runtime/edge-sql.ts

@@ -158,6 +158,11 @@ export function createEdgeSqlDatabase(binding: EdgeSqlBinding) {
     await one(prepare(sql, params), method);
 
   const batchCallback: AsyncBatchRemoteCallback = async (batch) => {
+    if (batch.length < 1) {
+      throw new EdgeSqlShapeError(
+        "edge.sql: a transaction requires at least one statement",
+      );
+    }
     if (batch.length > EDGE_SQL_MAX_STATEMENTS) {
       throw new EdgeSqlShapeError(
         `edge.sql: a batch of ${batch.length} statements exceeds the facade ` +
@@ -165,9 +170,26 @@ export function createEdgeSqlDatabase(binding: EdgeSqlBinding) {
       );
     }
     const prepared = batch.map((entry) => prepare(entry.sql, entry.params));
-    const results = await binding.transaction(
+    const transaction = await binding.transaction(
       prepared.map((entry) => ({ sql: entry.sql, params: entry.params })),
     );
+    const keys =
+      typeof transaction === "object" &&
+      transaction !== null &&
+      !Array.isArray(transaction)
+        ? Object.keys(transaction)
+        : [];
+    if (
+      keys.length !== 1 ||
+      keys[0] !== "results" ||
+      !Array.isArray(transaction?.results)
+    ) {
+      throw new ProxyColumnMismatchError(
+        `edge.sql: transaction returned a malformed result envelope for ` +
+          `${prepared.length} statements`,
+      );
+    }
+    const { results } = transaction;
     if (results.length !== prepared.length) {
       throw new ProxyColumnMismatchError(
         `edge.sql: transaction returned ${results.length} results for ` +