日本語入力直後にキーボード操作で文字を選択すると選択した文字が消える

[UMA0626]

Description

掲題のとおり

Steps to reproduce

1. UbuntuにGoogle Chromeとmozcをインストールする
   バージョンは後述
2. 設定 > システム > 地域と言語 > Manage Installed Languages > キーボード入力に使うIMシステム をIBusに設定し、再起動
3. Chromeで、テキストボックスが存在する任意のWebページを表示する(Googleなど)
4. テキストボックスにMozcで日本語を入力する
   このとき、以下をしない
   • 変換
   • 半角/全角切り替え
   • テキストボックスからフォーカスを外す
5. キーボード操作によって入力した文字を選択する
   Ctrl + A, Shift + ←, Shift + HOMEなど

Expected behavior

文字が消えずに正しく選択される

Actual behavior

選択した文字が消える

• Ctrl + Aで全選択した場合、すべての文字が消える
• Shift + ←でカーソルの左の1文字を選択した場合、その文字が消える
  続けてShift + ←を押すと、残りの文字は正しく選択できる
• Shift + HOMEでカーソルの左の全文字を選択した場合、その文字が消える

Screenshots

1. 「noproblem」と入力し、Ctrl + A: 問題なく全選択される
2. 「にほんご」と入力し、Ctrl + A: 全削除される
3. 再度「にほんご」と入力し、Ctrl + ←: 「ご」だけ削除される
4. そのまま続けてCtrl + ←: 以降は正しく選択される
5. 「ばーじょん」を変換した後にCtrl + A: 問題なく全選択される

Screencast.from.2025-11-20.14-17-01.webm

Version or commit-id

Mozc-2.32.5994.102+24.11.oss

Environment

• OS: Ubuntu 24.04.3 LTS
• IMF (for Linux): iBus
• Related Applications
  • Google Chrome: 142.0.7444.175（Official Build） （64 ビット）

Investigations

• Whether this issue happens on both Ibus and other IMF (e.g. Fcitx, uim).
  • no (fcitxでは発生しなかった)
• Whether this issue happens on other IMEs (e.g. Anthy, SKK).
  • N/A (試していない)
• What applications this issue happens on (e.g. Chromium, gedit).
  • Google Chrome (Chromium)
• What applications this issue does not happen on (e.g. Chromium, gedit).
  • Firefox
  • Sublime Text
  • VS Code

Additional context

本事象はChromeによって表示されたテキストボックス上で発生しており、
URLバーへの入力やブックマークの編集などでは発生しなかった。

AIによる見解
※私はC++, linux, IME, iBus, mozcに詳しくないので下記見解が正しそうか自己検証できていませんが、参考までに記載します 🙇

原因:
UpdateDeletionRange が output.consumed() を確認せず、deletion_range があれば常に DeleteSurroundingText を呼んでいます。
EchoBack で consumed(false) でも、deletion_range が残っていると削除が実行されます。

修正:
UpdateDeletionRange で output.consumed() が false の場合は deletion_range を無視するようにします。

[yukawa]

Thank you for the detailed report.

So far this doesn't look to be something we can fix in the Mozc side, but if someone happens to find the root cause then figure out how to fix this in the Mozc side without causing any side effect, we are happy to review pull requests.

I've converted this to discussion so that other contributors can continue posting their findings and/or new information.

For those who decide to try to look into this issue, here are my suggestions:

• First, let's see if the issue is reproducible with other IBus-based IMEs such as ibus-anthy and ibus-skk. If the same issue is reproducible, then it confirms that the issue needs to be fixed in somewhere else.
• The second step might be to see the difference between Chrome and VSCode, as those two apps share lots of code. If the issue started happening in a recent version of Chromium, then figuring out which commit introduced the behavior change would be super helpful.

Hope this helps.

[UMA0626]

ありがとうございます！

AIにコードを修正させてみたところ、以下変更をしました。

bool MozcEngine::UpdateDeletionRange(IbusEngineWrapper *engine,
                                     const commands::Output &output) {
+  if (!output.consumed()) {
+    return true;
+  }
  if (output.has_deletion_range() && output.deletion_range().offset() < 0 &&
      output.deletion_range().offset() + output.deletion_range().length() >=
          0) {
    engine->DeleteSurroundingText(output.deletion_range().offset(),
                                  output.deletion_range().length());
  }
  return true;
}

上記をビルドして私の環境で試したところ、事象が解消しました！
このコード変更に他の副作用があるか未確認なので、参考までに記載しておきます 🙇

ibus-anthy, ibus-skkでの確認も、手が空いたときに試してみます 👍

[yukawa]

Thanks! That's super helpful information.

@hiroyuki-komatsu Can you confirm the behavior that output.consumed() is false while output.has_deletion_range() is true? If yes, I suppose we may want to address this in the mozc_server side.

[hiroyuki-komatsu]

Thank you @UMA0626 ! Your information is very helpful.

I'm now wondering why deletion_range is filled in this situation.

@UMA0626, could you please help us by adding logs and checking them?

Adding logs

UpdateDeletionRange is only called by UpdateAll. So I'd like to check which UpdateAll invokes it. Please add a log command before UpdateAll

void MozcEngine::CandidateClicked(IbusEngineWrapper *engine, uint index,
                                  uint button, uint state) {
  if (index >= unique_candidate_ids_.size()) {
    return;
  }
  const int32_t id = unique_candidate_ids_[index];
  if (id == kBadCandidateId) {
    return;
  }
  commands::Output output;
  commands::SessionCommand command;
  command.set_type(commands::SessionCommand::SELECT_CANDIDATE);
  command.set_id(id);
  client_->SendCommand(command, &output);
+  LOG(INFO) << "CandidateClicked: " << output;
  UpdateAll(engine, output);
}

bool MozcEngine::ProcessKeyEvent(IbusEngineWrapper *engine, uint keyval,
                                 uint keycode, uint modifiers) {
...
  commands::Output output;
  if (!client_->SendKeyWithContext(key, context, &output)) {
    LOG(ERROR) << "SendKey failed";
    return false;
  }

  MOZC_VLOG(2) << output;
+  LOG(INFO) << "ProcessKeyEvent: " << output;
  UpdateAll(engine, output);

  return output.consumed();
}

void MozcEngine::RevertSession(IbusEngineWrapper *engine) {
  // TODO(team): We should skip following actions when there is no on-going
  // composition.
  commands::SessionCommand command;
  command.set_type(commands::SessionCommand::REVERT);
  commands::Output output;
  if (!client_->SendCommand(command, &output)) {
    LOG(ERROR) << "RevertSession() failed";
    return;
  }
+  LOG(INFO) << "RevertSession: " << output;
  UpdateAll(engine, output);
}

bool MozcEngine::ExecuteCallback(IbusEngineWrapper *engine,
                                 const commands::Output &output) {
...

  // Here uses recursion of UpdateAll but it's okay because the converter
  // ensures that the second output never contains callback.
+  LOG(INFO) << "ExecuteCallback: " << output;
  UpdateAll(engine, new_output);
  return true;
}

Checking logs

Logs will be available at ~/.config/mozc/ (or ~/.mozc/). Please let us know the result of the above change.

Thank you!

[yukawa]

Logs will be available at ~/.config/mozc/ (or ~/.mozc/). Please let us know the result of the above change.

Jus fyi, I guess to enable logging we need to build Mozc without --config release_build. One of the most reliable way to enable logging might be just building Mozc as debug build as follows.

bazelisk build package --config oss_linux --compilation_mode dbg

[UMA0626]

I'll check later, thank you!