amosblomqvist
2d7b004fda
The pi-style ack protocol (previous commit) left the client describing the
old world: artifact cards read the ack string as a result ("could not be
displayed"), the only job poller rendered on input-available parts that no
longer exist (so wake turns landed invisibly until re-navigation), steers
rendered as learner bubbles with no SVG anywhere, and server-persisted
assistant messages carried empty SDK ids (duplicate React keys).
- persistTurn (turn-service): fill empty message ids at the canonical write
— server-only messages (last turn's, wake turns') now key uniquely;
client-authored ids untouched.
- chat.tsx: ONE central /api/jobs poll (fast while jobs run, slow discovery
cadence, off for job-less sessions) drives every job surface, re-syncs
once when a job settles (the record already holds the steer + wake
response at flip time), and adopts server-side wake turns via the same
turnInFlight machinery as returning mid-turn. Steers branch to the
artifact card — never a bubble. Composer no longer gates on jobs.
- /api/jobs now reports turnInFlight alongside the job list.
- dispatch-card: the tool-call box — live activity by toolCallId while
running, quiet "delivered"/"failed" one-liner after; the ack is visible
observability, the artifact lives in the steer.
- steer-card: renders the delivered artifact by kind (research brief /
sanitized SVG / sandboxed demo iframe) from the data-subagent-artifact
part; result renderers extracted and exported from the artifact cards.
- sub-agents-panel: right-sidebar section above the plan tree listing
running sub-agents (kind, elapsed, live activity) — vanishes when idle.
- tests: dispatch/steer/panel render tests re-target the new protocol;
persistTurn id-hygiene suite. 229/229 green, build green.
Out-of-lane touches for master review: chat.tsx + components/** (cleared
this wave), app/api/jobs/route.ts, src/server/turn-service.ts, contracts
protocol doc §2. Live-verified on :3003 (full loop, SSR DOM checked).
66 lines
2.4 KiB
TypeScript
66 lines
2.4 KiB
TypeScript
"use client";
|
|
|
|
import { DispatchCard } from "@/components/artifacts/dispatch-card";
|
|
import { MessageResponse } from "@/components/ai-elements/message";
|
|
import type { ArtifactPartProps, ResearchResult } from "@/src/core";
|
|
|
|
/**
|
|
* The research tool-call box (contracts.ts "THE ASYNC PROTOCOL" §4).
|
|
*
|
|
* The part's output is the fire-and-forget dispatch ACK — the box itself is
|
|
* the DispatchCard (live status via the central jobs poll; quiet one-liner
|
|
* once delivered). The research brief itself no longer renders from the
|
|
* tool part at all: it rides the steer message (the user-role delivery
|
|
* injected after the wake turn) and renders there via ResearchBriefCard.
|
|
*/
|
|
export function JobCard({ part }: ArtifactPartProps) {
|
|
if (part.state !== "output-available") return null;
|
|
return <DispatchCard toolCallId={part.toolCallId} kind="research" />;
|
|
}
|
|
|
|
/**
|
|
* The research brief — markdown body + a small source link list. Exported:
|
|
* the steer card (sub-agent result delivery) renders the same artifact.
|
|
*/
|
|
export function ResearchBriefCard({ output }: { output: unknown }) {
|
|
const result = output as ResearchResult | undefined;
|
|
if (!result || typeof result.brief !== "string") {
|
|
return (
|
|
<div className="rounded-xl border bg-card p-4 text-sm text-muted-foreground">
|
|
Research completed, but the result could not be displayed.
|
|
</div>
|
|
);
|
|
}
|
|
const failed = result.brief.startsWith("Research failed");
|
|
return (
|
|
<div className="rounded-xl border bg-card p-4 text-card-foreground" data-testid="research-brief-card">
|
|
<div className="mb-1 flex items-center gap-2 text-xs font-medium uppercase tracking-wide text-muted-foreground">
|
|
Research brief
|
|
{failed && <span className="rounded bg-red-500/15 px-1.5 py-0.5 text-red-500">failed</span>}
|
|
</div>
|
|
<div className="text-sm [&_p:first-child]:mt-0 [&_p:last-child]:mb-0">
|
|
<MessageResponse>{result.brief}</MessageResponse>
|
|
</div>
|
|
{result.sources.length > 0 && (
|
|
<div className="mt-3 border-t pt-2">
|
|
<div className="mb-1 text-xs font-medium uppercase tracking-wide text-muted-foreground">Sources</div>
|
|
<ul className="flex flex-col gap-0.5 text-xs">
|
|
{result.sources.map((s) => (
|
|
<li key={s.url} className="truncate">
|
|
<a
|
|
href={s.url}
|
|
target="_blank"
|
|
rel="noopener noreferrer"
|
|
className="text-primary underline-offset-2 hover:underline"
|
|
>
|
|
{s.title}
|
|
</a>
|
|
</li>
|
|
))}
|
|
</ul>
|
|
</div>
|
|
)}
|
|
</div>
|
|
);
|
|
}
|