fix: normalize trailing slashes for page backlinks (#42)

src/server/lib/dataforseoBacklinks.test.ts

@@ -24,6 +24,24 @@ describe("normalizeBacklinksTarget", () => {
     });
   });
 
+  it("trims trailing slashes from non-root page URLs", () => {
+    expect(
+      normalizeBacklinksTarget("https://github.com/every-app/open-seo/"),
+    ).toEqual({
+      apiTarget: "https://github.com/every-app/open-seo",
+      displayTarget: "https://github.com/every-app/open-seo",
+      scope: "page",
+    });
+  });
+
+  it("keeps trailing slashes for root page URLs", () => {
+    expect(normalizeBacklinksTarget("https://example.com/")).toEqual({
+      apiTarget: "https://example.com/",
+      displayTarget: "https://example.com/",
+      scope: "page",
+    });
+  });
+
   it("treats bare hostnames as domain lookups", () => {
     expect(normalizeBacklinksTarget("Example.com")).toEqual({
       apiTarget: "example.com",

src/server/lib/dataforseoBacklinksTarget.ts

@@ -11,6 +11,17 @@ type NormalizeBacklinksTargetOptions = {
   scope?: BacklinksLookupInput["scope"];
 };
 
+function normalizePageTargetUrl(url: URL, hostname: string): string {
+  const normalizedUrl = new URL(url.toString());
+  normalizedUrl.hostname = hostname;
+
+  if (normalizedUrl.pathname.length > 1) {
+    normalizedUrl.pathname = normalizedUrl.pathname.replace(/\/+$/, "");
+  }
+
+  return normalizedUrl.toString();
+}
+
 export function normalizeBacklinksTarget(
   input: string,
   options: NormalizeBacklinksTargetOptions = {},
@@ -63,23 +74,26 @@ export function normalizeBacklinksTarget(
 
   if (requestedScope === "page") {
     const normalizedUrl = new URL(parsed.toString());
-    normalizedUrl.hostname = exactHostname;
     if (!hasExplicitProtocol && !hasMeaningfulPath) {
       normalizedUrl.pathname = "/";
     }
+
+    const normalizedTarget = normalizePageTargetUrl(
+      normalizedUrl,
+      exactHostname,
+    );
     return {
-      apiTarget: normalizedUrl.toString(),
+      apiTarget: normalizedTarget,
-      displayTarget: normalizedUrl.toString(),
+      displayTarget: normalizedTarget,
       scope: "page",
     };
   }
 
   if (hasExplicitProtocol || hasMeaningfulPath) {
-    const normalizedUrl = new URL(parsed.toString());
+    const normalizedTarget = normalizePageTargetUrl(parsed, exactHostname);
-    normalizedUrl.hostname = exactHostname;
     return {
-      apiTarget: normalizedUrl.toString(),
+      apiTarget: normalizedTarget,
-      displayTarget: normalizedUrl.toString(),
+      displayTarget: normalizedTarget,
       scope: "page",
     };
   }