From 3eefbbe8839315b53171523bf43fa65bde41803c Mon Sep 17 00:00:00 2001 From: nsfisis Date: Sun, 22 Feb 2026 19:35:40 +0900 Subject: fix(resolver): handle virtual packages and deduplicate pool exploration Virtual/meta packages (e.g. "psr/http-client-implementation") don't exist on Packagist and caused a fatal error during transitive dependency exploration. These packages are resolved via provides/replaces from other packages already in the pool, so 404 errors are now skipped. Also fix PoolBuilder::next_pending() repeatedly returning the same virtual package name by tracking explored names in a HashSet, since virtual packages are never added to inputs and the old check (inputs.any(name)) never matched them. Co-Authored-By: Claude Opus 4.6 --- crates/mozart-registry/src/resolver.rs | 20 ++++++++++++-------- 1 file changed, 12 insertions(+), 8 deletions(-) (limited to 'crates/mozart-registry/src/resolver.rs') diff --git a/crates/mozart-registry/src/resolver.rs b/crates/mozart-registry/src/resolver.rs index 66864f7..3e0936f 100644 --- a/crates/mozart-registry/src/resolver.rs +++ b/crates/mozart-registry/src/resolver.rs @@ -467,14 +467,18 @@ pub async fn resolve(request: &ResolveRequest) -> Result, R continue; } - let versions = handle - .block_on(packagist::fetch_package_versions( - &name, - repo_cache.as_ref(), - )) - .map_err(|e| { - ResolveError::DependencyFetchError(format!("Failed to fetch {}: {}", name, e)) - })?; + let versions = match handle.block_on(packagist::fetch_package_versions( + &name, + repo_cache.as_ref(), + )) { + Ok(v) => v, + Err(_) => { + // Virtual/meta packages (e.g. "psr/http-client-implementation") + // don't exist on Packagist. They are resolved via provides/replaces + // from other packages already in the pool. + continue; + } + }; for pv in &versions { let inputs = -- cgit v1.3.1