Skip to content

fix(tui): preserve renderer initialization errors#33996

Merged
nexxeln merged 5 commits into
anomalyco:devfrom
Hona:fix/tui-preserve-renderer-error
Jun 26, 2026
Merged

fix(tui): preserve renderer initialization errors#33996
nexxeln merged 5 commits into
anomalyco:devfrom
Hona:fix/tui-preserve-renderer-error

Conversation

@Hona

@Hona Hona commented Jun 26, 2026

Copy link
Copy Markdown
Member

Summary

  • preserve the original OpenTUI renderer initialization error instead of Effect.tryPromise's generic fallback
  • cover native library loading failures with a lifecycle regression test

Related to #33903 and #32706.

Testing

  • bun test test/app-lifecycle.test.tsx --timeout 30000
  • bun typecheck (packages/tui)
  • bunx prettier --check packages/tui/src/app.tsx packages/tui/test/app-lifecycle.test.tsx
  • pre-push workspace typecheck (29 packages)

The full TUI suite passes 191 tests but retains one unrelated Windows path separator failure in test/runtime.test.tsx: expected ~/project, received ~\project.

Copilot AI review requested due to automatic review settings June 26, 2026 02:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@nexxeln nexxeln enabled auto-merge (squash) June 26, 2026 09:17
@Hona Hona disabled auto-merge June 26, 2026 09:18
@nexxeln nexxeln merged commit 753d312 into anomalyco:dev Jun 26, 2026
8 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.

3 participants