bug #121 Keep process.env.NODE_ENV === "development" in the built files, close #120 (Kocal)

This PR was merged into the main branch.

Discussion
----------

Keep `process.env.NODE_ENV === "development"` in the built files, close #120

Commits
-------

9805e0d Keep `process.env.NODE_ENV === "development"` in the built files, close #120

Author: Kocal

CHANGELOG.md

@@ -1,5 +1,9 @@
 # CHANGELOG
 
+## 4.0.1
+
+* Fix `application.debug = true` by keeping `process.env.NODE_ENV` check in dist files, see https://github.com/symfony/stimulus-bridge/pull/121
+
 ## 4.0.0
 
 ### Breaking changes

dist/index.js

@@ -25,7 +25,7 @@ function identifierForContextKey(key) {
 //#region src/index.ts
 function startStimulusApp(context) {
 	const application = Application.start();
-	application.debug = true;
+	if (process.env.NODE_ENV === "development") application.debug = true;
 	if (context) application.load(definitionsFromContext(context));
 	for (const controllerName in symfonyControllers) {
 		if (!Object.prototype.hasOwnProperty.call(symfonyControllers, controllerName)) continue;

rolldown.config.mjs

@@ -18,6 +18,10 @@ export default defineConfig([
             '@hotwired/stimulus',
         ],
         plugins: [typescript()],
+        define: {
+            // Keep `process.env.NODE_ENV` as a string, so it can be replaced by Webpack when user builds its app
+            'process.env.NODE_ENV': 'process.env.NODE_ENV',
+        },
     },
 
     /*