fix(web): keep goal controls beside the composer#26023

Approvability

Verdict: Would Approve

Macroscope's review found this PR approvable — This is a small, localized UI bug fix that relocates existing goal controls beside the composer without changing data, APIs, or production infrastructure. An edge case remains where those controls may be hidden while an approval or question drawer is open.

Needs human approval because:

• 1 blocking correctness issue found at or above your repo's Minimum Blocking Severity

^{Adjust the Minimum Blocking Severity for this repo — including turning it Off — in Settings. You can add or adjust custom eligibility rules. Learn more.}

Explain Change →

No actionable comments were generated in the recent review.

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

External image Recent review info

External image Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: main
• File: .coderabbit.config.ts
• Commit: 6577c6b
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Advanced
• Run ID: dc16e35a-a374-4f7b-b7e0-9ff18b159f26

External image Commits

Reviewing files that changed from the base of the PR and between 8557fce687729b525d72e15ba615927fc95e554a and 4b79e5f9d7687eb9742f1f18ec28c943b2f7eef2.

External image Files selected for processing: 1

• apps/web/src/components/ChatView.tsx

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 7 remain after this review.

thanks @diddletime. the original correctly identified that goal controls disappear when reading earlier messages. i kept your commit and merged current main into this branch.

the final change reuses the existing goal row beside the composer, outside the question/approval filter and on the common path used by provider-native subagents. this removes the widened banner variant entirely. the row uses the existing composer surface so attached drawers cannot leave its text over the transcript. the final diff against main is two files, 11 additions and 11 deletions.

verification passed on d6a967be737e3665193ec89926833f42e6a84ddd:

• blacksmith vp run check:changed: format, lint, both knip checks, affected typechecks, 42 existing tests and both affected react compiler checks.
• github CI, agent context budget and mobile fingerprint check. preview workflows correctly skipped.
• real codex luna goal in the web client: reproduced main's scrolling problem, paused the goal, scrolled through earlier messages, opened full goal details and used Clear through the provider/client path. Resume and Clear remained reachable at 390px in light and dark themes.
• a persisted question fixture confirms the goal row stays visible and painted while the question is open. this is visual fixture evidence, not a live provider question test.
• two independent source reviews approved the final head. the original drawer/subagent findings and the first review's missing-background finding are addressed.

unverified: packaged electron, native mobile, live provider-native subagents, live approval/question responses and Resume. their command callbacks and provider paths are unchanged.

before on main, scrolled to the beginning of the real goal thread:

before: goal controls are outside the visible scrolling timeline

after, matching thread, goal and scroll position:

after: paused goal, Resume and Clear stay beside the composer

scrolling, details and Clear through codex:

real client scrolling, goal details and Clear

persisted question fixture, cropped to the goal and question controls:

visual fixture: goal controls retain their surface above an open question

merged current github/main into the existing head again without rebasing (4b79e5f9d7). the only conflict was main re-indenting the composer block in ChatView.tsx and adding an import; i took main's side and re-applied this PR's three edits unchanged. diff against main is still 2 files, +11/-9.

blacksmith vp run check:changed passes: format, lint, knip, typecheck, 49 focused tests, and compiler coverage.

chrome captures at desktop and narrow web widths use isolated goal fixture state. they verify appearance, not provider-backed resume/clear actions; those actions and pending-request integration remain unverified.

goal beside the desktop composer

goal beside the narrow web composer

Verified in a real dev environment

verified as-is at 8557fce687729b525d72e15ba615927fc95e554a on this Linux Boat VM, using repository Playwright and /usr/bin/google-chrome against vp run dev. no mocks, fixtures, seeded conversations, or source changes.

• reproduced on github/main (afb8619e1d) in ../main: created a project pointing at a real git repository through the UI, selected Codex gpt-6.1-sol, generated a long conversation, and set a real goal that asked a question. scrolling to the beginning put the goal row below the viewport (y=1847 in a 1000px viewport).
• stopped main before starting the exact PR head. copied only that isolated app-created dev state for matching captures. the goal remains beside the composer at the same scroll position (y=697), exactly once, with Resume and Clear reachable while the real provider question is open.
• clicked details, Resume, and Clear. Resume started a real Codex goal turn. Clear paused/interrupted goal work, received the provider's thread/goal/cleared event, and removed the row.
• created another real goal/question on the PR. checked 390px web layouts in light and dark themes, opened full details, cleared the goal, and reloaded: the row stayed absent. submitting the question still started a real Codex response. ordinary provider turns and timeline scrolling also worked.
• Blacksmith vp run check:changed passed: format, lint, knip, affected typechecks, 42 focused tests, and React compiler checks. GitHub CI is green. worker setup required fetching git history and installing its Playwright Chromium before the successful run.

short provider transcript from clicking the controls:

before, main, same thread and scroll position:

main: goal controls disappear while reading earlier messages with a live question open

after, exact PR head:

pr: goal controls stay above the live question beside the composer

scrolling, full details, Resume starting Codex, and Clear removing the goal (idle gaps shortened; no performance claim):

real codex goal scrolling, details, resume and clear

390px light web: resume and clear beside a fresh provider question

390px dark web: resume and clear beside a fresh provider question

Not verified

packaged Electron, native iOS/Android, macOS/Windows, relay/tunnel connections, other providers, provider-native subagent views, and live approval drawers. the narrow captures are responsive web, not native mobile. no PR commits or pushes were needed.

main's first checklist response required a page reload to become visible despite a completed provider response. the PR client also needed a reload after its initial cold load of ChatView. both recovered within minutes; neither changed the goal-control result. these startup/live-render observations were not investigated as part of this two-file UI verification.