aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/mozart-registry/src/installer_executor/mod.rs
diff options
context:
space:
mode:
authornsfisis <nsfisis@gmail.com>2026-05-03 22:26:55 +0900
committernsfisis <nsfisis@gmail.com>2026-05-03 22:26:55 +0900
commit2a680f571eae31737525ef96105e929ae420b061 (patch)
tree6297d7b76753de69e9934550a739c39459d8a681 /crates/mozart-registry/src/installer_executor/mod.rs
parent55e6f5367bf86d1dc6e99b7492d86c5208dd1f1c (diff)
downloadphp-mozart-2a680f571eae31737525ef96105e929ae420b061.tar.gz
php-mozart-2a680f571eae31737525ef96105e929ae420b061.tar.zst
php-mozart-2a680f571eae31737525ef96105e929ae420b061.zip
fix(resolver): strip inline #ref and gate alias trace on alias stability
Two related parity gaps surfaced by the `alias-with-reference` fixture: 1. A root require like `dev-main#abcd as 1.0.0` left the SAT-side constraint as `dev-main#abcd`, which no candidate matched, so resolution failed before the alias could be materialized. Mirror Composer's `extractAliases` regex (which captures only the constraint up to `#`) and `RootPackageLoader::extractReferences` (which records the hash separately): drop the trailing `#hex` from the resolver-side constraint and from the alias's left-hand side. Lockfile generation already pulls the reference back out of the raw require map for the post-resolve override. 2. `MarkAliasInstalled`'s trace line gated the reference suffix on the *target* package's stability, so a stable alias like `1.0.0` pointing at a dev-branch target rendered as `1.0.0 abcd`. Mirror `AliasPackage::getFullPrettyVersion`: the alias decides on its own whether to append the suffix based on its own normalized version, so a stable alias skips the suffix even when the target is dev.
Diffstat (limited to 'crates/mozart-registry/src/installer_executor/mod.rs')
-rw-r--r--crates/mozart-registry/src/installer_executor/mod.rs24
1 files changed, 24 insertions, 0 deletions
diff --git a/crates/mozart-registry/src/installer_executor/mod.rs b/crates/mozart-registry/src/installer_executor/mod.rs
index c344ba4..c29e32c 100644
--- a/crates/mozart-registry/src/installer_executor/mod.rs
+++ b/crates/mozart-registry/src/installer_executor/mod.rs
@@ -110,6 +110,30 @@ pub fn format_full_pretty_with_pretty(pretty_version: &str, pkg: &LockedPackage)
)
}
+/// Render an alias's full pretty version: the alias's own pretty form for
+/// the visible text, the alias's *normalized* version for the dev-stability
+/// gate, and the target package's source/dist references for the suffix.
+/// Mirrors `AliasPackage::getFullPrettyVersion`, where the alias decides on
+/// its own whether to append a reference based on its own stability — so a
+/// stable alias like `1.0.0` skips the suffix even when the target is a dev
+/// branch.
+pub fn format_full_pretty_alias(
+ alias_pretty: &str,
+ alias_version: &str,
+ target: &LockedPackage,
+) -> String {
+ let source_ref = target.source.as_ref().and_then(|s| s.reference.as_deref());
+ let dist_ref = target.dist.as_ref().and_then(|d| d.reference.as_deref());
+ let source_type = target.source.as_ref().map(|s| s.source_type.as_str());
+ format_full_pretty_with_refs(
+ alias_pretty,
+ alias_version,
+ source_ref,
+ dist_ref,
+ source_type,
+ )
+}
+
/// Same as [`format_full_pretty_version_for_installed`] but lets the caller
/// supply an alternate pretty version. Used when emitting
/// `MarkAliasUninstalled`: the alias's `1.0.x-dev` text needs to be rendered