docs: update jsdocs

Address review comments from jfmengels#301.

Author: lishaduck

.eslintrc.js

@@ -128,7 +128,9 @@ module.exports = {
     'no-unused-expressions': 'off', // This rule is replaced with the TSESlint version.
     '@typescript-eslint/no-unused-expressions': 'error', // Support TS stuff.
     '@typescript-eslint/no-throw-literal': 'error', // Recommended in v8 (w/rename to `only-throw-error`)
-    '@typescript-eslint/prefer-namespace-keyword': 'error', // Recommended in v8
+    '@typescript-eslint/prefer-find': 'error', // Recommended in v8
+    '@typescript-eslint/prefer-includes': 'error', // Recommended in v8
+    '@typescript-eslint/prefer-regexp-exec': 'error', // Recommended in v8
 
     // Unsafe
     '@typescript-eslint/no-unsafe-assignment': 'off', // Blocked on typescript-eslint/typescript-eslint#1682.

lib/anonymize.js

@@ -1,5 +1,5 @@
 /**
- * This module aims to make the paths and versions used in the CLI generic
+ * @file This module aims to make the paths and versions used in the CLI generic,
  * so that the CLI tests (in the `test/` folder) have the same output on different
  * machines, and also the same output when only the CLI version changes.
  */

lib/app-wrapper.js

@@ -81,6 +81,7 @@ function initWithWorker(elmModulePath, flags) {
 
 /**
  * @param {string | symbol} port
+ * @returns {Listened<unknown>}
  */
 function send(port) {
   return (/** @type {unknown} */ data) => {
@@ -137,6 +138,7 @@ function initializeListeners() {
 /**
  * @param {string} elmModulePath
  * @param {Flags} flags
+ * @returns {ReviewApp}
  */
 function initWithoutWorker(elmModulePath, flags) {
   const elmModule = loadCompiledElmApp(elmModulePath);
@@ -146,6 +148,9 @@ function initWithoutWorker(elmModulePath, flags) {
   return app;
 }
 
+/**
+ * @returns {void}
+ */
 function stop() {
   if (worker) {
     worker.terminate();

lib/autofix.js

@@ -27,6 +27,7 @@ const StyledMessage = require('./styled-message');
  * @param {Options} options
  * @param {ReviewApp} app
  * @param {VersionString} elmVersion
+ * @returns {void}
  */
 function subscribe(options, app, elmVersion) {
   AppState.subscribe(

lib/build.js

@@ -1,10 +1,11 @@
 /**
+ * @import {CompileOptions, Sources} from '../vendor/types/node-elm-compiler';
  * @import {AppHash, BuildResult} from './types/build';
  * @import {ApplicationElmJson, ElmJson} from './types/content';
- * @import {VersionString} from './types/version';
+ * @import {ErrorMessageInfo} from './types/error-message';
  * @import {Options, Template} from './types/options';
  * @import {Path} from './types/path';
- * @import {CompileOptions, Sources} from '../vendor/types/node-elm-compiler';
+ * @import {VersionString} from './types/version';
  */
 const path = require('node:path');
 const crypto = require('node:crypto');
@@ -334,6 +335,7 @@ async function createTemplateProject(
  * @param {Options} options
  * @param {string} userSrc
  * @param {ApplicationElmJson} elmJson
+ * @returns {ApplicationElmJson}
  */
 function updateSourceDirectories(options, userSrc, elmJson) {
   let sourceDirectories = [
@@ -359,10 +361,10 @@ function updateSourceDirectories(options, userSrc, elmJson) {
 
 /**
  * @param {Options} options
- * @param {string} dest
- * @param {string} elmModulePath
+ * @param {Path} dest
+ * @param {Path} elmModulePath
  * @param {Sources} compileTargets
- * @param {string} elmBinary
+ * @param {Path} elmBinary
  * @param {boolean} isReviewApp
  * @returns {Promise<string | null>}
  */
@@ -455,6 +457,7 @@ async function compileElmProject(
 /**
  * @param {Options} options
  * @param {string} stderr
+ * @returns {ErrorMessageInfo}
  */
 function compilationError(options, stderr) {
   if (stderr.includes('DEBUG REMNANTS')) {
@@ -496,6 +499,7 @@ function compilationError(options, stderr) {
  * @param {Options} options
  * @param {Path} reviewElmJsonPath
  * @param {ElmJson} reviewElmJson
+ * @returns {void}
  */
 function validateElmReviewVersion(options, reviewElmJsonPath, reviewElmJson) {
   if (options.localElmReviewSrc) {

lib/debug.js

@@ -4,6 +4,7 @@ const options = AppState.getOptions();
 
 /**
  * @param {string} message
+ * @returns {void}
  */
 function log(message) {
   if (options.debug) {

lib/elm-files.js

@@ -1,5 +1,5 @@
 /**
- * @import {ElmFile, Source, Readme, ElmJson, ElmJsonData, SourceDirectories, Ast} from './types/content';
+ * @import {Ast, ElmFile, ElmJson, ProjectFiles, Readme, Source, SourceDirectories} from './types/content';
  * @import {ReviewOptions} from './types/options';
  * @import {Path} from './types/path';
  */
@@ -19,14 +19,6 @@ const AppState = require('./state');
 
 const defaultGlob = '**/*.elm$';
 
-/**
- * @typedef {object} ProjectFiles
- * @property {ElmJsonData} elmJsonData
- * @property {Readme | null} readme
- * @property {ElmFile[]} elmFiles
- * @property {Path[]} sourceDirectories
- */
-
 /**
  * Get all files from the project.
  *

lib/error-message.js

@@ -97,7 +97,7 @@ function stacktrace(debug, error) {
  * @param {boolean} debug
  * @param {CustomError} error
  * @param {Path} [defaultPath]
- * @returns {unknown}
+ * @returns {{type: string; title: string; path: string | undefined; message: string[]; stack: string | undefined}}
  */
 function formatJson(debug, error, defaultPath) {
   return {

lib/fs-wrapper.js

@@ -9,7 +9,7 @@ const fsp = fs.promises;
  * Read a JSON file.
  *
  * @param {Path} file
- * @param {Reviver | undefined} [reviver=undefined]
+ * @param {Reviver} [reviver]
  * @returns {Promise<unknown>}
  */
 async function readJsonFile(file, reviver) {
@@ -19,10 +19,9 @@ async function readJsonFile(file, reviver) {
 }
 
 /**
- *
  * @param {string} data
  * @param {Path} file
- * @param {Reviver | undefined} reviver
+ * @param {Reviver} [reviver]
  * @returns {unknown}
  */
 function parseJsonFromFile(data, file, reviver) {
@@ -46,8 +45,8 @@ function readJsonFileSync(file) {
 /**
  * Read a file using Promises.
  *
- * @param {string} file
- * @param {fs.BaseEncodingOptions & { flag?: fs.OpenMode | undefined }} [options={encoding: 'utf8'}]
+ * @param {Path} file
+ * @param {fs.BaseEncodingOptions & { flag?: fs.OpenMode | undefined }} [options]
  * @returns {Promise<string>}
  */
 async function readFile(file, options) {
@@ -58,10 +57,10 @@ async function readFile(file, options) {
 /**
  * Write a JSON file.
  *
- * @param {string} file
+ * @param {Path} file
  * @param {unknown} content
- * @param {string | number | undefined} [space=undefined]
- * @param {Replacer | undefined} [replacer=undefined]
+ * @param {string | number | undefined} [space]
+ * @param {Replacer} [replacer]
  * @returns {Promise<void>}
  */
 async function writeJson(file, content, space, replacer) {
@@ -85,7 +84,8 @@ function mkdirpSync(dir) {
 }
 
 /**
- * @param {fs.PathLike} file
+ * @param {Path} file
+ * @returns {Promise<void>}
  */
 async function remove(file) {
   await fsp.rm(

lib/help.js

@@ -56,6 +56,9 @@ ${Flags.buildFlags('fix', null)}
   );
 }
 
+/**
+ * @returns {void}
+ */
 function suppress() {
   const orange = chalk.keyword('orange');
 
@@ -97,6 +100,9 @@ ${Flags.buildFlags('suppress', 'suppress')}
   );
 }
 
+/**
+ * @returns {void}
+ */
 function init() {
   console.log(
     // prettier-ignore
@@ -117,6 +123,9 @@ ${Flags.buildFlags('init', 'init')}
   );
 }
 
+/**
+ * @returns {void}
+ */
 function newPackage() {
   console.log(
     // prettier-ignore
@@ -138,6 +147,9 @@ ${Flags.buildFlags('new-package', 'new-package')}
   );
 }
 
+/**
+ * @returns {void}
+ */
 function newRule() {
   console.log(`The new-rule command adds an empty rule to your review configuration or
 review package.
@@ -161,6 +173,9 @@ ${Flags.buildFlags('new-rule', 'new-rule')}
 `);
 }
 
+/**
+ * @returns {void}
+ */
 function prepareOffline() {
   console.log(`The prepare-offline command allows the tool to run in offline mode using
 the ${chalk.cyan('--offline')} flag.