Remove /docs/ segments in categoryHrefToDocID

Author: ptgott

src/theme/DocCardList/href-to-id.test.ts

@@ -19,6 +19,16 @@ describe("categoryHrefToDocID", () => {
       href: "/reference/agent-services/",
       expectedID: "reference/agent-services/agent-services",
     },
+    {
+      description: "docs URL segment in versioned path",
+      href: "/docs/ver/15.x/admin-guides/access-controls/guides/joining-sessions",
+      expectedID: "admin-guides/access-controls/guides/joining-sessions/joining-sessions",
+    },
+    {
+      description: "docs URL segment in unversioned path",
+      href: "/docs/admin-guides/access-controls/guides/joining-sessions",
+      expectedID: "admin-guides/access-controls/guides/joining-sessions/joining-sessions",
+    },
   ];
 
   test.each(testCases)("$description", (c) => {

src/theme/DocCardList/href-to-id.ts

@@ -3,12 +3,16 @@
 // IDs in the items prop, so we generate a page ID based on the assumption that
 // category page slugs are the same as their containing directory names.
 export function categoryHrefToDocID(href: string): string {
-  const idPrefix = href.replace(new RegExp(`^(/ver/[0-9]+\\.x)?/`), "");
-  const slugRE = new RegExp(`/([^/]+)/$`);
+  // Remove initial path segments that are not involved in generating the page
+  // ID, such as "/docs/" and "/ver/15.x/".
+  let idPrefix = href.replace(new RegExp(`^/(docs/)?(ver/[0-9]+\\.x/)?`), "");
+  // Ensure that trailing slashes are trimmed so we can uniformly add the slug
+  // segment.
+  idPrefix = idPrefix.replace(new RegExp(`/$`), "");
+  const slugRE = new RegExp(`/([^/]+)/?$`);
   const slug = slugRE.exec(href);
-  if (!slug || slug.length != 2) {
-    return "";
+  if (!slug || slug.length < 2) {
+    throw new Error(`could not identify a category page ID for href ${href}`);
   }
-  return idPrefix + slug[1];
+  return idPrefix + "/" + slug[1];
 }
-