aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/mozart-registry/src/inline_package.rs
diff options
context:
space:
mode:
authornsfisis <nsfisis@gmail.com>2026-05-03 20:03:42 +0900
committernsfisis <nsfisis@gmail.com>2026-05-03 20:03:42 +0900
commitd3cdb9e3f73314e04061d4d18f654e7e80b0dc18 (patch)
tree92dfda3903a7a7f1c631b18eaf544063e9b7f4f6 /crates/mozart-registry/src/inline_package.rs
parent756e0b9c18af8b2b5887ae1fb265a03187ca9c00 (diff)
downloadphp-mozart-d3cdb9e3f73314e04061d4d18f654e7e80b0dc18.tar.gz
php-mozart-d3cdb9e3f73314e04061d4d18f654e7e80b0dc18.tar.zst
php-mozart-d3cdb9e3f73314e04061d4d18f654e7e80b0dc18.zip
feat(repository): support only/exclude/canonical repo filters
Composer's FilterRepository wraps a repository with three knobs: `only` / `exclude` to drop packages by name, and `canonical: false` to relax the repo's authoritative claim on its package names so lower-priority repos can still answer. Mozart was ignoring all three, so first-listed inline / composer-repo entries always shadowed later repos and `only` / `exclude` lists were silently no-ops. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Diffstat (limited to 'crates/mozart-registry/src/inline_package.rs')
-rw-r--r--crates/mozart-registry/src/inline_package.rs18
1 files changed, 17 insertions, 1 deletions
diff --git a/crates/mozart-registry/src/inline_package.rs b/crates/mozart-registry/src/inline_package.rs
index 0cc38d6..509193b 100644
--- a/crates/mozart-registry/src/inline_package.rs
+++ b/crates/mozart-registry/src/inline_package.rs
@@ -6,6 +6,7 @@
//! pool and into the generated lockfile entry verbatim.
use crate::packagist::PackagistVersion;
+use crate::repository_filter::RepositoryFilter;
use indexmap::IndexSet;
use mozart_core::package::RawRepository;
@@ -36,6 +37,7 @@ pub fn collect_inline_packages(repositories: &[RawRepository]) -> Vec<InlinePack
let Some(value) = &repo.package else {
continue;
};
+ let filter = RepositoryFilter::from_repo(repo);
let mut from_this_repo: Vec<InlinePackage> = Vec::new();
match value {
@@ -56,13 +58,21 @@ pub fn collect_inline_packages(repositories: &[RawRepository]) -> Vec<InlinePack
let mut names_this_repo: IndexSet<String> = IndexSet::new();
for pkg in from_this_repo {
+ if !filter.is_allowed(&pkg.name) {
+ continue;
+ }
if claimed.contains(&pkg.name) {
continue;
}
names_this_repo.insert(pkg.name.clone());
packages.push(pkg);
}
- claimed.extend(names_this_repo);
+ // canonical: false → packages enter the pool but the name is not
+ // claimed, so lower-priority repositories may still answer for it.
+ // Mirrors `FilterRepository::loadPackages`'s `namesFound = []` reset.
+ if filter.canonical {
+ claimed.extend(names_this_repo);
+ }
}
packages
}
@@ -101,6 +111,9 @@ mod tests {
repo_type: "package".to_string(),
url: None,
package: Some(value),
+ only: None,
+ exclude: None,
+ canonical: None,
}
}
@@ -135,6 +148,9 @@ mod tests {
repo_type: "vcs".to_string(),
url: Some("https://example.com/foo.git".to_string()),
package: None,
+ only: None,
+ exclude: None,
+ canonical: None,
}];
assert!(collect_inline_packages(&repos).is_empty());
}