Skip to content

Render structured log panels with SyntaxEditorUI#31

Closed
lynnswap wants to merge 7 commits into
mainfrom
codex/generic-log-panels-syntax-editor
Closed

Render structured log panels with SyntaxEditorUI#31
lynnswap wants to merge 7 commits into
mainfrom
codex/generic-log-panels-syntax-editor

Conversation

@lynnswap

@lynnswap lynnswap commented Jun 8, 2026

Copy link
Copy Markdown
Owner

Purpose

Render tool, MCP, and app log results through generic log panels instead of command-output-only metadata fields, while using SyntaxEditorUI only as a ReviewUI display dependency.

Changes

  • Replaces result/error metadata with structured ReviewLogEntry.ContentBlock values.
  • Converts app-server object and array results into pretty JSON content blocks and scalar values into plaintext blocks.
  • Consolidates command output and structured result display through ReviewMonitorLogPanel payloads.
  • Embeds SyntaxEditorView(model:) for syntax panels with JSON/plaintext language handling and bounded panel height.
  • Adds SyntaxEditorUI v0.11.0 to the ReviewUI target only.
  • Updates app-server, model, projection, and interaction tests for the new structured panel flow.

Testing

  • swift test --build-system swiftbuild --no-parallel
  • xcodebuild test -project Tools/ReviewMonitor/CodexReviewMonitor.xcodeproj -scheme CodexReviewMonitor -destination 'platform=macOS,arch=arm64' CODE_SIGNING_ALLOWED=NO CODE_SIGNING_REQUIRED=NO
  • git diff --check

@lynnswap lynnswap marked this pull request as ready for review June 8, 2026 13:12
@lynnswap lynnswap closed this Jun 9, 2026
@lynnswap lynnswap deleted the codex/generic-log-panels-syntax-editor branch June 9, 2026 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant