feat: F-Droidビルドにおける自己アップデート機能の無効化と関連設定の追加

yuubinnkyoku · yuubinnkyoku · commit a86ac7604b8c · 2025-09-26T20:34:49.000+09:00
diff --git a/README.md b/README.md
@@ -52,30 +52,81 @@ AtCoderの精進をスマホでも。
 
 ビルド済みのバイナリ（APKなど）は[GitHubリリーズページ](https://github.com/Shojin-App/shojin_app/releases)からダウンロードできます。これが最も簡単な開始方法です。
 
-### F-Droid対応版
+### F-Droid 対応版 / Reproducible Build
 
-このアプリはF-Droidポリシーに完全準拠したビルドが可能です。F-Droid対応版では以下の機能が制限されます：
+このアプリは F-Droid Inclusion Policy に準拠したビルドが可能です。F-Droid フレーバー (`fdroid`) では以下が強制されます:
 
-- **自己アップデート機能**: 無効化（F-Droidが管理）
-- **オンラインフォント取得**: 無効化（システムフォント使用）
-- **外部APKダウンロード**: 完全に無効
+| 機能 | 通常ビルド | F-Droid ビルド |
+|------|------------|----------------|
+| 自己アップデート (アプリ内ダウンロード & APKインストール) | 有効 (ユーザー設定可) | 完全無効 (コードパス停止) |
+| 起動時自動チェック UI | 表示 | 非表示 |
+| オンラインフォント (Google Fonts) | 利用可 | 無効 (同梱/システムフォント) |
+| 外部APKダウンロード | 有効 | 無効 |
+| Git 依存 (動的取得) | なし (除去済) | なし |
 
-#### F-Droid対応ビルド手順
+自己アップデート関連クラス (`AutoUpdateManager`, `UpdateManager`) は F-Droid ビルドでは早期 return するノーオペ実装になり、不用意に APK を取得・実行しません。
+
+#### ビルド手順 (F-Droid フレーバー)
 
 ```bash
-# F-Droid互換ビルドの実行
+# 推奨: スクリプトで再現性確保
 ./build_fdroid.sh
 
-# または手動ビルド
+# 手動:
 flutter build apk \
-  --dart-define=FDROID_BUILD=true \
-  --dart-define=ENABLE_SELF_UPDATE=false \
-  --dart-define=ENABLE_ONLINE_FONTS=false \
-  --flavor=fdroid \
-  --release
+    --dart-define=FDROID_BUILD=true \
+    --dart-define=ENABLE_SELF_UPDATE=false \
+    --dart-define=ENABLE_ONLINE_FONTS=false \
+    --flavor=fdroid \
+    --release
+```
+
+#### 確認ポイント (F-Droid 申請前チェックリスト)
+- [ ] `git tag vX.Y.Z` が `pubspec.yaml` の `version` と一致
+- [ ] 起動後「設定 > 更新設定」セクションに自己更新 UI が表示されない
+- [ ] ネットワークトラフィックが AtCoder / コード実行 API / 静的リソース以外へ行かない
+- [ ] `flutter analyze` と `flutter test` が成功
+- [ ] `FONT_LICENSES.md` / LICENSE 表示画面から辿れる
+
+#### メタデータ作成例 (参考)
+F-Droid リポジトリ (fdroiddata) 用 `metadata/io.github.shojinapp.kyopro.yml` の骨子:
+```yaml
+Categories: Development
+License: GPL-3.0-only
+SourceCode: https://github.com/Shojin-App/shojin_app
+IssueTracker: https://github.com/Shojin-App/shojin_app/issues
+Changelog: https://github.com/Shojin-App/shojin_app/releases
+AutoName: Shojin App
+RepoType: git
+Repo: https://github.com/Shojin-App/shojin_app.git
+Builds:
+    - versionName: 1.0.0
+        versionCode: 100
+        commit: v1.0.0
+        gradle:
+            - fdroid
+        output: app-fdroid-release.apk
+        srclibs:
+            - flutter@stable
+        prebuild: |
+            sed -i 's/publish_to: .*/publish_to: none/' pubspec.yaml
+        build: |
+            flutter pub get
+            flutter build apk --dart-define=FDROID_BUILD=true --dart-define=ENABLE_SELF_UPDATE=false --dart-define=ENABLE_ONLINE_FONTS=false --flavor=fdroid --release
 ```
 
-**注意**: 実際のF-Droid提出には、Git依存関係の除去またはベンダリングが必要です。詳細は `pubspec_fdroid.yaml` と `FONT_LICENSES.md` を参照してください。
+#### ライセンス/資産
+- ハックジェンフォント (HackGen) : OFL 1.1 (`FONT_LICENSES.md`)
+- その他依存関係: アプリ内「ライセンス」画面でオフライン参照可
+
+#### セキュリティ指針
+- 外部実行バイナリをダウンロード・実行しない
+- 自己更新コードはビルドフラグ下で死コード化 (早期 no-op)
+- 追跡/広告/分析用ライブラリを含まない
+
+> F-Droid 寄稿時の最終チェックに役立つ簡易スクリプトは `build_fdroid.sh` を参照してください。
+
+**注意**: メタデータ例は参考であり、実際に提出する際は最新ポリシー・バージョンに合わせて調整してください。
 
 ### ソースからビルド
 
@@ -154,3 +205,5 @@ https://github.com/inotia00/revanced-manager
 
 本プロジェクトおよびその内容は、AtCoder株式会社及びその関連会社とは一切関係がなく、資金提供、承認、支持、またはその他いかなる形での関連もありません。
 本プロジェクトで使用されている商標、サービスマーク、商号、またはその他の知的財産権は、それぞれの所有者に帰属します。
+
+GitHub ロゴ, YouTube ロゴ, X(Twitter) ロゴ は各社の登録商標/商標です。これらのロゴはリンク誘導/識別目的のみで使用されており、本アプリによる公式な提携・後援・保証を意味しません。
diff --git a/lib/main.dart b/lib/main.dart
@@ -5,7 +5,6 @@ import 'package:dynamic_color/dynamic_color.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
 import 'package:flutter_localizations/flutter_localizations.dart'; // 追加
-import 'package:google_fonts/google_fonts.dart';
 import 'package:provider/provider.dart';
 
 import 'config/build_config.dart'; // Add build configuration
@@ -26,6 +25,16 @@ void main() async {
   // Flutter Engineの初期化を保証
   WidgetsFlutterBinding.ensureInitialized();
 
+  // F-Droid ビルド安全性アサート: 自己アップデートが無効であること
+  assert(() {
+    if (BuildConfig.isF_DroidBuild && BuildConfig.enableSelfUpdate) {
+      throw StateError(
+        'FDROID_BUILD=true なのに enableSelfUpdate が true です。ビルドフラグ/defines を再確認してください。',
+      );
+    }
+    return true;
+  }());
+
   // NotificationServiceの初期化と権限リクエスト
   final notificationService = NotificationService();
   await notificationService.initialize();
diff --git a/lib/screens/settings_screen.dart b/lib/screens/settings_screen.dart
@@ -471,21 +471,20 @@ class _SettingsScreenState extends State<SettingsScreen> {
       title: '更新設定',
       icon: Icons.system_update_alt,
       children: [
-        _HapticSwitchListTile(
-          title: 'アプリ起動時に自動で更新を確認',
-          value: _autoUpdateCheckEnabled,
-          onChanged: _setAutoUpdatePreference,
-          icon: Icons.sync_outlined,
-        ),
-        _HapticSwitchListTile(
-          title: 'アップデート通知を表示',
-          subtitle: '新しいバージョンが利用可能な時に通知を表示',
-          value: _showUpdateDialog,
-          onChanged: _setShowUpdateDialog,
-          icon: Icons.notifications_outlined,
-        ),
-        // Manual update section - hidden for F-Droid builds
         if (BuildConfig.enableSelfUpdate) ...[
+          _HapticSwitchListTile(
+            title: 'アプリ起動時に自動で更新を確認',
+            value: _autoUpdateCheckEnabled,
+            onChanged: _setAutoUpdatePreference,
+            icon: Icons.sync_outlined,
+          ),
+          _HapticSwitchListTile(
+            title: 'アップデート通知を表示',
+            subtitle: '新しいバージョンが利用可能な時に通知を表示',
+            value: _showUpdateDialog,
+            onChanged: _setShowUpdateDialog,
+            icon: Icons.notifications_outlined,
+          ),
           Padding(
             padding: const EdgeInsets.symmetric(
               horizontal: 20.0,
@@ -522,6 +521,23 @@ class _SettingsScreenState extends State<SettingsScreen> {
               ],
             ),
           ),
+        ] else ...[
+          Padding(
+            padding: const EdgeInsets.fromLTRB(20, 12, 20, 12),
+            child: Row(
+              crossAxisAlignment: CrossAxisAlignment.start,
+              children: [
+                const Icon(Icons.info_outline, color: Colors.grey),
+                const SizedBox(width: 12),
+                Expanded(
+                  child: Text(
+                    'このビルドではアプリ内自己アップデート機能は無効化されています。最新バージョンは公式GitHubリリースまたはF-Droidリポジトリ経由で入手してください。',
+                    style: AppFonts.notoSansJp(fontSize: 14),
+                  ),
+                ),
+              ],
+            ),
+          ),
         ],
       ],
     );
diff --git a/lib/services/auto_update_manager.dart b/lib/services/auto_update_manager.dart
@@ -18,19 +18,21 @@ class AutoUpdateManager {
 
   // Check if auto update is enabled
   Future<bool> isAutoUpdateEnabled() async {
-    if (!kEnableSelfUpdate) return false;
+    if (!kEnableSelfUpdate) return false; // Hard off for FDroid builds
     final prefs = await SharedPreferences.getInstance();
     return prefs.getBool(_autoUpdateKey) ?? true;
   }
 
   // Set auto update preference
   Future<void> setAutoUpdateEnabled(bool enabled) async {
+    if (!kEnableSelfUpdate) return; // no-op in FDroid
     final prefs = await SharedPreferences.getInstance();
     await prefs.setBool(_autoUpdateKey, enabled);
   }
 
   // Get last update check timestamp
   Future<DateTime?> getLastUpdateCheck() async {
+    if (!kEnableSelfUpdate) return null; // no tracking when disabled
     final prefs = await SharedPreferences.getInstance();
     final timestamp = prefs.getInt(_lastUpdateCheckKey);
     return timestamp != null
@@ -40,24 +42,28 @@ class AutoUpdateManager {
 
   // Set last update check timestamp
   Future<void> setLastUpdateCheck(DateTime dateTime) async {
+    if (!kEnableSelfUpdate) return; // no-op
     final prefs = await SharedPreferences.getInstance();
     await prefs.setInt(_lastUpdateCheckKey, dateTime.millisecondsSinceEpoch);
   }
 
   // Get skipped version
   Future<String?> getSkippedVersion() async {
+    if (!kEnableSelfUpdate) return null;
     final prefs = await SharedPreferences.getInstance();
     return prefs.getString(_skippedVersionKey);
   }
 
   // Set skipped version
   Future<void> setSkippedVersion(String version) async {
+    if (!kEnableSelfUpdate) return; // no-op
     final prefs = await SharedPreferences.getInstance();
     await prefs.setString(_skippedVersionKey, version);
   }
 
   // Check if should check for updates (based on time interval)
   Future<bool> shouldCheckForUpdates() async {
+    if (!kEnableSelfUpdate) return false;
     if (!await isAutoUpdateEnabled()) return false;
 
     final lastCheck = await getLastUpdateCheck();
@@ -75,7 +81,9 @@ class AutoUpdateManager {
     String repo = 'Shojin_App',
   }) async {
     if (!kEnableSelfUpdate) {
-      debugPrint('[FDROID] Self-update disabled by build flag');
+      debugPrint(
+        '[FDROID] Self-update disabled by build flag (startup check skipped)',
+      );
       return;
     }
     if (!await shouldCheckForUpdates()) {
@@ -127,6 +135,7 @@ class AutoUpdateManager {
     BuildContext context,
     EnhancedAppUpdateInfo updateInfo,
   ) {
+    if (!kEnableSelfUpdate) return; // safety
     if (!context.mounted) return;
 
     // Show as a SnackBar first, then dialog on tap
@@ -154,6 +163,7 @@ class AutoUpdateManager {
     BuildContext context,
     EnhancedAppUpdateInfo updateInfo,
   ) {
+    if (!kEnableSelfUpdate) return; // safety
     if (!context.mounted) return;
 
     showDialog(
@@ -177,6 +187,7 @@ class AutoUpdateManager {
     BuildContext context,
     EnhancedAppUpdateInfo updateInfo,
   ) {
+    if (!kEnableSelfUpdate) return; // safety
     if (!context.mounted) return;
 
     showDialog(
@@ -214,6 +225,10 @@ class AutoUpdateManager {
     String owner = 'yuubinnkyoku',
     String repo = 'Shojin_App',
   }) async {
+    if (!kEnableSelfUpdate) {
+      debugPrint('[FDROID] Manual update check suppressed');
+      return null;
+    }
     try {
       debugPrint('=== 手動アップデートチェック開始 ===');
       debugPrint('Repository: $owner/$repo');
@@ -249,6 +264,7 @@ class AutoUpdateManager {
     BuildContext context,
     EnhancedAppUpdateInfo updateInfo,
   ) {
+    if (!kEnableSelfUpdate) return; // safety
     _showUpdateDialog(context, updateInfo);
   }
 }
diff --git a/lib/services/update_manager.dart b/lib/services/update_manager.dart
@@ -27,9 +27,10 @@ class UpdateManager extends ChangeNotifier {
     );
     if (Platform.isAndroid) {
       if (!kEnableSelfUpdate) {
-        _statusMessage = 'F-Droidビルドではアプリ内アップデートは無効です。リリースページから手動で更新してください。';
+        // Hard block for FDroid builds
+        _statusMessage = 'F-Droidビルドではアプリ内アップデートは無効です (コードパス停止)。';
         notifyListeners();
-        return;
+        return; // Never attempt install
       }
       if (fileExtension == 'apk') {
         try {
@@ -138,6 +139,10 @@ class UpdateManager extends ChangeNotifier {
   }
 
   Future<void> _installApk(String filePath) async {
+    if (!kEnableSelfUpdate) {
+      debugPrint('[FDROID] _installApk() suppressed');
+      return;
+    }
     _statusMessage = 'APKのインストールプロセスを開始しています: $filePath';
     notifyListeners();
     debugPrint(_statusMessage);
