Skip to content

Restore Ghostty Pure prompt redraw fix - #1363

Merged
lawrencecchen merged 3 commits into
mainfrom
task-nightly-pure-prompt-followup
Mar 13, 2026
Merged

lawrencecchen merged 3 commits into
mainfrom
task-nightly-pure-prompt-followup

Conversation

@lawrencecchen

@lawrencecchen lawrencecchen commented Mar 13, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • point the Ghostty submodule at the published Pure prompt redraw fix on manaflow-ai/ghostty main
  • document the restored zsh redraw patch in the Ghostty fork notes
  • add the matching GhosttyKit checksum pin once the xcframework-404a3f175ba6baafabc46cac807194883e040980 release is published

Testing

  • python3 tests/test_ghostty_zsh_pure_hidden_cr_omits_continuation_marker.py
  • python3 tests/test_ghostty_zsh_pure_preprompt_redraw.py
  • python3 tests/test_shell_zsh_prefers_bundled_ghostty_integration.py
  • tagged build pure-followup passed scripts/probe-pure-prompt-duplication.py against a minimal Pure repro home

Dependency

Checklist

  • Ghostty fix published to manaflow-ai/ghostty main
  • GhosttyKit checksum pin added for the published Ghostty SHA
  • manual Pure prompt repro verified locally in tagged cmux build

@vercel

vercel Bot commented Mar 13, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cmux Ready Ready Preview, Comment Mar 13, 2026 0:57am

@greptile-apps greptile-apps Bot 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.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@coderabbitai

coderabbitai Bot commented Mar 13, 2026 •

Copy link
Copy Markdown

Warning

Rate limit exceeded

@lawrencecchen has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 1 minutes and 12 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 9ca3d54f-026f-48f5-a277-90a6ebfcf448

📥 Commits

Reviewing files that changed from the base of the PR and between fd79034 and dfcb633.

📒 Files selected for processing (1)
  • scripts/ghosttykit-checksums.txt
📝 Walkthrough

Walkthrough

The ghostty submodule pointer and documentation were updated to reference a new fork commit (404a3f1) that fixes Pure-style zsh prompt redraw markers; no functional code, public APIs, or control flow were changed.

Changes

Cohort / File(s) Summary
Submodule Reference
ghostty
Updated submodule commit pointer from a50579bd5ddec81c6244b9b349d4bf781f667cec to 404a3f175ba6baafabc46cac807194883e040980. No code or API changes.
Documentation
docs/ghostty-fork.md
Added fork commit 404a3f175 entry and adjusted section heading and summary to describe the Pure hidden-CR redraw fix and restoration of prompt-marker behavior. Minor text edits (+3/-1 lines).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 A hop, a tweak, a tiny new hash,
Pure prompts align with a gentle dash,
Docs whisper the fix, the pointer's set right,
No APIs changed — just a quiet delight. ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: restoring a Ghostty Pure prompt redraw fix, which matches the submodule update and documentation changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed PR description includes all major required sections: Summary, Testing, and Checklist with relevant details about the changes and verification performed.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch task-nightly-pure-prompt-followup
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 1 file

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 099f0a931a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread ghostty Outdated

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@ghostty`:
- Line 1: The ghostty submodule pointer references commit
099f0a931ae0538051d9adba98cef8bc6986f5d1 which is not present on the remote main
branch; push that commit to the manaflow-ai/ghostty remote main branch first
(ensure the commit is part of origin/main on the ghostty repo), then update the
submodule pointer in the parent repo (the ghostty entry) and amend the PR so the
parent repo references the now-published main commit.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 26cfe40d-0468-4dad-b38f-327b1222a1c8

📥 Commits

Reviewing files that changed from the base of the PR and between 09ea29a and 099f0a9.

📒 Files selected for processing (1)
  • ghostty

Comment thread ghostty Outdated
@lawrencecchen

Copy link
Copy Markdown
Contributor Author

Addressed the Ghostty submodule review threads.

  • Ghostty PR by Lawrence Chen merged to manaflow-ai/ghostty main: Fix Pure prompt redraw markers ghostty#12
  • cmux now points at published Ghostty commit 404a3f175ba6baafabc46cac807194883e040980
  • pinned GhosttyKit.xcframework.tar.gz checksum for that exact SHA in scripts/ghosttykit-checksums.txt
  • verified scripts/download-prebuilt-ghosttykit.sh succeeds against the published release asset

@lawrencecchen
lawrencecchen merged commit c0693e1 into main Mar 13, 2026
12 checks passed
@lawrencecchen
lawrencecchen deleted the task-nightly-pure-prompt-followup branch March 13, 2026 12:59
bn-l pushed a commit to bn-l/cmux that referenced this pull request Apr 3, 2026
…e-prompt-followup

Restore Ghostty Pure prompt redraw fix

This branch was successfully deployed

1 active deployment
Preview — dfcb6335 Deployed Mar 13, 2026 by vercel[bot]
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