Skip to content

fix(errors): separate informational notes from actionable alternatives in ContextError#651

Merged
BYK merged 1 commit intomainfrom
fix/context-error-note
Apr 3, 2026
Merged

fix(errors): separate informational notes from actionable alternatives in ContextError#651
BYK merged 1 commit intomainfrom
fix/context-error-note

Conversation

@BYK
Copy link
Copy Markdown
Member

@BYK BYK commented Apr 3, 2026

Summary

  • Adds an optional note parameter to ContextError for diagnostic context, rendered as a separate "Note:" section after the actionable "Or:" alternatives
  • Fixes the CLI-VN error where the "Or:" section contained non-actionable DSN resolution info instead of steps the user can take
  • Moves DSN diagnostic info from alternatives to note in issue list and project view commands

Before:

Or:
  - Found 2 DSN(s) that could not be resolved
  - You may not have access to these projects, or you can specify the target explicitly

After:

Or:
  - Run from a directory with a Sentry-configured project
  - Set SENTRY_ORG and SENTRY_PROJECT (or SENTRY_DSN) environment variables
  - Run 'sentry org list' to find your organization slug
  - Run 'sentry project list <org>/' to find project slugs

Note: Found 2 DSN(s) that could not be resolved — you may not have access to these projects

Fixes CLI-VN

…s in ContextError

Add optional `note` parameter to ContextError for diagnostic context,
rendered as a "Note:" section after the actionable "Or:" alternatives.
Fixes CLI-VN where the Or: section contained non-actionable DSN info.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026

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).


Bug Fixes 🐛

  • (errors) Separate informational notes from actionable alternatives in ContextError by BYK in #651

Internal Changes 🔧

  • Regenerate skill files and command docs by github-actions[bot] in eb1b19e7

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026

Codecov Results 📊

134 passed | Total: 134 | 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 1376 uncovered lines.
✅ Project coverage is 95.64%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    95.64%    95.64%        —%
==========================================
  Files          219       219         —
  Lines        31540     31551       +11
  Branches         0         0         —
==========================================
+ Hits         30164     30175       +11
- Misses        1376      1376         —
- Partials         0         0         —

Generated by Codecov Action

@BYK BYK merged commit a5df9c3 into main Apr 3, 2026
24 checks passed
@BYK BYK deleted the fix/context-error-note branch April 3, 2026 12:49
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