[CY-6271] Support .cjs extension about codacy-plugins-api

[github-actions]

Internal ticket created : CY-6271

[jhildenbiddle]

Ping @franciscodua since this seems very similar to #61. Thanks!

[stefanvacareanu7]

Hi there,

Thanks for reaching out!

You can use the file extensions feature to configure the accepted extensions for Javascript language.

Let me know if this helps.

Stefan

[jhildenbiddle]

Thanks @stefanvacareanu7.

TLDR: PMD breaks with .cjs files.

Details...

The project I was working on is now live, so for context:

• Project: https://github.com/jhildenbiddle/mergician
• Codacy Dashboard: https://app.codacy.com/gh/jhildenbiddle/mergician/

The tooltip on the web UI states that only .js, .jsx, .jsm, .vue, and .mjs are supported:

Without adding the extension, the log for the latest commit analysis shows the following error:

If I add .cjs to the file extensions section as instructed, the logs show a different error:

Error on file src/index.cjs:1. Cause: PMDException: Error while parsing /src/src/index.cjs
Error on file src/util.cjs:1. Cause: PMDException: Error while parsing /src/src/util.cjs
Error on file tests/index.test.js:1. Cause: PMDException: Error while processing /src/tests/index.test.js

Since PMD is failing, I disabled this tool and all tests now pass as expected. I can live without PMD, but it would be nice if this tool worked as well since I'm using it to analyze other repos.

Also, while I'm happy to find a workaround for analyzing .cjs files with Codacy, both .mjs and .cjs are widely-used extensions within the Node ecosystem so supporting one by default without the other feels like a miss. Just my $0.02.

[DMarinhoCodacy]

Hello,

We already bumped PMD to the latest stable version and we tested already .cjs files with success.

Feel free to make more tests and let us know.

Kind regards,
David Marinho