diff --git a/clients/web/src/components/elements/MessageDirectionBadge/MessageDirectionBadge.stories.tsx b/clients/web/src/components/elements/MessageDirectionBadge/MessageDirectionBadge.stories.tsx index 51a34ff8..a4aa55e0 100644 --- a/clients/web/src/components/elements/MessageDirectionBadge/MessageDirectionBadge.stories.tsx +++ b/clients/web/src/components/elements/MessageDirectionBadge/MessageDirectionBadge.stories.tsx @@ -22,6 +22,6 @@ export const Incoming: Story = { args: { direction: "incoming" }, play: async ({ canvasElement }) => { const canvas = within(canvasElement); - expect(canvas.getByText("client ← server")).toBeInTheDocument(); + expect(canvas.getByText("server → client")).toBeInTheDocument(); }, }; diff --git a/clients/web/src/components/elements/MessageDirectionBadge/MessageDirectionBadge.test.tsx b/clients/web/src/components/elements/MessageDirectionBadge/MessageDirectionBadge.test.tsx index 6c297daa..d7257206 100644 --- a/clients/web/src/components/elements/MessageDirectionBadge/MessageDirectionBadge.test.tsx +++ b/clients/web/src/components/elements/MessageDirectionBadge/MessageDirectionBadge.test.tsx @@ -8,8 +8,8 @@ describe("MessageDirectionBadge", () => { expect(screen.getByText("client → server")).toBeInTheDocument(); }); - it("renders client ← server for incoming", () => { + it("renders server → client for incoming", () => { renderWithMantine(); - expect(screen.getByText("client ← server")).toBeInTheDocument(); + expect(screen.getByText("server → client")).toBeInTheDocument(); }); }); diff --git a/clients/web/src/components/elements/MessageDirectionBadge/MessageDirectionBadge.tsx b/clients/web/src/components/elements/MessageDirectionBadge/MessageDirectionBadge.tsx index f02426bb..785456cb 100644 --- a/clients/web/src/components/elements/MessageDirectionBadge/MessageDirectionBadge.tsx +++ b/clients/web/src/components/elements/MessageDirectionBadge/MessageDirectionBadge.tsx @@ -4,19 +4,19 @@ export interface MessageDirectionBadgeProps { /** * Direction of travel for the entry: "outgoing" = the inspector sent it to * the server (client → server); "incoming" = the server sent it to the - * inspector (client ← server). + * inspector (server → client). */ direction: "outgoing" | "incoming"; } const LABEL: Record = { outgoing: "client → server", - incoming: "client ← server", + incoming: "server → client", }; /** * Dual-state badge showing which way a Protocol/Network entry traveled. Outgoing - * (client → server) is green; incoming (client ← server) is purple — not yellow, + * (client → server) is green; incoming (server → client) is purple — not yellow, * which (paired with green) reads as caution/ok status rather than direction. * Shared by `ProtocolEntry` and `NetworkEntry`. */ diff --git a/clients/web/src/components/groups/MessageDirectionFilter/MessageDirectionFilter.test.tsx b/clients/web/src/components/groups/MessageDirectionFilter/MessageDirectionFilter.test.tsx index 33b96770..eb19384e 100644 --- a/clients/web/src/components/groups/MessageDirectionFilter/MessageDirectionFilter.test.tsx +++ b/clients/web/src/components/groups/MessageDirectionFilter/MessageDirectionFilter.test.tsx @@ -17,7 +17,7 @@ describe("MessageDirectionFilter", () => { screen.getByRole("button", { name: "client → server" }), ).toBeInTheDocument(); expect( - screen.getByRole("button", { name: "client ← server" }), + screen.getByRole("button", { name: "server → client" }), ).toBeInTheDocument(); }); @@ -30,7 +30,7 @@ describe("MessageDirectionFilter", () => { onToggleDirection={onToggleDirection} />, ); - await user.click(screen.getByRole("button", { name: "client ← server" })); + await user.click(screen.getByRole("button", { name: "server → client" })); expect(onToggleDirection).toHaveBeenCalledWith("server", false); }); diff --git a/clients/web/src/components/groups/MessageDirectionFilter/MessageDirectionFilter.tsx b/clients/web/src/components/groups/MessageDirectionFilter/MessageDirectionFilter.tsx index 029e66c8..033f3871 100644 --- a/clients/web/src/components/groups/MessageDirectionFilter/MessageDirectionFilter.tsx +++ b/clients/web/src/components/groups/MessageDirectionFilter/MessageDirectionFilter.tsx @@ -9,14 +9,14 @@ const SubtleButton = Button.withProps({ // The two message directions, in display order. Label + color mirror the // MessageDirectionBadge: outgoing (client → server) is green, incoming -// (client ← server) is violet. +// (server → client) is violet. const MESSAGE_DIRECTIONS: { origin: MessageOrigin; label: string; color: string; }[] = [ { origin: "client", label: "client → server", color: "green" }, - { origin: "server", label: "client ← server", color: "violet" }, + { origin: "server", label: "server → client", color: "violet" }, ]; export interface MessageDirectionFilterProps { @@ -27,7 +27,7 @@ export interface MessageDirectionFilterProps { /** * "Filter by Message Direction" section — a Select/Deselect All control plus a - * FilterToggleButton per direction (client → server / client ← server). Used by + * FilterToggleButton per direction (client → server / server → client). Used by * the Protocol controls. (Kept as its own component so the section is testable in * isolation and reusable if another screen ever needs a direction filter.) */ diff --git a/clients/web/src/components/groups/ProtocolControls/ProtocolControls.test.tsx b/clients/web/src/components/groups/ProtocolControls/ProtocolControls.test.tsx index f902def4..c182bd46 100644 --- a/clients/web/src/components/groups/ProtocolControls/ProtocolControls.test.tsx +++ b/clients/web/src/components/groups/ProtocolControls/ProtocolControls.test.tsx @@ -91,7 +91,7 @@ describe("ProtocolControls", () => { screen.getByRole("button", { name: "client → server" }), ).toBeInTheDocument(); expect( - screen.getByRole("button", { name: "client ← server" }), + screen.getByRole("button", { name: "server → client" }), ).toBeInTheDocument(); }); @@ -102,7 +102,7 @@ describe("ProtocolControls", () => { , ); // Currently visible → clicking turns it off. - await user.click(screen.getByRole("button", { name: "client ← server" })); + await user.click(screen.getByRole("button", { name: "server → client" })); expect(onToggleDirection).toHaveBeenCalledWith("server", false); }); diff --git a/clients/web/src/components/groups/ProtocolEntry/ProtocolEntry.test.tsx b/clients/web/src/components/groups/ProtocolEntry/ProtocolEntry.test.tsx index ee7caeed..16560220 100644 --- a/clients/web/src/components/groups/ProtocolEntry/ProtocolEntry.test.tsx +++ b/clients/web/src/components/groups/ProtocolEntry/ProtocolEntry.test.tsx @@ -158,11 +158,11 @@ describe("ProtocolEntry", () => { expect(screen.getByText("client → server")).toBeInTheDocument(); }); - it("shows client ← server for a server-originated entry", () => { + it("shows server → client for a server-originated entry", () => { renderWithMantine( , ); - expect(screen.getByText("client ← server")).toBeInTheDocument(); + expect(screen.getByText("server → client")).toBeInTheDocument(); }); it("renders Pin label when not pinned", () => { diff --git a/clients/web/src/components/groups/ProtocolListPanel/ProtocolListPanel.test.tsx b/clients/web/src/components/groups/ProtocolListPanel/ProtocolListPanel.test.tsx index 04df5740..115fa1d3 100644 --- a/clients/web/src/components/groups/ProtocolListPanel/ProtocolListPanel.test.tsx +++ b/clients/web/src/components/groups/ProtocolListPanel/ProtocolListPanel.test.tsx @@ -116,7 +116,7 @@ describe("ProtocolListPanel", () => { pinnedIds={new Set(["req-1"])} />, ); - expect(screen.getByText("Pinned Requests (1)")).toBeInTheDocument(); + expect(screen.getByText("Pinned Messages (1)")).toBeInTheDocument(); expect(screen.getByText("History (2)")).toBeInTheDocument(); }); @@ -186,7 +186,7 @@ describe("ProtocolListPanel", () => { />, ); const pinned = screen.getByRole("button", { - name: "Pinned Requests (1)", + name: "Pinned Messages (1)", }); const history = screen.getByRole("button", { name: "History (2)" }); await user.click(pinned); @@ -346,7 +346,7 @@ describe("ProtocolListPanel", () => { pinnedIds={new Set(["req-1"])} />, ); - expect(screen.getByText("Pinned Requests (1)")).toBeInTheDocument(); + expect(screen.getByText("Pinned Messages (1)")).toBeInTheDocument(); expect(screen.queryByText(/^History \(/)).not.toBeInTheDocument(); }); diff --git a/clients/web/src/components/groups/ProtocolListPanel/ProtocolListPanel.tsx b/clients/web/src/components/groups/ProtocolListPanel/ProtocolListPanel.tsx index d6c8bdaa..7962cb4f 100644 --- a/clients/web/src/components/groups/ProtocolListPanel/ProtocolListPanel.tsx +++ b/clients/web/src/components/groups/ProtocolListPanel/ProtocolListPanel.tsx @@ -98,7 +98,7 @@ const SectionLinkButton = Button.withProps({ }); function formatPinnedTitle(count: number): string { - return `Pinned Requests (${count})`; + return `Pinned Messages (${count})`; } function formatHistoryTitle(count: number): string { diff --git a/clients/web/src/components/screens/ProtocolScreen/ProtocolScreen.test.tsx b/clients/web/src/components/screens/ProtocolScreen/ProtocolScreen.test.tsx index b71f5959..00ba2ae1 100644 --- a/clients/web/src/components/screens/ProtocolScreen/ProtocolScreen.test.tsx +++ b/clients/web/src/components/screens/ProtocolScreen/ProtocolScreen.test.tsx @@ -78,7 +78,7 @@ describe("ProtocolScreen", () => { renderWithMantine( , ); - await user.click(screen.getByRole("button", { name: "client ← server" })); + await user.click(screen.getByRole("button", { name: "server → client" })); expect(onUiChange).toHaveBeenCalledWith( expect.objectContaining({ visibleDirections: { client: true, server: false }, @@ -130,7 +130,7 @@ describe("ProtocolScreen", () => { it("drops the filter sidebar when embedded, keeping the request list", () => { renderWithMantine(); - expect(screen.getByText("Requests")).toBeInTheDocument(); + expect(screen.getByText("Messages")).toBeInTheDocument(); // The sidebar (ProtocolControls, with its Search box) is not rendered. expect(screen.queryByPlaceholderText("Search...")).toBeNull(); });