@larkiny/astro-github-loader 0.14.4 → 0.14.6

package/dist/github.link-transform.js

@@ -1,4 +1,3 @@
-import { slug } from "github-slugger";
 import path from "node:path";
 /**
  * Extract anchor fragment from a link
@@ -148,12 +147,11 @@ function generateSiteUrl(targetPath, stripPrefixes) {
     else if (url === "index") {
         url = "";
     }
-    // Split path into segments and slugify each
-    const segments = url
+    // Filter empty segments (from double slashes etc.)
+    url = url
         .split("/")
-        .map((segment) => (segment ? slug(segment) : ""));
-    // Reconstruct URL
-    url = segments.filter((s) => s).join("/");
+        .filter((s) => s)
+        .join("/");
     // Ensure leading slash
     if (url && !url.startsWith("/")) {
         url = "/" + url;
@@ -199,7 +197,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.4",
+  "version": "0.14.6",
   "description": "Load content from GitHub repositories into Astro content collections with asset management and content transformations",
   "keywords": [
     "astro",
@@ -46,7 +46,6 @@
   },
   "dependencies": {
     "@octokit/auth-app": "^8.1.1",
-    "github-slugger": "^2.0.0",
     "octokit": "^5.0.4",
     "picomatch": "^4.0.2"
   },

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

@@ -244,6 +244,69 @@ describe("globalLinkTransform", () => {
       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 dots and case in generated site URLs", () => {
+      // TypeDoc output uses dot-separated filenames like types_account_manager.AccountManager.md
+      // The generated URL should preserve dots and case to match Starlight's generateId
+      const files: ImportedFile[] = [
+        createImportedFile(
+          "latest/api/classes/types_account_manager.AccountManager.md",
+          "src/content/docs/docs/algokit-utils/typescript/latest/api/classes/types_account_manager.AccountManager.md",
+          "# AccountManager",
+        ),
+        createImportedFile(
+          "latest/api/classes/types_app_factory.AppFactory.md",
+          "src/content/docs/docs/algokit-utils/typescript/latest/api/classes/types_app_factory.AppFactory.md",
+          "[AccountManager](types_account_manager.AccountManager.md)",
+        ),
+      ];
+
+      const result = globalLinkTransform(files, {
+        stripPrefixes: ["src/content/docs"],
+        linkMappings: [
+          { pattern: /\.md(#|$)/, replacement: "$1", global: true },
+        ],
+        logger,
+      });
+
+      // URL should preserve dots and case — NOT slugify to types_account_manageraccountmanager
+      expect(result[1].content).toBe(
+        "[AccountManager](/docs/algokit-utils/typescript/latest/api/classes/types_account_manager.AccountManager/)",
+      );
+    });
+
     it("should preserve anchors in transformed links", () => {
       const files: ImportedFile[] = [
         createImportedFile(

package/src/github.link-transform.ts

@@ -1,4 +1,3 @@
-import { slug } from "github-slugger";
 import path from "node:path";
 import type {
   LinkMapping,
@@ -270,13 +269,11 @@ function generateSiteUrl(targetPath: string, stripPrefixes: string[]): string {
     url = "";
   }
 
-  // Split path into segments and slugify each
-  const segments = url
+  // Filter empty segments (from double slashes etc.)
+  url = url
     .split("/")
-    .map((segment) => (segment ? slug(segment) : ""));
-
-  // Reconstruct URL
-  url = segments.filter((s) => s).join("/");
+    .filter((s) => s)
+    .join("/");
 
   // Ensure leading slash
   if (url && !url.startsWith("/")) {
@@ -337,7 +334,7 @@ function transformLink(
         globalMappings,
         context,
       );
-      if (rawMapped !== linkPath + anchor) {
+      if (rawMapped !== linkPath + anchor && rawMapped.startsWith("/")) {
         return `[${linkText}](${rawMapped})`;
       }
     }