fix: use file url to import tsx at runtime

[mdonnalley]

• Use file url to import tsx at runtime
• debug errors caused by importing ts-node and/or tsx

QA

• create new ESM cli on windows: oclif generate my-cli --yes
• install tsx globally: npm install -g tsx
• add tsx to my-cli: npm install --save-dev tsx
• remove ts-node from my-cli: npm uninstall ts-node
• update shebang in dev.js to #!/usr/bin/env -S npx tsx --no-warnings
• update dev.cmd to use npx tsx instead of node --loader ts-node/esm --no-warnings=ExperimentalWarning
• delete the dist directory
• set DEBUG=*
• run bin\dev.cmd hello world

The command should successfully run and the logs should indicate that tsx did not fail to be imported:

  oclif:config:ts-path registering tsx at C:\Users\username\code\tsx-bug-esm +7ms
  oclif:config:ts-path tsx path: C:\Users\username\code\tsx-bug-esm\node_modules\tsx\dist\esm\api\index.cjs +8ms
  oclif:config:ts-path Successfully imported tsx +0ms

It's probably worth repeating these steps on mac or linux too

Fixes #1170
@W-16530475@

[acerbisgianluca]

Hey @mdonnalley, thanks for the quick fix!

One note, since we're on Windows, in point 5 of the QA steps the tester should change the command in ./bin/dev.cmd as well in order to run tsx instead of ts-node, right?

Btw I found out why it wasn't loading the commands from the src folder. I mistakenly removed rootDir from the tsconfig, so it wasn't looking for commands in the src folder.

[mdonnalley]

One note, since we're on Windows, in point 5 of the QA steps the tester should change the command in ./bin/dev.cmd as well in order to run tsx instead of ts-node, right?

Good catch. I've updated the description

[acerbisgianluca]

Tested following the QA steps and it loads tsx correctly now.

[cristiand391]

QA notes:

✅ macos: still works, can see tsx being registered in debug logs
also changed the apiPath on purpose and can see the error being logged

✅ windows

before

can't load tsx, throws Error: command hellow:world not found

after

ls dist shows no build, tsx is loaded and transpile command

Thanks @acerbisgianluca for the double check!