xinyin025/LangBot
1
0
Fork 0
mirror of https://github.com/langbot-app/LangBot.git synced 2026-07-22 20:36:08 +00:00
Code Issues Packages Projects Releases Wiki Activity

feat(agent-platform): productize bot event route testing

Browse Source
This commit is contained in:
huanghuoguoguo
2026-07-11 10:44:50 +08:00
parent 4cbb9415bf
commit 00fb6f8236
17 changed files with 1313 additions and 376 deletions
Download Patch File Download Diff File Expand all files Collapse all files
web/src/app/home/bots/components/bot-form/BotForm.tsx
+23 -21
View File
@@ -381,6 +381,8 @@ export default function BotForm({
description: form.getValues().description ?? '',
adapter: form.getValues().adapter,
adapter_config: form.getValues().adapter_config,
enable: form.getValues().enable,
event_bindings: form.getValues().event_bindings ?? [],
};
httpClient
.createBot(newBot)
@@ -448,27 +450,7 @@ export default function BotForm({
</CardContent>
</Card>
{/* Card 2: Event Routing (edit mode only) */}
{initBotId && (
<Card>
<CardHeader>
<CardTitle>{t('bots.eventRouting')}</CardTitle>
<CardDescription>
{t('bots.eventRoutingDescription')}
</CardDescription>
</CardHeader>
<CardContent>
<EventBindingsEditor
form={form}
botId={initBotId}
supportedEvents={adapterSupportedEvents[currentAdapter] || []}
agentOptions={agentNameList}
/>
</CardContent>
</Card>
)}
{/* Card 4: Adapter Configuration */}
{/* Card 2: Adapter Configuration */}
<Card>
<CardHeader>
<CardTitle>{t('bots.adapterConfig')}</CardTitle>
@@ -672,6 +654,26 @@ export default function BotForm({
)}
</CardContent>
</Card>
{/* Card 3: Event Routing */}
{currentAdapter && (
<Card>
<CardHeader>
<CardTitle>{t('bots.eventRouting')}</CardTitle>
<CardDescription>
{t('bots.eventRoutingDescription')}
</CardDescription>
</CardHeader>
<CardContent>
<EventBindingsEditor
form={form}
botId={initBotId}
supportedEvents={adapterSupportedEvents[currentAdapter] || []}
agentOptions={agentNameList}
/>
</CardContent>
</Card>
)}
</form>
</Form>
);
web/src/app/home/bots/components/bot-form/EventBindingsEditor.tsx
+263 -34
View File
@@ -81,6 +81,7 @@ import {
Agent,
BotRouteDryRunResult,
BotEventRouteStatus,
BotRouteTestResult,
} from '@/app/infra/entities/api';
import { backendClient } from '@/app/infra/http';
@@ -240,11 +241,98 @@ function routeStatusBadgeClass(
return 'border-border bg-muted/40 text-muted-foreground';
}
function localizedFailureReason(
failureCode: string | null | undefined,
fallback: string | null | undefined,
t: TFunction,
) {
if (!failureCode) return fallback || '';
const key = `bots.routeFailure.${failureCode}`;
const label = t(key);
return label === key ? fallback || failureCode : label;
}
function routeStatusDetail(
status: BotEventRouteStatus | undefined,
t: TFunction,
) {
if (!status) return '';
if (status.failure_code) {
return localizedFailureReason(status.failure_code, status.reason, t);
}
if (status.last_status) {
const key = `bots.routeStatusDetail.${status.last_status}`;
const label = t(key);
if (label !== key) return label;
}
return status.reason || formatRouteStatusTime(status.timestamp);
}
function diagnosticDetailLabel(
detail: Record<string, unknown>,
fallbackIndex: number,
t: TFunction,
) {
const order =
typeof detail.binding_index === 'number'
? detail.binding_index
: typeof detail.order === 'number'
? detail.order
: fallbackIndex;
const route = t('bots.dryRunRuleIndex', { index: order + 1 });
if (detail.selected) {
return t('bots.dryRunDiagnosticSelected', { route });
}
const failureCode =
typeof detail.failure_code === 'string' ? detail.failure_code : null;
const reason = localizedFailureReason(
failureCode,
typeof detail.reason === 'string' ? detail.reason : null,
t,
);
if (detail.matched) {
return t('bots.dryRunDiagnosticMatched', { route, reason });
}
return t('bots.dryRunDiagnosticSkipped', { route, reason });
}
function formatRouteStatusTime(timestamp: number | null | undefined) {
if (!timestamp) return '';
return new Date(timestamp * 1000).toLocaleString();
}
function samplePayloadForEvent(eventType: string): Record<string, unknown> {
if (eventType === 'message.received') {
return {
message_text: 'Hello',
chat_type: 'private',
chat_id: 'test-user',
user_id: 'test-user',
};
}
if (eventType === 'group.member_joined') {
return {
group_id: 'test-group',
group_name: 'Test Group',
user_id: 'test-user',
user_name: 'Test User',
};
}
if (eventType === 'group.member_left') {
return {
group_id: 'test-group',
group_name: 'Test Group',
user_id: 'test-user',
user_name: 'Test User',
is_kicked: false,
};
}
if (eventType === 'platform.specific') {
return { action: 'test', data: {} };
}
return {};
}
// ── target combobox (type + target merged, with search + groups) ───────────────
// Encoded value: "discard", "agent:<uuid>", "pipeline:<uuid>"
@@ -639,21 +727,29 @@ function RouteDryRunDialog({
bindings,
eventOptions,
agentOptions,
onRouteStatusUpdate,
}: {
botId?: string;
bindings: EventBinding[];
eventOptions: string[];
agentOptions: Agent[];
onRouteStatusUpdate?: (statuses: BotEventRouteStatus[]) => void;
}) {
const { t } = useTranslation();
const firstEvent = eventOptions[0] ?? DEFAULT_EVENTS[0];
const [open, setOpen] = useState(false);
const [eventType, setEventType] = useState(firstEvent);
const [payloadText, setPayloadText] = useState('{\n "message_text": ""\n}');
const [payloadText, setPayloadText] = useState(() =>
JSON.stringify(samplePayloadForEvent(firstEvent), null, 2),
);
const [advancedPayloadOpen, setAdvancedPayloadOpen] = useState(false);
const [isRunning, setIsRunning] = useState(false);
const [isDispatching, setIsDispatching] = useState(false);
const [payloadError, setPayloadError] = useState<string | null>(null);
const [runError, setRunError] = useState<string | null>(null);
const [result, setResult] = useState<BotRouteDryRunResult | null>(null);
const [dispatchResult, setDispatchResult] =
useState<BotRouteTestResult | null>(null);
useEffect(() => {
if (!eventOptions.includes(eventType)) {
@@ -661,6 +757,13 @@ function RouteDryRunDialog({
}
}, [eventOptions, eventType, firstEvent]);
useEffect(() => {
setPayloadText(JSON.stringify(samplePayloadForEvent(eventType), null, 2));
setPayloadError(null);
setResult(null);
setDispatchResult(null);
}, [eventType]);
function resolveTargetName(resultTarget?: BotRouteDryRunResult['target']) {
if (!resultTarget) return '';
if (resultTarget.target_name) return resultTarget.target_name;
@@ -669,11 +772,8 @@ function RouteDryRunDialog({
return agent ? targetLabel(agent) : (resultTarget.target_uuid ?? '');
}
async function runDryRun() {
function parsePayload(): Record<string, unknown> | null {
setPayloadError(null);
setRunError(null);
setResult(null);
let payload: Record<string, unknown> = {};
if (payloadText.trim()) {
try {
@@ -684,14 +784,24 @@ function RouteDryRunDialog({
Array.isArray(parsed)
) {
setPayloadError(t('bots.dryRunPayloadObjectError'));
return;
return null;
}
payload = parsed as Record<string, unknown>;
} catch {
setPayloadError(t('bots.dryRunPayloadJsonError'));
return;
return null;
}
}
return payload;
}
async function runDryRun() {
setRunError(null);
setResult(null);
setDispatchResult(null);
const payload = parsePayload();
if (payload === null) return;
if (!botId) {
setRunError(t('bots.dryRunNeedsSavedBot'));
@@ -717,6 +827,43 @@ function RouteDryRunDialog({
}
}
async function dispatchTestEvent() {
setRunError(null);
setDispatchResult(null);
const payload = parsePayload();
if (payload === null) return;
if (!botId) {
setRunError(t('bots.dryRunNeedsSavedBot'));
return;
}
setIsDispatching(true);
try {
const testResult = await backendClient.testBotEventRoute(botId, {
event_type: eventType,
payload,
});
setDispatchResult(testResult);
onRouteStatusUpdate?.(testResult.route_status?.routes || []);
if (!testResult.dispatched) {
setRunError(
localizedFailureReason(
testResult.failure_code,
testResult.reason,
t,
) || t('bots.routeTestFailed'),
);
}
} catch (error) {
const err = error as { msg?: string };
setRunError(err.msg || t('bots.routeTestFailed'));
} finally {
setIsDispatching(false);
}
}
const targetName = result ? resolveTargetName(result.target) : '';
return (
@@ -738,7 +885,7 @@ function RouteDryRunDialog({
</DialogHeader>
<div className="space-y-4">
<div className="grid gap-3 sm:grid-cols-[220px_1fr]">
<div className="space-y-3">
<div className="space-y-1.5">
<label className="text-sm font-medium">
{t('bots.dryRunEventType')}
@@ -756,23 +903,55 @@ function RouteDryRunDialog({
</SelectContent>
</Select>
</div>
<div className="space-y-1.5">
<label className="text-sm font-medium">
{t('bots.dryRunPayload')}
</label>
<Textarea
value={payloadText}
onChange={(e) => setPayloadText(e.target.value)}
className="min-h-[118px] font-mono text-xs"
spellCheck={false}
placeholder='{"message_text": "hello"}'
/>
{payloadError ? (
<p className="text-xs text-destructive">{payloadError}</p>
) : (
<p className="text-xs text-muted-foreground">
{t('bots.dryRunPayloadHint')}
</p>
<div className="rounded-md border bg-muted/20 px-3 py-2.5">
<div className="flex items-start justify-between gap-3">
<div className="min-w-0">
<p className="text-sm font-medium">
{t('bots.dryRunSampleReady')}
</p>
<p className="mt-0.5 text-xs leading-relaxed text-muted-foreground">
{t('bots.dryRunSampleDescription', {
event: eventLabel(eventType, t),
})}
</p>
</div>
<Button
type="button"
variant="ghost"
size="sm"
className="h-8 shrink-0 px-2 text-xs"
onClick={() => setAdvancedPayloadOpen((value) => !value)}
>
{advancedPayloadOpen ? (
<ChevronDown className="h-3.5 w-3.5" />
) : (
<ChevronRight className="h-3.5 w-3.5" />
)}
{advancedPayloadOpen
? t('bots.dryRunHidePayload')
: t('bots.dryRunEditPayload')}
</Button>
</div>
{advancedPayloadOpen && (
<div className="mt-3 space-y-1.5 border-t pt-3">
<label className="text-xs font-medium">
{t('bots.dryRunPayload')}
</label>
<Textarea
value={payloadText}
onChange={(e) => setPayloadText(e.target.value)}
className="min-h-[118px] font-mono text-xs"
spellCheck={false}
placeholder='{"message_text": "hello"}'
/>
{payloadError ? (
<p className="text-xs text-destructive">{payloadError}</p>
) : (
<p className="text-xs text-muted-foreground">
{t('bots.dryRunPayloadHint')}
</p>
)}
</div>
)}
</div>
</div>
@@ -832,23 +1011,54 @@ function RouteDryRunDialog({
</div>
</div>
{result.diagnostic_steps.length > 0 && (
{(result.diagnostic_details?.length ||
result.diagnostic_steps.length > 0) && (
<div className="mt-3">
<div className="mb-1.5 flex items-center gap-1.5 text-xs font-medium text-muted-foreground">
<ListChecks className="h-3.5 w-3.5" />
{t('bots.dryRunDiagnostics')}
</div>
<ol className="space-y-1 pl-5 text-xs text-muted-foreground">
{result.diagnostic_steps.map((step, index) => (
<li key={`${index}:${step}`} className="list-decimal">
{step}
</li>
))}
{result.diagnostic_details?.length
? result.diagnostic_details.map((detail, index) => (
<li
key={`${index}:${String(detail.binding_id ?? '')}`}
className="list-decimal"
>
{diagnosticDetailLabel(detail, index, t)}
</li>
))
: result.diagnostic_steps.map((step, index) => (
<li
key={`${index}:${step}`}
className="list-decimal"
>
{step}
</li>
))}
</ol>
</div>
)}
</div>
)}
{dispatchResult?.dispatched && (
<Alert>
<CheckCircle2 className="h-4 w-4" />
<AlertDescription>
{t('bots.routeTestDispatched', {
count: dispatchResult.suppressed_outputs?.length || 0,
})}
</AlertDescription>
</Alert>
)}
<Alert className="border-amber-200 bg-amber-50/60 text-amber-900 dark:border-amber-900/50 dark:bg-amber-950/20 dark:text-amber-200">
<AlertCircle className="h-4 w-4" />
<AlertDescription>
{t('bots.routeTestSideEffectWarning')}
</AlertDescription>
</Alert>
</div>
<DialogFooter>
@@ -859,10 +1069,25 @@ function RouteDryRunDialog({
>
{t('common.close')}
</Button>
<Button type="button" onClick={runDryRun} disabled={isRunning}>
<Button
type="button"
onClick={runDryRun}
disabled={isRunning || isDispatching}
>
<Play className="h-4 w-4 mr-1" />
{isRunning ? t('bots.dryRunRunning') : t('bots.dryRunAction')}
</Button>
<Button
type="button"
variant="outline"
onClick={dispatchTestEvent}
disabled={isRunning || isDispatching}
>
<Activity className="h-4 w-4 mr-1" />
{isDispatching
? t('bots.routeTestRunning')
: t('bots.routeTestAction')}
</Button>
</DialogFooter>
</DialogContent>
</Dialog>
@@ -905,6 +1130,7 @@ function BindingCardContent({
const filterCount = (binding.filters as FilterRow[] | undefined)?.length ?? 0;
const pipelineAllowed = isMessageEventPattern(binding.event_pattern);
const statusTime = formatRouteStatusTime(routeStatus?.timestamp);
const statusDetail = routeStatusDetail(routeStatus, t);
return (
<div className="rounded-lg border bg-card">
@@ -1002,9 +1228,11 @@ function BindingCardContent({
</Badge>
<span
className="max-w-full truncate text-[11px] text-muted-foreground"
title={routeStatus?.reason || routeStatus?.message || statusTime}
title={
statusTime ? `${statusDetail} · ${statusTime}` : statusDetail
}
>
{routeStatus?.failure_code || routeStatus?.reason || statusTime}
{statusDetail || statusTime}
</span>
</div>
@@ -1325,6 +1553,7 @@ export default function EventBindingsEditor({
bindings={bindings}
eventOptions={dryRunEventOptions}
agentOptions={agentOptions}
onRouteStatusUpdate={setRouteStatuses}
/>
<Button
type="button"
web/src/app/infra/entities/api/index.ts
+16
View File
@@ -252,6 +252,11 @@ export interface BotRouteDryRunRequest {
event_bindings?: EventBinding[];
}
export interface BotRouteTestRequest {
event_type: string;
payload?: Record<string, unknown>;
}
export interface BotRouteDryRunTarget {
target_type: EventBinding['target_type'];
target_uuid?: string | null;
@@ -306,6 +311,17 @@ export interface BotEventRouteStatusResponse {
stale_routes: BotEventRouteStatus[];
}
export interface BotRouteTestResult {
dispatched: boolean;
event_type: string;
status?: BotEventRouteStatus['last_status'];
binding_id?: string | null;
failure_code?: string | null;
reason?: string | null;
suppressed_outputs: Array<Record<string, unknown>>;
route_status: BotEventRouteStatusResponse;
}
export interface ApiRespKnowledgeBases {
bases: KnowledgeBase[];
}
web/src/app/infra/http/BackendClient.ts
+12
View File
@@ -61,6 +61,8 @@ import {
ApiRespSkill,
BotRouteDryRunRequest,
BotRouteDryRunResult,
BotRouteTestRequest,
BotRouteTestResult,
BotEventRouteStatusResponse,
} from '@/app/infra/entities/api';
import { Plugin } from '@/app/infra/entities/plugin';
@@ -473,6 +475,16 @@ export class BackendClient extends BaseHttpClient {
return this.get(`/api/v1/platform/bots/${botId}/event-routes/status`);
}
public testBotEventRoute(
botId: string,
request: BotRouteTestRequest,
): Promise<BotRouteTestResult> {
return this.post(
`/api/v1/platform/bots/${botId}/event-routes/test`,
request,
);
}
public deleteBot(uuid: string): Promise<object> {
return this.delete(`/api/v1/platform/bots/${uuid}`);
}