tools: fix jsdoc lint

PR-URL: #47789
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>

Author: MoLow

benchmark/common.js

@@ -374,7 +374,6 @@ function getUrlData(withBase) {
  * The 'wpt' type contains about 400 data points when `withBase` is true,
  * and 200 data points when `withBase` is false.
  * Other types contain 200 data points with or without base.
- *
  * @param {string} type Type of the data, 'wpt' or a key of `urls`
  * @param {number} e The repetition of the data, as exponent of 2
  * @param {boolean} withBase Whether to include a base URL

lib/internal/async_hooks.js

@@ -445,7 +445,6 @@ function clearDefaultTriggerAsyncId() {
 
 /**
  * Sets a default top level trigger ID to be used
- *
  * @template {Array<unknown>} T
  * @template {unknown} R
  * @param {number} triggerAsyncId

lib/internal/errors.js

@@ -499,7 +499,6 @@ const captureLargerStackTrace = hideStackFrames(
  * function UVException using a context object with data assembled in C++.
  * The goal is to migrate them to ERR_* errors later when compatibility is
  * not a concern.
- *
  * @param {object} ctx
  * @returns {Error}
  */
@@ -551,7 +550,6 @@ const uvException = hideStackFrames(function uvException(ctx) {
  * This creates an error compatible with errors produced in the C++
  * This function should replace the deprecated
  * `exceptionWithHostPort()` function.
- *
  * @param {number} err - A libuv error number
  * @param {string} syscall
  * @param {string} address
@@ -591,7 +589,6 @@ const uvExceptionWithHostPort = hideStackFrames(
 
 /**
  * This used to be util._errnoException().
- *
  * @param {number} err - A libuv error number
  * @param {string} syscall
  * @param {string} [original]
@@ -725,7 +722,6 @@ let maxStack_ErrorMessage;
  * Returns true if `err.name` and `err.message` are equal to engine-specific
  * values indicating max call stack size has been exceeded.
  * "Maximum call stack size exceeded" in V8.
- *
  * @param {Error} err
  * @returns {boolean}
  */
@@ -857,7 +853,6 @@ class AbortError extends Error {
 
 /**
  * This creates a generic Node.js error.
- *
  * @param {string} message The error message.
  * @param {object} errorProperties Object with additional properties to be added to the error.
  * @returns {Error}

lib/internal/modules/esm/fetch_module.js

@@ -235,7 +235,6 @@ async function isLocalAddress(hostname) {
  *
  * In cases where the request & response have already settled, this returns the
  * cache value synchronously.
- *
  * @param {URL} parsed
  * @param {ESModuleContext} context
  * @returns {ReturnType<typeof fetchWithRedirects>}

lib/internal/modules/esm/hooks.js

@@ -193,7 +193,6 @@ class Hooks {
    * Internally, this behaves like a backwards iterator, wherein the stack of
    * hooks starts at the top and each call to `nextResolve()` moves down 1 step
    * until it reaches the bottom or short-circuits.
-   *
    * @param {string} originalSpecifier The specified URL path of the module to
    *                                   be resolved.
    * @param {string} [parentURL] The URL path of the module's parent.
@@ -324,7 +323,6 @@ class Hooks {
    * Internally, this behaves like a backwards iterator, wherein the stack of
    * hooks starts at the top and each call to `nextLoad()` moves down 1 step
    * until it reaches the bottom or short-circuits.
-   *
    * @param {URL['href']} url The URL/path of the module to be loaded
    * @param {object} context Metadata about the module
    * @returns {Promise<{ format: ModuleFormat, source: ModuleSource }>}

lib/internal/modules/esm/loader.js

@@ -236,7 +236,6 @@ class DefaultModuleLoader {
    *
    * This method must NOT be renamed: it functions as a dynamic import on a
    * loader module.
-   *
    * @param {string | string[]} specifiers Path(s) to the module.
    * @param {string} parentURL Path of the parent importing the module.
    * @param {Record<string, string>} importAssertions Validations for the
@@ -286,7 +285,6 @@ class DefaultModuleLoader {
 
   /**
    * Resolve the location of the module.
-   *
    * @param {string} originalSpecifier The specified URL path of the module to
    *                                   be resolved.
    * @param {string} [parentURL] The URL path of the module's parent.
@@ -309,7 +307,6 @@ class DefaultModuleLoader {
 
   /**
    * Provide source that is understood by one of Node's translators.
-   *
    * @param {URL['href']} url The URL/path of the module to be loaded
    * @param {object} [context] Metadata about the module
    * @returns {Promise<{ format: ModuleFormat, source: ModuleSource }>}
@@ -354,7 +351,6 @@ class CustomizedModuleLoader extends DefaultModuleLoader {
 
   /**
    * Resolve the location of the module.
-   *
    * @param {string} originalSpecifier The specified URL path of the module to
    *                                   be resolved.
    * @param {string} [parentURL] The URL path of the module's parent.
@@ -389,7 +385,6 @@ class CustomizedModuleLoader extends DefaultModuleLoader {
 
   /**
    * Provide source that is understood by one of Node's translators.
-   *
    * @param {URL['href']} url The URL/path of the module to be loaded
    * @param {object} [context] Metadata about the module
    * @returns {Promise<{ format: ModuleFormat, source: ModuleSource }>}

lib/internal/per_context/primordials.js

@@ -314,7 +314,6 @@ const {
  * Because these functions are used by `makeSafe`, which is exposed on the
  * `primordials` object, it's important to use const references to the
  * primordials that they use.
- *
  * @template {Iterable} T
  * @template {*} TReturn
  * @template {*} TNext

lib/internal/policy/manifest.js

@@ -418,7 +418,6 @@ class Manifest {
    * the prototype to `null` for values or by running prior to any user code.
    *
    * `manifestURL` is a URL to resolve relative locations against.
-   *
    * @param {object} obj
    * @param {string} manifestHREF
    */

lib/internal/url.js

@@ -692,7 +692,6 @@ ObjectDefineProperties(URLSearchParams.prototype, {
  *
  * We use `href` and `protocol` as they are the only properties that are
  * easy to retrieve and calculate due to the lazy nature of the getters.
- *
  * @param {*} self
  * @returns {self is URL}
  */

lib/internal/util/inspect.js

@@ -301,7 +301,6 @@ function getUserOptions(ctx, isCrossContext) {
 /**
  * Echos the value of any input. Tries to print the value out
  * in the best way possible given the different types.
- *
  * @param {any} value The value to print out.
  * @param {object} opts Optional options object that alters the output.
  */