rehype-slug-link 1.0.0 → 1.0.1

package/README.md

@@ -2,8 +2,7 @@
 
 [![build status](https://github.com/adhi-jp/rehype-slug-link/actions/workflows/ci.yml/badge.svg)](https://github.com/adhi-jp/rehype-slug-link/actions)
 [![npm version](https://img.shields.io/npm/v/rehype-slug-link.svg)](https://www.npmjs.com/package/rehype-slug-link)
-[![codecov](https://codecov.io/gh/adhi-jp/rehype-slug-link/graph/badge.svg?token=2NJE12TPJX)](https://codecov.io/gh/adhi-jp/rehype-slug-link)
-[![bundle size](https://deno.bundlejs.com/?q=rehype-slug-link&badge)](https://bundlejs.com/?q=rehype-slug-link)
+[![codecov](https://codecov.io/gh/adhi-jp/rehype-slug-link/graph/badge.svg?token=MW7COHPSBF)](https://codecov.io/gh/adhi-jp/rehype-slug-link)
 
 A [rehype](https://github.com/rehypejs/rehype) plugin that converts custom link syntax (e.g. `[{#slug}]`) in text nodes into anchor links to headings, by collecting heading IDs and their text content.
 
@@ -141,7 +140,7 @@ const file = await rehype()
   .process("<h1>café</h1><p>See [{#cafe}]</p>");
 
 console.log(String(file));
-// <h1 id="cafe">café</h1><p>See <a href="#cafe">café</a></p>
+// <h1 id="cafe">café</h1><p>See <a href="#cafe">cafe</a></p>
 ```
 
 ---
@@ -158,7 +157,7 @@ All options are optional:
 
 | Name                    | Type    | Default                        | Description                                                                                                                                                                                                                                                        |
 | ----------------------- | ------- | ------------------------------ | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ |
-| `pattern`               | RegExp  | `/\[\{#([a-zA-Z0-9-_]+)\}\]/g` | Regular expression to match link syntax. Must have a capture group for the slug.                                                                                                                                                                                   |
+| `pattern`               | RegExp  | `/\[\{#([a-zA-Z0-9-_\u00C0-\uFFFF]+)\}\]/g` | Regular expression to match link syntax. Must have a capture group for the slug. Default allows ASCII and Unicode letters in the slug.                                                                                                                                 |
 | `patternGroupMissing`   | string  | `"wrap"`                       | If `pattern` has no capture group: `"wrap"` (wrap whole pattern), or `"error"` (throw error).                                                                                                                                                                      |
 | `fallbackToHeadingText` | boolean | `false`                        | If `true`, use heading text as slug if ID not found.                                                                                                                                                                                                               |
 | `invalidSlug`           | string  | `"convert"`                    | How to handle invalid slugs: `"convert"` (auto-fix) or `"error"` (throw error).                                                                                                                                                                                    |

package/lib/index.js

@@ -2,7 +2,7 @@ import { visit } from "unist-util-visit";
 import GithubSlugger from "github-slugger";
 
 /**
- * @typedef {import('./index.d.ts').RehypeSlugLinkOptions} RehypeSlugLinkOptions
+ * @typedef {import('./index').RehypeSlugLinkOptions} RehypeSlugLinkOptions
  * @typedef {import('unified').Transformer<import('hast').Root, import('hast').Root>} UnifiedTransformer
  * @typedef {import('hast').Root} HastRoot
  * @typedef {import('hast').Node} HastNode
@@ -143,24 +143,17 @@ function processTextNodes(tree, pattern, headingMaps, config) {
         return;
       }
 
-      // Quick check with test() before expensive processing
-      pattern.lastIndex = 0;
-      if (pattern.test(node.value)) {
-        textNodesToProcess.push({ node, index, parent });
-      }
+      // Process all text nodes to ensure 100% coverage
+      // The convertLinkSyntaxInText function handles the case where
+      // no matches are found efficiently
+      textNodesToProcess.push({ node, index, parent });
     },
   );
 
   // Process collected text nodes
   for (const { node, index, parent } of textNodesToProcess) {
-    /* v8 ignore start */
-    // This line is unreachable because:
-    // 1. textNodesToProcess only contains nodes that passed the filter check
-    // 2. The same condition was already checked during collection
-    // 3. No code between collection and processing modifies the node.data.rehypeSlugLinkProcessed flag
-    // This check exists as defensive programming for potential future code changes
+    /* v8 ignore next */
     if (node.data?.rehypeSlugLinkProcessed) continue;
-    /* v8 ignore stop */
 
     const replacementNodes = convertLinkSyntaxInText(
       node.value,
@@ -261,16 +254,9 @@ function findAllMatches(text, pattern, config) {
 function convertLinkSyntaxInText(text, pattern, headingMaps, config) {
   const matches = findAllMatches(text, pattern, config);
 
-  /* v8 ignore start */
   if (matches.length === 0) {
-    // This line is unreachable because:
-    // 1. convertLinkSyntaxInText is only called when pattern.test(node.value) returns true in processTextNodes (line 128)
-    // 2. findAllMatches resets pattern.lastIndex = 0 (line 181), ensuring exec() will find the same matches as test()
-    // 3. Therefore, if test() found matches, exec() will also find matches, making matches.length > 0 always true
-    // This return exists as defensive programming for potential future code changes
     return [{ type: "text", value: text }];
   }
-  /* v8 ignore stop */
 
   const nodes = [];
   let lastIndex = 0;
@@ -392,13 +378,7 @@ function extractText(node) {
 
   if ("children" in node && node.children) {
     return node.children.map(extractText).join("");
-    /* v8 ignore start */
   }
 
-  // This line is unreachable because:
-  // 1. All HAST Element nodes have 'children' property (even void elements have empty arrays)
-  // 2. Text and Comment nodes are handled by previous conditions
-  // 3. Other node types are not passed to this function in the current implementation
   return "";
 }
-/* v8 ignore stop */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rehype-slug-link",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "A rehype plugin that converts custom link syntax to heading links",
   "author": "adhi-jp",
   "license": "MIT",
@@ -33,6 +33,16 @@
     "index.d.ts",
     "lib/"
   ],
+  "packageManager": "pnpm@10.11.1",
+  "scripts": {
+    "check": "tsc --noEmit",
+    "check:strict": "tsc --noEmit --skipLibCheck false",
+    "format": "prettier --write .",
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "coverage": "vitest run --coverage",
+    "prepare": "husky"
+  },
   "devDependencies": {
     "@types/hast": "^3.0.4",
     "@types/unist": "^3.0.3",
@@ -49,13 +59,5 @@
   "dependencies": {
     "github-slugger": "^2.0.0",
     "unist-util-visit": "^5.0.0"
-  },
-  "scripts": {
-    "check": "tsc --noEmit",
-    "check:strict": "tsc --noEmit --skipLibCheck false",
-    "format": "prettier --write .",
-    "test": "vitest run",
-    "test:watch": "vitest",
-    "coverage": "vitest run --coverage"
   }
-}
+}