feat(remix-dev): add suppport for .mjs and .cjs configs (#3675)

remix-run · Aug 11, 2022 · 6c19f72 · 6c19f72
1 parent e1ec2b5
commit 6c19f72
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 9 deletions.
diff --git a/.changeset/afraid-peaches-accept.md b/.changeset/afraid-peaches-accept.md
@@ -0,0 +1,5 @@
+---
+"@remix-run/dev": patch
+---
+
+Add support for .mjs and .cjs remix.config files.
diff --git a/packages/remix-dev/config.ts b/packages/remix-dev/config.ts
@@ -287,15 +287,18 @@ export async function readConfig(
   }
 
   let rootDirectory = path.resolve(remixRoot);
-  let configFile = path.resolve(rootDirectory, "remix.config.js");
-
-  let appConfig: AppConfig;
-  try {
-    appConfig = require(configFile);
-  } catch (error) {
-    throw new Error(
-      `Error loading Remix config in ${configFile}\n${String(error)}`
-    );
+  let configFile = findConfig(rootDirectory, "remix.config");
+
+  let appConfig: AppConfig = {};
+  if (configFile) {
+    try {
+      let appConfigModule = await import(configFile);
+      appConfig = appConfigModule?.default || appConfig;
+    } catch (error) {
+      throw new Error(
+        `Error loading Remix config at ${configFile}\n${String(error)}`
+      );
+    }
   }
 
   let customServerEntryPoint = appConfig.server;
@@ -475,3 +478,14 @@ function findEntry(dir: string, basename: string): string | undefined {
 
   return undefined;
 }
+
+const configExts = [".js", ".cjs", ".mjs"];
+
+function findConfig(dir: string, basename: string): string | undefined {
+  for (let ext of configExts) {
+    let file = path.resolve(dir, basename + ext);
+    if (fse.existsSync(file)) return file;
+  }
+
+  return undefined;
+}