アプリ指定の読み上げ速度を受け付け既定速度を等速へ変更

[TinyKitten]

概要

/tts がアプリ指定の読み上げ速度を受け付けるようにし、TTS_SPEED の既定値を dev / 本番とも等速（1.0）へ変更する。

アプリ側で「アナウンス速度」（ゆっくり / 普通 / 早め）を選べるようにする変更（TrainLCD/MobileApp 側の PR）と対になる。これまで読み上げ速度は Worker の環境変数（dev 1.15 / 本番 1.1）に固定で、利用者が調整する手段が無かった。

変更の種類

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

変更内容

• src/lib/google/tts.ts: parseClientSpeed と ALLOWED_CLIENT_SPEEDS（0.85 / 1.0 / 1.15）を追加。リクエスト指定の速度を正規化する。
• src/routes/tts.ts: TtsRequest.speed を追加し、parseClientSpeed(data.speed) ?? parseSpeed(env.TTS_SPEED) で解決する。
• wrangler.jsonc: TTS_SPEED を dev 1.15 → 1.0、本番 1.1 → 1.0 へ変更。

設計上の判断

• 許可リスト方式にした。parseSpeed の許容範囲（0.25〜4.0）をそのまま開放すると、同じ文が速度違いで R2 / KV へ無制限に積み上がり、合成の文字数課金も増える。アプリのプリセットと一対一で対応する3値だけを受理し、範囲外は「未指定」として扱う。
• プリセット外・未指定はエラーにしない。速度を送ってこない旧バージョンのアプリが正常系なので、環境変数の既定値へ倒す。
• 既定値を等速に揃えた。アプリ側の設定の既定が「普通」（1.0）なので、速度を送ってこないアプリの体感と一致させる。あわせて dev と本番で値が異なる状態（1.15 / 1.1）も解消し、開発時の体感が本番とズレないようにした。

リグレッションリスクと緩和

• 既存の利用者の体感速度が変わる: 本番は 1.1 から 1.0 へ落ちる。意図した変更で、アプリ側の設定既定「普通」と一致する。速めが好みの利用者はアプリ側で「早め」（1.15）を選べる。
• キャッシュの取り違えは起きない: 速度は既に computeId の入力に含まれているため、値が変われば別 ID になる。HASH_VERSION の更新は不要。
• 旧バージョンのアプリへの影響: speed を送らないリクエストは従来どおり環境変数の既定値で合成される。互換性は壊れない。
• 既存キャッシュ: 1.1 / 1.15 で合成済みの音声は参照されなくなるが残存する。掃除するなら npm run find-orphaned-tts。

テスト

npm run lint / npm test（16 suites / 201 tests）/ npm run typecheck をローカルで実行し、すべて成功することを確認済み。

追加したテスト（src/lib/google/tts.test.ts）:

• プリセット3値を数値・文字列のいずれでも受理すること

• プリセット外（0.9 / 1.05 / 1.2 / 2 / 0.25 / 4）を無視すること

• 未指定・不正値（null / '' / 'fast' / {} / [] / NaN / Infinity）を「未指定」として扱うこと

• npm run lint が通ること

• npm test が通ること

• npm run typecheck が通ること

関連Issue

対になるアプリ側の PR: TrainLCD/MobileApp#6737

デプロイ順序

アプリ側の変更より 先に デプロイする。逆順でもアプリが送る speed が無視されるだけで害はないが、設定が効かない期間ができる。

Summary by CodeRabbit

• 新機能

  • 読み上げ速度を「遅め（0.85倍）」「標準（1.0倍）」「速め（1.15倍）」から指定できるようになりました。
  • 無効な速度指定は自動的に標準速度へ戻ります。

• 変更

  • 開発環境・本番環境の標準読み上げ速度を1.0倍に統一しました。

[coderabbitai]

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: a9cf8817-d9fc-4294-b9bf-74c650ec4ccb

📥 Commits

Reviewing files that changed from the base of the PR and between 7370018 and 9b0566c.

📒 Files selected for processing (4)

• src/lib/google/tts.test.ts
• src/lib/google/tts.ts
• src/routes/tts.ts
• wrangler.jsonc

Limit details: You’ve used the included review currently available. Your 60 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

---

📝 Walkthrough

Walkthrough

クライアント指定の読み上げ速度に、0.85、1.0、1.15 のプリセットを追加しました。TTSルートは有効な速度を優先し、無効または未指定の場合は環境変数の既定値を使用します。

Changes

クライアント読み上げ速度

Layer / File(s)	Summary
速度プリセットの解析契約 src/lib/google/tts.ts, src/lib/google/tts.test.ts	ALLOWED_CLIENT_SPEEDS と parseClientSpeed を追加しました。数値または文字列を解析し、許可されたプリセット以外は undefined にします。
TTSルートと既定値の統合 src/routes/tts.ts, wrangler.jsonc	TtsRequest.speed を受け付けます。有効なクライアント指定を優先し、選択した速度をキャッシュキーに含めます。dev と production の既定値を 1.0 に統一しました。

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: ⚪ Minimal · up to 9b056

The change adds selectable speech speeds and aligns the default speed across environments; no actionable merge-blocking risk remains after normal checks and review.

Sequence Diagram(s)

sequenceDiagram
  participant Client
  participant TtsRequest
  participant parseClientSpeed
  participant computeId
  Client->>TtsRequest: speed を送信
  TtsRequest->>parseClientSpeed: speed を解析
  parseClientSpeed-->>TtsRequest: プリセット速度または undefined
  TtsRequest->>computeId: 選択した速度を渡す
  computeId-->>TtsRequest: 速度を含むキャッシュキー

Loading

Poem

うさぎは速度を耳で聞く
速さの札は三つ並ぶ
無効な値は月へ返す
キャッシュの道に速度を刻む
TTSの声が整って響く

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
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 3…
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.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	タイトルは、アプリ指定の読み上げ速度への対応と既定速度を1.0へ変更する主要な変更内容を明確に要約しています。

Full details: Docstring Coverage

Explanation

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 3 files. (1 skipped: 1 unsupported.)

✨ 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 feature/tts-speed-preference

---

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