From 1b6d124f148e11289d919457be3d8ca109d95cff Mon Sep 17 00:00:00 2001
From: vishnukvmd <vishnu@ente.io>
Date: Mon, 21 Jul 2025 16:29:25 +0530
Subject: [PATCH] Lint network

---
 mobile/packages/network/analysis_options.yaml | 72 +++++++++++++++++++
 mobile/packages/network/lib/ente_network.dart |  2 -
 mobile/packages/network/pubspec.yaml          |  6 +-
 3 files changed, 75 insertions(+), 5 deletions(-)
 create mode 100644 mobile/packages/network/analysis_options.yaml

diff --git a/mobile/packages/network/analysis_options.yaml b/mobile/packages/network/analysis_options.yaml
new file mode 100644
index 0000000000..1bd78bc1b0
--- /dev/null
+++ b/mobile/packages/network/analysis_options.yaml
@@ -0,0 +1,72 @@
+# For more linters, we can check https://dart-lang.github.io/linter/lints/index.html
+# or https://pub.dev/packages/lint (Effective dart)
+# use "flutter analyze ." or "dart  analyze ." for running lint checks
+
+include: package:flutter_lints/flutter.yaml
+linter:
+  rules:
+    # Ref https://github.com/flutter/packages/blob/master/packages/flutter_lints/lib/flutter.yaml
+    # Ref https://dart-lang.github.io/linter/lints/
+    - avoid_print
+    - avoid_unnecessary_containers
+    - avoid_web_libraries_in_flutter
+    - no_logic_in_create_state
+    - prefer_const_constructors
+    - prefer_const_constructors_in_immutables
+    - prefer_const_declarations
+    - prefer_const_literals_to_create_immutables
+    - prefer_final_locals
+    - require_trailing_commas
+    - sized_box_for_whitespace
+    - use_full_hex_values_for_flutter_colors
+    - use_key_in_widget_constructors
+    - cancel_subscriptions
+
+
+    - avoid_empty_else
+    - exhaustive_cases
+
+    # just style suggestions
+    - sort_pub_dependencies
+    - use_rethrow_when_possible
+    - prefer_double_quotes
+    - directives_ordering
+    - always_use_package_imports
+    - sort_child_properties_last
+    - unawaited_futures
+
+analyzer:
+  errors:
+    avoid_empty_else: error
+    exhaustive_cases: error
+    curly_braces_in_flow_control_structures: error
+    directives_ordering: error
+    require_trailing_commas: error
+    always_use_package_imports: warning
+    prefer_final_fields: error
+    unused_import: error
+    camel_case_types: error
+    prefer_is_empty: warning
+    use_rethrow_when_possible: info
+    unused_field: warning
+    use_key_in_widget_constructors: warning
+    sort_child_properties_last: warning
+    sort_pub_dependencies: warning
+    library_private_types_in_public_api: warning
+    constant_identifier_names: ignore
+    prefer_const_constructors: warning
+    prefer_const_declarations: warning
+    prefer_const_constructors_in_immutables: warning
+    prefer_final_locals: warning
+    unnecessary_const: error
+    cancel_subscriptions: error
+    unrelated_type_equality_checks: error
+    unnecessary_cast: info
+
+
+    unawaited_futures: warning # convert to warning after fixing existing issues
+    invalid_dependency: info
+    use_build_context_synchronously: ignore # experimental lint, requires many changes
+    prefer_interpolation_to_compose_strings: ignore # later too many warnings
+    prefer_double_quotes: ignore # too many warnings
+    avoid_renaming_method_parameters: ignore # incorrect warnings for `equals` overrides
diff --git a/mobile/packages/network/lib/ente_network.dart b/mobile/packages/network/lib/ente_network.dart
index d269ce93e5..418fd5eba1 100644
--- a/mobile/packages/network/lib/ente_network.dart
+++ b/mobile/packages/network/lib/ente_network.dart
@@ -1,3 +1 @@
-library ente_network;
-
 export 'network.dart';
diff --git a/mobile/packages/network/pubspec.yaml b/mobile/packages/network/pubspec.yaml
index bdcf9f9868..1e2e9b13d9 100644
--- a/mobile/packages/network/pubspec.yaml
+++ b/mobile/packages/network/pubspec.yaml
@@ -8,21 +8,21 @@ environment:
   flutter: ">=1.17.0"
 
 dependencies:
-  flutter:
-    sdk: flutter
   dio: ^5.8.0+1
   ente_configuration:
     path: ../../packages/configuration
   ente_events:
     path: ../../packages/events
+  flutter:
+    sdk: flutter
   native_dio_adapter: ^1.4.0
   package_info_plus: ^8.3.0
   ua_client_hints: ^1.4.1
   uuid: ^4.5.1
 
 dev_dependencies:
+  flutter_lints: ^5.0.0
   flutter_test:
     sdk: flutter
-  flutter_lints: ^5.0.0
 
 flutter: