Skip to content

phonemeタグにxml:lang属性を追加してTTSの日本語発音を改善#5527

Merged
TinyKitten merged 1 commit intodevfrom
feature/tts-phoneme-xml-lang
Mar 6, 2026
Merged

phonemeタグにxml:lang属性を追加してTTSの日本語発音を改善#5527
TinyKitten merged 1 commit intodevfrom
feature/tts-phoneme-xml-lang

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Mar 6, 2026

Summary

  • wrapIpa が生成する <phoneme> タグに xml:lang="ja-JP" 属性を追加
  • en-USボイスで英語表記の駅名(例: Kasai Rinkai Park)を読み上げる際、IPAが日本語の音素体系として解釈されるようにヒントを付与
  • ローマ字表記と日本語表記が異なる駅名(葛西臨海公園 / Kasai Rinkai Park 等)で正しい日本語発音が得られることを期待

Test plan

  • npm test パス確認済み
  • 実機でTTS再生し、日本語駅名の発音が改善されていることを確認

🤖 Generated with Claude Code

Summary by CodeRabbit

リリースノート

  • 改善
    • 音声合成における言語タグの指定精度を改善しました。日本語コンテンツの処理がより正確になります。

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions github-actions bot added the react label Mar 6, 2026
@TinyKitten TinyKitten self-assigned this Mar 6, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 6, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e27a4d1e-eabb-4340-922d-17f54e4efe92

📥 Commits

Reviewing files that changed from the base of the PR and between 3e3b906 and c5a81cd.

📒 Files selected for processing (1)
  • src/utils/ssml.ts

📝 Walkthrough

ウォークスルー

src/utils/ssml.tswrapIpa 関数が変更され、nameIpanameRoman の両方が提供される場合、返される phoneme タグに xml:lang="ja-JP" 属性が明示的に追加されました。周辺のロジックやエラー処理は変わっていません。

変更内容

コホート / ファイル 概要
SSML 言語属性追加
src/utils/ssml.ts
wrapIpa 関数の phoneme タグに xml:lang="ja-JP" 属性を追加し、言語指定を明示化

コード レビューの推定工数

🎯 2 (シンプル) | ⏱️ ~5 分

関連する可能性のある PR

提案ラベル

react

🐰 ウサギからのお祝い詩

言葉には地域の香り✨
日本語タグで、より正確に
xml:lang が輝く phoneme タグ
小さな変更だけど、大きな気配り🌸

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed タイトルは、phonemeタグにxml:lang属性を追加してTTSの日本語発音を改善する主な変更内容を正確に反映しており、変更セット全体と完全に関連している。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/tts-phoneme-xml-lang

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

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.

1 participant