Skip to content

fix(tui): restore integrated Design and reliable LSP recovery - #171

Merged
filipeforattini merged 6 commits into
mainfrom
restore-design-tui
Sep 8, 2026
Merged

fix(tui): restore integrated Design and reliable LSP recovery#171
filipeforattini merged 6 commits into
mainfrom
restore-design-tui

Conversation

@filipeforattini

@filipeforattini filipeforattini commented Sep 8, 2026

Copy link
Copy Markdown

Restore Design as a cyan primary mode in the fullscreen TUI. Tab and /design select it in the current conversation; /design-review opens that conversation's browser review. The TUI retains its session runner, permissions and questions while using the newer Design revision, asset, rendering, whiteboard and SVG-to-GIF services. Pre-0.22 prototypes can be imported without overwriting their source or private review files. Approval hands the exact revision to Plan in the same conversation and respects Design-only goals.

Fix Biome and other Node LSP initialization failures caused by rejected NODE_OPTIONS. Recovery now reads structured initialization stderr, isolates bounded retries per server, preserves quoted options, and monitors the replacement process rather than the exited original.

Validation:

  • 149 focused Redcode tests passed; an additional queued-feedback interruption regression passed.
  • 22 Core Design/rendering/MCP tests passed using the installed Chromium cache.
  • TUI mode tests and all three app lifecycle tests passed.
  • Real fullscreen TUI plus browser: create/write/publish, feedback, whiteboard, SVG-to-GIF download (128px, six frames), restart, and approval continuing in Plan.
  • Real Biome 2.5.12 initialized with the rejected NODE_OPTIONS after recovery.
  • Native Linux build, fullscreen TUI, preview, same-session feedback and SVG-to-GIF passed outside the checkout. Package typechecks passed for Redcode, TUI, Core and Design.

The review shell now has compact toolbars, theme-aware cyan controls and keyboard-accessible Review/Assets/Details tabs. CSS stays inside Shadow DOM, with the prototype isolated in its iframe; no Tailwind or global reset is added. Browser regression coverage verifies style isolation, desktop viewport sizing and mobile overflow.

Includes patch changesets. Release versions are left to the Version PR.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@filipeforattini
filipeforattini enabled auto-merge (squash) September 8, 2026 14:01
@filipeforattini
filipeforattini merged commit 5ce06f9 into main Sep 8, 2026
13 checks passed
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