fix: JSON.stringify(...) throwing an error (#31)

james-elicx · Aug 27, 2023 · 8527c95 · 8527c95
1 parent 0b7bfb2
commit 8527c95
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
diff --git a/.changeset/green-bobcats-notice.md b/.changeset/green-bobcats-notice.md
@@ -0,0 +1,5 @@
+---
+'cf-bindings-proxy': patch
+---
+
+Fix using `JSON.stringify(...)` throwing an error.
diff --git a/src/proxy.ts b/src/proxy.ts
@@ -88,6 +88,8 @@ const createResponseProxy = <T extends object>(
 				return data[Number(prop)];
 			}
 
+			if (['toJSON'].includes(prop as string)) return data;
+
 			// eslint-disable-next-line @typescript-eslint/no-use-before-define
 			const newProxy = createBindingProxy<BindingRequest>(bindingId, true);
 

diff --git a/tests/proxy.spec.ts b/tests/proxy.spec.ts
@@ -320,6 +320,8 @@ suite('bindings', () => {
 				{ key: 'json-key' },
 				{ key: 'second-key' },
 			]);
+
+			expect(JSON.stringify(list.truncated)).toEqual('false');
 		});
 
 		test('head', async () => {