Adding dependency inference for module.declare(factory) signature.

nobleModules.js

@@ -608,6 +608,7 @@
         }
         //#endregion
 
+        //#region Module declaration implementation helpers
         function initializeMainModule(dependencies, moduleFactory) {
             moduleObjectFactory.setMainModuleExports({});
             memoizeAndProvideDependencies(MAIN_MODULE_ID, dependencies, moduleFactory, function onMainModuleReady() {
@@ -616,10 +617,30 @@
             });
         }
 
+        // RegExp and function based on BravoJS, which says "mostly borrowed from FlyScript."
+        // NB: requireRegExp has two capturing clauses: one for double quotes, and one for single.
+        // It does not capture the quotes (unlike BravoJS/FlyScript).
+        var requireRegExp = /\/\/.*|\/\*[\s\S]*?\*\/|"(?:\\[\s\S]|[^"\\])*"|'(?:\\[\s\S]|[^'\\])*'|[;=(,:!^]\s*\/(?:\\.|[^\/\\])+\/|(?:^|\W)\s*require\s*\(\s*(?:"((?:\\[\s\S]|[^"\\])*)"|'((?:\\[\s\S]|[^'\\])*)')\s*\)/g;
+
+        function scrapeDependenciesFrom(rawSource) {
+            var dependencies = [];
+
+            var result = null;
+            while ((result = requireRegExp.exec(rawSource)) !== null) {
+                var moduleIdentifier = result[1] || result[2];
+                if (moduleIdentifier) {
+                    dependencies.push(moduleIdentifier);
+                }
+            }
+
+            return dependencies;
+        }
+        //#endregion
+
         function declareImpl(dependencies, moduleFactory) {
             if (moduleFactory === undefined) {
                 moduleFactory = dependencies;
-                dependencies = [];
+                dependencies = scrapeDependenciesFrom(moduleFactory.toString());
             }
 
             if (!globalModule.main) {

test/demos/squares.js

@@ -0,0 +1,10 @@
+module.declare(function(require, exports, module) {
+	var area = require("./area");
+	var squarePerimeter = require       (  './perimeter').square;
+
+    // var commentedOut = require("./diamond/bottom");
+    /* var noGoodEither = require("./diamond/bottom"); */
+
+	exports.area = area.square;
+	exports.perimeter = squarePerimeter;
+})

test/moduleNamespace.js

@@ -13,6 +13,19 @@ asyncTest("declare: accepts labeled dependency objects and correctly provides th
     });
 });
 
+asyncModuleTest("declare: infers dependencies if only a factory function is given", function () {
+    module.provide(["demos/squares"], function () {
+        var squares = require("demos/squares");
+
+        strictEqual(squares.area(3), 9, "demos/area dependency was inferred and works");
+        strictEqual(squares.perimeter(3), 12, "demos/perimeter dependency was inferred and works");
+
+        strictEqual(require.isMemoized(require.id("demos/diamond/bottom")), false, "Commented-out demos/diamond/bottom dependency was not memoized");
+
+        start();
+    });
+});
+
 asyncModuleTest("load: when called twice in a row for the same module, both callbacks fire", function (require, exports, module) {
     var numberOfLoadsSoFar = 0;