feat(config): replace tsx with node@>=22.6 type stripping

Author: JamieMason

package.json

@@ -39,9 +39,6 @@
     "Stuart Knightley (https://github.com/Stuk)",
     "Tom Fletcher (https://github.com/tom-fletcher)"
   ],
-  "dependencies": {
-    "tsx": "^4.21.0"
-  },
   "devDependencies": {
     "@biomejs/biome": "^2.3.11",
     "@release-it/conventional-changelog": "^10.0.4",

pnpm-lock.yaml

@@ -16,22 +16,24 @@ pub enum NodeJsResult {
 
 #[derive(Debug, Error)]
 pub enum RcfileError {
-  #[error("Failed to read config file: {0}")]
+  #[error("Failed to read config file:\n\n{0}")]
   FileReadFailed(#[from] std::io::Error),
-  #[error("Failed to run node to retrieve JS/TS config file: {0}")]
+  #[error("Failed to run node to retrieve JS/TS config file:\n\n{0}")]
   NodeJsExecutionFailed(#[source] std::io::Error),
-  #[error("Executing a JavaScript config file failed with stderr: {stderr}")]
+  #[error("Node.js v22.6.0 or higher is needed for a TypeScript config file, try a JavaScript or JSON config file:\n\n{stderr}")]
+  NodeJsCannotStripTypes { stderr: String },
+  #[error("Executing a JavaScript config file failed with stderr:\n\n{stderr}")]
   ProcessFailed { stderr: String },
-  #[error("Config file contains invalid UTF-8: {0}")]
+  #[error("Config file contains invalid UTF-8:\n\n{0}")]
   InvalidUtf8(#[from] std::string::FromUtf8Error),
-  #[error("Config file failed validation: {0}")]
+  #[error("Config file failed validation:\n\n{0}")]
   InvalidConfig(#[from] serde_json::Error),
-  #[error("Failed to import or require config file: {import_error} {require_error}")]
+  #[error("Node.js threw when trying to import() your config file:\n\n{import_error}\n\n{require_error}")]
   JavaScriptImportFailed { import_error: String, require_error: String },
-  #[error("Failed to parse JSON in config file: {0}")]
+  #[error("Failed to parse JSON in config file:\n\n{0}")]
   JsonParseFailed(#[source] serde_json::Error),
-  #[error("Failed to parse YAML in config file: {0}")]
+  #[error("Failed to parse YAML in config file:\n\n{0}")]
   YamlParseFailed(#[from] serde_yaml::Error),
-  #[error("Config defined as a property in package.json failed validation: {0}")]
+  #[error("Config defined as a property in package.json failed validation:\n\n{0}")]
   PackageJsonConfigInvalid(#[source] serde_json::Error),
 }

src/rcfile/error.rs

@@ -14,9 +14,7 @@ pub fn from_javascript_path(file_path: &Path) -> Result<Rcfile, RcfileError> {
   let escaped_file_path_for_nodejs = file_path.to_string_lossy().replace('\\', "\\\\");
   let nodejs_script = format!(
     r#"
-    import('tsx')
-      .catch(() => null)
-      .then(() => import('{escaped_file_path_for_nodejs}'))
+    import('{escaped_file_path_for_nodejs}')
       .then(findConfig)
       .then((value) => {{
         if (isNonEmptyObject(value)) {{
@@ -71,18 +69,31 @@ pub fn from_javascript_path(file_path: &Path) -> Result<Rcfile, RcfileError> {
     "#
   );
 
+  let is_typescript = file_path.to_string_lossy().ends_with("ts");
+  let mut args = vec![];
+
+  if is_typescript {
+    args.push("--experimental-strip-types");
+  }
+
+  args.push("--eval");
+  args.push(&nodejs_script);
+
   Command::new("node")
-    .args(vec!["-e", &nodejs_script])
+    .args(args)
     .current_dir(file_path.parent().unwrap_or_else(|| Path::new(".")))
     .output()
     .map_err(RcfileError::NodeJsExecutionFailed)
     .and_then(|output| {
       if output.status.success() {
         Ok(output.stdout)
       } else {
-        Err(RcfileError::ProcessFailed {
-          stderr: String::from_utf8_lossy(&output.stderr).to_string(),
-        })
+        let stderr = String::from_utf8_lossy(&output.stderr).to_string();
+        if stderr.contains("experimental-strip-types") {
+          Err(RcfileError::NodeJsCannotStripTypes { stderr })
+        } else {
+          Err(RcfileError::ProcessFailed { stderr })
+        }
       }
     })
     .and_then(|stdout| String::from_utf8(stdout).map_err(RcfileError::InvalidUtf8))