Describe the bug
kit remove exits with code 0 even when extra comma-separated tags fail to remove. The function removeModel() logs the errors via output.Errorf but always returns nil, so the caller never knows a failure happened. This means kit remove myrepo:tag1,tag2,bogus silently reports success despite bogus failing.
To Reproduce
kit pull alice/my-model:latest
kit tag alice/my-model:latest alice/my-model:extra1 alice/my-model:extra2
kit remove alice/my-model:latest,extra1,extra2,nonexistent
echo $?
extra2 and nonexistent both fail (they don't exist), but exit code is 0.
Version
latest
Additional context
The loop at pkg/cmd/remove/remove.go:115-126 iterates opts.extraTags, logs failures, but returns nil. Compare with lines 108-110 where the primary reference correctly returns an error.
Workaround: remove each tag individually instead of comma-separated.
Describe the bug
kit removeexits with code 0 even when extra comma-separated tags fail to remove. The functionremoveModel()logs the errors viaoutput.Errorfbut always returnsnil, so the caller never knows a failure happened. This meanskit remove myrepo:tag1,tag2,bogussilently reports success despite bogus failing.To Reproduce
extra2andnonexistentboth fail (they don't exist), but exit code is0.Version
latest
Additional context
The loop at
pkg/cmd/remove/remove.go:115-126iteratesopts.extraTags, logs failures, but returnsnil. Compare with lines 108-110 where the primary reference correctly returns an error.Workaround: remove each tag individually instead of comma-separated.