Skip to content

feat(fix): accept UV in --package-managers - #1709

Merged
Oskar Haarklou Veileborg (BarrensZeppelin) merged 1 commit into
v1.xfrom
oskar/package-managers-uv
Nov 11, 2026
Merged

Oskar Haarklou Veileborg (BarrensZeppelin) merged 1 commit into
v1.xfrom
oskar/package-managers-uv

Conversation

@BarrensZeppelin

@BarrensZeppelin Oskar Haarklou Veileborg (BarrensZeppelin) commented Nov 10, 2026 •

Copy link
Copy Markdown
Member

Coana now gives uv lockfiles (uv.lock, *.py.lock) their own --package-managers value, UV (SocketDev/coana-package-manager#2711). Before that change they answered to no package manager, so every filter let them through.

socket fix checks --package-managers values against its own copy of Coana's list, which has no UV. So socket fix --package-managers UV fails with "Invalid package manager" before Coana runs. Worse, a filter that names every other Python package manager (POETRY PIPENV PIP_REQUIREMENTS) would silently drop uv projects once Coana is bumped.

This adds UV to the list, fixes the stale Coana path in its doc comment, and adds a changelog entry.

Merge order: Coana 15.14.2, the version pinned here, rejects UV. This should merge together with, or after, the bump to a Coana release that includes #2711.

Expected reviewer effort: Stamp

Coana now answers uv.lock and *.py.lock files to UV, so uv projects can be included or left out by name. Without UV in this list, socket fix rejects the value before Coana sees it, and a filter that names every other Python package manager silently drops uv projects.
@cursor

cursor Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

PR Summary

Low Risk
Small allowlist and changelog update for CLI argument validation; no auth, security, or data-path changes.

Overview
socket fix --package-managers now accepts UV, matching Coana’s new filter for uv lockfiles (uv.lock, *.py.lock). Without this, UV was rejected as invalid locally, and filters that named only other Python package managers would exclude uv projects once Coana started classifying them separately.

The change is limited to adding UV to ALL_PACKAGE_MANAGERS in package-manager.mts, correcting the mirrored Coana source path in the file comment, and documenting the behavior under Unreleased in the changelog.

Reviewed by Cursor Bugbot for commit 10f3c21. Configure here.

@BarrensZeppelin
Oskar Haarklou Veileborg (BarrensZeppelin) merged commit ff46ee1 into v1.x Nov 11, 2026
18 checks passed
@BarrensZeppelin
Oskar Haarklou Veileborg (BarrensZeppelin) deleted the oskar/package-managers-uv branch November 11, 2026 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants