Eslint config passport next

+babel.js

@@ -0,0 +1,82 @@
+'use strict';
+
+// Move to own repo so can have `@babel/eslint-parser` and `@babel/eslint-plugin` as deps.
+
+// Adapted from MIT-licensed: https://github.com/futagoza/eslint-config-futagozaryuu/blob/master/packages/%40futagoza/eslint-config-core/stylistic-issues.js
+
+/**
+ * Allows passing in a whole config to wrap.
+ * @param {ESLintConfig} config
+ * @returns {ESLintConfig}
+ */
+module.exports = function (config) {
+  return {
+    parser: '@babel/eslint-parser',
+    parserOptions: {
+      ...config.parserOptions,
+      requireConfigFile: false
+    },
+
+    plugins: [
+      ...config.plugins,
+      '@babel/eslint-plugin'
+    ],
+    rules: {
+
+      ...config.rules,
+
+      // `@babel/eslint-plugin` provides better support for the following rules
+
+      'new-cap': 'off',
+      'no-invalid-this': 'off',
+      'no-unused-expressions': 'off',
+      'object-curly-spacing': 'off',
+      semi: 'off',
+
+      /**
+       * Require constructor names to begin with a capital letter.
+       *
+       * @see
+       * - http://eslint.org/docs/rules/new-cap
+       * - https://github.com/babel/babel/tree/master/eslint/babel-eslint-plugin#rules
+       */
+      '@babel/new-cap': config.rules['new-cap'] || 'off',
+
+      /**
+       * Disallow `this` keywords outside of classes or class-like objects.
+       *
+       * @see
+       * - http://eslint.org/docs/rules/no-invalid-this
+       * - https://github.com/babel/babel/tree/master/eslint/babel-eslint-plugin#rules
+       */
+      '@babel/no-invalid-this': config.rules['no-invalid-this'] || 'off',
+
+      /**
+       * Disallow unused expressions.
+       *
+       * @see
+       * - http://eslint.org/docs/rules/no-unused-expressions
+       * - https://github.com/babel/babel/tree/master/eslint/babel-eslint-plugin#rules
+       */
+      '@babel/no-unused-expressions': config.rules['no-unused-expressions'] || 'off',
+
+      /**
+       * Enforce consistent spacing inside braces (🔧 ).
+       *
+       * @see
+       * - http://eslint.org/docs/rules/object-curly-spacing
+       * - https://github.com/babel/babel/tree/master/eslint/babel-eslint-plugin#rules
+       */
+      '@babel/object-curly-spacing': config.rules['object-curly-spacing'] || 'off',
+
+      /**
+       * Require or disallow semicolons instead of ASI (🔧).
+       *
+       * @see
+       * - http://eslint.org/docs/rules/semi
+       * - https://github.com/babel/babel/tree/master/eslint/babel-eslint-plugin#rules
+       */
+      '@babel/semi': config.rules.semi || 'off'
+    }
+  };
+};

+modules.js

@@ -0,0 +1,24 @@
+'use strict';
+
+module.exports = {
+  parserOptions: {
+    ecmaFeatures: {globalReturn: false},
+    sourceType: 'module'
+  },
+  globals: {
+    __dirname: 'off',
+    __filename: 'off',
+    exports: 'off',
+    module: 'off',
+    require: 'off'
+  },
+  rules: {
+    'node/no-unsupported-features/es-syntax': [
+      'error',
+      // We could check for `ecmaVersion < 2020` to restrict `dynamicImport`,
+      //   but if present, typically compiling, and seems support was available
+      //   in earlier Node experimentally anyways
+      {ignores: ['modules', 'dynamicImport']}
+    ]
+  }
+};

+script-node.js

@@ -0,0 +1,17 @@
+'use strict';
+
+// As adding after defaults, recalculate ecmaVersion (Node plugins sets own
+//  `ecmaVersion`)
+const {env, ecmaVersion} = require('./detectEnv.js');
+
+module.exports = {
+  env,
+  extends: ['plugin:node/recommended-script'],
+  parserOptions: {
+    ecmaVersion
+  },
+  rules: {
+    'import/no-commonjs': 'off',
+    strict: ['error']
+  }
+};

+script.js

@@ -0,0 +1,14 @@
+'use strict';
+
+module.exports = {
+  parserOptions: {
+    sourceType: 'script',
+    ecmaFeatures: {
+      globalReturn: false
+    }
+  },
+  rules: {
+    'import/no-commonjs': 'off',
+    strict: ['error']
+  }
+};

.editorconfig

@@ -0,0 +1,15 @@
+; EditorConfig file: https://EditorConfig.org
+; Install the "EditorConfig" plugin into your editor to use
+
+root = true
+
+[*]
+charset = utf-8
+end_of_line = lf
+insert_final_newline = true
+indent_style = space
+indent_size = 2
+trim_trailing_whitespace = true
+
+[*.md]
+indent_size = 4

.eslintignore

@@ -0,0 +1,2 @@
+node_modules
+!*.js

.eslintrc.js

@@ -1,45 +1,32 @@
+'use strict';
 module.exports = {
-    env: {
-        browser: true,
-        commonjs: true,
-        es6: true,
-        node: true
+  extends: [
+    './sauron-node-script-overrides.js'
+  ],
+  settings: {
+    polyfills: [
+      'Object.entries',
+      'Object.values'
+    ]
+  },
+  overrides: [{
+    files: 'build.js',
+    globals: {
+      // Todo: Get rid of this usage (and esm) with native Node only
+      __dirname: true
     },
-    extends: ['eslint:recommended'],
-    plugins: ['jsdoc', 'markdown'],
-    parserOptions: {
-        ecmaVersion: 2018,
-        sourceType: 'module'
-    },
-    overrides: [{
-      files: ['**/*.md'],
-      rules: {
-        'eol-last': ['off'],
-        'no-console': ['off'],
-        'no-undef': ['off'],
-        'no-unused-vars': ['warn'],
-        'padded-blocks': ['off'],
-        'import/unambiguous': ['off'],
-        'import/no-unresolved': ['off'],
-        'node/no-missing-import': ['off']
-      }
-    }],
+    extends: [
+      './sauron-node-overrides.js'
+    ]
+  }, {
+    files: ['inherited-rules/**'],
     rules: {
-        indent: [
-            'error',
-            2
-        ],
-        'linebreak-style': [
-            'error',
-            'unix'
-        ],
-        quotes: [
-            'error',
-            'double'
-        ],
-        semi: [
-            'error',
-            'always'
-        ]
+      // Uses JSON.stringify
+      quotes: ['error', 'double'],
+      'quote-props': 'off'
     }
+  }],
+  rules: {
+    'max-len': 'off'
+  }
 };

.npmignore

@@ -0,0 +1 @@
+pnpm-lock.yaml