mirror of
https://github.com/CopilotKit/CopilotKit.git
synced 2026-09-14 16:26:20 +08:00
f102d16c44
User-surfaced on production-Railway: gen-ui-interrupt and
interrupt-headless cells render nothing when pills are clicked —
only the agent's "[Scheduling...]" tool-call placeholder text shows.
hitl cell silently no-ops on the langgraph-interrupt path.
Three distinct breakages, same root family:
1. `gen-ui-interrupt/page.tsx` used `useInterrupt({ renderInChat })` —
a LangGraph-specific hook that listens for AG-UI `interrupt` events.
MAF has no `interrupt()` primitive; `interrupt_agent.py` emits a
regular `schedule_meeting` tool call instead. The hook never fires,
so the inline TimePickerCard never mounts. Replaced with
`useHumanInTheLoop({ name: "schedule_meeting" })` that listens for
the actual tool call — UX matches LGP, mechanism differs. Un-skipped
the two formerly-skipped tests (`picking a slot transitions to
picked`, `cancel path transitions to cancelled`); both now pass.
2. `interrupt-headless/page.tsx` mixed V1 `CopilotKit` provider with
V2 `useFrontendTool` hook (per GOTCHAS.md: "V1 + V2 mixing silently
fails — tool rendering pipeline never wires up"). The async handler
never ran, so the TimeSlotPopup in the app surface never opened.
Moved the `CopilotKit` import to V2.
3. `hitl/page.tsx` also mixed V1 and V2 imports for the same reason.
Switched fully to V2 and dropped the `useLangGraphInterrupt` block —
dead code on MAF (no interrupt events to listen for); the
coexisting `useHumanInTheLoop({ name: "generate_task_steps" })` is
the actual frontend handler.
`StepSelector` in hitl/page.tsx is now unused but retained — TypeScript
flags it as unused but doesn't fail; it's harmless and worth keeping
for parity if LangGraph interrupts ever get adapter-emulated. Cleanup
later.
129 lines
4.7 KiB
TypeScript
129 lines
4.7 KiB
TypeScript
import { test, expect } from "@playwright/test";
|
|
|
|
// QA reference: qa/gen-ui-interrupt.md
|
|
// Demo source: src/app/demos/gen-ui-interrupt/{page.tsx, time-picker-card.tsx}
|
|
//
|
|
// Uses `useInterrupt({ renderInChat: true })` — the low-level CopilotKit
|
|
// primitive wired to LangGraph's `interrupt()` on the `interrupt_agent`
|
|
// graph (shared with `interrupt-headless`). When the agent invokes the
|
|
// backend `schedule_meeting` tool, the graph interrupts and a
|
|
// `TimePickerCard` renders INLINE in the chat transcript (no portal).
|
|
//
|
|
// Card states (mutually exclusive, per-interrupt):
|
|
// - `time-picker-card` — initial, 4 slot buttons + "None of these work"
|
|
// - `time-picker-picked` — after a slot is clicked
|
|
// - `time-picker-cancelled` — after the ghost cancel button
|
|
//
|
|
// Typed prompts (not suggestion pills) are used for the tool-trigger flows:
|
|
// pill-click was observed to not always drive schedule_meeting on Railway.
|
|
// No LLM-text assertions — only testid state transitions plus the inline
|
|
// (non-body) render contract.
|
|
|
|
test.describe("Gen UI via useInterrupt (inline time picker)", () => {
|
|
test.setTimeout(120_000);
|
|
|
|
test.beforeEach(async ({ page }) => {
|
|
await page.goto("/demos/gen-ui-interrupt");
|
|
});
|
|
|
|
test("page loads with chat input and no picker rendered", async ({
|
|
page,
|
|
}) => {
|
|
await expect(page.getByPlaceholder("Type a message")).toBeVisible();
|
|
await expect(page.locator('[data-testid="time-picker-card"]')).toHaveCount(
|
|
0,
|
|
);
|
|
});
|
|
|
|
test("both suggestion pills render", async ({ page }) => {
|
|
const suggestions = page.locator('[data-testid="copilot-suggestion"]');
|
|
await expect(
|
|
suggestions.filter({ hasText: "Book a call with sales" }).first(),
|
|
).toBeVisible({ timeout: 15_000 });
|
|
await expect(
|
|
suggestions.filter({ hasText: "Schedule a 1:1 with Alice" }).first(),
|
|
).toBeVisible({ timeout: 15_000 });
|
|
});
|
|
|
|
// The MS Agent Framework port renders the picker via
|
|
// `useHumanInTheLoop({ name: "schedule_meeting" })` because MAF has no
|
|
// `interrupt()` primitive — the agent emits a regular tool call instead.
|
|
// Skip rationale from the LGP version (Railway flakiness on the
|
|
// langgraph-interrupt path) does not apply here; the deterministic aimock
|
|
// tool-call fixture makes the picker land reliably.
|
|
test("picking a slot transitions the card to the picked state", async ({
|
|
page,
|
|
}) => {
|
|
const input = page.getByPlaceholder("Type a message");
|
|
await input.fill(
|
|
"Use schedule_meeting to book an intro call with the sales team about pricing.",
|
|
);
|
|
await page.locator('[data-testid="copilot-send-button"]').first().click();
|
|
|
|
const card = page.locator('[data-testid="time-picker-card"]').first();
|
|
await expect(card).toBeVisible({ timeout: 60_000 });
|
|
|
|
// Contract: inline render, NOT a body portal (unlike hitl-in-app).
|
|
await expect(
|
|
page.locator('body > [data-testid="time-picker-card"]'),
|
|
).toHaveCount(0);
|
|
|
|
const expectedSlots = [
|
|
"Tomorrow 10:00 AM",
|
|
"Tomorrow 2:00 PM",
|
|
"Monday 9:00 AM",
|
|
"Monday 3:30 PM",
|
|
];
|
|
for (const label of expectedSlots) {
|
|
await expect(card.getByRole("button", { name: label })).toBeVisible();
|
|
}
|
|
await expect(
|
|
card.getByRole("button", { name: "None of these work" }),
|
|
).toBeVisible();
|
|
|
|
await card.getByRole("button", { name: "Monday 9:00 AM" }).click();
|
|
|
|
const picked = page.locator('[data-testid="time-picker-picked"]').first();
|
|
await expect(picked).toBeVisible({ timeout: 10_000 });
|
|
await expect(picked).toContainText("Monday 9:00 AM");
|
|
|
|
// The picked-state card replaces the interactive card.
|
|
await expect(page.locator('[data-testid="time-picker-card"]')).toHaveCount(
|
|
0,
|
|
);
|
|
|
|
await expect(
|
|
page.locator('[data-testid="copilot-assistant-message"]').first(),
|
|
).toBeVisible({
|
|
timeout: 45_000,
|
|
});
|
|
});
|
|
|
|
test("cancel path: None-of-these-work transitions to cancelled state", async ({
|
|
page,
|
|
}) => {
|
|
const input = page.getByPlaceholder("Type a message");
|
|
await input.fill(
|
|
"Use schedule_meeting to book a 1:1 with Alice next week to review Q2 goals.",
|
|
);
|
|
await page.locator('[data-testid="copilot-send-button"]').first().click();
|
|
|
|
const card = page.locator('[data-testid="time-picker-card"]').first();
|
|
await expect(card).toBeVisible({ timeout: 60_000 });
|
|
|
|
await card.getByRole("button", { name: "None of these work" }).click();
|
|
|
|
const cancelled = page
|
|
.locator('[data-testid="time-picker-cancelled"]')
|
|
.first();
|
|
await expect(cancelled).toBeVisible({ timeout: 10_000 });
|
|
await expect(cancelled).toContainText("Cancelled");
|
|
|
|
await expect(
|
|
page.locator('[data-testid="copilot-assistant-message"]').first(),
|
|
).toBeVisible({
|
|
timeout: 45_000,
|
|
});
|
|
});
|
|
});
|