chore: fixes

Author: kirrg001

packages/collector/src/announceCycle/agentready.js

@@ -152,18 +152,18 @@ function enter(_ctx) {
   // TODO: Add an EventEmitter functionality for the current process
   //       such as `instana.on('instana.collector.initialized')`.
   // eslint-disable-next-line no-unused-expressions
-  // process?.send?.('instana.collector.initialized');
+  if (process.env.INSTANA_IPC_ENABLED === 'true') {
+    process?.send?.('instana.collector.initialized');
 
-  /*
-  if (!isMainThread) {
-    const { parentPort } = require('worker_threads');
+    if (!isMainThread) {
+      const { parentPort } = require('worker_threads');
 
-    if (parentPort) {
-      // CASE: This is for the worker thread if available.
-      parentPort.postMessage('instana.collector.initialized');
+      if (parentPort) {
+        // CASE: This is for the worker thread if available.
+        parentPort.postMessage('instana.collector.initialized');
+      }
     }
   }
-  */
 }
 
 function leave() {

packages/collector/src/index.js

@@ -100,7 +100,9 @@ function init(userConfig = {}) {
     }
 
     if (collectorIndexCacheKey) {
-      // process?.send?.('instana.collector.initialized');
+      if (process.env.INSTANA_IPC_ENABLED === 'true') {
+        process?.send?.('instana.collector.initialized');
+      }
 
       return require.cache[collectorIndexCacheKey].exports;
     } else {

packages/collector/test/test_util/ProcessControls.js

@@ -151,7 +151,8 @@ class ProcessControls {
         INSTANA_FIRE_MONITORING_EVENT_DURATION_IN_MS: 500,
         INSTANA_RETRY_AGENT_CONNECTION_IN_MS: 500,
         APP_USES_HTTPS: this.appUsesHttps ? 'true' : 'false',
-        INSTANA_DISABLE_USE_OPENTELEMETRY: !this.enableOtelIntegration
+        INSTANA_DISABLE_USE_OPENTELEMETRY: !this.enableOtelIntegration,
+        INSTANA_IPC_ENABLED: 'true'
       },
       opts.env
     );

packages/core/src/util/requireHook.js

@@ -66,18 +66,20 @@ function patchedModuleLoad(moduleName) {
   //       However, when an ESM library imports a CommonJS package, our requireHook is triggered.
   //       For native ESM libraries the iitmHook is triggered.
   if (path.isAbsolute(moduleName) && ['.node', '.json', '.ts'].indexOf(path.extname(moduleName)) === -1) {
+    // Normalize Windows paths (backslashes) to forward slashes for regex matching
+    const normalizedModuleName = moduleName.replace(/\\/g, '/');
     // EDGE CASE for ESM: mysql2/promise.js
-    if (moduleName.indexOf('node_modules/mysql2/promise.js') !== -1) {
+    if (normalizedModuleName.indexOf('node_modules/mysql2/promise.js') !== -1) {
       moduleName = 'mysql2/promise';
     } else {
       // e.g. path is node_modules/@elastic/elasicsearch/index.js
-      let match = moduleName.match(/node_modules\/(@.*?(?=\/)\/.*?(?=\/))/);
+      let match = normalizedModuleName.match(/node_modules\/(@.*?(?=\/)\/.*?(?=\/))/);
 
       if (match && match.length > 1) {
         moduleName = match[1];
       } else {
         // e.g. path is node_modules/mysql/lib/index.js
-        match = moduleName.match(/node_modules\/(.*?(?=\/))/);
+        match = normalizedModuleName.match(/node_modules\/(.*?(?=\/))/);
 
         if (match && match.length > 1) {
           moduleName = match[1];
@@ -145,8 +147,11 @@ function patchedModuleLoad(moduleName) {
   }
 
   if (!cacheEntry.byFileNamePatternTransformersApplied) {
+    // Normalize Windows paths (backslashes) to forward slashes for regex pattern matching
+    // This ensures patterns with forward slashes (like /\/mongodb-core\/lib\/connection\/pool\.js/) work on Windows
+    const normalizedFilename = filename.replace(/\\/g, '/');
     for (let i = 0; i < byFileNamePatternTransformers.length; i++) {
-      if (byFileNamePatternTransformers[i].pattern.test(filename)) {
+      if (byFileNamePatternTransformers[i].pattern.test(normalizedFilename)) {
         cacheEntry.moduleExports =
           byFileNamePatternTransformers[i].fn(cacheEntry.moduleExports, filename) || cacheEntry.moduleExports;
       }
@@ -191,15 +196,27 @@ exports.buildFileNamePattern = function buildFileNamePattern(arr) {
   return new RegExp(`${arr.reduce(buildFileNamePatternReducer, '')}$`);
 };
 
+/**
+ * Escapes special regex characters in a string
+ * @param {string} str
+ * @returns {string}
+ */
+function escapeRegex(str) {
+  return str.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+}
+
 /**
  * @param {string} agg
  * @param {string} pathSegment
  * @returns {string}
  */
 function buildFileNamePatternReducer(agg, pathSegment) {
   if (agg.length > 0) {
-    agg += `\\${path.sep}`;
+    // Always use forward slashes in patterns since we normalize filenames to forward slashes
+    // This ensures patterns work consistently on both Windows and Unix systems
+    agg += '\\/';
   }
-  agg += pathSegment;
+  // Escape special regex characters in path segments (e.g., '.' in 'express.js' should be '\.')
+  agg += escapeRegex(pathSegment);
   return agg;
 }