Skip to content

Polish CodexUI thread, inspector, and conversation UX #49

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
VolkerChristian wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Polish CodexUI thread, inspector, and conversation UX #49

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
44d949e
Polish thread panel interactions
VolkerChristian Aug 25, 2026
67c5451
Unify popup menu styling
VolkerChristian Aug 25, 2026
022082b
Define semantic UI palettes
VolkerChristian Aug 25, 2026
7805ebe
Polish inspector navigation
VolkerChristian Aug 25, 2026
42b1f0e
Add libgit2 change review
VolkerChristian Aug 25, 2026
cf1975f
Resolve and monitor thread repositories
VolkerChristian Aug 25, 2026
693e997
Polish changes inspector layout
VolkerChristian Aug 25, 2026
58e407f
Unify primary panel headers
VolkerChristian Aug 25, 2026
b9f1935
Polish command execution cards
VolkerChristian Aug 26, 2026
955e042
Retain composer draft across threads
VolkerChristian Aug 26, 2026
10524a4
Keep nested output scroll ownership
VolkerChristian Aug 26, 2026
4463964
Render user messages as Markdown
VolkerChristian Aug 26, 2026
4c51325
Promote production shell to canonical application
VolkerChristian Aug 26, 2026
c078438
Install libgit2 in CI
VolkerChristian Aug 26, 2026
985f76c
Align transport and sort chevrons
VolkerChristian Aug 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,7 @@ jobs:
cmake \
iproute2 \
libbluetooth-dev \
libgit2-dev \
libmagic-dev \
libssl-dev \
ninja-build \
Expand Down
245 changes: 127 additions & 118 deletions CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,8 @@ find_package(
COMPONENTS net-rc-stream-legacy net-rc-stream-tls
)
find_package(Qt6 REQUIRED COMPONENTS Widgets)
find_package(PkgConfig REQUIRED)
pkg_check_modules(LIBGIT2 REQUIRED IMPORTED_TARGET libgit2)
find_package(Threads REQUIRED)

set(
Expand All @@ -37,6 +39,8 @@ set(
src/codex/DiffViewer.h
src/codex/FrontendSession.cpp
src/codex/FrontendSession.h
src/codex/GitDiffProvider.cpp
src/codex/GitDiffProvider.h
src/codex/FileSelectionDialog.cpp
src/codex/FileSelectionDialog.h
src/codex/MainWindow.cpp
Expand Down Expand Up @@ -65,52 +69,46 @@ set(
)

set(
CODEXUI_GREENFIELD_MIDDLE_SOURCES
src/greenfield/codex/ShellWidget.cpp
src/greenfield/codex/ShellWidget.h
src/greenfield/codex/middle/ComposerPane.cpp
src/greenfield/codex/middle/ComposerPane.h
src/greenfield/codex/middle/ConversationCards.cpp
src/greenfield/codex/middle/ConversationCards.h
src/greenfield/codex/middle/ConversationProjection.cpp
src/greenfield/codex/middle/ConversationProjection.h
src/greenfield/codex/middle/ConversationView.cpp
src/greenfield/codex/middle/ConversationView.h
src/greenfield/codex/middle/InspectorPane.cpp
src/greenfield/codex/middle/InspectorPane.h
src/greenfield/codex/middle/MiddleRegionWidget.cpp
src/greenfield/codex/middle/MiddleRegionWidget.h
src/greenfield/codex/middle/MiddleTypes.cpp
src/greenfield/codex/middle/MiddleTypes.h
src/greenfield/codex/middle/PromptCoordinator.cpp
src/greenfield/codex/middle/PromptCoordinator.h
src/greenfield/codex/middle/ThreadPane.cpp
src/greenfield/codex/middle/ThreadPane.h
CODEXUI_SHELL_SOURCES
src/codex/ShellWidget.cpp
src/codex/ShellWidget.h
src/codex/middle/ComposerPane.cpp
src/codex/middle/ComposerPane.h
src/codex/middle/ConversationCards.cpp
src/codex/middle/ConversationCards.h
src/codex/middle/ConversationProjection.cpp
src/codex/middle/ConversationProjection.h
src/codex/middle/ConversationView.cpp
src/codex/middle/ConversationView.h
src/codex/middle/InspectorPane.cpp
src/codex/middle/InspectorPane.h
src/codex/middle/MiddleRegionWidget.cpp
src/codex/middle/MiddleRegionWidget.h
src/codex/middle/MiddleTypes.cpp
src/codex/middle/MiddleTypes.h
src/codex/middle/PromptCoordinator.cpp
src/codex/middle/PromptCoordinator.h
src/codex/middle/ThreadPane.cpp
src/codex/middle/ThreadPane.h
)

qt_add_executable(
codex-ui
${CODEXUI_CODEX_COMMON_SOURCES}
${CODEXUI_GREENFIELD_MIDDLE_SOURCES}
${CODEXUI_SHELL_SOURCES}
src/codex/PendingRequestDialog.cpp
src/codex/PendingRequestDialog.h
src/codex/TurnSettingsWidget.cpp
src/codex/TurnSettingsWidget.h
)
qt_add_executable(
codex-ui-harness
${CODEXUI_CODEX_COMMON_SOURCES}
src/codex/WorkbenchWidget.cpp
src/codex/WorkbenchWidget.h
)

function(configure_codexui_target target)
target_compile_features(${target} PRIVATE cxx_std_20)
target_include_directories(${target} PRIVATE src)
target_link_libraries(
${target}
PRIVATE
AISuite::OpenAICodex
PkgConfig::LIBGIT2
Qt6::Widgets
Threads::Threads
snodec::net-un-stream-legacy
Expand All @@ -120,33 +118,24 @@ function(configure_codexui_target target)
endfunction()

configure_codexui_target(codex-ui)
target_include_directories(codex-ui BEFORE PRIVATE src/greenfield)
configure_codexui_target(codex-ui-harness)
target_compile_definitions(codex-ui-harness PRIVATE CODEXUI_DEVELOPMENT_HARNESS=1)

if(TARGET snodec::net-in-stream-tls AND TARGET snodec::net-in6-stream-tls)
foreach(target codex-ui codex-ui-harness)
target_compile_definitions(${target} PRIVATE CODEXUI_CODEX_FRONTEND_TLS=1)
target_link_libraries(
${target} PRIVATE snodec::net-in-stream-tls snodec::net-in6-stream-tls
)
endforeach()
target_compile_definitions(codex-ui PRIVATE CODEXUI_CODEX_FRONTEND_TLS=1)
target_link_libraries(
codex-ui PRIVATE snodec::net-in-stream-tls snodec::net-in6-stream-tls
)
endif()

if(TARGET snodec::net-rc-stream-legacy AND TARGET snodec::net-rc-stream-tls)
foreach(target codex-ui codex-ui-harness)
target_compile_definitions(${target} PRIVATE CODEXUI_CODEX_FRONTEND_RFCOMM=1)
target_link_libraries(
${target} PRIVATE snodec::net-rc-stream-legacy snodec::net-rc-stream-tls
)
endforeach()
target_compile_definitions(codex-ui PRIVATE CODEXUI_CODEX_FRONTEND_RFCOMM=1)
target_link_libraries(
codex-ui PRIVATE snodec::net-rc-stream-legacy snodec::net-rc-stream-tls
)
endif()

if(TARGET snodec::http-client AND TARGET snodec::websocket-client)
foreach(target codex-ui codex-ui-harness)
target_compile_definitions(${target} PRIVATE CODEXUI_CODEX_FRONTEND_WEBSOCKET=1)
target_link_libraries(${target} PRIVATE snodec::http-client snodec::websocket-client)
endforeach()
target_compile_definitions(codex-ui PRIVATE CODEXUI_CODEX_FRONTEND_WEBSOCKET=1)
target_link_libraries(codex-ui PRIVATE snodec::http-client snodec::websocket-client)
endif()

if(BUILD_TESTING)
Expand Down Expand Up @@ -207,65 +196,63 @@ if(BUILD_TESTING)
)

add_executable(
codexui-greenfield-projection-test
tests/codex/GreenfieldProjectionTest.cpp
src/greenfield/codex/middle/ConversationProjection.cpp
src/greenfield/codex/middle/ConversationProjection.h
src/greenfield/codex/middle/MiddleTypes.cpp
src/greenfield/codex/middle/MiddleTypes.h
src/greenfield/codex/middle/PromptCoordinator.cpp
src/greenfield/codex/middle/PromptCoordinator.h
codexui-conversation-projection-test
tests/codex/ConversationProjectionTest.cpp
src/codex/middle/ConversationProjection.cpp
src/codex/middle/ConversationProjection.h
src/codex/middle/MiddleTypes.cpp
src/codex/middle/MiddleTypes.h
src/codex/middle/PromptCoordinator.cpp
src/codex/middle/PromptCoordinator.h
)
target_compile_features(
codexui-greenfield-projection-test PRIVATE cxx_std_20
)
target_include_directories(
codexui-greenfield-projection-test BEFORE PRIVATE src/greenfield src
codexui-conversation-projection-test PRIVATE cxx_std_20
)
target_include_directories(codexui-conversation-projection-test PRIVATE src)
target_link_libraries(
codexui-greenfield-projection-test PRIVATE Qt6::Widgets
codexui-conversation-projection-test PRIVATE Qt6::Widgets
)
add_test(
NAME codexui-greenfield-projection
COMMAND codexui-greenfield-projection-test
NAME codexui-conversation-projection
COMMAND codexui-conversation-projection-test
)
set_tests_properties(
codexui-greenfield-projection PROPERTIES TIMEOUT 10
codexui-conversation-projection PROPERTIES TIMEOUT 10
)

qt_add_executable(
codexui-greenfield-middle-test
tests/codex/GreenfieldMiddleTest.cpp
src/greenfield/codex/middle/ConversationCards.cpp
src/greenfield/codex/middle/ConversationCards.h
src/greenfield/codex/middle/ConversationView.cpp
src/greenfield/codex/middle/ConversationView.h
src/greenfield/codex/middle/MiddleTypes.cpp
src/greenfield/codex/middle/MiddleTypes.h
codexui-conversation-cards-test
tests/codex/ConversationCardsTest.cpp
src/codex/middle/ConversationCards.cpp
src/codex/middle/ConversationCards.h
src/codex/middle/ConversationView.cpp
src/codex/middle/ConversationView.h
src/codex/middle/MiddleTypes.cpp
src/codex/middle/MiddleTypes.h
)
target_compile_features(
codexui-greenfield-middle-test PRIVATE cxx_std_20
)
target_include_directories(
codexui-greenfield-middle-test BEFORE PRIVATE src/greenfield src
codexui-conversation-cards-test PRIVATE cxx_std_20
)
target_include_directories(codexui-conversation-cards-test PRIVATE src)
target_link_libraries(
codexui-greenfield-middle-test PRIVATE Qt6::Widgets
codexui-conversation-cards-test PRIVATE Qt6::Widgets
)
add_test(
NAME codexui-greenfield-middle
COMMAND codexui-greenfield-middle-test
NAME codexui-conversation-cards
COMMAND codexui-conversation-cards-test
)
set_tests_properties(
codexui-greenfield-middle
codexui-conversation-cards
PROPERTIES TIMEOUT 15 ENVIRONMENT "QT_QPA_PLATFORM=offscreen"
)

qt_add_executable(
codexui-greenfield-layout-test
tests/codex/GreenfieldLayoutTest.cpp
codexui-application-layout-test
tests/codex/ApplicationLayoutTest.cpp
src/codex/DiffViewer.cpp
src/codex/DiffViewer.h
src/codex/GitDiffProvider.cpp
src/codex/GitDiffProvider.h
src/codex/FileSelectionDialog.cpp
src/codex/FileSelectionDialog.h
src/codex/PresentationModel.cpp
Expand All @@ -276,73 +263,95 @@ if(BUILD_TESTING)
src/codex/TurnSettingsWidget.h
src/codex/ui/ExpandingPromptEditor.cpp
src/codex/ui/ExpandingPromptEditor.h
src/greenfield/codex/middle/ComposerPane.cpp
src/greenfield/codex/middle/ComposerPane.h
src/greenfield/codex/middle/ConversationCards.cpp
src/greenfield/codex/middle/ConversationCards.h
src/greenfield/codex/middle/ConversationView.cpp
src/greenfield/codex/middle/ConversationView.h
src/greenfield/codex/middle/InspectorPane.cpp
src/greenfield/codex/middle/InspectorPane.h
src/greenfield/codex/middle/MiddleRegionWidget.cpp
src/greenfield/codex/middle/MiddleRegionWidget.h
src/greenfield/codex/middle/MiddleTypes.cpp
src/greenfield/codex/middle/MiddleTypes.h
src/greenfield/codex/middle/ThreadPane.cpp
src/greenfield/codex/middle/ThreadPane.h
src/codex/ui/UiStyle.cpp
src/codex/ui/UiStyle.h
src/codex/middle/ComposerPane.cpp
src/codex/middle/ComposerPane.h
src/codex/middle/ConversationCards.cpp
src/codex/middle/ConversationCards.h
src/codex/middle/ConversationView.cpp
src/codex/middle/ConversationView.h
src/codex/middle/InspectorPane.cpp
src/codex/middle/InspectorPane.h
src/codex/middle/MiddleRegionWidget.cpp
src/codex/middle/MiddleRegionWidget.h
src/codex/middle/MiddleTypes.cpp
src/codex/middle/MiddleTypes.h
src/codex/middle/ThreadPane.cpp
src/codex/middle/ThreadPane.h
)
target_compile_features(
codexui-greenfield-layout-test PRIVATE cxx_std_20
)
target_include_directories(
codexui-greenfield-layout-test BEFORE PRIVATE src/greenfield src
codexui-application-layout-test PRIVATE cxx_std_20
)
target_include_directories(codexui-application-layout-test PRIVATE src)
target_link_libraries(
codexui-greenfield-layout-test PRIVATE AISuite::OpenAICodex Qt6::Widgets
codexui-application-layout-test
PRIVATE AISuite::OpenAICodex PkgConfig::LIBGIT2 Qt6::Widgets
)
add_test(
NAME codexui-greenfield-layout
COMMAND codexui-greenfield-layout-test
NAME codexui-application-layout
COMMAND codexui-application-layout-test
)
set_tests_properties(
codexui-greenfield-layout
codexui-application-layout
PROPERTIES TIMEOUT 15 ENVIRONMENT "QT_QPA_PLATFORM=offscreen"
)

set(CODEXUI_GREENFIELD_SHELL_TEST_SOURCES ${CODEXUI_CODEX_COMMON_SOURCES})
qt_add_executable(
codexui-git-changes-live-test
tests/codex/GitChangesLiveTest.cpp
src/codex/DiffViewer.cpp
src/codex/DiffViewer.h
src/codex/GitDiffProvider.cpp
src/codex/GitDiffProvider.h
src/codex/ui/UiStyle.cpp
src/codex/ui/UiStyle.h
)
target_compile_features(codexui-git-changes-live-test PRIVATE cxx_std_20)
target_include_directories(codexui-git-changes-live-test PRIVATE src)
target_link_libraries(
codexui-git-changes-live-test PRIVATE PkgConfig::LIBGIT2 Qt6::Widgets
)
add_test(
NAME codexui-git-changes-live
COMMAND codexui-git-changes-live-test
)
set_tests_properties(
codexui-git-changes-live
PROPERTIES TIMEOUT 30 ENVIRONMENT "QT_QPA_PLATFORM=offscreen"
)

set(CODEXUI_SHELL_TEST_SOURCES ${CODEXUI_CODEX_COMMON_SOURCES})
list(
REMOVE_ITEM CODEXUI_GREENFIELD_SHELL_TEST_SOURCES
REMOVE_ITEM CODEXUI_SHELL_TEST_SOURCES
src/codex/main.cpp
src/codex/MainWindow.cpp
src/codex/MainWindow.h
)
qt_add_executable(
codexui-greenfield-shell-test
tests/codex/GreenfieldShellIntegrationTest.cpp
${CODEXUI_GREENFIELD_SHELL_TEST_SOURCES}
${CODEXUI_GREENFIELD_MIDDLE_SOURCES}
codexui-shell-integration-test
tests/codex/ShellIntegrationTest.cpp
${CODEXUI_SHELL_TEST_SOURCES}
${CODEXUI_SHELL_SOURCES}
src/codex/PendingRequestDialog.cpp
src/codex/PendingRequestDialog.h
src/codex/TurnSettingsWidget.cpp
src/codex/TurnSettingsWidget.h
)
configure_codexui_target(codexui-greenfield-shell-test)
target_include_directories(
codexui-greenfield-shell-test BEFORE PRIVATE src/greenfield
)
configure_codexui_target(codexui-shell-integration-test)
add_test(
NAME codexui-greenfield-shell
COMMAND codexui-greenfield-shell-test
NAME codexui-shell-integration
COMMAND codexui-shell-integration-test
)
set_tests_properties(
codexui-greenfield-shell
codexui-shell-integration
PROPERTIES TIMEOUT 20 ENVIRONMENT "QT_QPA_PLATFORM=offscreen"
)

endif()

install(
TARGETS codex-ui codex-ui-harness
TARGETS codex-ui
RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR}
BUNDLE DESTINATION .
)
Expand Down
Loading