chore(deps): Drop SwiftTerm fork — all fixes now upstream#102
Merged
Conversation
Replace per-file exclusions inside TermQ.app/ with a top-level ignore for the entire directory, which is fully generated at build time. Also removes previously-tracked Info-Debug.plist and AppIcon.icns. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
All three fixes from eyelock/SwiftTerm are now in migueldeicaza/SwiftTerm main: - Cursor ghosting (DECTCEM / CaretView): landed independently in upstream Ref: migueldeicaza/SwiftTerm#452 - EV_VANISHED crash (childMonitor race): PR #401 by ipodishima (Oct 2025) Ref: migueldeicaza/SwiftTerm#401 - PTY resize / Auto Layout: PR #469 by colemaco (Feb 2026) Ref: migueldeicaza/SwiftTerm#469 The eyelock/SwiftTerm fork was never wired into Package.swift — these plans were documented but not executed. The fork can now be archived. Also tightens MCP swift-sdk lower bound to 0.12.0, matching the API used by tests since commit 48c5823. Updates ToolHandlers.swift to use the new non-deprecated Tool.Content.text(text:annotations:_meta:) enum case. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The .gitignore simplification in the previous commit excluded the entire TermQ.app/ bundle, but Info-Debug.plist is a source template (not a build artifact) that the Makefile depends on. Move it to Config/ and update the INFO_PLIST reference accordingly. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
All three fixes staged in
eyelock/SwiftTermhave landed inmigueldeicaza/SwiftTermupstream. The fork was never wired intoPackage.swift(plans were documented but never executed), so this PR simply advances the upstream pin and cleans up.The three fixes now in upstream
setFrameSizenot callingprocessSizeChange)Changes
75d0fd9→b6ce28a4(post-v1.13.0, Apr 6 2026 — all three fixes included)0.9.0→0.12.0(tightened to match the API already used by tests since commit48c5823)Tool.Content.text(text:annotations:_meta:)enum case (required by the sdk bump)Follow-up
After merge,
eyelock/SwiftTermcan be archived via Settings → Danger Zone.The
fix/tmux-panelsbranch should be rebased onto updated main (git rebase origin/main—Package.swiftwas not modified on that branch so the rebase should be clean).Test plan
make build— passes, zero errors, zero deprecation warningsmake lint— 0 serious violationsmake format-check— passes (warnings are pre-existing)make test— all tests pass🤖 Generated with Claude Code