review: Address review feedback

- Add nil checks for Snapshot
- Change format string for supported output formats

Author: bobbyhouse

cmd/docker-mcp/commands/workingset.go

@@ -344,7 +344,7 @@ Filters use key=value format (e.g., name=github, profile=my-dev-env).`,
 
 	flags := cmd.Flags()
 	flags.StringArrayVarP(&opts.Filters, "filter", "f", []string{}, "Filter output (e.g., name=github, profile=my-dev-env)")
-	flags.StringVar(&opts.Format, "format", string(workingset.OutputFormatHumanReadable), fmt.Sprintf("Output format (%s)", strings.Join(workingset.SupportedFormats(), "|")))
+	flags.StringVar(&opts.Format, "format", string(workingset.OutputFormatHumanReadable), fmt.Sprintf("Supported: %s.", strings.Join(workingset.SupportedFormats(), ", ")))
 
 	return cmd
 }

pkg/workingset/server.go

@@ -122,7 +122,8 @@ func RemoveServers(ctx context.Context, dao db.DAO, id string, serverNames []str
 	originalCount := len(workingSet.Servers)
 	filtered := make([]Server, 0, len(workingSet.Servers))
 	for _, server := range workingSet.Servers {
-		if !namesToRemove[server.Snapshot.Server.Name] {
+		// TODO: Remove when Snapshot is required
+		if server.Snapshot == nil || !namesToRemove[server.Snapshot.Server.Name] {
 			filtered = append(filtered, server)
 		}
 	}
@@ -247,6 +248,10 @@ func buildSearchResults(dbSets []db.WorkingSet, nameFilter string) []SearchResul
 }
 
 func matchesNameFilter(server Server, nameLower string) bool {
+	// TODO: Remove when Snapshot is required
+	if server.Snapshot == nil {
+		return false
+	}
 	if nameLower == "" {
 		return true
 	}