From 389b87e15024e6fb0aa4d4ca97403745aac31a7d Mon Sep 17 00:00:00 2001 From: Erhnysr Date: Mon, 8 Jun 2026 13:11:31 +0300 Subject: [PATCH] fix: throw instead of return in sendRequestToSubAccountSigner and harden production guard --- .../src/interface/public-utilities/spend-permission/utils.ts | 5 ++--- packages/account-sdk/src/sign/base-account/Signer.ts | 2 +- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/packages/account-sdk/src/interface/public-utilities/spend-permission/utils.ts b/packages/account-sdk/src/interface/public-utilities/spend-permission/utils.ts index eaf7fe31..2ad81365 100644 --- a/packages/account-sdk/src/interface/public-utilities/spend-permission/utils.ts +++ b/packages/account-sdk/src/interface/public-utilities/spend-permission/utils.ts @@ -124,9 +124,8 @@ export function createSpendPermissionTypedDataWithSeconds( // Runtime check to prevent misuse if (process.env.NODE_ENV === 'production') { - console.warn( - '⚠️ createSpendPermissionTypedDataWithSeconds is being used. ' + - 'This function is intended for testing purposes only.' + throw new Error( + 'createSpendPermissionTypedDataWithSeconds is for testing only and must not be called in production.' ); } diff --git a/packages/account-sdk/src/sign/base-account/Signer.ts b/packages/account-sdk/src/sign/base-account/Signer.ts index 1cffe605..58ee5ad2 100644 --- a/packages/account-sdk/src/sign/base-account/Signer.ts +++ b/packages/account-sdk/src/sign/base-account/Signer.ts @@ -792,7 +792,7 @@ export class Signer { correlationId, errorMessage: parseErrorMessageFromAny(error), }); - return standardErrors.provider.unauthorized( + throw standardErrors.provider.unauthorized( 'failed to add sub account owner when sending request to sub account signer' ); }