fix: add dynamic import to tracing deps if possible

Author: 3cp

src/transformers/name-amd-define.js

@@ -16,6 +16,8 @@ const amdRequireDepFinder = astMatcher.depFinder(
   'requirejs(__any, [__deps])',
   'requirejs(__any, [__deps], __any)'
 );
+// import() has been translated to imp0r_() in cjs-to-amd.js
+const dynamicDepFinder = astMatcher.depFinder('imp0r_(__dep)');
 
 // transform an anonymous amd define() into named define.
 // find deps.
@@ -28,6 +30,12 @@ export default function(unit) {
     }
   });
 
+  dynamicDepFinder(unit.contents).forEach(d => {
+    if (result.deps.indexOf(d) === -1) {
+      result.deps.push(d);
+    }
+  });
+
   result.deps = result.deps.filter(d => {
     // remove cjs constants
     if (d === 'require' || d === 'exports' || d === 'module') return false;

test/transformers/name-amd-define.spec.js

@@ -559,8 +559,8 @@ test('nameDefine get requirejs deps', t => {
 });
 
 test('nameDefine fills up module name', t => {
-  const cjs = "define(['require','exports','module','./a'],function(){});";
-  const cjsExpected = "define('cjs',['require','exports','module','./a'],function(){});";
+  const cjs = "define(['require','exports','module','./a'],function(require, exports, module){var imp0r_ = function(d){return requirejs([requirejs.resolveModuleId(module.id,d)]).then(function(r){return r[0]&&r[0].default?r[0].default:r;});}; imp0r_('hello');});";
+  const cjsExpected = "define('cjs',['require','exports','module','./a'],function(require, exports, module){var imp0r_ = function(d){return requirejs([requirejs.resolveModuleId(module.id,d)]).then(function(r){return r[0]&&r[0].default?r[0].default:r;});}; imp0r_('hello');});";
 
   const unit = {
     contents: cjs,
@@ -570,7 +570,7 @@ test('nameDefine fills up module name', t => {
   const r = nameDefine(unit);
   t.deepEqual(r.defined, ['cjs']);
   t.notOk(r.shimed);
-  t.deepEqual(r.deps, ['./a']);
+  t.deepEqual(r.deps, ['./a', 'hello']);
   t.equal(r.contents, cjsExpected);
   t.end();
 });