Some of dependencies are removed from headless package.json, but still required

At https://github.com/conversejs/converse.js/commit/3f7ffd025a98ffb5deb4709c2f4bb25139d3ddbc#diff-8d8848d744f09155a6ed06111726618c some of dependencies have been removed, but they are still required.

For instance jed is not in the package.json anymore, but it is required in https://github.com/conversejs/converse.js/blob/master/src/headless/i18n.js which is required in https://github.com/conversejs/converse.js/blob/master/src/headless/converse-core.js. Therefore the only way to use @converse/headless is to manually add dependencies like jed to your local dependencies.

We should either remove the depending code from @converse/headless or add the dependencies to package.json back again.

[jcbrand]

Hi @kevinfaguiar,

thanks for your detailed response and for your patience in trying to use the headless build.

You’re right, we need to have a distribution build of converse-headless.js to be used by 3rd party projects.

I’ve now made the relevant changes (see here) so that there is a file dist/converse-headless.js inside @converse/headless and the main key of the package.json points to it.

I’ve also created a quick and dirty headless-example which uses @converse/headless and which you can look at.

Because these changes are not released yet, you’ll need to clone this repo and then npm link in @converse/headless so that you can have the latest code in your project. The headless-example README explains how you can do this.

I’m also closing this issue because I’ve now added Jed as a dependency, which is what this was about. Feel free to create new issues if you run into any problems.

Good luck and let me know how it goes!

[jcbrand]

@kevinfaguiar: backbone, es6-promise, moment, pluggable.js are all set as dependencies in the package.json of @converse/headless.

See here: https://github.com/conversejs/converse.js/blob/master/src/headless/package.json#L25

jQuery is not a dependency of @converse/headless and is not used, so I don’t know why you think it’s a missing dependency.

Concerning the snabdom errors, that’s because you need webpack aliases. See here: https://github.com/conversejs/converse.js/blob/master/webpack.config.js#L91

We should probably document that somewhere, or make a fix so that those aliases are no longer necessary.