Update linter

Author: AndreasArvidsson

.oxfmtrc.json

@@ -6,5 +6,5 @@
     "/packages/lib-engine/src/customCommandGrammar/generated/",
     "/packages/lib-engine/src/snippets/vendor/",
     "/resources/fixtures/recorded/**/*.yml"
-  ]
+  ],
 }

.oxlintrc.json

@@ -1,76 +1,70 @@
 {
   "$schema": "./node_modules/oxlint/configuration_schema.json",
   "plugins": ["eslint", "typescript", "unicorn", "oxc", "import"],
+  "jsPlugins": [
+    {
+      "name": "mocha",
+      "specifier": "eslint-plugin-mocha"
+    }
+  ],
+
   "ignorePatterns": [
-    ".git/**",
     "resources/playground/**",
-    "packages/*/out/**",
-    "packages/*/dist/**",
-    "packages/app-web-docs/.docusaurus/**",
-    "packages/app-neovim/cursorless.nvim/**",
-    "packages/app-vscode/src/keyboard/grammar/generated/**",
-    "packages/lib-engine/src/customCommandGrammar/generated/**",
-    "packages/lib-engine/src/snippets/vendor/**",
-    "**/node_modules/**"
+    "packages/app-neovim/cursorless.nvim/**"
   ],
+
+  "options": {
+    "typeAware": true,
+    "typeCheck": false
+  },
+
   "rules": {
-    "import/no-relative-packages": "error",
-    "import/no-duplicates": "error",
-    "unused-imports/no-unused-imports": "error",
-    "@typescript-eslint/consistent-type-imports": "error",
-    "@typescript-eslint/consistent-type-assertions": [
-      "error",
+    "import/no-relative-packages": "warn",
+    "import/no-duplicates": "warn",
+    "typescript/consistent-type-imports": "warn",
+    "typescript/consistent-type-assertions": [
+      "warn",
       {
         "assertionStyle": "as"
       }
     ],
-    "@typescript-eslint/naming-convention": [
-      "error",
-      {
-        "selector": ["objectLiteralProperty"],
-        "format": ["camelCase"],
-        "filter": {
-          "regex": "[.]",
-          "match": false
-        }
-      }
-    ],
-    "@typescript-eslint/no-floating-promises": "error",
-    "@typescript-eslint/no-explicit-any": "off",
-    "@typescript-eslint/no-inferrable-types": "off",
-    "@typescript-eslint/no-non-null-assertion": "off",
-    "@typescript-eslint/no-unused-vars": [
-      "error",
+    "typescript/no-explicit-any": "off",
+    "typescript/no-inferrable-types": "off",
+    "typescript/restrict-template-expressions": "off",
+    "typescript/no-base-to-string": "off",
+    "typescript/unbound-method": "off",
+    "eslint/no-unused-vars": [
+      "warn",
       {
         "argsIgnorePattern": "^_",
         "varsIgnorePattern": "^_",
         "caughtErrorsIgnorePattern": "^_",
         "ignoreRestSiblings": true
       }
     ],
-    "curly": "error",
+    "curly": "warn",
     "eqeqeq": [
-      "error",
+      "warn",
       "always",
       {
         "null": "never"
       }
     ],
-    "no-constant-condition": [
-      "error",
+    "no-warning-comments": [
+      "warn",
       {
-        "checkLoops": false
+        "terms": ["todo"]
       }
     ],
-    "no-restricted-syntax": [
-      "error",
-      "MemberExpression[object.property.name='constructor'][property.name='name']"
+    "eslint/no-constant-condition": [
+      "warn",
+      {
+        "checkLoops": false
+      }
     ],
-    "no-throw-literal": "error",
-    "semi": "off",
-    "unicorn/prefer-module": "error",
-    "mocha/no-pending-tests": "error",
-    "mocha/no-exclusive-tests": "error"
-  },
-  "overrides": []
+    "eslint/no-throw-literal": "warn",
+    "unicorn/prefer-module": "warn",
+    "mocha/no-pending-tests": "warn",
+    "mocha/no-exclusive-tests": "warn"
+  }
 }

package.json

@@ -37,12 +37,13 @@
     "@pnpm/meta-updater": "^2.0.6",
     "@types/node": "^24.12.0",
     "esbuild": "^0.27.4",
-    "oxfmt": "^0.16.0",
-    "oxlint-tsgolint": "^0.17.1",
-    "oxlint": "^1.15.0",
-    "stylelint-config-standard-scss": "^17.0.0",
-    "stylelint-config-standard": "^40.0.0",
+    "eslint-plugin-mocha": "^11.2.0",
+    "oxfmt": "^0.41.0",
+    "oxlint": "^1.56.0",
+    "oxlint-tsgolint": "^0.17.2",
     "stylelint": "^17.5.0",
+    "stylelint-config-standard": "^40.0.0",
+    "stylelint-config-standard-scss": "^17.0.0",
     "syncpack": "^14.2.1",
     "tsx": "^4.21.0",
     "typescript": "^5.9.3"

packages/app-vscode/.oxlintrc.json

@@ -0,0 +1,22 @@
+{
+  "extends": ["../../.oxlintrc.json"],
+  "ignorePatterns": ["src/keyboard/grammar/generated/**"],
+  "overrides": [
+    {
+      "files": ["src/scripts/**/*.ts"],
+      "rules": {
+        "no-restricted-imports": [
+          "warn",
+          {
+            "paths": [
+              {
+                "name": "vscode",
+                "message": "Scripts shouldn't depend on vscode"
+              }
+            ]
+          }
+        ]
+      }
+    }
+  ]
+}

packages/app-vscode/src/ide/vscode/VSCodeScopeVisualizer/VscodeFancyRangeHighlighter/generateDifferentiatedRanges.ts

@@ -27,11 +27,9 @@ export function* generateDifferentiatedRanges(
 
   for (const range of ranges) {
     // Remove any ranges that have ended before the start of the current range.
-    currentRanges = [
-      ...currentRanges.filter(({ range: previousRange }) =>
-        generalizedRangeTouches(previousRange, range),
-      ),
-    ];
+    currentRanges = currentRanges.filter(({ range: previousRange }) =>
+      generalizedRangeTouches(previousRange, range),
+    );
 
     const differentiatedRange = {
       range,

packages/app-vscode/src/ide/vscode/VscodeToggleBreakpoint.ts

@@ -5,7 +5,7 @@ import type { VscodeTextEditor } from "./VscodeTextEditor";
 
 export async function vscodeToggleBreakpoint(
   editor: VscodeTextEditor,
-  ranges?: GeneralizedRange[] | undefined,
+  ranges?: GeneralizedRange[],
 ): Promise<void> {
   if (ranges == null) {
     return await vscode.commands.executeCommand(

packages/app-web-docs/package.json

@@ -45,7 +45,6 @@
     "@cursorless/lib-node-common": "workspace:*",
     "@docusaurus/module-type-aliases": "^3.9.2",
     "@docusaurus/types": "^3.9.2",
-    "@tsconfig/docusaurus": "^2.0.9",
     "@types/mdast": "^4.0.4",
     "@types/react": "^19.2.14",
     "sass": "^1.98.0",

packages/app-web-docs/sidebar.js

@@ -1,4 +1,4 @@
-// @ts-check
+// @ts-nocheck
 
 /** @type {import('@docusaurus/plugin-content-docs').SidebarsConfig} */
 export default {

packages/app-web-docs/src/plugins/scope-tests-plugin.ts

@@ -6,13 +6,16 @@ import {
 import type { LoadContext, Plugin, PluginOptions } from "@docusaurus/types";
 import * as fs from "node:fs";
 import * as path from "node:path";
+import { fileURLToPath } from "node:url";
 import type {
   Fixture,
   Scope,
   ScopeTests,
   Target,
 } from "../docs/components/types";
 
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+
 export default function prepareAssetsPlugin(
   _context: LoadContext,
   _options: PluginOptions,

packages/app-web-docs/tsconfig.json

@@ -1,14 +1,16 @@
 {
-  "extends": ["@tsconfig/docusaurus/tsconfig.json", "../../tsconfig.web.json"],
+  "extends": ["../../tsconfig.web.json"],
   "compilerOptions": {
     "jsx": "react-jsx",
-    "jsxImportSource": "react"
+    "jsxImportSource": "react",
+    "skipLibCheck": true,
+    "types": ["@docusaurus/theme-classic"],
   },
   "include": [
     "src/**/*.ts",
     "src/**/*.tsx",
     "src/**/*.json",
-    "../../resources/typings/**/*.d.ts"
+    "../../resources/typings/**/*.d.ts",
   ],
-  "exclude": ["node_modules"]
+  "exclude": ["node_modules"],
 }