mirror of
https://github.com/bytedance/deer-flow.git
synced 2026-08-07 05:18:53 +00:00
* docs: design frontend performance remediation * docs: plan frontend performance remediation * test(frontend): add route asset performance budgets * perf(nginx): compress textual responses safely * perf(frontend): lazy load case study media * perf(frontend): bound static demo file tracing * perf(frontend): restore static locale boundaries * perf(frontend): defer closed workspace panels * perf(frontend): split editors and deduplicate highlighting * perf(frontend): index incremental message derivation * perf(frontend): stabilize paged history cache policy * perf(frontend): bound streaming markdown renders * perf(frontend): virtualize message history * perf(frontend): bound and virtualize chat lists * perf(frontend): suspend inactive decorative animation * perf(browser): stream latest frames as binary * perf(artifacts): stream bounded text previews * docs: finalize performance runtime boundaries * style(backend): apply test formatting * fix(frontend): keep translation functions client-side * perf(frontend): defer decorative animation bundles * test(frontend): lock optimized route budgets * fix: harden frontend performance boundaries * test(frontend): update i18n provider fixture * fix(frontend): preserve sidebar pagination position * style(backend): format artifact range test
250 lines
7.3 KiB
TypeScript
250 lines
7.3 KiB
TypeScript
import { describe, expect, it } from "@rstest/core";
|
|
import { createElement, type KeyboardEvent } from "react";
|
|
import { renderToStaticMarkup } from "react-dom/server";
|
|
|
|
import {
|
|
findMissingRequiredFields,
|
|
HumanInputCard,
|
|
shouldSubmitHumanInputTextOnKeyDown,
|
|
} from "@/components/workspace/messages/human-input-card";
|
|
import { I18nContext } from "@/core/i18n/context";
|
|
import { enUS } from "@/core/i18n/locales/en-US";
|
|
import type {
|
|
HumanInputRequest,
|
|
HumanInputResponse,
|
|
} from "@/core/messages/human-input";
|
|
|
|
const request: HumanInputRequest = {
|
|
version: 1,
|
|
kind: "human_input_request",
|
|
source: "ask_clarification",
|
|
request_id: "clarification:call-abc",
|
|
tool_call_id: "call-abc",
|
|
clarification_type: "approach_choice",
|
|
question: "Which environment should I deploy to?",
|
|
context: "Need the target environment.",
|
|
input_mode: "choice_with_other",
|
|
options: [
|
|
{ id: "option-1", label: "development", value: "development" },
|
|
{ id: "option-2", label: "staging", value: "staging" },
|
|
],
|
|
};
|
|
|
|
describe("HumanInputCard", () => {
|
|
it("renders request text, options, and the other-answer input", () => {
|
|
const html = renderCard();
|
|
|
|
expect(html).toContain("Need your help");
|
|
expect(html).toContain("Need the target environment.");
|
|
expect(html).toContain("Which environment should I deploy to?");
|
|
expect(html).toContain("development");
|
|
expect(html).toContain("staging");
|
|
expect(html).toContain("Other answer");
|
|
expect(html).toContain("Type another answer...");
|
|
});
|
|
|
|
it("renders answered state as disabled with the selected value", () => {
|
|
const response: HumanInputResponse = {
|
|
version: 1,
|
|
kind: "human_input_response",
|
|
source: "ask_clarification",
|
|
request_id: "clarification:call-abc",
|
|
response_kind: "option",
|
|
option_id: "option-2",
|
|
value: "staging",
|
|
};
|
|
const html = renderCard({ answeredResponse: response });
|
|
|
|
expect(html).toContain("Answered");
|
|
expect(html).toContain("Answered: staging");
|
|
expect(html).toContain("disabled");
|
|
});
|
|
|
|
it("renders read-only state when no submit handler is available", () => {
|
|
const html = renderCard({ onSubmit: undefined });
|
|
|
|
expect(html).toContain("Read only");
|
|
expect(html).toContain("disabled");
|
|
});
|
|
|
|
it("renders markdown in question field (bold, lists)", () => {
|
|
const html = renderCard({
|
|
request: {
|
|
...request,
|
|
question:
|
|
"你想写什么样的小说?\n\n1. **题材/类型**:科幻、奇幻\n2. **篇幅**:短篇、中篇",
|
|
input_mode: "free_text",
|
|
options: undefined,
|
|
},
|
|
});
|
|
|
|
expect(html).toContain("题材/类型");
|
|
expect(html).toContain("篇幅");
|
|
expect(html).not.toContain("**题材/类型**");
|
|
expect(html).not.toContain("**篇幅**");
|
|
});
|
|
|
|
it("renders a form request with labeled fields and required markers", () => {
|
|
const html = renderCard({
|
|
request: {
|
|
...request,
|
|
version: 2,
|
|
request_id: "clarification:call-form",
|
|
question: "Please provide the expense details.",
|
|
input_mode: "form",
|
|
options: undefined,
|
|
fields: [
|
|
{ name: "amount", label: "Amount", type: "number", required: true },
|
|
{
|
|
name: "receipts",
|
|
label: "Receipts",
|
|
type: "multi_select",
|
|
required: false,
|
|
options: [
|
|
{ id: "receipts-option-1", label: "A-1", value: "A-1" },
|
|
{ id: "receipts-option-2", label: "A-2", value: "A-2" },
|
|
],
|
|
},
|
|
{ name: "note", label: "Note", type: "textarea", required: false },
|
|
{
|
|
name: "design",
|
|
label: "DesignPriority",
|
|
type: "checkbox",
|
|
required: false,
|
|
},
|
|
],
|
|
},
|
|
});
|
|
|
|
expect(html).toContain("Amount");
|
|
expect(html).toContain("*");
|
|
expect(html).toContain("Receipts");
|
|
expect(html).toContain("A-1");
|
|
expect(html).toContain("Note");
|
|
expect(html).toContain("Submit");
|
|
// Checkbox fields render as a single toggle row — the label must not be
|
|
// duplicated by an additional field label above the control.
|
|
expect(html.split("DesignPriority").length - 1).toBe(1);
|
|
});
|
|
|
|
it("associates form labels with controls and marks required fields", () => {
|
|
const html = renderCard({
|
|
request: {
|
|
...request,
|
|
version: 2,
|
|
request_id: "clarification:call-form",
|
|
question: "Please provide the expense details.",
|
|
input_mode: "form",
|
|
options: undefined,
|
|
fields: [
|
|
{ name: "amount", label: "Amount", type: "number", required: true },
|
|
{ name: "note", label: "Note", type: "textarea", required: false },
|
|
],
|
|
},
|
|
});
|
|
|
|
// Every visible label is linked to its control via htmlFor/id.
|
|
const htmlForIds = [...html.matchAll(/<label[^>]*for="([^"]+)"/g)].map(
|
|
(match) => match[1],
|
|
);
|
|
expect(htmlForIds.length).toBe(2);
|
|
for (const id of htmlForIds) {
|
|
expect(html).toContain(`id="${id}"`);
|
|
}
|
|
expect(html).toContain('aria-required="true"');
|
|
});
|
|
|
|
it("findMissingRequiredFields flags empty required values only", () => {
|
|
const fields = [
|
|
{
|
|
name: "amount",
|
|
label: "Amount",
|
|
type: "number" as const,
|
|
required: true,
|
|
},
|
|
{
|
|
name: "note",
|
|
label: "Note",
|
|
type: "text" as const,
|
|
required: false,
|
|
},
|
|
{
|
|
name: "receipts",
|
|
label: "Receipts",
|
|
type: "multi_select" as const,
|
|
required: true,
|
|
options: [{ id: "o1", label: "A-1", value: "A-1" }],
|
|
},
|
|
];
|
|
|
|
expect(
|
|
findMissingRequiredFields(fields, {}).map((field) => field.name),
|
|
).toEqual(["amount", "receipts"]);
|
|
expect(
|
|
findMissingRequiredFields(fields, {
|
|
amount: " ",
|
|
receipts: [],
|
|
}).map((field) => field.name),
|
|
).toEqual(["amount", "receipts"]);
|
|
expect(
|
|
findMissingRequiredFields(fields, {
|
|
amount: "300",
|
|
receipts: ["A-1"],
|
|
}),
|
|
).toEqual([]);
|
|
});
|
|
|
|
it("does not submit text with Enter while IME composition is active", () => {
|
|
expect(shouldSubmitHumanInputTextOnKeyDown(keyEvent())).toBe(true);
|
|
expect(
|
|
shouldSubmitHumanInputTextOnKeyDown(keyEvent({ shiftKey: true })),
|
|
).toBe(false);
|
|
expect(
|
|
shouldSubmitHumanInputTextOnKeyDown(keyEvent({ isComposing: true })),
|
|
).toBe(false);
|
|
expect(
|
|
shouldSubmitHumanInputTextOnKeyDown(keyEvent({ keyCode: 229 })),
|
|
).toBe(false);
|
|
expect(shouldSubmitHumanInputTextOnKeyDown(keyEvent(), true)).toBe(false);
|
|
});
|
|
});
|
|
|
|
function renderCard(props: Partial<Parameters<typeof HumanInputCard>[0]> = {}) {
|
|
return renderToStaticMarkup(
|
|
createElement(
|
|
I18nContext.Provider,
|
|
{
|
|
value: {
|
|
locale: "en-US",
|
|
setLocale: () => undefined,
|
|
t: enUS,
|
|
},
|
|
},
|
|
createElement(HumanInputCard, {
|
|
request,
|
|
onSubmit: () => undefined,
|
|
...props,
|
|
}),
|
|
),
|
|
);
|
|
}
|
|
|
|
function keyEvent({
|
|
isComposing = false,
|
|
key = "Enter",
|
|
keyCode = 13,
|
|
shiftKey = false,
|
|
}: {
|
|
isComposing?: boolean;
|
|
key?: string;
|
|
keyCode?: number;
|
|
shiftKey?: boolean;
|
|
} = {}) {
|
|
return {
|
|
key,
|
|
keyCode,
|
|
nativeEvent: { isComposing },
|
|
shiftKey,
|
|
} as unknown as KeyboardEvent<HTMLTextAreaElement>;
|
|
}
|