fix: reuse shared issue path parsing in router

ui/src/lib/issue-reference.test.ts

@@ -12,6 +12,12 @@ describe("issue-reference", () => {
     expect(parseIssuePathIdFromPath("http://localhost:3100/PAP/issues/PAP-1179")).toBe("PAP-1179");
   });
 
+  it("ignores placeholder issue paths", () => {
+    expect(parseIssuePathIdFromPath("/issues/:id")).toBeNull();
+    expect(parseIssuePathIdFromPath("http://localhost:3100/issues/:id")).toBeNull();
+    expect(parseIssueReferenceFromHref("/issues/:id")).toBeNull();
+  });
+
   it("normalizes bare identifiers, issue URLs, and issue scheme links into internal links", () => {
     expect(parseIssueReferenceFromHref("pap-1271")).toEqual({
       issuePathId: "PAP-1271",

ui/src/lib/router.tsx

@@ -9,12 +9,7 @@ import {
   extractCompanyPrefixFromPath,
   normalizeCompanyPrefix,
 } from "@/lib/company-routes";
-
-function parseIssuePathIdFromPath(pathname: string | null | undefined): string | null {
-  if (!pathname) return null;
-  const match = pathname.match(/(?:^|\/)issues\/([^/?#]+)/);
-  return match?.[1] ?? null;
-}
+import { parseIssuePathIdFromPath } from "@/lib/issue-reference";
 
 function resolveTo(to: To, companyPrefix: string | null): To {
   if (typeof to === "string") {