@larkiny/astro-github-loader 0.14.1 → 0.14.3

package/dist/github.link-transform.js

@@ -181,6 +181,26 @@ function transformLink(linkText, linkUrl, context) {
         return `[${linkText}](${linkUrl})`;
     }
     const { path: linkPath, anchor } = extractAnchor(linkUrl);
+    // Try global linkMappings on the RAW link path before normalization.
+    // Bare-path links (e.g., "docs/markdown/autoapi/foo/") are repo-root-relative
+    // but normalizePath() treats them as file-relative, mangling the path.
+    // Applying global mappings first lets patterns match the link as written.
+    // Only for bare paths — relative (./, ../) and absolute (/) links must flow
+    // through normalizePath() first to avoid over-matching by generic global
+    // mappings like .md-stripping from generateStarlightLinkMappings().
+    const isBareBarePath = !linkPath.startsWith("./") &&
+        !linkPath.startsWith("../") &&
+        !linkPath.startsWith("/") &&
+        !linkPath.includes("://");
+    if (isBareBarePath && context.global.linkMappings) {
+        const globalMappings = context.global.linkMappings.filter((m) => m.global);
+        if (globalMappings.length > 0) {
+            const rawMapped = applyLinkMappings(linkPath + anchor, globalMappings, context);
+            if (rawMapped !== linkPath + anchor) {
+                return `[${linkText}](${rawMapped})`;
+            }
+        }
+    }
     // Normalize the link path relative to current file FIRST
     const normalizedPath = normalizePath(linkPath, context.currentFile.sourcePath, context.global.logger);
     // Apply global path mappings to the normalized path

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@larkiny/astro-github-loader",
   "type": "module",
-  "version": "0.14.1",
+  "version": "0.14.3",
   "description": "Load content from GitHub repositories into Astro content collections with asset management and content transformations",
   "keywords": [
     "astro",

package/src/github.link-transform.spec.ts

@@ -120,6 +120,97 @@ describe("globalLinkTransform", () => {
       expect(result[0].content).toBe("[See other](/other/)");
     });
 
+    it("should apply global linkMappings to bare-path links before normalization", () => {
+      const files: ImportedFile[] = [
+        createImportedFile(
+          "docs/markdown/capabilities/guide.md",
+          "src/content/docs/guide.md",
+          "[`AccountManager`](docs/markdown/autoapi/algokit_utils/accounts/account_manager/#algokit_utils.accounts.account_manager.AccountManager)",
+        ),
+      ];
+
+      const result = globalLinkTransform(files, {
+        stripPrefixes: ["src/content/docs"],
+        linkMappings: [
+          {
+            pattern: /^docs\/markdown\/autoapi\/algokit_utils\/(.+)/,
+            replacement: "/docs/algokit-utils/python/latest/api/$1",
+            global: true,
+          },
+        ],
+        logger,
+      });
+
+      expect(result[0].content).toBe(
+        "[`AccountManager`](/docs/algokit-utils/python/latest/api/accounts/account_manager/#algokit_utils.accounts.account_manager.AccountManager)",
+      );
+    });
+
+    it("should not early-return relative ./ links when .md-stripping global mappings exist", () => {
+      const files: ImportedFile[] = [
+        createImportedFile(
+          "docs/guide.md",
+          "src/content/docs/guide.md",
+          "[Subscriber](./subscriber.md)",
+        ),
+        createImportedFile(
+          "docs/subscriber.md",
+          "src/content/docs/subscriber.md",
+          "# Subscriber",
+        ),
+      ];
+
+      const result = globalLinkTransform(files, {
+        stripPrefixes: ["src/content/docs"],
+        linkMappings: [
+          {
+            pattern: /\.md(#|$)/,
+            replacement: "$1",
+            global: true,
+          },
+          {
+            pattern: /\/index(\.md)?$/,
+            replacement: "/",
+            global: true,
+          },
+        ],
+        logger,
+      });
+
+      // Should resolve via sourceToTargetMap, not early-return from .md stripping
+      expect(result[0].content).toBe("[Subscriber](/subscriber/)");
+    });
+
+    it("should not early-return relative ../ links when global mappings exist", () => {
+      const files: ImportedFile[] = [
+        createImportedFile(
+          "docs/guides/intro.md",
+          "src/content/docs/guides/intro.md",
+          "[Overview](../overview.md)",
+        ),
+        createImportedFile(
+          "docs/overview.md",
+          "src/content/docs/overview.md",
+          "# Overview",
+        ),
+      ];
+
+      const result = globalLinkTransform(files, {
+        stripPrefixes: ["src/content/docs"],
+        linkMappings: [
+          {
+            pattern: /\.md(#|$)/,
+            replacement: "$1",
+            global: true,
+          },
+        ],
+        logger,
+      });
+
+      // Should resolve via normalization + sourceToTargetMap
+      expect(result[0].content).toBe("[Overview](/overview/)");
+    });
+
     it("should preserve anchors in transformed links", () => {
       const files: ImportedFile[] = [
         createImportedFile(

package/src/github.link-transform.ts

@@ -314,6 +314,32 @@ function transformLink(
 
   const { path: linkPath, anchor } = extractAnchor(linkUrl);
 
+  // Try global linkMappings on the RAW link path before normalization.
+  // Bare-path links (e.g., "docs/markdown/autoapi/foo/") are repo-root-relative
+  // but normalizePath() treats them as file-relative, mangling the path.
+  // Applying global mappings first lets patterns match the link as written.
+  // Only for bare paths — relative (./, ../) and absolute (/) links must flow
+  // through normalizePath() first to avoid over-matching by generic global
+  // mappings like .md-stripping from generateStarlightLinkMappings().
+  const isBareBarePath =
+    !linkPath.startsWith("./") &&
+    !linkPath.startsWith("../") &&
+    !linkPath.startsWith("/") &&
+    !linkPath.includes("://");
+  if (isBareBarePath && context.global.linkMappings) {
+    const globalMappings = context.global.linkMappings.filter((m) => m.global);
+    if (globalMappings.length > 0) {
+      const rawMapped = applyLinkMappings(
+        linkPath + anchor,
+        globalMappings,
+        context,
+      );
+      if (rawMapped !== linkPath + anchor) {
+        return `[${linkText}](${rawMapped})`;
+      }
+    }
+  }
+
   // Normalize the link path relative to current file FIRST
   const normalizedPath = normalizePath(
     linkPath,