@larkiny/astro-github-loader 0.14.3 → 0.14.5

package/dist/github.link-transform.js

@@ -185,10 +185,13 @@ function transformLink(linkText, linkUrl, context) {
     // 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("./") &&
+    // Only for multi-segment bare paths — relative (./, ../) and absolute (/) links
+    // must flow through normalizePath() first. Single-segment bare paths (no "/")
+    // like "api-algopy" or "types_amount.AlgoAmount" are sibling-file references
+    // that also need normalization, so we require at least one "/" to distinguish
+    // repo-root-relative paths from sibling references.
+    const isBareBarePath = linkPath.includes("/") &&
+        !linkPath.startsWith("./") &&
         !linkPath.startsWith("../") &&
         !linkPath.startsWith("/") &&
         !linkPath.includes("://");
@@ -196,7 +199,7 @@ function transformLink(linkText, linkUrl, context) {
         const globalMappings = context.global.linkMappings.filter((m) => m.global);
         if (globalMappings.length > 0) {
             const rawMapped = applyLinkMappings(linkPath + anchor, globalMappings, context);
-            if (rawMapped !== linkPath + anchor) {
+            if (rawMapped !== linkPath + anchor && rawMapped.startsWith("/")) {
                 return `[${linkText}](${rawMapped})`;
             }
         }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@larkiny/astro-github-loader",
   "type": "module",
-  "version": "0.14.3",
+  "version": "0.14.5",
   "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

@@ -211,6 +211,72 @@ describe("globalLinkTransform", () => {
       expect(result[0].content).toBe("[Overview](/overview/)");
     });
 
+    it("should not early-return single-segment bare-path sibling references", () => {
+      // Single-segment bare paths like "page-b.md" are sibling-file references.
+      // They must go through normalizePath() (joining with current dir) to resolve
+      // via sourceToTargetMap, NOT be caught by the bare-path pre-normalization check.
+      const files: ImportedFile[] = [
+        createImportedFile(
+          "docs/api/page-a.md",
+          "src/content/docs/api/page-a.md",
+          "[See B](page-b.md#section)",
+        ),
+        createImportedFile(
+          "docs/api/page-b.md",
+          "src/content/docs/api/page-b.md",
+          "# Page B",
+        ),
+      ];
+
+      const result = globalLinkTransform(files, {
+        stripPrefixes: ["src/content/docs"],
+        linkMappings: [
+          {
+            pattern: /\.md(#|$)/,
+            replacement: "$1",
+            global: true,
+          },
+        ],
+        logger,
+      });
+
+      // "page-b.md" normalizes to "docs/api/page-b.md", resolves via sourceToTargetMap
+      expect(result[0].content).toBe("[See B](/api/page-b/#section)");
+    });
+
+    it("should not early-return multi-segment bare-path sibling references with .md", () => {
+      // Multi-segment bare paths like "types/subscription.md" are file-relative
+      // sibling references (e.g., from subscriber.md to types/subscription.md).
+      // The .md-stripping global mapping should NOT cause an early return — the
+      // link must flow through normalizePath() + sourceToTargetMap to resolve.
+      const files: ImportedFile[] = [
+        createImportedFile(
+          "latest/api/subscriber.md",
+          "src/content/docs/docs/algokit-subscriber/typescript/latest/api/subscriber.md",
+          "[`AlgorandSubscriberConfig`](types/subscription.md#algorandsubscriberconfig)",
+        ),
+        createImportedFile(
+          "latest/api/types/subscription.md",
+          "src/content/docs/docs/algokit-subscriber/typescript/latest/api/types/subscription.md",
+          "# subscription",
+        ),
+      ];
+
+      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 to absolute URL, NOT leave as relative
+      expect(result[0].content).toBe(
+        "[`AlgorandSubscriberConfig`](/docs/algokit-subscriber/typescript/latest/api/types/subscription/#algorandsubscriberconfig)",
+      );
+    });
+
     it("should preserve anchors in transformed links", () => {
       const files: ImportedFile[] = [
         createImportedFile(

package/src/github.link-transform.ts

@@ -318,10 +318,13 @@ function transformLink(
   // 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().
+  // Only for multi-segment bare paths — relative (./, ../) and absolute (/) links
+  // must flow through normalizePath() first. Single-segment bare paths (no "/")
+  // like "api-algopy" or "types_amount.AlgoAmount" are sibling-file references
+  // that also need normalization, so we require at least one "/" to distinguish
+  // repo-root-relative paths from sibling references.
   const isBareBarePath =
+    linkPath.includes("/") &&
     !linkPath.startsWith("./") &&
     !linkPath.startsWith("../") &&
     !linkPath.startsWith("/") &&
@@ -334,7 +337,7 @@ function transformLink(
         globalMappings,
         context,
       );
-      if (rawMapped !== linkPath + anchor) {
+      if (rawMapped !== linkPath + anchor && rawMapped.startsWith("/")) {
         return `[${linkText}](${rawMapped})`;
       }
     }