Skip to content

Adopt package manager command classes - #1675

Closed
edvilme wants to merge 2 commits into
package-manager-command-refactorfrom
package-manager-command-migration
Closed

Adopt package manager command classes#1675
edvilme wants to merge 2 commits into
package-manager-command-refactorfrom
package-manager-command-migration

Conversation

@edvilme

@edvilme edvilme commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Adopts the package manager command classes introduced in #1621 across Pip/UV, Conda, and Poetry.

Changes

  • Replaces existing command construction and execution paths with the new command classes.
  • Preserves existing command flags, environment targeting, version ordering, prerelease defaults, timeouts, and Poetry cwd behavior.
  • Removes superseded utility wrappers and obsolete parser tests.
  • Adds regression coverage for available-version precision, UV selection and tree targeting, and Conda upgrade semantics.

Dependency

This PR is stacked on #1621 and intentionally contains only the migration and cleanup commit. Review and merge #1621 first; this PR can then be retargeted to main.

Testing

  • npm run lint
  • npm run compile-tests
  • npm run unittest (1,440 passing, 4 pending)

@edvilme
edvilme force-pushed the package-manager-command-refactor branch from bd9c3b0 to b620149 Compare July 28, 2026 17:48
@edvilme
edvilme force-pushed the package-manager-command-migration branch from 8a29f4f to 8db3b41 Compare July 28, 2026 17:56
@edvilme
edvilme force-pushed the package-manager-command-migration branch from 8db3b41 to e1a77dd Compare July 28, 2026 18:11
@edvilme

edvilme commented Jul 28, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #1678. GitHub prevented retargeting this PR after registering it as part of the original stack; #1678 contains the same adoption commit and is correctly based on the smoke-test PR #1677.

@edvilme edvilme closed this Jul 28, 2026
@edvilme
edvilme deleted the package-manager-command-migration branch July 28, 2026 20:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant