Merge pull request #477 from observablehq/observable-update-3

Observable update 3

.editorconfig

@@ -10,7 +10,12 @@ insert_final_newline = true
 
 [cspell.json]
 indent_size = 4
-insert_final_newline = false
 
-[website/blog/*.md]
+[{,website/blog/,changelog_unreleased/**/}*.md]
 trim_trailing_whitespace = false
+
+[tests/{**/__snapshots__/*, tests/format/**/*}]
+trim_trailing_whitespace = false
+
+[tests/format/**/jsfmt.spec.js]
+trim_trailing_whitespace = true

.eslintignore

@@ -1,13 +1,15 @@
+.tmp
 !/*.js
-/tests/**/*.js
-!/tests/**/jsfmt.spec.js
+/tests/format/**/*.js
+/tests/integration/cli/
+!/tests/format/**/jsfmt.spec.js
 !/**/.eslintrc.js
-/test*.js
+/test*.*
 /scripts/release/node_modules
 /coverage/
 /dist/
 **/node_modules/**
 /website/build/
 /website/static/playground.js
 /website/static/lib/
-/tests_integration/cli/
+/vendors/

.eslintrc.js

@@ -0,0 +1,333 @@
+"use strict";
+const { isCI } = require("ci-info");
+
+module.exports = {
+  root: true,
+  env: {
+    es2022: true,
+    node: true,
+  },
+  parserOptions: {
+    ecmaVersion: "latest",
+  },
+  reportUnusedDisableDirectives: true,
+  extends: ["eslint:recommended", "prettier"],
+  plugins: [
+    "prettier-internal-rules",
+    "import",
+    "regexp",
+    "unicorn",
+    "@typescript-eslint",
+  ],
+  settings: {
+    "import/internal-regex": "^linguist-languages/",
+  },
+  rules: {
+    "@typescript-eslint/prefer-ts-expect-error": "error",
+    "arrow-body-style": ["error", "as-needed"],
+    curly: "error",
+    "dot-notation": "error",
+    eqeqeq: "error",
+    "no-console": isCI ? "error" : "warn",
+    "no-else-return": [
+      "error",
+      {
+        allowElseIf: false,
+      },
+    ],
+    "no-implicit-coercion": "error",
+    "no-inner-declarations": "error",
+    "no-restricted-syntax": [
+      "error",
+      // `!foo === bar` and `!foo !== bar`
+      'BinaryExpression[operator=/^[!=]==$/] > UnaryExpression.left[operator="!"]',
+    ],
+    "no-return-await": "error",
+    "no-unneeded-ternary": "error",
+    "no-useless-return": "error",
+    "no-unused-vars": [
+      "error",
+      {
+        ignoreRestSiblings: true,
+      },
+    ],
+    "no-var": "error",
+    "object-shorthand": "error",
+    "one-var": ["error", "never"],
+    "prefer-arrow-callback": "error",
+    "prefer-const": [
+      "error",
+      {
+        destructuring: "all",
+      },
+    ],
+    "prefer-destructuring": [
+      "error",
+      {
+        VariableDeclarator: {
+          array: false,
+          object: true,
+        },
+        AssignmentExpression: {
+          array: false,
+          object: false,
+        },
+      },
+      {
+        enforceForRenamedProperties: false,
+      },
+    ],
+    "prefer-object-spread": "error",
+    "prefer-rest-params": "error",
+    "prefer-spread": "error",
+    "prettier-internal-rules/jsx-identifier-case": "error",
+    "prettier-internal-rules/no-identifier-n": "error",
+    quotes: [
+      "error",
+      "double",
+      {
+        avoidEscape: true,
+      },
+    ],
+    "require-await": "error",
+    strict: "error",
+    "symbol-description": "error",
+    yoda: [
+      "error",
+      "never",
+      {
+        exceptRange: true,
+      },
+    ],
+
+    "import/extensions": ["error", "ignorePackages"],
+    "import/no-extraneous-dependencies": [
+      "error",
+      {
+        devDependencies: ["tests*/**", "scripts/**"],
+      },
+    ],
+    "import/order": "error",
+    "import/no-anonymous-default-export": "error",
+
+    "regexp/match-any": [
+      "error",
+      {
+        allows: ["dotAll"],
+      },
+    ],
+    "regexp/no-unused-capturing-group": "error",
+    "regexp/no-useless-flag": [
+      "error",
+      {
+        strictTypes: false,
+      },
+    ],
+    "regexp/no-useless-lazy": "error",
+
+    "unicorn/better-regex": "error",
+    "unicorn/explicit-length-check": "error",
+    "unicorn/filename-case": "error",
+    "unicorn/new-for-builtins": "error",
+    "unicorn/no-array-for-each": "error",
+    "unicorn/no-array-push-push": "error",
+    "unicorn/no-new-array": "error",
+    "unicorn/no-unreadable-iife": "error",
+    "unicorn/no-useless-length-check": "error",
+    "unicorn/no-useless-promise-resolve-reject": "error",
+    "unicorn/no-useless-undefined": "error",
+    "unicorn/prefer-array-flat": [
+      "error",
+      {
+        functions: ["flat", "flatten"],
+      },
+    ],
+    "unicorn/prefer-array-flat-map": "error",
+    "unicorn/prefer-array-some": "error",
+    "unicorn/prefer-includes": "error",
+    "unicorn/prefer-json-parse-buffer": "error",
+    "unicorn/prefer-native-coercion-functions": "error",
+    "unicorn/prefer-number-properties": "error",
+    "unicorn/prefer-optional-catch-binding": "error",
+    "unicorn/prefer-regexp-test": "error",
+    "unicorn/prefer-spread": "error",
+    "unicorn/prefer-string-slice": "error",
+    "unicorn/prefer-string-starts-ends-with": "error",
+    "unicorn/prefer-switch": "error",
+    "unicorn/prefer-type-error": "error",
+    "unicorn/template-indent": "error",
+    "unicorn/text-encoding-identifier-case": "error",
+  },
+  overrides: [
+    {
+      files: [
+        "scripts/**/*.js",
+        "scripts/**/*.mjs",
+        "tests/config/install-prettier.js",
+      ],
+      rules: {
+        "no-console": "off",
+      },
+    },
+    {
+      files: ["scripts/**/*.mjs"],
+      rules: {
+        "unicorn/prefer-top-level-await": "error",
+      },
+    },
+    {
+      files: [
+        "**/*.mjs",
+        "scripts/release/**/*.js",
+        "scripts/tools/bundle-test/**/*.js",
+      ],
+      parserOptions: {
+        sourceType: "module",
+      },
+      rules: {
+        "unicorn/prefer-module": "error",
+        "unicorn/prefer-node-protocol": "error",
+      },
+    },
+    {
+      files: [
+        "tests/format/**/jsfmt.spec.js",
+        "tests/config/**/*.js",
+        "tests/integration/**/*.js",
+        "scripts/release/__tests__/**/*.spec.js",
+      ],
+      env: {
+        jest: true,
+      },
+      plugins: ["jest"],
+      rules: {
+        "jest/valid-expect": [
+          "error",
+          {
+            alwaysAwait: true,
+          },
+        ],
+        "jest/prefer-to-be": "error",
+      },
+    },
+    {
+      files: ["tests/integration/**/*.js"],
+      rules: {
+        "prettier-internal-rules/await-cli-tests": "error",
+      },
+    },
+    {
+      files: ["tests/**/*.js"],
+      rules: {
+        strict: "off",
+        "unicorn/prefer-array-flat": "off",
+        "unicorn/prefer-array-flat-map": "off",
+      },
+      globals: {
+        run_spec: false,
+      },
+    },
+    {
+      files: ["src/cli/*.js"],
+      rules: {
+        "no-restricted-modules": [
+          "error",
+          {
+            patterns: ["../"],
+          },
+        ],
+      },
+    },
+    {
+      files: ["src/cli/*/*.js"],
+      rules: {
+        "no-restricted-modules": [
+          "error",
+          {
+            patterns: ["../../"],
+          },
+        ],
+      },
+    },
+    {
+      files: "src/language-js/needs-parens.js",
+      rules: {
+        "prettier-internal-rules/better-parent-property-check-in-needs-parens":
+          "error",
+      },
+    },
+    {
+      files: "src/**/*.js",
+      rules: {
+        "prettier-internal-rules/consistent-negative-index-access": "error",
+        "prettier-internal-rules/flat-ast-path-call": "error",
+        "prettier-internal-rules/no-conflicting-comment-check-flags": "error",
+        "prettier-internal-rules/no-doc-builder-concat": "error",
+        "prettier-internal-rules/no-empty-flat-contents-for-if-break": "error",
+        "prettier-internal-rules/no-unnecessary-ast-path-call": "error",
+        "prettier-internal-rules/prefer-ast-path-each": "error",
+        "prettier-internal-rules/prefer-indent-if-break": "error",
+        "prettier-internal-rules/prefer-is-non-empty-array": "error",
+      },
+    },
+    {
+      files: ["src/language-*/**/*.js"],
+      rules: {
+        "prettier-internal-rules/directly-loc-start-end": "error",
+      },
+    },
+    {
+      files: ["src/language-js/**/*.js"],
+      rules: {
+        "prettier-internal-rules/no-node-comments": [
+          "error",
+          {
+            file: "src/language-js/utils/index.js",
+            functions: ["hasComment", "getComments"],
+          },
+          "src/language-js/pragma.js",
+          "src/language-js/parse/postprocess/*.js",
+          "src/language-js/parse/babel.js",
+          "src/language-js/parse/meriyah.js",
+          "src/language-js/parse/json.js",
+          "src/language-js/parse/acorn.js",
+        ],
+      },
+    },
+    {
+      files: ["website/**/*"],
+      env: {
+        browser: true,
+        worker: true,
+      },
+      extends: ["plugin:react/recommended"],
+      settings: {
+        react: {
+          version: "17",
+        },
+      },
+      rules: {
+        "import/no-extraneous-dependencies": "off",
+        "react/display-name": "off",
+        "react/no-deprecated": "off",
+        "react/prop-types": "off",
+        "unicorn/filename-case": "off",
+      },
+    },
+    {
+      files: ["website/playground/**/*"],
+      parserOptions: {
+        sourceType: "module",
+      },
+    },
+    {
+      files: ["bin/prettier.js"],
+      parserOptions: {
+        ecmaVersion: 5,
+      },
+      rules: {
+        "no-var": "off",
+      },
+    },
+  ],
+};