Skip to content

チューニング画面を廃止し設定画面から開発者向けセクションを削除 - #6630

Merged
TinyKitten merged 1 commit into
devfrom
claude/remove-tuning-screen-dev-menu-l5k1n2
Aug 9, 2026
Merged

チューニング画面を廃止し設定画面から開発者向けセクションを削除#6630
TinyKitten merged 1 commit into
devfrom
claude/remove-tuning-screen-dev-menu-l5k1n2

Conversation

@TinyKitten

@TinyKitten TinyKitten commented Aug 9, 2026

Copy link
Copy Markdown
Member

概要

チューニング画面(TuningSettings)を廃止し、設定画面から「開発者向け」セクションおよび「チューニング」メニューを削除しました。あわせて、この削除によって参照元がなくなった翻訳キーを ja/en 双方から除去しています。

変更の種類

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

変更内容

  • src/components/TuningSettings.tsx を削除
  • src/index.tsx から TuningSettings の import と Stack.Screen 登録を削除
  • src/screens/AppSettings.tsx から、SETTING_ITEM_ID_MAPdeveloper_tuning・アイコン解決の developer_tuning ケース・developerItemsuseMemo・「開発者向け」セクションの JSX を削除
  • assets/translations/{ja,en}.json から、本変更で参照元がなくなった以下のキーを削除
    • tuning / tuningItemTiming / tuningItemHeaderDelay / tuningItemHeaderDuration / tuningItemBottomTransitionDelay / headerDelayTooShortErrorText / nanErrorText / disableDevOverlay / forDevelopers

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

  • 影響範囲はカナリア(dev アプリ)限定: 削除した「開発者向け」セクションは isDevApp でガードされており、本番ビルドには元から表示されていません。設定画面のパーソナライズ/アプリについての各セクションは無変更です。
  • TuningSettings ルートへの遷移: navigation.navigate('TuningSettings') の呼び出し箇所は設定画面のみで、リポジトリ全体を検索して他に遷移元がないことを確認済みです(ディープリンク・クイックアクションからの参照もなし)。
  • 翻訳キー: 削除したキーが他から参照されていないことを検索で確認し、削除後に ja/en のキー差分が本変更前と同一(既存の local2 のみ)であることを検証しました。
  • tuningState atom は据え置き: telemetryEnabled / untouchableModeEnabled は試験的機能設定画面から、devOverlayEnabled は走行画面の長押しアクションシート(Permitted.tsx)から引き続き操作されるため、atom 自体は削除していません。
  • 既知の残課題(本 PR のスコープ外): headerTransitionInterval / headerTransitionDelay / bottomTransitionInterval はチューニング画面が唯一の書き込み口だったため、本変更以降は書き込み手段がなくなります。Permitted.tsx のストレージ復元処理は残しているので、既に値を保存済みのカナリア利用者は変更不能なカスタム値を保持し続けます。これらを atom から削除して DEFAULT_* 定数へ一本化する対応は、6 ファイルと atom をモックしている約 10 のテストファイルに波及するため、別途対応が必要であれば切り出します。
  • 既存のデッドキーは未整理: tuningItemTTS* / tuningItemLocationAccuracy は本変更以前から未使用のため、無関係な差分を混ぜないよう今回は温存しています。

テスト

ローカルで以下を実行し、すべて成功することを確認しました(node_modules 未インストール状態だったため事前に npm install を実行。差分の出た package-lock.json はコミットに含めていません)。

  • npm run lint — 626 ファイル、指摘なし

  • npm test — 214 suites / 2247 tests すべて成功

  • npm run typecheck — エラーなし

  • npm run lint が通ること

  • npm test が通ること

  • npm run typecheck が通ること

関連Issue

スクリーンショット(任意)

設定画面から「開発者向け」セクションが消える差分ですが、カナリアビルド限定の表示かつ実機ビルドを起動できる環境がないため、スクリーンショットは未添付です。


🤖 Generated with Claude Code

https://claude.ai/code/session_01TQwaU562LMNnu96xUrsQse


Generated by Claude Code

Summary by CodeRabbit

  • 変更
    • 開発者向けのチューニング設定画面と設定項目を削除しました。
    • ヘッダー遅延、表示タイミング、入力値エラー、開発者オーバーレイに関する設定を削除しました。
    • 設定画面から「開発者向け」セクションを削除しました。
    • TTS音声・プロンプトおよび位置情報の精度に関する設定は引き続き利用できます。

@github-actions github-actions Bot added the react label Aug 9, 2026
@TinyKitten TinyKitten self-assigned this Aug 9, 2026
@coderabbitai

coderabbitai Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

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: 3ef1d26a-25e5-49de-b194-d5e1abe2d5af

📥 Commits

Reviewing files that changed from the base of the PR and between fe8c648 and bc0e86f.

📒 Files selected for processing (5)
  • assets/translations/en.json
  • assets/translations/ja.json
  • src/components/TuningSettings.tsx
  • src/index.tsx
  • src/screens/AppSettings.tsx
💤 Files with no reviewable changes (5)
  • src/index.tsx
  • src/components/TuningSettings.tsx
  • src/screens/AppSettings.tsx
  • assets/translations/ja.json
  • assets/translations/en.json

📝 Walkthrough

Walkthrough

TuningSettings コンポーネントとナビゲーション登録を削除しました。AppSettings から開発者向け設定表示を削除しました。英語と日本語の関連翻訳キーを削除しました。

Changes

開発者向けチューニング設定の削除

Layer / File(s) Summary
チューニング画面とナビゲーションの削除
src/components/TuningSettings.tsx, src/index.tsx
TuningSettings コンポーネント、設定値の処理、開発用オーバーレイ、画面登録を削除しました。
設定画面の開発者向け表示削除
src/screens/AppSettings.tsx
developer_tuning の設定項目、アイコン分岐、項目生成処理、開発者向けセクションを削除しました。
関連翻訳キーの削除
assets/translations/en.json, assets/translations/ja.json
旧チューニング設定、ヘッダー遅延、入力値エラー、開発者オーバーレイ、開発者向け表示の翻訳キーを削除しました。TTS音声、プロンプト、位置精度のキーは維持しました。

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Poem

うさぎが設定画面を跳ねてゆく
チューニングの道具は静かに消える
翻訳の鍵も整理され
TTSの声は残って歌う
ぴょん、と軽い設定画面

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed 概要、変更内容、テスト結果、関連Issue、任意のスクリーンショット欄を含み、テンプレートに沿って具体的に記載されています。
Title 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/remove-tuning-screen-dev-menu-l5k1n2

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

@TinyKitten
TinyKitten merged commit 9d7e73c into dev Aug 9, 2026
7 checks passed
@TinyKitten
TinyKitten deleted the claude/remove-tuning-screen-dev-menu-l5k1n2 branch August 9, 2026 10:35
@TinyKitten TinyKitten mentioned this pull request Aug 9, 2026
9 tasks
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