More PR feedback

crates/turborepo-lib/src/boundaries.rs

@@ -184,7 +184,6 @@ impl Run {
         // We assume the tsconfig.json is at the root of the package
         let tsconfig_path = package_root.join_component("tsconfig.json");
 
-        // TODO: Load tsconfig.json
         let resolver =
             Tracer::create_resolver(tsconfig_path.exists().then(|| tsconfig_path.as_ref()));
 
@@ -296,9 +295,9 @@ impl Run {
         }
         // We use `relation_to_path` and not `contains` because `contains`
         // panics on invalid paths with too many `..` components
-        if matches!(
+        if !matches!(
             package_path.relation_to_path(&resolved_import_path),
-            PathRelation::Divergent | PathRelation::Child
+            PathRelation::Parent
         ) {
             Ok(Some(BoundariesDiagnostic::ImportLeavesPackage {
                 import: import.to_string(),

crates/turborepo-lib/src/query/mod.rs

@@ -1,6 +1,5 @@
-
-mod external_package;
 mod boundaries;
+mod external_package;
 mod file;
 mod package;
 mod server;
@@ -15,7 +14,6 @@ use std::{
 use async_graphql::{http::GraphiQLSource, *};
 use axum::{response, response::IntoResponse};
 use external_package::ExternalPackage;
-use miette::Diagnostic;
 use package::Package;
 pub use server::run_server;
 use thiserror::Error;
@@ -153,7 +151,6 @@ impl RepositoryQuery {
 #[graphql(concrete(name = "Packages", params(Package)))]
 #[graphql(concrete(name = "ChangedPackages", params(ChangedPackage)))]
 #[graphql(concrete(name = "Files", params(File)))]
-#[graphql(concrete(name = "TraceErrors", params(file::TraceError)))]
 #[graphql(concrete(name = "ExternalPackages", params(ExternalPackage)))]
 #[graphql(concrete(name = "Diagnostics", params(Diagnostic)))]
 pub struct Array<T: OutputType> {