Make session_todos_changed deterministic via single sql call

The test asked the model to run three FK-dependent INSERTs. The model issued
them as three parallel sql tool calls; their execution order is not
deterministic, so on some platforms the todo_deps INSERT raced ahead of its
parent todos rows and hit 'FOREIGN KEY constraint failed', diverging from the
recorded snapshot (cache miss on macOS CI).

Reword the prompt to ask for a single sql tool call containing all three
statements in order. Statements within one call run sequentially, so the FK
constraint is always satisfied and the snapshot replays deterministically on
all platforms. Re-recorded the snapshot accordingly. Verified stable across
3 replay runs.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: stephentoub

nodejs/test/e2e/session_todos_changed.e2e.test.ts

@@ -33,7 +33,9 @@ describe("Todos changed event + readSqlTodosWithDependencies", async () => {
 
             await session.sendAndWait({
                 prompt:
-                    "Use the sql tool to execute exactly these statements, in order, with no extra rows:\n" +
+                    "Use the sql tool exactly once to execute all three of the following statements " +
+                    "together, in this exact order, in a single sql tool call (a single query string " +
+                    "containing all three statements):\n" +
                     "1. INSERT INTO todos (id, title, status) VALUES ('alpha', 'First todo', 'pending');\n" +
                     "2. INSERT INTO todos (id, title, status) VALUES ('beta', 'Second todo', 'done');\n" +
                     "3. INSERT INTO todo_deps (todo_id, depends_on) VALUES ('beta', 'alpha');\n" +

test/snapshots/session_todos_changed/fires_session_todos_changed_and_exposes_rows_and_dependencies.yaml

@@ -5,77 +5,65 @@ conversations:
       - role: system
         content: ${system}
       - role: user
-        content: |-
-          Use the sql tool to execute exactly these statements, in order, with no extra rows:
+        content: >-
+          Use the sql tool exactly once to execute all three of the following statements together, in this exact order,
+          in a single sql tool call (a single query string containing all three statements):
+
           1. INSERT INTO todos (id, title, status) VALUES ('alpha', 'First todo', 'pending');
+
           2. INSERT INTO todos (id, title, status) VALUES ('beta', 'Second todo', 'done');
+
           3. INSERT INTO todo_deps (todo_id, depends_on) VALUES ('beta', 'alpha');
+
           Then stop. Do not insert any other rows or create any other tables.
       - role: assistant
-        content: I'll execute those three SQL statements in order.
+        content: "I'll execute all three SQL statements together in a single call:"
       - role: assistant
         tool_calls:
           - id: toolcall_0
             type: function
             function:
               name: sql
-              arguments: "{\"description\":\"Insert alpha todo\",\"query\":\"INSERT INTO todos (id, title, status) VALUES ('alpha',
-                'First todo', 'pending');\"}"
-      - role: assistant
-        tool_calls:
-          - id: toolcall_1
-            type: function
-            function:
-              name: sql
-              arguments: "{\"description\":\"Insert beta todo\",\"query\":\"INSERT INTO todos (id, title, status) VALUES ('beta',
-                'Second todo', 'done');\"}"
-      - role: assistant
-        tool_calls:
-          - id: toolcall_2
-            type: function
-            function:
-              name: sql
-              arguments: "{\"description\":\"Insert dependency\",\"query\":\"INSERT INTO todo_deps (todo_id, depends_on) VALUES
-                ('beta', 'alpha');\"}"
+              arguments: "{\"description\":\"Insert todos and dependency\",\"query\":\"INSERT INTO todos (id, title, status) VALUES
+                ('alpha', 'First todo', 'pending');\\nINSERT INTO todos (id, title, status) VALUES ('beta', 'Second
+                todo', 'done');\\nINSERT INTO todo_deps (todo_id, depends_on) VALUES ('beta', 'alpha');\"}"
   - messages:
       - role: system
         content: ${system}
       - role: user
-        content: |-
-          Use the sql tool to execute exactly these statements, in order, with no extra rows:
+        content: >-
+          Use the sql tool exactly once to execute all three of the following statements together, in this exact order,
+          in a single sql tool call (a single query string containing all three statements):
+
           1. INSERT INTO todos (id, title, status) VALUES ('alpha', 'First todo', 'pending');
+
           2. INSERT INTO todos (id, title, status) VALUES ('beta', 'Second todo', 'done');
+
           3. INSERT INTO todo_deps (todo_id, depends_on) VALUES ('beta', 'alpha');
+
           Then stop. Do not insert any other rows or create any other tables.
       - role: assistant
-        content: I'll execute those three SQL statements in order.
+        content: "I'll execute all three SQL statements together in a single call:"
         tool_calls:
           - id: toolcall_0
             type: function
             function:
               name: sql
-              arguments: "{\"description\":\"Insert alpha todo\",\"query\":\"INSERT INTO todos (id, title, status) VALUES ('alpha',
-                'First todo', 'pending');\"}"
-          - id: toolcall_1
-            type: function
-            function:
-              name: sql
-              arguments: "{\"description\":\"Insert beta todo\",\"query\":\"INSERT INTO todos (id, title, status) VALUES ('beta',
-                'Second todo', 'done');\"}"
-          - id: toolcall_2
-            type: function
-            function:
-              name: sql
-              arguments: "{\"description\":\"Insert dependency\",\"query\":\"INSERT INTO todo_deps (todo_id, depends_on) VALUES
-                ('beta', 'alpha');\"}"
+              arguments: "{\"description\":\"Insert todos and dependency\",\"query\":\"INSERT INTO todos (id, title, status) VALUES
+                ('alpha', 'First todo', 'pending');\\nINSERT INTO todos (id, title, status) VALUES ('beta', 'Second
+                todo', 'done');\\nINSERT INTO todo_deps (todo_id, depends_on) VALUES ('beta', 'alpha');\"}"
       - role: tool
         tool_call_id: toolcall_0
-        content: "1 row(s) inserted. Last inserted row ID: 1."
-      - role: tool
-        tool_call_id: toolcall_1
-        content: "1 row(s) inserted. Last inserted row ID: 2."
-      - role: tool
-        tool_call_id: toolcall_2
-        content: "1 row(s) inserted. Last inserted row ID: 1."
+        content: |-
+          Executed 3 statement(s):
+
+          1. INSERT
+             1 row(s) inserted. Last inserted row ID: 1.
+
+          2. INSERT
+             1 row(s) inserted. Last inserted row ID: 2.
+
+          3. INSERT
+             1 row(s) inserted. Last inserted row ID: 1.
       - role: assistant
-        content: Done. All three statements executed successfully.
+        content: Done. All three statements executed successfully in a single SQL call.