Skip to content

feat(issue): batch view and explain commands - #1609

Merged
betegon merged 4 commits into
mainfrom
feat/issue-view-batch
Sep 28, 2026
Merged

betegon merged 4 commits into
mainfrom
feat/issue-view-batch

Conversation

@betegon

@betegon betegon commented Sep 21, 2026 •

Copy link
Copy Markdown
Member

Summary

sentry issue view and sentry issue explain now accept space- or newline-separated issue IDs with bounded parallel work and deterministic batch output.

Single-issue contracts do not change: view --json remains a flat issue object, while explain --json remains an array of root causes. Multi-issue view returns an array of those issue objects; multi-issue explain returns labeled envelopes (issue, org, issueId, rootCauses). Partial failures warn on stderr without changing the JSON shape, and all-failed batches rethrow the primary error.

issue view --web opens every requested issue up to a five-tab safety cap. --force bypasses the cap. -f remains the repository-wide --fresh alias instead of acquiring a second meaning.

The shared single-issue positional remains unchanged for plan, resolve, unresolve, archive, and events; batch commands use a separate variadic positional. issue list --expand remains out of scope for 1b.

Validation

  • TZ=UTC pnpm exec vitest run test/lib test/commands test/types test/script — 9,819 passed, 17 skipped
  • pnpm run typecheck
  • pnpm run lint
  • pnpm run check:fragments — valid; existing unrelated proguard upload coverage warning only
  • Generated command docs, agent skill, and SDK
  • Clean merge-tree against current origin/main

Agents currently loop `issue view` per ID. Match `event view` with
space-separated/newline IDs and a JSON array only for multi-id calls,
without changing the shared single-id positional used by explain/plan/resolve.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Sep 21, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
cli Ready Ready Preview Sep 28, 2026 9:16am UTC
1 Skipped Deployment
Project Deployment Actions Updated
sentry-local Skipped Skipped Sep 28, 2026 9:16am UTC

Request Review

@BYK

BYK commented Sep 21, 2026

Copy link
Copy Markdown
Member

sentry issue view A B -w opens only A and warns that extras are ignored

I'd say it should open all up to a limit and the limit can be bypassed with -f

@BYK

BYK commented Sep 21, 2026

Copy link
Copy Markdown
Member

sentry issue explain A B is still single-id (shared positional untouched)

I'd also make this support multiple issues

Open multiple issue pages with a five-tab safety cap, and allow an explicit
--force override without repurposing the shared -f/--fresh convention. Add
batch root-cause analysis while preserving every single-issue output shape.

Extract issue rendering and share bounded batch orchestration to keep the
command implementations focused.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel
vercel Bot temporarily deployed to Preview – sentry-local September 22, 2026 18:30 Inactive
@betegon betegon changed the title feat(issue): view multiple issues in one invocation feat(issue): batch view and explain commands Sep 22, 2026
@betegon

betegon commented Sep 22, 2026

Copy link
Copy Markdown
Member Author

Addressed both points from @BYK:

  • issue view A B -w now opens both. The default safety cap is 5 tabs, and --force opens all requested issues. I kept -f bound to the shared --fresh flag used across read-only commands, so the override is long-form --force rather than giving -f two meanings.
  • issue explain A B now batches too, with the same space/newline input, concurrency bound, partial-failure behavior, and single-invocation compatibility guarantees as issue view.

The cleanup also fixed the CI trace nullability error, extracted issue formatting from the command, and centralized issue-batch orchestration.

@betegon
betegon marked this pull request as ready for review September 28, 2026 08:58
@github-actions github-actions Bot added the risk: medium PR risk score: medium label Sep 28, 2026
Resolve the issue utility import conflict by preserving both main's shared
issue URL parsing types and the PR's variadic batch argument helper.

Co-authored-by: Cursor <cursoragent@cursor.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 5cd800a. Configure here.

Comment thread packages/cli/src/commands/issue/utils.ts
Only report per-item warnings when a batch has at least one successful result.
For total failure, rethrow the primary error directly so stderr contains one
formatted error instead of a warning plus the same error.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel
vercel Bot temporarily deployed to Preview – sentry-local September 28, 2026 09:16 Inactive
@betegon
betegon merged commit e37a425 into main Sep 28, 2026
37 checks passed
@betegon
betegon deleted the feat/issue-view-batch branch September 28, 2026 09:50

This branch was successfully deployed

1 active and 1 inactive deployments
Preview – cli — d73b2e58 Deployed Sep 28, 2026 by vercel[bot]
Preview – sentry-local — d73b2e58 Deployed Sep 28, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: medium PR risk score: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants