fix: Prevent unlink from continuing after processing a file

The previous implementation of the unlink command would continue
processing files even after encountering an error or skipping a file
due to it not being a symlink. This commit modifies the loop to return
after processing each file, ensuring that the command stops processing
the remaining files in such cases.

Author: rivethorn

cmd/unlink.go

@@ -52,9 +52,10 @@ var unlinkCmd = &cobra.Command{
 							fmt.Printf("Skipping %s\n", destPath)
 							continue
 						}
+						os.Remove(destPath)
+						internal.LogVerbose(verbose, "Removed %s symlink", destPath)
 					}
-					os.Remove(destPath)
-					internal.LogVerbose(verbose, "Removed %s symlink", destPath)
+					return nil
 				}
 				if dryRun {
 					fmt.Printf("Would restore backup %s -> %s\n", backupPath, destPath)
@@ -68,9 +69,11 @@ var unlinkCmd = &cobra.Command{
 					} else {
 						fmt.Printf("󰩹 Removed symlink %s (no backup found)\n", destPath)
 					}
+					return nil
 				}
 			} else {
 				fmt.Printf(" %s is not a symlink, skipping\n", destPath)
+				return nil
 			}
 		}
 		return nil