Update @types/hast, @types/nlcst, utilities

Author: wooorm

Diff for: lib/index.js

@@ -152,7 +152,7 @@ export function toNlcst(tree, file, Parser) {
     throw new Error('hast-util-to-nlcst expected parser')
   }
 
-  if (!pointStart(tree).line || !pointStart(tree).column) {
+  if (!pointStart(tree)) {
     throw new Error('hast-util-to-nlcst expected position on nodes')
   }
 
@@ -164,10 +164,14 @@ export function toNlcst(tree, file, Parser) {
 
   find(tree)
 
+  const start = loc.toPoint(0)
+  const end = loc.toPoint(doc.length)
+
   return {
     type: 'RootNode',
     children: results,
-    position: {start: loc.toPoint(0), end: loc.toPoint(doc.length)}
+    /* c8 ignore next - prevent crashes if some nodes don’t have positional info */
+    position: start && end ? {start, end} : undefined
   }
 
   /**
@@ -354,6 +358,7 @@ export function toNlcst(tree, file, Parser) {
         replacement = [{type: 'WhiteSpaceNode', value: '\n'}]
         change = true
       } else if (source(node)) {
+        // @ts-expect-error: remove when `to-string` is updated.
         replacement = [{type: 'SourceNode', value: toString(node)}]
         change = true
       } else {
@@ -403,30 +408,30 @@ export function toNlcst(tree, file, Parser) {
    *   Nodes to patch.
    * @param {ReturnType<location>} location
    *   Location info.
-   * @param {number} offset
+   * @param {number | undefined} offset
    *   Current offset.
    * @returns {void}
    *   Nothing.
    */
   function patch(nodes, location, offset) {
     let index = -1
-    let start = offset
+    let from = offset
 
-    while (++index < nodes.length) {
+    while (from !== undefined && ++index < nodes.length) {
       const node = nodes[index]
 
       if ('children' in node) {
-        patch(node.children, location, start)
+        patch(node.children, location, from)
       }
 
-      const end = start + nlcstToString(node).length
+      const to = from + nlcstToString(node).length
+      const start = location.toPoint(from)
+      const end = location.toPoint(to)
 
-      node.position = {
-        start: location.toPoint(start),
-        end: location.toPoint(end)
-      }
+      /* c8 ignore next - prevent crashes if some nodes don’t have positional info */
+      node.position = start && end ? {start, end} : undefined
 
-      start = end
+      from = to
     }
   }
 }

Diff for: package.json

@@ -36,38 +36,38 @@
     "index.js"
   ],
   "dependencies": {
-    "@types/hast": "^2.0.0",
-    "@types/nlcst": "^1.0.0",
-    "@types/unist": "^2.0.0",
-    "hast-util-embedded": "^2.0.0",
-    "hast-util-is-element": "^2.0.0",
-    "hast-util-phrasing": "^2.0.0",
+    "@types/hast": "^3.0.0",
+    "@types/nlcst": "^2.0.0",
+    "@types/unist": "^3.0.0",
+    "hast-util-embedded": "^3.0.0",
+    "hast-util-is-element": "^3.0.0",
+    "hast-util-phrasing": "^3.0.0",
     "hast-util-to-string": "^2.0.0",
-    "hast-util-whitespace": "^2.0.0",
-    "nlcst-to-string": "^3.0.0",
-    "unist-util-position": "^4.0.0",
-    "vfile": "^5.0.0",
-    "vfile-location": "^4.0.0"
+    "hast-util-whitespace": "^3.0.0",
+    "nlcst-to-string": "^4.0.0",
+    "unist-util-position": "^5.0.0",
+    "vfile": "^6.0.0",
+    "vfile-location": "^5.0.0"
   },
   "devDependencies": {
-    "@types/node": "^18.0.0",
-    "c8": "^7.0.0",
-    "hast-util-from-html": "^1.0.0",
+    "@types/node": "^20.0.0",
+    "c8": "^8.0.0",
+    "hast-util-from-html": "^2.0.0",
     "is-hidden": "^2.0.0",
-    "parse-dutch": "^6.0.0",
-    "parse-english": "^6.0.0",
-    "parse-latin": "^6.0.0",
-    "prettier": "^2.0.0",
+    "parse-dutch": "^7.0.0",
+    "parse-english": "^7.0.0",
+    "parse-latin": "^7.0.0",
+    "prettier": "^3.0.0",
     "remark-cli": "^11.0.0",
     "remark-preset-wooorm": "^9.0.0",
     "type-coverage": "^2.0.0",
-    "typescript": "^4.0.0",
-    "xo": "^0.53.0"
+    "typescript": "^5.0.0",
+    "xo": "^0.55.0"
   },
   "scripts": {
     "prepack": "npm run build && npm run format",
     "build": "tsc --build --clean && tsc --build && type-coverage",
-    "format": "remark . -qfo && prettier . -w --loglevel warn && xo --fix",
+    "format": "remark . -qfo && prettier . -w --log-level warn && xo --fix",
     "test-api": "node --conditions development test/index.js",
     "test-coverage": "c8 --check-coverage --100 --reporter lcov npm run test-api",
     "test": "npm run build && npm run format && npm run test-coverage"
@@ -89,7 +89,10 @@
           "no-await-in-loop": 0
         }
       }
-    ]
+    ],
+    "rules": {
+      "unicorn/prefer-at": "off"
+    }
   },
   "remarkConfig": {
     "plugins": [