Skip to content

chore(fork): show the app as DoubleCode - #6

Merged
doublepi123 merged 2 commits into
mainfrom
feat/rename-app-doublecode
Oct 17, 2026
Merged

doublepi123 merged 2 commits into
mainfrom
feat/rename-app-doublecode

Conversation

@doublepi123

Copy link
Copy Markdown
Owner

Fork-only: show the app as DoubleCode so it is easy to tell apart from upstream's AndCode.

  • app_name -> DoubleCode in all 8 locales (launcher, recents, notifications, drawer, onboarding, voice input service label)
  • Voice assistant title used a hardcoded "AndCode"; it now uses R.string.app_name (also fixes an i18n-rule violation; worth upstreaming)
  • Other strings mentioning AndCode (legal docs, dialog bodies, cd_app_logo) are intentionally left as-is to keep upstream syncs conflict-free

Pre-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)); stringResource and R were 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、法務文書など)がそのまま残っていることをソースで確かめて、変更概要と一致すると判断しました。

@doublepi123
doublepi123 merged commit 740278e into main Oct 17, 2026
6 checks passed
@doublepi123
doublepi123 deleted the feat/rename-app-doublecode branch October 17, 2026 02:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant