No need to JSON.stringify when using Response.json()

Jason Barry · Jason Barry · commit ad19bcd07431 · 2024-03-12T19:04:08.000-07:00
diff --git a/netlify/functions/blob.ts b/netlify/functions/blob.ts
@@ -6,7 +6,7 @@ export default async (req: Request) => {
   if (req.method === "GET") {
     const todos = await store.get("todos", { type: "json" });
     // @ts-expect-error `new` operator not needed for Response.json()
-    return Response.json(todos || JSON.stringify([]), { status: 200 });
+    return Response.json(todos || [], { status: 200 });
   }
 
   if (req.method === "PUT") {

Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@ export default async (req: Request) => {`
`6`	`6`	`if (req.method === "GET") {`
`7`	`7`	`const todos = await store.get("todos", { type: "json" });`
`8`	`8`	// @ts-expect-error `new` operator not needed for Response.json()
`9`		`- return Response.json(todos \|\| JSON.stringify([]), { status: 200 });`
	`9`	`+ return Response.json(todos \|\| [], { status: 200 });`
`10`	`10`	`}`
`11`	`11`
`12`	`12`	`if (req.method === "PUT") {`