diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-02-24 01:00:48 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-02-24 01:00:48 +0900 |
| commit | 64ed53cf184fb05cbfe9f0336bc8695ff0e800f8 (patch) | |
| tree | cea8c1854c5c7bf0fcffcaf175f1bf188d9f8904 /crates/mozart/src/commands/archive.rs | |
| parent | d9858fbd322ce20de84f44ce67ae11edf081578c (diff) | |
| download | php-mozart-64ed53cf184fb05cbfe9f0336bc8695ff0e800f8.tar.gz php-mozart-64ed53cf184fb05cbfe9f0336bc8695ff0e800f8.tar.zst php-mozart-64ed53cf184fb05cbfe9f0336bc8695ff0e800f8.zip | |
fix(cache): enable dist archive caching for all commands
files_cache was Option<&Cache> and install_from_lock always passed None,
so downloaded zip/tar archives were never cached. Make the parameter
non-optional (&Cache) and wire it through every command that downloads
dist archives (install, update, require, remove, create-project,
archive). The Cache internally respects --no-cache via its enabled flag,
so the Option wrapper was unnecessary.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Diffstat (limited to 'crates/mozart/src/commands/archive.rs')
| -rw-r--r-- | crates/mozart/src/commands/archive.rs | 21 |
1 files changed, 17 insertions, 4 deletions
diff --git a/crates/mozart/src/commands/archive.rs b/crates/mozart/src/commands/archive.rs index 2490246..faa0e94 100644 --- a/crates/mozart/src/commands/archive.rs +++ b/crates/mozart/src/commands/archive.rs @@ -96,6 +96,7 @@ pub async fn execute( let cache_config = mozart_registry::cache::build_cache_config(cli.no_cache); let repo_cache = mozart_registry::cache::Cache::repo(&cache_config); + let files_cache = mozart_registry::cache::Cache::files(&cache_config); // 1. Determine working directory let working_dir = match &cli.working_dir { @@ -153,7 +154,14 @@ pub async fn execute( let meta: PackageMeta = if let Some(ref pkg_name) = args.package { // Remote package mode console.info("Searching for the specified package."); - resolve_remote_package(pkg_name, args.version.as_deref(), &repo_cache, console).await? + resolve_remote_package( + pkg_name, + args.version.as_deref(), + &repo_cache, + &files_cache, + console, + ) + .await? } else { // Root package mode if !composer_json_path.exists() { @@ -248,6 +256,7 @@ async fn resolve_remote_package( package_name: &str, version_constraint: Option<&str>, repo_cache: &mozart_registry::cache::Cache, + files_cache: &mozart_registry::cache::Cache, console: &mozart_core::console::Console, ) -> anyhow::Result<PackageMeta> { use mozart_core::package::Stability; @@ -334,9 +343,13 @@ async fn resolve_remote_package( let temp_dir = temp_base.join(&unique); std::fs::create_dir_all(&temp_dir)?; - let bytes = - mozart_registry::downloader::download_dist(&dist.url, dist.shasum.as_deref(), None, None) - .await?; + let bytes = mozart_registry::downloader::download_dist( + &dist.url, + dist.shasum.as_deref(), + None, + files_cache, + ) + .await?; match dist.dist_type.as_str() { "zip" => mozart_registry::downloader::extract_zip(&bytes, &temp_dir)?, |
