@nextclaw/x-harness 0.2.55 → 0.2.56

package/SOURCE.md

@@ -1,8 +1,8 @@
 # Corresponding source
 
-Version 0.2.55, commit 7fc4a6a0e9bcea24ea2ac8d739572dc15afad136.
+Version 0.2.56, commit 84766d97c35a9cecbee5b8c6461aaa2e28d02c81.
 
-Complete source, build scripts and dependencies: https://github.com/Peiiii/x-harness/tree/7fc4a6a0e9bcea24ea2ac8d739572dc15afad136
+Complete source, build scripts and dependencies: https://github.com/Peiiii/x-harness/tree/84766d97c35a9cecbee5b8c6461aaa2e28d02c81
 
 Clone this revision, run npm ci, npm run bootstrap and npm run release:build.
 

package/dist/browser-dom-editing.js

@@ -198,7 +198,9 @@ export class DOMEditing {
             .locator(`a[href=${JSON.stringify(primary)}]`)
             .filter({ has: this.#tab.playwright.locator("time") }));
         this.#session.current = null;
-        await anchor.press("PageDown");
+        // A retained timestamp can be far below the viewport. Validate it, then
+        // scroll the page without waiting for that offscreen link to take focus.
+        await (await this.#unique(this.#tab.playwright.locator("body"))).press("PageDown");
         return { completed: true };
         return { completed: false, loc, submittedEditor };
     }

package/dist/platform-collections.js

@@ -158,38 +158,50 @@ export class PlatformCollections {
     }
     async advanceSourceWindow(previous, seen) {
         const renderedIds = previous.renderedIds ?? [...seen];
-        await this.#browser.act({ operation: "nextPosts" });
-        let advanced = await this.#ready({ view: "posts", limit: 5 });
-        if (advanced.needsStateCheck || advanced.alerts?.length)
-            return advanced;
-        let timedOut = false;
-        try {
-            // Wait for a newly rendered card, even if that source was processed on an
-            // earlier run. Global processed IDs do not describe the current DOM window.
-            await this.#browser.act({
-                operation: "waitForPosts",
-                excludeIds: renderedIds,
-            });
-        }
-        catch (error) {
-            if (!/timed out|timeout/i.test(error.message))
-                throw error;
-            timedOut = true;
+        let advanced;
+        for (let step = 0; step < 10; step++) {
+            await this.#browser.act({ operation: "nextPosts" });
+            advanced = await this.#ready({ view: "posts", limit: 5 });
+            if (advanced.needsStateCheck || advanced.alerts?.length)
+                return advanced;
+            let timedOut = false;
+            try {
+                // Wait for a newly rendered card, even if that source was processed on an
+                // earlier run. Global processed IDs do not describe the current DOM window.
+                await this.#browser.act({
+                    operation: "waitForPosts",
+                    excludeIds: renderedIds,
+                });
+            }
+            catch (error) {
+                if (!/timed out|timeout/i.test(error.message))
+                    throw error;
+                timedOut = true;
+            }
+            advanced = await this.#ready({ view: "posts", limit: 5 });
+            if (timedOut &&
+                advanced.atRenderedTail &&
+                !advanced.loading &&
+                advanced.firstRenderedId === previous.firstRenderedId &&
+                advanced.renderedArticles === previous.renderedArticles)
+                advanced.paginationBoundary = {
+                    kind: "stable-rendered-tail",
+                    lastId: advanced.renderedIds?.at(-1),
+                    renderedCount: advanced.renderedArticles,
+                    waitMs: 5000,
+                    scopeComplete: false,
+                };
+            if (advanced.paginationBoundary ||
+                !Number.isFinite(advanced.viewportScrollTop) ||
+                advanced.renderedArticles !== previous.renderedArticles ||
+                (advanced.renderedIds ?? advanced.posts?.map((p) => p.id))?.some((id) => !renderedIds.includes(id)))
+                return advanced;
         }
-        advanced = await this.#ready({ view: "posts", limit: 5 });
-        if (timedOut &&
-            advanced.atRenderedTail &&
-            !advanced.loading &&
-            advanced.firstRenderedId === previous.firstRenderedId &&
-            advanced.renderedArticles === previous.renderedArticles)
-            advanced.paginationBoundary = {
-                kind: "stable-rendered-tail",
-                lastId: advanced.renderedIds?.at(-1),
-                renderedCount: advanced.renderedArticles,
-                waitMs: 5000,
-                scopeComplete: false,
-            };
-        return advanced;
+        return {
+            ...advanced,
+            needsStateCheck: true,
+            missing: ["bounded-scroll-window-exhausted"],
+        };
     }
     async handleReadCreator(args) {
         if (typeof this.#browser.collectCreator !== "function")

package/dist/readers-posts.js

@@ -108,6 +108,7 @@ export function readPosts({ offset = 0, limit = 5, targetId = "", } = {}) {
             ?.getAttribute("href")
             ?.match(/\/status\/(\d+)/)?.[1])
             .filter(Boolean),
+        viewportScrollTop: document.scrollingElement?.scrollTop,
         atRenderedTail: Boolean(document.scrollingElement) &&
             document.scrollingElement.scrollTop +
                 document.scrollingElement.clientHeight >=

package/generated/observation.mjs

@@ -109,6 +109,7 @@ export const readXObservation = (query) => {
             ?.getAttribute("href")
             ?.match(/\/status\/(\d+)/)?.[1])
             .filter(Boolean),
+        viewportScrollTop: document.scrollingElement?.scrollTop,
         atRenderedTail: Boolean(document.scrollingElement) &&
             document.scrollingElement.scrollTop +
                 document.scrollingElement.clientHeight >=

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nextclaw/x-harness",
-  "version": "0.2.55",
+  "version": "0.2.56",
   "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": "7fc4a6a0e9bcea24ea2ac8d739572dc15afad136"
+  "gitHead": "84766d97c35a9cecbee5b8c6461aaa2e28d02c81"
 }