Skip to content

英語TTSで「Seibu」が「さいぶ」と読まれる誤読を端末内蔵TTS向けの読み替え表に追加して修正 - #6876

Merged
TinyKitten merged 1 commit into
devfrom
claude/keiseibus-english-reading-bug-q4db4b
Sep 6, 2026
Merged

英語TTSで「Seibu」が「さいぶ」と読まれる誤読を端末内蔵TTS向けの読み替え表に追加して修正#6876
TinyKitten merged 1 commit into
devfrom
claude/keiseibus-english-reading-bug-q4db4b

Conversation

@TinyKitten

@TinyKitten TinyKitten commented Sep 6, 2026

Copy link
Copy Markdown
Member

概要

英語アナウンスで「Seibu(西武)」が「さいぶ」と読み上げられる誤読を、#6869 で入れた「Keisei → Kay-say」と同じ方式で修正します。端末内蔵 TTS 向けの読み替え表に Seibu → Say-boo を追加します。リモート TTS 向けの同じ置換は TrainLCD/Functions#27 で行います。

変更の種類

  • バグ修正
  • 新機能
  • リファクタリング
  • ドキュメント
  • CI/CD
  • その他

変更内容

  • src/utils/englishReading.ts: ENGLISH_READING_RULES/\bSeibu\b/gi → 'Say-boo' を追加
    • 英語 TTS は "ei" を /aɪ/ と推定して「さいぶ」と読むため、英語の辞書語 "Say" + "boo" で /seɪ.buː/(せいぶ)を確定させます
    • 単語境界で一致させるため、Seibu Ikebukuro Line / Seibu-Shinjuku は置換され、1 語の Seibuen(西武園)は対象外です
  • src/utils/englishReading.test.ts: 上記のケースを追加
  • リモートTTSへは「Keisei」の読み替えを適用せず原文のまま送るよう端末内蔵TTS限定に変更 #6872 の分担どおり、この置換は端末内蔵 TTS(engine: 'native')の経路だけに適用されます。リモート TTS へは原文の「Seibu」のまま送ります

テスト

  • npm run lint が通ること
  • npm test が通ること
  • npm run typecheck が通ること

ローカル実行結果(Node 24.20.0 / npm 11.19.0):

npm run lint      -> Checked 721 files. No fixes applied.
npm test -- src/utils/englishReading.test.ts src/utils/speakableText.test.ts src/hooks/tts
                  -> Test Suites: 6 passed, Tests: 92 passed
npm run typecheck -> エラーなし

関連Issue

スクリーンショット(任意)

UI 変更なし: src/utils/** の読み上げテキスト生成ロジックのみの変更で、画面表示には変化がありません

🤖 Generated with Claude Code

https://claude.ai/code/session_01WS1LSBnYTMfm7CyrzqKxuW


Generated by Claude Code

Summary by CodeRabbit

  • 改善
    • 英語音声読み上げで、「Seibu」を「Say-boo」と発音しやすくなるよう改善しました。
    • 「Seibuen」など、対象外の単語には意図せず適用されません。

@github-actions github-actions Bot added the react label Sep 6, 2026
@TinyKitten TinyKitten self-assigned this Sep 6, 2026
@coderabbitai

coderabbitai Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Essentials

Run ID: 4697bf93-33e8-4462-a2ce-176df359536b

📥 Commits

Reviewing files that changed from the base of the PR and between a6de12e and e6330ef.

📒 Files selected for processing (2)
  • src/utils/englishReading.test.ts
  • src/utils/englishReading.ts

Included review availability: 3 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


📝 Walkthrough

Walkthrough

英語 TTS 向けに、単語境界の SeibuSay-boo に置換する規則を追加しました。通常表記とハイフン連結表記、および Seibuen の非置換をテストします。

Changes

英語読み上げ

Layer / File(s) Summary
Seibu 発音規則と検証
src/utils/englishReading.ts, src/utils/englishReading.test.ts
単語境界にある SeibuSay-boo に置換します。通常表記とハイフン連結表記を検証します。Seibuen は置換しないことを検証します。

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to e6330

This change corrects the built-in English TTS reading of standalone and hyphenated “Seibu” while leaving “Seibuen” unchanged. The covered behavior is ready to merge with no identified material risk.

Poem

うさぎが辞書をひらいたよ
Seibu は Say-boo と読もう
ハイフンつなぎも元気よく
Seibuen には触れないで
TTS の耳へ月明かり
ぴょんと正しい音になる

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed PRの目的、変更内容、変更の種類、テスト結果、関連Issue、UI変更なしの理由を記載しています。テンプレートの必須項目を満たしています。
Title check ✅ Passed タイトルは、端末内蔵TTS向けに「Seibu」の誤読を修正する主要な変更を具体的に示しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 2…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/keiseibus-english-reading-bug-q4db4b

Comment @coderabbitai help to get the list of available commands.

@TinyKitten
TinyKitten merged commit be360b6 into dev Sep 6, 2026
7 checks passed
@TinyKitten
TinyKitten deleted the claude/keiseibus-english-reading-bug-q4db4b branch September 6, 2026 16:21
This was referenced Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants