Error transforming bundle with 'uglify' plugin #13

ammar91 · 2016-10-11T19:00:12Z

I have an Angular 2 sample app that's bundle up with Rollup, but it throws an error when using this plugin. It says

Error transforming bundle with 'uglify' plugin: SyntaxError: Unexpected token: name (AppComponent).

This is how my rollup.config.s looks like


import rollup from 'rollup'
import nodeResolve from 'rollup-plugin-node-resolve'
import commonjs from 'rollup-plugin-commonjs';
import uglify from 'rollup-plugin-uglify';

export default {
    entry: 'dist/app/main.js',
    dest: 'dist/build.js', // output a single application bundle
    sourceMap: false,
    format: 'iife',
    plugins: [
        nodeResolve({ jsnext: true, module: true }),
        commonjs({
            namedExports: {
                // left-hand side can be an absolute path, a path 
                // relative to the current directory, or the name 
                // of a module in node_modules 
                'node_modules/primeng/primeng.js': ['DialogModule']
            },
            include: ['node_modules/rxjs/**', 'node_modules/primeng/**']
        }),
        uglify()
    ]
}

The text was updated successfully, but these errors were encountered:

ds8k · 2016-11-30T21:28:39Z

Having the same issue, were you able to resolve this?

TrySound · 2016-11-30T22:59:09Z

I guess that's because of es2015 in dependencies code. Uglify supports only es5.

mattpilott · 2016-12-08T23:35:03Z

I also have this. My config is as follows, I am using the harmony branch though.

// Rollup plugins
import babel from 'rollup-plugin-babel';
import uglify from 'rollup-plugin-uglify';
import { minify } from 'uglify-js';

export default {
    entry: './app/scripts/framework/framework.js',
    dest: './app/scripts/framework/framework.min.js',
    format: 'iife',
    moduleName: 'UIKit',
    moduleContext: {
        './app/scripts/framework/parties/promise.js' : 'window',
        './app/scripts/framework/parties/fetch.js' : 'window'
    },
    plugins: [
        babel({
            exclude: ['./node_modules/**', './app/scripts/framework/parties/**'],
            "presets": [[
                "env", {
                    "targets": {
                        "chrome": 52
                    },
                    "modules": false
                }
            ]],
            "plugins": [
                "external-helpers"
            ]
        }),
        uglify({}, minify)
    ]
};

I get this error Error transforming bundle with 'uglify' plugin: SyntaxError: Unexpected token: operator (>)

Any help is much appreciated

dotnetCarpenter · 2016-12-13T21:22:58Z

@TrySound Can you elaborate on how to, use the UglifyJS harmony branch by passing its minify function to minify your code, described in https://github.com/TrySound/rollup-plugin-uglify#warning?
It doesn't work out of the box.

TrySound · 2016-12-14T06:24:07Z

@dotnetCarpenter What's exactly doesn't work?

dotnetCarpenter · 2016-12-14T10:02:15Z

@TrySound Uglify is not able to minify because of the es2015 syntax, when using the work-around, out of the box. I think we're all a bit clueless to how we can use the harmony branch of uglify-js. Well, maybe not @matt3224 but I for one don't know how to install a branch of a npm module.

