Add logging when new sourcedb is loaded

Reviewed By: kinto0

Differential Revision: D92773270

fbshipit-source-id: 9c004d541daec54fbd5e91fe318722186e9633b1

Author: connernilsen

crates/pyrefly_build/src/lib.rs

@@ -25,6 +25,7 @@
 #![feature(const_type_name)]
 #![feature(if_let_guard)]
 
+use std::fmt::Display;
 use std::path::Path;
 use std::path::PathBuf;
 use std::sync::Arc;
@@ -42,6 +43,7 @@ pub mod source_db;
 pub use source_db::SourceDatabase;
 use starlark_map::small_map::SmallMap;
 mod query;
+use tracing::info;
 #[cfg(not(target_arch = "wasm32"))]
 use which::which;
 
@@ -92,6 +94,15 @@ impl BuildSystemArgs {
     }
 }
 
+impl Display for BuildSystemArgs {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        match self {
+            Self::Buck(args) => write!(f, "Buck({})", args),
+            Self::Custom(args) => write!(f, "Custom({})", args),
+        }
+    }
+}
+
 #[derive(Debug, Clone, Deserialize, Serialize, PartialEq, Eq)]
 #[serde(rename_all = "kebab-case")]
 pub struct BuildSystem {
@@ -147,6 +158,12 @@ impl BuildSystem {
             return Some(Ok(result.dupe()));
         }
 
+        info!(
+            "Loading new build system at {}: {}",
+            config_root.display(),
+            &self.args
+        );
+
         for path in &mut self.search_path_prefix {
             *path = config_root.join(&path);
         }

crates/pyrefly_build/src/query/buck.rs

@@ -6,6 +6,7 @@
  */
 
 use std::fmt::Debug;
+use std::fmt::Display;
 use std::path::Path;
 use std::path::PathBuf;
 use std::process::Command;
@@ -77,6 +78,16 @@ impl BxlArgs {
     }
 }
 
+impl Display for BxlArgs {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        write!(
+            f,
+            "isolation_dir: {:?}, extras: {:?}",
+            self.isolation_dir, self.extras
+        )
+    }
+}
+
 #[derive(Debug)]
 pub struct BxlQuerier(BxlArgs);
 

crates/pyrefly_build/src/query/custom.rs

@@ -6,6 +6,7 @@
  */
 
 use std::fmt::Debug;
+use std::fmt::Display;
 use std::path::Path;
 use std::path::PathBuf;
 use std::process::Command;
@@ -45,6 +46,16 @@ impl CustomQueryArgs {
     }
 }
 
+impl Display for CustomQueryArgs {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        write!(
+            f,
+            "command: {:?}, repo_root: {:?}",
+            self.command, self.repo_root
+        )
+    }
+}
+
 /// A querier allowing for a custom command when querying and constructing source DB.
 #[derive(Debug)]
 pub struct CustomQuerier(CustomQueryArgs);

test/basic.md

@@ -20,6 +20,7 @@ $ echo -e "from typing import reveal_type\nreveal_type(1)" > $TMPDIR/empty.py &&
 
 ```scrut {output_stream: stderr}
 $ $PYREFLY check $TEST_PY
+ INFO Loading new build system at * (glob?)
  INFO Querying Buck for source DB (glob?)
  INFO Source DB build ID: * (glob?)
  INFO Finished querying Buck for source DB (glob?)