Refactor types

Author: wooorm

packages/remark-lint-fenced-code-flag/index.js

@@ -6,7 +6,7 @@
  * @fileoverview
  *   Check fenced code block flags.
  *
- *   Options: `Array.<string>` or `Object`, optional.
+ *   Options: `Array<string>` or `Object`, optional.
  *
  *   Providing an array is as passing `{flags: Array}`.
  *
@@ -83,7 +83,7 @@
 /**
  * @typedef {import('mdast').Root} Root
  *
- * @typedef {string[]} Flags
+ * @typedef {Array<string>} Flags
  *
  * @typedef FlagMap
  * @property {Flags} [flags]
@@ -108,7 +108,7 @@ const remarkLintFencedCodeFlag = lintRule(
   (tree, file, option) => {
     const value = String(file)
     let allowEmpty = false
-    /** @type {string[]} */
+    /** @type {Array<string>} */
     let allowed = []
 
     if (typeof option === 'object') {

packages/remark-lint-fenced-code-flag/readme.md

@@ -12,7 +12,7 @@
 
 Check fenced code block flags.
 
-Options: `Array.<string>` or `Object`, optional.
+Options: `Array<string>` or `Object`, optional.
 
 Providing an array is as passing `{flags: Array}`.
 

packages/remark-lint-no-duplicate-headings-in-section/index.js

@@ -78,7 +78,7 @@ const remarkLintNoDuplicateHeadingsInSection = lintRule(
   },
   /** @type {import('unified-lint-rule').Rule<Root, void>} */
   (tree, file) => {
-    /** @type {Array.<Record<string, Heading>>} */
+    /** @type {Array<Record<string, Heading>>} */
     let stack = []
 
     visit(tree, 'heading', (node) => {

packages/remark-lint-no-reference-like-url/index.js

@@ -41,7 +41,7 @@ const remarkLintNoReferenceLikeUrl = lintRule(
   },
   /** @type {import('unified-lint-rule').Rule<Root, void>} */
   (tree, file) => {
-    /** @type {string[]} */
+    /** @type {Array<string>} */
     const identifiers = []
 
     visit(tree, 'definition', (node) => {

packages/remark-lint-no-undefined-references/index.js

@@ -72,9 +72,9 @@
  * @typedef {import('mdast').Paragraph} Paragraph
  *
  * @typedef Options
- * @property {string[]} [allow]
+ * @property {Array<string>} [allow]
  *
- * @typedef {number[]} Range
+ * @typedef {Array<number>} Range
  */
 
 import {normalizeIdentifier} from 'micromark-util-normalize-identifier'
@@ -133,7 +133,7 @@ const remarkLintNoUndefinedReferences = lintRule(
      * @param {Heading|Paragraph} node
      */
     function findInPhrasing(node) {
-      /** @type {Range[]} */
+      /** @type {Array<Range>} */
       let ranges = []
 
       visit(node, (child) => {
@@ -158,7 +158,7 @@ const remarkLintNoUndefinedReferences = lintRule(
         }
 
         const source = contents.slice(start, end)
-        /** @type {Array.<[number, string]>} */
+        /** @type {Array<[number, string]>} */
         const lines = [[start, '']]
         let last = 0
 

packages/remark-lint-table-cell-padding/index.js

@@ -212,9 +212,9 @@ const remarkLintTableCellPadding = lintRule(
       // To do: fix types to always have `align` defined.
       /* c8 ignore next */
       const align = node.align || []
-      /** @type {number[]} */
+      /** @type {Array<number>} */
       const sizes = []
-      /** @type {Entry[]} */
+      /** @type {Array<Entry>} */
       const entries = []
       let index = -1
 
@@ -294,7 +294,7 @@ const remarkLintTableCellPadding = lintRule(
      * @param {'start'|'end'} side
      * @param {Entry} entry
      * @param {0|1} style
-     * @param {number[]} sizes
+     * @param {Array<number>} sizes
      */
     function checkSide(side, entry, style, sizes) {
       const cell = entry.node

packages/remark-lint-table-pipe-alignment/index.js

@@ -66,7 +66,7 @@ const remarkLintTablePipeAlignment = lintRule(
     const value = String(file)
 
     visit(tree, 'table', (node) => {
-      /** @type {number[]} */
+      /** @type {Array<number>} */
       const indices = []
       let index = -1
 

packages/remark-lint/index.js

@@ -9,13 +9,13 @@ import remarkMessageControl from 'remark-message-control'
  * This adds support for ignoring stuff from messages (`<!--lint ignore-->`).
  * All rules are in their own packages and presets.
  *
- * @type {import('unified').Plugin<void[], Root>}
+ * @type {import('unified').Plugin<Array<void>, Root>}
  */
 export default function remarkLint() {
   this.use(lintMessageControl)
 }
 
-/** @type {import('unified').Plugin<void[], Root>} */
+/** @type {import('unified').Plugin<Array<void>, Root>} */
 function lintMessageControl() {
   return remarkMessageControl({name: 'lint', source: 'remark-lint'})
 }

packages/unified-lint-rule/lib/index.js

@@ -4,7 +4,7 @@
  *
  * @typedef {0|1|2} Severity
  * @typedef {'warn'|'on'|'off'|'error'} Label
- * @typedef {[Severity, ...unknown[]]} SeverityTuple
+ * @typedef {[Severity, ...Array<unknown>]} SeverityTuple
  *
  * @typedef RuleMeta
  * @property {string} origin name of the lint rule
@@ -38,7 +38,7 @@ export function lintRule(meta, rule) {
 
   return plugin
 
-  /** @type {import('unified').Plugin<[unknown]|void[]>} */
+  /** @type {import('unified').Plugin<[unknown]|Array<void>>} */
   function plugin(raw) {
     const [severity, options] = coerce(ruleId, raw)
 
@@ -80,7 +80,7 @@ export function lintRule(meta, rule) {
  * @returns {SeverityTuple}
  */
 function coerce(name, value) {
-  /** @type {unknown[]} */
+  /** @type {Array<unknown>} */
   let result
 
   if (typeof value === 'boolean') {
@@ -89,11 +89,11 @@ function coerce(name, value) {
     result = [1]
   } else if (
     Array.isArray(value) &&
-    // `isArray(unknown)` is turned into `any[]`:
+    // `isArray(unknown)` is turned into `Array<any>`:
     // type-coverage:ignore-next-line
     primitives.has(typeof value[0])
   ) {
-    // `isArray(unknown)` is turned into `any[]`:
+    // `isArray(unknown)` is turned into `Array<any>`:
     // type-coverage:ignore-next-line
     result = [...value]
   } else {

script/build-presets.js

@@ -64,7 +64,7 @@ presets(root).then((presetObjects) => {
       )
     }
 
-    /** @type {TableContent[]} */
+    /** @type {Array<TableContent>} */
     const rows = []
 
     rows.push({
@@ -110,11 +110,11 @@ presets(root).then((presetObjects) => {
       }
     }
 
-    /** @type {BlockContent[]} */
+    /** @type {Array<BlockContent>} */
     // @ts-expect-error: fine.
     const descriptionContent = remark().parse(description).children
 
-    /** @type {BlockContent[]} */
+    /** @type {Array<BlockContent>} */
     const children = [
       {type: 'html', value: '<!--This file is generated-->'},
       {