diff --git a/packages/junior/src/chat/logging.ts b/packages/junior/src/chat/logging.ts index 040e0277..fbd64a20 100644 --- a/packages/junior/src/chat/logging.ts +++ b/packages/junior/src/chat/logging.ts @@ -36,6 +36,7 @@ export interface LogContext { slackThreadId?: string; slackUserId?: string; slackUserName?: string; + slackUserEmail?: string; slackChannelId?: string; runId?: string; assistantUserName?: string; @@ -1375,6 +1376,7 @@ export function setSentryTagsFromContext(context: LogContext): void { Sentry.setUser({ id: context.slackUserId, username: context.slackUserName, + email: context.slackUserEmail, }); } } @@ -1390,7 +1392,11 @@ export function setSentryScopeContext( } } if (context.slackUserId) { - scope.setUser({ id: context.slackUserId, username: context.slackUserName }); + scope.setUser({ + id: context.slackUserId, + username: context.slackUserName, + email: context.slackUserEmail, + }); } scope.setContext("app", attrs); } diff --git a/packages/junior/src/chat/runtime/reply-executor.ts b/packages/junior/src/chat/runtime/reply-executor.ts index a614bc09..3757b753 100644 --- a/packages/junior/src/chat/runtime/reply-executor.ts +++ b/packages/junior/src/chat/runtime/reply-executor.ts @@ -432,9 +432,11 @@ export function createReplyToThread(deps: ReplyExecutorDeps) { ); const resolvedUserName = message.author.userName ?? fallbackIdentity?.userName; - if (resolvedUserName) { - setTags({ slackUserName: resolvedUserName }); - } + setTags({ + slackUserId: message.author.userId, + slackUserName: resolvedUserName, + slackUserEmail: fallbackIdentity?.email, + }); const userAttachments = await deps.resolveUserAttachments( message.attachments, {