fix: correct format verb errors in error messages and logging output

Fixes multiple format string bugs found by auditing Go format verbs:
- %ws typo in remove.go: %ws is not a valid Go verb; produces stray 's' in output
- %w used in output.Errorf/Fatalf/Debugf: %w is only supported in fmt.Errorf;
  using it in fmt.Sprintf-based logging functions produces garbled %!w(...) output

Affected files:
  pkg/cmd/remove/remove.go        (%ws -> %w, %w -> %s)
  pkg/cmd/unpack/cmd.go           (%w -> %v)
  pkg/cmd/info/cmd.go             (%w -> %v)
  pkg/lib/filesystem/cache/cache.go (%w -> %s)
  pkg/lib/harness/llm-harness.go  (%w -> %v)

Verification: go vet, go build, go test all pass.
Signed-off-by: Adesh Deshmukh <adeshkd123@gmail.com>

Author: AdeshDeshmukh

pkg/cmd/info/cmd.go

@@ -119,7 +119,7 @@ func runCommand(opts *infoOptions) func(*cobra.Command, []string) error {
 		} else {
 			yamlBytes, err := config.MarshalToYAML()
 			if err != nil {
-				return output.Fatalf("Error formatting manifest: %w", err)
+				return output.Fatalf("Error formatting manifest: %v", err)
 			}
 			fmt.Print(string(yamlBytes))
 		}

pkg/cmd/remove/remove.go

@@ -57,7 +57,7 @@ func removeAllModels(ctx context.Context, opts *removeOptions) error {
 			// First untag all manifests for this digest
 			for _, tag := range tags {
 				if err := localRepo.Untag(ctx, tag); err != nil {
-					output.Errorf("Failed to untag %s:%s: %w", repository, tag, err)
+					output.Errorf("Failed to untag %s:%s: %s", repository, tag, err)
 				}
 				output.Infof("Untagged %s:%s", repository, tag)
 			}
@@ -139,7 +139,7 @@ func removeModelRef(ctx context.Context, localRepo local.LocalRepo, ref *registr
 	if err := ref.ValidateReferenceAsDigest(); err == nil || forceDelete {
 		output.Debugf("Deleting manifest with digest %s", ref.Reference)
 		if err := localRepo.Delete(ctx, desc); err != nil {
-			return ocispec.DescriptorEmptyJSON, fmt.Errorf("failed to delete model: %ws", err)
+			return ocispec.DescriptorEmptyJSON, fmt.Errorf("failed to delete model: %w", err)
 		}
 		return desc, nil
 	}

pkg/cmd/unpack/cmd.go

@@ -189,7 +189,7 @@ func runCommand(opts *unpackOptions) func(*cobra.Command, []string) error {
 		}
 		// Make sure target directory exists, in case user is using the -d flag
 		if err := os.MkdirAll(opts.unpackDir, 0755); err != nil {
-			return output.Fatalf("failed to create directory %s: %w", opts.unpackDir, err)
+			return output.Fatalf("failed to create directory %s: %v", opts.unpackDir, err)
 		}
 		output.Infof("Unpacking to %s", unpackTo)
 

pkg/lib/filesystem/cache/cache.go

@@ -89,7 +89,7 @@ func MkCacheFile(subDir CacheSubDir, basename string) (tempFile *os.File, cleanu
 			output.Errorf("Error closing temporary file %s: %s", tempFilePath, err)
 		}
 		if err := os.Remove(f.Name()); err != nil && !os.IsNotExist(err) {
-			output.Errorf("Failed to remove temporary file %s: %w", tempFilePath, err)
+			output.Errorf("Failed to remove temporary file %s: %s", tempFilePath, err)
 		}
 	}
 	return f, cleanup, nil

pkg/lib/harness/llm-harness.go

@@ -163,7 +163,7 @@ func (harness *LLMHarness) Stop() error {
 
 	err = process.Signal(os.Interrupt) // Try to kill it gently
 	if err != nil {
-		output.Debugf("Error killing process %w", err)
+		output.Debugf("Error killing process %v", err)
 		// If SIGTERM failed, kill it with SIGKILL
 		err = process.Kill()
 		if err != nil {