refactor!: use native JSON.parse (#520)

Author: pi0

README.md

@@ -33,15 +33,15 @@ import { ofetch } from "ofetch";
 
 ## ✔️ Parsing Response
 
-`ofetch` will smartly parse JSON and native values using [destr](https://github.com/unjs/destr), falling back to the text if it fails to parse.
+`ofetch` smartly parse JSON responses.
 
 ```js
 const { users } = await ofetch("/api/users");
 ```
 
 For binary content types, `ofetch` will instead return a `Blob` object.
 
-You can optionally provide a different parser than `destr`, or specify `blob`, `arrayBuffer`, `text` or `stream` to force parsing the body with the respective `FetchResponse` method.
+You can optionally provide a different parser than `JSON.parse`, or specify `blob`, `arrayBuffer`, `text` or `stream` to force parsing the body with the respective `FetchResponse` method.
 
 ```js
 // Use JSON.parse
@@ -398,9 +398,9 @@ myFetch("/foo", { requiresAuth: true });
 
 ## 📦 Bundler Notes
 
-- All targets are exported with Module and CommonJS format and named exports
+- All targets are exported with ESM format and named exports
 - No export is transpiled for the sake of modern syntax
-  - You probably need to transpile `ofetch`, `destr`, and `ufo` packages with Babel for ES5 support
+  - You probably need to transpile `ofetch` for ES5 support
 - You need to polyfill `fetch` global for supporting legacy browsers like using [unfetch](https://github.com/developit/unfetch)
 
 ## ❓ FAQ

package.json

@@ -23,10 +23,6 @@
     "release": "pnpm test && pnpm build && changelogen --release --prerelease --publish --publishTag alpha --push",
     "test": "pnpm lint && vitest run --coverage"
   },
-  "dependencies": {
-    "destr": "^2.0.5",
-    "ufo": "^1.6.1"
-  },
   "devDependencies": {
     "@types/node": "^24.9.1",
     "@vitest/coverage-v8": "^4.0.4",

pnpm-lock.yaml

@@ -1,5 +1,4 @@
 import type { Readable } from "node:stream";
-import destr from "destr";
 import { withBase, withQuery } from "./utils.url.ts";
 import { createFetchError } from "./error.ts";
 import {
@@ -213,12 +212,13 @@ export function createFetch(globalOptions: CreateFetchOptions = {}): $Fetch {
           : context.options.responseType) ||
         detectResponseType(context.response.headers.get("content-type") || "");
 
-      // We override the `.json()` method to parse the body more securely with `destr`
       switch (responseType) {
         case "json": {
           const data = await context.response.text();
-          const parseFunction = context.options.parseResponse || destr;
-          context.response._data = parseFunction(data);
+          if (data) {
+            const parseFunction = context.options.parseResponse || JSON.parse;
+            context.response._data = parseFunction(data);
+          }
           break;
         }
         case "stream": {

src/fetch.ts

@@ -252,7 +252,7 @@ describe("ofetch", () => {
 
   it("204 no content", async () => {
     const res = await $fetch(getURL("204"));
-    expect(res).toBe("");
+    expect(res).toBe(undefined);
   });
 
   it("HEAD no content", async () => {