chore: lint rules -> default restriction to opt-out (from opt-in)

Author: rstoenescu

.oxlintrc.json

@@ -25,13 +25,13 @@
 
   "plugins": ["vue", "import", "eslint"],
 
-  // Enable all rules in the `correctness` and `suspicious` categories
   "categories": {
     "style": "error",
     "pedantic": "warn",
     "correctness": "error",
     "suspicious": "error",
-    "perf": "error"
+    "perf": "error",
+    "restriction": "error"
   },
 
   "rules": {
@@ -105,7 +105,23 @@
     ],
 
     /** Restriction rule overrides */
+    "no-void": "off",
+    "no-plusplus": "off",
+    "no-param-reassign": "off",
+    "no-console": "off",
+    "complexity": "off",
+    "no-empty-function": "off",
+    "no-empty": "off",
+    "no-relative-parent-imports": "off",
+    "class-methods-use-this": "off",
+    "no-bitwise": "off",
+    "max-props": "off",
+    "default-case": "off",
+    "no-div-regex": "off",
     "import/no-cycle": ["error", { "maxDepth": 3 }],
+    "import/no-default-export": "off",
+    "import/no-dynamic-require": "off",
+    "import/unambiguous": "off",
 
     /** Pedantic rule overrides */
     "unicorn/explicit-length-check": [
@@ -135,22 +151,17 @@
     },
 
     {
-      "files": [
-        "{app-vite,cli,create-quasar,extras,icongenie,app-webpack}/**/*"
-      ],
-
+      "files": ["{app-vite,cli,create-quasar,extras,icongenie,app-webpack}/**"],
       "env": {
         "node": true
       },
-
       "rules": {
         "import/first": "off"
       }
     },
 
     {
       "files": ["ui/src/**/*.js"],
-
       "globals": {
         "__QUASAR_VERSION__": "readonly",
         "__QUASAR_SSR__": "readonly",
@@ -165,6 +176,23 @@
       "rules": {
         "unicorn/consistent-function-scoping": "off"
       }
+    },
+
+    {
+      "files": [
+        "{extras,app-webpack,utils/babel-preset-app,utils/ssr-helpers}/**",
+        "**/*.cjs"
+      ],
+      "rules": {
+        "no-commonjs": "off"
+      }
+    },
+
+    {
+      "files": ["docs/src/examples/**"],
+      "rules": {
+        "no-alert": "off"
+      }
     }
   ]
 }

app-webpack/lib/utils/print-webpack-issue/extractWebpackError.js

@@ -9,7 +9,7 @@ const requestShortener = new RequestShortener(process.cwd())
 */
 
 function getOriginalErrorStack(e) {
-  // oxlint-disable-next-line eqeqeq
+  // oxlint-disable-next-line eqeqeq no-eq-null
   while (e.error != null) {
     e = e.error
   }

docs/.oxlintrc.json

@@ -10,7 +10,7 @@
     "prepare-docs": "node build/copy-quasar-api.js",
     "dev": "pnpm prepare-docs && quasar dev",
     "dev:ssr": "pnpm prepare-docs && quasar dev -m ssr",
-    "build": "quasar clean && pnpm prepare-docs && node build/search.js && pnpm run lint && quasar build -m ssr && node build/fix-ssr-manifest.js && mkdir -p dist/quasar.dev/client/examples/ && cp -R src/examples/* dist/quasar.dev/client/examples/ && node build/ssg.js",
+    "build": "quasar clean && pnpm prepare-docs && node build/search.js && quasar build -m ssr && node build/fix-ssr-manifest.js && mkdir -p dist/quasar.dev/client/examples/ && cp -R src/examples/* dist/quasar.dev/client/examples/ && node build/ssg.js",
     "relnotes": "node build/release-notes/index.js",
     "lint": "oxfmt && oxlint --fix",
     "lint:check": "oxfmt --check && oxlint",

docs/package.json

@@ -4,6 +4,7 @@
   "private": true,
   "license": "ISC",
   "author": "",
+  "type": "module",
   "main": "index.js",
   "scripts": {
     "dev": "node server.js",

ui/playground/upload-server/package.json

@@ -1,10 +1,10 @@
-const express = require('express')
-const app = express()
-const formidable = require('formidable')
-const path = require('path')
-const fse = require('fs-extra')
-const throttle = require('express-throttle-bandwidth')
+import express from 'express'
+import formidable from 'formidable'
+import path from 'node:path'
+import fse from 'fs-extra'
+import throttle from 'express-throttle-bandwidth'
 
+const app = express()
 const port = process.env.PORT || 4444,
   folder = path.join(__dirname, 'files')
 

ui/playground/upload-server/server.js

@@ -74,7 +74,7 @@ export function isDeepEqual(a, b) {
       return true
     }
 
-    // oxlint-disable-next-line eqeqeq
+    // oxlint-disable-next-line eqeqeq no-eq-null
     if (a.buffer != null && a.buffer.constructor === ArrayBuffer) {
       length = a.length
 

ui/src/utils/is/is.js

@@ -65,7 +65,7 @@ export default function viteSingleFile() {
 
         for (const jsName of jsAssets) {
           const jsChunk = bundle[jsName]
-          // oxlint-disable-next-line eqeqeq
+          // oxlint-disable-next-line eqeqeq no-eq-null
           if (jsChunk.code != null) {
             deleteList.push(jsName)
             replacedHtml = replaceScript(