From f6bb148f74f523375ff168ff071e8a993011ae8e Mon Sep 17 00:00:00 2001
From: Claude Bot <claude-bot@bun.sh>
Date: Thu, 22 Jan 2026 19:57:42 +0000
Subject: [PATCH] fix(cli): make --compile imply --production as documented

The `--compile` flag help text claims it "Implies --production", but
it did not actually enable production mode or minification. This fix
ensures that when `--compile` is used, it also enables:
- All minification flags (minify_syntax, minify_whitespace, minify_identifiers)
- The production bundler option

Fixes #26373

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 src/cli/Arguments.zig               |   7 +-
 test/regression/issue/26373.test.ts | 105 ++++++++++++++++++++++++++++
 2 files changed, 109 insertions(+), 3 deletions(-)
 create mode 100644 test/regression/issue/26373.test.ts

diff --git a/src/cli/Arguments.zig b/src/cli/Arguments.zig
index f3c3c299da..bd62cd8d2b 100644
--- a/src/cli/Arguments.zig
+++ b/src/cli/Arguments.zig
@@ -915,7 +915,8 @@ pub fn parse(allocator: std.mem.Allocator, ctx: Command.Context, comptime cmd: C
         ctx.bundler_options.transform_only = args.flag("--no-bundle");
         ctx.bundler_options.bytecode = args.flag("--bytecode");
 
-        const production = args.flag("--production");
+        const compile = args.flag("--compile");
+        const production = args.flag("--production") or compile;
 
         if (args.flag("--app")) {
             if (!bun.FeatureFlags.bake()) {
@@ -1043,7 +1044,7 @@ pub fn parse(allocator: std.mem.Allocator, ctx: Command.Context, comptime cmd: C
             }
         }
 
-        if (args.flag("--compile")) {
+        if (compile) {
             ctx.bundler_options.compile = true;
             ctx.bundler_options.inline_entrypoint_import_meta_main = true;
         }
@@ -1436,7 +1437,7 @@ pub fn parse(allocator: std.mem.Allocator, ctx: Command.Context, comptime cmd: C
             Global.exit(1);
         }
 
-        if (args.flag("--production")) {
+        if (args.flag("--production") or ctx.bundler_options.compile) {
             const any_html = for (opts.entry_points) |entry_point| {
                 if (strings.hasSuffixComptime(entry_point, ".html")) {
                     break true;
diff --git a/test/regression/issue/26373.test.ts b/test/regression/issue/26373.test.ts
new file mode 100644
index 0000000000..f5dc315882
--- /dev/null
+++ b/test/regression/issue/26373.test.ts
@@ -0,0 +1,105 @@
+import { expect, test } from "bun:test";
+import { bunEnv, bunExe, tempDir } from "harness";
+
+// Test that --compile implies --production (minification enabled)
+// https://github.com/oven-sh/bun/issues/26373
+
+test("--compile implies --production and enables minification", async () => {
+  using dir = tempDir("issue-26373", {
+    // Use code with long variable names that will be minified in production mode
+    "index.ts": `
+const myVeryLongVariableName = "hello";
+const anotherLongVariableName = "world";
+console.log(myVeryLongVariableName, anotherLongVariableName);
+`,
+  });
+
+  // Build without flags (no minification, larger bundle)
+  await using noMinifyProc = Bun.spawn({
+    cmd: [bunExe(), "build", "--outfile", "no-minify.js", "./index.ts"],
+    cwd: String(dir),
+    env: bunEnv,
+    stdout: "pipe",
+    stderr: "pipe",
+  });
+
+  const [, noMinifyStderr, noMinifyExitCode] = await Promise.all([
+    noMinifyProc.stdout.text(),
+    noMinifyProc.stderr.text(),
+    noMinifyProc.exited,
+  ]);
+
+  expect(noMinifyStderr).toBe("");
+  expect(noMinifyExitCode).toBe(0);
+
+  // Build with --production (minification enabled, smaller bundle)
+  await using productionProc = Bun.spawn({
+    cmd: [bunExe(), "build", "--production", "--outfile", "production.js", "./index.ts"],
+    cwd: String(dir),
+    env: bunEnv,
+    stdout: "pipe",
+    stderr: "pipe",
+  });
+
+  const [, productionStderr, productionExitCode] = await Promise.all([
+    productionProc.stdout.text(),
+    productionProc.stderr.text(),
+    productionProc.exited,
+  ]);
+
+  expect(productionStderr).toBe("");
+  expect(productionExitCode).toBe(0);
+
+  // Read the bundle files
+  const noMinifyContent = await Bun.file(`${dir}/no-minify.js`).text();
+  const productionContent = await Bun.file(`${dir}/production.js`).text();
+
+  // Non-minified should contain the original long variable names
+  expect(noMinifyContent).toContain("myVeryLongVariableName");
+  expect(noMinifyContent).toContain("anotherLongVariableName");
+
+  // Production (minified) should NOT contain the original long variable names
+  expect(productionContent).not.toContain("myVeryLongVariableName");
+  expect(productionContent).not.toContain("anotherLongVariableName");
+
+  // Build with --compile only (should imply --production, including minification)
+  // The output should show "minify" step
+  await using compileProc = Bun.spawn({
+    cmd: [bunExe(), "build", "--compile", "--outfile", "compiled", "./index.ts"],
+    cwd: String(dir),
+    env: bunEnv,
+    stdout: "pipe",
+    stderr: "pipe",
+  });
+
+  const [compileStdout, compileStderr, compileExitCode] = await Promise.all([
+    compileProc.stdout.text(),
+    compileProc.stderr.text(),
+    compileProc.exited,
+  ]);
+
+  expect(compileStderr).toBe("");
+  expect(compileExitCode).toBe(0);
+
+  // The stdout should contain "minify" indicating minification was performed
+  // This is the key assertion that proves --compile implies --production
+  expect(compileStdout).toContain("minify");
+
+  // The compiled executable should run correctly
+  await using runCompiled = Bun.spawn({
+    cmd: [`${dir}/compiled`],
+    env: bunEnv,
+    stdout: "pipe",
+    stderr: "pipe",
+  });
+
+  const [compiledOutput, compiledStderr, runExitCode] = await Promise.all([
+    runCompiled.stdout.text(),
+    runCompiled.stderr.text(),
+    runCompiled.exited,
+  ]);
+
+  expect(compiledStderr).toBe("");
+  expect(compiledOutput).toBe("hello world\n");
+  expect(runExitCode).toBe(0);
+});