Repository navigation
Conversation
## Problem `clime init` remembers remote URLs but discards local manifest paths. Later runs without an argument therefore use an older URL or the built-in defaults. Bulk plugin updates also run installers for stale manifest entries whose executable no longer exists. ## Reproduction Run `clime init ./tools.yaml`, change directories, then run `clime init` without an argument. The local source is not reused. Removing a managed plugin's executable while retaining its manifest entry still causes `clime plugin update all` to invoke its updater. ## Fix Save local init sources as absolute paths in the existing `init_url` field, retaining URL support and reusing either source when omitted. Filter bulk updates to managed plugins whose executable exists in `~/.clime/plugins/` or on `PATH`, leaving missing entries unchanged. ## Test - `go test ./... -v`, `go vet ./...`, and `go build` pass. - Regression tests cover path reuse from another directory and bulk updates with both installed and missing plugins. - CLI smoke checks cover URL reuse, invalid source preservation, and skipping missing executables, non-executable files, and broken links. With no installed managed plugins, bulk update now reports: ```text No installed managed plugins found. ``` Assistant-By GPT-6
git-hulk
changed the title
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.
Problem
clime initremembers remote URLs but discards local manifest paths.Later runs without an argument therefore use an older URL or the built-in
defaults. Bulk plugin updates also run installers for stale manifest
entries whose executable no longer exists.
Reproduction
Run
clime init ./tools.yaml, change directories, then runclime initwithout an argument. The local source is not reused. Removing a managed
plugin's executable while retaining its manifest entry still causes
clime plugin update allto invoke its updater.Fix
Save local init sources as absolute paths in the existing
init_urlfield, retaining URL support and reusing either source when omitted.
Filter bulk updates to managed plugins whose executable exists in
~/.clime/plugins/or onPATH, leaving missing entries unchanged.Test
go test ./... -v,go vet ./..., andgo buildpass.updates with both installed and missing plugins.
skipping missing executables, non-executable files, and broken links.
With no installed managed plugins, bulk update now reports:
Assistant-By GPT-6