simplify public API snapshots (#1114)

- omit auto trait impls.
- omit blanket impls.

crates/infra/utils/src/cargo/manifest.rs

@@ -10,18 +10,18 @@ use crate::paths::PathExtensions;
 
 #[derive(Deserialize)]
 pub struct WorkspaceManifest {
-    pub workspace: Workspace,
+    workspace: Workspace,
 }
 
 #[derive(Deserialize)]
-pub struct Workspace {
-    pub package: WorkspacePackage,
-    pub dependencies: HashMap<String, Dependency>,
+struct Workspace {
+    package: WorkspacePackage,
+    dependencies: HashMap<String, Dependency>,
 }
 
 #[derive(Deserialize)]
-pub struct WorkspacePackage {
-    pub version: Version,
+struct WorkspacePackage {
+    version: Version,
 }
 
 #[derive(Deserialize)]
@@ -40,4 +40,17 @@ impl WorkspaceManifest {
         toml::from_str(&manifest_path.read_to_string()?)
             .with_context(|| format!("Failed to deserialize manifest: {manifest_path:?}"))
     }
+
+    pub fn version(&self) -> &Version {
+        &self.workspace.package.version
+    }
+
+    pub fn dependency(&self, dependency_name: impl AsRef<str>) -> Result<&Dependency> {
+        let dependency_name = dependency_name.as_ref();
+
+        self.workspace
+            .dependencies
+            .get(dependency_name)
+            .with_context(|| format!("Cannot find dependency '{dependency_name}' in workspace."))
+    }
 }

crates/infra/utils/src/cargo/public_api.rs

@@ -37,7 +37,11 @@ mod public_api_snapshots {
             .toolchain(env!("RUST_NIGHTLY_VERSION"))
             .build()?;
 
-        let public_api = public_api::Builder::from_rustdoc_json(rustdoc_json).build()?;
+        let public_api = public_api::Builder::from_rustdoc_json(rustdoc_json)
+            .omit_auto_derived_impls(false)
+            .omit_auto_trait_impls(true)
+            .omit_blanket_impls(true)
+            .build()?;
 
         let output_path = crate_dir.join("generated/public_api.txt");
 

crates/infra/utils/src/cargo/workspace.rs

@@ -18,16 +18,12 @@ impl CargoWorkspace {
 
         let manifest = WorkspaceManifest::load()?;
 
-        let dependency = manifest
-            .workspace
-            .dependencies
-            .get(crate_name)
-            .with_context(|| format!("Cannot find dependency '{crate_name}' in workspace."))?;
-
         let mut command = Command::new("cargo")
             .args(["install", crate_name])
             .flag("--locked");
 
+        let dependency = manifest.dependency(crate_name)?;
+
         command = match dependency {
             Dependency::Local { path, version } => command
                 .property("--path", path.unwrap_str())
@@ -54,15 +50,8 @@ impl CargoWorkspace {
     }
 
     pub fn locate_source_crate(crate_name: impl AsRef<str>) -> Result<PathBuf> {
-        let crate_name = crate_name.as_ref();
-
         let manifest = WorkspaceManifest::load()?;
-
-        let dependency = manifest
-            .workspace
-            .dependencies
-            .get(crate_name)
-            .with_context(|| format!("Cannot find dependency '{crate_name}' in workspace."))?;
+        let dependency = manifest.dependency(crate_name)?;
 
         match dependency {
             Dependency::Local { path, version: _ } => Ok(Path::repo_path(path)),
@@ -76,7 +65,7 @@ impl CargoWorkspace {
     }
 
     pub fn local_version() -> Result<Version> {
-        Ok(WorkspaceManifest::load()?.workspace.package.version)
+        Ok(WorkspaceManifest::load()?.version().to_owned())
     }
 
     pub fn published_version(crate_name: impl AsRef<str>) -> Result<Version> {