@uniflowed/vite 0.0.0-alpha.17 → 0.0.0-alpha.18

package/driver.js

@@ -980,6 +980,26 @@ const ADAPTERS = {
       server: nodeEntrySource("./handler.js"),
     }),
   },
+  // The same two files as `node`, with `@uniflowed/server/bun` in place of
+  // `@uniflowed/server/node`. That module is `./internal/static.js` for every
+  // decision and `Bun.file` for the bytes, which is where the measured win is
+  // — see ubugeeei-prod/uf#391 and the header of `packages/server/bun.js`.
+  //
+  // No `conditions` of its own, unlike `edge`. Bun honours a `bun` export
+  // condition and this deliberately does not ask for it: the measured win is
+  // `Bun.serve` and `Bun.file` and not resolution, so asking would trade a
+  // `handler.js` that is byte-for-byte `node`'s — which is what
+  // `every_adapter_answers_exactly_what_the_node_adapter_answers` checks, and
+  // what makes an adapter one file rather than one application — for whichever
+  // build a dependency happens to ship behind that condition, tested by
+  // nobody here. `edge` pays that price because it must: there is no
+  // `node:stream` in a Worker.
+  bun: {
+    entries: (document, cache) => ({
+      handler: handlerEntrySource(document, cache, BUN_CAPABILITIES),
+      server: bunEntrySource("./handler.js"),
+    }),
+  },
   // The same two files. What `--adapter container` adds is a `Dockerfile` and
   // a `.dockerignore`, and both are plain text that `uf` writes beside this
   // output rather than anything the bundler produces — see `uf_cli`'s
@@ -1027,6 +1047,7 @@ const ADAPTERS = {
  * dropped.
  */
 const NODE_CAPABILITIES = { module: "@uniflowed/server/node", name: "nodeCapabilities" };
+const BUN_CAPABILITIES = { module: "@uniflowed/server/bun", name: "bunCapabilities" };
 const EDGE_CAPABILITIES = { module: "@uniflowed/server/edge", name: "edgeCapabilities" };
 const SERVERLESS_CAPABILITIES = { module: "@uniflowed/server/lambda", name: "lambdaCapabilities" };
 
@@ -1296,6 +1317,41 @@ serve({ handle: fetch, staticDir, beginRequest }).catch((error) => {
 `;
 }
 
+/**
+ * The source of `server.js`: the Bun socket around that handler.
+ *
+ * `nodeEntrySource`'s twin, and identical but for the module it imports
+ * `serve` from — `@uniflowed/server/bun` exports the same signature on
+ * purpose, so that the two adapters are one contract and a project moving
+ * between them changes a flag and nothing else.
+ */
+function bunEntrySource(handlerSpecifier) {
+  return `// Generated by \`uf build --adapter bun\`. Not checked in, not edited.
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+
+import { serve } from "@uniflowed/server/bun";
+
+// \`beginRequest\` comes from the handler beside this file rather than from
+// \`@uniflowed/server/bun\` above, because the request has to be established in
+// the storage the *application* reads, which is the copy bundled into
+// \`handler.js\`. See ubugeeei-prod/uf#389.
+import { beginRequest, fetch } from ${JSON.stringify(handlerSpecifier)};
+
+// Resolved from this file and not from the working directory: a process
+// manager, a container entrypoint and a person in a shell each start a server
+// from wherever they happen to be, and a directory that only served its own
+// assets when it was started from inside itself would be a deployment with a
+// trap in it.
+const staticDir = path.join(path.dirname(fileURLToPath(import.meta.url)), "static");
+
+serve({ handle: fetch, staticDir, beginRequest }).catch((error) => {
+  process.stderr.write(\`uf: \${error?.message ?? String(error)}\\n\`);
+  process.exit(1);
+});
+`;
+}
+
 /**
  * The source of `worker.js`: the Cloudflare Workers entry around that handler.
  *

package/index.js

@@ -696,15 +696,25 @@ function flowPlugin({ routerRoot, appEntry, strictMode, command, accessibility }
                   // result by then — so the terminal hears about it here or not
                   // at all.
                   onError: (error) => reportRenderError(devServer, url, error),
+                  // Vite sees the head and only the head. That is what lets the
+                  // development server stream like every other host — see below.
+                  transformHead: (head) => devServer.transformIndexHtml(url, head),
                 },
               );
               if (result.error != null) reportRenderError(devServer, url, result.error);
-              // Collected rather than piped: `transformIndexHtml` is a
-              // whole-document hook, so there is no first byte to send until it
-              // has run. `uf start` and `uf preview` stream — see
-              // `internal/serve.js` — and that is a property of the development
-              // server rather than of the renderer. ubugeeei-prod/uf#374.
-              const html = await devServer.transformIndexHtml(url, await result.text());
+              // Piped, like every other host. This used to collect the whole
+              // document and transform it at the end, because
+              // `transformIndexHtml` is a *whole document* hook — which made the
+              // one place a developer would notice streaming the one place it
+              // did not happen: a slow page showed nothing until it was finished
+              // and `_uf.loading.js` looked broken.
+              //
+              // `transformHead` above is the seam. `internal/stream.js` already
+              // held the opening chunk back until the head was complete and
+              // forwarded everything after it untouched, so the hook only ever
+              // sees the head — and Vite's dev hook handles a document that ends
+              // mid-`<body>` without complaint, which was the open question on
+              // ubugeeei-prod/uf#374.
               response.statusCode = result.status ?? 200;
               // The render's own headers, then the content type over the top:
               // exactly the order `@uniflowed/server`'s `fetch.js` writes them
@@ -714,7 +724,7 @@ function flowPlugin({ routerRoot, appEntry, strictMode, command, accessibility }
                 response.setHeader(name, value);
               }
               response.setHeader("content-type", "text/html; charset=utf-8");
-              response.end(html);
+              await result.pipe(response);
               return true;
             });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uniflowed/vite",
-  "version": "0.0.0-alpha.17",
+  "version": "0.0.0-alpha.18",
   "description": "Vite, driven by uf.config.js: every Flow module through `uf transform`, MDX, the file-system router and static rendering as Vite plugins.",
   "type": "module",
   "license": "MIT",
@@ -33,8 +33,8 @@
   "dependencies": {
     "@mdx-js/rollup": "^3.1.1",
     "@shikijs/rehype": "^3.23.0",
-    "@uniflowed/host": "0.0.0-alpha.17",
-    "@uniflowed/server": "0.0.0-alpha.17",
+    "@uniflowed/host": "0.0.0-alpha.18",
+    "@uniflowed/server": "0.0.0-alpha.18",
     "rehype-slug": "^6.0.0",
     "remark-frontmatter": "^5.0.0",
     "remark-gfm": "^4.0.1",