From ae1aa6540761e54a76b8f7984cf93cd3a0d011d0 Mon Sep 17 00:00:00 2001
From: nsfisis <nsfisis@gmail.com>
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) <noreply@anthropic.com>
---
 crates/mozart-vcs/src/driver/gitlab.rs | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

(limited to 'crates/mozart-vcs/src/driver/gitlab.rs')

diff --git a/crates/mozart-vcs/src/driver/gitlab.rs b/crates/mozart-vcs/src/driver/gitlab.rs
index f96c078..c1afbcb 100644
--- a/crates/mozart-vcs/src/driver/gitlab.rs
+++ b/crates/mozart-vcs/src/driver/gitlab.rs
@@ -1,4 +1,5 @@
-use std::collections::{BTreeMap, HashMap};
+use indexmap::IndexMap;
+use std::collections::BTreeMap;
 
 use anyhow::{Result, bail};
 use regex::Regex;
@@ -21,7 +22,7 @@ pub struct GitLabDriver {
     root_identifier: Option<String>,
     tags: Option<BTreeMap<String, String>>,
     branches: Option<BTreeMap<String, String>>,
-    info_cache: HashMap<String, Option<serde_json::Value>>,
+    info_cache: IndexMap<String, Option<serde_json::Value>>,
     git_driver: Option<Box<GitDriver>>,
     http_client: Client,
     config: DriverConfig,
@@ -41,7 +42,7 @@ impl GitLabDriver {
             root_identifier: None,
             tags: None,
             branches: None,
-            info_cache: HashMap::new(),
+            info_cache: IndexMap::new(),
             git_driver: None,
             http_client: Client::new(),
             config,
-- 
cgit v1.3.1