fix(check): key content_overlap collapse by pair, not text

`staticIssueKey` put `issue.text` in the collapse key, but `overlapIssue`
already names a collision by both of its selectors. When the leading element's
text changes per sample — a count-up, typewriter, or rotating word — every
dense-pass sample formed its own group with `occurrences: 1`, so
`applyPersistenceTier` demoted all of them to `info` and a genuinely held
collision never reached the `error` tier that gates a run.

Empirically: two projects with identical geometry and an identical 5.9s
overlap, differing only in DOM order, verdicted differently — label-first gave
`error`/`ok=false`, count-up-first gave 48x `info`/`ok=true`. After this change
both give `error`/`ok=false` with `occurrences: 48`. Seven other overlap
fixtures (crossfades, static, masked reel, 600ms transient) are unchanged.

Codes that identify themselves by text keep it; only `content_overlap`, which
carries both selectors, drops it.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Author: xuanruli

packages/cli/src/utils/layoutAudit.test.ts

@@ -255,6 +255,42 @@ describe("persistence-tiered severity (#U10)", () => {
     expect(collapsed[0]).toMatchObject({ severity: "warning", occurrences: 2 });
   });
 
+  it("promotes a content_overlap whose text changes every sample (count-up over a label)", () => {
+    // The colliding pair is named by both selectors, so per-sample text must not split one held collision into transient groups.
+    const collapsed = collapseStaticLayoutIssues(
+      [
+        {
+          ...issue("content_overlap", "warning"),
+          time: 4.0,
+          containerSelector: ".num",
+          text: "$1,204",
+        },
+        {
+          ...issue("content_overlap", "warning"),
+          time: 4.5,
+          containerSelector: ".num",
+          text: "$8,930",
+        },
+      ],
+      73,
+    );
+
+    expect(collapsed).toHaveLength(1);
+    expect(collapsed[0]).toMatchObject({ severity: "error", occurrences: 2 });
+  });
+
+  it("still separates two distinct text_box_overflow findings that differ only by text", () => {
+    const collapsed = collapseStaticLayoutIssues(
+      [
+        { ...issue("text_box_overflow", "warning"), time: 4.0, text: "first" },
+        { ...issue("text_box_overflow", "warning"), time: 4.5, text: "second" },
+      ],
+      73,
+    );
+
+    expect(collapsed).toHaveLength(2);
+  });
+
   it("promotes content_overlap whose two occurrences span exactly 500ms (at the floor)", () => {
     const collapsed = collapseStaticLayoutIssues(
       [

packages/cli/src/utils/layoutAudit.ts

@@ -342,12 +342,18 @@ function staticIssueKey(issue: LayoutIssue): string {
     issue.severity,
     issue.selector,
     issue.containerSelector ?? "",
-    issue.text ?? "",
+    collapseTextKey(issue),
     issue.overflow ? formatOverflow(issue.overflow) : "",
     framePositionKey(issue),
   ].join("|");
 }
 
+/** Text identity for the collapse key, omitted where the code identifies itself by geometry instead. */
+function collapseTextKey(issue: LayoutIssue): string {
+  // content_overlap names its pair by both selectors, so animated text would split one held collision into per-sample groups.
+  return issue.code === "content_overlap" ? "" : (issue.text ?? "");
+}
+
 function framePositionKey(issue: LayoutIssue): string {
   // connector_detached shares it: id-less paths collapse to one selector, so distinct lines need geometry in the key.
   return issue.code === "frame_out_of_frame" || issue.code === "connector_detached"