From d0cafb526f14a579350e2c5c3c3caabab9e5e775 Mon Sep 17 00:00:00 2001 From: JonasBa Date: Fri, 10 Mar 2023 15:17:51 -0500 Subject: [PATCH] ref(utils): use readonly value so we dont mutate arguments --- packages/utils/src/stacktrace.ts | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/packages/utils/src/stacktrace.ts b/packages/utils/src/stacktrace.ts index 93b62490a250..8a304001efc6 100644 --- a/packages/utils/src/stacktrace.ts +++ b/packages/utils/src/stacktrace.ts @@ -57,16 +57,12 @@ export function stackParserFromStackParserOptions(stackParser: StackParser | Sta /** * @hidden */ -export function stripSentryFramesAndReverse(stack: StackFrame[]): StackFrame[] { +export function stripSentryFramesAndReverse(stack: ReadonlyArray): StackFrame[] { if (!stack.length) { return []; } - let localStack = stack; - - if (stack.length >= STACKTRACE_LIMIT) { - localStack.slice(0, STACKTRACE_LIMIT); - } + let localStack = stack.slice(0, STACKTRACE_LIMIT); const lastFrameFunction = localStack[localStack.length - 1].function; // If stack ends with one of our internal API calls, remove it (ends, meaning it's the bottom of the stack - aka top-most call)