module.hot not replace to false in production build

[MegaJSDeveloper]

🐛 bug report

module.hot not replace to false in production build

🤔 Expected Behavior

index.html

<script src="script.js"></script>

script.js

if (module.hot) {
	module.hot.dispose(function () {
		console.log('OLOLOOOOO')
	})
}

in production build all code like module.hot will be replace to false, like:

if (false) {
	module.hot.dispose(function () {
		console.log('OLOLOOOOO')
	})
}

and minificator will remove all if(false){ } code blocks from production build

😯 Current Behavior in Parcel v1.12.4

hot reload code get to the production bundle:

}({
	"mpVp": [function (require, module, exports) {
		module.hot && module.hot.dispose(function () {
			console.log("OLOLOOOOO")
		});
	}, {}]
}, {}, ["mpVp"], null)

this is not a good idea

💁 Possible Solution

you must replace module.hot in all scripts to false keyword, that's exactly what webpack does.

I use Parcel v1.12.4

[DeMoorJasper]

what is the parcel version? Or the command you ran?

[MegaJSDeveloper]

@DeMoorJasper parcel v1.12.4

command:

parcel build index.html