Doesn't detect circular dependency when ./ is in module path

[apaatsio]

Problem:

It says no circular dependencies even though there is one.

To reproduce:

Directory structure

a.coffee
foo/b.coffee

a.coffee:

define 'a', ['./foo/b'], ->

b.coffee:

define 'foo/b', ['a'], ->

Now run madge --circular --format amd ./ and notice that it doesn't detect the circular dependency.

If you remove the ./ from a.coffee it detects the circularity.

I noticed this on Windows. I didn't have time yet to try on *nix.

[pahen]

Thanks for a good bug report! I have fixed it in v.0.3.3 now.

[apaatsio]

Cool, thanks for the prompt fix!

[RoyalBingBong]

I think this one back!
This example will say it does not have cyclic dependencies:
index.ts

export class Bar {}

export * from './foo';

foo.ts

import { Bar } from './';

export class Foo extends Bar {}

If I change import { Bar } from './'; to import { Bar } from './index'; then --circular correctly shows that I have a circular dependency.

Edit: Seems to only be the case for Typescript.

[pahen]

Do you include your tsconfig with the --ts-config option when using Madge @RoyalBingBong ?

[RoyalBingBong]

Oh, I did not.
I tried again with the example from above and a default tsconfig.json from tsc --init and the circular dependency detection seems to work just fine.