Understanding Comma-to-Newline Chat Display Change
Understanding Comma-to-Newline Chat Display Change
Agent
Claude Code
Model
Claude Opus 4.6
Shared
Feb 18, 12:58 AM
Messages30
Downloads5
i'm viewing a chat replay and i see that we are somehow parsing a comma and turning it into a newline in the chat display. this is the final event output but we build up the chat with deltas so the user can see the info streaming in. what is happening here?
{
"timestamp": "2026-02-17T21:16:00.224Z",
"type": "response_item",
"payload": {
"type": "message",
"role": "assistant",
"content": [
{
"type": "output_text",
"text": "The active RFC is currently just a blank template, so I’m inspecting project architecture and existing RFC patterns to produce a concrete, repo-aligned plan with defaults filled in."
}
],
"phase": "commentary"
}
} {
"timestamp": "2026-02-17T21:16:00.223Z",
"type": "event_msg",
"payload": {
"type": "agent_message",
"message": "The active RFC is currently just a blank template, so I’m inspecting project architecture and existing RFC patterns to produce a concrete, repo-aligned plan with defaults filled in."
}
}
Here's the problem. In ChatMessageList.tsx:27-43, the splitSentences() function has a comma-splitting fallback:
The text in your example is a single sentence (one period at the end), so the first regex split produces only 1 part. It then falls through to the comma fallback, which splits on , so and , repo-aligned.
Then paragraphize() (line 65) groups those comma-split fragments into separate paragraphs, which renders them on separate lines with gaps between them.
The comma fallback was probably intended to break up long comma-separated lists, but it's incorrectly breaking up normal prose that happens to have commas. The fix is to remove or restrict that comma-splitting fallback. What would you like to do — remove it entirely, or restrict it to only fire when there are many comma-separated items (like 4+)?
where did splitSentences come from in the commit history?
git log --all --oneline -n 20 -- electron_app/src/renderer/features/chat/components/ChatMessageList.tsx0792429 edit chat and spec editor styles 1a6d805 Guard chat disambiguation async updates 8c1fa19 chat appearance e85c5b6 Update ChatMessageList.tsx f669a99 take ai response out of text bubble 371137b Guard chat disambiguation async updates fa30536 chat appearance d7e1205 Update ChatMessageList.tsx bdac0d3 take ai response out of text bubble 5a68af2 refactor(electron): remove chat activity message card (by Codex) 22688e3 refactor(chat): extract reusable chat message components 0f042ec refactor(chat): extract reusable chat message components 888323b refactor(chat): extract reusable chat message components
git log --all --oneline -S "splitSentences" -- electron_app/src/renderer/features/chat/components/ChatMessageList.tsxe85c5b6 Update ChatMessageList.tsx d7e1205 Update ChatMessageList.tsx
[Image: source: /Users/josh/Library/Application Support/CleanShot/media/media_VTK6nLMAGe/CleanShot 2026-02-17 at 14.31.40@2x.png]
git show d7e1205 --stat && echo "---" && git show d7e1205 --format="%H %ai %an%n%s%n%b" | head -5commit d7e1205aa09957e2f80e13c6a2067805d458e99f
Author: Ayman Nadeem <875834+aymannadeem@users.noreply.github.com>
Date: Thu Feb 12 13:50:59 2026 -0800
Update ChatMessageList.tsx
.../features/chat/components/ChatMessageList.tsx | 76 +++++++++++++++++++++-
1 file changed, 75 insertions(+), 1 deletion(-)
---
d7e1205aa09957e2f80e13c6a2067805d458e99f 2026-02-12 13:50:59 -0800 Ayman Nadeem
Update ChatMessageList.tsx
diff --git a/electron_app/src/renderer/features/chat/components/ChatMessageList.tsx b/electron_app/src/renderer/features/chat/components/ChatMessageList.tsxgit show d7e1205 -- electron_app/src/renderer/features/chat/components/ChatMessageList.tsxcommit d7e1205aa09957e2f80e13c6a2067805d458e99f
Author: Ayman Nadeem <875834+aymannadeem@users.noreply.github.com>
Date: Thu Feb 12 13:50:59 2026 -0800
Update ChatMessageList.tsx
diff --git a/electron_app/src/renderer/features/chat/components/ChatMessageList.tsx b/electron_app/src/renderer/features/chat/components/ChatMessageList.tsx
index dbd43f0..e85fd52 100644
--- a/electron_app/src/renderer/features/chat/components/ChatMessageList.tsx
+++ b/electron_app/src/renderer/features/chat/components/ChatMessageList.tsx
@@ -19,6 +19,78 @@ type AssistantBlock =
| { type: "list"; items: string[] }
| { type: "code"; content: string; language?: string };
+function normalizeSentence(value: string): string {
+ return value.replace(/\s+/g, " ").trim().toLowerCase();
+}
+
+function splitSentences(text: string): string[] {
+ const normalized = text.replace(/\s+/g, " ").trim();
+ if (!normalized) {
+ return [];
+ }
+ const rough = normalized
+ .split(/(?<=[.!?])\s+(?=[A-Z0-9`"'([])/)
+ .map((part) => part.trim())
+ .filter(Boolean);
+ if (rough.length > 1) {
+ return rough;
+ }
+ return normalized
+ .split(/,\s+(?=[a-z])/i)
+ .map((part) => part.trim())
+ .filter(Boolean);
+}
+
+function collapseNearDuplicateSentences(sentences: string[]): string[] {
+ const result: string[] = [];
+ const recentNormalized: string[] = [];
+ for (const sentence of sentences) {
+ const normalized = normalizeSentence(sentence);
+ if (!normalized) {
+ continue;
+ }
+ if (recentNormalized.includes(normalized)) {
+ continue;
+ }
+ result.push(sentence);
+ recentNormalized.push(normalized);
+ if (recentNormalized.length > 8) {
+ recentNormalized.shift();
+ }
+ }
+ return result;
+}
+
+function paragraphize(text: string): string[] {
+ const sentences = collapseNearDuplicateSentences(splitSentences(text));
+ if (sentences.length <= 1) {
+ return [text];
+ }
+
+ const paragraphs: string[] = [];
+ let current = "";
+ let sentenceCount = 0;
+
+ for (const sentence of sentences) {
+ const next = current.length > 0 ? `${current} ${sentence}` : sentence;
+ const shouldBreak = sentenceCount >= 2 || next.length > 280;
+ if (shouldBreak && current.length > 0) {
+ paragraphs.push(current);
+ current = sentence;
+ sentenceCount = 1;
+ } else {
+ current = next;
+ sentenceCount += 1;
+ }
+ }
+
+ if (current.length > 0) {
+ paragraphs.push(current);
+ }
+
+ return paragraphs;
+}
+
function isSpecialLine(line: string): boolean {
const trimmed = line.trim();
return trimmed.startsWith("```") || /^#{1,3}\s+/.test(trimmed) || /^[-*]\s+/.test(trimmed);
@@ -78,7 +150,9 @@ function parseAssistantBlocks(content: string): AssistantBlock[] {
index += 1;
paragraphLines.push(next.trim());
}
- blocks.push({ type: "paragraph", text: paragraphLines.join(" ") });
+ for (const paragraph of paragraphize(paragraphLines.join(" "))) {
+ blocks.push({ type: "paragraph", text: paragraph });
+ }
}
return blocks;It was introduced by Ayman on Feb 12 in commit d7e1205 ("Update ChatMessageList.tsx"). The entire splitSentences, collapseNearDuplicateSentences, paragraphize, and normalizeSentence suite was added in that single commit.
The intent was to break up long assistant messages into readable paragraphs — splitting first on sentence boundaries (.!?), then falling back to comma splitting when there's only a single sentence. The comma fallback is what's causing your bug: single-sentence prose with commas gets split into separate paragraph blocks.
what other commits happened around then
git log --all --oneline --since="2026-02-12" --until="2026-02-13" --author="Ayman"4b85ea5 move disambiguation into inline chat 8c1fa19 chat appearance e85c5b6 Update ChatMessageList.tsx f669a99 take ai response out of text bubble 2581f4b move disambiguation into inline chat fa30536 chat appearance d7e1205 Update ChatMessageList.tsx bdac0d3 take ai response out of text bubble
git log --all --format="%h %ai %s" --since="2026-02-11" --until="2026-02-14" --author="Ayman"2330d1b 2026-02-13 17:51:27 -0800 edit-preview toggle a12643c 2026-02-13 17:47:04 -0800 fix nested numbered list rendering a5167f2 2026-02-13 17:39:42 -0800 shift spec text vertically for improved spacing f8d186d 2026-02-13 17:35:29 -0800 fix preview bug 33a5881 2026-02-13 17:32:53 -0800 remove title 0792429 2026-02-13 17:28:51 -0800 edit chat and spec editor styles 1aaeb6f 2026-02-13 15:54:49 -0800 add subtle repo next to spec title 4b85ea5 2026-02-12 18:42:59 -0800 move disambiguation into inline chat 8c1fa19 2026-02-12 17:59:07 -0800 chat appearance e85c5b6 2026-02-12 13:50:59 -0800 Update ChatMessageList.tsx f669a99 2026-02-12 13:30:58 -0800 take ai response out of text bubble 2581f4b 2026-02-12 18:42:59 -0800 move disambiguation into inline chat fa30536 2026-02-12 17:59:07 -0800 chat appearance d7e1205 2026-02-12 13:50:59 -0800 Update ChatMessageList.tsx bdac0d3 2026-02-12 13:30:58 -0800 take ai response out of text bubble 72f31dd 2026-02-11 20:53:27 -0800 remove color distinction da3a181 2026-02-11 20:46:34 -0800 move title out of spec 15b2174 2026-02-11 20:39:31 -0800 clean up left pane 97fa4e3 2026-02-11 20:28:43 -0800 have separator extend all the way up to fill vertical space addf77f 2026-02-11 20:21:50 -0800 add separator to sidebar 81cc5e8 2026-02-11 19:58:44 -0800 add toggle side bar + command b support 09fd2bb 2026-02-11 19:44:33 -0800 focused CSS pass now: shared tokens, sidebar grid alignment, label/type harmonization, editor/composer rhythm adjustments, and wider effective chat footprint c995055 2026-02-11 19:20:54 -0800 add hover over text 3f899c8 2026-02-11 19:13:17 -0800 Move add spec out of header to make it cleaner 9d61650 2026-02-11 19:09:32 -0800 Update styles.css e659eb7 2026-02-11 18:55:08 -0800 Update styles.css d848432 2026-02-11 18:47:36 -0800 Merge title field with editor 766e3f3 2026-02-11 18:32:12 -0800 Update styles.css 6f828ce 2026-02-11 18:08:36 -0800 improve spacing and highlight current workspace d87cb4a 2026-02-11 17:45:44 -0800 improve send button appearance 3734a68 2026-02-11 20:53:27 -0800 remove color distinction af69b55 2026-02-11 20:46:34 -0800 move title out of spec c25481f 2026-02-11 20:39:31 -0800 clean up left pane 95f9a64 2026-02-11 20:28:43 -0800 have separator extend all the way up to fill vertical space 0bd5e5b 2026-02-11 20:21:50 -0800 add separator to sidebar c04bcef 2026-02-11 19:58:44 -0800 add toggle side bar + command b support 2f5044c 2026-02-11 19:44:33 -0800 focused CSS pass now: shared tokens, sidebar grid alignment, label/type harmonization, editor/composer rhythm adjustments, and wider effective chat footprint a159eff 2026-02-11 19:20:54 -0800 add hover over text 6b42d58 2026-02-11 19:13:17 -0800 Move add spec out of header to make it cleaner 39e7266 2026-02-11 19:09:32 -0800 Update styles.css 8a8d823 2026-02-11 18:55:08 -0800 Update styles.css 48032e8 2026-02-11 18:47:36 -0800 Merge title field with editor 519a48a 2026-02-11 18:32:12 -0800 Update styles.css f52ac11 2026-02-11 18:08:36 -0800 improve spacing and highlight current workspace fa4c288 2026-02-11 17:45:44 -0800 improve send button appearance 7994309 2026-02-11 16:50:48 -0800 remove center add workspace cta once workspace has been added e7fe1ef 2026-02-11 16:50:48 -0800 remove center add workspace cta once workspace has been added 61ab5dd 2026-02-11 16:37:29 -0800 try to fix ci issue 3225757 2026-02-11 16:30:00 -0800 remove add spec until workspace is added b7d7dcd 2026-02-11 16:26:21 -0800 improve spacing and styling a758349 2026-02-11 16:20:27 -0800 Update chat.css f116634 2026-02-11 16:12:36 -0800 add + icon before add workspace 7307554 2026-02-11 13:52:21 -0800 fixing start screen
The user is seeing commas being turned into newlines in the chat replay display. Let me look at the ReplayChatView component to understand how chat messages are built up from deltas.
It was part of a UI polish sprint on Feb 12. The sequence that day was:
bdac0d3 — "take ai response out of text bubble"d7e1205 — "Update ChatMessageList.tsx" (introduced splitSentences + paragraphize)f669a99 — "take ai response out of text bubble" (duplicate on another branch)e85c5b6 — "Update ChatMessageList.tsx" (duplicate)8c1fa19 / fa30536 — "chat appearance"4b85ea5 / 2581f4b — "move disambiguation into inline chat"It looks like Ayman was restyling the chat — removing the bubble/card treatment for assistant messages and switching to a more open layout. The splitSentences/paragraphize logic was added as part of making the bubble-less text more readable by breaking long blocks into visual paragraphs. The comma fallback was likely meant to handle cases where the AI produces a single long sentence, but it's too aggressive for normal prose.
Perfect! Now I have all the information I need. Let me create a comprehensive summary of my findings:
## Summary
Based on my search of the codebase, here are the key files and code patterns related to how chat messages are built up from deltas/streaming events in the replay and codex session context:
### 1. **Text Delta Accumulation for Chat Messages**
**File:** `/Users/josh/Code/rfc/electron_app/src/main/services/appController/chatDomain.ts`
The primary function that accumulates text deltas into chat messages is `applyToken()` (lines 362-386):
```typescript
export async function applyToken(ctx: ChatDomainContext, delta: string, requestId: string): Promise<void> {
const run = ctx.getActiveChatRun(requestId);
if (!run) {
return;
}
const specUi = ctx.getSpecUi(run.workspaceId, run.specId);
if (!specUi) {
return;
}
const isCurrentSpec = ctx.isCurrentSpec(run.workspaceId, run.specId);
const targetMessages = specUi.chatMessages;
const idx = findLastIndex(
targetMessages,
(message) => message.role === "assistant" && message.requestId === requestId && message.isStreaming === true
);
if (idx < 0) {
return;
}
targetMessages[idx].content += delta; // <-- TEXT ACCUMULATION HERE
if (isCurrentSpec && !ctx.isSelectingSpec()) {
ctx.emitSnapshot();
}
}
```
The key line is `targetMessages[idx].content += delta;` which directly appends incoming text deltas to the message content.
### 2. **Delta Extraction and Multi-Item Message Handling**
**File:** `/Users/josh/Code/rfc/electron_app/src/main/services/codexRunner/notificationHandlers.ts` (lines 212-241)
The `item/agentMessage/delta` notification handler accumulates text from multiple agent message items:
```typescript
if (method === "item/agentMessage/delta") {
const delta = args.extractDeltaText(params, item);
if (!delta || !run) {
return;
}
const isNewItem = itemId && !run.agentMessageTextByItemId.has(itemId);
if (itemId) {
const previous = run.agentMessageTextByItemId.get(itemId) ?? "";
run.agentMessageTextByItemId.set(itemId, `${previous}${delta}`); // <-- ACCUMULATION
}
if (isNewItem && run.agentMessageItemCount > 0) {
run.onToken("\n\n"); // <-- NEWLINE INSERTION BETWEEN ITEMS
}
if (isNewItem) {
run.agentMessageItemCount += 1;
}
run.onToken(delta);
return;
}
```
**Important:** When a new agent message item starts (after the first one), the handler inserts `"\n\n"` to separate multiple agent messages (line 224).
### 3. **Delta Text Extraction**
**File:** `/Users/josh/Code/rfc/electron_app/src/main/services/codexRunner/protocolParsers.ts` (lines 284-301)
```typescript
export function extractDeltaText(params: Record<string, unknown>, item: Record<string, unknown>): string | undefined {
const deltaObj = params.delta as Record<string, unknown> | undefined;
if (deltaObj) {
const text = readString(deltaObj.text) || readString(deltaObj.content);
if (text) {
return text;
}
}
return (
readString(params.delta) ||
readString(params.text) ||
readString(params.content) ||
readString(item.text) ||
readString(item.content) ||
readString(((params.message as Record<string, unknown> | undefined) ?? {}).text)
);
}
```
### 4. **Chat Message Structure**
**File:** `/Users/josh/Code/rfc/electron_app/src/shared/models.ts` (lines 61-70)
```typescript
export interface ChatMessage {
id: string;
role: ChatRole;
content: string;
timestamp: ISODateString;
requestId?: string;
isStreaming?: boolean;
error?: string;
activity?: ActivityPayload;
}
```
Chat messages are stored with a simple `content: string` field that gets accumulated.
### 5. **Comma-to-Newline Pattern (Sentence Splitting)**
**File:** `/Users/josh/Code/rfc/electron_app/src/renderer/features/chat/components/ChatMessageList.tsx` (lines 27-43)
The rendering layer has a `splitSentences()` function that **splits on commas**:
```typescript
function splitSentences(text: string): string[] {
const normalized = text.replace(/\s+/g, " ").trim();
if (!normalized) {
return [];
}
const rough = normalized
.split(/(?<=[.!?])\s+(?=[A-Z0-9`"'([])/) // <-- First tries to split on sentence boundaries
.map((part) => part.trim())
.filter(Boolean);
if (rough.length > 1) {
return rough;
}
return normalized
.split(/,\s+(?=[a-z])/i) // <-- FALLS BACK TO SPLITTING ON COMMAS
.map((part) => part.trim())
.filter(Boolean);
}
```
This is used by `paragraphize()` for rendering assistant messages, converting comma-separated items into separate lines for display purposes.
### 6. **Codex Session Log Text Extraction**
**File:** `/Users/josh/Code/rfc/electron_app/src/replay/views/ReplayCodexSessionView.tsx` (lines 408-451)
When extracting searchable text from Codex session entries, parts are **joined with newlines**:
```typescript
function extractSearchableText(entry: CodexSessionEntry): string {
const parts: string[] = [];
const payload = entry.payload;
if (typeof payload.message === "string") {
parts.push(payload.message);
}
if (typeof payload.text === "string") {
parts.push(payload.text);
}
// ... more field extraction ...
return parts.join("\n"); // <-- JOINS WITH NEWLINES
}
```
This is used for search functionality in the Codex session viewer, not for building actual chat messages.
### Key Findings:
1. **Text accumulation** happens through simple string concatenation: `content += delta`
2. **Newlines between items** are explicitly inserted when new agent message items start
3. **Comma splitting** only happens at the **rendering layer** for display purposes, not during message accumulation
4. **No comma-to-newline conversion** occurs during the streaming/delta accumulation process itself
5. The codex session searchable text uses newline joins for combining multiple text fields, but this is separate from chat message construction
agentId: a2722a4 (for resuming to continue this agent's work if needed)
<usage>total_tokens: 74515
tool_uses: 36
duration_ms: 84457</usage>The agent found the issue. There's a splitSentences() function in ChatMessageList.tsx that splits on commas and renders them as separate paragraphs. Let me look at that code directly.
Found it. The issue is in the rendering layer. Let me look at the exact code.