fix: support comma-separated webhook events in --events flag

[felipefreitag]

No description provided.

[cubic-dev-ai]

cubic analysis

2 issues found across 5 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="src/commands/webhooks/update.ts">

<violation number="1" location="src/commands/webhooks/update.ts:64">
P2: Comma-separated input containing `all` is not handled correctly; `all` is passed as a literal event instead of expanding to all webhook events. According to linked Linear issue ENG-4700, comma-separated events should be supported, so `all,email.bounced` should also be interpreted correctly.</violation>
</file>

<file name="src/commands/webhooks/listen.ts">

<violation number="1" location="src/commands/webhooks/listen.ts:188">
P2: Comma-separated input containing `all` is not handled correctly because `all` detection happens before normalization.</violation>
</file>

Linked issue analysis

Linked issue: ENG-4700: - webhooks create: make it clearer events are separated by spaces, maybe support commas

Status	Acceptance criteria	Notes
✅	Support comma-separated webhook events in --events flag	Added normalizeEvents and wired it into create/listen/update
✅	Accept mixed comma and space-separated events	normalizeEvents handles comma split and multiple args; test added
✅	Trim spaces around comma-separated events	normalizeEvents trims entries and test verifies trimming
✅	Update help text and examples to indicate comma or space-separated events	Option descriptions and examples updated to mention commas
✅	Ensure listen and update commands also support comma-separated events	listen/update now import and use normalizeEvents like create
✅	Add tests verifying comma-splitting behavior	Three tests added covering comma split, mixed, and trimming
❌	Improve error message to hint about space vs comma separation when user passes comma-separated input	No changes to error handling or user-facing error messages

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}