Breaking change after version 2.0.1 with Node v0.6 about typechecker HOT 6 CLOSED

The problem is with cyclic.js and the existsSync function on line 1. Which is not available before node v0.8 (was path.existsSync)

The Watchr library is depending on "typechecker": "~2.0.1", which is now broken on node v0.6.

The change to fs.existsSync is breaking other dependencies at the moment.

Here is a solution for making cyclic.js work again with older versions of node.

try {
    if (require('fs').lstatSync('.git')) {
        require('child_process').spawn(
            'npm',
            ['install', '--force', require('./package.json').name],
            {env: process.env, cwd: process.cwd(), stdio: 'inherit'}
        );
    }
} catch (err) {
    return err;
}

So the problem can be solved by using lstatSync with a try/catch around it. Easy!

I tried path.existsSync as well, but node > 0.6 prints the following warning. (not recommend)

path.existsSync is now called `fs.existsSync`.

Any chance you can fix this minor compatibility issue? Thanks!

from typechecker.

+1. Facing the same issue. Would be good to get this compatibility fixed.

from typechecker.

What is the purpose of this file cyclic.js, and why has it found its way into so many projects?

from typechecker.

@amb26 if project A has a development dependency called project B that has a dependency of project A, then that creates a cyclic dependency, which npm will error about, cyclic.js fixes that error.

I'm unsure if newer versions of npm still have this issue.

from typechecker.

cyclic.js is maintained by the bevry/base project, https://github.com/bevry/base/blob/master/cyclic.js

TypeChecker will be updated in coming months when it is converted along with our over projects to ES6, and will get this update - details. As it seems this update is only for Node 0.6 from 2013, I don't think this of any urgency.

from typechecker.

Removed cyclic.js in v2.1.0, seems npm now handles cyclic dependencies correctly.

from typechecker.