keepNames default true, not experimental now

Author: matt-aitken

packages/cli-v3/src/build/bundle.ts

@@ -176,7 +176,10 @@ async function createBuildOptions(
   const customConditions = options.resolvedConfig.build?.conditions ?? [];
   const conditions = [...customConditions, "trigger.dev", "module", "node"];
 
-  const keepNames = options.resolvedConfig.build?.experimental_keepNames ?? false;
+  const keepNames =
+    options.resolvedConfig.build?.keepNames ??
+    options.resolvedConfig.build?.experimental_keepNames ??
+    true;
   const minify = options.resolvedConfig.build?.experimental_minify ?? false;
 
   const $buildPlugins = await buildPlugins(options.target, options.resolvedConfig);

packages/core/src/v3/config.ts

@@ -184,17 +184,22 @@ export type TriggerConfig = {
     experimental_autoDetectExternal?: boolean;
 
     /**
-     * **WARNING: This is an experimental feature and might be removed in a future version.**
+     * This still works but use `keepNames` instead.
      *
-     * Preserve the original names of functions and classes in the bundle. This can fix issues with frameworks that rely on the original names for registration and binding, for example MikroORM.
+     * @default true
      *
-     * @link https://esbuild.github.io/api/#keep-names
+     * @deprecated (use keepNames instead)
+     */
+    experimental_keepNames?: boolean;
+
+    /* Set to false to minify the original names of functions and classes in the bundle.
+     * This can make bundles smaller at the cost of compatibility with frameworks that rely on function/class/variable names.
      *
-     * @default false
+     * @link https://esbuild.github.io/api/#keep-names
      *
-     * @deprecated (experimental)
+     * @default true
      */
-    experimental_keepNames?: boolean;
+    keepNames?: boolean;
 
     /**
      * **WARNING: This is an experimental feature and might be removed in a future version.**