@bobfrankston/rmfmail 1.0.703 → 1.0.705

package/bin/build-bundles.mjs

@@ -23,16 +23,66 @@ import { fileURLToPath } from "node:url";
 const root = path.resolve(fileURLToPath(import.meta.url), "..", "..");
 const watch = process.argv.includes("--watch");
 
+// Each entry has its own external list because the Android bootstrap bundle
+// needs to INCLUDE @bobfrankston/mailx-store-web (entire purpose of the
+// bundle), while app.bundle.js externalizes it so the desktop build doesn't
+// pull a 1-MB blob into a path that only Android takes.
 const entries = [
     {
         in:  path.join(root, "client", "app.ts"),
         out: path.join(root, "client", "app.bundle.js"),
         label: "app",
+        external: [
+            "@bobfrankston/rmf-tiny",
+            "@bobfrankston/mailx-store-web/*",
+            "tinymce",
+            "tinymce/*",
+            "quill",
+            "quill/*",
+            "jsonc-parser",
+            "mammoth",
+            "html-to-docx",
+        ],
     },
     {
         in:  path.join(root, "client", "compose", "compose.ts"),
         out: path.join(root, "client", "compose", "compose.bundle.js"),
         label: "compose",
+        external: [
+            "@bobfrankston/rmf-tiny",
+            "@bobfrankston/mailx-store-web/*",
+            "tinymce",
+            "tinymce/*",
+            "quill",
+            "quill/*",
+            "jsonc-parser",
+            "mammoth",
+            "html-to-docx",
+        ],
+    },
+    {
+        // 2026-05-13: dedicated Android bootstrap bundle. The previous
+        // arrangement leaned on a runtime import map to resolve
+        // `@bobfrankston/mailx-store-web/android-bootstrap.js` →
+        // `file:///android_asset/node_modules/@bobfrankston/...` at WebView
+        // load time. That URL never resolved on real APKs ("Failed to
+        // fetch dynamically imported module" 100% reproducible across
+        // v695 → v702), apparently because MAUI's MauiAsset machinery
+        // mangles deep `node_modules/@bobfrankston/...` paths in ways the
+        // WebView can't undo. Bundling inlines every transitive dep
+        // (mailx-types, sql.js, etc.) and ships ONE self-contained file at
+        // `file:///android_asset/client/android-bootstrap.bundle.js` —
+        // same asset tree as `client/index.html`, the path we already
+        // know works.
+        in:  path.join(root, "packages", "mailx-store-web", "android-bootstrap.ts"),
+        out: path.join(root, "client", "android-bootstrap.bundle.js"),
+        label: "android-bootstrap",
+        external: [
+            // sql.js loads its own wasm at runtime via `locateFile`, and the
+            // wasm binary is shipped as its own MauiAsset — don't try to
+            // inline a binary into JS.
+            "sql.js",
+        ],
     },
 ];
 
@@ -42,29 +92,7 @@ const shared = {
     target:    "es2022",
     platform:  "browser",
     sourcemap: true,
-    // Keep dynamic imports for code paths that should still load on demand:
-    // - @bobfrankston/rmf-tiny is loaded only when the user picks TinyMCE.
-    // - @bobfrankston/mailx-store-web/android-bootstrap is only on Android.
-    // - Quill / TinyMCE themselves are loaded by their adapters at runtime.
-    // - jsonc-parser is large and only used by the JSONC config editor.
-    external: [
-        "@bobfrankston/rmf-tiny",
-        "@bobfrankston/mailx-store-web/*",
-        "tinymce",
-        "tinymce/*",
-        "quill",
-        "quill/*",
-        "jsonc-parser",
-        "mammoth",
-        "html-to-docx",
-    ],
-    // The output has to be ESM-compatible — index.html loads it via
-    // <script type="module"> through the boot router's `await import()`.
-    // Splitting would create extra chunks served as separate fetches,
-    // defeating the purpose; the whole point is one fetch per entry.
     splitting: false,
-    // Suppress the "minify saved N bytes" chatter; we want the source
-    // readable in DevTools (sourcemap doubles up).
     minify: false,
     logLevel: "info",
 };
@@ -72,7 +100,7 @@ const shared = {
 async function buildAll() {
     const t0 = Date.now();
     const results = await Promise.all(entries.map(e =>
-        esbuild.build({ ...shared, entryPoints: [e.in], outfile: e.out })
+        esbuild.build({ ...shared, external: e.external, entryPoints: [e.in], outfile: e.out })
             .then(r => ({ ok: true, label: e.label, out: e.out }))
             .catch(err => ({ ok: false, label: e.label, err }))
     ));
@@ -87,7 +115,7 @@ async function buildAll() {
 
 if (watch) {
     const ctxs = await Promise.all(entries.map(e =>
-        esbuild.context({ ...shared, entryPoints: [e.in], outfile: e.out })
+        esbuild.context({ ...shared, external: e.external, entryPoints: [e.in], outfile: e.out })
     ));
     for (const c of ctxs) await c.watch();
     console.log("build-bundles: watch mode");