This is not good because we mutate the scope, so that will also affect other events that have the same scope (unless I am missing something and we are always forking a scope for this scenario?). So we should do one of these:

1. Fork a scope here, and mutate that one:

withScope(scope => 
  scope.setLevel(...);
  prepareFeedbackEvent({ scope, ... })
})

2. put the level directly on the feedback event somehow

Good catch! I updated to use pushScope and popScope -- I don't love the withScope API.... I wish withScope returned the new scope.

I'd recommend to use withScope instead (we'll probably get rid of pushScope in v8):

withScope((scope) => {
  scope.clearBreadcrumbs(); 

    if ([FEEDBACK_API_SOURCE, FEEDBACK_WIDGET_SOURCE].includes(String(source))) {
      scope.setLevel('info');
    }

    const feedbackEvent = ...
});

Updated using withScope, had to create a promise because I need the return value inside of withScope callback

just a note, nothing to do here, but this may change a bit in v8. We're thinking to rework how/where we keep breadcrumbs, in the browser we'll probably keep them globally. At this point we'll have to change this logic here. Feel free to leave this for later, but if there is an easy way to make this forwards-compatible, we may as well do it already now 👍

Good to know, would it be better to delete breadcrumbs from the event rather than the scope?