@yysng/astro-boilerplate 1.1.35 → 1.1.36

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yysng/astro-boilerplate",
-  "version": "1.1.35",
+  "version": "1.1.36",
   "description": "Astro + Sanity Boilerplate with AEO Layers 1–5",
   "type": "module",
   "exports": {

package/src/content-system/loader.js

@@ -1,17 +1,28 @@
 import { CONTENT_REGISTRY } from "./registry.js";
 
+const isEdge = typeof process === "undefined";
+
 export async function loadContent(key, env = {}) {
   const entry = CONTENT_REGISTRY[key];
   if (!entry) throw new Error(`Unknown content key: ${key}`);
 
-  // Cloudflare runtime
+  // Cloudflare runtime (KV)
   if (env?.CONTENT_KV) {
     const stored = await env.CONTENT_KV.get(entry.file, "json");
     if (stored) return stored;
     throw new Error(`KV missing content for key "${key}"`);
   }
 
-  // Local dev — dynamic import prevents fs from entering Worker bundle
-  const { readLocal } = await import("./storage.node.js");
+  // 🚨 Edge runtime must NEVER attempt local filesystem
+  if (isEdge) {
+    throw new Error(
+      `Edge runtime missing CONTENT_KV binding for "${key}". ` +
+      `Make sure you pass Astro.locals.runtime.env into loadContent() and your Pages project has CONTENT_KV bound.`
+    );
+  }
+
+  // Local dev only — prevent bundling into Worker by using @vite-ignore + variable specifier
+  const modPath = "./storage.node.js";
+  const { readLocal } = await import(/* @vite-ignore */ modPath);
   return await readLocal(entry.file);
 }

package/src/content-system/updater.js

@@ -4,13 +4,21 @@ import { CONTENT_REGISTRY } from "./registry.js";
 
 const isEdge = typeof process === "undefined";
 
-const { writeLocal } = isEdge
-  ? await import("@yysng/astro-boilerplate/content-storage/edge")
-  : await import("@yysng/astro-boilerplate/content-storage/node");
+// IMPORTANT:
+// - On edge, never import node storage.
+// - Use @vite-ignore + variable specifier to avoid bundling node fs into worker.
+async function getStorage() {
+  if (isEdge) {
+    const p = "@yysng/astro-boilerplate/content-storage/edge";
+    return await import(/* @vite-ignore */ p);
+  } else {
+    const p = "@yysng/astro-boilerplate/content-storage/node";
+    return await import(/* @vite-ignore */ p);
+  }
+}
 
 function mergeDefined(existing, incoming) {
   const result = { ...existing };
-
   for (const k in incoming) {
     const v = incoming[k];
     if (v === undefined) continue;
@@ -21,7 +29,6 @@ function mergeDefined(existing, incoming) {
       result[k] = v;
     }
   }
-
   return result;
 }
 
@@ -35,13 +42,21 @@ export async function updateContent(key, incoming, env = {}) {
 
   schema.validate(merged);
 
-  // Cloudflare
+  // Cloudflare KV
   if (env?.CONTENT_KV) {
     await env.CONTENT_KV.put(entry.file, JSON.stringify(merged));
     return merged;
   }
 
-  // Local dev
+  // Edge must never write local
+  if (isEdge) {
+    throw new Error(
+      `Edge runtime missing CONTENT_KV binding for updateContent("${key}").`
+    );
+  }
+
+  // Local dev filesystem write
+  const { writeLocal } = await getStorage();
   await writeLocal(entry.file, merged);
   return merged;
 }