Skip to content

test(cli): Add regression tests for space-separated issue arguments (CLI-1G1) - #2084

Closed
sentry[bot] wants to merge 1 commit into
mainfrom
seer/test-issue-args-space-regression
Closed

sentry[bot] wants to merge 1 commit into
mainfrom
seer/test-issue-args-space-regression

Conversation

@sentry

@sentry sentry Bot commented Nov 7, 2026

Copy link
Copy Markdown
Contributor

Issue arguments that contain whitespace (such as agent input combining a short ID and a numeric ID, multiple IDs passed as one argument, or search queries) previously threw ValidationError: Invalid issue identifier: contains a space. via validateResourceId.

The parsing changes in PRs #1729, #1730, and #1747 (shipped in 0.48.0) addressed the underlying behavior by routing whitespace-containing inputs to issue search in single-issue parsers and splitting them on whitespace in multi-issue parsers.

This change adds regression tests in packages/cli/test/lib/issue-arguments.test.ts to ensure these cases (parseIssueArgument for searches like "CLI-1G1 7441717161" and parseIssueArguments splitting space-separated IDs) remain covered.

Fixes CLI-1G1

@sentry <feedback>: Autofix iterates on these changes
@sentry stop iterating: Autofix stops iterating on this run

This PR was automatically generated by Sentry. You can adjust this setting at any time.

@vercel

vercel Bot commented Nov 7, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
sentry-local Skipped Skipped Nov 7, 2026 10:47am UTC

Request Review

@github-actions github-actions Bot added the risk: low PR risk score: low label Nov 7, 2026
@sentry-junior

sentry-junior Bot commented Nov 7, 2026

Copy link
Copy Markdown
Contributor

Closing this as redundant. It only adds tests, and main already covers the same behavior.

The new cases repeat these checks with different strings. A failure they would catch is already caught by the tests above. Merging this would also mark CLI-1G1 fixed by a PR that changes no code.

@sentry-junior sentry-junior Bot closed this Nov 7, 2026

This branch was previously deployed

1 inactive deployment
Preview — c160ec26 Deployed Nov 7, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: low PR risk score: low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants