grunt-amd-commonjs-wrapper
v0.0.4
Published
Convert AMD Modules with declared dependencies into the Simplified CommonJS Wrapper style
Downloads
4
Maintainers
Readme
grunt-amd-commonjs-wrapper
Convert a normal amd module to the simplified commonjs style amd module
What problem does this solve?
Specifying all the dependencies in an array, then receiving them in a callback makes it very difficult for a linter to detect unused dependencies. This in turn makes it hard to detect if files aren't being built into any requirejs build files.
require(['dep1', 'dep2', 'dep3'], function (dep1, dep2, dep3) {
// dep1 and dep2 are unused, but this will pass lint
return dep3;
});
When to use?
This is the opposite of most grunt tasks that you'd use as part of your build, in that it (hopefully) makes your code more readable and easy to lint. This means that you're only likely to use it a single time to convert the style of a huge project.
In addition, some people prefer writing their modules in the commonjs style (especially if you want to run your code in node as well). Once you've used this task to convert your modules, you could even just strip off the outer define call and re-add it at build time with something like grunt-amd-wrap.
Examples
define(['dep1', 'dep2'], function (var1, var2) {
return var1 + var2;
});
becomes
define(function (require, exports, module) {
var var1 = require('dep1');
var var2 = require('dep2');
return var1 + var2;
});
#####supports variable-less dependencies
define(['dep1', 'dep2'], function () {
});
becomes
define(function (require, exports, module) {
require('dep1');
require('dep2');
});
#####supports named modules
define('modulename', ['dep'], function () {
});
becomes
define('modulename', function (require, exports, module) {
require('dep');
});
#####supports 'use strict'
define('modulename', ['dep'], function () {
'use strict';
});
becomes
define('modulename', function (require, exports, module) {
'use strict';
require('dep');
});
#####leaves commonjs wrapper modules alone
define('modulename', function (require, exports, module) {
'use strict';
});
remains
define('modulename', function (require, exports, module) {
'use strict';
});
Whitespace Warning
There is an attempt to match the indentation of the rest of the file, but its far from perfect. Might upset your linter.
Relative Path Warning
If you're using paths not declared in your requirejs config file, then this will probably break all your paths. This is due to the commonjs style require statements using relative paths, rather than making all paths relative to your requirejs config baseUrl
setting.