Skip to content

Commit 8b7cede

Browse files
committed
refactor(web): keep SimpleWorkEntryRow name on extracted module
Revert the identifier rename from the extraction — the component was called `SimpleWorkEntryRow` in MessagesTimeline and should keep that name in its new file too. Only the module path changes.
1 parent 479e876 commit 8b7cede

3 files changed

Lines changed: 10 additions & 10 deletions

File tree

apps/web/src/components/chat/MessagesTimeline.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ import {
3232
type MessagesTimelineRow,
3333
} from "./MessagesTimeline.logic";
3434
import { TerminalContextInlineChip } from "./TerminalContextInlineChip";
35-
import { WorkEntryRow } from "./WorkEntryRow";
35+
import { SimpleWorkEntryRow } from "./SimpleWorkEntryRow";
3636
import {
3737
deriveDisplayedUserMessageState,
3838
type ParsedTerminalContextEntry,
@@ -550,7 +550,7 @@ const WorkGroupSection = memo(function WorkGroupSection({
550550
)}
551551
<div className="space-y-0.5">
552552
{visibleEntries.map((workEntry) => (
553-
<WorkEntryRow
553+
<SimpleWorkEntryRow
554554
key={`work-row:${workEntry.id}`}
555555
workEntry={workEntry}
556556
workspaceRoot={workspaceRoot}

apps/web/src/components/chat/WorkEntryRow.tsx renamed to apps/web/src/components/chat/SimpleWorkEntryRow.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ import { normalizeCompactToolLabel } from "./MessagesTimeline.logic";
2424
* subscriptions — so both the live timeline and the Appearance settings
2525
* preview can render it identically.
2626
*/
27-
export const WorkEntryRow = memo(function WorkEntryRow(props: {
27+
export const SimpleWorkEntryRow = memo(function SimpleWorkEntryRow(props: {
2828
workEntry: WorkLogEntry;
2929
workspaceRoot: string | undefined;
3030
}) {

apps/web/src/components/settings/ChatAppearancePreview.tsx

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { type CSSProperties } from "react";
22
import ChatMarkdown from "../ChatMarkdown";
3-
import { WorkEntryRow } from "../chat/WorkEntryRow";
3+
import { SimpleWorkEntryRow } from "../chat/SimpleWorkEntryRow";
44
import { type WorkLogEntry } from "../../session-logic";
55

66
const SAMPLE_MARKDOWN = `Sure — here's the refactor:
@@ -38,15 +38,15 @@ interface ChatAppearancePreviewProps {
3838

3939
/**
4040
* Decorative, read-only chat mock rendered in the Appearance settings
41-
* panel. Reuses the real timeline's `WorkEntryRow` and `ChatMarkdown` so
41+
* panel. Reuses the real timeline's `SimpleWorkEntryRow` and `ChatMarkdown` so
4242
* whatever styling they carry, the preview carries too — no risk of drift.
4343
*
4444
* Every text tier the real timeline uses is exercised here so scaling is
4545
* visible end-to-end:
4646
* - `text-chat-4xs` — work-log section header
47-
* - `text-chat-xs` — tool-row heading (inside `WorkEntryRow`), user-bubble timestamp
48-
* - `text-chat-2xs` — tooltip command preview (inside `WorkEntryRow`)
49-
* - `text-chat-3xs` — changed-file chip (inside `WorkEntryRow`), assistant meta line
47+
* - `text-chat-xs` — tool-row heading (inside `SimpleWorkEntryRow`), user-bubble timestamp
48+
* - `text-chat-2xs` — tooltip command preview (inside `SimpleWorkEntryRow`)
49+
* - `text-chat-3xs` — changed-file chip (inside `SimpleWorkEntryRow`), assistant meta line
5050
* - `text-chat-body` — user / assistant message bodies
5151
* Plus `ChatMarkdown`, whose `.chat-markdown` rules in `index.css` scale
5252
* via `[data-timeline-root]` — set here on the wrapper along with
@@ -67,8 +67,8 @@ export function ChatAppearancePreview({ fontSize }: ChatAppearancePreviewProps)
6767
Tool calls (2)
6868
</p>
6969
<div className="space-y-0.5">
70-
<WorkEntryRow workEntry={PREVIEW_BASH_ROW} workspaceRoot={undefined} />
71-
<WorkEntryRow workEntry={PREVIEW_EDIT_ROW} workspaceRoot={undefined} />
70+
<SimpleWorkEntryRow workEntry={PREVIEW_BASH_ROW} workspaceRoot={undefined} />
71+
<SimpleWorkEntryRow workEntry={PREVIEW_EDIT_ROW} workspaceRoot={undefined} />
7272
</div>
7373
</div>
7474
<div className="flex justify-end">

0 commit comments

Comments
 (0)