Hi @Benjamin-Lee, thanks for reporting this! I haven't reproduced this, but can totally see why it happens.

The issue is because we're using require.resolve in this line here: https://github.com/redwoodjs/redwood/blob/main/packages/cli/src/commands/execHandler.js#L103 - which would pick up the json file ofcourse. Something similar probably happens when trying to actually execute: https://github.com/redwoodjs/redwood/blob/main/packages/cli/src/lib/exec.js#L14

Off the top of my head, I can't think of a clean way to make it ignore the other files, but happy to hear suggestions before you start!

Things to note:

• we need to support .js, .jsx, .tsand.tsx` I think
• you can totally avoid using require.resolve

Hey @Benjamin-Lee have you had a chance to look at a fix for this yet? Are you still interested? No pressure, just wanted an update 🙂 Have a great day!

Hi @Tobbe and @Benjamin-Lee, I'd like to pick this up if it's alright with you? This would be my first time contributing so I may need some guidance.

It seems like there are two problems/requirements:

1. The handler needs to ensure there aren't ambiguous scripts defined such as [foo.js, foo.ts]
2. If someone wants to include other files like foo.json or foo.whatever, it should have no impact on their script named foo.{js,jsx,ts,tsx}.

For the first, the execHandler is currently using require.resolve to check that the script actually exists before trying to run it. We can use the file system to do this instead. Something like this:

  // - execHandler.js
  const tgtScriptName = path.parse(scriptPath)?.name
  const tgtMatch = findScripts()
    .map((p) => path.parse(p)?.name)
    .filter((n) => n == tgtScriptName)

With this, we'd expect tgtMatch to contain exactly one matching script with the extension *.{js,jsx,ts,tsx}. If it has zero, or more than 1, then it's a problem.

For the second requirement, we can use a similar approach to first resolve the full script path including file extension, and pass this in to the call to require. That will prevent require from accidentally loading foo.json when it should have loaded foo.js. Something along these lines:

// - files.ts
export const resolveJSPath = (pathNoExt: string) => {
  const files = fg.sync(`${pathNoExt}.{js,jsx,ts,tsx}`, {
    absolute: true,
    ignore: ['node_modules'],
  })
  if (files.length == 1) {
    return files[0]
  }
  return false
}

// - exec.js
const script = require(resolveJSPath(scriptPath))

Questions:

1. Does this approach seem directionally on track?
2. Any gotchas or special cases I should be on the lookout for?