修复 macOS 端到端交互稳定性

Author: 1625567290

apps/desktop/e2e/prompt-rail.spec.ts

@@ -211,28 +211,43 @@ test('the pointer is always on a tick while it travels down the rail', async ({
   await expect
     .poll(async () => {
       const travel = await page.evaluate(() => {
-        const ticks = [...document.querySelectorAll('.maka-prompt-rail-tick')];
+        const ticks = [...document.querySelectorAll<HTMLElement>('.maka-prompt-rail-tick')];
         if (ticks.length < 2) throw new Error('the prompt rail needs at least two ticks');
-        const first = ticks[0]!.getBoundingClientRect();
-        const last = ticks[ticks.length - 1]!.getBoundingClientRect();
-        const x = Math.round(first.left + first.width / 2);
+        const boxes = ticks.map((tick) => tick.getBoundingClientRect());
+        const first = boxes[0]!;
+        const last = boxes[boxes.length - 1]!;
         const describe = (found: Element | null) =>
           found instanceof Element
             ? `${found.tagName.toLowerCase()}.${found.className.toString().trim().replace(/\s+/g, '.')}`
             : 'null';
-        const startY = Math.round(first.top + first.height / 2);
-        const endY = Math.round(last.top + last.height / 2);
-        const misses: Array<{ y: number; hit: string }> = [];
-        for (let y = startY; y <= endY; y += 1) {
-          const found = document.elementFromPoint(x, y);
-          if (!found?.closest('.maka-prompt-rail-tick')) {
-            misses.push({ y, hit: describe(found) });
+        const misses: Array<{ index: number; kind: 'center' | 'seam'; hit: string }> = [];
+        for (const [index, box] of boxes.entries()) {
+          const found = document.elementFromPoint(
+            box.left + box.width / 2,
+            box.top + box.height / 2,
+          );
+          if (found?.closest('.maka-prompt-rail-tick') !== ticks[index]) {
+            misses.push({ index, kind: 'center', hit: describe(found) });
+          }
+        }
+        const gaps = boxes.slice(1).map((box, index) => box.top - boxes[index]!.bottom);
+        for (let index = 0; index < boxes.length - 1; index += 1) {
+          const before = boxes[index]!;
+          const after = boxes[index + 1]!;
+          const found = document.elementFromPoint(
+            before.left + before.width / 2,
+            (before.bottom + after.top) / 2,
+          );
+          const landed = found?.closest('.maka-prompt-rail-tick');
+          if (landed !== ticks[index] && landed !== ticks[index + 1]) {
+            misses.push({ index, kind: 'seam', hit: describe(found) });
           }
         }
         return {
-          insideViewport: startY >= 0 && endY < window.innerHeight,
+          insideViewport: first.top >= 0 && last.bottom <= window.innerHeight,
           misses: misses.length,
-          hasSpan: endY > startY,
+          hasSpan: last.bottom > first.top,
+          continuousBoxes: Math.max(...gaps) <= 0.25,
           sample: misses.slice(0, 3),
         };
       });
@@ -242,6 +257,7 @@ test('the pointer is always on a tick while it travels down the rail', async ({
       insideViewport: true,
       misses: 0,
       hasSpan: true,
+      continuousBoxes: true,
     });
 });
 

apps/desktop/e2e/skill-draft-lifecycle.spec.ts

@@ -54,9 +54,19 @@ async function composeWithSkill(page: Page, text: string, name: RegExp): Promise
   await composer.fill(text);
   await composer.click();
   await composer.pressSequentially(' /');
-  const option = page.getByRole('listbox', { name: /技能/ }).getByRole('option', { name });
+  const listbox = page.getByRole('listbox', { name: /技能/ });
+  const option = listbox.getByRole('option', { name });
   await expect(option).toBeVisible();
-  await option.click();
+  // The suggestion list can re-rank once when the cold Skill projection
+  // catches up. Walk its active-descendant state instead of racing a pointer
+  // target that is being replaced under the cursor.
+  const optionCount = await listbox.getByRole('option').count();
+  for (let index = 0; index < optionCount; index += 1) {
+    if ((await option.getAttribute('aria-selected')) === 'true') break;
+    await composer.press('ArrowDown');
+  }
+  await expect(option).toHaveAttribute('aria-selected', 'true');
+  await composer.press('Enter');
 }
 
 async function beginRevision(page: Page): Promise<void> {