@cabify/eslint-config 3.0.1-beta-6 → 3.0.1-beta-9

package/configs/base.js

@@ -1,28 +1,30 @@
-const globals = require('globals');
+/* eslint-disable import/extensions */
+import globals from 'globals';
 
-const { isPackageAvailable } = require('../utils');
+import { isPackageAvailable } from '../utils.js';
+import bestPractices from './best-practices.js';
+import errors from './errors.js';
+import es6 from './es6.js';
+import formats from './formats.js';
+import imports from './imports.js';
+import jest from './jest.js';
+import lodash from './lodash.js';
+import node from './node.js';
+import postcss from './postcss.js';
+import promises from './promises.js';
+import react from './react.js';
+import reactA11y from './react-a11y.js';
+import storybook from './storybook.js';
+import strict from './strict.js';
+import style from './style.js';
+import variables from './variables.js';
 
-const isTSAvailable = isPackageAvailable('typescript');
-const isJestAvailable = isPackageAvailable('jest');
-
-const storybook = require('./storybook');
-const ts = require('./ts');
-const postcss = require('./postcss');
-
-const bestPractices = require('./best-practices');
-const errors = require('./errors');
-const es6 = require('./es6');
-const imports = require('./imports');
-const node = require('./node');
-const promises = require('./promises');
-const strict = require('./strict');
-const style = require('./style');
-const variables = require('./variables');
-const react = require('./react');
-const lodash = require('./lodash');
-const reactA11y = require('./react-a11y');
-const jest = require('./jest');
-const formats = require('./formats');
+const isTSAvailable = await isPackageAvailable('typescript');
+let tsConfigs = [];
+if (isTSAvailable) {
+  tsConfigs = await import('./ts.js');
+}
+const isJestAvailable = await isPackageAvailable('jest');
 
 const configs = [
   bestPractices,
@@ -42,10 +44,6 @@ const configs = [
 ].filter(Boolean);
 
 const overrides = [
-  isTSAvailable && {
-    files: ['**/*.ts', '**/*.tsx'],
-    ...ts,
-  },
   {
     files: ['*.story.tsx', '*.stories.tsx'],
     ...storybook,
@@ -56,7 +54,7 @@ const overrides = [
   },
 ].filter(Boolean);
 
-module.exports = [
+export default [
   ...configs,
   {
     name: 'base-cabify-eslint-config',
@@ -72,5 +70,6 @@ module.exports = [
       strict: 'error',
     },
   },
+  ...tsConfigs,
   ...overrides,
 ];

package/configs/best-practices.js

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   name: 'best-practices-cabify-eslint-config',
   rules: {
     // enforces getter/setter pairs in objects

package/configs/errors.js

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   name: 'errors-cabify-eslint-config',
   rules: {
     // Enforce “for” loop update clause moving the counter in the right direction

package/configs/es6.js

@@ -1,18 +1,20 @@
-module.exports = {
+import globals from 'globals';
+
+export default {
   name: 'ES6-cabify-eslint-config',
-  // languageOptions: {
-  //   globals: {
-  //     es6: true,
-  //   },
-  //   ecmaVersion: 6,
-  //   sourceType: 'module',
-  //   parserOptions: {
-  //     ecmaFeatures: {
-  //       generators: false,
-  //       objectLiteralDuplicateProperties: false,
-  //     },
-  //   },
-  // },
+  languageOptions: {
+    ecmaVersion: 6,
+    sourceType: 'module',
+    globals: {
+      ...globals.es2015,
+    },
+    parserOptions: {
+      ecmaFeatures: {
+        generators: false,
+        objectLiteralDuplicateProperties: false,
+      },
+    },
+  },
 
   rules: {
     // verify super() callings in constructors

package/configs/formats.js

@@ -1,6 +1,6 @@
-const eslintPluginPrettierRecommended = require('eslint-plugin-prettier/recommended');
+import eslintPluginPrettierRecommended from 'eslint-plugin-prettier/recommended';
 
-module.exports = {
+export default {
   name: 'formats-cabify-eslint-config',
   ...eslintPluginPrettierRecommended,
 };

package/configs/imports.js

@@ -1,13 +1,14 @@
-const importPlugin = require('eslint-plugin-import');
-const simpleImportSort = require('eslint-plugin-simple-import-sort');
+import importPlugin from 'eslint-plugin-import';
+import simpleImportSort from 'eslint-plugin-simple-import-sort';
+import globals from 'globals';
 
-module.exports = {
+export default {
   name: 'imports-cabify-eslint-config',
-  // languageOptions: {
-  //   globals: {
-  //     es6: true,
-  //   },
-  // },
+  languageOptions: {
+    globals: {
+      ...globals.es2015,
+    },
+  },
   plugins: {
     import: importPlugin,
     'simple-import-sort': simpleImportSort,

package/configs/jest.js

@@ -1,6 +1,6 @@
-const jest = require('eslint-plugin-jest');
+import jest from 'eslint-plugin-jest';
 
-module.exports = {
+export default {
   name: 'jest-cabify-eslint-config',
   ...jest.configs['flat/recommended'],
   rules: {

package/configs/lodash.js

@@ -1,6 +1,6 @@
-const lodashPlugin = require('eslint-plugin-lodash');
+import lodashPlugin from 'eslint-plugin-lodash';
 
-module.exports = {
+export default {
   name: 'lodash-cabify-eslint-config',
   plugins: { lodash: lodashPlugin },
   rules: {

package/configs/node.js

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   name: 'node-cabify-eslint-config',
   rules: {
     // enforce return after a callback

package/configs/postcss.js

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   name: 'post-css-cabify-eslint-config',
   rules: {
     'global-require': 'off',

package/configs/promises.js

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   name: 'promises-cabify-eslint-config',
   rules: {
     'no-floating-promises': 'off',

package/configs/react-a11y.js

@@ -1,17 +1,15 @@
-const jsxAllyPlugin = require('eslint-plugin-jsx-a11y');
+import jsxAllyPlugin from 'eslint-plugin-jsx-a11y';
 
-module.exports = {
+export default {
   name: 'react-a11y-cabify-eslint-config',
   plugins: { 'jsx-a11y': jsxAllyPlugin },
-
-  // languageOptions: {
-  //   parserOptions: {
-  //     ecmaFeatures: {
-  //       jsx: true,
-  //     },
-  //   },
-  // },
-
+  languageOptions: {
+    parserOptions: {
+      ecmaFeatures: {
+        jsx: true,
+      },
+    },
+  },
   rules: {
     // Enforce that anchors have content
     // https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/anchor-has-content.md

package/configs/react.js

@@ -1,7 +1,7 @@
-const react = require('eslint-plugin-react');
-const reactHooks = require('eslint-plugin-react-hooks');
+import react from 'eslint-plugin-react';
+import reactHooks from 'eslint-plugin-react-hooks';
 
-module.exports = {
+export default {
   name: 'react-cabify-eslint-config',
   plugins: {
     react,

package/configs/storybook.js

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   name: 'storybook-cabify-eslint-config',
   rules: {
     'import/no-default-export': 'off',

package/configs/strict.js

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   name: 'strict-cabify-eslint-config',
   rules: {
     // babel inserts `'use strict';` for us

package/configs/style.js

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   name: 'style-cabify-eslint-config',
   rules: {
     // require camel case names

package/configs/ts.js

@@ -1,10 +1,10 @@
-module.exports = {
+/* eslint-disable import/no-unresolved */
+/* eslint-disable no-param-reassign */
+import tseslint from 'typescript-eslint';
+
+const tsCustomConfig = {
   name: 'ts-cabify-eslint-config',
-  plugins: ['@typescript-eslint'],
-  extends: [
-    'plugin:@typescript-eslint/recommended',
-    'plugin:@typescript-eslint/recommended-requiring-type-checking',
-  ],
+  files: ['**/*.ts', '**/*.tsx'],
   rules: {
     '@typescript-eslint/restrict-template-expressions': [
       'error',
@@ -136,9 +136,26 @@ module.exports = {
     'no-redeclare': 'off', // superseeded by @typescript-eslint/no-redeclare
     'no-shadow': 'off', // superseeded by @typescript-eslint/no-shadow
   },
-  overrides: [
-    {
-      excludedFiles: '*.d.ts',
+  languageOptions: {
+    parserOptions: {
+      projectService: true,
+      tsconfigRootDir: import.meta.dirname,
     },
-  ],
+  },
+  ignores: ['*.d.ts'],
 };
+
+const tsLintConfig = tseslint.config(
+  ...tseslint.configs.recommended,
+  ...tseslint.configs.recommendedTypeChecked,
+  { ...tsCustomConfig },
+);
+
+if (tsLintConfig.length) {
+  tsLintConfig.forEach((tsconfig) => {
+    tsconfig.files = ['**/*.ts', '**/*.tsx'];
+    tsconfig.ignores = ['**/*.d.ts'];
+  });
+}
+
+export default tsLintConfig;

package/configs/variables.js

@@ -1,6 +1,6 @@
-const confusingBrowserGlobals = require('confusing-browser-globals');
+import confusingBrowserGlobals from 'confusing-browser-globals';
 
-module.exports = {
+export default {
   name: 'variables-cabify-eslint-config',
   rules: {
     // enforce or disallow variable initializations at definition

package/eslint.config.js

@@ -1,3 +1,4 @@
-const recommended = require('./recommended');
+// eslint-disable-next-line import/extensions
+import recommended from './recommended.js';
 
-module.exports = [...recommended];
+export default [...recommended];

package/package.json

@@ -1,7 +1,8 @@
 {
   "name": "@cabify/eslint-config",
-  "version": "3.0.1-beta-6",
+  "version": "3.0.1-beta-9",
   "description": "ESLint config for Cabify Javascript projects",
+  "type": "module",
   "scripts": {
     "build": "echo 'No build to perform'",
     "test": "npm run lint:check && npm run format:check",
@@ -38,7 +39,6 @@
     ".": "./recommended.js"
   },
   "dependencies": {
-    "@typescript-eslint/eslint-plugin": "^8.4.0",
     "@typescript-eslint/parser": "^8.4.0",
     "confusing-browser-globals": "^1.0.10",
     "eslint-config-prettier": "^9.1.0",
@@ -50,14 +50,15 @@
     "eslint-plugin-react": "^7.36.0",
     "eslint-plugin-react-hooks": "^4.6.2",
     "eslint-plugin-simple-import-sort": "^12.1.1",
-    "globals": "^15.9.0"
+    "globals": "^15.9.0",
+    "typescript-eslint": "^8.8.0"
   },
   "peerDependencies": {
-    "eslint": "9.11.0",
+    "eslint": ">= 9.11.0",
     "prettier": ">= 2.2.1"
   },
   "devDependencies": {
-    "eslint": "9.11.0",
+    "eslint": "^9.11.1",
     "prettier": "3.3.3"
   },
   "publishConfig": {

package/recommended.js

@@ -1,4 +1,6 @@
-const base = require('./configs/base');
-const eslintConfigPrettier = require('eslint-config-prettier');
+import eslintConfigPrettier from 'eslint-config-prettier';
 
-module.exports = [...base, eslintConfigPrettier];
+// eslint-disable-next-line import/extensions
+import base from './configs/base.js';
+
+export default [...base, eslintConfigPrettier];

package/utils.js

@@ -1,11 +1,11 @@
-module.exports = {
-  isPackageAvailable(packageName) {
-    try {
-      // eslint-disable-next-line global-require, import/no-dynamic-require
-      return !!require(packageName);
-      // eslint-disable-next-line no-unused-vars
-    } catch (e) {
-      return false;
-    }
-  },
-};
+// eslint-disable-next-line import/prefer-default-export
+export async function isPackageAvailable(packageName) {
+  try {
+    // Dynamically import the package
+    await import(packageName);
+    return true;
+  } catch (error) {
+    console.log(error);
+    return false;
+  }
+}