chore: fix husky, lint and format code

.eslintrc.cjs

@@ -13,11 +13,18 @@ module.exports = {
     // disable the rule for all files
     '@typescript-eslint/explicit-function-return-type': 'off',
     '@typescript-eslint/no-non-null-assertion': 'off',
+    '@typescript-eslint/no-unused-vars': ['error', {
+      "varsIgnorePattern": "_.*",
+      "argsIgnorePattern": "_.*"
+    }],
     'import/named': 'off',
     'import/no-unresolved': 'off',
     'import/extensions': ['error', 'always', { ignorePackages: true }],
     'import/no-duplicates': 'off',
     'no-duplicate-imports': 'off',
     'tsdoc/syntax': 'warn'
   },
+  "env": {
+    "browser": true,
+  }
 };

.husky/commit-msg

@@ -0,0 +1,4 @@
+#!/usr/bin/env sh
+. "$(dirname -- "$0")/_/husky.sh"
+
+npx --no -- commitlint --edit ${1}

.husky/pre-commit

@@ -0,0 +1,4 @@
+#!/usr/bin/env sh
+. "$(dirname -- "$0")/_/husky.sh"
+
+npx --no -- lint-staged

package.json

@@ -49,13 +49,13 @@
     "format:prettier": "prettier \"**/*.js\" \"**/*.ts\" --write --ignore-path .gitignore",
     "lint": "npm run lint:eslint && npm run lint:prettier",
     "format": "npm run format:eslint && npm run format:prettier",
-    "test": "web-test-runner --coverage",
+    "test": "npm run lint && web-test-runner --coverage",
     "test:snapshot": "web-test-runner --update-snapshots --coverage",
     "test:manual": "web-test-runner --manual",
     "test:watch": "web-test-runner --watch",
     "test:unit": "web-test-runner --watch --group unit",
     "test:integration": "web-test-runner --watch --group integration",
-    "doc:clean": "npx rimraf doc",
+    "doc:clean": "npx --no -- rimraf doc",
     "doc:typedoc": "typedoc --plugin none --out doc src",
     "doc:wca": "wca src --outDir doc/components",
     "doc": "npm run doc:clean && npm run doc:typedoc && npm run doc:wca",
@@ -65,7 +65,8 @@
     "release:major": "standard-version --release-as major",
     "build": "npm run test && npm run build:notest && cp .nojekyll build/",
     "build:notest": "npm run doc && snowpack build && workbox generateSW workbox-config.cjs",
-    "start": "snowpack dev"
+    "start": "snowpack dev",
+    "prepare": "husky install"
   },
   "devDependencies": {
     "@commitlint/cli": "^13.1.0",
@@ -88,7 +89,7 @@
     "eslint-plugin-babel": "^5.3.1",
     "eslint-plugin-tsdoc": "^0.2.14",
     "fast-check": "^2.19.0",
-    "husky": "^7.0.1",
+    "husky": "^8.0.0",
     "lint-staged": "^11.1.2",
     "prettier": "^2.3.2",
     "sinon": "^11.1.2",
@@ -112,12 +113,6 @@
     "singleQuote": true,
     "arrowParens": "avoid"
   },
-  "husky": {
-    "hooks": {
-      "pre-commit": "lint-staged",
-      "commit-msg": "commitlint -E HUSKY_GIT_PARAMS"
-    }
-  },
   "lint-staged": {
     "*.ts": [
       "eslint --fix",

public/js/plugins.js

@@ -232,6 +232,6 @@ export const officialPlugins = [
     default: false,
     kind: 'menu',
     requireDoc: true,
-    position: 'middle'
-  }
+    position: 'middle',
+  },
 ];

public/js/worker.js

@@ -1,7 +1,7 @@
 importScripts('xmlvalidate.js');
 
-onmessage = function(e) {
-  Module.ready.then(function(mod) {
+onmessage = function (e) {
+  Module.ready.then(function (mod) {
     if (e.data.name.toLowerCase().endsWith('.xsd'))
       mod.init(e.data.content, e.data.name);
     else mod.validate(e.data.content, e.data.name);

src/Editing.ts

@@ -31,6 +31,7 @@ export type EditingElement = Mixin<typeof Editing>;
 /** @typeParam TBase - a type extending `LitElement`
  * @returns `Base` with an `XMLDocument` property "`doc`" and an event listener
  * applying [[`EditorActionEvent`]]s and dispatching [[`LogEvent`]]s. */
+// eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types
 export function Editing<TBase extends LitElementConstructor>(Base: TBase) {
   class EditingElement extends Base {
     /** The `XMLDocument` to be edited */
@@ -445,6 +446,7 @@ export function Editing<TBase extends LitElementConstructor>(Base: TBase) {
       );
     }
 
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     constructor(...args: any[]) {
       super(...args);
 

src/Hosting.ts

@@ -42,7 +42,10 @@ interface MenuPlugin {
 /** Mixin that hosts the UI for Plugins, Settings and Logging */
 export type HostingElement = Mixin<typeof Hosting>;
 
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
+// eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types
 export function Hosting<
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   TBase extends new (...args: any[]) => PluggingElement &
     LoggingElement &
     SettingElement
@@ -207,6 +210,7 @@ export function Hosting<
       ];
     }
 
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     constructor(...args: any[]) {
       super(...args);
 

src/Logging.ts

@@ -68,6 +68,7 @@ function getPluginName(src: string): string {
  */
 export type LoggingElement = Mixin<typeof Logging>;
 
+// eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types
 export function Logging<TBase extends LitElementConstructor>(Base: TBase) {
   class LoggingElement extends Base {
     /** All [[`LogEntry`]]s received so far through [[`LogEvent`]]s. */
@@ -185,6 +186,7 @@ export function Logging<TBase extends LitElementConstructor>(Base: TBase) {
       super.performUpdate();
     }
 
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     constructor(...args: any[]) {
       super(...args);
 

src/Plugging.ts

@@ -99,7 +99,7 @@ function staticTagHtml(
 
 type PluginKind = 'editor' | 'menu' | 'validator';
 const menuPosition = ['top', 'middle', 'bottom'] as const;
-type MenuPosition = typeof menuPosition[number];
+type MenuPosition = (typeof menuPosition)[number];
 
 export type Plugin = {
   name: string;
@@ -176,7 +176,9 @@ const loadedPlugins = new Set<string>();
 /** Mixin that manages Plugins in `localStorage` */
 export type PluggingElement = Mixin<typeof Plugging>;
 
+// eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types
 export function Plugging<
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   TBase extends new (...args: any[]) => EditingElement & LoggingElement
 >(Base: TBase) {
   class PluggingElement extends Base {
@@ -345,6 +347,7 @@ export function Plugging<
       this.pluginDownloadUI.close();
     }
 
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     constructor(...args: any[]) {
       super(...args);
 

src/Setting.ts

@@ -82,10 +82,18 @@ export function newLoadNsdocEvent(
   });
 }
 
+// TODO(ca-d): unify document loading mechanisms
+declare global {
+  interface ElementEventMap {
+    ['load-nsdoc']: LoadNsdocEvent;
+  }
+}
+
 /** Mixin that saves [[`Settings`]] to `localStorage`, reflecting them in the
  * `settings` property, setting them through `setSetting(setting, value)`. */
 export type SettingElement = Mixin<typeof Setting>;
 
+// eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types
 export function Setting<TBase extends LitElementConstructor>(Base: TBase) {
   class SettingElement extends Base {
     /** Current [[`Settings`]] in `localStorage`, default to [[`defaults`]]. */
@@ -358,13 +366,14 @@ export function Setting<TBase extends LitElementConstructor>(Base: TBase) {
       return new DOMParser().parseFromString(text, 'application/xml');
     }
 
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     constructor(...params: any[]) {
       super(...params);
 
       registerTranslateConfig({ loader, empty: key => key });
       use(this.settings.language);
 
-      (<any>this).addEventListener('load-nsdoc', this.onLoadNsdoc);
+      this.addEventListener('load-nsdoc', this.onLoadNsdoc);
     }
 
     render(): TemplateResult {

src/Waiting.ts

@@ -13,6 +13,7 @@ import {
  * [Pending State](https://github.com/justinfagnani/pending-state-protocol) */
 export type WaitingElement = Mixin<typeof Waiting>;
 
+// eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types
 export function Waiting<TBase extends LitElementConstructor>(Base: TBase) {
   class WaitingElement extends Base {
     /** Whether the element is currently waiting for some async work. */
@@ -32,6 +33,7 @@ export function Waiting<TBase extends LitElementConstructor>(Base: TBase) {
       this.waiting = this.work.size > 0;
     }
 
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     constructor(...args: any[]) {
       super(...args);
 

src/Wizarding.ts

@@ -14,6 +14,7 @@ import { WizardDialog } from './wizard-dialog.js';
  * queued onto on incoming [[`WizardEvent`]]s, first come first displayed. */
 export type WizardingElement = Mixin<typeof Wizarding>;
 
+// eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types
 export function Wizarding<TBase extends LitElementConstructor>(Base: TBase) {
   class WizardingElement extends Base {
     /** FIFO queue of [[`Wizard`]]s to display. */
@@ -37,6 +38,7 @@ export function Wizarding<TBase extends LitElementConstructor>(Base: TBase) {
       );
     }
 
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     constructor(...args: any[]) {
       super(...args);