Downgrade valid-jsdoc lint to warn

.eslintrc.json

@@ -34,7 +34,7 @@
       },
       "ignore": ["constructor"]
     }],
-    "valid-jsdoc": ["error", {
+    "valid-jsdoc": ["warn", {
       "prefer": {
         "arg": "param",
         "argument": "param",

src/components/PasswordBox.js

@@ -17,7 +17,6 @@
  */
 
 class PasswordBox extends Observable {
-    // eslint-disable-next-line valid-jsdoc
     /**
      * @param {?HTMLFormElement} [$el]
      * @param {Partial<PasswordBoxOptions>} [options]

src/components/PasswordSetterBox.js

@@ -6,7 +6,6 @@
 /* global TemplateTags */
 
 class PasswordSetterBox extends Observable {
-    // eslint-disable-next-line valid-jsdoc
     /**
      * @param {?HTMLFormElement} [$el]
      * @param {{bgColor?: string, buttonI18nTag?: string}} [options]

src/components/QrVideoScanner.js

@@ -4,7 +4,6 @@
 /* global TemplateTags */
 
 class QrVideoScanner extends Observable {
-    // eslint-disable-next-line valid-jsdoc
     /**
      * @param {?HTMLDivElement} [$el]
      * @param {(result: string) => boolean} [validator]

src/lib/CookieStorage.js

@@ -116,7 +116,6 @@ class CookieStorage {
         return !!CookieJar.readCookie(this._getChunkCookieName(name, 0, namespace));
     }
 
-    // eslint-disable-next-line valid-jsdoc
     /**
      * @param {string} name
      * @param {Uint8Array} data

src/lib/FiatApi.js

@@ -1,7 +1,6 @@
 // Adapted and reduced version of FiatApi.ts from @nimiq/utils
 // This file should also be updated whenever FiatApi.ts in @nimiq/utils is updated.
 class FiatApi {
-    // eslint-disable-next-line valid-jsdoc
     /**
      * @param {Array<FiatApi.SupportedCryptoCurrency>} cryptoCurrencies
      * @param {Array<FiatApi.SupportedFiatCurrency>} vsCurrencies

src/lib/I18n.js

@@ -58,7 +58,6 @@ class I18n { // eslint-disable-line no-unused-vars
     static translateDom(dom = document.body, enforcedLanguage) {
         const language = enforcedLanguage ? this.getClosestSupportedLanguage(enforcedLanguage) : this.language;
 
-        /* eslint-disable-next-line valid-jsdoc */ // Multi-line descriptions are not valid JSDoc, apparently
         /**
          * @param {string} tag
          * @param {(element: HTMLElement, translation: string) => void} callback - callback(element, translation) for

src/lib/JsonUtils.js

@@ -53,7 +53,6 @@ class JsonUtils {
         return value;
     }
 
-    /* eslint-disable-next-line valid-jsdoc */
     /**
      * @param {number} type
      * @param {string} value

src/lib/Key.js

@@ -145,7 +145,6 @@ class Key {
         return Key.deriveHash(input);
     }
 
-    // eslint-disable-next-line valid-jsdoc
     /**
      * @param {unknown} other
      * @returns {other is Key}

src/lib/KeyInfo.js

@@ -85,7 +85,6 @@ class KeyInfo {
         return new KeyInfo(obj.id, obj.type, encrypted, obj.hasPin, defaultAddress);
     }
 
-    // eslint-disable-next-line valid-jsdoc
     /**
      * @param {unknown} other
      * @returns {other is KeyInfo}

src/lib/Observable.js

@@ -16,7 +16,6 @@ class Observable { // eslint-disable-line no-unused-vars
         this._listeners = new Map();
     }
 
-    // eslint-disable-next-line valid-jsdoc
     /**
      * @param {string} type
      * @param {(...arg: any[]) => any} callback

src/lib/QrScanner.js

@@ -29,7 +29,6 @@ class QrScanner {
         }
     }
 
-    // eslint-disable-next-line valid-jsdoc
     /**
      * @param {HTMLVideoElement} video
      * @param {(result: string) => any} onDecode

src/lib/RequestParser.js

@@ -217,7 +217,6 @@ class RequestParser { // eslint-disable-line no-unused-vars
         }
     }
 
-    // eslint-disable-next-line valid-jsdoc
     /**
      * @template {boolean} T
      * @param {any} address

src/lib/TemplateTags.js

@@ -1,6 +1,5 @@
 class TemplateTags {
     // Let typescript infer return value. (Seems to be impossible to declare it the right way in standard JSDoc)
-    // eslint-disable-next-line valid-jsdoc
     /**
      * @template {number} T
      * @param {T} variableCount

src/lib/bitcoin/BitcoinRequestParserMixin.js

@@ -8,7 +8,6 @@
  * @typedef {new (...args: any[]) => T} BitcoinRequestParserMixinConstructor;
  */
 
-// eslint-disable-next-line valid-jsdoc
 /**
  * A mixin to add Bitcoin-related request parsers to a RequestParser class.
  *

src/lib/polygon/PolygonRequestParserMixin.js

@@ -7,7 +7,6 @@
  * @typedef {new (...args: any[]) => T} PolygonRequestParserMixinConstructor;
  */
 
-// eslint-disable-next-line valid-jsdoc
 /**
  * A mixin to add Polygon-related request parsers to a RequestParser class.
  *

src/request/sign-polygon-transaction/SignPolygonTransactionApi.js

@@ -54,7 +54,6 @@ class SignPolygonTransactionApi extends PolygonRequestParserMixin(TopLevelApi) {
         return parsedRequest;
     }
 
-    // eslint-disable-next-line valid-jsdoc
     /**
      *
      * @param {KeyguardRequest.PolygonTransactionInfo} request

src/request/sign-swap/SignSwapApi.js

@@ -330,7 +330,6 @@ class SignSwapApi extends PolygonRequestParserMixin(BitcoinRequestParserMixin(To
         return direction;
     }
 
-    // eslint-disable-next-line valid-jsdoc
     /**
      *
      * @param {Omit<KeyguardRequest.PolygonTransactionInfo, 'amount'>} request