Invalid loader value: "cjs"

[Spotchi]

I am using Lambda Live Debugger for the first time with CDK.
I am using a Slack client in one of my lambdas, with axios as a transitive dependency.

Here's the offending dependency :

"@slack/web-api": "^7.8.0",

The LLD output

Starting the debugger (ID 10a57bf796ffb60a02253f2a2a376d72)...
Changing current folder to /Users/quentinlaurent/Desktop/app/cdk
Getting resources with 'cdk' framework
[CDK] Entry file: /Users/quentinlaurent/Desktop/app/cdk/bin/cdk.ts
[CDK] Using ESM format
✘ [ERROR] Invalid loader value: "cjs" [plugin injectCode]

    node_modules/@slack/web-api/dist/WebClient.js:72:40:
      72 │ const axios_1 = __importDefault(require("axios"));
         ╵                                         ~~~~~~~

Error: Error building CDK code: Build failed with 1 error:
node_modules/@slack/web-api/dist/WebClient.js:72:40: ERROR: [plugin: injectCode] Invalid loader value: "cjs"
    at CdkFramework.getLambdasDataFromCdkByCompilingAndRunning (file:///Users/quentinlaurent/Desktop/qualia/lambda-live-debugger/dist/frameworks/cdkFramework.mjs:288:19)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async CdkFramework.getLambdas (file:///Users/quentinlaurent/Desktop/qualia/lambda-live-debugger/dist/frameworks/cdkFramework.mjs:51:30)
    at async Object.getLambdas (file:///Users/quentinlaurent/Desktop/qualia/lambda-live-debugger/dist/resourceDiscovery.mjs:43:21)
    at async Object.discoverLambdas (file:///Users/quentinlaurent/Desktop/qualia/lambda-live-debugger/dist/configuration.mjs:115:26)
    at async run (file:///Users/quentinlaurent/Desktop/qualia/lambda-live-debugger/dist/lldebugger.mjs:54:5) {
  [cause]: Error: Build failed with 1 error:
  node_modules/@slack/web-api/dist/WebClient.js:72:40: ERROR: [plugin: injectCode] Invalid loader value: "cjs"
      at failureErrorWithLog (/Users/quentinlaurent/Desktop/qualia/lambda-live-debugger/node_modules/esbuild/lib/main.js:1651:15)
      at /Users/quentinlaurent/Desktop/qualia/lambda-live-debugger/node_modules/esbuild/lib/main.js:1059:25
      at runOnEndCallbacks (/Users/quentinlaurent/Desktop/qualia/lambda-live-debugger/node_modules/esbuild/lib/main.js:1486:45)
      at buildResponseToResult (/Users/quentinlaurent/Desktop/qualia/lambda-live-debugger/node_modules/esbuild/lib/main.js:1057:7)
      at /Users/quentinlaurent/Desktop/qualia/lambda-live-debugger/node_modules/esbuild/lib/main.js:1086:16
      at responseCallbacks.<computed> (/Users/quentinlaurent/Desktop/qualia/lambda-live-debugger/node_modules/esbuild/lib/main.js:704:9)
      at handleIncomingPacket (/Users/quentinlaurent/Desktop/qualia/lambda-live-debugger/node_modules/esbuild/lib/main.js:764:9)
      at Socket.readFromStdout (/Users/quentinlaurent/Desktop/qualia/lambda-live-debugger/node_modules/esbuild/lib/main.js:680:7)
      at Socket.emit (node:events:517:28)
      at addChunk (node:internal/streams/readable:368:12) {
    errors: [Getter/Setter],
    warnings: [Getter/Setter]
  }
}

I did find a similar issue in this repo with mjs files : #78

It seems to me that a similar fix could be applied for cjs files?

[Spotchi]

I made a small PR for this, which fixes my problem, though did not have time to test it extensively
#90

[ServerlessLife]

Thank you very much, @Spotchi, for reporting the problem and for already preparing the PR.

I will look into this later today or over the weekend.

I am pretty sure the PR will not solve the issue. I assume the problem is a combination of tsconfig.json, package.json, and the @slack/web-api module. Is there any chance that you could prepare a sample project or at least share tsconfig.json

[Spotchi]

Thank you, I will try to put a sample project together next week, but note that this did fix the issue for me and I was able to run Lld with that fix

[ServerlessLife]

Thank you, @Spotchi, for reporting the bug and fixing it. 🙌

I reproduced the issue by just importing the @slack/web-api. I will deploy the new version as soon as all the tests run.

It's a bit confusing to me why you import @slack/web-api in CDK code. That should be imported into Lambda 🤔. But the bug is clear. It could also happen with any other library that uses js files.

I really appreciate this. I added you to the list of contributors. If you would like me to write your full name or a different link, please let me know what to do.

[ServerlessLife]

🎉 This issue has been resolved in version 1.4.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[Spotchi]

Thank you, @Spotchi, for reporting the bug and fixing it. 🙌

Thank you for making this tool! I appreciate the quick reaction and PR review too!

It's a bit confusing to me why you import @slack/web-api in CDK code. That should be imported into Lambda 🤔. But the bug is clear. It could also happen with any other library that uses js files.
Well,
I actually don't import the @slack/web-api in CDK code, only in lambda code and shared common code. As far as I understand you are building lambdas with esbuild with the injectCodePlugin for each lambda, this is when this error happens, so I think the bug can arise due to lambda dependencies too I think

I really appreciate this. I added you to the list of contributors. If you would like me to write your full name or a different link, please let me know what to do.
Full name is Quentin Laurent, link to github profile is good thanks!

[ServerlessLife]

I update your name on the website. On npm, it will updated with the next release, which will be next week, because I am preparing some new features.