Skip to content

Validate --status and --type values in sd list, search, and ready#90

Merged
jayminwest merged 2 commits into
mainfrom
warren/run_1kp421xa7hd6
Jun 16, 2026
Merged

Validate --status and --type values in sd list, search, and ready#90
jayminwest merged 2 commits into
mainfrom
warren/run_1kp421xa7hd6

Conversation

@jayminwest

Copy link
Copy Markdown
Owner

Summary

feat(list/search/ready): validate --status and --type flag values (seeds-d305)

Run

  • Warren run: run_1kp421xa7hd6
  • Agent: pi
  • Cost: $0.771 (40 in / 7.8k out / 809.1k cache-r)

Seeds

  • seeds-d305 — Validate --status and --type values in sd list, search, and ready

Commits (2)

  • 4d966cb feat(list/search/ready): validate --status and --type flag values (seeds-d305)
  • 6426e99 chore(warren): seeds state

Files changed

.canopy/agent.json                                 |  1 +
 ...270Z_019ed01f-1826-7eaf-9b33-d3e7c8dc88bf.jsonl | 69 ++++++++++++++++++++++
 .seeds/issues.jsonl                                |  6 +-
 src/commands/list.test.ts                          | 40 +++++++++++++
 src/commands/list.ts                               | 22 +++++++
 src/commands/ready.test.ts                         | 23 ++++++++
 src/commands/ready.ts                              | 12 ++++
 src/commands/search.test.ts                        | 46 +++++++++++++++
 src/commands/search.ts                             | 22 +++++++
 9 files changed, 238 insertions(+), 3 deletions(-)

Prompt

Show prompt
work on sd seeds-d305

🤖 Opened by warren run run_1kp421xa7hd6

warren-agent and others added 2 commits June 16, 2026 11:12
…eds-d305)

Add up-front validation for --status and --type in sd list, search, and
ready, mirroring the idiom already used by sd update. Invalid values now
exit 1 with 'Invalid --<flag> value: X. Valid: a|b|c' on stderr (or a
{success:false, command, error} envelope under --json) instead of
silently filtering to an empty result.

Tests cover both human and --json paths for each command.

Refs: pl-a786 step 1
@jayminwest jayminwest enabled auto-merge (squash) June 16, 2026 11:13
@jayminwest jayminwest merged commit 476111d into main Jun 16, 2026
2 checks passed
@jayminwest jayminwest deleted the warren/run_1kp421xa7hd6 branch June 16, 2026 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant