@cubejs-backend/linter 1.7.36 → 1.7.38

package/node.json

@@ -0,0 +1,31 @@
+// The Node preset: Cube's shared oxlint configuration plus what only applies to the
+// backend packages. Frontend packages (@cubejs-client/*) extend the repo-root config or
+// airbnb-react.json instead, so nothing here reaches them.
+{
+  "$schema": "../../node_modules/oxlint/configuration_schema.json",
+  "extends": ["./.oxlintrc.json"],
+  "rules": {
+    // A control-flow block that opens right under an unrelated statement reads as part of
+    // it. Backend code is dense with them -- driver retry loops, connection teardown --
+    // so require the separating blank line. `for` covers for/for-in/for-of, `do` covers
+    // do/while; a plain `while` is left alone because its condition sits on the keyword
+    // line and already reads as its own statement.
+    "@stylistic/padding-line-between-statements": [
+      "warn",
+      { "blankLine": "always", "prev": "*", "next": ["try", "do", "for"] }
+    ]
+  },
+  "overrides": [
+    {
+      // A top-level `env` key is not inherited through `extends` -- oxlint treats it as
+      // per-config-file -- but an `overrides` entry's `env` is. Declaring the Node
+      // environment this way is what lets the backend packages extend this preset and get
+      // it, instead of each restating `{ "node": true, "es6": true }`.
+      "files": ["**"],
+      "env": {
+        "node": true,
+        "es6": true
+      }
+    }
+  ]
+}

package/package.json

@@ -1,35 +1,29 @@
 {
   "name": "@cubejs-backend/linter",
-  "description": "Cube.js ESLint (virtual package) for linting code",
+  "description": "Cube shared oxlint configuration (virtual package) for linting code",
   "author": "Cube Dev, Inc.",
-  "version": "1.7.36",
+  "version": "1.7.38",
   "repository": {
     "type": "git",
     "url": "https://github.com/cube-js/cube.git",
-    "directory": "packages/cubejs-mssql-driver"
+    "directory": "packages/cubejs-linter"
   },
   "engines": {
     "node": ">=20.0.0"
   },
-  "main": "index.js",
-  "peerDependencies": {
-    "eslint": ">=8.57"
-  },
   "dependencies": {
-    "@stylistic/eslint-plugin-ts": "^3.1.0",
-    "@typescript-eslint/eslint-plugin": "^8.46.0",
-    "@typescript-eslint/parser": "^8.46.0",
-    "eslint": "^8.57.1",
-    "eslint-config-airbnb-base": "^14.2.1",
-    "eslint-plugin-import": "^2.22.1",
-    "eslint-plugin-node": "^10.0.0"
+    "@stylistic/eslint-plugin": "^5.10.0",
+    "oxlint": "^1.82.0"
   },
   "files": [
-    "index.js"
+    ".oxlintrc.json",
+    "node.json",
+    "airbnb-base.json",
+    "airbnb-react.json"
   ],
   "license": "Apache-2.0",
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "0e1c035e2b48a6b44cfd1f10d5c6e94b4bba76d6"
+  "gitHead": "9ea1fda5dade5b8a4256acd20bcf77318fe13a72"
 }

package/index.js

@@ -1,114 +0,0 @@
-module.exports = {
-  root: true,
-  extends: 'airbnb-base',
-  env: {
-    node: true,
-  },
-  plugins: ['import', '@typescript-eslint/eslint-plugin', '@stylistic/ts'],
-  parser: '@typescript-eslint/parser',
-  parserOptions: {
-    sourceType: 'module',
-    ecmaVersion: 2020,
-    ecmaFeatures: {
-      legacyDecorators: true,
-    },
-  },
-  rules: {
-    'no-useless-constructor': 0,
-    'max-classes-per-file': 0,
-    'prefer-object-spread': 0,
-    'import/no-unresolved': 0,
-    'comma-dangle': 0,
-    'no-console': 0,
-    'arrow-parens': 0,
-    'import/prefer-default-export': 0,
-    'import/extensions': 0,
-    quotes: ['warn', 'single'],
-    'no-prototype-builtins': 0,
-    'class-methods-use-this': 0,
-    'no-param-reassign': 0,
-    'no-mixed-operators': 0,
-    'no-else-return': 0,
-    'prefer-promise-reject-errors': 0,
-    'no-plusplus': 0,
-    'no-await-in-loop': 0,
-    'operator-linebreak': 0,
-    // linter can't fix this itself and, in some cases, conflicts with `arrow-body-style`
-    'max-len': 0,
-    'no-trailing-spaces': ['warn', { skipBlankLines: true }],
-    'object-curly-newline': 0,
-    // TypeScript Recommended
-    'no-array-constructor': 'off',
-    '@typescript-eslint/no-array-constructor': 'error',
-    'no-empty-function': 'off',
-    '@typescript-eslint/no-empty-function': 'error',
-    'no-extra-semi': 'off',
-    '@stylistic/ts/no-extra-semi': 'error',
-    'no-underscore-dangle': 'off',
-    'no-unused-vars': 'off',
-    '@typescript-eslint/no-unused-vars': [
-      'warn',
-      {
-        argsIgnorePattern: '^_.*',
-        varsIgnorePattern: '^_.*',
-      },
-    ],
-    // '@typescript-eslint/no-var-requires': 'error',
-    '@typescript-eslint/prefer-as-const': 'error',
-    '@typescript-eslint/prefer-namespace-keyword': 'error',
-    '@typescript-eslint/triple-slash-reference': 'error',
-    '@stylistic/ts/type-annotation-spacing': 'error',
-    '@stylistic/ts/space-infix-ops': 'error',
-    'no-restricted-syntax': [
-      'error',
-      {
-        selector: 'ForInStatement',
-        message: 'for..in loops iterate over the entire prototype chain, which is virtually never what you want. Use Object.{keys,values,entries}, and iterate over the resulting array.',
-      },
-      {
-        selector: 'LabeledStatement',
-        message: 'Labels are a form of GOTO; using them makes code confusing and hard to maintain and understand.',
-      },
-      {
-        selector: 'WithStatement',
-        message: '`with` is disallowed in strict mode because it makes code impossible to predict and optimize.',
-      },
-    ],
-  },
-  overrides: [
-    {
-      files: ['*.ts', '*.tsx'],
-      rules: {
-        // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/src/configs/eslint-recommended.ts
-        'constructor-super': 'off', // ts(2335) & ts(2377)
-        'getter-return': 'off', // ts(2378)
-        'no-const-assign': 'off', // ts(2588)
-        'no-dupe-args': 'off', // ts(2300)
-        'no-dupe-class-members': 'off', // ts(2393) & ts(2300)
-        'no-dupe-keys': 'off', // ts(1117)
-        'no-func-assign': 'off', // ts(2539)
-        'no-import-assign': 'off', // ts(2539) & ts(2540)
-        'no-new-symbol': 'off', // ts(2588)
-        'no-obj-calls': 'off', // ts(2349)
-        'no-redeclare': 'off', // ts(2451)
-        'no-setter-return': 'off', // ts(2408)
-        'no-this-before-super': 'off', // ts(2376)
-        'no-undef': 'off', // ts(2304)
-        'no-unreachable': 'off', // ts(7027)
-        'no-unsafe-negation': 'off', // ts(2365) & ts(2360) & ts(2358)
-        'no-var': 'error', // ts transpiles let/const to var, so no need for vars any more
-        'prefer-const': 'error', // ts provides better types with const
-        'prefer-rest-params': 'error', // ts provides better types with rest args over arguments
-        'prefer-spread': 'error', // ts transpiles spread to apply, so no need for manual apply
-        'valid-typeof': 'off', // ts(2367)
-        // TypeScript Additional
-        '@typescript-eslint/explicit-member-accessibility': 'error',
-        'no-shadow': 'off',
-        '@typescript-eslint/no-shadow': ['error', { ignoreTypeValueShadow: true }],
-        'no-duplicate-imports': 'off',
-        semi: 'off',
-        '@stylistic/ts/semi': 'error',
-      },
-    },
-  ],
-};