Skip to content

SOL の端末で Edge の書く手伝いを切る - #91

Merged
5ym merged 1 commit into
mainfrom
sol-no-writing-assist
Nov 2, 2026
Merged

5ym merged 1 commit into
mainfrom
sol-no-writing-assist

Conversation

@5ym

@5ym 5ym commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

概要

SOL の端末で、xterm.js がキー入力を受ける隠れた textarea に次の属性を付けた。

  • writingsuggestions="false"
  • spellcheck="false"
  • autocomplete / autocorrect / autocapitalize を off

背景

端末の左上に出ていた青い点の正体は、Edge の「Ask Edge to help me write」(書く手伝い)の目印だった。この欄はブラウザに「文章を書く欄」とみなされていたので、書く手伝い・スペルチェック・自動補完を切った。

#90 で欄を画面の外に置いたのは、保険として残す。style.css のコメントも正体に合わせて直した。

確かめたこと

  • bun x tsc --noEmit・bun test web・bun run lint・bun run build:web が通る
  • 青い点は headless のブラウザでは出ないので、実機の Edge で確かめる

🤖 Generated with Claude Code

https://claude.ai/code/session_01T8TqVdo6LUrWKdVQgbScVU

xterm.js の隠れた textarea に writingsuggestions=false・spellcheck=false・
autocomplete/autocorrect/autocapitalize=off を付け、Edge の「Ask Edge to help me write」と
その目印(端末の左上の青い点)が出ないようにした。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T8TqVdo6LUrWKdVQgbScVU
@claude

claude Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Claude finished @5ym's task in 14s —— View job


レビュー結果

  • 差分の確認 (web/amt/view.ts, web/style.css)
  • 不具合 / セキュリティ / 設計 / テストの観点で確認

結論: 問題ありません。

不具合

  • term.open() の後に term.textarea を参照しており、順序は正しいです。if (term.textarea) で undefined も防いでいます。
  • writingsuggestions="false" は Edge と Chromium の標準属性です。未対応のブラウザでは無視されるだけなので、副作用はありません。

セキュリティ

  • 入力値や権限に関わる変更はありません。

設計

  • 画面外に置く CSS (SOL の入力欄を画面の外に置き、つないでいる間はページを巻かない #90) を保険として残し、コメントだけ実際の原因に合わせて直しています。この判断は妥当です。
  • 【要確認・任意】xterm.js は自前で textarea に autocorrect, autocapitalize, spellcheck を設定している版が多いため、そのうち 3 つは重複するかもしれません。害はなく、使っている xterm の版で挙動を変えられても困らないので、残して問題ありません。

テスト

  • DOM 属性を付けるだけの変更で、青い点は headless では再現できないため、自動テストが無いのは妥当です。説明にあるとおり、実機の Edge で確認してください。この確認だけは未完了です。

@5ym
5ym merged commit 8a444d6 into main Nov 2, 2026
7 checks passed
@5ym
5ym deleted the sol-no-writing-assist branch November 2, 2026 12:14
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