feat(base): cache uv directory
Metadata
Current evaluation
Adds uv cache directory caching to craft-providers base, approved by 1 reviewer with 0 unresolved comments. CI mostly passing, some slow tests pending.
Suggested action: needs review
Reason: PR is approved with 0 unresolved review comments and CI mostly passing. @astrojuanlu asked about blockers on 2026-03-30, and @lengau noted a new opt-in cache-sharing approach is in development. The PR addresses charmcraft#2461 with a straightforward +43/-22 line change and should be reviewed for merge or to determine if the new cache approach supersedes it.
Impact:
35
Quick Win:
28.0
Staleness:
25
Complexity:
20
Confidence:
75
Issue body
Based on https://github.com/canonical/craft-providers/pull/394
Fixes https://github.com/canonical/charmcraft/issues/2461
uv's cache directory is located in the same parent directory as pip's cache directory
https://github.com/canonical/craft-providers/blob/b5b4a45ea0e8ba2ff1cc984a9676aa5f46c367d4/craft_providers/base.py#L831-L832
https://docs.astral.sh/uv/reference/storage/#cache-directory
https://pip.pypa.io/en/stable/topics/caching/#default-paths
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen3.6-35b-a3b-mtp-q6 |
Impact:
35
Quick Win:
28.0
Staleness:
25
Complexity:
20
Confidence:
75
|
needs review | Adds uv cache directory caching to craft-providers base, approved by 1 reviewer with 0 unresolved comments. CI mostly passing, some slow tests pending. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
55
Complexity:
20
Confidence:
85
|
keep open | Implements uv cache directory caching to match pip's location. PR is approved, CI failures are known or fixed elsewhere, and maintainers are evaluating a new cache-sharing approach without rejecting this change. | |
| qwen3.6-35b-a3b-mtp-q6 |
Staleness:
70
Complexity:
20
Confidence:
85
|
close stale | Adds caching for the uv directory alongside pip's cache. Inactive for 104 days; a maintainer indicated a broader opt-in cache sharing solution is in development, superseding this targeted change. | |
| qwen3.6-35b-a3b-mtp-q6 |
Staleness:
55
Complexity:
15
Confidence:
85
|
needs review | Adds uv cache directory to shared cache paths in the base module. PR is awaiting maintainer review/decision, with recent comments noting unrelated test failures and a future opt-in cache sharing approach. |
Update history
| Date | Change |
|---|---|
| updated |
Related work
-
Related To:
canonical/charmcraft#2461
(confidence 90%)
Issue this PR fixes
-
Related To:
canonical/craft-providers#394
(confidence 85%)
PR explicitly based on this related PR
Related issues
No related issues found above the similarity threshold.