Skip to content

TTSキャッシュキーにjaVoiceNameを追加#5593

Merged
TinyKitten merged 1 commit intodevfrom
feature/fix-tts-ja-voice-cache-key
Mar 12, 2026
Merged

TTSキャッシュキーにjaVoiceNameを追加#5593
TinyKitten merged 1 commit intodevfrom
feature/fix-tts-ja-voice-cache-key

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Mar 12, 2026

Summary

  • buildCacheKeyjaVoiceName が含まれていなかったため、日本語ボイスを変更しても同じテキストに対してキャッシュがヒットし、古いボイスの音声が再生され続ける不具合を修正
  • キャッシュキーに jaVoiceName を追加

Test plan

  • 日本語ボイスを変更した後、TTS再生で新しいボイスが反映されることを確認
  • 英語ボイスの変更が引き続き正常に反映されることを確認

🤖 Generated with Claude Code

Summary by CodeRabbit

リリースノート

このプルリクエストでは内部的なキャッシュロジックの最適化が行われました。エンドユーザーに対する機能的な変更や影響はございません。

@github-actions github-actions bot added the react label Mar 12, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 12, 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: 91d3ddb3-90b1-4a28-91d7-f5cf62348849

📥 Commits

Reviewing files that changed from the base of the PR and between bea5927 and 39b2ccc.

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

📝 Walkthrough

歩き方

TTSスピーチフェッチャーのキャッシュキー生成ロジックが更新されました。キャッシュキーにjaVoiceNameが新たに含められるようになり、日本語音声による区別が追加されました。公開APIには変更なく、内部キャッシング処理のセマンティクスのみが影響を受けます。

変更

コホート / ファイル 概要
TTSキャッシュキー生成
src/utils/ttsSpeechFetcher.ts
キャッシュキー生成ロジックにjaVoiceNameを追加。既存のtextJatextEnenVoiceNameに加え、日本語音声名でキャッシュエントリを区別するように変更。

関連する可能性のあるPR

  • tts関数の改修 #4472: TTSキャッシュキー生成においてjaVoiceNameを含めるよう修正する同様の変更が実装されており、直接関連性があります。

🐰 ウサギからのお祝い

キャッシュの鍵に声を添えて、
日本語の音も忘れずに。
細かな工夫が大きな力、
音声の違いが今ここに輝く!🎵

推定コードレビュー工数

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

🚥 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 プルリクエストのタイトルは、変更内容の主要ポイント(TTSキャッシュキーにjaVoiceNameを追加)を明確かつ簡潔に説明しており、ファイルサマリーとPRの目的と完全に関連している。
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/fix-tts-ja-voice-cache-key

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

@TinyKitten TinyKitten self-assigned this Mar 12, 2026
@TinyKitten TinyKitten merged commit e8c029f into dev Mar 12, 2026
7 checks passed
@TinyKitten TinyKitten deleted the feature/fix-tts-ja-voice-cache-key branch March 12, 2026 10:37
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