Skip to content

[codex] Phase 3G: chat UI polish#17

Merged
sukar0972 merged 3 commits into
mainfrom
codex/phase3g-chat-ui-polish
Jul 4, 2026
Merged

[codex] Phase 3G: chat UI polish#17
sukar0972 merged 3 commits into
mainfrom
codex/phase3g-chat-ui-polish

Conversation

@sukar0972

Copy link
Copy Markdown
Owner

Summary

Phase 3G applies a narrow web UI polish slice:

  • restore the ultrathink composer frame border effect
  • stabilize composer provider chrome while typing by deriving a coarse prompt-injection state
  • add a small regression assertion documenting that ordinary prompt body edits stay in the same ultrathink state

Intentionally deferred from this phase

  • RightPanelTabs middle-click/inset commits: this fork currently uses RightPanelSheet and does not have the upstream tabbed right-panel store/component path.
  • Main sidebar toggle: conflicts broadly across fork-customized layout/keybinding files; should be a separate focused PR if we still want it before Phase 4.
  • Persistent word-wrap: pulls in deleted desktop/file-preview paths plus larger ChatMarkdown/DiffPanel refactors; better grouped with the markdown/scroll UI phase.

Grok Composer 2.5 review

Reviewed with grok -m grok-composer-2.5-fast.

Verdict: approve, no merge-blocking issues. I applied Grok's small non-blocking suggestion to add a stabilization-contract assertion and a comment explaining why the raw prompt became a coarse enum.

Verification

  • vp test apps/web/src/components/chat/composerProviderState.test.tsx — 10 passed
  • vp run typecheck — passed
    • existing suggestion remains: src/diagnostics/ProcessResourceMonitor.ts(315,5)
  • vp check — passed
    • existing react(no-unstable-nested-components) warnings remain

juliusmarminge and others added 3 commits July 4, 2026 07:42
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Julius Marminge <juliusmarminge@users.noreply.github.com>
(cherry picked from commit f5f98cf)
@github-actions github-actions Bot added size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Jul 4, 2026
@sukar0972 sukar0972 marked this pull request as ready for review July 4, 2026 20:50
@sukar0972 sukar0972 merged commit ba4614b into main Jul 4, 2026
10 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants