chore: add pre-commit hook with lint + format checks (#17)

* chore: add pre-commit hook configuration

Adds .pre-commit-config.yaml for the pino monorepo.

Hooks mirror what CI enforces:
- trailing-whitespace, end-of-file-fixer, check-yaml, check-json,
  check-merge-conflict, check-added-large-files (general hygiene)
- dart format --set-exit-if-changed (matches flutter-ci.yml)
- flutter analyze --no-pub --fatal-infos (matches flutter-ci.yml)
- pnpm typecheck (matches server-ci.yml)

Generated build output (server/dist/) is excluded from all checks.

Install locally: pip install pre-commit && pre-commit install

Co-authored-by: Milan Le <leduckhc@users.noreply.github.com>

* fix: apply trailing whitespace and end-of-file fixes

Auto-fixed by pre-commit hooks (trailing-whitespace, end-of-file-fixer)
on first repo-wide run. Establishes a clean baseline so future commits
don't accumulate whitespace noise.

Co-authored-by: Milan Le <leduckhc@users.noreply.github.com>

* chore: simplify install comment (pre-commit already on PATH)

Co-authored-by: Milan Le <leduckhc@users.noreply.github.com>

* chore: switch hooks from pre-commit to pre-push stage

Slow checks (flutter analyze, pnpm typecheck) are better suited
to pre-push where they don't interrupt every local commit.

Install: pre-commit install --hook-type pre-push

Co-authored-by: Milan Le <leduckhc@users.noreply.github.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Milan Le <leduckhc@users.noreply.github.com>

Author: leduckhc

.agents/skills/dart-run-static-analysis/SKILL.md

@@ -98,7 +98,7 @@ void processData() {
   // Suppress for a specific line
   // ignore: invalid_assignment
   int x = '';
-  
+
   const y = 10; // ignore: constant_identifier_names
 }
 ```

.agents/skills/dart-setup-ffi-assets/SKILL.md

@@ -281,19 +281,19 @@ Future<File> downloadAsset(
 ) async {
   final fileName = targetOS.dylibFileName('native_add_${targetOS.name}_${targetArchitecture.name}');
   final uri = downloadUri(fileName);
-  
+
   final client = HttpClient()..findProxy = HttpClient.findProxyFromEnvironment;
   final request = await client.getUrl(uri);
   final response = await request.close();
-  
+
   if (response.statusCode != 200) {
     throw ArgumentError('Download target $uri failed: Code ${response.statusCode}');
   }
-  
+
   final targetFile = File.fromUri(outputDir.uri.resolve(fileName));
   await targetFile.create(recursive: true);
   await response.pipe(targetFile.openWrite());
-  
+
   return targetFile;
 }
 

.agents/skills/flutter-add-integration-test/SKILL.md

@@ -41,7 +41,7 @@ Use the Dart/Flutter MCP server tools to interactively explore and manipulate th
 
 ## Test Authoring Guidelines
 
-Structure integration tests using the `flutter_test` API paradigm. 
+Structure integration tests using the `flutter_test` API paradigm.
 
 - Create a dedicated `integration_test/` directory at the project root.
 - Name all test files using the `<name>_test.dart` convention.
@@ -63,7 +63,7 @@ Execute tests using the `flutter drive` command. Require a host driver script lo
   `flutter drive --driver=test_driver/integration_test.dart --target=integration_test/app_test.dart -d chrome`
 - **If testing headless web:** Run with `-d web-server`.
 - **If testing on Android (Local):** Run `flutter drive --driver=test_driver/integration_test.dart --target=integration_test/app_test.dart`.
-- **If testing on Firebase Test Lab (Android):** 
+- **If testing on Firebase Test Lab (Android):**
   1. Build debug APK: `flutter build apk --debug`
   2. Build test APK: `./gradlew app:assembleAndroidTest`
   3. Upload both APKs to the Firebase Test Lab console.

.agents/skills/flutter-add-widget-preview/SKILL.md

@@ -15,7 +15,7 @@ metadata:
 
 ## Preview Guidelines
 
-Use the Flutter Widget Previewer to render widgets in real-time, isolated from the full application context. 
+Use the Flutter Widget Previewer to render widgets in real-time, isolated from the full application context.
 
 - **Target Elements:** Apply the `@Preview` annotation to top-level functions, static methods within a class, or public widget constructors/factories that have no required arguments and return a `Widget` or `WidgetBuilder`.
 - **Imports:** Always import `package:flutter/widget_previews.dart` to access the preview annotations.
@@ -98,7 +98,7 @@ final class TransformativePreview extends Preview {
   Preview transform() {
     final originalPreview = super.transform();
     final builder = originalPreview.toBuilder();
-    
+
     builder
       ..name = 'Transformed - ${originalPreview.name}'
       ..theme = _themeBuilder;

.agents/skills/flutter-add-widget-test/SKILL.md

@@ -50,10 +50,10 @@ Copy the following checklist to track progress when implementing a new widget te
 Apply the following conditional logic based on the type of interaction or state change being tested:
 
 *   **If testing static rendering:** Call `await tester.pumpWidget()` once, then immediately run `expect()` assertions.
-*   **If testing standard state changes (e.g., button taps):** 
+*   **If testing standard state changes (e.g., button taps):**
     1. Call `await tester.tap(finder)`.
     2. Call `await tester.pump()` to trigger a single frame rebuild.
-*   **If testing animations, transitions, or asynchronous UI updates:** 
+*   **If testing animations, transitions, or asynchronous UI updates:**
     1. Trigger the action (e.g., `await tester.drag(finder, Offset(500, 0))`).
     2. Call `await tester.pumpAndSettle()` to repeatedly pump frames until no more frames are scheduled (animation completes).
 *   **If testing text input:** Call `await tester.enterText(textFieldFinder, 'Input string')`.

.agents/skills/flutter-apply-architecture-best-practices/SKILL.md

@@ -83,13 +83,13 @@ class ApiClient {
 // 2. Repository (Single source of truth, returns Domain Model)
 class UserRepository {
   UserRepository({required ApiClient apiClient}) : _apiClient = apiClient;
-  
+
   final ApiClient _apiClient;
   User? _cachedUser;
 
   Future<User> getUser(String id) async {
     if (_cachedUser != null) return _cachedUser!;
-    
+
     final apiModel = await _apiClient.fetchUser(id);
     _cachedUser = User(id: apiModel.id, name: apiModel.fullName); // Transform to Domain Model
     return _cachedUser!;
@@ -102,7 +102,7 @@ class UserRepository {
 ```dart
 // 3. ViewModel (State management and presentation logic)
 class ProfileViewModel extends ChangeNotifier {
-  ProfileViewModel({required UserRepository userRepository}) 
+  ProfileViewModel({required UserRepository userRepository})
       : _userRepository = userRepository;
 
   final UserRepository _userRepository;
@@ -140,7 +140,7 @@ class ProfileView extends StatelessWidget {
         if (viewModel.isLoading) {
           return const Center(child: CircularProgressIndicator());
         }
-        
+
         final user = viewModel.user;
         if (user == null) {
           return const Center(child: Text('User not found'));

.agents/skills/flutter-fix-layout-issues/SKILL.md

@@ -14,7 +14,7 @@ metadata:
 
 ## Constraint Violation Diagnostics
 
-Flutter layout operates on a strict rule: **Constraints go down. Sizes go up. Parent sets position.** Layout errors occur when this negotiation fails, typically due to unbounded constraints or unconstrained children. 
+Flutter layout operates on a strict rule: **Constraints go down. Sizes go up. Parent sets position.** Layout errors occur when this negotiation fails, typically due to unbounded constraints or unconstrained children.
 
 Diagnose layout failures using the following error signatures:
 
@@ -85,7 +85,7 @@ Column(
 Row(
   children: [
     const Icon(Icons.search),
-    TextField(), 
+    TextField(),
   ],
 )
 ```

.agents/skills/flutter-implement-json-serialization/SKILL.md

@@ -48,7 +48,7 @@ Use this conditional workflow when retrieving and parsing JSON from a network re
 - [ ] Decode and map the JSON to the model.
 
 1. **Execute Request**: Use the `http` package to perform the network call.
-2. **Validate Response**: 
+2. **Validate Response**:
    - If `response.statusCode == 200` (or 201 for POST), proceed to parsing.
    - If the status code indicates failure, throw an `Exception`.
 3. **Determine Parsing Strategy**:
@@ -81,7 +81,7 @@ class User {
         'id': int id,
         'name': String name,
         'email': String email,
-      } => 
+      } =>
         User(
           id: id,
           name: name,

.agents/skills/flutter-setup-declarative-routing/SKILL.md

@@ -16,7 +16,7 @@ metadata:
 
 ## Core Concepts
 
-Use the `go_router` package for declarative routing in Flutter. It provides a robust API for complex routing scenarios, deep linking, and nested navigation. 
+Use the `go_router` package for declarative routing in Flutter. It provides a robust API for complex routing scenarios, deep linking, and nested navigation.
 
 - **GoRouter**: The central configuration object defining the application's route tree.
 - **GoRoute**: A standard route mapping a URL path to a Flutter screen.
@@ -120,7 +120,7 @@ Configure the native platforms to intercept specific URLs and route them into th
 ```
 
 ### If configuring for iOS:
-1. **Modify `Info.plist`**: Opt-in to Flutter's default deep link handler. 
+1. **Modify `Info.plist`**: Opt-in to Flutter's default deep link handler.
 *Note: If using a third-party deep linking plugin (e.g., `app_links`), set this to `NO` to prevent conflicts.*
 ```xml
 <key>FlutterDeepLinkingEnabled</key>

.agents/skills/flutter-setup-localization/SKILL.md

@@ -190,8 +190,8 @@ class GreetingWidget extends StatelessWidget {
   final int notificationCount;
 
   const GreetingWidget({
-    super.key, 
-    required this.userName, 
+    super.key,
+    required this.userName,
     required this.notificationCount,
   });