chore(deps-dev): upgrade biome and vitest

Author: erunion

biome.jsonc

@@ -1,9 +1,9 @@
 {
   "$schema": "./node_modules/@biomejs/biome/configuration_schema.json",
-  "extends": ["@readme/standards/biome"],
+  "extends": ["@readme/standards/biome", "@readme/standards/biome/esm"],
   "files": {
     "ignoreUnknown": false,
-    "includes": ["**/*.{cjs,mjs,js,mts,ts}", "!coverage/**", "!dist/**", "!src/targets/**/fixtures/**"],
+    "includes": ["**/*.{cjs,mjs,js,mts,ts}", "!coverage", "!dist", "!src/targets/**/fixtures"],
   },
   "linter": {
     "enabled": true,
@@ -28,4 +28,19 @@
       },
     },
   },
+  "overrides": [
+    {
+      "includes": ["src/**/*.test.ts"],
+      "linter": {
+        "rules": {
+          "correctness": {
+            // Because we dynamically load test fixtures, and have to use `require` to do so, those
+            // files need to be `.cjs` files which collides with this extension as it wants them
+            // to be `.js`.
+            "useImportExtensions": "off",
+          },
+        },
+      },
+    },
+  ],
 }

package-lock.json

@@ -91,12 +91,12 @@
     "@types/node": "^24.0.8",
     "@types/qs": "^6.9.10",
     "@types/stringify-object": "^4.0.5",
-    "@vitest/coverage-v8": "^3.0.5",
+    "@vitest/coverage-v8": "^4.0.2",
     "prettier": "^3.0.3",
     "require-directory": "^2.1.1",
     "tsup": "^8.0.1",
     "typescript": "^5.8.3",
-    "vitest": "^3.0.5"
+    "vitest": "^4.0.2"
   },
   "prettier": "@readme/standards/prettier"
 }

package.json

@@ -283,7 +283,7 @@ export class HTTPSnippet {
         if (request.postData.text) {
           try {
             request.postData.jsonObj = JSON.parse(request.postData.text);
-          } catch (e) {
+          } catch {
             // force back to `text/plain` if headers have proper content-type value, then this should also work
             request.postData.mimeType = 'text/plain';
           }

src/index.ts

@@ -74,6 +74,7 @@ const inputFileNames = readdirSync(path.join(...expectedBasePath), 'utf-8');
 
 const fixtures: [string, Request][] = inputFileNames.map(inputFileName => [
   inputFileName.replace(path.extname(inputFileName), ''),
+  // biome-ignore lint/style/noCommonJs: Because we're dynamically loading fixtures we need to use `require`.
   require(path.resolve(...expectedBasePath, inputFileName)),
 ]);
 
@@ -204,7 +205,7 @@ function integrationTest(
 
       try {
         expect(stdout).toStrictEqual(harResponse.content.text);
-      } catch (err) {
+      } catch {
         // Some targets always assume that their response is JSON and for this case
         // (`custom-method`) will print out an empty string instead.
         expect(stdout).toBe('');

src/integration.test.ts

@@ -17,6 +17,7 @@ const inputFileNames = readdirSync(path.join(...expectedBasePath), 'utf-8');
 
 const fixtures: [string, Request][] = inputFileNames.map(inputFileName => [
   inputFileName.replace(path.extname(inputFileName), ''),
+  // biome-ignore lint/style/noCommonJs: Because we're dynamically loading fixtures we need to use `require`.
   require(path.resolve(...expectedBasePath, inputFileName)),
 ]);
 

src/targets/index.test.ts

@@ -187,7 +187,7 @@ export const curl: Client<CurlOptions> = {
                   `${binary ? '--data-binary' : arg('data')} '\n${JSON.stringify(jsonPayload, null, indentJSON)}\n'`,
                 );
               }
-            } catch (err) {
+            } catch {
               // no-op
             }
           }

src/targets/shell/curl/client.ts

@@ -1,5 +1,6 @@
 import { defineConfig } from 'tsup';
 
+// biome-ignore lint/style/noDefaultExport: `tsup` uses default exports.
 export default defineConfig(options => ({
   ...options,