Skip to content

[P2] Remplacer les catch(_) silencieux par du logging et de l'UI #77

Description

@r9r-dev

Description

14 instances de catch (_) silencieux dans le code, ce qui rend le debugging difficile.

Pattern problématique

} catch (_) {
  // Ignore errors
}

Fichiers concernés

  • lib/features/settings/providers/theme_provider.dart (2x)
  • lib/features/settings/providers/settings_provider.dart (3x)
  • lib/features/settings/providers/window_geometry_provider.dart (3x)
  • lib/main.dart (1x)
  • lib/features/chat/widgets/irc_logs_view.dart (1x)
  • lib/features/users/services/user_actions_service.dart (1x)
  • lib/core/irc/protocol/sts_policy.dart (1x)
  • lib/core/irc/connection/socket_manager_impl.dart (1x)

Solution proposée

} catch (e, st) {
  _logger.e('Description de l\'erreur', error: e, stackTrace: st);
  // Ou rethrow si nécessaire
}

Ajouter également une info sur l'erreur dans les logs de l'application. Indiquer qu'il y a eu une erreur ainsi qu'un moyen de voir les détails (sans polluer les logs dans l'interface).

Enfin, quand il y a une erreur, l'indicateur en bas à gauche à côté de ke.network doit changer de couleur vers le orange.

Investigations

Pour les autres erreurs potentielles non mentionnées dans ce ticket, contrôler que l'erreur est bien logguée dans l'interface des logs de l'application.

Exceptions acceptables

  • Erreurs lors du shutdown (uniquement les logs)
  • Opérations non critiques où l'échec est attendu

Effort estimé

1 jour

Référence

docs/CODE_REVIEW.md - Section 4.6

Metadata

Metadata

Assignees

No one assigned

    Labels

    qualityQualité de code

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions