SyntaxError in micromatch when starting browser-sync

[mxeff]

Issue details

A SyntaxError is thrown when starting browser-sync:

[13:07:39] Finished 'build' after 20 s
[13:07:39] Starting 'server'...
[13:07:39] 'server' errored after 7.79 ms
[13:07:39] /xxx/xxx/node_modules/micromatch/index.js:44
    let isMatch = picomatch(String(patterns[i]), { ...options, onResult }, true);
                                                   ^^^

SyntaxError: Unexpected token ...
    at createScript (vm.js:56:10)
    at Object.runInThisContext (vm.js:97:10)
    at Module._compile (module.js:542:28)
    at Module._extensions..js (module.js:579:10)
    at Object.require.extensions.(anonymous function) [as .js] (/xxx/xxx/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)

Steps to reproduce/test case

1. Run npm install with browser-sync@^2.10.0 as dev dependency
2. Run gulp

Please specify which version of Browsersync, node and npm you're running

• Browsersync [2.26.3]
• Node [6.12.2]
• Npm [3.10.10]

Affected platforms

• linux
• windows
• OS X
• freebsd
• solaris
• other (please specify which)

Browsersync use-case

• API
• Gulp
• Grunt
• CLI

for all other use-cases, (gulp, grunt etc), please show us exactly how you're using Browsersync

gulp.task('default',
    gulp.series(
        'build',
        server,
        watch
    )
);

function server(done) {
  'use strict';
  browser.init({
    server: PATHS.dist.base, port: PORT, open: false
  });
  done();
}

PATHS.dist.base: 'dist'
PORT: 1234

+1

I have the same issue.

Browsersync [2.26.4]

[michaellenahan]

Introduced here (v2.26.4) as far as I can tell: in my own project, pinning browser-sync to "2.26.3" in my own package.json prevented the issue from occuring.

This is the commit in browser-sync which introduces this issue:

05a58db

Specifically, in the package.json file:
packages/browser-sync/package.json

"micromatch": "2.3.11",
... is replaced with:
"micromatch": "^4.0.2",

Please see:
micromatch/micromatch#159

My suggestion is to revert to the latest 3.x version of micromatch ("3.1.10"), which does not make use of the "..." three-dots spread syntax https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Spread_syntax in index.js

Compare:
https://github.com/micromatch/micromatch/blob/master/index.js
https://github.com/micromatch/micromatch/blob/3.1.10/index.js

[jonschlinkert]

This is happening because the engine property specified in the browser-sync package.json is 0.8.0.

If we want to keep compatibility with older vesions of Node I suppose we could add a pre-publish step to transpile. But we’d have to do it in all dependencies of micromatch as well. Id be willing to discuss further if someone wants to create an issue at micromatch with ideas for a solution that will make everyone happy.

[wesselbaum]

Did I get it right that this is a node 6 problem? So the solution would be to stay on 6 and use previous versions or update to at least node 8?

[franco-testagrossa]

+1

I have the same issue.

Browsersync [2.26.4]

[atmospherix]

Encountered this today and lost several hours of work diagnosing the problem.

Based on @michaellenahan 's comment I changed the version of micromatch to "3.1.10" in browser-sync's package.json ... That got me running again, but I have no idea what other problems that may introduce, if any.

[vnayanac]

I'm also facing the same issue. Changed the micromatch version to 3.1.10 in browser-sync package.json.
It is fine until local environment.
Even I tried with browser-sync new version in main package.json. But no luck.

[shakyShane]

fixed in browser-sync@2.26.5