feat(core): Apply scope attributes to logs

Author: Lms24

packages/core/src/logs/internal.ts

@@ -1,10 +1,12 @@
+import type { TypedAttributes } from '../attributes';
+import { attributeValueToTypedAttributeValue } from '../attributes';
 import { getGlobalSingleton } from '../carrier';
 import type { Client } from '../client';
 import { getClient, getCurrentScope, getGlobalScope, getIsolationScope } from '../currentScopes';
 import { DEBUG_BUILD } from '../debug-build';
 import type { Scope, ScopeData } from '../scope';
 import type { Integration } from '../types-hoist/integration';
-import type { Log, SerializedLog, SerializedLogAttributeValue } from '../types-hoist/log';
+import type { Log, SerializedLog } from '../types-hoist/log';
 import { mergeScopeData } from '../utils/applyScopeDataToEvent';
 import { consoleSandbox, debug } from '../utils/debug-logger';
 import { isParameterizedString } from '../utils/is';
@@ -16,51 +18,6 @@ import { createLogEnvelope } from './envelope';
 
 const MAX_LOG_BUFFER_SIZE = 100;
 
-/**
- * Converts a log attribute to a serialized log attribute.
- *
- * @param key - The key of the log attribute.
- * @param value - The value of the log attribute.
- * @returns The serialized log attribute.
- */
-export function logAttributeToSerializedLogAttribute(value: unknown): SerializedLogAttributeValue {
-  switch (typeof value) {
-    case 'number':
-      if (Number.isInteger(value)) {
-        return {
-          value,
-          type: 'integer',
-        };
-      }
-      return {
-        value,
-        type: 'double',
-      };
-    case 'boolean':
-      return {
-        value,
-        type: 'boolean',
-      };
-    case 'string':
-      return {
-        value,
-        type: 'string',
-      };
-    default: {
-      let stringValue = '';
-      try {
-        stringValue = JSON.stringify(value) ?? '';
-      } catch {
-        // Do nothing
-      }
-      return {
-        value: stringValue,
-        type: 'string',
-      };
-    }
-  }
-}
-
 /**
  * Sets a log attribute if the value exists and the attribute key is not already present.
  *
@@ -139,6 +96,7 @@ export function _INTERNAL_captureLog(
 
   const {
     user: { id, email, username },
+    attributes: scopeAttributes,
   } = getMergedScopeData(currentScope);
   setLogAttribute(processedLogAttributes, 'user.id', id, false);
   setLogAttribute(processedLogAttributes, 'user.email', email, false);
@@ -201,13 +159,13 @@ export function _INTERNAL_captureLog(
     body: message,
     trace_id: traceContext?.trace_id,
     severity_number: severityNumber ?? SEVERITY_TEXT_TO_SEVERITY_NUMBER[level],
-    attributes: Object.keys(attributes).reduce(
-      (acc, key) => {
-        acc[key] = logAttributeToSerializedLogAttribute(attributes[key]);
+    attributes: {
+      ...scopeAttributes,
+      ...Object.keys(attributes).reduce((acc, key) => {
+        acc[key] = attributeValueToTypedAttributeValue(attributes[key]);
         return acc;
-      },
-      {} as Record<string, SerializedLogAttributeValue>,
-    ),
+      }, {} as TypedAttributes),
+    },
   };
 
   captureSerializedLog(client, serializedLog);

packages/core/src/types-hoist/log.ts

@@ -1,3 +1,4 @@
+import type { TypedAttributes } from '../attributes';
 import type { ParameterizedString } from './parameterize';
 
 export type LogSeverityLevel = 'trace' | 'debug' | 'info' | 'warn' | 'error' | 'fatal';
@@ -30,12 +31,6 @@ export interface Log {
   severityNumber?: number;
 }
 
-export type SerializedLogAttributeValue =
-  | { value: string; type: 'string' }
-  | { value: number; type: 'integer' }
-  | { value: number; type: 'double' }
-  | { value: boolean; type: 'boolean' };
-
 export interface SerializedLog {
   /**
    * Timestamp in seconds (epoch time) indicating when the  log occurred.
@@ -60,7 +55,7 @@ export interface SerializedLog {
   /**
    * Arbitrary structured data that stores information about the log - e.g., userId: 100.
    */
-  attributes?: Record<string, SerializedLogAttributeValue>;
+  attributes?: TypedAttributes;
 
   /**
    * The severity number.