Skip to content

fix(terminal): commit native IME composition safely - #47

Merged
ZacharyZcR merged 8 commits into
Termix-SSH:dev-1.5.0from
ZacharyZcR:fix/native-terminal-ime
Aug 12, 2026
Merged

fix(terminal): commit native IME composition safely#47
ZacharyZcR merged 8 commits into
Termix-SSH:dev-1.5.0from
ZacharyZcR:fix/native-terminal-ime

Conversation

@ZacharyZcR

Copy link
Copy Markdown
Member

Summary

  • replace heuristic hidden TextInput handling with a local Expo native input view
  • send only committed iOS and Android IME text, excluding marked/composing text
  • keep special-key translation and cross-source deduplication in one TypeScript mapping
  • preserve the current remote-desktop protocol session behavior
  • include only the native IME module and required autolinking configuration

This extracts the terminal input work from #38 onto dev-1.5.0. The original implementation commits remain authored by Tink (goxofy). Auth/session persistence, settings, and CI workflow changes were intentionally excluded because they are unrelated or already superseded.

Verification

  • npx tsc --noEmit
  • npm run lint (0 errors; existing warnings only)
  • npm run format:check
  • npx expo-modules-autolinking verify --platform ios
  • npx expo-modules-autolinking verify --platform android
  • resolved module discovery for both iOS and Android
  • git diff --check

Supersedes #38.

Tink and others added 8 commits August 13, 2026 06:17
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
@ZacharyZcR
ZacharyZcR merged commit 168f4ce into Termix-SSH:dev-1.5.0 Aug 12, 2026
2 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.

1 participant