Migrate to oxlint and oxfmt (#696)

<!-- Please read
https://github.com/SableClient/Sable/blob/dev/CONTRIBUTING.md before
submitting your pull request -->

### Description

Migrates from eslint and prettier to oxlint and oxfmt + some rule
hardening and - some rules that don't exist in oxlint, but I decided it
probably wasn't worth running both in parallel just for a few rules.

Notably, got rid of many "as any"s and swapped to .toSorted() for safer
stuff and "type" in imports. Added comments to all the existing lint
issues (console logging and the alert for app updates) and the few
places "as any" was justified (test cases) and some message object
whatnots.

Also notably, fmt now takes a second or two, and linting now takes no
more than a few seconds. Yay!

Recommended vscode plugins, workflows, and configs should all be updated
assuming I didn't miss anything.

<!-- Please include a summary of the change. Please also include
relevant motivation and context. List any dependencies that are required
for this change. -->

#### Type of change

- [x] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [x] Breaking change - breaking I guess since not all rules exist? I
don't really know exactly what I uh... didn't include.
- [ ] This change requires a documentation update

### Checklist:

- [x] My code follows the style guidelines of this project
- [x] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings

### AI disclosure:

- [x] Partially AI assisted (clarify which code was AI assisted and
briefly explain what it does).
- [ ] Fully AI generated (explain what all the generated code does in
moderate detail).
<!-- Write any explanation required here, but do not generate the
explanation using AI!! You must prove you understand what the code in
this PR does. -->

I deployed a swarm of silly AIs to fix many of the simpler as any casts
and subsequent typechecks because there were hundreds and I'm not doing
that manually :p. I did review it, mostly just seamless swapping to the
correct cast type instead of as any, nothing particularly complicated.

Author: 7w1

.prettierignore

@@ -1,3 +1,3 @@
 {
-  "recommendations": ["dbaeumer.vscode-eslint", "esbenp.prettier-vscode", "webpro.vscode-knip"]
+  "recommendations": ["webpro.vscode-knip", "oxc.oxc-vscode"]
 }

.prettierrc.json

@@ -1,11 +1,16 @@
 {
+  "editor.codeActionsOnSave": {
+    "source.fixAll.oxc": "always"
+  },
   "editor.formatOnSave": true,
-  "editor.defaultFormatter": "esbenp.prettier-vscode",
+  "editor.defaultFormatter": "oxc.oxc-vscode",
+  "oxc.fmt.configPath": "oxfmt.config.ts",
+  "oxc.typeAware": true,
   "typescript.tsdk": "node_modules/typescript/lib",
   "[jsonc]": {
-    "editor.defaultFormatter": "esbenp.prettier-vscode"
+    "editor.defaultFormatter": "oxc.oxc-vscode"
   },
   "[json]": {
-    "editor.defaultFormatter": "esbenp.prettier-vscode"
+    "editor.defaultFormatter": "oxc.oxc-vscode"
   }
 }

.vscode/extensions.json

@@ -1,15 +1,15 @@
 {
   "$schema": "https://unpkg.com/knip@5/schema.json",
   "entry": ["src/sw.ts", "scripts/normalize-imports.js"],
+  "ignore": ["oxlint.config.ts", "oxfmt.config.ts"],
   "ignoreExportsUsedInFile": {
     "interface": true,
     "type": true
   },
   "ignoreDependencies": [
     "buffer",
     "@sableclient/sable-call-embedded",
-    "@matrix-org/matrix-sdk-crypto-wasm",
-    "@testing-library/user-event"
+    "@matrix-org/matrix-sdk-crypto-wasm"
   ],
   "ignoreBinaries": ["knope"],
   "rules": {

.vscode/settings.json

@@ -2,8 +2,12 @@
 versioned_files = ["package.json"]
 changelog = "CHANGELOG.md"
 extra_changelog_sections = [
-    { name = "Documentation", types = ["docs"] },
-    { name = "Notes", types = ["note"] },
+  { name = "Documentation", types = [
+    "docs",
+  ] },
+  { name = "Notes", types = [
+    "note",
+  ] },
 ]
 # assets = "marker" // TODO: add this later once we have assets
 
@@ -64,6 +68,6 @@ repo = "Sable"
 [release_notes]
 # The <!-- commit:$commit_hash --> marker is used by prepare-release.yml
 change_templates = [
-    "### $summary <!-- commit:$commit_hash -->\n\n$details",
-    "* $summary <!-- commit:$commit_hash -->",
+  "### $summary <!-- commit:$commit_hash -->\n\n$details",
+  "* $summary <!-- commit:$commit_hash -->",
 ]

eslint.config.js

@@ -0,0 +1,18 @@
+import { defineConfig } from 'oxfmt';
+
+export default defineConfig({
+  printWidth: 100,
+  tabWidth: 2,
+  singleQuote: true,
+  trailingComma: 'es5',
+  ignorePatterns: [
+    'dist',
+    'node_modules',
+    'package.json',
+    'pnpm-lock.yaml',
+    'LICENSE',
+    'README.md',
+    'CHANGELOG.md',
+    './changeset',
+  ],
+});

knip.json

@@ -0,0 +1,62 @@
+import { defineConfig } from 'oxlint';
+
+export default defineConfig({
+  options: {
+    typeAware: true,
+  },
+  plugins: ['react', 'jsx-a11y', 'typescript', 'import', 'unicorn', 'oxc', 'vitest', 'promise'],
+  categories: {
+    correctness: 'error',
+    suspicious: 'warn',
+    perf: 'warn',
+    style: 'off',
+  },
+  env: {
+    browser: true,
+    builtin: true,
+  },
+  rules: {
+    'import/no-unassigned-import': 'off',
+    'import/no-named-as-default': 'off',
+    'import/no-named-as-default-member': 'off',
+    'no-console': ['error', { allow: ['warn', 'error'] }],
+    'react/react-in-jsx-scope': 'off',
+    'react/jsx-filename-extension': ['error', { extensions: ['.tsx', '.jsx'] }],
+    'react/rules-of-hooks': 'error',
+    'react/exhaustive-deps': 'error',
+    'react/iframe-missing-sandbox': 'off',
+    'jsx-a11y/no-autofocus': 'off',
+    'jsx-a11y/prefer-tag-over-role': 'off',
+    'typescript/no-explicit-any': 'error',
+    'typescript/consistent-type-imports': 'error',
+    'typescript/only-throw-error': 'error',
+    'typescript/no-unsafe-type-assertion': 'off',
+    'typescript/no-floating-promises': 'off',
+    'typescript/no-unnecessary-type-arguments': 'off',
+    'oxc/no-map-spread': 'off',
+    'promise/always-return': 'off',
+  },
+  overrides: [
+    {
+      files: ['**/*.ts', '**/*.tsx', '**/*.cts', '**/*.mts'],
+      rules: {
+        'typescript/no-unused-vars': [
+          'error',
+          {
+            args: 'after-used',
+            ignoreRestSiblings: true,
+            vars: 'all',
+          },
+        ],
+        'typescript/no-shadow': 'error',
+      },
+    },
+    {
+      files: ['**/*.test.ts', '**/*.test.tsx'],
+      rules: {
+        'typescript/unbound-method': 'off',
+        'typescript/no-unsafe-enum-comparison': 'off',
+      },
+    },
+  ],
+});

knope.toml

@@ -12,10 +12,10 @@
     "dev": "vite dev",
     "build": "vite build",
     "preview": "vite preview",
-    "lint": "eslint .",
-    "lint:fix": "eslint . --fix",
-    "fmt": "prettier --write .",
-    "fmt:check": "prettier --check .",
+    "lint": "oxlint .",
+    "lint:fix": "oxlint . --fix && oxfmt .",
+    "fmt": "oxfmt .",
+    "fmt:check": "oxfmt --check .",
     "typecheck": "tsc",
     "test": "vitest",
     "test:ui": "vitest --ui",
@@ -57,7 +57,6 @@
     "dompurify": "^3.3.3",
     "emojibase": "^15.3.1",
     "emojibase-data": "^15.3.2",
-    "eslint-plugin-react": "7.37.5",
     "eventemitter3": "^5.0.4",
     "file-saver": "^2.0.5",
     "focus-trap-react": "^10.3.1",
@@ -96,8 +95,6 @@
   "devDependencies": {
     "@cloudflare/vite-plugin": "^1.26.0",
     "@esbuild-plugins/node-globals-polyfill": "^0.2.3",
-    "@eslint/compat": "2.0.2",
-    "@eslint/js": "9.39.3",
     "@rollup/plugin-inject": "^5.0.5",
     "@rollup/plugin-wasm": "^6.2.2",
     "@sableclient/sable-call-embedded": "v1.1.4",
@@ -118,14 +115,11 @@
     "@vitest/ui": "^4.1.0",
     "buffer": "^6.0.3",
     "cloudflared": "^0.7.1",
-    "eslint": "9.39.3",
-    "eslint-config-airbnb-extended": "3.0.1",
-    "eslint-config-prettier": "10.1.8",
-    "eslint-plugin-prettier": "5.5.5",
-    "globals": "17.3.0",
     "jsdom": "^29.0.0",
     "knip": "5.85.0",
-    "prettier": "3.8.1",
+    "oxfmt": "^0.45.0",
+    "oxlint": "^1.60.0",
+    "oxlint-tsgolint": "^0.21.0",
     "typescript": "^5.9.3",
     "vite": "^7.3.1",
     "vite-plugin-compression2": "2.5.0",