Allow for serializing int as BigInt

graphql · Oct 11, 2024 · 99dca99 · 99dca99
1 parent 50607e4
commit 99dca99
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/type/__tests__/scalars-test.ts b/src/type/__tests__/scalars-test.ts
@@ -614,6 +614,7 @@ describe('Type System: Specified scalar types', () => {
       expect(serialize(123)).to.equal('123');
       expect(serialize(0)).to.equal('0');
       expect(serialize(-1)).to.equal('-1');
+      expect(serialize(BigInt(123))).to.equal('123');
 
       const valueOf = () => 'valueOf ID';
       const toJSON = () => 'toJSON ID';

diff --git a/src/type/scalars.ts b/src/type/scalars.ts
@@ -252,7 +252,7 @@ export const GraphQLID = new GraphQLScalarType<string>({
     if (typeof coercedValue === 'string') {
       return coercedValue;
     }
-    if (Number.isInteger(coercedValue)) {
+    if (Number.isInteger(coercedValue) || typeof coercedValue === 'bigint') {
       return String(coercedValue);
     }
     throw new GraphQLError(