dev: run cargo clippy

Author: sandydoo

devenv-eval-cache/src/lib.rs

@@ -42,12 +42,10 @@ mod integration_tests {
 
     fn get_nix_binary() -> Result<String, Box<dyn std::error::Error>> {
         match env::var("DEVENV_NIX") {
-            Ok(path) => Ok(format!("{}/bin/nix", path)),
-            Err(_) => Err(format!(
-                "DEVENV_NIX environment variable not set. \
+            Ok(path) => Ok(format!("{path}/bin/nix")),
+            Err(_) => Err("DEVENV_NIX environment variable not set. \
                 Please set DEVENV_NIX to point to the store path of the custom Nix build. \
-                Example: DEVENV_NIX=/nix/store/...-nix-devenv-2.30.0... cargo test --features integration-tests"
-            ).into())
+                Example: DEVENV_NIX=/nix/store/...-nix-devenv-2.30.0... cargo test --features integration-tests".to_string().into())
         }
     }
 
@@ -77,7 +75,7 @@ mod integration_tests {
         let nix_binary = get_nix_binary()?;
         let cached_cmd = CachedCommand::new(pool);
         let mut cmd = Command::new(nix_binary);
-        cmd.args(&["eval", "--impure", "--expr", expr]);
+        cmd.args(["eval", "--impure", "--expr", expr]);
 
         Ok(cached_cmd.output(&mut cmd).await?)
     }
@@ -209,7 +207,7 @@ mod integration_tests {
             env::set_var(test_env_var, test_env_value);
         }
 
-        let nix_expr = format!(r#"builtins.getEnv "{}""#, test_env_var);
+        let nix_expr = format!(r#"builtins.getEnv "{test_env_var}""#);
 
         // Run nix eval with caching
         let output = run_nix_eval_cached(&pool, &nix_expr).await?;
@@ -329,7 +327,7 @@ mod integration_tests {
         pool: sqlx::SqlitePool,
     ) -> Result<(), Box<dyn std::error::Error>> {
         let test_env_var = "TEST_CACHE_INVALIDATION_VAR";
-        let nix_expr = format!(r#"builtins.getEnv "{}""#, test_env_var);
+        let nix_expr = format!(r#"builtins.getEnv "{test_env_var}""#);
 
         // Set initial value
         unsafe {

devenv/src/mcp.rs

@@ -462,7 +462,7 @@ mod tests {
         let packages = server.fetch_packages().await;
 
         // Should be able to fetch packages without error
-        assert!(packages.is_ok(), "Failed to fetch packages: {:?}", packages);
+        assert!(packages.is_ok(), "Failed to fetch packages: {packages:?}");
 
         let packages = packages.unwrap();
 
@@ -540,8 +540,7 @@ mod tests {
                 for known_option in known_options {
                     assert!(
                         options.iter().any(|opt| opt.name == known_option),
-                        "Expected option '{}' not found",
-                        known_option
+                        "Expected option '{known_option}' not found"
                     );
                 }
 
@@ -562,7 +561,7 @@ mod tests {
             }
             Err(e) => {
                 // Expected to fail in test environment
-                eprintln!("Expected failure in test environment: {:?}", e);
+                eprintln!("Expected failure in test environment: {e:?}");
                 eprintln!(
                     "This test requires running from a devenv project root with proper setup"
                 );

devenv/src/snix_backend.rs

@@ -3,13 +3,11 @@
 //! This module provides a Rust-native Nix evaluator backend using Snix
 //! as an alternative to the traditional C++ Nix binary.
 
-#![cfg(feature = "snix")]
-
 use crate::nix_backend::{DevenvPaths, NixBackend, Options};
 use crate::{cli, config};
 use async_trait::async_trait;
 use devenv_eval_cache::Output;
-use miette::{bail, Result};
+use miette::{Result, bail};
 use snix_build::buildservice::{BuildService, DummyBuildService};
 use snix_castore::blobservice::from_addr as blob_from_addr;
 use snix_castore::directoryservice::from_addr as directory_from_addr;
@@ -125,7 +123,9 @@ impl NixBackend for SnixBackend {
     async fn dev_env(&self, _json: bool, _gc_root: &Path) -> Result<Output> {
         // TODO: This is a complex operation that requires implementing the equivalent
         // of `nix print-dev-env`. For now, we'll return a placeholder error.
-        bail!("dev_env is not yet implemented for Snix backend. This requires implementing shell environment generation.")
+        bail!(
+            "dev_env is not yet implemented for Snix backend. This requires implementing shell environment generation."
+        )
     }
 
     async fn add_gc(&self, _name: &str, _path: &Path) -> Result<()> {
@@ -157,7 +157,7 @@ impl NixBackend for SnixBackend {
         } else {
             // Build an attribute path expression like ".#foo.bar"
             let attr_path = attributes.join(".");
-            format!("(import ./flake.nix).{}", attr_path)
+            format!("(import ./flake.nix).{attr_path}")
         };
 
         // For now, return a placeholder - proper implementation would need generator context