@akanjs/devkit 3.0.0-alpha.95 → 3.0.0-alpha.96

package/applicationBuildRunner.test.ts

@@ -1,5 +1,6 @@
 import { describe, expect, test } from "bun:test";
-import { AKAN_OPTIONAL_BACKEND_EXTERNALS } from "./applicationBuildRunner";
+import { rm } from "node:fs/promises";
+import { AKAN_BACKEND_MINIFY, AKAN_OPTIONAL_BACKEND_EXTERNALS } from "./applicationBuildRunner";
 
 describe("ApplicationBuildRunner", () => {
   test("externalizes Akan optional backend dependencies", () => {
@@ -7,4 +8,38 @@ describe("ApplicationBuildRunner", () => {
       expect.arrayContaining(["@libsql/client", "bullmq", "ioredis", "postgres", "protobufjs"]),
     );
   });
+
+  test("keeps backend identifiers so a class names its own logger and stack frames", () => {
+    expect(AKAN_BACKEND_MINIFY.identifiers).toBe(false);
+    expect(AKAN_BACKEND_MINIFY.whitespace).toBe(true);
+    expect(AKAN_BACKEND_MINIFY.syntax).toBe(true);
+  });
+
+  test("a bundler that mangles identifiers loses the class name a Logger reads", async () => {
+    const dir = `${process.env.TMPDIR ?? "/tmp"}/akan-minify-${Bun.randomUUIDv7()}`;
+    const entry = `${dir}/entry.ts`;
+    await Bun.write(
+      entry,
+      `const serve = () => class Service { readonly name = this.constructor.name; };
+export class SampleService extends serve() {}
+console.info(new SampleService().name);
+`,
+    );
+    const build = async (minify: Bun.BuildConfig["minify"]) => {
+      const result = await Bun.build({ entrypoints: [entry], target: "bun", minify, outdir: `${dir}/out` });
+      expect(result.success).toBe(true);
+      const proc = Bun.spawn([process.execPath, result.outputs[0].path], { stdout: "pipe" });
+      return (await new Response(proc.stdout).text()).trim();
+    };
+    try {
+      expect(await build(AKAN_BACKEND_MINIFY)).toBe("SampleService");
+      expect(await build(true)).not.toBe("SampleService");
+      // Bun 1.4.2 accepts `keepNames` and ignores it; drop this expectation once it keeps the name.
+      expect(await build({ whitespace: true, syntax: true, identifiers: true, keepNames: true })).not.toBe(
+        "SampleService",
+      );
+    } finally {
+      await rm(dir, { recursive: true, force: true });
+    }
+  });
 });

package/applicationBuildRunner.ts

@@ -60,6 +60,11 @@ const SSR_RENDER_EXTERNALS = [
   "react-server-dom-webpack/client.browser",
 ] as const;
 
+// Identifier mangling renames every class, and `this.constructor.name` is what names a service's logger, an
+// `Exception`, a guard, and every frame of a stack trace — for ~2% of boot on server bytes nothing downloads.
+// `minify.keepNames` typechecks and does nothing as of Bun 1.4.2.
+export const AKAN_BACKEND_MINIFY = { whitespace: true, syntax: true, identifiers: false } as const;
+
 export const AKAN_OPTIONAL_BACKEND_EXTERNALS = [
   "@libsql/client",
   "bullmq",
@@ -68,16 +73,6 @@ export const AKAN_OPTIONAL_BACKEND_EXTERNALS = [
   "protobufjs",
 ] as const;
 
-const MINIFY_PROBE = ((): { minify: Bun.BuildConfig["minify"]; sourcemap?: Bun.BuildConfig["sourcemap"] } => {
-  const mode = process.env.AKAN_MINIFY_PROBE ?? "all";
-  if (mode === "noident") return { minify: { whitespace: true, syntax: true, identifiers: false } };
-  if (mode === "noident-map")
-    return { minify: { whitespace: true, syntax: true, identifiers: false }, sourcemap: "linked" };
-  if (mode === "syntax-map") return { minify: { whitespace: false, syntax: true, identifiers: false }, sourcemap: "linked" };
-  if (mode === "none") return { minify: false, sourcemap: "linked" };
-  return { minify: true };
-})();
-
 export class ApplicationBuildRunner {
   #app: App;
   #fast: boolean;
@@ -213,7 +208,7 @@ export class ApplicationBuildRunner {
       entrypoints: backendEntryPoints,
       outdir: this.#app.dist.cwdPath,
       target: "bun",
-      ...MINIFY_PROBE,
+      minify: AKAN_BACKEND_MINIFY,
       naming: { entry: "[name].[ext]", chunk: "chunk-[hash].[ext]" },
       define: { "process.env.NODE_ENV": JSON.stringify("production") },
       plugins: backendExternals.length > 0 ? [this.#createExternalSpecifiersPlugin(backendExternals)] : [],
@@ -224,7 +219,7 @@ export class ApplicationBuildRunner {
           entrypoints: [this.#resolveRscWorkerBuildEntry()],
           outdir: this.#app.dist.cwdPath,
           target: "bun",
-          ...MINIFY_PROBE,
+          minify: AKAN_BACKEND_MINIFY,
           naming: { entry: "[name].[ext]", chunk: "chunk-[hash].[ext]" },
           conditions: ["react-server"],
           // `akan build` must embed production react-server-dom regardless of the shell's NODE_ENV.
@@ -236,7 +231,7 @@ export class ApplicationBuildRunner {
       entrypoints: [this.#resolveConsoleRuntimeBuildEntry()],
       outdir: this.#app.dist.cwdPath,
       target: "bun",
-      ...MINIFY_PROBE,
+      minify: AKAN_BACKEND_MINIFY,
       naming: { entry: "console-runtime.[ext]", chunk: "chunk-[hash].[ext]" },
       define: { "process.env.NODE_ENV": JSON.stringify("production") },
     });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@akanjs/devkit",
-  "version": "3.0.0-alpha.95",
+  "version": "3.0.0-alpha.96",
   "sourceType": "module",
   "type": "module",
   "publishConfig": {
@@ -44,7 +44,7 @@
     "@langchain/openai": "^1.4.6",
     "@tailwindcss/node": "^4.3.0",
     "@trapezedev/project": "^7.1.4",
-    "akanjs": "3.0.0-alpha.95",
+    "akanjs": "3.0.0-alpha.96",
     "chalk": "^5.6.2",
     "commander": "^14.0.3",
     "dayjs": "^1.11.20",