From 1d4c5ee34d57427694b2fcc148bf5ae1e1b2bbfd Mon Sep 17 00:00:00 2001
From: Cursor Agent <cursoragent@cursor.com>
Date: Wed, 5 Aug 2026 20:17:08 +0000
Subject: [PATCH] fix(test): settle visible search-query-ribbon in ui-tools
 composer check

CI can leave a hidden Next streaming `#S:1` clone beside the live favourites
ribbon (outstanding #093), so bare getByTestId fails strict mode. Match the
settlement helpers already used elsewhere in this file.
---
 tests/ui-tools.spec.ts | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/tests/ui-tools.spec.ts b/tests/ui-tools.spec.ts
index 7c4d1f13b8..87077e42c9 100644
--- a/tests/ui-tools.spec.ts
+++ b/tests/ui-tools.spec.ts
@@ -980,7 +980,9 @@ test.describe("Clinical KB tools launcher", () => {
           timeout: 20_000,
         });
         if (route.ribbonQuery) {
-          const ribbon = page.getByTestId("search-query-ribbon");
+          // Prefer the settled visible ribbon — CI can leave a hidden Next
+          // streaming `#S:1` clone beside the live band (outstanding #093).
+          const ribbon = visibleByTestId(page, "search-query-ribbon");
           await expect(ribbon, `${route.path} at ${viewport.name}`).toBeVisible({ timeout: 20_000 });
           await expect(ribbon.getByRole("heading", { name: route.ribbonQuery })).toBeVisible();
           await expect(ribbon.getByRole("status")).toBeVisible();