Skip to content

feat(feedback): add list and view commands#1266

Open
betegon wants to merge 10 commits into
mainfrom
bt/feat-feedback-list-view
Open

feat(feedback): add list and view commands#1266
betegon wants to merge 10 commits into
mainfrom
bt/feat-feedback-list-view

Conversation

@betegon

@betegon betegon commented Jul 17, 2026

Copy link
Copy Markdown
Member

Summary

  • add the top-level sentry feedback route with list and default view commands
  • query modern User Feedback through the issue index with a mandatory issue.category:feedback filter, mailbox mapping, target resolution, and bidirectional pagination
  • show Feedback details with best-effort latest-event, replay, and attachment enrichment
  • add markdown-first formatters, generated CLI skill docs, property/unit tests, and mocked E2E coverage
  • fix shared issue auto-pagination so non-multiple limits preserve the next cursor

Part of #1262

Test plan

  • pnpm run generate:docs
  • pnpm run typecheck
  • pnpm run lint
  • pnpm run test:unit
  • pnpm run test:e2e
  • 182 focused Feedback and shared-regression tests
  • 7 completion/route drift tests
  • pnpm run check:deps
  • pnpm run check:errors
  • pnpm run check:fragments
  • pnpm run check:docs-sections
  • CI: unit, E2E, generated files, binary/npm builds, docs, and security checks all passing

@github-actions

github-actions Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://cli.sentry.dev/_preview/pr-1266/

Built to branch gh-pages at 2026-07-21 19:09 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@github-actions

github-actions Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

✅ Patch coverage is 87.84%. Project has 5504 uncovered lines.
✅ Project coverage is 81.75%. Comparing base (base) to head (head).

Files with missing lines (5)
File Patch % Lines
src/lib/formatters/feedback.ts 89.08% ⚠️ 13 Missing and 10 partials
src/commands/feedback/utils.ts 45.83% ⚠️ 13 Missing and 2 partials
src/commands/feedback/list.ts 92.68% ⚠️ 3 Missing and 4 partials
src/commands/feedback/view.ts 100.00% ⚠️ 3 partials
src/lib/api/feedback.ts 71.43% ⚠️ 2 Missing
Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    81.68%    81.75%    +0.07%
==========================================
  Files          426       433        +7
  Lines        29909     30160      +251
  Branches     19433     19642      +209
==========================================
+ Hits         24431     24656      +225
- Misses        5478      5504       +26
- Partials      2035      2054       +19

Generated by Codecov Action

@betegon
betegon marked this pull request as ready for review July 21, 2026 14:27
@github-actions github-actions Bot added the risk: high PR risk score: high label Jul 21, 2026
Comment thread src/commands/feedback/utils.ts
Comment thread src/lib/formatters/feedback.ts Outdated
Sanitize untrusted feedback and event data before terminal rendering, preserve limit-scoped cursor navigation, and fetch attachment metadata across bounded API pages.
Comment thread src/lib/formatters/feedback.ts Fixed
Comment thread src/commands/feedback/view.ts
Comment thread src/lib/api/issues.ts
Comment thread src/commands/issue/utils.ts Outdated
Comment thread src/lib/sentry-url-parser.ts
Comment thread src/lib/sentry-url-parser.ts
Comment thread src/lib/api/events.ts

@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 7901a55. Configure here.

Comment thread src/lib/formatters/feedback.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: high PR risk score: high

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants