jquery-ui "requirable" as CJS module

[G-rom]

Hi,

Tools like webpack (or browserify, or ...) allow us to use a great range of module syntax (AMD, CJS, global, event ES6 with babel-loader) but

1/ jquery-ui is not up to date on npm see #11101 2/ jquery-ui package.json is not correct and it's missing a "main" section, i.e. "main": "./ui/core.js" or an "index.js" file / module at the root. With this fixed, jquery-ui cannot be required inside a project using modern project builder.

Would appreciate an update of package.json, at least #11101 can be by-passed by using github tarball url, but this one cannot by passed without a fork.

[G-rom]

Hi,

Tools like webpack (or browserify, or ...) allow us to use a great range of module syntax (AMD, CJS, global, event ES6 with babel-loader) but

1/ jquery-ui is not up to date on npm see #11101

2/ jquery-ui package.json is not correct and it's missing a "main" section, i.e. "main": "./ui/core.js" or an "index.js" file / module at the root. Without this fixed, jquery-ui cannot be required inside a project using modern project builder.

Would appreciate an update of package.json, at least #11101 can be by-passed by using github tarball url, but this one cannot by passed without a fork.

[Scott González]

There is no main file for jQuery UI. Please clearly explain what you're looking for, or open a PR that shows the requested changes.

[G-rom]

I know there is no main file, but a valid package.json must have one. So you should have a valid package.json or no package.json file at all. In this current state, we cannot use npm & jquery-ui github tarball release

Solutions:

1/ removing the package.json

2/ fix the package.json & add a "main" entry

3/ add an "index.js" module at the package root, seems like it's the default "main" for npm package, but this solution is kind of hackish

[Scott González]

Replying to G-rom:

I know there is no main file, but a valid package.json must have one. So you should have a valid package.json or no package.json file at all. In this current state, we cannot use npm & jquery-ui github tarball release

We do have a valid package.json. The main field is *not* required, only name and version are.

Solutions:

1/ removing the package.json

Not at all a solution, we absolutely need package.json, it is our dependency manager.

2/ fix the package.json & add a "main" entry

There's no reasonable entry to add that has been proposed.

3/ add an "index.js" module at the package root, seems like it's the default "main" for npm package, but this solution is kind of hackish

Definitely a hack, especially if it does nothing.

[G-rom]

a "main" entry is not required, if you have an index.js

Keep rejecting all the solutions and jquery-ui will remain unusable by builder relying on node require resolving logic.

Another solution could be to add an index.js which will export a simple object listing all "ui" modules, like

module.exports = {
  accordion: require('./ui/accordion'),
  autocomplete: require('./ui/autocomplete'),
  ...

};

[G-rom]

In fact, this could be the best solution, this way someone could do

var ui = require('jquery-ui');
var autocomplete = ui.autocomplete

or

var autocomplete = require('jquery-ui/ui/autocomplete');

for a fine grain load and a lighter build

[Scott González]

@G-rom please see ​https://github.com/jquery/jquery-ui/pull/1600

[Jörn Zaefferer]

Build: Add main entry for webpack support; actual value doesn't matter

Fixes #14375 Closes gh-1600

Changeset: 6308a261fd5d1be2362863e6a59fede4420d9f8c