import uglify from 'rollup-plugin-uglify'
import { minify } from 'uglify-js'
...
export default {
    entry: "js/src/main.js",
    plugins: [
        babel(babelrc()),
        uglify({}, minify)

Running the above through rollup gives me:

npm run rollup -- --config rollup.main.js

> timetracker-frontend@1.0.0 rollup /var/www/xmaspeer/timetracker/public
> rollup "--config" "rollup.main.js"

Compiling source to execute on chrome 55, firefox 50, safari 10
Treating 'app' as external dependency
Error transforming bundle with 'uglify' plugin: SyntaxError: Unexpected token: operator (>)
Error
    at new JS_Parse_Error (eval at <anonymous> (/var/www/xmaspeer/timetracker/public/node_modules/uglify-js/tools/node.js:28:1), <anonymous>:1547:18)
    at js_error (eval at <anonymous> (/var/www/xmaspeer/timetracker/public/node_modules/uglify-js/tools/node.js:28:1), <anonymous>:1555:11)
    at croak (eval at <anonymous> (/var/www/xmaspeer/timetracker/public/node_modules/uglify-js/tools/node.js:28:1), <anonymous>:2094:9)
    at token_error (eval at <anonymous> (/var/www/xmaspeer/timetracker/public/node_modules/uglify-js/tools/node.js:28:1), <anonymous>:2102:9)
    at unexpected (eval at <anonymous> (/var/www/xmaspeer/timetracker/public/node_modules/uglify-js/tools/node.js:28:1), <anonymous>:2108:9)
    at expr_atom (eval at <anonymous> (/var/www/xmaspeer/timetracker/public/node_modules/uglify-js/tools/node.js:28:1), <anonymous>:2635:9)
    at maybe_unary (eval at <anonymous> (/var/www/xmaspeer/timetracker/public/node_modules/uglify-js/tools/node.js:28:1), <anonymous>:2797:19)
    at expr_ops (eval at <anonymous> (/var/www/xmaspeer/timetracker/public/node_modules/uglify-js/tools/node.js:28:1), <anonymous>:2832:24)
    at maybe_conditional (eval at <anonymous> (/var/www/xmaspeer/timetracker/public/node_modules/uglify-js/tools/node.js:28:1), <anonymous>:2837:20)
    at maybe_assign (eval at <anonymous> (/var/www/xmaspeer/timetracker/public/node_modules/uglify-js/tools/node.js:28:1), <anonymous>:2861:20)
Type rollup --help for help, or visit https://github.com/rollup/rollup/wiki

My .babelrc look like this (I also use env preset to only transpile the code that is not supported by my target browsers):

{
  "presets": [
    ["env",
    {
      "debug": false,
      "targets": {
        "browsers": [
          "last 1 chrome version",
          "last 1 firefox version",
          "last 1 safari version"
        ]
      },
      "modules": false
    }]
  ],
  "plugins": [
    "external-helpers"
  ]
}

OS: Ubuntu 16.04 (Linux 4.4.0-53-generic)
node: v7.2.1
npm: 3.10.10

TrySound · 2016-12-14T10:22:59Z

npm i mishoo/UglifyJS2#harmony

dotnetCarpenter · 2016-12-14T11:48:37Z

@matt3224 @TrySound At first it didn't work after installing mishoo/UglifyJS2#harmony. I had to manually delete the uglify-js folder and then install again. Now it works! 💃

dotnetCarpenter · 2016-12-14T11:49:07Z

@TrySound Perhaps you should update the README with this as it's not apparent.

TrySound · 2016-12-14T12:20:34Z

@dotnetCarpenter PR welcome.

mattpilott · 2016-12-14T13:12:09Z

Just a quick update from me, I have this working now which is awesome, since installing the harmony branch via the npm i mishoo/UglifyJS2#harmony

@dotnetCarpenter i think its better to use either npm un uglify-js or npm uninstall uglify-js rather than deleting the folder manually 👍

Shyam-Chen · 2016-12-15T03:38:53Z

same here

Error transforming bundle with 'uglify' plugin: SyntaxError: Invalid syntax: 0a.h

OS: Windows 10

dotnetCarpenter · 2016-12-15T13:31:32Z

@matt3224 I am using yarn and since uglify-js isn't in my dependency list but is a dependency of rollup-plugin-uglify, yarn won't let me uninstall uglify-js. Unfortunately, yarn add --dev mishoo/UglifyJS2#harmony does not install the correct branch, it just does nothing but report back everything is ok. I did a git diff between the two uglify-js branches and then looked into the uglify-js folder to see if I had any of the differences, when I found none, I deleted the folder and did yarn add mishoo/UglifyJS2#harmony to get the correct version.

If I remember correctly, then npm un uglify-js just delete the uglify-js folder and update your package.json if you have it there. So a manual delete should be the same.

When I yarn add mishoo/UglifyJS2#harmony, without having uglify-js in node_modules/, the correct version of uglify-js will be installed. Had I done that initially, it might had work out from the beginning... But it's untested. E.g. yarn add --dev rollup-plugin-uglify mishoo/UglifyJS2#harmony might work.

TrySound · 2016-12-15T13:34:34Z

Report an issue in yarn.

dotnetCarpenter · 2016-12-15T13:36:40Z

@TrySound I know. But I'm pressed on time the next two weeks and got a lot of other priorities on my plate. I'll have to wait. But at least now, I'm not the only one with this information. ;)

TrySound · 2016-12-15T13:38:00Z

You may always use npm again.

jonataswalker · 2016-12-20T15:27:04Z

For me what it worked was:

{
  "devDependencies": {
    "uglify-js": "git://github.com/mishoo/UglifyJS2#harmony"
  }
}

Instead of:

npm i mishoo/UglifyJS2#harmony

piotr-cz · 2016-12-28T11:42:56Z

This worked for me:

npm uninstall --save-dev uglify-js
npm install --save-dev mishoo/UglifyJS2#harmony

aviddiviner · 2017-07-10T05:33:57Z

Just a note to anyone coming to this issue a few months later. That harmony branch has moved on quite a bit since it worked, so rather use the harmony-v2.8.0 tag, which is from around the time the last comments were made.

So, either

{"devDependencies": {"uglify-js": "git://github.com/mishoo/UglifyJS2#harmony-v2.8.0"}}

Or

npm install --save-dev mishoo/UglifyJS2#harmony-v2.8.0

TrySound · 2017-07-10T08:37:45Z

@aviddiviner harmony branch is released now as uglify-es

dotnetCarpenter · 2018-01-01T14:48:02Z

@TrySound I think you can safely close this issue as you have it documented in https://github.com/TrySound/rollup-plugin-uglify#warning

dotnetCarpenter · 2018-01-01T14:48:57Z

Or even better, switch to use uglify-es instead of uglify-js as a dependency.

LuisSevillano · 2018-04-12T11:29:37Z

For me what it worked was uninstall the plugin and install it again.
Thanks!

TrySound · 2018-04-12T11:53:08Z

@LuisSevillano yarn upgrade-interactive --latest

TrySound closed this as completed Dec 15, 2016

dotnetCarpenter mentioned this issue Dec 15, 2016

Reinstall a package from another branch does nothing yarnpkg/yarn#2265

Open

unional mentioned this issue Jan 13, 2017

Export default class with no name cause uglify to fail rollup/rollup-plugin-commonjs#161

Closed

Error transforming bundle with 'uglify' plugin #13

Error transforming bundle with 'uglify' plugin #13

Comments

ammar91 commented Oct 11, 2016

ds8k commented Nov 30, 2016

Uh oh!

TrySound commented Nov 30, 2016

Uh oh!

mattpilott commented Dec 8, 2016

Uh oh!

dotnetCarpenter commented Dec 13, 2016

Uh oh!

TrySound commented Dec 14, 2016

Uh oh!

dotnetCarpenter commented Dec 14, 2016

Uh oh!

TrySound commented Dec 14, 2016

Uh oh!

dotnetCarpenter commented Dec 14, 2016

Uh oh!

dotnetCarpenter commented Dec 14, 2016

Uh oh!

TrySound commented Dec 14, 2016

Uh oh!

mattpilott commented Dec 14, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Shyam-Chen commented Dec 15, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

dotnetCarpenter commented Dec 15, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

TrySound commented Dec 15, 2016

Uh oh!

dotnetCarpenter commented Dec 15, 2016

Uh oh!

TrySound commented Dec 15, 2016

Uh oh!

jonataswalker commented Dec 20, 2016

Uh oh!

piotr-cz commented Dec 28, 2016

Uh oh!

aviddiviner commented Jul 10, 2017

Uh oh!

TrySound commented Jul 10, 2017

Uh oh!

dotnetCarpenter commented Jan 1, 2018

Uh oh!

dotnetCarpenter commented Jan 1, 2018

Uh oh!

LuisSevillano commented Apr 12, 2018

Uh oh!

TrySound commented Apr 12, 2018

Uh oh!

mattpilott commented Dec 14, 2016 •

edited

Loading

Shyam-Chen commented Dec 15, 2016 •

edited

Loading

dotnetCarpenter commented Dec 15, 2016 •

edited

Loading