chore(deps-dev): moving from Biome+Prettier to Oxc

.github/workflows/ci.yml

@@ -31,10 +31,6 @@ jobs:
       - run: npm ci
       - run: npm run build
       - run: npm run lint
-      - name: Prettier formatting check
-        run: npm run prettier
-      - name: Insensitive language check for Markdown docs
-        run: npm run alex
 
   test:
     runs-on: ubuntu-latest

knip.ts

@@ -9,10 +9,17 @@ const config: KnipConfig = {
   ignoreDependencies: [
     '@vitest/coverage-v8', // used by vitest --coverage
     'conventional-changelog-cli', // used by `npm version`
+    'eslint-plugin-readme', // This is used in our Oxlint config through another dependency.
     'har-format', // this is loaded via `@types/har-format`
     'vitest', // installed in individual packages
   ],
   ignoreBinaries: ['vitest'],
+  oxfmt: {
+    config: ['oxfmt.config.ts'],
+  },
+  oxlint: {
+    config: ['oxlint.config.ts'],
+  },
   workspaces: {
     'packages/test-utils': {
       entry: ['nock-mocks.ts'],

oxfmt.config.ts

@@ -0,0 +1,17 @@
+import oxfmtConfig from '@readme/oxlint-config/oxfmt';
+import { defineConfig } from 'oxfmt';
+
+export default defineConfig(
+  Object.assign(structuredClone(oxfmtConfig), {
+    sortImports: {
+      ...oxfmtConfig.sortImports,
+    },
+    ignorePatterns: [
+      'coverage/',
+      'dist/',
+      'packages/api/schema.json',
+      'packages/httpsnippet-client-api/test/__datasets__/**/index.ts',
+      'packages/httpsnippet-client-api/test/__datasets__/**/output.js',
+    ],
+  }),
+);

oxlint.config.ts

@@ -0,0 +1,67 @@
+import oxlintConfig from '@readme/oxlint-config';
+import oxlintConfigVitest from '@readme/oxlint-config/testing/vitest';
+import oxlintConfigTS from '@readme/oxlint-config/typescript';
+import { defineConfig } from 'oxlint';
+
+export default defineConfig({
+  extends: [oxlintConfig, oxlintConfigTS],
+  options: {
+    reportUnusedDisableDirectives: 'error',
+  },
+  ignorePatterns: [
+    '**/coverage',
+    '**/dist',
+
+    // `httpsnippet-client-api` datasets
+    'packages/httpsnippet-client-api/test/__datasets__/**/*.js',
+  ],
+  categories: {
+    suspicious: 'error',
+  },
+  env: {
+    browser: true,
+    commonjs: true,
+    es2022: true,
+    node: true,
+  },
+  rules: {
+    'class-methods-use-this': 'off',
+    'import/no-cycle': 'off', // @todo
+    'no-underscore-dangle': ['error', { allow: ['_key'] }],
+    'no-void': 'off',
+    'unicorn/no-array-sort': 'off', // @todo
+  },
+  overrides: [
+    {
+      files: ['packages/**/*.test.{js,ts}'],
+      ...oxlintConfigVitest,
+      rules: Object.assign(structuredClone(oxlintConfigVitest.rules), {
+        'vitest/no-conditional-expect': 'off',
+        'vitest/require-hook': 'off',
+        'vitest/warn-todo': 'off',
+      }),
+    },
+    {
+      files: ['packages/api/bin/api.js', 'packages/**/test/global.d.ts'],
+      rules: {
+        'import/no-unassigned-import': 'off',
+      },
+    },
+    {
+      files: ['packages/core/**'],
+      rules: {
+        'no-restricted-imports': [
+          'error',
+          {
+            paths: [
+              {
+                name: 'fs/promises',
+                message: "Please use `fs` instead as some client frameworks don't polyfill `fs/promises`.",
+              },
+            ],
+          },
+        ],
+      },
+    },
+  ],
+});