不要なフォントのサブモジュールクローンをCIで行わないようにした

[TinyKitten]

Summary by CodeRabbit

リリースノート

• Chores
  • GitHub Actionsのチェックアウトステップを簡略化しました。複数のワークフロー設定を更新し、不要なパラメータを削除しました。
  • 開発ガイダンスドキュメントを更新しました。

[coderabbitai]

Walkthrough

GitHub Actions ワークフロー内の SSH キーベースの認証およびサブモジュール設定を削除し、チェックアウトステップを簡素化しました。また、関連するドキュメント内の GitHub Actions シークレット再利用に関するガイドラインを削除しています。

Changes

Cohort / File(s)	Summary
ワークフローのチェックアウトステップ簡素化 ​.github/workflows/deploy_android_production.yml, ​.github/workflows/test.yml, ​.github/workflows/typecheck.yml	各ワークフローのチェックアウトステップから submodules: recursive と ssh-key: ${{ secrets.FONTS_SSH_KEY }} パラメータを削除し、actions/checkout@v4 のデフォルト動作に統一
ドキュメントの指針削除 ​.github/copilot-instructions.md, AGENTS.md	GitHub Actions シークレット名の再利用に関する指針（FONTS_SSH_KEY の例）を削除

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• SSH キー設定の削除に伴う認証メカニズム変更の影響を確認
• 3つのワークフロー内の同一パターン削除が他の機能に与える影響を確認
• ドキュメント削除による通知/ガイドラインの廃止と整合性を確認

Possibly related PRs

• デプロイ設定漏れ修正 #4573: GitHub Actions のチェックアウト/サブモジュール処理と認証情報（本 PR は SSH キーベースのサブモジュールチェックアウトを削除、参照 PR はサブモジュール認証/URL とフェッチステップを調整）の関連変更
• GitHub Copilotの活用に関する指示書を新規作成 #4562: .github/copilot-instructions.md および AGENTS.md 同一ドキュメントの編集と GitHub Actions シークレット/設定に関するガイダンスの重複

Poem

🐰 SSH キーさよなら、シンプルは正義
チェックアウトは軽く、設定は明快
ドキュメントも整理して、迷いなし
ウサギが走った、秘密のない道を 🏃✨

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	プルリクエストのタイトル「不要なフォントのサブモジュールクローンをCIで行わないようにした」は、変更セットの主な内容と完全に関連しています。複数のワークフローファイル（deploy_android_production.yml、test.yml、typecheck.yml）から submodules: recursive と SSH キー設定を削除し、さらに関連するドキュメント（copilot-instructions.md と AGENTS.md）からも対応する指示を削除する変更が含まれています。タイトルは、CI パイプラインからフォントサブモジュールの不要なクローン操作を削除するという、この PR の本質的な目的を正確に反映しており、簡潔かつ明確です。

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/clone-by-ci

---

📜 Recent review details

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d8e53ab and a9fe00a.

📒 Files selected for processing (5)

• .github/copilot-instructions.md (0 hunks)
• .github/workflows/deploy_android_production.yml (1 hunks)
• .github/workflows/test.yml (1 hunks)
• .github/workflows/typecheck.yml (1 hunks)
• AGENTS.md (0 hunks)

💤 Files with no reviewable changes (2)

• .github/copilot-instructions.md
• AGENTS.md

🧰 Additional context used

🧠 Learnings (1)

📓 Common learnings

Learnt from: TinyKitten
Repo: TrainLCD/MobileApp PR: 0
File: :0-0
Timestamp: 2025-08-11T21:13:39.083Z
Learning: TinyKitten prefers concise code reviews for the TrainLCD/MobileApp repository. Keep review comments brief and to the point.

Learnt from: TinyKitten
Repo: TrainLCD/MobileApp PR: 4448
File: functions/src/funcs/tts.ts:214-232
Timestamp: 2025-08-11T20:23:23.071Z
Learning: PR #4448 in TrainLCD/MobileApp is specifically about AI feedback summarization and labeling functionality, not about TTS (Text-to-Speech) improvements. The TTS code changes appear to be part of a modularization refactoring but are not the focus of this PR.

🔇 Additional comments (3)

.github/workflows/test.yml (1)

20-21: 簡潔で適切な変更

Jest テストワークフローでは不要なサブモジュール（フォント）のクローンを削除しているため、チェックアウトステップの簡素化は正しい判断です。

.github/workflows/deploy_android_production.yml (1)

19-20: 一貫性のある改善

Android 本番ビルドでもサブモジュールが不要なため、チェックアウトステップの統一化は妥当です。

.github/workflows/typecheck.yml (1)

20-21: 全ワークフロー間の統一性を確認完了

.github/workflows/ 内の全ワークフロー定義を確認しました。SSH キーと再帰的サブモジュール指定の残存がないことが確認されました。3 つのワークフロー（typecheck.yml、lint.yml、test.yml）が一貫して簡素化されており、変更は完全かつ安全です。

---

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

[TinyKitten]

本来ここ消しちゃ駄目だけど本番デプロイもEASに寄せる予定だからいずれこのファイルごと消える