Skip to content

Complete uvs hardening and prepare for public feedback#4

Merged
maphew merged 5 commits into
mainfrom
agent/phase-5-public-cleanup
Jul 21, 2026
Merged

Complete uvs hardening and prepare for public feedback#4
maphew merged 5 commits into
mainfrom
agent/phase-5-public-cleanup

Conversation

@maphew

@maphew maphew commented Jul 21, 2026

Copy link
Copy Markdown
Owner

What changed

  • harden lifecycle, registry recovery, update behavior, and CLI failure handling
  • add isolated real-uv lifecycle coverage and a Windows/Linux, Python 3.10–3.13 CI matrix
  • resolve independent release-review findings around entry points, versioning, simple output, and debug behavior
  • narrow and clarify the public product contract, remove obsolete project artifacts, and add contribution, security, bug-report, and usefulness-feedback paths
  • position 0.1 honestly as an experimental GitHub-installed tool seeking validation rather than a proven or PyPI-published product

Why

The earlier prototype and Phase 2 package-generation work established the core adapter, but the remaining lifecycle behavior, integration evidence, repository surface, and public claims needed to be made dependable and internally consistent before inviting outside use.

Impact

A supported single-file Python script now has deterministic snapshot install/update/uninstall behavior, recovery-oriented registry persistence, clearer errors, and documented limits. Newcomers get a reversible trial path and focused ways to report bugs or candid usefulness feedback.

Validation

  • uv run pytest — 170 passed
  • uv run pytest -m integration -n 0 — 1 passed
  • GitHub Actions run 29476378538 — passed across Windows and Ubuntu on Python 3.10–3.13
  • independent release review completed with all findings resolved

@maphew
maphew merged commit 64ba375 into main Jul 21, 2026
16 checks passed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9c9cd8510d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/uvs/cli.py
Comment on lines +326 to +328
rollback_result = run_uv_uninstall(
cli_name, quiet=options.get("quiet", False)
)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Avoid uninstalling existing tools on update registry failures

When this helper is called from uvs update for an already installed tool, a save_registry failure after uv tool install succeeds now runs uv tool uninstall. Since uv has already replaced the previous environment during the install, this rollback deletes the command entirely while the old registry file may remain unchanged; for example, a disk-full or permission error during registry persistence turns a failed update bookkeeping write into loss of the installed tool. Track whether this operation is a fresh install versus an update, and avoid destructive uninstall or restore the prior snapshot for updates.

Useful? React with 👍 / 👎.

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