fix(slack): render approval controls for every pending tool call

The HITL renderer only posted approve/deny buttons for the first pending tool call, so when a turn paused on multiple tool approvals the rest never got controls and the turn could not finish. It now renders a section plus an approve/deny pair for each pending call.

Author: ericallam

packages/slack/src/index.test.ts

@@ -69,6 +69,22 @@ describe("slack channel", () => {
     expect(text.length).toBeLessThan(3000);
   });
 
+  it("renderInteraction posts controls for every pending tool call, not just the first", () => {
+    const c = slack({ id: "s-hitl-multi", token: "t" });
+    const msg = c.renderInteraction?.(
+      [
+        { toolCallId: "call-a", toolName: "refund", input: { amount: 1 } },
+        { toolCallId: "call-b", toolName: "sendEmail", input: { to: "x" } },
+      ],
+      { event: messageEvent(), deliveryId: "d1" }
+    );
+    expect(msg).not.toBeNull();
+    const values = (msg!.blocks as any[]).flatMap((b) => b.elements ?? []).map((e: any) => e.value);
+    expect(values).toEqual(
+      expect.arrayContaining(["call-a::approve", "call-a::deny", "call-b::approve", "call-b::deny"])
+    );
+  });
+
   it("onInteraction resolves a block_actions click to a tool output; ignores messages", () => {
     const c = slack({ id: "s-hitl2", token: "t" });
     const approve = c.onInteraction?.({

packages/slack/src/index.ts

@@ -203,20 +203,19 @@ function defaultSlackRenderInteraction(
   pending: ChannelPendingToolCall[],
   _ctx: ChannelInteractionCtx<unknown>
 ): ChannelMessage | null {
-  const call = pending[0];
-  if (!call) return null;
-  let detail = "";
-  if (call.input !== undefined) {
-    const serialized = safeStringify(call.input);
-    const shown =
-      serialized.length > MAX_INTERACTION_INPUT_CHARS
-        ? serialized.slice(0, MAX_INTERACTION_INPUT_CHARS) + "\n... (truncated)"
-        : serialized;
-    detail = "\n```" + shown + "```";
-  }
-  return {
-    text: `Approval needed: ${call.toolName}`,
-    blocks: [
+  if (pending.length === 0) return null;
+
+  const blocks = pending.flatMap((call) => {
+    let detail = "";
+    if (call.input !== undefined) {
+      const serialized = safeStringify(call.input);
+      const shown =
+        serialized.length > MAX_INTERACTION_INPUT_CHARS
+          ? serialized.slice(0, MAX_INTERACTION_INPUT_CHARS) + "\n... (truncated)"
+          : serialized;
+      detail = "\n```" + shown + "```";
+    }
+    return [
       {
         type: "section",
         text: { type: "mrkdwn", text: `*Approval needed* for \`${call.toolName}\`${detail}` },
@@ -240,7 +239,15 @@ function defaultSlackRenderInteraction(
           },
         ],
       },
-    ],
+    ];
+  });
+
+  return {
+    text:
+      pending.length === 1
+        ? `Approval needed: ${pending[0]!.toolName}`
+        : `Approval needed: ${pending.length} tool calls`,
+    blocks,
   };
 }