feat: add .gitignore support to fix duplicate package detection

Replace walkdir with ignore crate to respect .gitignore patterns when
scanning for package manifests. This fixes the issue where packages
would appear multiple times when found in both the source and target
directories.

Changes:
- Add ignore crate dependency
- Replace WalkDir with WalkBuilder in discover_members()
- Configure walker to respect .gitignore, global git ignore, and .git/info/exclude
- Remove unused walkdir dependency
- Fix clippy warning about unnecessary map_or

Fixes duplicate entries like:
  elf-magic: 0.2.7
  elf-magic: 0.2.7  <- from target/

Now correctly shows:
  elf-magic: 0.2.7

Author: levicook

Cargo.lock

@@ -35,12 +35,12 @@ path = "src/bin/odo.rs"
 [dependencies]
 anyhow = "1.0.98"
 clap = { version = "4.5", features = ["derive"] }
+ignore = "0.4"
 json-patch = "4.0.0"
 semver = "1.0.26"
 serde = { version = "1.0", features = ["derive"] }
 serde_json = "1.0"
 toml_edit = "0.22.27"
-walkdir = "2.5.0"
 
 [features]
 fixture-tests = []

Cargo.toml

@@ -3,8 +3,8 @@ mod package_json;
 
 use crate::domain::{Workspace, WorkspaceMember};
 use anyhow::{Context, Result};
+use ignore::WalkBuilder;
 use std::path::Path;
-use walkdir::WalkDir;
 
 /// Load the current workspace from the file system
 ///
@@ -51,8 +51,24 @@ where
     }
 
     let mut members = Vec::new();
-    for entry in WalkDir::new(root).into_iter().filter_map(Result::ok) {
+
+    // Use WalkBuilder to respect .gitignore files
+    for result in WalkBuilder::new(root)
+        .hidden(false) // Don't ignore hidden files by default
+        .git_ignore(true) // Respect .gitignore files
+        .git_global(true) // Respect global git ignore
+        .git_exclude(true) // Respect .git/info/exclude
+        .build()
+    {
+        let entry = result.with_context(|| "Failed to walk directory tree")?;
         let path = entry.path();
+
+        // Skip directories - we only care about files
+        if entry.file_type().is_some_and(|ft| ft.is_dir()) {
+            continue;
+        }
+
+        // Apply custom ignore filter
         if ignore(path) {
             continue;
         }