fix(mobile): simplify ordered list width detection

Author: PR Batch Tester

apps/mobile/src/lib/orderedListMarkers.test.ts

@@ -2,43 +2,24 @@ import { describe, expect, it } from "vite-plus/test";
 
 import { wideOrderedListMarkerDigits } from "./orderedListMarkers";
 
-const numberedList = (count: number, marker = (index: number) => `${index + 1}.`) =>
-  Array.from({ length: count }, (_, index) => `${marker(index)} item`).join("\n");
+const numberedList = (count: number, marker = "1.") =>
+  Array.from({ length: count }, () => `${marker} item`).join("\n");
 
 describe("wideOrderedListMarkerDigits", () => {
-  it("returns 0 while every number fits in two digits", () => {
+  it("ignores text and lists that fit the normal gutter", () => {
     expect(wideOrderedListMarkerDigits("just a message")).toBe(0);
-    expect(wideOrderedListMarkerDigits("- one\n- two")).toBe(0);
-    expect(wideOrderedListMarkerDigits(numberedList(99))).toBe(0);
     expect(wideOrderedListMarkerDigits("We shipped it in 2026. More later.")).toBe(0);
-    expect(wideOrderedListMarkerDigits("Version 1.2 is out")).toBe(0);
-    const withSubLists = Array.from(
-      { length: 60 },
-      (_, index) => `${index + 1}. item\n    1. sub\n    2. sub`,
-    ).join("\n");
-    expect(wideOrderedListMarkerDigits(withSubLists)).toBe(0);
-  });
-
-  it("measures the widest written number", () => {
-    expect(wideOrderedListMarkerDigits(numberedList(130))).toBe(3);
-    expect(wideOrderedListMarkerDigits("1000) a")).toBe(4);
-    expect(wideOrderedListMarkerDigits("text\r\n\r\n100.\r\n")).toBe(3);
-    expect(wideOrderedListMarkerDigits("007. agent")).toBe(0);
+    expect(wideOrderedListMarkerDigits(numberedList(99))).toBe(0);
   });
 
-  it("covers lists the renderer numbers itself", () => {
-    expect(wideOrderedListMarkerDigits(numberedList(101, () => "1."))).toBe(3);
+  it("covers written and renderer-generated three-digit markers", () => {
+    expect(wideOrderedListMarkerDigits("100. item")).toBe(3);
+    expect(wideOrderedListMarkerDigits(numberedList(100))).toBe(3);
+    expect(wideOrderedListMarkerDigits(numberedList(100, "> 1."))).toBe(3);
   });
 
-  it("finds lists nested in other blocks", () => {
-    expect(wideOrderedListMarkerDigits("- 100000. nested")).toBe(6);
-    expect(wideOrderedListMarkerDigits("> > 250. quoted")).toBe(3);
-    expect(wideOrderedListMarkerDigits("1. a\n\n   100. nested")).toBe(3);
+  it("measures ordered markers nested in other list items", () => {
     expect(wideOrderedListMarkerDigits("100. 1. item")).toBe(3);
-    expect(wideOrderedListMarkerDigits(numberedList(120, () => "1. 1."))).toBe(3);
-  });
-
-  it("stays linear on long runs of bullets and spaces", () => {
-    expect(wideOrderedListMarkerDigits(`${"-    ".repeat(40)}x`)).toBe(0);
+    expect(wideOrderedListMarkerDigits("- 1000. nested")).toBe(4);
   });
 });

apps/mobile/src/lib/orderedListMarkers.ts

@@ -5,29 +5,20 @@
 const ORDERED_MARKER_LINE =
   /^([ \t>]*(?:(?:[-+*]|\d{1,9}[.)])[ \t]+(?:>[ \t>]*)?)*)(\d{1,9})[.)](?:[ \t\r]|$)/;
 const PREFIX_NUMBER = /(\d{1,9})[.)]/g;
-const TOP_LEVEL_ITEM = /^ {0,3}\d/;
-
-/**
- * Digit count of the widest ordered-list number in `markdown`, when that is
- * three or more digits; otherwise 0.
- *
- * Agents write their list numbers out, so the largest written number covers
- * normal replies. Counting top-level items also covers lists the renderer
- * numbers itself, such as `1.` on every line. Code blocks are not skipped; a
- * stray match there only widens the gutter.
- */
 export function wideOrderedListMarkerDigits(markdown: string): number {
   let widest = 0;
-  let topLevelItems = 0;
+  let items = 0;
   for (const line of markdown.split("\n")) {
     const marker = ORDERED_MARKER_LINE.exec(line);
     if (!marker) continue;
+    items += 1;
     widest = Math.max(widest, Number(marker[2]));
     for (const outer of marker[1]!.matchAll(PREFIX_NUMBER)) {
       widest = Math.max(widest, Number(outer[1]));
     }
-    if (TOP_LEVEL_ITEM.test(line)) topLevelItems += 1;
   }
-  const digits = String(Math.max(widest, topLevelItems)).length;
+  // The renderer increments repeated `1.` markers. Counting every matched item
+  // may over-size messages with many separate lists, but it cannot clip them.
+  const digits = String(Math.max(widest, items)).length;
   return digits >= 3 ? digits : 0;
 }