Repository navigation
chore(fork): show the app as DoubleCode - #6
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fork-only: show the app as DoubleCode so it is easy to tell apart from upstream's AndCode.
app_name->DoubleCodein all 8 locales (launcher, recents, notifications, drawer, onboarding, voice input service label)"AndCode"; it now usesR.string.app_name(also fixes an i18n-rule violation; worth upstreaming)cd_app_logo) are intentionally left as-is to keep upstream syncs conflict-freePre-PR review
Reviewed at 7949208. The follow-up commit 0704af8 applies the reviewer's first non-blocking suggestion only (one line,
Text("AndCode")->Text(stringResource(R.string.app_name));stringResourceandRwere already imported).判定: APPROVE
ブロッカー
なし
提案(非ブロッキング)
app/src/main/java/com/yugahashimoto/andcode/feature/assistant/AndCodeVoiceSession.kt:520— 音声アシスタント画面の見出しはText("AndCode", ...)とハードコードされていて、app_nameを参照していません。fork では、ランチャー・通知・ドロワー・オンボーディングが "DoubleCode" になるのに、アシスタントの見出しだけ "AndCode" のまま残ります。upstream 由来の既存のハードコードなので今回の回帰ではありません。ブランド表示を揃えるならstringResource(R.string.app_name)に置き換えてください。このハードコードは i18n ルールにも反しているので、upstream に返す修正としても妥当です。cd_app_logo(全 8 ロケールで "AndCode"、AppDrawerContent.kt:352などで使用)— TalkBack の利用者には、"AndCode" のロゴの隣に "DoubleCode" と読み上げられます。ロゴの画像自体が AndCode の意匠なので、意図どおりであればこのままで構いません。AndCodeComponents.kt:48のOpenCodeBrandにも "AndCode" がハードコードされていますが、呼び出し元が無い未使用の composable なので影響はありません。チェック済み項目
app_nameの値:全 8 ロケール(values / ar / es / fr / ja / pt-rBR / ru / zh-rCN)のstrings.xml:3がDoubleCodeで、translatable属性も他の属性も付いていないことを確認しました。リポジトリ内の他の XML にapp_nameを二重に定義している箇所はありません。i18n キー整合:キーの追加・削除は無く、全ロケールに
app_nameが残っています。i18n-check.ymlが比べるのはキー名だけなので、値を変えても失敗しません。app_nameの参照箇所:AndroidManifest.xml:57(application の label)と:128(AndCodeRecognitionServiceの label)。システムの音声入力設定にも "DoubleCode" と表示されるので、upstream 版との区別という目的に合っています。widget_quick_input.xml:16(contentDescription)。AppDrawerContent.kt:356(ドロワーの見出し)。OnboardingChoiceScreen.kt:82。onboarding_welcome_titleは全 8 ロケールで%1$s形式なので、「Welcome to DoubleCode」などと正しく展開されます。LocalRuntimeService.kt:626(通知タイトル)。いずれも文字列リソースとして参照しているだけで、表示以外の処理(ID や比較など)に値を使っている箇所はありません。
テスト:
app/src/testとandroidTestでapp_name、DoubleCode、"AndCode" リテラルを検索しました。"AndCode" を前提にしているテストは次の 4 種類で、どれもapp_nameに依存していません。LegalScreenInstrumentedTest.kt:49:法務文書 asset の本文を確認しており、asset は変更されていません。LegalDisclosureComplianceTest.kt:109:README と pages を確認しています。*ReleaseClientTestの User-Agent。LocalRuntimeProcessLauncherTest.kt:343:エージェントに渡すコンテキストの文面です。app_nameの値を検証しているテストはありません。差分の範囲について:shell が使えないため
git diffは実行していません。全ロケールのapp_nameの現在値を確認し、"AndCode" を含む他の文字列(cd_app_logo、image_saved、法務文書など)がそのまま残っていることをソースで確かめて、変更概要と一致すると判断しました。