試験的機能からETAアシストを廃止しリモート設定のみで自動有効化するよう変更

[TinyKitten]

概要

リモート設定(eta_assist_enabled)が許可している場合に ETA アシスト（到着判定の改善）を自動的に有効化するよう変更し、試験的機能設定から手動トグル項目を削除しました。従来はリモート設定のマスタースイッチに加えてユーザーの手動トグルが必要でしたが、手動トグルを廃止し、リモート設定のみで判定します。リモート設定はサーバー側キルスイッチとして継続して残します。

変更の種類

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

変更内容

• 試験的機能設定画面(ExperimentalSettings)から ETA アシストのトグル項目・説明文・トグル処理を削除
• isEtaAssistEnabled() を「リモート設定 AND 手動トグル」から「リモート設定(eta_assist_enabled)のみ」に変更し、統合により不要となった isEtaAssistRemoteEnabled() を削除
• 手動トグル用の atom(etaAssistManualEnabledAtom)を削除
• 起動時の手動トグル復元処理(Permitted)と永続化ストレージキー(ETA_ASSIST_MANUAL_ENABLED)を削除
• 未使用となった翻訳文言(etaAssistTitle / etaAssistDescription、ja/en)を削除
• 関連テスト(remoteConfig.test.ts / ExperimentalSettings.test.tsx)を新仕様へ更新

テスト

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

関連Issue

スクリーンショット（任意）

---

Generated by Claude Code

Summary by CodeRabbit

• 変更

  • 設定画面からETA補助の手動切り替え項目を削除しました。
  • ポートレートモードの設定項目を追加し、縦持ち時のレイアウトを切り替えられるようにしました。
  • テレメトリに関する説明文を更新し、端末の地理座標が分析サーバーへ送信されることを明示しました。
  • 設定値の保存・復元対象を更新しました。

• テスト

  • ETA補助がリモート設定に基づいてのみ有効になる動作を確認するテストを更新しました。

[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: 354d8d4d-6759-48f8-bbd8-f47a2cc62d7c

📥 Commits

Reviewing files that changed from the base of the PR and between 6bfde55 and 2a919e7.

📒 Files selected for processing (9)

• assets/translations/en.json
• assets/translations/ja.json
• src/components/Permitted.tsx
• src/constants/storage.ts
• src/lib/remoteConfig.test.ts
• src/lib/remoteConfig.ts
• src/screens/ExperimentalSettings.test.tsx
• src/screens/ExperimentalSettings.tsx
• src/store/atoms/experimental.ts

💤 Files with no reviewable changes (4)

• src/constants/storage.ts
• assets/translations/ja.json
• assets/translations/en.json
• src/store/atoms/experimental.ts

---

📝 Walkthrough

Walkthrough

ETA補助をRemote Configのマスタースイッチのみで判定するよう変更し、手動トグル、関連する画面表示・保存復元処理・翻訳を削除しました。ポートレートモードなどの設定キーと翻訳を追加しています。

Changes

ETA補助設定の整理

Layer / File(s)	Summary
Remote Config単独判定 src/lib/remoteConfig.ts, src/lib/remoteConfig.test.ts	ETA補助の有効判定から手動トグルとの合成を削除し、Remote Configの値またはfalseを返す仕様へ変更しました。
実験設定画面のETA補助削除 src/screens/ExperimentalSettings.tsx, src/screens/ExperimentalSettings.test.tsx, assets/translations/*.json	実験設定画面からETA補助手動トグルを削除し、ポートレートモードとテレメトリに関する表示・テスト・翻訳を更新しました。
設定キーと復元処理の整理 src/constants/storage.ts, src/components/Permitted.tsx, src/store/atoms/experimental.ts	ETA補助手動設定のストレージキーと復元処理を削除し、ポートレートモードなどの設定キーを追加しました。

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

Possibly related PRs

• TrainLCD/MobileApp#6204: ポートレートモードの追加と関連するストレージ・アトムを変更しています。
• TrainLCD/MobileApp#6366: ETA補助のRemote Config判定を変更しています。
• TrainLCD/MobileApp#6369: ETA補助の判定と手動トグルアトムを変更しています。

Suggested labels: react

Poem

ぴょんと跳ねたら、補助は消え
遠隔スイッチが道を決め
縦持ち画面が顔を出す
設定の鍵も軽くなり
月夜のうさぎは、テストを祝う ─ 🐇

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	PRの主変更であるETAアシストの廃止とリモート設定のみでの自動有効化を的確に表しています。
Description check	✅ Passed	テンプレートの必須項目である概要、変更の種類、変更内容、テストが揃っており、内容も十分具体的です。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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/eta-assist-auto-enable-nenfjv

---

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

[TinyKitten]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.