Fixed a TODO within a test that I happened to be looking at.
Got rid of the aliasDependencies hack as fixing a TODO in a test (in the
previous commit) means there are no longer any tests breaking, and I
can't think of a reason why we'd need interfaces associated with aliases
to be bundled before a class that references that alias.
Updated SourceModuleDependencyOrderCalculator so that it builds a
dependency graph in an earlier phase, allowing us to change the
dependency graph we use to order dependencies by.
Cleaned up CircularDependencyException code before attempting to upgrade
so that we can have comparable circular exception messages on the server
to what we currently have on the client.
Updated CircularDependencyException to produce error messages with the
same format as that used on the client.