mirror of
https://github.com/CopilotKit/CopilotKit.git
synced 2026-09-14 16:26:20 +08:00
Ran Shem Tov
9a12dcc345
Cancel handling was asymmetric in both languages: the TypeScript-shaped
sentinel was only recognised on the outer envelope, so a cancel that arrived
wrapped as {"response": {"status": "cancelled"}} was reported to the model as
"did not pick a time". Both tools now check the sentinel at both levels, and
the test matrix covers the shape it missed (three cases fail without the fix).
The TypeScript tool also coalesced the label with ?? where Python uses or, so
an empty label swallowed a valid time; both now fall back the same way.
Frontend and probes:
- The headless resume waited on a single animation frame, which never fires in
a background tab, stranding the run. A timer races it and the resume fires
once, whichever lands first.
- The picker latches after the first answer, so a second click cannot race the
resume already in flight.
- A rejected resume now replaces the green "Booked" badge with a failure state
instead of leaving a success the user never got.
- The headless probe searched the whole page for booking words, which the
demo's own static copy already satisfies; it now reads assistant bubbles only
and matches phrases the page does not contain.
- The pick-path spec asserted a bubble was visible, which the pre-pause bubble
already satisfies; it now asserts the post-resume narration. The headless
spec scopes the picker to the app-surface pane, which is the contract it
claims to hold.
Prose and fixtures:
- The dedicated-mount rationale named the wrong tool: the in-chat demo
registers book_call, and the real collision is the shared agent's own
immediate schedule_meeting. Corrected in the parity notes and both tools.
- The TypeScript integration no longer cites Python API names, and the picker
no longer cites a slot generator that does not exist.
- The shared docs no longer describe a hook the embedded snippet does not
show, no longer place a standard interrupt's payload on event.value, and
carry the correct RUN_FINISHED outcome shape.
- The d4 fixture edits are back to UTF-8 literals, dropping 86 lines of
re-encoding noise so only the six intended key changes remain.
- The agent package exposes a test script, so the new unit test is runnable.