Skip to content

fix: add 400 suggestions to org-all issue list path (CLI-BY)#497

Merged
BYK merged 1 commit intomainfrom
fix/cli-by-orgall-400
Mar 20, 2026
Merged

fix: add 400 suggestions to org-all issue list path (CLI-BY)#497
BYK merged 1 commit intomainfrom
fix/cli-by-orgall-400

Conversation

@BYK
Copy link
Copy Markdown
Member

@BYK BYK commented Mar 20, 2026

Problem

The org-all code path (sentry issue list org/) was missing the 400 Bad Request suggestions added in PR #489. When the API returns 400, users see a bare error without guidance. Affects 23 users (CLI-BY).

Fix

Wrapped the fetchOrgAllIssues call in handleOrgAllIssues with a try-catch that enriches 400 errors using the existing build400Detail() helper — the same one used by handleResolvedTargets. Users now see suggestions about query syntax, period, and project access in both code paths.

The org-all code path (sentry issue list org/) was missing the 400
suggestions added in PR #489 for the multi-target path. When the API
returns 400 Bad Request in org-all mode, users now see the same
context-aware suggestions about query syntax, period, and access.
Affects 23 users (CLI-BY).
@github-actions
Copy link
Copy Markdown
Contributor

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • (telemetry) Track TTY vs non-TTY invocations via metric by betegon in #482
  • Dynamic cache-backed shell completions with fuzzy matching by BYK in #465

Bug Fixes 🐛

  • (project) Fallback to org listing when bare slug matches an organization by betegon in #475
  • Add 400 suggestions to org-all issue list path (CLI-BY) by BYK in #497
  • Lowercase project slug in issue arg parsing (CLI-C8) by BYK in #496
  • Enrich short ID 404 with org context and suggestions (CLI-A1) by BYK in #494
  • Suggest similar projects when project not found in org (CLI-C0) by BYK in #493
  • Event 404 hint should suggest different project, not repeat failing command by BYK in #492
  • Enrich event 404 errors with retention and format suggestions (CLI-6F) by BYK in #491
  • Add actionable suggestions for 400 Bad Request on issue list (CLI-BM, CLI-7B) by BYK in #489
  • Detect issue short IDs passed to issue list (CLI-C3) by BYK in #488
  • Add Glob.match() polyfill + improve auto-detect diagnostics (CLI-7T) by BYK in #487
  • Add org-slug pre-check to dispatchOrgScopedList (CLI-9A) by BYK in #485

Internal Changes 🔧

  • (issue) Skip getProject round-trip in project-search resolution by betegon in #473
  • (resolve) Carry project data through resolution to eliminate redundant getProject calls by BYK in #486
  • HTTP latency optimizations — diagnostics, cache warming, concurrency limits by BYK in #490
  • Switch from @sentry/bun to @sentry/node-core/light (~170ms startup savings) by BYK in #474
  • Regenerate skill files by github-actions[bot] in b7b240ec

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Copy Markdown
Contributor

Codecov Results 📊

126 passed | Total: 126 | Pass Rate: 100% | Execution Time: 0ms

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests
Failed Tests
Skipped Tests

✨ No test changes detected

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 1056 uncovered lines.
✅ Project coverage is 95.72%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    95.72%    95.72%        —%
==========================================
  Files          180       180         —
  Lines        24679     24686        +7
  Branches         0         0         —
==========================================
+ Hits         23623     23630        +7
- Misses        1056      1056         —
- Partials         0         0         —

Generated by Codecov Action

@BYK BYK merged commit a8e4fbb into main Mar 20, 2026
22 checks passed
@BYK BYK deleted the fix/cli-by-orgall-400 branch March 20, 2026 09:24
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