The regex for loadChildren misses a case

[Ayyash]

this regex to catch loadChildren var loadChildrenRegex = /loadChildren[\s]*:[\s]*['|"](.*?)['|"]/gm;
does not catch loadChildren that do not use strings, or prefix with a variable like what I do:

loadChildren: config.moduleUrl + 'modulename#ModuleName';

[Brandon Roberts]

This is intentional. The loader is doing a string replacement, so if you're using variables or functions in the loadChildren value it's going to skip it.

[Ayyash]

why is it skipping it? whats the side effect?

[Brandon Roberts]

The string replacement is static. Code isn't going to be evaluated before the loader does the replacement since it won't be evaluated until runtime.