From 0bff6ef5fd3ba89babbf8740b14a6c42ab31eb92 Mon Sep 17 00:00:00 2001 From: Steve Yegge Date: Tue, 14 Oct 2025 03:29:41 -0700 Subject: [PATCH] polish: Trim whitespace in --deps flag parsing Handle edge cases in dependency spec parsing: - Skip empty dependency specs (e.g., from trailing commas) - Trim whitespace around type and ID (e.g., 'discovered-from: bd-20') This makes the flag more forgiving of user input errors. --- cmd/bd/main.go | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/cmd/bd/main.go b/cmd/bd/main.go index 984d26c4..3df4998a 100644 --- a/cmd/bd/main.go +++ b/cmd/bd/main.go @@ -588,6 +588,12 @@ var createCmd = &cobra.Command{ // Add dependencies if specified (format: type:id or just id for default "blocks" type) for _, depSpec := range deps { + // Skip empty specs (e.g., from trailing commas) + depSpec = strings.TrimSpace(depSpec) + if depSpec == "" { + continue + } + var depType types.DependencyType var dependsOnID string @@ -598,8 +604,8 @@ var createCmd = &cobra.Command{ fmt.Fprintf(os.Stderr, "Warning: invalid dependency format '%s', expected 'type:id' or 'id'\n", depSpec) continue } - depType = types.DependencyType(parts[0]) - dependsOnID = parts[1] + depType = types.DependencyType(strings.TrimSpace(parts[0])) + dependsOnID = strings.TrimSpace(parts[1]) } else { // Default to "blocks" if no type specified depType = types.DepBlocks