[codex] Add clean command for managed tools#24
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
ptm clean [--apply]for removing ptm-managed release tools that are no longer configured.${XDG_DATA_HOME:-~/.local/share}/ptm/tools/<bin>/currentand publish$XDG_BIN_HOMEentries as symlinks.opt_dirsupport; full archive extraction is now driven bybin_path_in_archive/extra_bins..ptm.jsonmetadata for managed release installs.Breaking Changes / Migration Notes
opt_diris no longer a supported config field and is ignored by config loading.$XDG_BIN_HOME; they are stored under the ptm managed root and exposed through symlinks.ptm clean --applyremoves only directories under the ptm managed root and symlinks that point into those directories.npm / Bun Follow-up
ptm cleancurrently manages release-tool directories only. The CLI shape should support npm/bun later, but the internal clean plan will need to grow beyondtool_dirdeletion to model package-manager uninstall actions.Validation
uv run pytestpassed: 164 testsuv run ruff check src testspasseduv run ruff format --check src testspasseduv run ty check src testspassed