Rename babel-plugin-minify-computed-member-expressions to the existin…

[hzoo]

…g babel-plugin-transform-member-expression-literals

Ref #19 (comment)

Should all of the plugins be namespaced under minify-? or should we reuse the old ones and move them over?

• Move babel-plugin-transform-member-expression-literals
• Move merge-sibling-variables + move out of simplify plugin
• Move property-literals into repo instead of babel-plugin-minify-computed-properties
• move minify-booleans
• move remove console/debugger (not priority)
• move simplfy-comparison-operators to babel-plugin-minify-strict-equals (although the latter sounds better?)
• undefined-to-void
• Fix preset names

After this PR/Release

• Remove plugins from babel
• Move plugins into minify section/preset on babeljs.io

[hzoo]

fixup minify-type-constructors and rm from minify-simplify (be75675) - it looks like the changes from https://github.com/amasad/babel-minify/pull/17/files were lost in the split up pr b510d6e

[amasad]

LGTM -- merge whenever you're ready (make sure tests pass and no regression in benchmarks)

[hzoo]

I might as well move over node-env-inline and inline-environment-variables and we can decide to deprecate or move them again

npm t

> @ test /Users/hzhu/dev/babel-minify
> eslint src/ && jest

Using Jest CLI v0.6.1
 PASS  packages/babel-helper-is-nodes-equiv/__tests__/is-nodes-equiv-test.js (1.764s)
 PASS  packages/babel-plugin-minify-empty-function/__tests__/empty-function-test.js (1.826s)
 PASS  packages/babel-plugin-minify-replace/__tests__/replace-test.js (1.864s)
 PASS  packages/babel-plugin-minify-flip-comparisons/__tests__/flip-comparisons-test.js (1.846s)
 PASS  packages/babel-plugin-minify-infinity/__tests__/infinity-test.js (1.891s)
 PASS  packages/babel-plugin-minify-mangle-names/__tests__/mangle-names-test.js (2.382s)
 PASS  packages/babel-plugin-minify-simplify/__tests__/simplify-test.js (2.932s)
 PASS  packages/babel-plugin-minify-guarded-expressions/__tests__/guarded-expressions-test.js (1.114s)
 PASS  packages/babel-plugin-minify-constant-folding/__tests__/constant-folding-test.js (1.076s)
 PASS  packages/babel-plugin-transform-merge-sibling-variables/__tests__/transform-merge-sibling-variables-test.js (1.134s)
 PASS  packages/babel-plugin-transform-simplify-comparison-operators/__tests__/strict-equals-test.js (1.088s)
 PASS  packages/babel-plugin-transform-remove-debugger/__tests__/remove-debugger-test.js (1.046s)
 PASS  packages/babel-plugin-transform-member-expression-literals/__tests__/transform-member-expression-literals-test.js (1.055s)
 PASS  packages/babel-plugin-transform-property-literals/__tests__/transform-property-literals-test.js (0.877s)
 PASS  packages/babel-plugin-minify-type-constructors/__tests__/type-constructors-test.js (0.802s)
 PASS  packages/babel-plugin-transform-undefined-to-void/__tests__/undefined-test.js (0.854s)
 PASS  packages/babel-plugin-transform-minify-booleans/__tests__/booleans-test.js (0.841s)
 PASS  packages/babel-plugin-transform-remove-console/__tests__/remove-console-test.js (1.1s)
 PASS  packages/babel-plugin-minify-dead-code-elimination/__tests__/dead-code-elimination-test.js (4.479s)
241 tests passed (241 total in 19 test suites, run time 6.467s)

[hzoo]

Not sure why I'm getting different results from cloning/npm i/bootstrap

Looks like it's a bit bigger/slower but negligible. Might be a good idea to see the diff though

new

./scripts/benchmark.js ./scripts/fixtures/backbone.js
        raw     raw win gzip   gzip win parse time run
babel   21.93kB 219%    7.3kB  169%     2ms        748ms
uglify  21.81kB 221%    7.31kB 169%     1ms        313ms
closure 21.67kB 223%    7.37kB 167%     2ms        2015ms

./scripts/benchmark.js ./scripts/fixtures/react.js
        raw      raw win gzip    gzip win parse time run
closure 171.46kB 265%    52.97kB 168%     12ms       2989ms
uglify  176.39kB 255%    53.17kB 167%     9ms        1473ms
babel   179.17kB 250%    53.28kB 166%     10ms       3176ms

./scripts/benchmark.js ./scripts/fixtures/jquery.js
        raw     raw win gzip    gzip win parse time run
uglify  94.38kB 217%    32.82kB 157%     6ms        1122ms
babel   94.84kB 216%    33.02kB 156%     6ms        3455ms
closure 94.23kB 218%    33.38kB 153%     8ms        2657ms

Ignore: Ok I cloned the repo and was getting different results for some reason: https://gist.github.com/hzoo/3072a8f72f12a8aeb1dc0e91347e00e9

[hzoo]

Ok just found out the benchmark doesn't run all the plugins

[hzoo]

./scripts/benchmark.js ./scripts/fixtures/jquery.js
              raw     raw win gzip    gzip win parse time run
uglify        94.38kB 217%    32.82kB 157%     6ms        971ms
minify-preset 93.63kB 220%    32.94kB 156%     6ms        3224ms
babel         94.91kB 216%    33.04kB 156%     6ms        2585ms
closure       94.23kB 218%    33.38kB 153%     7ms        2101ms

./scripts/benchmark.js ./scripts/fixtures/react.js
              raw      raw win gzip    gzip win parse time run
minify-preset 176.03kB 256%    52.8kB  168%     9ms        2872ms
closure       171.46kB 265%    52.97kB 168%     11ms       2455ms
uglify        176.39kB 255%    53.17kB 167%     9ms        1267ms
babel         182.69kB 243%    53.7kB  164%     10ms       2263ms

./scripts/benchmark.js ./scripts/fixtures/backbone.js
              raw     raw win gzip   gzip win parse time run
minify-preset 21.74kB 222%    7.28kB 170%     1ms        725ms
uglify        21.81kB 221%    7.31kB 169%     2ms        278ms
babel         22.17kB 215%    7.32kB 169%     1ms        414ms
closure       21.67kB 223%    7.37kB 167%     2ms        1411ms

[hzoo]

we can just replace below one with this (wasn't sure if that was specifically needed) @amasad

[hzoo]

Oops.. mean't to do a regular merge commit (oh well)

Will do

• is http://babeljs.io/docs/plugins/transform-node-env-inline/ the same as http://babeljs.io/docs/plugins/transform-inline-environment-variables/? and babel-plugin-minify-replace
• Docs

in another pr