codeclimate-tslint icon indicating copy to clipboard operation
codeclimate-tslint copied to clipboard

fails to use tslint rules from sonarTS

Open abenhamdine opened this issue 7 years ago • 3 comments

When I try to use CodeClimate on ts files with SonarTS tslint plugin, I encounter the following error :

/usr/src/app/node_modules/tslint/lib/configuration.js:55
        throw new error_1.FatalError("Failed to load " + configPath + ": " + error.message, error);
        ^

FatalError: Failed to load /tmp/codeclimate-tslint/a550b73b242c7b7d3edc418468b8310b4f8de8536e008b0b6d93cb3c73e2a5c9.json: Invalid "extends" configuration value - could not require "tslint-sonarts". Review the Node lookup algorithm (https://nodejs.org/api/modules.html#modules_all_together) for the approximate method TSLint uses to find the referenced configuration file.
    at FatalError.Error (native)
    at new FatalError (/usr/src/app/node_modules/tslint/lib/error.js:27:28)
    at Object.findConfiguration (/usr/src/app/node_modules/tslint/lib/configuration.js:55:15)
    at new TsLinter (/usr/src/app/dist/tsLinter.js:33:57)
    at Object.<anonymous> (/usr/src/app/dist/index.js:26:18)
    at Module._compile (module.js:577:32)
    at Object.Module._extensions..js (module.js:586:10)
    at Module.load (module.js:494:32)
    at tryModuleLoad (module.js:453:12)
    at Function.Module._load (module.js:445:3)

According to your support :

Dave Henton (Code Climate)

Jun 1, 12:03 PM EDT

Hey Arnaud, Thanks for writing in, and sorry about the trouble with this. I believe what's happening here is related to us not installing a repo's node_modules when running our plugins. For other repos where I've seen this pop up: when you run npm install locally to get a local node_modules directory containing the library (and then use our CLI to analyze your repo locally), you won't encounter this issue. The catch with that is we haven't figured out a good way to install the node_modules that are specified in a project that doesn't break aspects of the limits we place on engines for security reasons, and so it's going to take a bit more work on our side to map that work out and get it moving. Really sorry about the inconvenience with this in the meantime. I'd encourage you to reach out to the devs over at the Github repo for this engine. It looks like there are some dependencies they've included in the engine in order to support certain rules on Code Climate, so perhaps you could open a PR to get support for those modules you're using. In the meantime, you should be able to disable those rules which rely on NPM modules, and then see a successful analysis. Thanks, Dave Dave Henton, Support Specialist / Stay tuned to our Changelog for all the latest news: https://codeclimate.com/changelog and https://codeclimate.com/changelog.rss

Could you guys do something to solve that ? Sonar linters are very powerful profesionnal analysis tools, it would be convenient to be able to run it with CodeClimate.

FWIW, I also opened an issue in SonarTS repo : https://github.com/SonarSource/SonarTS/issues/653

abenhamdine avatar Jun 06 '18 12:06 abenhamdine

I am running into the same issue.

rafaelrinaldi avatar Oct 08 '18 18:10 rafaelrinaldi

Same issue for me as well.

koladilip avatar Nov 29 '18 06:11 koladilip

Same here

k3nsei avatar Dec 15 '18 17:12 k3nsei