mirror of
https://github.com/logos-co/logos-package-downloader-module.git
synced 2026-08-27 10:31:11 +00:00
Dario LipicarandClaude Opus 4.8
6b8790eceb
* fix: downloadResolvedDependencies keeps already-installed satisfying deps downloadResolvedDependencies re-resolved with NO installed set (resolveDependenciesJson(deps) — installed defaulted to empty), while the preview resolveDependencies(deps, installed) was installed-aware. So the preview could say 'nothing changes' while the actual download silently upgraded an installed transitive dep to the newest version. It now takes an installedPackagesJson arg and threads it through, so download matches preview. Also re-pins the downloader lib to pick up the resolver fix where a top-level pinned dep is no longer re-resolved to newest as a transitive dep. BREAKING: downloadResolvedDependencies now takes 2 args; callers must pass the installed set (or ""). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * docs: update downloadResolvedDependencies to the 2-arg signature Addresses Copilot review on #17 — README and docs/project.md still showed the old 1-arg downloadResolvedDependencies(depsJson); they now show the installedPackagesJson parameter. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * chore: re-pin logos-package-downloader to merged master (#18) The resolver dedup (a top-level pinned dep is no longer re-resolved to newest) merged; move the pin from the branch head to master (8d5128b). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>