Fix BigInt handling

Urigo · Oct 10, 2023 · 0d73e69 · 0d73e69
1 parent 5b66cd5
commit 0d73e69
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/.changeset/tender-spoons-burn.md b/.changeset/tender-spoons-burn.md
@@ -0,0 +1,5 @@
+---
+'graphql-scalars': patch
+---
+
+Fix BigInt handling
diff --git a/src/scalars/BigInt.ts b/src/scalars/BigInt.ts
@@ -80,7 +80,7 @@ export const GraphQLBigIntConfig: GraphQLScalarTypeConfig<
     if (inputValue.toString() !== bigint.toString()) {
       throw createGraphQLError(`BigInt cannot represent value: ${inputValue}`);
     }
-    if (!isSafeInteger(bigint) && !isBigIntSerializable()) {
+    if (isSafeInteger(bigint) && !isBigIntSerializable()) {
       return Number(bigint.toString());
     }
     return bigint;
@@ -96,7 +96,7 @@ export const GraphQLBigIntConfig: GraphQLScalarTypeConfig<
     if (strOrBooleanValue.toString() !== bigint.toString()) {
       throw createGraphQLError(`BigInt cannot represent value: ${strOrBooleanValue}`);
     }
-    if (!isSafeInteger(bigint) && !isBigIntSerializable()) {
+    if (isSafeInteger(bigint) && !isBigIntSerializable()) {
       return Number(bigint.toString());
     }
     return bigint;