@friggframework/core 2.0.0-next.111 → 2.0.0-next.113

package/types/logs/index.d.ts

@@ -1,5 +1,64 @@
-declare module "@friggframework/logs/logger" {
+declare module "@friggframework/core/logs" {
+  export type LogLevel = "TRACE" | "DEBUG" | "INFO" | "WARN" | "ERROR" | "FATAL";
+
+  export type LogFields = Record<string, unknown>;
+
+  export interface LogRecord {
+    readonly timestamp: string;
+    readonly level: LogLevel;
+    readonly message: string;
+    readonly logger: string;
+    readonly [key: string]: unknown;
+  }
+
+  export interface Logger {
+    readonly name: string;
+    trace(message: unknown, fields?: LogFields): void;
+    debug(message: unknown, fields?: LogFields): void;
+    info(message: unknown, fields?: LogFields): void;
+    warn(message: unknown, fields?: LogFields): void;
+    error(message: unknown, fields?: LogFields): void;
+    fatal(message: unknown, fields?: LogFields): void;
+    child(bindings: LogFields | (() => LogFields)): Logger;
+    isLevelEnabled(level: string): boolean;
+  }
+
+  export interface LogSink {
+    name: string;
+    write(record: LogRecord): void;
+    flush?(options: { signal?: AbortSignal }): Promise<void> | void;
+  }
+
+  export interface MemorySink extends LogSink {
+    readonly records: LogRecord[];
+    clear(): void;
+  }
+
+  export interface SerializedError {
+    type: string;
+    message: string;
+    code?: string | number;
+    status?: number;
+    stack?: string;
+    cause?: SerializedError | string;
+  }
+
+  export function getLogger(name?: string): Logger;
+  export function createMemorySink(options?: { install?: boolean }): MemorySink;
+  export function resetLoggerForTests(options?: {
+    level?: LogLevel | Lowercase<LogLevel>;
+    sinks?: LogSink[];
+    /** Record frigg.* WARN+ records without eventName. Kept across resets that omit it. */
+    trackViolations?: boolean;
+  }): void;
+  export function serializeError(error: unknown): SerializedError;
+  export function redactValue(value: unknown): unknown;
+  export function toSanitizedSurrogate(error: unknown): Error;
+
+  /** @deprecated Use getLogger or this.logger. */
   export function debug(...messages: any[]): void;
+  /** @deprecated Use getLogger or this.logger. */
   export function initDebugLog(...initMessages: any[]): void;
-  export function flushDebugLog(error: any): void;
+  /** @deprecated Use getLogger or this.logger. */
+  export function flushDebugLog(error?: any): void;
 }

package/user/repositories/user-repository-documentdb.js

@@ -17,6 +17,9 @@ const { ClientSafeError } = require('../../errors');
 const {
     DocumentDBEncryptionService,
 } = require('../../database/documentdb-encryption-service');
+const { getLogger } = require('../../logs');
+
+const log = getLogger('frigg.user');
 
 /**
  * User repository for DocumentDB.
@@ -145,19 +148,8 @@ class UserRepositoryDocumentDB extends UserRepositoryInterface {
 
         // Defensive check: verify document was found after insert
         if (!created) {
-            console.error(
-                '[UserRepositoryDocumentDB] User not found after insert',
-                {
-                    insertedId: fromObjectId(insertedId),
-                    params: {
-                        username: params.username,
-                        appUserId: params.appUserId,
-                        email: params.email,
-                    },
-                }
-            );
             throw new Error(
-                'Failed to create individual user: Document not found after insert. ' +
+                `Failed to create individual user: Document not found after insert (insertedId ${fromObjectId(insertedId)}). ` +
                     'This indicates a database consistency issue.'
             );
         }
@@ -195,18 +187,8 @@ class UserRepositoryDocumentDB extends UserRepositoryInterface {
 
         // Defensive check: verify document was found after insert
         if (!created) {
-            console.error(
-                '[UserRepositoryDocumentDB] Organization user not found after insert',
-                {
-                    insertedId: fromObjectId(insertedId),
-                    params: {
-                        appOrgId: params.appOrgId,
-                        name: params.name,
-                    },
-                }
-            );
             throw new Error(
-                'Failed to create organization user: Document not found after insert. ' +
+                `Failed to create organization user: Document not found after insert (insertedId ${fromObjectId(insertedId)}). ` +
                     'This indicates a database consistency issue.'
             );
         }
@@ -291,15 +273,8 @@ class UserRepositoryDocumentDB extends UserRepositoryInterface {
 
         // Defensive check: verify document was found after update
         if (!updated) {
-            console.error(
-                '[UserRepositoryDocumentDB] Individual user not found after update',
-                {
-                    userId: fromObjectId(objectId),
-                    updates,
-                }
-            );
             throw new Error(
-                'Failed to update individual user: Document not found after update. ' +
+                `Failed to update individual user: Document not found after update (userId ${userId}). ` +
                     'This indicates a database consistency issue.'
             );
         }
@@ -332,15 +307,8 @@ class UserRepositoryDocumentDB extends UserRepositoryInterface {
         const updated = await findOne(this.prisma, 'User', { _id: objectId });
 
         if (!updated) {
-            console.error(
-                '[UserRepositoryDocumentDB] Organization user not found after update',
-                {
-                    userId: fromObjectId(objectId),
-                    updates,
-                }
-            );
             throw new Error(
-                'Failed to update organization user: Document not found after update. ' +
+                `Failed to update organization user: Document not found after update (userId ${userId}). ` +
                     'This indicates a database consistency issue.'
             );
         }
@@ -363,9 +331,9 @@ class UserRepositoryDocumentDB extends UserRepositoryInterface {
 
     _mapUser(doc) {
         if (!doc) {
-            console.warn(
-                '[UserRepositoryDocumentDB] _mapUser received null/undefined document'
-            );
+            log.warn('_mapUser received a null document', {
+                eventName: 'frigg.user.map_user_null',
+            });
             return null;
         }