Refactor code-style

Author: wooorm

lib/find-repo.node.js

@@ -3,9 +3,9 @@
  * @import {VFile} from 'vfile'
  */
 
+import {exec as execCallback} from 'node:child_process'
 import path from 'node:path'
 import {promisify} from 'node:util'
-import {exec as execCallback} from 'node:child_process'
 
 const exec = promisify(execCallback)
 

lib/index.js

@@ -8,7 +8,9 @@
 /**
  * @typedef {Map<string, Map<string, boolean>>} Landmarks
  *   Landmarks.
- *
+ */
+
+/**
  * @typedef Options
  *   Configuration.
  * @property {string | false | null | undefined} [repository]
@@ -26,28 +28,33 @@
  *   Config on how hosted Git works (default: detected from repo);
  *   `github.com`, `gitlab.com`, or `bitbucket.org` work automatically;
  *   otherwise, pass `urlConfig` manually.
- *
+ */
+
+/**
  * @callback Propose
  * @param {string} value
  * @param {ReadonlyArray<string>} dictionary
  * @param {Readonly<ProposeOptions> | null | undefined} [options]
  * @returns {string | undefined}
- *
+ */
+
+/**
  * @typedef ProposeOptions
  *   Configuration for `propose`.
  * @property {number| null | undefined} [threshold]
  *   Threshold.
- *
- * @typedef {Extract<Nodes, Resource>} Resources
- *   Resources.
- *
+ */
+
+/**
  * @typedef Reference
  *   Reference to something.
  * @property {string} filePath
  *   Path to file.
  * @property {string | undefined} hash
  *   Hash.
- *
+ */
+
+/**
  * @typedef ReferenceInfo
  *   Info on a reference.
  * @property {VFile} file
@@ -56,7 +63,14 @@
  *   Reference.
  * @property {ReadonlyArray<Readonly<Resources>>} nodes
  *   Nodes that reference it.
- *
+ */
+
+/**
+ * @typedef {Extract<Nodes, Resource>} Resources
+ *   Resources.
+ */
+
+/**
  * @typedef State
  *   Info passed around.
  * @property {string} base
@@ -67,7 +81,9 @@
  *   Path to Git folder.
  * @property {Readonly<UrlConfig>} urlConfig
  *   Configuration.
- *
+ */
+
+/**
  * @typedef UrlConfig
  *   Hosted Git info.
  *
@@ -101,7 +117,6 @@
  *     lines: false
  *   }
  *   ```
- *
  * @property {string | null | undefined} [headingPrefix]
  *   Prefix of headings (example: `'#'`, `'#markdown-header-'`).
  * @property {string | null | undefined} [hostname]
@@ -136,7 +151,7 @@ const propose = /** @type {Propose} */ (propose_)
 cliCompleter.pluginId = constants.sourceId
 
 /** @type {Readonly<Partial<Record<Hosts, string>>>} */
-const viewPaths = {github: 'blob', gitlab: 'blob', bitbucket: 'src'}
+const viewPaths = {bitbucket: 'src', github: 'blob', gitlab: 'blob'}
 /** @type {Readonly<Partial<Record<Hosts, string>>>} */
 const headingPrefixes = {
   bitbucket: '#markdown-header-',
@@ -159,7 +174,7 @@ const slashes = '//'
 const lineExpression = /^#l\d/i
 
 // List from: https://github.com/github/markup#markups
-const readmeExtensions = new Set(['.markdown', '.mdown', '.mkdn', '.md'])
+const readmeExtensions = new Set(['.markdown', '.mdown', '.md', '.mkdn'])
 const readmeBasename = /^readme$/i
 
 /**
@@ -409,13 +424,10 @@ export default function remarkValidateLinks(options, fileSet) {
           } catch {}
 
           const files = entries.sort()
-          let index = -1
           /** @type {string | undefined} */
           let file
 
-          while (++index < files.length) {
-            const entry = entries[index]
-
+          for (const entry of files) {
             if (readme(entry)) {
               file = entry
               break
@@ -464,10 +476,8 @@ async function checkAll(files) {
   // Merge landmarks.
   /** @type {Landmarks} */
   const landmarks = new Map()
-  let index = -1
 
-  while (++index < files.length) {
-    const file = files[index]
+  for (const file of files) {
     const fileLandmarks = /** @type {Landmarks | undefined} */ (
       file.data[constants.landmarkId]
     )
@@ -482,10 +492,8 @@ async function checkAll(files) {
   // Merge references.
   /** @type {Map<string, Map<string, Array<ReferenceInfo>>>} */
   const references = new Map()
-  index = -1
 
-  while (++index < files.length) {
-    const file = files[index]
+  for (const file of files) {
     const fileReferences =
       /** @type {Map<string, Map<string, Array<Resources>>> | undefined} */ (
         file.data[constants.referenceId]
@@ -538,10 +546,8 @@ async function checkAll(files) {
     }
   }
 
-  index = -1
-
-  while (++index < missing.length) {
-    warn(landmarks, missing[index])
+  for (const reference of missing) {
+    warn(landmarks, reference)
   }
 }
 
@@ -613,10 +619,7 @@ function warn(landmarks, reference) {
     reason += '; did you mean `' + suggestion + '`'
   }
 
-  let index = -1
-
-  while (++index < reference.nodes.length) {
-    const node = reference.nodes[index]
+  for (const node of reference.nodes) {
     const message = reference.file.message(reason, {
       place: node.position,
       source: origin,