fix eslint errors

Author: anonrig

lib/internal/crypto/hash.js

@@ -68,6 +68,7 @@ const kFinalized = Symbol('kFinalized');
 
 /**
  * @param {string} name
+ * @returns {string}
  */
 function normalizeAlgorithm(name) {
   return StringPrototypeReplace(StringPrototypeToLowerCase(name), '-', '');

lib/internal/crypto/random.js

@@ -503,7 +503,7 @@ function generatePrimeSync(size, options = kEmptyObject) {
  * 48 is the ASCII code for '0', 97 is the ASCII code for 'a'.
  * @param {number} number An integer between 0 and 15.
  * @returns {number} corresponding to the ASCII code of the hex representation
- * of the parameter.
+ *   of the parameter.
  */
 const numberToHexCharCode = (number) => (number < 10 ? 48 : 87) + number;
 

lib/internal/dns/promises.js

@@ -124,7 +124,7 @@ function onlookupall(err, addresses) {
  * @param {string} hostname - The hostname to resolve.
  * @param {boolean} all - Whether to resolve with all IP addresses for the hostname.
  * @param {number} hints - One or more supported getaddrinfo flags (supply multiple via
- * bitwise OR).
+ *   bitwise OR).
  * @param {number} dnsOrder - How to sort results. Must be `ipv4first`, `ipv6first` or `verbatim`.
  * @returns {Promise<DNSLookupResult | DNSLookupResult[]>} The IP address(es) of the hostname.
  * @typedef {object} DNSLookupResult
@@ -187,9 +187,9 @@ function createLookupPromise(family, hostname, all, hints, dnsOrder) {
  * @param {boolean} [options.all] - Whether to return all or just the first resolved address.
  * @param {0 | 4 | 6} [options.family] - The record family. Must be 4, 6, or 0 (for both).
  * @param {number} [options.hints] - One or more supported getaddrinfo flags (supply multiple via
- * bitwise OR).
+ *   bitwise OR).
  * @param {'ipv4first' | 'ipv6first' | 'verbatim'} [options.order] - Return results in same order DNS resolved them;
- * New code should supply `verbatim`.
+ *   New code should supply `verbatim`.
  * @returns {Promise<object>}
  */
 function lookup(hostname, options) {

lib/internal/errors.js

@@ -126,8 +126,8 @@ function isPermissionModelError(err) {
  * of error objects created in a VM Context will always invoke the
  * prepareStackTraceCallback of the principal realm.
  * @param {object} globalThis The global object of the realm that the error was
- * created in. When the error object is created in a VM Context, this is the
- * global object of that VM Context.
+ *   created in. When the error object is created in a VM Context, this is the
+ *   global object of that VM Context.
  * @param {object} error The error object.
  * @param {CallSite[]} trace An array of CallSite objects, read more at https://v8.dev/docs/stack-trace-api#customizing-stack-traces.
  * @returns {string}

lib/internal/event_target.js

@@ -373,9 +373,6 @@ function isCustomEvent(value) {
   return isEvent(value) && (value?.[kDetail] !== undefined);
 }
 
-/**
- * @class
- */
 class CustomEvent extends Event {
   /**
    * @param {string} type

lib/internal/modules/cjs/loader.js

@@ -1146,7 +1146,7 @@ function getDefaultLoad(url, filename) {
  * @param {string} url The module URL (with the node: prefix)
  * @param {string} format Format from resolution.
  * @returns {any} If there are no load hooks or the load hooks do not override the format of the
- * builtin, load and return the exports of the builtin. Otherwise, return undefined.
+ *   builtin, load and return the exports of the builtin. Otherwise, return undefined.
  */
 function loadBuiltinWithHooks(id, url, format) {
   if (loadHooks.length) {
@@ -1666,7 +1666,7 @@ function wrapSafe(filename, content, cjsModuleInstance, format) {
  * @param {string} content The source code of the module
  * @param {string} filename The file path of the module
  * @param {'module'|'commonjs'|'commonjs-typescript'|'module-typescript'|'typescript'} format
- * Intended format of the module.
+ *   Intended format of the module.
  * @returns {any}
  */
 Module.prototype._compile = function(content, filename, format) {

lib/internal/modules/customization_hooks.js

@@ -175,7 +175,7 @@ function buildHooks(hooks, name, defaultStep, validate, mergedContext) {
    * in order to fill in missing arguments or check returned results.
    * Due to the merging of the context, this must be a closure.
    * @param {number} index Index in the chain. Default step is 0, last added hook is 1,
-   * and so on.
+   *   and so on.
    * @param {Function} userHookOrDefault Either the user hook or the default step to invoke.
    * @param {Function|undefined} next The next wrapped step. If this is the default step, it's undefined.
    * @returns {Function} Wrapped hook or default step.

lib/internal/modules/esm/assert.js

@@ -48,7 +48,7 @@ const supportedTypeAttributes = ArrayPrototypeFilter(
  * @param {string} url The URL of the imported module, for error reporting.
  * @param {string} format One of Node's supported translators
  * @param {Record<string, string>} importAttributes Validations for the
- * module import.
+ *   module import.
  * @returns {true}
  * @throws {TypeError} If the format and type attribute are incompatible.
  */

lib/internal/modules/esm/hooks.js

@@ -143,7 +143,7 @@ class Hooks {
    * @param {string} urlOrSpecifier
    * @param {string} parentURL
    * @param {any} [data] Arbitrary data to be passed from the custom
-   * loader (user-land) to the worker.
+   *   loader (user-land) to the worker.
    */
   async register(urlOrSpecifier, parentURL, data, isInternal) {
     const cascadedLoader = require('internal/modules/esm/loader').getOrInitializeCascadedLoader();
@@ -162,7 +162,7 @@ class Hooks {
    * @param {string} url Custom loader specifier
    * @param {Record<string, unknown>} exports
    * @param {any} [data] Arbitrary data to be passed from the custom loader (user-land)
-   * to the worker.
+   *   to the worker.
    * @returns {any | Promise<any>} User data, ignored unless it's a promise, in which case it will be awaited.
    */
   addCustomLoader(url, exports, data) {
@@ -190,10 +190,10 @@ class Hooks {
    * 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.
+   *   be resolved.
    * @param {string} [parentURL] The URL path of the module's parent.
    * @param {ImportAttributes} [importAttributes] Attributes from the import
-   * statement or expression.
+   *   statement or expression.
    * @returns {Promise<{ format: string, url: URL['href'] }>}
    */
   async resolve(
@@ -688,18 +688,18 @@ function pluckHooks({
  * chain, and generate and supply the `next<HookName>` argument to the custom
  * hook.
  * @param {KeyedHook} current The (currently) first hook in the chain (this shifts
- * on every call).
+ *   on every call).
  * @param {object} meta Properties that change as the current hook advances
- * along the chain.
+ *   along the chain.
  * @param {boolean} meta.chainFinished Whether the end of the chain has been
- * reached AND invoked.
+ *   reached AND invoked.
  * @param {string} meta.hookErrIdentifier A user-facing identifier to help
- * pinpoint where an error occurred. Ex "file:///foo.mjs 'resolve'".
+ *   pinpoint where an error occurred. Ex "file:///foo.mjs 'resolve'".
  * @param {string} meta.hookName The kind of hook the chain is (ex 'resolve')
  * @param {boolean} meta.shortCircuited Whether a hook signaled a short-circuit.
  * @param {function(string, unknown): void} validate A wrapper function
- * containing all validation of a custom loader hook's intermediary output. Any
- * validation within MUST throw.
+ *   containing all validation of a custom loader hook's intermediary output. Any
+ *   validation within MUST throw.
  * @returns {Function} The next hook in the chain.
  */
 function nextHookFactory(current, meta, { validateArgs, validateOutput }) {

lib/internal/modules/esm/initialize_import_meta.js

@@ -22,7 +22,7 @@ function createImportMetaResolve(defaultParentURL, loader, allowParentURL) {
   /**
    * @param {string} specifier
    * @param {URL['href']} [parentURL] When `--experimental-import-meta-resolve` is specified, a
-   * second argument can be provided.
+   *   second argument can be provided.
    * @returns {string}
    */
   return function resolve(specifier, parentURL = defaultParentURL) {