Fix for entry point detection on windows

Author: ericallam

packages/cli-v3/src/build/packageModules.ts

@@ -1,7 +1,6 @@
+import { BuildTarget } from "@trigger.dev/core/v3";
 import { join } from "node:path";
 import { sourceDir } from "../sourceDir.js";
-import { BuildTarget } from "@trigger.dev/core/v3";
-import { logger } from "../utilities/logger.js";
 
 export const devRunWorker = join(sourceDir, "entryPoints", "dev-run-worker.js");
 export const devIndexWorker = join(sourceDir, "entryPoints", "dev-index-worker.js");
@@ -26,54 +25,58 @@ export const esmShimPath = join(sourceDir, "shims", "esm.js");
 
 export const shims = [esmShimPath];
 
+// IMPORTANT: this may look like it should not work on Windows, but it does (and changing to using path.join will break stuff)
 function isDevRunWorker(entryPoint: string) {
   return (
-    entryPoint.includes(join("dist", "esm", "entryPoints", "dev-run-worker.js")) ||
-    entryPoint.includes(join("src", "entryPoints", "dev-run-worker.ts"))
+    entryPoint.includes("dist/esm/entryPoints/dev-run-worker.js") ||
+    entryPoint.includes("src/entryPoints/dev-run-worker.ts")
   );
 }
 
+// IMPORTANT: this may look like it should not work on Windows, but it does (and changing to using path.join will break stuff)
 function isDevIndexWorker(entryPoint: string) {
   return (
-    entryPoint.includes(join("dist", "esm", "entryPoints", "dev-index-worker.js")) ||
-    entryPoint.includes(join("src", "entryPoints", "dev-index-worker.ts"))
+    entryPoint.includes("dist/esm/entryPoints/dev-index-worker.js") ||
+    entryPoint.includes("src/entryPoints/dev-index-worker.ts")
   );
 }
 
+// IMPORTANT: this may look like it should not work on Windows, but it does (and changing to using path.join will break stuff)
 function isDeployIndexController(entryPoint: string) {
   return (
-    entryPoint.includes(join("dist", "esm", "entryPoints", "deploy-index-controller.js")) ||
-    entryPoint.includes(join("src", "entryPoints", "deploy-index-controller.ts"))
+    entryPoint.includes("dist/esm/entryPoints/deploy-index-controller.js") ||
+    entryPoint.includes("src/entryPoints/deploy-index-controller.ts")
   );
 }
 
+// IMPORTANT: this may look like it should not work on Windows, but it does (and changing to using path.join will break stuff)
 function isDeployIndexWorker(entryPoint: string) {
   return (
-    entryPoint.includes(join("dist", "esm", "entryPoints", "deploy-index-worker.js")) ||
-    entryPoint.includes(join("src", "entryPoints", "deploy-index-worker.ts"))
+    entryPoint.includes("dist/esm/entryPoints/deploy-index-worker.js") ||
+    entryPoint.includes("src/entryPoints/deploy-index-worker.ts")
   );
 }
 
 function isDeployRunController(entryPoint: string) {
   return (
-    entryPoint.includes(join("dist", "esm", "entryPoints", "deploy-run-controller.js")) ||
-    entryPoint.includes(join("src", "entryPoints", "deploy-run-controller.ts"))
+    entryPoint.includes("dist/esm/entryPoints/deploy-run-controller.js") ||
+    entryPoint.includes("src/entryPoints/deploy-run-controller.ts")
   );
 }
 
+// IMPORTANT: this may look like it should not work on Windows, but it does (and changing to using path.join will break stuff)
 function isDeployRunWorker(entryPoint: string) {
   return (
-    entryPoint.includes(join("dist", "esm", "entryPoints", "deploy-run-worker.js")) ||
-    entryPoint.includes(join("src", "entryPoints", "deploy-run-worker.ts"))
+    entryPoint.includes("dist/esm/entryPoints/deploy-run-worker.js") ||
+    entryPoint.includes("src/entryPoints/deploy-run-worker.ts")
   );
 }
 
+// IMPORTANT: this may look like it should not work on Windows, but it does (and changing to using path.join will break stuff)
 export function isLoaderEntryPoint(entryPoint: string) {
-  logger.debug("isLoaderEntryPoint", entryPoint, join("src", "entryPoints", "loader.ts"));
-
   return (
-    entryPoint.includes(join("dist", "esm", "entryPoints", "loader.js")) ||
-    entryPoint.includes(join("src", "entryPoints", "loader.ts"))
+    entryPoint.includes("dist/esm/entryPoints/loader.js") ||
+    entryPoint.includes("src/entryPoints/loader.ts")
   );
 }