fix: tsconfig project reference it self should throw error (#211)

---------

Co-authored-by: Boshen <[email protected]>

Author: jerrykingxyz

src/error.rs

@@ -28,6 +28,10 @@ pub enum ResolveError {
     #[error("Tsconfig not found {0}")]
     TsconfigNotFound(PathBuf),
 
+    /// Tsconfig's project reference path points to it self
+    #[error("Tsconfig's project reference path points to this tsconfig {0}")]
+    TsconfigSelfReference(PathBuf),
+
     #[error("{0}")]
     IOError(IOError),
 

src/lib.rs

@@ -1099,7 +1099,14 @@ impl<Fs: FileSystem> ResolverGeneric<Fs> {
                     let tsconfig = self.cache.tsconfig(
                         /* root */ true,
                         &reference_tsconfig_path,
-                        |_| Ok(()),
+                        |reference_tsconfig| {
+                            if reference_tsconfig.path == tsconfig.path {
+                                return Err(ResolveError::TsconfigSelfReference(
+                                    reference_tsconfig.path.clone(),
+                                ));
+                            }
+                            Ok(())
+                        },
                     )?;
                     reference.tsconfig.replace(tsconfig);
                 }

src/tests/tsconfig_project_references.rs

@@ -99,3 +99,34 @@ fn manual() {
         assert_eq!(resolved_path, expected, "{request} {path:?}");
     }
 }
+
+#[test]
+fn self_reference() {
+    let f = super::fixture_root().join("tsconfig/cases/project_references");
+
+    #[rustfmt::skip]
+    let pass = [
+        (f.join("app"), vec!["./tsconfig.json".into()]),
+        (f.join("app/tsconfig.json"), vec!["./tsconfig.json".into()]),
+        (f.join("app"), vec![f.join("app")]),
+        (f.join("app/tsconfig.json"), vec![f.join("app")]),
+        (f.join("app/tsconfig.json"), vec![f.join("project_b"), f.join("app")]),
+    ];
+
+    for (config_file, reference_paths) in pass {
+        let resolver = Resolver::new(ResolveOptions {
+            tsconfig: Some(TsconfigOptions {
+                config_file: config_file.clone(),
+                references: TsconfigReferences::Paths(reference_paths.clone()),
+            }),
+            ..ResolveOptions::default()
+        });
+        let path = f.join("app");
+        let resolved_path = resolver.resolve(&path, "@/index.ts").map(|f| f.full_path());
+        assert_eq!(
+            resolved_path,
+            Err(ResolveError::TsconfigSelfReference(f.join("app/tsconfig.json"))),
+            "{config_file:?} {reference_paths:?}"
+        );
+    }
+}

src/tsconfig.rs

@@ -29,7 +29,7 @@ pub struct TsConfig {
 
     /// Path to `tsconfig.json`. Contains the `tsconfig.json` filename.
     #[serde(skip)]
-    path: PathBuf,
+    pub(crate) path: PathBuf,
 
     #[serde(default)]
     pub extends: Option<ExtendsField>,