Skip to content

feat: ios16 support#2469

Open
Yash-Singh1 wants to merge 1 commit intot3code/mobile-remote-connectfrom
yash/ios16-mobile
Open

feat: ios16 support#2469
Yash-Singh1 wants to merge 1 commit intot3code/mobile-remote-connectfrom
yash/ios16-mobile

Conversation

@Yash-Singh1
Copy link
Copy Markdown
Collaborator

@Yash-Singh1 Yash-Singh1 commented May 2, 2026

What Changed

Adds ios 16 support to the GhosttyKit library used in mobile app

Why

So we support ios 16

Checklist

  • This PR is small and focused
  • I explained what changed and why

Note

Medium Risk
Updates a vendored native terminal rendering framework and its C/Swift integration points, which could introduce runtime/ABI issues on iOS if the new Ghostty build diverges. Dependency/lockfile changes (adding shiki) also affect build output and dependency resolution.

Overview
Adds iOS 16 support for the mobile terminal by switching the vendored GhosttyKit.xcframework to a new fork/revision and updating the vendored ghostty.h API surface (notably adding GHOSTTY_API visibility macros and changing read_clipboard_cb to return bool).

Updates T3TerminalView.swift to match the new callback signature, refreshes Ghostty vendoring/license docs, and introduces scripts/build-libghostty-ios16.sh to rebuild and replace the iOS slices/headers of GhosttyKit.xcframework. Also pins [email protected] in apps/mobile/package.json with corresponding bun.lock updates.

Reviewed by Cursor Bugbot for commit 2e04b58. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Add iOS 16 support to the mobile terminal using a custom Ghostty fork

  • Updates vendored GhosttyKit.xcframework to a custom-I/O fork of Ghostty with an iOS 16 compatibility patch (revision d36c3b8).
  • Adds build-libghostty-ios16.sh, a script to rebuild the xcframework from source using Zig 0.15.2, replacing only the ios-arm64 and ios-arm64-simulator slices.
  • Updates T3TerminalView.swift to return false from the read_clipboard_cb closure to match the updated ghostty_runtime_read_clipboard_cb signature (changed from void to bool).
  • Adds [email protected] as a dependency and updates the lockfile with scoped resolutions to avoid conflicts with packages expecting [email protected].

Macroscope summarized 2e04b58.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 2, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 13f208f5-32ff-4952-ade0-2d776c3093da

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch yash/ios16-mobile

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.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XL 500-999 changed lines (additions + deletions). labels May 2, 2026
@macroscopeapp
Copy link
Copy Markdown
Contributor

macroscopeapp Bot commented May 2, 2026

Approvability

Verdict: Needs human review

This PR updates a vendored native iOS library (GhosttyKit) with iOS 16 compatibility patches. Vendored binary updates cannot be fully verified through code review alone, and this enables new platform support, warranting human review.

You can customize Macroscope's approvability policy. Learn more.

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

Labels

size:XL 500-999 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant