Simplifications to the NamespacedJsSourceModule class before attempting
to change how it works.
Replaced the globalization block with requireAll() invocations inside
each module -- temporarilly commented out tests that break only because
of this change, so we can integration test this against CaplinTrader.
Updated CommonJs classes so that use-time depenencies to non-leaf
classes are added beneath `module.exports`, plus externalized a couple
of private inner classes to make the depenency requirements easier to
understand.
Re-enabled the globalization block, and corresponding tests, now that we
know this must be kept.
#931 adding getCanonicalPathWhenPossible method - fixes bug when we tried to create an aspect with an invalid name on windows
Merge pull request #996 from BladeRunnerJS/globalization-block-removal

Globalization block removal
Fixed bug recently introduced to Topiarist when improving it's circular
dependency detection -- will add unit tests around this when it's merged
upstream into the 'topiarist' project.
Merge pull request #1004 from BladeRunnerJS/topiarist-circular-dependency-bug-fix

Fixed bug recently introduced to Topiarist when improving it's circular
Merge branch 'develop' into test-key-events-fix