fix(remix-dev): Allow debugging for the remix shorthand (#3391)

* fix(remix-dev): Allow debugging for the remix shorthand

* Remove break for default statement

Co-authored-by: Michaël De Boey <info@michaeldeboey.be>

* refactor dev/default case shared code
and fix silly oversight

* Use TS shorthand

Co-authored-by: Pedro Cattori <pcattori@gmail.com>

Co-authored-by: Michaël De Boey <info@michaeldeboey.be>
Co-authored-by: Pedro Cattori <pcattori@gmail.com>

packages/remix-dev/cli/run.ts

@@ -126,6 +126,12 @@ const npxInterop = {
   pnpm: "pnpm exec",
 };
 
+async function dev(projectDir: string, flags: { debug?: boolean }) {
+  if (!process.env.NODE_ENV) process.env.NODE_ENV = "development";
+  if (flags.debug) inspector.open();
+  await commands.dev(projectDir, process.env.NODE_ENV);
+}
+
 /**
  * Programmatic interface for running the Remix CLI with the given command line
  * arguments.
@@ -426,13 +432,10 @@ export async function run(argv: string[] = process.argv.slice(2)) {
       break;
     }
     case "dev":
-      if (!process.env.NODE_ENV) process.env.NODE_ENV = "development";
-      if (flags.debug) inspector.open();
-      await commands.dev(input[1], process.env.NODE_ENV);
+      await dev(input[1], flags);
       break;
     default:
       // `remix ./my-project` is shorthand for `remix dev ./my-project`
-      if (!process.env.NODE_ENV) process.env.NODE_ENV = "development";
-      await commands.dev(input[0], process.env.NODE_ENV);
+      await dev(input[0], flags);
   }
 }