fix: share startup error logs as file and add export-to-file option

Author: i5hi

lib/core/screens/app_init_error_screen.dart

@@ -4,12 +4,12 @@ import 'package:bb_mobile/core/utils/constants.dart';
 import 'package:bb_mobile/core/utils/logger.dart';
 import 'package:bb_mobile/core/widgets/app_language_picker.dart';
 import 'package:bb_mobile/core/widgets/buttons/button.dart';
+import 'package:bb_mobile/core/widgets/share_logs_bottom_sheet.dart';
 import 'package:bb_mobile/core/widgets/snackbar_utils.dart';
 import 'package:bb_mobile/generated/l10n/localization.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_svg/flutter_svg.dart';
 import 'package:bull_ui/bull_ui.dart' show Gap;
-import 'package:share_plus/share_plus.dart';
 import 'package:url_launcher/url_launcher.dart';
 
 class AppInitErrorScreen extends StatefulWidget {
@@ -24,25 +24,40 @@ class AppInitErrorScreen extends StatefulWidget {
 class _AppInitErrorScreenState extends State<AppInitErrorScreen> {
   Language _language = Language.fromKeyboard();
 
-  Future<void> _shareLogs(BuildContext context) async {
+  Future<void> _shareLogs(BuildContext context, AppLocalizations loc) async {
     try {
       final logs = await log.readLogs();
+      await shareLogsAsFile(logs);
+    } catch (e) {
+      log.severe(
+        message: 'Failed to share logs',
+        error: e,
+        trace: StackTrace.current,
+      );
       if (!context.mounted) return;
-      await SharePlus.instance.share(
-        ShareParams(
-          text: logs.join('\n'),
-          subject: 'bull_logs.tsv',
-          title: 'bull_logs.tsv',
-        ),
+      SnackBarUtils.showSnackBar(
+        context,
+        loc.errorSharingLogsMessage(e.toString()),
       );
+    }
+  }
+
+  Future<void> _exportLogs(BuildContext context, AppLocalizations loc) async {
+    try {
+      final logs = await log.readLogs();
+      final saved = await exportLogsAsFile(logs);
+      if (!context.mounted) return;
+      if (saved) {
+        SnackBarUtils.showSnackBar(context, loc.logsExportedMessage);
+      }
     } catch (e) {
       log.severe(
-        message: 'Failed to share logs',
+        message: 'Failed to export logs',
         error: e,
         trace: StackTrace.current,
       );
       if (!context.mounted) return;
-      SnackBarUtils.showSnackBar(context, 'Failed to share logs: $e');
+      SnackBarUtils.showSnackBar(context, loc.logsExportFailedMessage);
     }
   }
 
@@ -128,7 +143,18 @@ class _AppInitErrorScreenState extends State<AppInitErrorScreen> {
                       textColor: context.appColors.text,
                       borderColor: context.appColors.border,
                       outlined: true,
-                      onPressed: () => _shareLogs(context),
+                      onPressed: () => _shareLogs(context, loc),
+                    ),
+                    const Gap(12),
+                    BBButton.big(
+                      label: loc.logsShareOptionExport,
+                      iconData: Icons.file_download_outlined,
+                      iconFirst: true,
+                      bgColor: context.appColors.surface,
+                      textColor: context.appColors.text,
+                      borderColor: context.appColors.border,
+                      outlined: true,
+                      onPressed: () => _exportLogs(context, loc),
                     ),
                     const Gap(12),
                     BBButton.big(

lib/core/widgets/share_logs_bottom_sheet.dart

@@ -1,9 +1,12 @@
 import 'dart:convert';
+import 'dart:io';
 
 import 'package:bb_mobile/core/utils/build_context_x.dart';
 import 'package:bull_ui/bull_ui.dart';
 import 'package:file_picker/file_picker.dart';
 import 'package:flutter/material.dart';
+import 'package:path/path.dart' as p;
+import 'package:path_provider/path_provider.dart';
 import 'package:share_plus/share_plus.dart';
 
 Future<void> showLogsShareSheet({
@@ -58,15 +61,30 @@ Future<void> shareLogsAsText(List<String> logs) async {
   );
 }
 
+Future<void> shareLogsAsFile(List<String> logs) async {
+  final path = p.join(
+    (await getTemporaryDirectory()).path,
+    _timestampedLogFileName(),
+  );
+  await File(path).writeAsString(logs.join('\n'));
+  await SharePlus.instance.share(
+    ShareParams(files: [XFile(path)], subject: 'bull_logs.tsv'),
+  );
+}
+
 /// Returns true if the file was saved, false if the user cancelled.
 Future<bool> exportLogsAsFile(List<String> logs) async {
-  final timestamp = DateTime.now()
-      .toIso8601String()
-      .replaceAll(':', '-')
-      .replaceAll('.', '-');
   final result = await FilePicker.platform.saveFile(
     bytes: utf8.encode(logs.join('\n')),
-    fileName: 'bull_logs_$timestamp.tsv',
+    fileName: _timestampedLogFileName(),
   );
   return result != null;
 }
+
+String _timestampedLogFileName() {
+  final timestamp = DateTime.now()
+      .toIso8601String()
+      .replaceAll(':', '-')
+      .replaceAll('.', '-');
+  return 'bull_logs_$timestamp.tsv';
+}

lib/core/widgets/share_logs_widget.dart

@@ -17,10 +17,18 @@ class ShareLogsWidget extends StatelessWidget {
           shape: RoundedRectangleBorder(borderRadius: BorderRadius.circular(2)),
           tileColor: context.appColors.transparent,
           title: Text(context.loc.shareLogsLabel),
-          onTap: () => _onShareTapped(context),
+          onTap: () => _shareLogs(context),
           trailing: const Icon(Icons.share_sharp),
         ),
         const Gap(8),
+        ListTile(
+          shape: RoundedRectangleBorder(borderRadius: BorderRadius.circular(2)),
+          tileColor: context.appColors.transparent,
+          title: Text(context.loc.logsShareOptionExport),
+          onTap: () => _exportLogs(context),
+          trailing: const Icon(Icons.file_download_outlined),
+        ),
+        const Gap(8),
         GestureDetector(
           onTap: () => _deleteLogs(context),
           child: Padding(
@@ -46,19 +54,10 @@ class ShareLogsWidget extends StatelessWidget {
     SnackBarUtils.showSnackBar(context, context.loc.logsDeletedMessage);
   }
 
-  Future<void> _onShareTapped(BuildContext context) async {
-    await showLogsShareSheet(
-      context: context,
-      onShare: () => _shareLogs(context),
-      onExport: () => _exportLogs(context),
-    );
-  }
-
   Future<void> _shareLogs(BuildContext context) async {
     try {
       final logs = await log.readLogs();
-      if (!context.mounted) return;
-      await shareLogsAsText(logs);
+      await shareLogsAsFile(logs);
     } catch (e) {
       if (!context.mounted) return;
       SnackBarUtils.showSnackBar(