lint - prettier

Linting with prettier

.eslintrc.cjs

@@ -0,0 +1,291 @@
+// @ts-check
+/** @typedef {import("@typescript-eslint/utils/dist/ts-eslint/Linter.d").Linter.Config} EslintConfig */
+/** @typedef {import("@typescript-eslint/utils/dist/ts-eslint/Linter.d").Linter.Config["ignorePatterns"]} EslintIgnorePatterns */
+/** @typedef {import("@typescript-eslint/utils/dist/ts-eslint/Linter.d").Linter.RulesRecord} EslintRules */
+/** @typedef {import("@typescript-eslint/utils/dist/ts-eslint/Linter.d").Linter.ConfigOverride} ConfigOverride */
+
+/** @type EslintIgnorePatterns */
+const ignorePatterns = ["dist", "**/*.d.ts", "node_modules", "**/*.min.js"];
+
+/** @type EslintRules */
+const jsRules = {
+  // See documentation at https://eslint.org/docs/latest/rules/
+
+  eqeqeq: "warn",
+  "no-cond-assign": "warn",
+  "no-const-assign": "error",
+  "no-constant-condition": "warn",
+  "no-else-return": "warn",
+  "no-eval": "warn",
+  "no-extra-bind": "warn",
+  "no-extra-label": "warn",
+  "no-fallthrough": "warn",
+  "no-label-var": "error",
+  "no-lone-blocks": "warn",
+  "no-lonely-if": "warn",
+  "no-negated-condition": "warn",
+  "no-self-compare": "warn",
+  "no-sparse-arrays": "warn",
+  "no-unmodified-loop-condition": "warn",
+  "no-unneeded-ternary": [
+    "warn",
+    {
+      defaultAssignment: false,
+    },
+  ],
+  "no-unreachable-loop": "warn",
+  "no-useless-computed-key": "warn",
+  "no-useless-concat": "warn",
+  "no-useless-escape": "warn",
+  "no-var": "warn",
+  "no-with": "warn",
+  "operator-assignment": "warn",
+  "prefer-const": "warn",
+  "prefer-exponentiation-operator": "warn",
+  "prefer-regex-literals": [
+    "warn",
+    {
+      disallowRedundantWrapping: true,
+    },
+  ],
+  "use-isnan": [
+    "error",
+    {
+      enforceForIndexOf: true,
+    },
+  ],
+};
+
+/** @type EslintRules */
+const jsRuleOverrides = {
+  "block-spacing": "off",
+  "brace-style": "off",
+  "comma-dangle": "off",
+  "comma-spacing": "off",
+  "default-param-last": "off",
+  "dot-notation": "off",
+  "func-call-spacing": "off",
+  indent: "off",
+  "init-declarations": "off",
+  "key-spacing": "off",
+  "keyword-spacing": "off",
+  "lines-between-class-members": "off",
+  "no-array-constructor": "off",
+  "no-dupe-class-members": "off",
+  "no-duplicate-imports": "off",
+  "no-empty-function": "off",
+  "no-extra-parens": "off",
+  "no-extra-semi": "off",
+  "no-implied-eval": "off",
+  "no-invalid-this": "off",
+  "no-loop-func": "off",
+  "no-loss-of-precision": "off",
+  "no-magic-numbers": "off",
+  "no-redeclare": "off",
+  "no-restricted-imports": "off",
+  "no-shadow": "off",
+  "no-throw-literal": "off",
+  "no-unused-expressions": "off",
+  "no-unused-vars": "off",
+  "no-use-before-define": "off",
+  "no-useless-constructor": "off",
+  "object-curly-spacing": "off",
+  "padding-line-between-statements": "off",
+  quotes: "off",
+  "require-await": "off",
+  "return-await": "off",
+  semi: "off",
+  "space-before-blocks": "off",
+  "space-before-function-paren": "off",
+  "space-infix-ops": "off",
+};
+
+/** @type EslintRules */
+const tsRules = {
+  // See documentation at https://typescript-eslint.io/rules/
+
+  "@typescript-eslint/adjacent-overload-signatures": "error",
+  "@typescript-eslint/array-type": [
+    "warn",
+    {
+      default: "array",
+    },
+  ],
+  "@typescript-eslint/ban-ts-comment": [
+    "error",
+    {
+      "ts-nocheck": "allow-with-description",
+    },
+  ],
+  "@typescript-eslint/ban-types": "error",
+  "@typescript-eslint/consistent-type-imports": [
+    "warn",
+    {
+      disallowTypeAnnotations: false,
+      fixStyle: "separate-type-imports",
+      prefer: "type-imports",
+    },
+  ],
+  "@typescript-eslint/dot-notation": "warn",
+  "@typescript-eslint/naming-convention": [
+    "warn",
+    {
+      format: null,
+      leadingUnderscore: "allow",
+      modifiers: ["requiresQuotes"],
+      // Allow any format for quoted properties
+      selector: "property",
+      trailingUnderscore: "allow",
+    },
+    {
+      format: ["camelCase", "UPPER_CASE"],
+      leadingUnderscore: "allow",
+      // Allow camelCase or upper_case for non-quoted properties
+      selector: "property",
+      trailingUnderscore: "allow",
+    },
+    {
+      format: ["PascalCase"],
+      // Force PascalCase for types and typeLikes
+      selector: "typeLike",
+    },
+    {
+      format: ["PascalCase"],
+      leadingUnderscore: "allow",
+      modifiers: ["unused"],
+      // Allow leading Underscores for *unused* types and typeLikes
+      selector: "typeLike",
+    },
+    {
+      format: ["camelCase", "UPPER_CASE"],
+      leadingUnderscore: "allow",
+      // Allow variables to be camelCase or CONSTANT and _underscores_
+      selector: "variable",
+      trailingUnderscore: "allow",
+    },
+    {
+      format: ["camelCase"],
+      leadingUnderscore: "allow",
+      // Allow underscores on either side of camelCased defaults
+      selector: "default",
+      trailingUnderscore: "allow",
+    },
+  ],
+  "@typescript-eslint/no-confusing-non-null-assertion": "warn",
+  "@typescript-eslint/no-empty-interface": "warn",
+  "@typescript-eslint/no-extra-non-null-assertion": "error",
+  "@typescript-eslint/no-invalid-void-type": "error",
+  "@typescript-eslint/no-loop-func": "error",
+  "@typescript-eslint/no-misused-new": "error",
+  "@typescript-eslint/no-non-null-asserted-nullish-coalescing": "warn",
+  "@typescript-eslint/no-non-null-asserted-optional-chain": "error",
+  "@typescript-eslint/no-redeclare": [
+    "error",
+    {
+      ignoreDeclarationMerge: true,
+    },
+  ],
+  "@typescript-eslint/no-require-imports": "warn",
+  "@typescript-eslint/no-throw-literal": "error",
+  "@typescript-eslint/no-unnecessary-type-constraint": "warn",
+  "@typescript-eslint/no-unsafe-declaration-merging": "warn",
+  "@typescript-eslint/no-unused-expressions": [
+    "warn",
+    {
+      allowShortCircuit: true,
+      allowTernary: true,
+      allowTaggedTemplates: true,
+    },
+  ],
+  "@typescript-eslint/no-unused-vars": "warn",
+  "@typescript-eslint/no-use-before-define": [
+    "warn",
+    {
+      allowNamedExports: true,
+      classes: true,
+      functions: false,
+      variables: true,
+    },
+  ],
+  "@typescript-eslint/no-useless-constructor": "warn",
+  "@typescript-eslint/no-useless-empty-export": "warn",
+  "@typescript-eslint/no-var-requires": "error",
+  "@typescript-eslint/prefer-as-const": "error",
+  "@typescript-eslint/prefer-enum-initializers": "warn",
+  "@typescript-eslint/prefer-nullish-coalescing": [
+    "warn",
+    {
+      ignoreTernaryTests: false,
+    },
+  ],
+  "@typescript-eslint/prefer-optional-chain": "warn",
+  "@typescript-eslint/prefer-ts-expect-error": "error",
+  "@typescript-eslint/return-await": ["warn", "always"],
+  "@typescript-eslint/triple-slash-reference": "error",
+  "@typescript-eslint/switch-exhaustiveness-check": "warn",
+  "@typescript-eslint/unified-signatures": "warn",
+};
+/** @type EslintRules */
+const extendsRulesOverrides = {
+  "@typescript-eslint/no-misused-promises": [
+    "error",
+    { checksVoidReturn: false },
+  ],
+};
+
+/** @type EslintRules */
+const RECOMMENDED_REQUIRING_TYPE_CHECKING = {
+  "@typescript-eslint/await-thenable": "error",
+  "@typescript-eslint/no-floating-promises": "error",
+  "@typescript-eslint/no-for-in-array": "error",
+  "no-implied-eval": "off",
+  "@typescript-eslint/no-implied-eval": "error",
+  "@typescript-eslint/no-misused-promises": "error",
+  "@typescript-eslint/no-unnecessary-type-assertion": "error",
+  "@typescript-eslint/no-unsafe-argument": "error",
+  "@typescript-eslint/no-unsafe-assignment": "error",
+  "@typescript-eslint/no-unsafe-call": "error",
+  "@typescript-eslint/no-unsafe-member-access": "error",
+  "@typescript-eslint/no-unsafe-return": "error",
+  "require-await": "off",
+  // DEV! Disabled due to bug
+  // "@typescript-eslint/require-await": "error",
+  "@typescript-eslint/restrict-plus-operands": "error",
+  "@typescript-eslint/restrict-template-expressions": "error",
+  "@typescript-eslint/unbound-method": "error",
+};
+
+/** @type EslintConfig */
+const config = {
+  root: true,
+  parser: "@typescript-eslint/parser",
+  parserOptions: {
+    ecmaVersion: "latest",
+    sourceType: "module",
+    impliedStrict: true,
+    project: "./tsconfig.eslint.json",
+  },
+  env: {
+    browser: true,
+    node: true,
+  },
+  plugins: ["@typescript-eslint"],
+  extends: [
+    "plugin:@typescript-eslint/eslint-recommended",
+    // DEV! Disabled due to bug
+    // "plugin:@typescript-eslint/recommended-requiring-type-checking",
+  ],
+
+  overrides: [],
+
+  rules: {
+    ...RECOMMENDED_REQUIRING_TYPE_CHECKING,
+    ...jsRules,
+    ...jsRuleOverrides,
+    ...tsRules,
+    ...extendsRulesOverrides,
+  },
+  ignorePatterns,
+};
+
+module.exports = config;

.github/workflows/vsce-release.yml

@@ -3,24 +3,24 @@ name: Publish
 on:
   push:
     tags:
-      - '*'
+      - "*"
 
 jobs:
   build:
     name: Publish binaries
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v3
-    - name: Package
-      run: |
-           npm install
-           npm install @vscode/vsce
-           npm run ext:package
-           ls
-    - name: Upload vsix file to release
-      uses: svenstaro/upload-release-action@v2
-      with:
-        file: "*.vsix"
-        overwrite: true
-        file_glob: true
+      - uses: actions/checkout@v3
+      - name: Package
+        run: |
+          npm install
+          npm install @vscode/vsce
+          npm run ext:package
+          ls
+      - name: Upload vsix file to release
+        uses: svenstaro/upload-release-action@v2
+        with:
+          file: "*.vsix"
+          overwrite: true
+          file_glob: true

.prettierrc.cjs

@@ -0,0 +1,15 @@
+// ~/vscode/.prettierrc.js
+module.exports = {
+  arrowParens: "avoid",
+  bracketSameLine: true,
+  bracketSpacing: true,
+  embeddedLanguageFormatting: "auto",
+  quoteProps: "as-needed",
+  endOfLine: "lf",
+  semi: true,
+  singleAttributePerLine: false,
+  singleQuote: false,
+  trailingComma: "es5",
+  useTabs: false,
+  tabWidth: 2,
+};