amdefine - npm Package Compare versions

Comparing version 0.0.2 to 0.0.3

amdefine.js

 /** vim: et:ts=4:sw=4:sts=4
- * @license amdefine 0.0.2 Copyright (c) 2011, The Dojo Foundation All Rights Reserved.
+ * @license amdefine 0.0.3 Copyright (c) 2011, The Dojo Foundation All Rights Reserved.
  * Available via the MIT or new BSD license.
@@ -11,134 +11,131 @@ * see: http://github.com/jrburke/amdefine for details
 
-var path = require('path'),
-    loaderCache = {},
-    makeRequire;
+var path = require('path');
 
 /**
- * Given a relative module name, like ./something, normalize it to
- * a real name that can be mapped to a path.
- * @param {String} name the relative name
- * @param {String} baseName a real name that the name arg is relative
- * to.
- * @returns {String} normalized name
+ * Creates a define for node.
+ * @param {Object} module the "module" object that is defined by Node for the
+ * current module.
+ * @param {Function} [require]. Node's require function for the current module.
+ * It only needs to be passed in Node versions before 0.5, when module.require
+ * did not exist.
+ * @returns {Function} a define function that is usable for the current node
+ * module.
  */
-function normalize(name, baseName) {
-    return path.normalize(path.join(baseName, name));
-}
+function amdefine(module, require) {
+    var defineCache = {},
+        loaderCache = {},
+        alreadyCalled = false,
+        makeRequire, stringRequire;
 
-/**
- * Create the normalize() function passed to a loader plugin's
- * normalize method.
- */
-function makeNormalize(relName) {
-    return function (name) {
-        return normalize(name, relName);
-    };
-}
-
-function makeLoad(id) {
-    function load(value) {
-        loaderCache[id] = value;
+    /**
+     * Trims the . and .. from an array of path segments.
+     * It will keep a leading path segment if a .. will become
+     * the first path segment, to help with module name lookups,
+     * which act like paths, but can be remapped. But the end result,
+     * all paths that use this function should look normalized.
+     * NOTE: this method MODIFIES the input array.
+     * @param {Array} ary the array of path segments.
+     */
+    function trimDots(ary) {
+        var i, part;
+        for (i = 0; ary[i]; i+= 1) {
+            part = ary[i];
+            if (part === '.') {
+                ary.splice(i, 1);
+                i -= 1;
+            } else if (part === '..') {
+                if (i === 1 && (ary[2] === '..' || ary[0] === '..')) {
+                    //End of the line. Keep at least one non-dot
+                    //path segment at the front so it can be mapped
+                    //correctly to disk. Otherwise, there is likely
+                    //no path mapping for a path starting with '..'.
+                    //This can still fail, but catches the most reasonable
+                    //uses of ..
+                    break;
+                } else if (i > 0) {
+                    ary.splice(i - 1, 2);
+                    i -= 2;
+                }
+            }
+        }
     }
 
-    load.fromText = function (id, text) {
-        //This one is difficult because the text can/probably uses
-        //define, and any relative paths and requires should be relative
-        //to that id was it would be found on disk. But this would require
-        //bootstrapping a module/require fairly deeply from node core.
-        //Not sure how best to go about that yet.
-        throw new Error('amdefine does not implement load.fromText');
-    };
+    function normalize(name, baseName) {
+        var baseParts;
 
-    return load;
-}
+        //Adjust any relative paths.
+        if (name && name.charAt(0) === '.') {
+            //If have a base name, try to normalize against it,
+            //otherwise, assume it is a top-level require that will
+            //be relative to baseUrl in the end.
+            if (baseName) {
+                baseParts = baseName.split('/');
+                baseParts = baseParts.slice(0, baseParts.length - 1);
+                baseParts = baseParts.concat(name.split('/'));
+                trimDots(baseParts);
+                name = baseParts.join('/');
+            }
+        }
 
-function stringRequire(module, require, id) {
-    //Split the ID by a ! so that
-    var index = id.indexOf('!'),
-        relId = path.dirname(module.filename),
-        prefix, plugin;
+        return name;
+    }
 
-    if (index === -1) {
-        //Straight module lookup. If it is one of the special dependencies,
-        //deal with it, otherwise, delegate to node.
-        if (id === 'require') {
-            return makeRequire(module, require);
-        } else if (id === 'exports') {
-            return module.exports;
-        } else if (id === 'module') {
-            return module;
-        } else {
-            return require(id);
-        }
-    } else {
-        //There is a plugin in play.
-        prefix = id.substring(0, index);
-        id = id.substring(index + 1, id.length);
+    /**
+     * Create the normalize() function passed to a loader plugin's
+     * normalize method.
+     */
+    function makeNormalize(relName) {
+        return function (name) {
+            return normalize(name, relName);
+        };
+    }
 
-        plugin = require(prefix);
-
-        if (plugin.normalize) {
-            id = plugin.normalize(id, makeNormalize(relId));
-        } else {
-            //Normalize the ID normally.
-            id = normalize(id, relId);
+    function makeLoad(id) {
+        function load(value) {
+            loaderCache[id] = value;
         }
 
-        if (loaderCache[id]) {
-            return loaderCache[id];
-        } else {
-            plugin.load(id, makeRequire(module, require), makeLoad(id), {});
+        load.fromText = function (id, text) {
+            //This one is difficult because the text can/probably uses
+            //define, and any relative paths and requires should be relative
+            //to that id was it would be found on disk. But this would require
+            //bootstrapping a module/require fairly deeply from node core.
+            //Not sure how best to go about that yet.
+            throw new Error('amdefine does not implement load.fromText');
+        };
 
-            return loaderCache[id];
-        }
+        return load;
     }
-}
 
-makeRequire = function (module, require) {
-    function amdRequire(deps, callback) {
-        if (typeof deps === 'string') {
-            //Synchronous, single module require('')
-            return stringRequire(module, require, deps);
-        } else {
-            //Array of dependencies with a callback.
+    makeRequire = function (systemRequire, exports, module, relId) {
+        function amdRequire(deps, callback) {
+            if (typeof deps === 'string') {
+                //Synchronous, single module require('')
+                return stringRequire(systemRequire, exports, module, deps, relId);
+            } else {
+                //Array of dependencies with a callback.
 
-            //Convert the dependencies to modules.
-            deps = deps.map(function (depName) {
-                return stringRequire(module, require, depName);
-            });
+                //Convert the dependencies to modules.
+                deps = deps.map(function (depName) {
+                    return stringRequire(systemRequire, exports, module, depName, relId);
+                });
 
-            //Wait for next tick to call back the require call.
-            process.nextTick(function () {
-                callback.apply(null, deps);
-            });
-
-            //Keeps strict checking in komodo happy.
-            return undefined;
+                //Wait for next tick to call back the require call.
+                process.nextTick(function () {
+                    callback.apply(null, deps);
+                });
+            }
         }
-    }
 
-    amdRequire.toUrl = function (filePath) {
-        if (filePath.indexOf('.') === 0) {
-            return normalize(filePath, path.dirname(module.filename));
-        } else {
-            return filePath;
-        }
+        amdRequire.toUrl = function (filePath) {
+            if (filePath.indexOf('.') === 0) {
+                return normalize(filePath, path.dirname(module.filename));
+            } else {
+                return filePath;
+            }
+        };
+
+        return amdRequire;
     };
 
-    return amdRequire;
-};
-
-/**
- * Creates a define for node.
- * @param {Object} module the "module" object that is defined by Node for the
- * current module.
- * @param {Function} [require]. Node's require function for the current module.
- * It only needs to be passed in Node versions before 0.5, when module.require
- * did not exist.
- * @returns {Function} a define function that is usable for the current node
- * module.
- */
-function amdefine(module, require) {
-    var alreadyCalled = false;
-
     //Favor explicit value, passed in if the module wants to support Node 0.4.
@@ -149,25 +146,28 @@ require = require || function req() {
 
-    //Create a define function specific to the module asking for amdefine.
-    function define() {
+    function runFactory(id, deps, factory) {
+        var r, e, m, result;
 
-        var args = arguments,
-            factory = args[args.length - 1],
-            isFactoryFunction = (typeof factory === 'function'),
-            deps, result;
+        if (id) {
+            e = loaderCache[id] = {};
+            m = {
+                id: id,
+                uri: __filename,
+                exports: e
+            };
+            r = makeRequire(undefined, e, m, id);
+        } else {
+            //Only support one define call per file
+            if (alreadyCalled) {
+                throw new Error('amdefine with no module ID cannot be called more than once per file.');
+            }
+            alreadyCalled = true;
 
-        //Only support one define call per file
-        if (alreadyCalled) {
-            throw new Error('amdefine cannot be called more than once per file.');
+            //Use the real variables from node
+            //Use module.exports for exports, since
+            //the exports in here is amdefine exports.
+            e = module.exports;
+            m = module;
+            r = makeRequire(require, e, m, module.id);
         }
-        alreadyCalled = true;
 
-        //Grab array of dependencies if it is there.
-        if (args.length > 1) {
-            deps = args[args.length - 2];
-            if (!Array.isArray(deps)) {
-                //deps is not an array, may be an ID. Discard it.
-                deps = null;
-            }
-        }
-
         //If there are dependencies, they are strings, so need
@@ -177,18 +177,17 @@ //to convert them to dependency values.
             deps = deps.map(function (depName) {
-                return stringRequire(module, require, depName);
+                return r(depName);
             });
-        } else if (isFactoryFunction) {
-            //Pass in the standard require, exports, module
-            deps = [makeRequire(module, require), module.exports, module];
         }
 
-        if (!isFactoryFunction) {
-            //Factory is an object that should just be used for the define call.
-            module.exports = factory;
+        //Call the factory with the right dependencies.
+        if (typeof factory === 'function') {
+            result = factory.apply(module.exports, deps);
         } else {
-            //Call the factory with the right dependencies.
-            result = factory.apply(module.exports, deps);
+            result = factory;
+        }
 
-            if (result !== undefined) {
-                module.exports = result;
+        if (result !== undefined) {
+            m.exports = result;
+            if (id) {
+                loaderCache[id] = m.exports;
             }
@@ -198,2 +197,102 @@ }
 
+    stringRequire = function (systemRequire, exports, module, id, relId) {
+        //Split the ID by a ! so that
+        var index = id.indexOf('!'),
+            originalId = id,
+            prefix, plugin;
+
+        if (index === -1) {
+            id = normalize(id, relId);
+
+            //Straight module lookup. If it is one of the special dependencies,
+            //deal with it, otherwise, delegate to node.
+            if (id === 'require') {
+                return makeRequire(systemRequire, exports, module, relId);
+            } else if (id === 'exports') {
+                return exports;
+            } else if (id === 'module') {
+                return module;
+            } else if (loaderCache.hasOwnProperty(id)) {
+                return loaderCache[id];
+            } else if (defineCache[id]) {
+                runFactory.apply(null, defineCache[id]);
+                return loaderCache[id];
+            } else {
+                if(systemRequire) {
+                    return systemRequire(originalId);
+                } else {
+                    throw new Error('No module with ID: ' + id);
+                }
+            }
+        } else {
+            //There is a plugin in play.
+            prefix = id.substring(0, index);
+            id = id.substring(index + 1, id.length);
+
+            plugin = stringRequire(systemRequire, exports, module, prefix, relId);
+
+            if (plugin.normalize) {
+                id = plugin.normalize(id, makeNormalize(relId));
+            } else {
+                //Normalize the ID normally.
+                id = normalize(id, relId);
+            }
+
+            if (loaderCache[id]) {
+                return loaderCache[id];
+            } else {
+                plugin.load(id, makeRequire(systemRequire, exports, module, relId), makeLoad(id), {});
+
+                return loaderCache[id];
+            }
+        }
+    };
+
+    //Create a define function specific to the module asking for amdefine.
+    function define(id, deps, factory) {
+        if (Array.isArray(id)) {
+            factory = deps;
+            deps = id;
+            id = undefined;
+        } else if (typeof id !== 'string') {
+            factory = id;
+            id = deps = undefined;
+        }
+
+        if (deps && !Array.isArray(deps)) {
+            factory = deps;
+            deps = undefined;
+        }
+
+        if (!deps) {
+            deps = ['require', 'exports', 'module'];
+        }
+
+        //Set up properties for this module. If an ID, then use
+        //internal cache. If no ID, then use the external variables
+        //for this node module.
+        if (id) {
+            //Put the module in deep freeze until there is a
+            //require call for it.
+            defineCache[id] = [id, deps, factory];
+        } else {
+            runFactory(id, deps, factory);
+        }
+    }
+
+    //define.require, which has access to all the values in the
+    //cache. Useful for AMD modules that all have IDs in the file,
+    //but need to finally export a value to node based on one of those
+    //IDs.
+    define.require = function (id) {
+        if (loaderCache[id]) {
+            return loaderCache[id];
+        }
+
+        if (defineCache[id]) {
+            runFactory.apply(null, defineCache[id]);
+            return loaderCache[id];
+        }
+    };
+
     define.amd = {};
@@ -200,0 +299,0 @@

package.json

 {
     "name": "amdefine",
     "description": "Provide AMD's define() API for declaring modules in the AMD format",
-    "version": "0.0.2",
+    "version": "0.0.3",
     "homepage": "http://github.com/jrburke/amdefine.js",
@@ -6,0 +6,0 @@ "author": "James Burke <[e-mail]> (http://github.com/jrburke)",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

No README

Quality

Package does not have a README. This may indicate a failed publish or a low quality package.

Found 1 instance in 1 package

Dynamic require

Supply chain risk

Dynamic require can indicate the package is performing dangerous or unsafe dynamic code execution.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

17238

increased by81.36%

Number of package files

5

increased by25%

Lines of code

260

increased by53.85%

Number of medium quality alerts

0

decreased by-100%

Number of lines in readme file

120

increased byInfinity%

Number of low supply chain risk alerts

2

decreased by-50%

Worsened metrics

Number of medium supply chain risk alerts

1

increased byInfinity%

No dependency changes