chore: resolve clippy warnings and add stricter CI check

.github/workflows/test.yml

@@ -145,6 +145,7 @@ jobs:
       - run: cargo machete --with-metadata
       - run: ./scripts/test-standalone.sh
       - run: mise run lint
+      - run: cargo clippy -- -D warnings
       - run: cargo clippy --all-features --all-targets -- -D warnings
 
   nightly:

src/cli/tasks/add.rs

@@ -100,8 +100,8 @@ impl TasksAdd {
             if let Some(description) = &self.description {
                 lines.push("#MISE description=\"".to_string() + description + "\"");
             }
-            if self.dir.is_some() {
-                lines.push("#MISE dir=".to_string() + &self.dir.unwrap());
+            if let Some(dir) = &self.dir {
+                lines.push("#MISE dir=".to_string() + dir);
             }
             if self.hide {
                 lines.push("#MISE hide=true".to_string());
@@ -165,8 +165,8 @@ impl TasksAdd {
                 }
                 task.insert("wait_for", Item::Value(wait_for.into()));
             }
-            if self.description.is_some() {
-                task.insert("description", self.description.unwrap().into());
+            if let Some(description) = &self.description {
+                task.insert("description", description.clone().into());
             }
             if !self.alias.is_empty() {
                 let mut alias = toml_edit::Array::new();
@@ -175,8 +175,8 @@ impl TasksAdd {
                 }
                 task.insert("alias", Item::Value(alias.into()));
             }
-            if self.dir.is_some() {
-                task.insert("dir", self.dir.unwrap().into());
+            if let Some(dir) = &self.dir {
+                task.insert("dir", dir.clone().into());
             }
             if self.hide {
                 task.insert("hide", true.into());
@@ -198,8 +198,8 @@ impl TasksAdd {
                 }
                 task.insert("outputs", Item::Value(outputs.into()));
             }
-            if self.shell.is_some() {
-                task.insert("shell", self.shell.unwrap().into());
+            if let Some(shell) = &self.shell {
+                task.insert("shell", shell.clone().into());
             }
             if self.quiet {
                 task.insert("quiet", true.into());

src/cmd.rs

@@ -529,7 +529,7 @@ enum ChildProcessOutput {
 mod tests {
     use pretty_assertions::assert_eq;
 
-    use crate::{cmd, config::Config};
+    use crate::config::Config;
 
     #[tokio::test]
     async fn test_cmd() {

src/lockfile.rs

@@ -487,10 +487,9 @@ fn merge_tool_entries_with_env(
                         entry.1.insert(e.clone());
                     }
                 }
-            } else if existing_tool.env.is_some() {
+            } else if let Some(existing_envs) = &existing_tool.env {
                 // Check if this env is already covered by a new entry
                 // If so, the existing entry is stale and should not be preserved
-                let existing_envs = existing_tool.env.as_ref().unwrap();
                 let env_already_covered = by_key
                     .values()
                     .any(|(_, new_envs, _)| existing_envs.iter().any(|e| new_envs.contains(e)));

src/path.rs

@@ -20,7 +20,7 @@ impl PathExt for Path {
     }
 
     fn mount(&self, on: &Path) -> PathBuf {
-        if self.is_empty() {
+        if PathExt::is_empty(self) {
             on.to_path_buf()
         } else {
             on.join(self)