Skip to content

fix: patch ghostty-opentui dist/ for logicalLineInfo crash#94

Merged
hyldmo merged 1 commit intomainfrom
hyldmo/ghostty-opentui-patch
Mar 23, 2026
Merged

fix: patch ghostty-opentui dist/ for logicalLineInfo crash#94
hyldmo merged 1 commit intomainfrom
hyldmo/ghostty-opentui-patch

Conversation

@hyldmo
Copy link
Copy Markdown
Owner

@hyldmo hyldmo commented Mar 23, 2026

Summary

  • Extend the ghostty-opentui patch to also cover dist/terminal-buffer.js — the existing patch only fixed src/ but Bun can resolve to dist/ in some setups, leaving the logicalLineInfo null guard unapplied
  • Print full stack traces (instead of just messages) in unhandled rejection handlers for better debugging

Upstream fix: remorses/ghostty-opentui#15

Test plan

  • Run numux with a project that triggers the logicalLineInfo crash — should no longer crash
  • Verify bun install applies the patch cleanly to both src/ and dist/

🤖 Generated with Claude Code

The existing patch only covered src/ but Bun can resolve to dist/
in some setups. Also print stack traces instead of just messages
in unhandled rejection handlers.

Upstream: remorses/ghostty-opentui#15

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@hyldmo hyldmo merged commit 4758f60 into main Mar 23, 2026
@hyldmo hyldmo deleted the hyldmo/ghostty-opentui-patch branch March 23, 2026 17:32
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 2.10.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant