From 0d0000874a08ccb76cb2fcabd723a9980f6211df Mon Sep 17 00:00:00 2001
From: Patrick Roza <contact@patrickroza.com>
Date: Mon, 13 Nov 2023 16:40:21 +0100
Subject: [PATCH] cleanup

---
 packages/infra/_src/errorReporter.ts | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/packages/infra/_src/errorReporter.ts b/packages/infra/_src/errorReporter.ts
index 47cbed0e1..cefb75610 100644
--- a/packages/infra/_src/errorReporter.ts
+++ b/packages/infra/_src/errorReporter.ts
@@ -1,3 +1,4 @@
+import { dropUndefined } from "@effect-app/core/utils"
 import * as Sentry from "@sentry/node"
 import { CauseException } from "./errors.js"
 import { RequestContextContainer } from "./services/RequestContextContainer.js"
@@ -15,11 +16,11 @@ export function reportError(
       yield* $(
         cause
           .logError
-          .annotateLogs({
+          .annotateLogs(dropUndefined({
             extras,
             // __cause__: error.toJSON(), // logs too much garbage
             __error_name__: name
-          })
+          }))
       )
     })
 }
@@ -46,17 +47,17 @@ export function logError<E>(
   return (cause: Cause<E>, extras?: Record<string, unknown>) =>
     Effect.gen(function*($) {
       if (cause.isInterrupted()) {
-        yield* $(Effect.logDebug("Interrupted").annotateLogs("extras", JSON.stringify(extras ?? {})))
+        yield* $(Effect.logDebug("Interrupted").annotateLogs(dropUndefined({ extras })))
         return
       }
       yield* $(
         cause
           .logWarning
-          .annotateLogs({
+          .annotateLogs(dropUndefined({
             extras,
             // __cause__: error.toJSON(), // logs too much garbage
             __error_name__: name
-          })
+          }))
       )
     })
 }