Skip to content

Commit 9b91652

Browse files
committed
Simplify logic
1 parent 45b7a05 commit 9b91652

1 file changed

Lines changed: 8 additions & 10 deletions

File tree

  • packages/core/src/tracing/langchain

packages/core/src/tracing/langchain/utils.ts

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -561,21 +561,19 @@ export function _INTERNAL_mergeLangChainCallbackHandler(existing: unknown, sentr
561561
}
562562

563563
if (Array.isArray(existing)) {
564-
if (existing.includes(sentryHandler)) {
565-
return existing;
564+
if (!existing.includes(sentryHandler)) {
565+
return [...existing, sentryHandler];
566566
}
567-
return [...existing, sentryHandler];
568567
}
569568

570569
if (isCallbackManager(existing)) {
571-
if (existing.handlers?.includes(sentryHandler)) {
572-
return existing;
570+
if (!existing.handlers?.includes(sentryHandler)) {
571+
const copied = existing.copy() as {
572+
addHandler: (handler: unknown, inherit?: boolean) => void;
573+
};
574+
copied.addHandler(sentryHandler, true);
575+
return copied;
573576
}
574-
const copied = existing.copy() as {
575-
addHandler: (handler: unknown, inherit?: boolean) => void;
576-
};
577-
copied.addHandler(sentryHandler, true);
578-
return copied;
579577
}
580578

581579
return existing;

0 commit comments

Comments
 (0)