Skip to content

Fix Android IME input and tmux clipboard copying - #198

Merged
bifrost0x merged 3 commits into
mainfrom
dev/fix-touch-input-and-tmux-copy
Sep 4, 2026
Merged

Fix Android IME input and tmux clipboard copying#198
bifrost0x merged 3 commits into
mainfrom
dev/fix-touch-input-and-tmux-copy

Conversation

@bifrost0x

Copy link
Copy Markdown
Owner

Summary

  • reset stale xterm helper input before Android IME composition starts, preventing duplicated display text and truncated committed input
  • accept bounded, strict UTF-8 OSC 52 clipboard updates only for confirmed persistent tmux sessions
  • keep plain SSH sessions isolated from remote OSC 52 clipboard writes and leave remote tmux configuration unchanged
  • document the persistent-tmux clipboard behavior and browser permission boundary

Validation

  • node --test tests/js/*.test.js (38/38 test files passed)
  • playwright test tests/e2e/session-workspace.spec.js (20/20 passed)
  • pytest -q (2,352 passed, 29 skipped; four loopback tests rerun with socket access)
  • eslint static/js
  • vendored frontend asset check
  • authentication i18n generated-file check
  • git diff --check

Closes #196
Closes #197

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-04T03:51:57.587156Z 32f4b5b Manual request
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@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: 3bb176e3de

ℹ️ 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 static/js/session-manager.js
@bifrost0x

Copy link
Copy Markdown
Owner Author

@codex review

@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: c00348ef2b

ℹ️ 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 static/js/terminal-manager.js Outdated
@bifrost0x

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Delightful!

Reviewed commit: 32f4b5b92f

ℹ️ 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".

@bifrost0x
bifrost0x merged commit 081db5e into main Sep 4, 2026
20 of 21 checks passed
@bifrost0x
bifrost0x deleted the dev/fix-touch-input-and-tmux-copy branch September 4, 2026 04:02
@github-project-automation github-project-automation Bot moved this from Backlog to Done in WebSSH Roadmap Sep 4, 2026
@bifrost0x bifrost0x self-assigned this Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

1 participant