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-registry/src/inline_package.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'crates/mozart-registry/src/inline_package.rs') diff --git a/crates/mozart-registry/src/inline_package.rs b/crates/mozart-registry/src/inline_package.rs index bad00fb..0cc38d6 100644 --- a/crates/mozart-registry/src/inline_package.rs +++ b/crates/mozart-registry/src/inline_package.rs @@ -6,8 +6,8 @@ //! pool and into the generated lockfile entry verbatim. use crate::packagist::PackagistVersion; +use indexmap::IndexSet; use mozart_core::package::RawRepository; -use std::collections::HashSet; /// One package extracted from a `type: package` repository. pub struct InlinePackage { @@ -28,7 +28,7 @@ pub struct InlinePackage { /// first-repo-wins priority via `RepositorySet::findPackages`. pub fn collect_inline_packages(repositories: &[RawRepository]) -> Vec { let mut packages = Vec::new(); - let mut claimed: HashSet = HashSet::new(); + let mut claimed: IndexSet = IndexSet::new(); for repo in repositories { if repo.repo_type != "package" { continue; @@ -54,7 +54,7 @@ pub fn collect_inline_packages(repositories: &[RawRepository]) -> Vec {} } - let mut names_this_repo: HashSet = HashSet::new(); + let mut names_this_repo: IndexSet = IndexSet::new(); for pkg in from_this_repo { if claimed.contains(&pkg.name) { continue; -- cgit v1.3.1