diff --git a/src/browser/features/Tools/WorkflowActionListToolCall.test.tsx b/src/browser/features/Tools/WorkflowActionListToolCall.test.tsx index be5f03430a..4029404f36 100644 --- a/src/browser/features/Tools/WorkflowActionListToolCall.test.tsx +++ b/src/browser/features/Tools/WorkflowActionListToolCall.test.tsx @@ -140,7 +140,9 @@ describe("WorkflowActionListToolCall", () => { expect(view.getByText("git.changedFiles")).toBeTruthy(); expect(view.getByText("read")).toBeTruthy(); - expect(view.getByText("blocked")).toBeTruthy(); + const blockedBadge = view.getByText("blocked"); + expect(blockedBadge.classList.contains("text-danger")).toBe(true); + expect(blockedBadge.classList.contains("text-warning")).toBe(false); // Blocked actions surface their reason in the row description slot. expect(view.getByText("Project is not trusted")).toBeTruthy(); }); diff --git a/src/browser/features/Tools/WorkflowActionListToolCall.tsx b/src/browser/features/Tools/WorkflowActionListToolCall.tsx index b714e15ad1..dc3900739f 100644 --- a/src/browser/features/Tools/WorkflowActionListToolCall.tsx +++ b/src/browser/features/Tools/WorkflowActionListToolCall.tsx @@ -126,7 +126,7 @@ function WorkflowActionListRow(props: { descriptor: WorkflowActionDescriptor }) {action.metadata.effect} ) : ( - blocked + blocked )} { clickToolHeader(view, "2 definitions"); expect(view.queryByText("executable")).toBeNull(); - expect(view.getByText("blocked")).toBeTruthy(); + const blockedBadge = view.getByText("blocked"); + expect(blockedBadge.classList.contains("text-danger")).toBe(true); + expect(blockedBadge.classList.contains("text-warning")).toBe(false); expect(view.getByText("Project is not trusted")).toBeTruthy(); }); }); diff --git a/src/browser/features/Tools/WorkflowDefinitionToolCall.tsx b/src/browser/features/Tools/WorkflowDefinitionToolCall.tsx index 9d6cbdc09f..338bad79f7 100644 --- a/src/browser/features/Tools/WorkflowDefinitionToolCall.tsx +++ b/src/browser/features/Tools/WorkflowDefinitionToolCall.tsx @@ -58,7 +58,7 @@ export function WorkflowKindBadge() { export function WorkflowBadge(props: { children: React.ReactNode; - tone?: "normal" | "success" | "warning"; + tone?: "normal" | "success" | "warning" | "danger"; }) { return ( @@ -149,7 +151,7 @@ function WorkflowDefinitionListRow(props: { descriptor: WorkflowDefinitionDescri {descriptor.scope} - {!descriptor.executable && blocked} + {!descriptor.executable && blocked} @@ -193,7 +195,7 @@ export function WorkflowDefinitionCard(props: { {descriptor.name} {descriptor.scope} - {!descriptor.executable && blocked} + {!descriptor.executable && blocked} {!props.compact && ( {descriptor.description}