@nextclaw/x-harness 0.2.51 → 0.2.52

package/SOURCE.md

@@ -1,8 +1,8 @@
 # Corresponding source
 
-Version 0.2.51, commit 4fcb5b63209a99a05a176cc9a7242c1f7de614ea.
+Version 0.2.52, commit b3e3310d4977db067f05ee409f58ae619f042c9a.
 
-Complete source, build scripts and dependencies: https://github.com/Peiiii/x-harness/tree/4fcb5b63209a99a05a176cc9a7242c1f7de614ea
+Complete source, build scripts and dependencies: https://github.com/Peiiii/x-harness/tree/b3e3310d4977db067f05ee409f58ae619f042c9a
 
 Clone this revision, run npm ci, npm run bootstrap and npm run release:build.
 

package/dist/browser-dom-navigation.js

@@ -71,6 +71,12 @@ export class DOMNavigation {
         return;
     }
     async handleWaitForFolder(command) {
+        if (command.selected) {
+            await this.#tab.playwright
+                .getByRole("heading", { name: command.folder, exact: true })
+                .waitFor({ state: "visible", timeoutMs: 5000 });
+            return;
+        }
         if (command.folderDiscovery) {
             // "All bookmarks" renders before the asynchronously loaded named cards.
             // Wait for any named card inside that specific menu, never an unrelated

package/dist/platform-collections.d.ts

@@ -9,6 +9,7 @@ export declare class PlatformCollections {
     private handleCollectSources;
     private handleReadCreator;
     private handleReadBookmarks;
+    private confirmBookmarkFolder;
     private bookmarkDiscovery;
     private waitForBookmarkLabel;
     private handleSearchSources;

package/dist/platform-collections.js

@@ -294,15 +294,9 @@ export class PlatformCollections {
                 folder,
                 controls: nav.controls,
             };
-        const selected = await this.#ready({ view: "navigation" });
-        if (selected.page === nav.page && !selected.headings?.includes(folder))
-            return {
-                status: "needsDecision",
-                complete: false,
-                missing: ["folder-selection-unconfirmed"],
-                folder,
-                page: selected.page,
-            };
+        const selected = await this.confirmBookmarkFolder(folder);
+        if (selected.status)
+            return selected;
         const value = await this.handleCollectSources(window);
         return {
             ...value,
@@ -314,6 +308,35 @@ export class PlatformCollections {
             },
         };
     }
+    async confirmBookmarkFolder(folder) {
+        let selected = await this.#ready({ view: "navigation" });
+        if (!selected.headings?.includes(folder)) {
+            try {
+                await this.#browser.act({
+                    operation: "waitForFolder",
+                    folder,
+                    selected: true,
+                });
+            }
+            catch (error) {
+                if (!/timed out|timeout/i.test(error.message))
+                    throw error;
+            }
+            selected = await this.#ready({ view: "navigation" });
+        }
+        if (!selected.headings?.includes(folder) ||
+            !/\/i\/(?:history\/)?bookmarks\//.test(selected.page ?? "") ||
+            selected.needsStateCheck ||
+            selected.alerts?.length)
+            return {
+                status: "needsDecision",
+                complete: false,
+                missing: ["folder-selection-unconfirmed"],
+                folder,
+                page: selected.page,
+            };
+        return selected;
+    }
     bookmarkDiscovery(nav) {
         const folders = nav.bookmarkFolders ?? [];
         return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nextclaw/x-harness",
-  "version": "0.2.51",
+  "version": "0.2.52",
   "type": "module",
   "description": "Typed X task harness with compact business results and explicit diagnostics.",
   "main": "./dist/index.js",
@@ -42,5 +42,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "4fcb5b63209a99a05a176cc9a7242c1f7de614ea"
+  "gitHead": "b3e3310d4977db067f05ee409f58ae619f042c9a"
 }