Publish autobahn-js-built to npm

[manas-chaudhari]

I get this error when creating production build.

Failed to compile.

static/js/main.955798b7.js from UglifyJs
Unexpected token: name (Simple) [./~/cbor/lib/simple.js:9,0][static/js/main.955798b7.js:10295,6]

error Command failed with exit code 1.

Content at static/js/main.955798b7.js:10295,6

class Simple {
      ^

Instructions to reproduce:

create-react-app test
cd test
yarn add autobahn

# In src/App.js, append the following line: 
import autobahn from 'autobahn';

yarn run build

I am very new to this. Here are my thoughts:
I think the discussion is facebook/create-react-app#984 (comment) is relevant here. It mentions that ES6 syntax in dependencies is not supported by create-react-app. If that's the case, can we transpile before publishing the autobahn npm module?

[adrlen]

Ok so the problem is hildjj/node-cbor is ES6 and UglifyJS doesn't support it. It should be transpiled by Babel but I'm not sure when this must be done..

[adrlen]

@oberstet Do you have any info on this topic ? Thanks for your time.

[oberstet]

@adrlen Sorry, I don't have info on that and no time to look into this also. Autobahn officially supports 2 ways of use and distribution:

1. browsers: pre-built library from https://github.com/crossbario/autobahn-js-built
2. Node: npm

[glortho]

We've had so many struggles getting autobahn to work with webpack that we should've stopped trying a long time ago. But we're committed to a predictable, ergonomic strategy for package bundling, versioning, etc. so this is our solution to the latest quagmire:

webpack.config.js

{
  test: /\.js$/,
  include: /mySrcDir|cbor/,
    use: {
      loader: 'babel-loader',
      ...
}

Note that if you're using create-react-app, this solution requires ejecting.

[offroff]

I manually copied the pre-built library to my create-react-app. That works but it's not a good solution in the long run. Can you publish the pre-built version to the npm registry?

[adrlen]

Actually, there is already an old pre-build version published on npmjs by a random stranger.
https://www.npmjs.com/package/autobahn-js-built

@oberstet Why isn't this repo published on npm by your team ?

[oberstet]

@adrlen as we don't have to intro code changes for this (to support yet another JS build thingy), we could push autobahn-js-built to npm of course when doing a release to this repo with low effort, yes.

ok, we probably should given this seems to be of interest to quite some developers, and before people use stuff released by strangers, definitely.

[oberstet]

we might need some mods to the packge.json in https://github.com/crossbario/autobahn-js-built

maybe a Makefile with a "publish" target that invokes npm publish

apart from that, this step of releasing also the browser built version to npm should then be documented in https://github.com/crossbario/autobahn-js/blob/master/doc/release-process.md