From 87608fb7e6d8a86b6b6e12c763b05cf998d9e009 Mon Sep 17 00:00:00 2001
From: "github-actions[bot]"
 <41898282+github-actions[bot]@users.noreply.github.com>
Date: Tue, 25 Aug 2026 08:06:16 +0000
Subject: [PATCH] test(workflow): add call.text and log parity tests in
 dynamic-workflow describe block

Add two tests to the 'dynamic-workflow parity' describe block that were
missing coverage for the two most fundamental Claude dynamic-workflow
primitives:

- call.text: validates that the rig equivalent of Claude's plain
  `await agent(prompt)` returns a string, matching the conversion-table
  row that's otherwise only covered in the 'workflow one-off agents' block.

- log: validates that `log(message)` emits a { type: 'log' } event in the
  onEvent stream, so the mapping is exercised in the parity section where
  a reader converting Claude code will look for it.

Both tests use the same fakeAgent / configureAgent pattern as the
surrounding block for consistency.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
---
 src/workflow.test.ts | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/src/workflow.test.ts b/src/workflow.test.ts
index 1138600..3a14bbf 100644
--- a/src/workflow.test.ts
+++ b/src/workflow.test.ts
@@ -273,6 +273,36 @@ describe("dynamic-workflow parity", () => {
     expect(warnings[0]).toMatchObject({ type: "warning", message: expect.stringContaining("2") });
   });
 
+  it("call.text returns a string — maps to await agent(prompt) without a schema", async () => {
+    // The most common Claude dynamic-workflow call: `await agent(prompt)` → `await call.text(prompt)`.
+    // Returns string | null; null signals an agent failure, not an empty string.
+    configureAgent(() => ({
+      ask: async () => '"pong"',
+      close: async () => {},
+    }));
+
+    const definition = workflow({
+      meta: { name: "call-text", description: "call.text returns string" },
+      body: ({ call }) => call.text("Reply with one word: pong."),
+    });
+
+    await expect(runWorkflow(definition)).resolves.toBe("pong");
+  });
+
+  it("log emits a log event visible in the onEvent stream", async () => {
+    // Claude dynamic workflows emit log(message); rig surfaces it as a { type: "log" } event.
+    const events: WorkflowEvent[] = [];
+    const definition = workflow({
+      meta: { name: "log-event", description: "log emits event" },
+      body: ({ log: wfLog }) => { wfLog("scanning repository"); },
+    });
+
+    await runWorkflow(definition, { onEvent: (event) => events.push(event) });
+    const logEvents = events.filter((event) => event.type === "log");
+    expect(logEvents).toHaveLength(1);
+    expect(logEvents[0]).toMatchObject({ type: "log", message: "scanning repository" });
+  });
+
   it("call.json accepts a non-object schema (s.enum) — rig advantage over Claude dynamic workflows", async () => {
     // Claude dynamic workflows only support object schemas in agent(prompt, { schema }).
     // rig's call.json accepts any s.* schema: s.enum, s.array, s.string, etc.