@data-fair/lib-agents-sim 0.6.0 → 0.7.0

package/chat-driver.d.ts

@@ -35,11 +35,19 @@ export type TurnOutcome = 'ended' | 'waiting';
  * words interpolated into a translated string, so any text match would be both
  * locale-dependent and at the mercy of what the assistant wrote.
  */
+/**
+ * How long an armed wait must persist before the driver calls it the person's
+ * move. Long enough for a resolving wait's indicator to clear, short enough to be
+ * nothing against a wait a person is actually thinking through.
+ */
+export declare const WAIT_SETTLE_MS = 500;
 export declare const WAITING_SELECTOR = "[data-testid=\"chat-activity\"][data-activity=\"waiting\"]";
 export declare function createChatDriver(root: ChatRoot, opts?: {
     locale?: ChatDriverLocale;
 }): {
-    sendMessage(text: string): Promise<void>;
+    sendMessage(text: string, opts?: {
+        readyTimeoutMs?: number;
+    }): Promise<void>;
     waitForTurn(timeoutMs?: number): Promise<TurnOutcome>;
     readConversation(): Promise<{
         role: "user" | "assistant";

package/chat-driver.js

@@ -51,14 +51,29 @@ export const SEND_TIMEOUT_MS = 15000;
  * words interpolated into a translated string, so any text match would be both
  * locale-dependent and at the mercy of what the assistant wrote.
  */
+/**
+ * How long an armed wait must persist before the driver calls it the person's
+ * move. Long enough for a resolving wait's indicator to clear, short enough to be
+ * nothing against a wait a person is actually thinking through.
+ */
+export const WAIT_SETTLE_MS = 500;
 export const WAITING_SELECTOR = '[data-testid="chat-activity"][data-activity="waiting"]';
 export function createChatDriver(root, opts = {}) {
     const strings = chatDriverStrings(opts.locale ?? 'en');
     return {
-        async sendMessage(text) {
+        async sendMessage(text, opts = {}) {
             const fillAndSend = async () => {
                 await root.getByPlaceholder(strings.input).fill(text, { timeout: SEND_TIMEOUT_MS });
-                await root.getByRole('button', { name: strings.send }).click({ timeout: SEND_TIMEOUT_MS });
+                // Wait for the composer to be able to take it. While the assistant is
+                // genuinely working the send control IS the Stop button, so there is no
+                // Send to click — and a caller that tried anyway spent SEND_TIMEOUT_MS
+                // failing, pressed Escape, failed again, and left the text sitting in the
+                // box. A judged run lost six of its nine turns exactly so, and read as an
+                // assistant that had gone silent. Waiting for the turn is not a wedged
+                // page; it is the normal case, so it gets the caller's own ceiling.
+                const send = root.getByRole('button', { name: strings.send });
+                await send.waitFor({ state: 'visible', timeout: opts.readyTimeoutMs ?? SEND_TIMEOUT_MS });
+                await send.click({ timeout: SEND_TIMEOUT_MS });
             };
             try {
                 await fillAndSend();
@@ -95,7 +110,20 @@ export function createChatDriver(root, opts = {}) {
             // recorded as a wedged turn; at a wait window as long as the harness's own
             // ceiling, that is every run.
             const ended = expect(stop).toHaveCount(0, { timeout: timeoutMs }).then(() => 'ended');
-            const armed = expect(waiting).toHaveCount(1, { timeout: timeoutMs }).then(() => 'waiting', 
+            const armed = expect(waiting).toHaveCount(1, { timeout: timeoutMs }).then(
+            // Still armed a moment later, not merely armed at the instant we looked.
+            // A wait that the person has just resolved keeps its indicator for as long
+            // as the click takes to round-trip, and reporting THAT as "control is
+            // yours" hands the caller a turn that is already resuming underneath: the
+            // simulation loop then sends into a working turn, where the message used
+            // to be dropped in silence. Settling costs half a second on a real wait,
+            // which is a pause measured in minutes.
+            async () => {
+                await waiting.page().waitForTimeout(WAIT_SETTLE_MS);
+                if (await waiting.count() === 0)
+                    return await new Promise(() => { });
+                return 'waiting';
+            }, 
             // Never rejects: a wait that is simply not what this turn did must not be
             // the error a caller sees. The Stop arm owns the timeout message.
             () => new Promise(() => { }));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@data-fair/lib-agents-sim",
-  "version": "0.6.0",
+  "version": "0.7.0",
   "description": "Primitives for judged browser simulations of the data-fair agents chat, plus a Claude Code bridge exposing the Agent SDK as an OpenAI-compatible provider.",
   "main": "index.js",
   "type": "module",