@drawbridge/drawbridge-telemetry 0.0.3 → 0.0.4

package/dist/express.cjs

@@ -35,29 +35,28 @@ __export(express_exports, {
 module.exports = __toCommonJS(express_exports);
 var Sentry = __toESM(require("@sentry/core"), 1);
 var applyRequestContext = (req) => {
-  const scope = Sentry.getCurrentScope();
   if (req == null ? void 0 : req.id) {
-    scope.setTag("traceId", req.id);
+    Sentry.setTag("traceId", req.id);
   }
   ;
   if (req == null ? void 0 : req.method) {
-    scope.setTag("method", req.method);
+    Sentry.setTag("method", req.method);
   }
   ;
   if (req == null ? void 0 : req.path) {
-    scope.setTag("route", req.path);
+    Sentry.setTag("route", req.path);
   }
   ;
   const user = (req == null ? void 0 : req.authenticated) || (req == null ? void 0 : req.user);
   if (user) {
-    scope.setUser({
+    Sentry.setUser({
       id: user.id || user._id,
       email: user.email
     });
   }
   ;
   if (req == null ? void 0 : req.organization) {
-    scope.setTag("organization", req.organization.id || req.organization._id);
+    Sentry.setTag("organization", req.organization.id || req.organization._id);
   }
   ;
 };

package/dist/express.d.cts

@@ -1,25 +1,30 @@
 import * as Sentry from '@sentry/core';
 
-// Apply all available context tags from a request to the active Sentry
-// scope. Idempotent — safe to call multiple times as the request enriches
-// (e.g. once at request entry for traceId/method/path, again from auth
-// middleware once req.authenticated and req.organization are populated).
+// Apply all available context tags from a request to the per-request
+// Sentry isolation scope. Idempotent — safe to call multiple times as
+// the request enriches (e.g. once at request entry for
+// traceId/method/path, again from auth middleware once req.authenticated
+// and req.organization are populated).
 //
 // Reads from req.authenticated first (drawbridge-api convention) and
 // falls back to req.user for compatibility with other services.
+//
+// Uses Sentry.setTag / Sentry.setUser (which target the isolation scope
+// in v8+) rather than getCurrentScope().setTag, because errors captured
+// later in the request from a different async fork only inherit the
+// isolation scope. Setting tags on the current scope at middleware time
+// can leave them out of the captureException event payload.
 
 const applyRequestContext = ( req ) => {
 
-	const scope = Sentry.getCurrentScope();
-
 	if( req?.id ){
 
-		scope.setTag( 'traceId', req.id );
+		Sentry.setTag( 'traceId', req.id );
 
 	}
 	if( req?.method ){
 
-		scope.setTag( 'method', req.method );
+		Sentry.setTag( 'method', req.method );
 
 	}
 	// req.route is set by Express only AFTER route matching, which hasn't
@@ -28,14 +33,14 @@ const applyRequestContext = ( req ) => {
 	// future res.on('finish') refinement.
 	if( req?.path ){
 
-		scope.setTag( 'route', req.path );
+		Sentry.setTag( 'route', req.path );
 
 	}
 	const user = req?.authenticated || req?.user;
 
 	if( user ){
 
-		scope.setUser({
+		Sentry.setUser({
 			id : user.id || user._id,
 			email : user.email
 		});
@@ -43,7 +48,7 @@ const applyRequestContext = ( req ) => {
 	}
 	if( req?.organization ){
 
-		scope.setTag( 'organization', req.organization.id || req.organization._id );
+		Sentry.setTag( 'organization', req.organization.id || req.organization._id );
 
 	}
 };

package/dist/express.d.ts

@@ -1,25 +1,30 @@
 import * as Sentry from '@sentry/core';
 
-// Apply all available context tags from a request to the active Sentry
-// scope. Idempotent — safe to call multiple times as the request enriches
-// (e.g. once at request entry for traceId/method/path, again from auth
-// middleware once req.authenticated and req.organization are populated).
+// Apply all available context tags from a request to the per-request
+// Sentry isolation scope. Idempotent — safe to call multiple times as
+// the request enriches (e.g. once at request entry for
+// traceId/method/path, again from auth middleware once req.authenticated
+// and req.organization are populated).
 //
 // Reads from req.authenticated first (drawbridge-api convention) and
 // falls back to req.user for compatibility with other services.
+//
+// Uses Sentry.setTag / Sentry.setUser (which target the isolation scope
+// in v8+) rather than getCurrentScope().setTag, because errors captured
+// later in the request from a different async fork only inherit the
+// isolation scope. Setting tags on the current scope at middleware time
+// can leave them out of the captureException event payload.
 
 const applyRequestContext = ( req ) => {
 
-	const scope = Sentry.getCurrentScope();
-
 	if( req?.id ){
 
-		scope.setTag( 'traceId', req.id );
+		Sentry.setTag( 'traceId', req.id );
 
 	}
 	if( req?.method ){
 
-		scope.setTag( 'method', req.method );
+		Sentry.setTag( 'method', req.method );
 
 	}
 	// req.route is set by Express only AFTER route matching, which hasn't
@@ -28,14 +33,14 @@ const applyRequestContext = ( req ) => {
 	// future res.on('finish') refinement.
 	if( req?.path ){
 
-		scope.setTag( 'route', req.path );
+		Sentry.setTag( 'route', req.path );
 
 	}
 	const user = req?.authenticated || req?.user;
 
 	if( user ){
 
-		scope.setUser({
+		Sentry.setUser({
 			id : user.id || user._id,
 			email : user.email
 		});
@@ -43,7 +48,7 @@ const applyRequestContext = ( req ) => {
 	}
 	if( req?.organization ){
 
-		scope.setTag( 'organization', req.organization.id || req.organization._id );
+		Sentry.setTag( 'organization', req.organization.id || req.organization._id );
 
 	}
 };

package/dist/express.js

@@ -1,29 +1,28 @@
 // express.js
 import * as Sentry from "@sentry/core";
 var applyRequestContext = (req) => {
-  const scope = Sentry.getCurrentScope();
   if (req == null ? void 0 : req.id) {
-    scope.setTag("traceId", req.id);
+    Sentry.setTag("traceId", req.id);
   }
   ;
   if (req == null ? void 0 : req.method) {
-    scope.setTag("method", req.method);
+    Sentry.setTag("method", req.method);
   }
   ;
   if (req == null ? void 0 : req.path) {
-    scope.setTag("route", req.path);
+    Sentry.setTag("route", req.path);
   }
   ;
   const user = (req == null ? void 0 : req.authenticated) || (req == null ? void 0 : req.user);
   if (user) {
-    scope.setUser({
+    Sentry.setUser({
       id: user.id || user._id,
       email: user.email
     });
   }
   ;
   if (req == null ? void 0 : req.organization) {
-    scope.setTag("organization", req.organization.id || req.organization._id);
+    Sentry.setTag("organization", req.organization.id || req.organization._id);
   }
   ;
 };

package/package.json

@@ -46,9 +46,12 @@
     "access": "public"
   },
   "scripts": {
-    "sync": ". \"$HOME/.nvm/nvm.sh\" && nvm use && npm prune && npm install",
+    "sync": ". \"$HOME/.nvm/nvm.sh\" && nvm use && npm prune && npm install && npx drawbridge-agents-sync",
     "build": "tsup && npm publish"
   },
   "types": "dist/index.d.ts",
-  "version": "0.0.3"
+  "version": "0.0.4",
+  "dependencies": {
+    "@drawbridge/drawbridge-agents": "0.0.4"
+  }
 }