feat(scope): Only set lastEventId for error events

Author: andreiborza

packages/core/src/baseclient.ts

@@ -650,9 +650,8 @@ export abstract class BaseClient<O extends ClientOptions> implements Client<O> {
 
     this.emit('preprocessEvent', event, hint);
 
-    const eventId = event.event_id || hint.event_id;
-    if (eventId) {
-      isolationScope.setLastEventId(eventId);
+    if (!event.type) {
+      isolationScope.setLastEventId(event.event_id || hint.event_id);
     }
 
     return prepareEvent(options, event, hint, currentScope, this, isolationScope).then(evt => {

packages/core/src/exports.ts

@@ -123,10 +123,10 @@ export function setUser(user: User | null): void {
 /**
  * The last error event id of the isolation scope.
  *
- * Warning: This function really returns the last recorded error event id on the current 
- * isolation scope.  If you call this function after handling a certain error and another error 
- * is captured in between, the last one is returned instead of the one you might expect. 
- * Also, ids of events that were never sent to Sentry (for example because 
+ * Warning: This function really returns the last recorded error event id on the current
+ * isolation scope. If you call this function after handling a certain error and another error
+ * is captured in between, the last one is returned instead of the one you might expect.
+ * Also, ids of events that were never sent to Sentry (for example because
  * they were dropped in `beforeSend`) could be returned.
  *
  * @returns The last event id of the isolation scope.