Skip to content

TTS設定画面からAndroid 16判定ロジックを撤廃#5405

Merged
TinyKitten merged 1 commit intodevfrom
claude/remove-android-16-tts-WEkr5
Feb 22, 2026
Merged

TTS設定画面からAndroid 16判定ロジックを撤廃#5405
TinyKitten merged 1 commit intodevfrom
claude/remove-android-16-tts-WEkr5

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Feb 22, 2026

https://claude.ai/code/session_01QJxW4UaH8moFYJvMQiL6rb

Summary by CodeRabbit

リリースノート

  • 改善

    • Android 16以上でのバックグラウンド音声再生の制限を削除しました。
    • テキスト読み上げ機能の設定ロジックを簡素化しました。
  • その他の変更

    • 不要な翻訳エントリを削除しました。

@github-actions github-actions bot added the react label Feb 22, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 22, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

ウォークスルー

Android 16以降の背景音声再生に関する条件分岐ロジックを完全に削除しました。翻訳ファイルから関連メッセージキーを削除し、Permitted.tzaとTTSSettings.tsxから版チェック処理とそれに伴うUI条件分岐を削除しました。

変更内容

コホート / ファイル(s) 概要
翻訳ファイル
assets/translations/en.json, assets/translations/ja.json
Android 16以降の背景TTS利用不可メッセージを示すbgTtsUnavailableOnAndroid16キーを削除。
背景TTS永続化ロジック
src/components/Permitted.tsx
Android版チェック(isAndroid16OrHigher)とそれに基づく条件分岐を削除。backgroundEnabledを常にbgTTSEnabledStr === 'true'から直接設定し、BG_TTS_ENABLED向けのAsyncStorage書き込み処理を削除。
TTS設定UI
src/screens/TTSSettings.tsx
Android版依存のロジック(isAndroid16OrHigherの計算、useEffect、AsyncStorage操作)およびAndroid 16+向けの条件付きUIメッセージを削除。PlatformとisDevAppのインポートをクリーンアップし、UI表示をspeechEnabledのみに依存するよう簡潔化。

推定レビュー工数

🎯 2 (Simple) | ⏱️ ~12分

関連する可能性のあるPR

推奨ラベル

react, android

ポエム

🐰 Android版の檻を外して
背景音声は自由に流れ
翻訳も条件分岐も削られて
コードはシンプル、そして軽い
ウサギが喜ぶ、すっきり整理

🚥 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設定画面からAndroid 16判定ロジックを撤廃」を明確に要約している。
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 claude/remove-android-16-tts-WEkr5

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

@TinyKitten TinyKitten merged commit 4e005aa into dev Feb 22, 2026
7 checks passed
@TinyKitten TinyKitten deleted the claude/remove-android-16-tts-WEkr5 branch February 22, 2026 01:07
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