Skip to content

fix: event 404 hint should suggest different project, not repeat failing command#492

Merged
BYK merged 1 commit intomainfrom
fix/cli-6f-hint-followup
Mar 20, 2026
Merged

fix: event 404 hint should suggest different project, not repeat failing command#492
BYK merged 1 commit intomainfrom
fix/cli-6f-hint-followup

Conversation

@BYK
Copy link
Member

@BYK BYK commented Mar 20, 2026

Follow-up to PR #491

Addresses unresolved Bugbot review comment that I missed before merging #491.

Problem

The Try: hint in fetchEventWithContext repeated the exact command that just produced the 404:

Event 'abc123' not found in my-org/my-project.

Try:
  sentry event view my-org/my-project abc123   ← same failing command!

Fix

Changed the hint to use a <project> placeholder to suggest trying a different project, and updated the Or: suggestion to explicitly mention org-wide search:

Event 'abc123' not found in my-org/my-project.

Try:
  sentry event view my-org/<project> abc123     ← try a different project

Or:
  - The event may have been deleted due to data retention policies
  - Verify the event ID is a 32-character hex string (e.g., a1b2c3d4...)
  - Search across all projects in the org: sentry event view my-org/ abc123

…ing command

Address unresolved Bugbot review from PR #491: the 'Try:' hint in
fetchEventWithContext repeated the exact command that just failed.
Changed to use a <project> placeholder so it suggests trying a
different project, and updated the suggestion bullet to explicitly
say 'Search across all projects in the org'.
@github-actions
Copy link
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
  • 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
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 1044 uncovered lines.
❌ Project coverage is 95.76%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
- Coverage    95.77%    95.76%    -0.01%
==========================================
  Files          180       180         —
  Lines        24622     24622         —
  Branches         0         0         —
==========================================
+ Hits         23580     23578        -2
- Misses        1042      1044        +2
- Partials         0         0         —

Generated by Codecov Action

@BYK BYK merged commit aaabc30 into main Mar 20, 2026
22 checks passed
@BYK BYK deleted the fix/cli-6f-hint-followup branch March 20, 2026 00:58
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