@poe-platform/safe-bash 0.1.784 → 0.1.786

package/README.md

@@ -640,7 +640,8 @@ Browser snapshots have no byte limit, including automatic snapshots after
 navigation. Legacy `maxSnapshotBytes` settings are ignored.
 Pass `onSnapshot` to `createPlaywrightCli` or `createPlaywrightController` to inspect
 navigation response headers and call `recapture()` before publishing YAML or JSON.
-Page URL and title reflect changes made by the hook. The `/playwright` entrypoint
+Page URL, title and console events reflect changes made by the hook. Recaptures
+re-resolve target selectors and finish before the session accepts another command. The `/playwright` entrypoint
 also exports `getPlaywrightMainFrameNavigation(context, page)` for passive metadata
 reads. See the [snapshot hook contract](src/contracts/playwright-snapshot.md).
 `playwright-cli --help` and `playwright-cli show --help` report dashboard

package/dist/safe-bash/playwright/capability-events.js

@@ -34,8 +34,18 @@ export function observePlaywrightCapabilities(context, registerCleanup, limits)
         history.set(record, { bytes: size, evict });
         if (size > state.maxBytes)
             forget(record);
-        while (history.size > (limits.maxEventEntries ?? Infinity) || bytes > state.maxBytes)
-            forget(history.keys().next().value);
+        while (history.size > (limits.maxEventEntries ?? Infinity) || bytes > state.maxBytes) {
+            // Preserve active navigation metadata ahead of rolling diagnostics, while
+            // still applying the configured byte and entry budgets to every record.
+            let diagnostic;
+            for (const record of history.keys()) {
+                if (!('mainFrameNavigation' in record) || !record.mainFrameNavigation) {
+                    diagnostic = record;
+                    break;
+                }
+            }
+            forget(diagnostic ?? history.keys().next().value);
+        }
     };
     const byteLength = (text) => new TextEncoder().encode(text).byteLength;
     const pageEvents = (page) => {

package/dist/safe-bash/playwright/controller.js

@@ -969,12 +969,7 @@ export function createPlaywrightController(options = {}) {
                 return { sections };
             }
             sections.push(...await downloadSections(session));
-            if (invocation.writeArtifact) {
-                const consoleLink = await flushPlaywrightConsole(session.lease.context, page, { ...(session.configuration ? { configuration: session.configuration } : {}), writeArtifact });
-                if (consoleLink)
-                    sections.push({ title: 'Events', content: `- New console entries: ${consoleLink}` });
-            }
-            const pageSectionIndex = sections.length;
+            let pageSectionIndex = sections.length;
             if (snapshot !== 'none' && (page.frames || page.ariaSnapshot || page._snapshotForAI || page.ariaSnapshotJSON || session.lease?.captureSnapshotJSON)) {
                 const snapshotOptions = { timeout: sessionSnapshotTimeout(session), ...(session.configuration?.snapshot?.boxes === undefined ? {} : { boxes: session.configuration.snapshot.boxes }), ...(session.lease?.captureSnapshotReferences ? { captureReferences: session.lease.captureSnapshotReferences } : {}) };
                 if (parsed.command === 'snapshot') {
@@ -986,18 +981,24 @@ export function createPlaywrightController(options = {}) {
                     }
                     if (parsed.options.boxes)
                         snapshotOptions.boxes = true;
-                    if (parsed.args[0])
-                        snapshotOptions.root = await resolveTarget(session, parsed.args[0]);
                 }
                 const captureWithHook = async (format, capture) => {
                     let active = true;
-                    const recapture = async () => {
+                    let pending = Promise.resolve();
+                    const recapture = () => {
                         if (!active)
-                            throw new Error('Playwright snapshot hook has completed');
-                        check();
-                        const value = await capture();
-                        check();
-                        return value;
+                            return Promise.reject(new Error('Playwright snapshot hook has completed'));
+                        const operation = pending.then(async () => {
+                            check();
+                            if (parsed.command === 'snapshot' && parsed.args[0])
+                                snapshotOptions.root = await resolveTarget(session, parsed.args[0]);
+                            const value = await capture();
+                            check();
+                            return value;
+                        });
+                        // Observe failures immediately, and serialize all admitted captures.
+                        pending = operation.then(() => { }, () => { });
+                        return operation;
                     };
                     try {
                         const initial = await recapture();
@@ -1011,6 +1012,7 @@ export function createPlaywrightController(options = {}) {
                     }
                     finally {
                         active = false;
+                        await pending;
                     }
                 };
                 retained = false;
@@ -1019,24 +1021,30 @@ export function createPlaywrightController(options = {}) {
                     check();
                     retained = parsed.command === 'snapshot';
                     sections.push({ title: 'Snapshot', content: { json: tree }, codeframe: 'json' });
-                    return { sections };
                 }
-                retained = false;
-                const text = await captureWithHook('yaml', () => session.snapshot.capture(page, local.signal, snapshotOptions));
-                check();
-                retained = parsed.command === 'snapshot';
-                if (filename !== undefined || snapshot === 'file' && invocation.writeArtifact) {
-                    const target = filename ?? capabilityArtifactName('page', 'yml', session.configuration);
-                    const bytes = new TextEncoder().encode(text);
-                    if (bytes.byteLength > maxArtifactBytes)
-                        throw new PlaywrightResourceLimitError('Artifact byte limit exceeded');
-                    if (!invocation.writeArtifact)
-                        throw new Error('Artifact byte destination unsupported');
-                    await writeArtifact(bytes, target);
-                    sections.push({ title: 'Snapshot', content: `- [Snapshot](${target})` });
+                else {
+                    retained = false;
+                    const text = await captureWithHook('yaml', () => session.snapshot.capture(page, local.signal, snapshotOptions));
+                    check();
+                    retained = parsed.command === 'snapshot';
+                    if (filename !== undefined || snapshot === 'file' && invocation.writeArtifact) {
+                        const target = filename ?? capabilityArtifactName('page', 'yml', session.configuration);
+                        const bytes = new TextEncoder().encode(text);
+                        if (bytes.byteLength > maxArtifactBytes)
+                            throw new PlaywrightResourceLimitError('Artifact byte limit exceeded');
+                        if (!invocation.writeArtifact)
+                            throw new Error('Artifact byte destination unsupported');
+                        await writeArtifact(bytes, target);
+                        sections.push({ title: 'Snapshot', content: `- [Snapshot](${target})` });
+                    }
+                    else
+                        sections.push({ title: 'Snapshot', content: text.trimEnd(), codeframe: 'yaml' });
                 }
-                else
-                    sections.push({ title: 'Snapshot', content: text.trimEnd(), codeframe: 'yaml' });
+            }
+            if (invocation.writeArtifact) {
+                const consoleLink = await flushPlaywrightConsole(session.lease.context, page, { ...(session.configuration ? { configuration: session.configuration } : {}), writeArtifact });
+                if (consoleLink)
+                    sections.splice(pageSectionIndex++, 0, { title: 'Events', content: `- New console entries: ${consoleLink}` });
             }
             if (parsed.command !== 'snapshot' || !parsed.json) {
                 const title = await page.title?.();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@poe-platform/safe-bash",
-  "version": "0.1.784",
+  "version": "0.1.786",
   "description": "Extensible virtual shell with pluggable filesystems and streaming commands",
   "type": "module",
   "license": "MIT",
@@ -841,7 +841,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@poe-platform/safe-fs": "0.1.784",
+    "@poe-platform/safe-fs": "0.1.786",
     "pdf-lib": "1.17.1",
     "@noble/hashes": "2.4.0",
     "pako": "3.0.1",