refactor: streamline beforeSend and beforeBreadcrumb hook handling by renaming variables for clarity

Author: Dobrunia

packages/javascript/src/addons/breadcrumbs.ts

@@ -237,20 +237,20 @@ export class BreadcrumbManager {
      */
     if (this.options.beforeBreadcrumb) {
       const breadcrumbClone = structuredClone(bc);
-      const modified = this.options.beforeBreadcrumb(breadcrumbClone, hint);
+      const result = this.options.beforeBreadcrumb(breadcrumbClone, hint);
 
       /**
        * false means discard
        */
-      if (modified === false) {
+      if (result === false) {
         return;
       }
 
       /**
        * Valid breadcrumb → apply changes from hook
        */
-      if (isValidBreadcrumb(modified)) {
-        Object.assign(bc, modified);
+      if (isValidBreadcrumb(result)) {
+        Object.assign(bc, result);
       } else {
         /**
          * Anything else is invalid — warn, bc stays untouched (hook only received a clone)

packages/javascript/src/catcher.ts

@@ -439,20 +439,20 @@ export default class Catcher {
      */
     if (typeof this.beforeSend === 'function') {
       const eventClone = structuredClone(payload);
-      const modified = this.beforeSend(eventClone);
+      const result = this.beforeSend(eventClone);
 
       /**
        * false → drop event
        */
-      if (modified === false) {
+      if (result === false) {
         throw new EventRejectedError('Event rejected by beforeSend method.');
       }
 
       /**
        * Valid event payload → use it instead of original
        */
-      if (isValidEventPayload(modified)) {
-        payload = modified as HawkJavaScriptEvent;
+      if (isValidEventPayload(result)) {
+        payload = result as HawkJavaScriptEvent;
       } else {
         /**
          * Anything else is invalid — warn, payload stays untouched (hook only received a clone)

packages/javascript/src/modules/socket.ts

@@ -1,12 +1,13 @@
 import log from '../utils/log';
 import type { CatcherMessage } from '@/types';
+import type { Transport } from '../types/transport';
 
 /**
  * Custom WebSocket wrapper class
  *
  * @copyright CodeX
  */
-export default class Socket {
+export default class Socket implements Transport {
   /**
    * Socket connection endpoint
    */