@sproutboat/wire 0.8.0 → 0.9.0

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # @sproutboat/wire
 
+## 0.9.0
+
+### Minor Changes
+
+- 4a7fe62: Cache API: `caches.default` / `caches.open(name)` (baronunread/sproutboat#59), with `match()`/`put()`/`delete()` backed by both transports (broker's `cache.*` ops and the standalone embedded transport, so behavior matches whichever way the app deploys). Keyed on method+URL for v1 (no `Vary` support yet). `put()` throws for a non-GET request, matching the spec; a 206 status or an explicit `no-store`/`private` `Cache-Control` is a silent no-op rather than an error, matching Workers' own `caches.default`. TTL derives from `s-maxage` (preferred) or `max-age`; entries with neither have no expiry.
+  
+  Deliberately synchronous, not Promise-returning: every other buffered-body method this runtime already ships (`text()`/`json()`/`arrayBuffer()`/`blob()`/`formData()`) is sync too, and this runtime's own coroutine machinery has a known, still-open memory-corruption issue tied specifically to nested `async function` calls accumulating across many requests (baronunread/sproutboat#168) — no reason to add more async surface here. `await caches.default.match(...)` still reads correctly: awaiting a non-promise value resolves to it immediately.
+
+### Patch Changes
+
+- Updated dependencies [6145afe]
+  - @sproutboat/assets@0.3.0
+
 ## 0.8.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sproutboat/wire",
-  "version": "0.8.0",
+  "version": "0.9.0",
   "license": "MIT",
   "type": "module",
   "repository": {
@@ -11,7 +11,7 @@
   "main": "./src/index.ts",
   "types": "./src/index.ts",
   "dependencies": {
-    "@sproutboat/assets": "^0.2.0"
+    "@sproutboat/assets": "^0.3.0"
   },
   "exports": {
     ".": "./src/index.ts",

package/src/broker.test.ts

@@ -75,6 +75,62 @@ test("an unbound KV namespace is rejected", async () => {
   await expect(b.dispatch({ op: "kv.put", ns: "OTHER", key: "k", value: "v" })).rejects.toThrow("not bound");
 });
 
+test("cache.match on an empty cache is a plain miss", async () => {
+  const b = make();
+  expect(await b.dispatch({ op: "cache.match", key: "GET https://example.com/a" })).toEqual({
+    ok: true,
+    found: false,
+  });
+});
+
+test("cache: put then match round-trips status/headers/body, deletes work, ambient (no binding needed)", async () => {
+  const b = make();
+  const key = "GET https://example.com/a";
+  expect(
+    await b.dispatch({ op: "cache.put", key, status: 200, headers: { "content-type": "text/plain" }, body: "hi" }),
+  ).toEqual({ ok: true, stored: true });
+  expect(await b.dispatch({ op: "cache.match", key })).toEqual({
+    ok: true,
+    found: true,
+    status: 200,
+    headers: { "content-type": "text/plain" },
+    body: "hi",
+  });
+  expect(await b.dispatch({ op: "cache.delete", key })).toEqual({ ok: true, deleted: true });
+  expect(await b.dispatch({ op: "cache.match", key })).toEqual({ ok: true, found: false });
+  expect(await b.dispatch({ op: "cache.delete", key })).toEqual({ ok: true, deleted: false });
+});
+
+test("cache.put skips storage (silently) for a 206, no-store, or private response", async () => {
+  const b = make();
+  await b.dispatch({ op: "cache.put", key: "k1", status: 206, body: "partial" });
+  await b.dispatch({ op: "cache.put", key: "k2", status: 200, headers: { "cache-control": "no-store" }, body: "x" });
+  await b.dispatch({ op: "cache.put", key: "k3", status: 200, headers: { "cache-control": "private" }, body: "x" });
+  expect(await b.dispatch({ op: "cache.match", key: "k1" })).toEqual({ ok: true, found: false });
+  expect(await b.dispatch({ op: "cache.match", key: "k2" })).toEqual({ ok: true, found: false });
+  expect(await b.dispatch({ op: "cache.match", key: "k3" })).toEqual({ ok: true, found: false });
+});
+
+test("cache.put derives TTL from s-maxage over max-age; an expired entry reads as a miss", async () => {
+  const b = make();
+  await b.dispatch({
+    op: "cache.put",
+    key: "k",
+    status: 200,
+    headers: { "cache-control": "max-age=3600, s-maxage=0" },
+    body: "x",
+  });
+  // s-maxage=0 wins over max-age=3600 -> expires immediately.
+  expect(await b.dispatch({ op: "cache.match", key: "k" })).toEqual({ ok: true, found: false });
+});
+
+test("cache namespaces (caches.open) don't collide on the same key", async () => {
+  const b = make();
+  await b.dispatch({ op: "cache.put", name: "images", key: "k", status: 200, body: "img" });
+  expect(await b.dispatch({ op: "cache.match", name: "images", key: "k" })).toMatchObject({ found: true, body: "img" });
+  expect(await b.dispatch({ op: "cache.match", key: "k" })).toEqual({ ok: true, found: false });
+});
+
 test("secrets resolve only when both bound and present", async () => {
   const b = make({ bindings: { secrets: ["API_KEY", "MISSING"] }, secrets: { API_KEY: "s3cr3t" } });
   expect(await b.dispatch({ op: "secret.get", name: "API_KEY" })).toEqual({ ok: true, value: "s3cr3t" });

package/src/broker.ts

@@ -332,6 +332,14 @@ export function createBroker(opts: BrokerOptions = {}): Broker {
     "CREATE TABLE IF NOT EXISTS ratelimit (name TEXT NOT NULL, key TEXT NOT NULL, window_start INTEGER NOT NULL, " +
       "count INTEGER NOT NULL, PRIMARY KEY (name, key))",
   );
+  // #59 — caches.default: ambient, not a resource a project binds (no config
+  // key), so it lives in the per-deployment db like do_storage/ae rather than
+  // going through the account-level resource registry. Key on method+URL for
+  // v1, per the issue's own scoped-down ask; no Vary support yet.
+  db.exec(
+    "CREATE TABLE IF NOT EXISTS cache (name TEXT NOT NULL, cache_key TEXT NOT NULL, status INTEGER NOT NULL, " +
+      "headers_json TEXT NOT NULL, body TEXT NOT NULL, expires_at INTEGER, PRIMARY KEY (name, cache_key))",
+  );
 
   // #74 — one SQLite file per account-level resource id, opened on first use.
   const resourceDbs = new Map<string, Database>();
@@ -1240,6 +1248,50 @@ export function createBroker(opts: BrokerOptions = {}): Broker {
             .map((r) => r.key),
         };
       }
+      case "cache.match": {
+        const name = str(msg.name) || "default";
+        const key = str(msg.key);
+        const row = db
+          .query<
+            { status: number; headers_json: string; body: string; expires_at: number | null },
+            [string, string]
+          >("SELECT status, headers_json, body, expires_at FROM cache WHERE name = ? AND cache_key = ?")
+          .get(name, key);
+        if (!row) return { ok: true, found: false };
+        if (row.expires_at != null && row.expires_at <= Date.now()) {
+          db.query("DELETE FROM cache WHERE name = ? AND cache_key = ?").run(name, key);
+          return { ok: true, found: false };
+        }
+        return { ok: true, found: true, status: row.status, headers: jsonObject(parseJsonValue(row.headers_json)) ?? {}, body: row.body };
+      }
+      case "cache.put": {
+        const name = str(msg.name) || "default";
+        const key = str(msg.key);
+        const status = Number(msg.status) || 200;
+        const headers = jsonObject(msg.headers ?? null) ?? {};
+        const cacheControl = str(headers["cache-control"] ?? headers["Cache-Control"] ?? "").toLowerCase();
+        // Mirrors Workers' own `caches.default.put()`: a 206 or an explicit
+        // no-store/private is a silent no-op, not an error — the handler's
+        // response still goes to the client either way.
+        if (status === 206 || cacheControl.includes("no-store") || cacheControl.includes("private")) {
+          return { ok: true, stored: false };
+        }
+        const sMaxAge = /s-maxage=(\d+)/.exec(cacheControl);
+        const maxAge = /max-age=(\d+)/.exec(cacheControl);
+        const ttlSeconds = sMaxAge ? Number(sMaxAge[1]) : maxAge ? Number(maxAge[1]) : null;
+        const expiresAt = ttlSeconds != null ? Date.now() + ttlSeconds * 1000 : null;
+        db.query(
+          "INSERT INTO cache (name, cache_key, status, headers_json, body, expires_at) VALUES (?1, ?2, ?3, ?4, ?5, ?6) " +
+            "ON CONFLICT (name, cache_key) DO UPDATE SET status = ?3, headers_json = ?4, body = ?5, expires_at = ?6",
+        ).run(name, key, status, JSON.stringify(headers), str(msg.body), expiresAt);
+        return { ok: true, stored: true };
+      }
+      case "cache.delete": {
+        const name = str(msg.name) || "default";
+        const key = str(msg.key);
+        const r = db.query("DELETE FROM cache WHERE name = ? AND cache_key = ?").run(name, key);
+        return { ok: true, deleted: r.changes > 0 };
+      }
       case "secret.get": {
         const name = str(msg.name);
         if (!bindings.secrets.includes(name)) throw new Error(`secret not bound: ${name}`);
@@ -1704,6 +1756,8 @@ export function createBroker(opts: BrokerOptions = {}): Broker {
   const REPLAYABLE = new Set([
     "kv.put",
     "kv.delete",
+    "cache.put",
+    "cache.delete",
     "queue.send",
     "queue.send_batch",
     "d1.query",