Repository navigation
fix(npm): trust packages pinned in mise lockfiles - #11384
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthrough
Changesnpm lockfile trust
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant ToolVersion
participant NPMBackend
participant AubeProjectWriter
participant npmrc
ToolVersion->>NPMBackend: resolved_from_lockfile()
NPMBackend->>AubeProjectWriter: pass lockfile provenance
AubeProjectWriter->>npmrc: emit allowedUnpopularPackages when applicable
Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile SummaryThis PR tracks lockfile-derived tool versions and uses that provenance to exempt only the requested npm package from aube’s low-download gate.
Confidence Score: 5/5The PR appears safe to merge, with the lockfile exemption correctly limited to the matching npm package and aube’s popularity check. Lockfile provenance is set only by the lockfile-resolution constructor, remains distinct from other locked states, reaches both aube installation paths unchanged, and produces a package-specific exemption. Important Files Changed
Reviews (1): Last reviewed commit: "fix(npm): trust packages pinned in mise ..." | Re-trigger Greptile |
Hyperfine Performance
|
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
mise-2026.7.14 x -- echo |
20.2 ± 1.2 | 17.1 | 29.3 | 1.00 |
mise x -- echo |
21.2 ± 1.5 | 18.1 | 28.8 | 1.05 ± 0.10 |
mise env
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
mise-2026.7.14 env |
18.9 ± 1.3 | 16.2 | 27.9 | 1.00 |
mise env |
20.4 ± 1.7 | 17.5 | 38.4 | 1.08 ± 0.12 |
mise hook-env
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
mise-2026.7.14 hook-env |
19.7 ± 1.2 | 16.6 | 26.8 | 1.00 |
mise hook-env |
21.4 ± 1.5 | 18.2 | 27.2 | 1.09 ± 0.10 |
mise ls
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
mise-2026.7.14 ls |
17.5 ± 1.2 | 14.2 | 24.7 | 1.00 |
mise ls |
19.1 ± 1.4 | 16.3 | 26.9 | 1.09 ± 0.11 |
xtasks/test/perf
| Command | mise-2026.7.14 | mise | Variance |
|---|---|---|---|
| install (cached) | 171ms | 184ms | -7% |
| ls (cached) | 64ms | 67ms | -4% |
| bin-paths (cached) | 65ms | 66ms | -1% |
| task-ls (cached) | 93ms | 98ms | -5% |
Summary
mise.lockaube_cli, scoped to the requested package onlyContext
This fixes the hidden embedded prompt reported in #11375 without requiring every locked npm tool to repeat
allow_low_downloads = truein configuration.The corresponding aube behavior for native npm project lockfiles is in aubepkg/aube#1143.
Validation
mise run lintmise run test:unit -- backend::npm::tests::test_write_aube_mise run test:unit -- toolset::tool_version::tests::AI-assisted — Tool: Codex; model: unavailable/unavailable; version: unavailable.
Note
Cursor Bugbot is generating a summary for commit 2406f88. Configure here.
Summary by CodeRabbit
New Features
allow_low_downloads.Bug Fixes
allow_low_downloads.Documentation
allow_low_downloadsis required and when lockfile resolution provides automatic approval.