The code cannot be efficiently minified

[lovasoa]

The code dynamically fetches it's own methods in https://github.com/sanity-io/mendoza-js/blob/master/src/internal-patcher.ts#L68-L69 :

let processor = `process${op}`
this[processor].apply(this)

When this code goes through a performant javascript minifier, it will break.

For the code to be able to go through minification efficiently and without breaking, the OPS constant should be a list of functions instead of a list of strings.

[judofyr]

What "performant javascript minifier" are you using? I wasn't aware that any of them would automatically optimize away method definitions in classes.

[lovasoa]

I think all major minifiers do it on their highest minification levels. See for instance mangle.properties in UglifyJS and terser, and ADVANCED_OPTIMIZATIONS in google closure compiler. Shipping long method names to the user increases bundle sizes significantly.

As a quick example, for this package :

$ npx rollup index.js | npx terser --mangle-props | wc -c
3104
$ npx rollup index.js | npx terser | wc -c
5446

Shipping property names to the user almost doubles the bundle size.