style: strip leading numbering from preview text before title deduplication

Author: marian2js

apps/desktop/src/features/dashboard/lib/strip-title-from-preview.ts

@@ -9,7 +9,11 @@ export function stripTitleFromPreview(title: string, summary: string): string {
 
   const stripped = stripMarkdown(summary);
   const normalizedTitle = title.replace(/\s+/g, " ").trim().toLowerCase();
-  const normalizedStripped = stripped.replace(/\s+/g, " ").trim();
+
+  // Strip leading list numbering (e.g. "1) ", "2. ") to match cleanArtifactTitle behavior
+  const withoutNumbering = stripped.replace(/^\d+[.)]\s+/, "");
+
+  const normalizedStripped = withoutNumbering.replace(/\s+/g, " ").trim();
   const normalizedStrippedLower = normalizedStripped.toLowerCase();
 
   if (normalizedTitle && normalizedStrippedLower.startsWith(normalizedTitle)) {

test/ui/strip-title-from-preview.test.ts

@@ -1,6 +1,7 @@
 import { describe, expect, it } from "vitest";
 import { readFileSync, existsSync } from "node:fs";
 import { resolve } from "node:path";
+import { stripTitleFromPreview } from "../../apps/desktop/src/features/dashboard/lib/strip-title-from-preview";
 
 const utilPath = resolve(
   __dirname,
@@ -51,6 +52,54 @@ describe("strip-title-from-preview utility", () => {
   });
 });
 
+describe("stripTitleFromPreview — behavioral", () => {
+  it("returns empty string for empty summary", () => {
+    expect(stripTitleFromPreview("Some title", "")).toBe("");
+  });
+
+  it("strips title prefix from summary", () => {
+    expect(
+      stripTitleFromPreview(
+        "Feature matrix",
+        "Feature matrix with grouped rows by use case",
+      ),
+    ).toBe("with grouped rows by use case");
+  });
+
+  it("returns full summary when title is not a prefix", () => {
+    expect(
+      stripTitleFromPreview("Unrelated title", "Some preview text here"),
+    ).toBe("Some preview text here");
+  });
+
+  it("strips leading numbering from preview before title comparison", () => {
+    expect(
+      stripTitleFromPreview(
+        "Rewritten Free vs Pro feature matrix with grouped rows by",
+        "1) Rewritten Free vs Pro feature matrix with grouped rows by use case > Important framing",
+      ),
+    ).toBe("use case > Important framing");
+  });
+
+  it("strips '2. ' numbering prefix from preview before title comparison", () => {
+    expect(
+      stripTitleFromPreview(
+        "Updated homepage hero copy",
+        "2. Updated homepage hero copy: new version with better CTA",
+      ),
+    ).toBe("new version with better CTA");
+  });
+
+  it("handles preview without numbering as before", () => {
+    expect(
+      stripTitleFromPreview(
+        "SEO Audit Key Findings",
+        "SEO Audit Key Findings — detailed breakdown follows",
+      ),
+    ).toBe("detailed breakdown follows");
+  });
+});
+
 describe("ArtifactCard title-echo removal", () => {
   it("ArtifactCard imports stripTitleFromPreview", () => {
     const src = readFileSync(artifactCardPath, "utf-8");