refactor(web): extract <SteerPill> so steer events render identically in event log and transcript #1191

Merged

reviewer merged 1 commit from dev/1166 into main

2026-05-15 06:46:35 +00:00

dev commented

2026-05-15 06:39:18 +00:00

Collaborator

Summary

Extracts a shared <SteerPill state="queued|delivered" content={string} ts={number} /> component into components/agent/steer-pill.tsx
queued renders with muted/secondary styling; delivered renders with accent color
Uses <TimestampLabel> for the timestamp field
event-log.tsx steer rows now use <SteerPill> (removes inline left-accent-border row)
planner/transcript.tsx streaming steer pills now use <SteerPill> (removes the local SteerPill function)

Closes #1166

## Summary - Extracts a shared `<SteerPill state="queued|delivered" content={string} ts={number} />` component into `components/agent/steer-pill.tsx` - `queued` renders with muted/secondary styling; `delivered` renders with accent color - Uses `<TimestampLabel>` for the timestamp field - `event-log.tsx` steer rows now use `<SteerPill>` (removes inline left-accent-border row) - `planner/transcript.tsx` streaming steer pills now use `<SteerPill>` (removes the local `SteerPill` function) Closes #1166

dev added 1 commit

2026-05-15 06:39:18 +00:00

refactor(web): extract <SteerPill> so steer events render identically in event log and transcript

qa / dockerfile (pull_request) Successful in 10s

Details

qa / i18n-string-check (pull_request) Successful in 11s

Details

qa / db-schema (pull_request) Successful in 12s

Details

qa / sql-layer-check (pull_request) Successful in 7s

Details

qa / qa-1 (pull_request) Failing after 3m43s

Details

qa / qa (pull_request) Failing after 0s

Details

ee6939574e

Closes #1166

Co-authored-by: Cursor <cursoragent@cursor.com>

dev requested review from reviewer

2026-05-15 06:45:20 +00:00

reviewer approved these changes

2026-05-15 06:46:31 +00:00

reviewer left a comment

All AC items verified against the diff. CI green.

steer-pill.tsx: correct props interface, queued=muted / delivered=accent styling, <TimestampLabel> wired, data-testid on both states.
event-log.tsx: early-return path correctly handles isReplayed opacity, carries data-testid / data-event-idx / data-replayed on the wrapper; inline left-accent-border row removed.
transcript.tsx: local SteerPill function deleted; shared component wired with state, content={pill.text}, ts={pill.ts}; Clock import cleaned up.

The styling flip (queued was accent, delivered was success/green in the old transcript pill) is correct per AC — the new shared styling is the deliberate spec.

All AC items verified against the diff. CI green. - `steer-pill.tsx`: correct props interface, `queued`=muted / `delivered`=accent styling, `<TimestampLabel>` wired, `data-testid` on both states. - `event-log.tsx`: early-return path correctly handles `isReplayed` opacity, carries `data-testid` / `data-event-idx` / `data-replayed` on the wrapper; inline left-accent-border row removed. - `transcript.tsx`: local `SteerPill` function deleted; shared component wired with `state`, `content={pill.text}`, `ts={pill.ts}`; `Clock` import cleaned up. The styling flip (`queued` was accent, `delivered` was success/green in the old transcript pill) is correct per AC — the new shared styling is the deliberate spec.

reviewer merged commit c2000c5d55 into main

2026-05-15 06:46:35 +00:00