From 2ea9f699aeb5fd41102eb3df04f6e91f7f1d8077 Mon Sep 17 00:00:00 2001
From: Abhijeet Prasad <aprasad@sentry.io>
Date: Wed, 7 Sep 2022 15:14:34 +0200
Subject: [PATCH] feat(node): Add Modules integration to default integrations

Start adding module information to NodeJS events. This is important so
we can start understanding what kind of packages people use, and better
inform strategies around future integration development + opentelemetry.
---
 packages/node/src/integrations/modules.ts | 5 ++++-
 packages/node/src/sdk.ts                  | 7 +++++--
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/packages/node/src/integrations/modules.ts b/packages/node/src/integrations/modules.ts
index df8bcc16d261..5140c5ee253a 100644
--- a/packages/node/src/integrations/modules.ts
+++ b/packages/node/src/integrations/modules.ts
@@ -87,7 +87,10 @@ export class Modules implements Integration {
       }
       return {
         ...event,
-        modules: this._getModules(),
+        modules: {
+          ...event.modules,
+          ...this._getModules(),
+        },
       };
     });
   }
diff --git a/packages/node/src/sdk.ts b/packages/node/src/sdk.ts
index db1e7e883ced..02566bc22f6c 100644
--- a/packages/node/src/sdk.ts
+++ b/packages/node/src/sdk.ts
@@ -24,6 +24,7 @@ import {
   ContextLines,
   Http,
   LinkedErrors,
+  Modules,
   OnUncaughtException,
   OnUnhandledRejection,
 } from './integrations';
@@ -35,16 +36,18 @@ export const defaultIntegrations = [
   // Common
   new CoreIntegrations.InboundFilters(),
   new CoreIntegrations.FunctionToString(),
-  new ContextLines(),
   // Native Wrappers
   new Console(),
   new Http(),
   // Global Handlers
   new OnUncaughtException(),
   new OnUnhandledRejection(),
+  // Event Info
+  new ContextLines(),
+  new Context(),
+  new Modules(),
   // Misc
   new LinkedErrors(),
-  new Context(),
 ];
 
 /**