From ae1aa6540761e54a76b8f7984cf93cd3a0d011d0 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Sun, 3 May 2026 11:55:03 +0900 Subject: refactor: switch internal maps/sets from HashMap to IndexMap Adopt indexmap workspace-wide so iteration order is deterministic and follows insertion order. The non-deterministic order of std HashMap otherwise leaks into resolver decisions when multiple valid solutions exist (e.g. cyclic require pairs under prefer-lowest), making behavior flaky and divergent from Composer's PHP-array semantics. Co-Authored-By: Claude Opus 4.7 (1M context) --- crates/mozart/src/commands/search.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'crates/mozart/src/commands/search.rs') diff --git a/crates/mozart/src/commands/search.rs b/crates/mozart/src/commands/search.rs index accd6af..023bfdf 100644 --- a/crates/mozart/src/commands/search.rs +++ b/crates/mozart/src/commands/search.rs @@ -139,7 +139,7 @@ pub async fn execute( // Deduplicate to unique vendor names (Composer returns vendor-only names // for SEARCH_VENDOR mode). - let mut seen = std::collections::HashSet::new(); + let mut seen = indexmap::IndexSet::new(); let mut vendor_names: Vec = Vec::new(); for r in &results { let vendor = r.name.split('/').next().unwrap_or("").to_string(); @@ -515,7 +515,7 @@ mod tests { ]; let refs: Vec<&SearchResult> = results.iter().collect(); - let mut seen = std::collections::HashSet::new(); + let mut seen = indexmap::IndexSet::new(); let mut vendor_names: Vec = Vec::new(); for r in &refs { let vendor = r.name.split('/').next().unwrap_or("").to_string(); -- cgit v1.3.1