Avoid extra JSON.stringify().

davidlehn · davidlehn · commit 1046c292e2ba · 2021-03-17T23:58:11.000-04:00
- This return value is only used where it would be immediately parsed
  again if a string. Just leave it as parsed data.
diff --git a/lib/documentLoaders/node.js b/lib/documentLoaders/node.js
@@ -172,8 +172,7 @@ async function _fetch({url, headers, strictSSL, httpAgent, httpsAgent}) {
       }
     }
     const res = await httpClient.get(url, options);
-    const body = JSON.stringify(res.data, null, 2);
-    return {res, body};
+    return {res, body: res.data};
   } catch(e) {
     // HTTP errors have a response in them
     // ky considers redirects HTTP errors

Original file line number	Diff line number	Diff line change
`@@ -172,8 +172,7 @@ async function _fetch({url, headers, strictSSL, httpAgent, httpsAgent}) {`
`172`	`172`	`}`
`173`	`173`	`}`
`174`	`174`	`const res = await httpClient.get(url, options);`
`175`		`- const body = JSON.stringify(res.data, null, 2);`
`176`		`- return {res, body};`
	`175`	`+ return {res, body: res.data};`
`177`	`176`	`} catch(e) {`
`178`	`177`	`// HTTP errors have a response in them`
`179`	`178`	`// ky considers redirects HTTP errors`