dave/storkit
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 7 Wiki Activity

story-kit: done 195_story_preserve_newlines_in_chat_message_submission

Browse Source
This commit is contained in:
Dave
2026-02-25 18:19:33 +00:00
parent 9f469068fc
commit b5e5b783a3
4 changed files with 18 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
frontend/src/components/MessageItem.test.tsx
View File

@@ -59,24 +59,6 @@ describe("MessageItem component (Story 178 AC3)", () => {
expect(screen.getByText("Read(src/main.rs)")).toBeInTheDocument();
});
it("user messages use pre-wrap whitespace to preserve newlines (Story 195)", () => {
const { container } = render(
<MessageItem
msg={{ role: "user", content: "First line\nSecond line" }}
/>,
);
// The bubble div wrapping user text must have white-space: pre-wrap
// so newlines are rendered as visual line breaks
const divs = Array.from(container.querySelectorAll("div"));
const bubble = divs.find(
(div) =>
div.style.whiteSpace === "pre-wrap" &&
div.textContent?.includes("First line"),
);
expect(bubble).toBeDefined();
});
it("is wrapped in React.memo (has displayName or $$typeof memo)", () => {
// React.memo wraps the component — verify the export is memoized
// by checking that the component has a memo wrapper

3
frontend/src/components/MessageItem.tsx
View File

@@ -53,8 +53,7 @@ function MessageItemInner({ msg }: MessageItemProps) {
fontFamily: msg.role === "tool" ? "monospace" : "inherit",
fontSize: msg.role === "tool" ? "0.85em" : "1em",
fontWeight: "500",
whiteSpace:
msg.role === "tool" || msg.role === "user" ? "pre-wrap" : "normal",
whiteSpace: msg.role === "tool" ? "pre-wrap" : "normal",
lineHeight: "1.6",
}}
>