Skip to content

feat(scope): Scoped user feedback capture#1916

Open
limbonaut wants to merge 3 commits into
limbonaut/feat/apply-scope-to-feedbackfrom
limbonaut/feat/scoped-feedback-capture
Open

feat(scope): Scoped user feedback capture#1916
limbonaut wants to merge 3 commits into
limbonaut/feat/apply-scope-to-feedbackfrom
limbonaut/feat/scoped-feedback-capture

Conversation

@limbonaut

@limbonaut limbonaut commented Jul 26, 2026

Copy link
Copy Markdown
Collaborator

Adds scoped capture of user feedback by introducing the sentry_scope_capture_feedback function, joining the existing family of sentry_scope_capture_x functions. It layers the given scope on top of the global scope when building the feedback event, so a single capture can carry its own tags, user, and contexts without mutating global state.

@github-actions

github-actions Bot commented Jul 26, 2026

Copy link
Copy Markdown
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 4f57ca8

Comment thread include/sentry.h
@codecov

codecov Bot commented Jul 26, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 92.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.66%. Comparing base (b834d70) to head (4f57ca8).

Additional details and impacted files
@@                            Coverage Diff                             @@
##           limbonaut/feat/apply-scope-to-feedback    #1916      +/-   ##
==========================================================================
+ Coverage                                   75.63%   75.66%   +0.03%     
==========================================================================
  Files                                          93       93              
  Lines                                       22039    22056      +17     
  Branches                                     3925     3927       +2     
==========================================================================
+ Hits                                        16669    16689      +20     
+ Misses                                       4487     4485       -2     
+ Partials                                      883      882       -1     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@limbonaut
limbonaut force-pushed the limbonaut/feat/scoped-feedback-capture branch from 4f57ca8 to 7f225fa Compare July 26, 2026 13:34
@limbonaut
limbonaut force-pushed the limbonaut/feat/apply-scope-to-feedback branch from 1073ed7 to b834d70 Compare July 26, 2026 13:56
@limbonaut
limbonaut force-pushed the limbonaut/feat/scoped-feedback-capture branch from 7f225fa to e6f1d00 Compare July 26, 2026 13:56

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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 e6f1d00. Configure here.

Comment thread src/sentry_core.c Outdated
options->run->cache_path, options->run->run_path);
}
}
sentry__attachments_free(all_attachments);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Local scope attachments dropped

Medium Severity

prepare_user_feedback applies and frees the passed-in scope but never merges local_scope->attachments into the envelope. Unlike sentry__prepare_event, only hint and global-scope attachments are included, so attachments set via sentry_scope_attach_* on the capture scope are silently omitted from scoped feedback.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit e6f1d00. Configure here.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This needs to be decided in #1915

Comment thread src/sentry_core.c
@limbonaut
limbonaut force-pushed the limbonaut/feat/scoped-feedback-capture branch from e6f1d00 to 4f57ca8 Compare July 26, 2026 14:01
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.

2 participants