style: dart format + whitespace normalization (pre-push hook)

Author: leduckhc

app/lib/main.dart

@@ -57,9 +57,7 @@ Future<void> main() async {
       input: input,
     );
     if (body == null) return;
-    container
-        .read(connectionControllerProvider.notifier)
-        .respondTo(rid, body);
+    container.read(connectionControllerProvider.notifier).respondTo(rid, body);
   };
   container.read(notificationControllerProvider);
 

app/lib/notifications/notification_request.dart

@@ -125,8 +125,7 @@ String encodeRequestPayload({
   required String sessionId,
   required String requestId,
   required String kind,
-}) =>
-    jsonEncode({'sid': sessionId, 'rid': requestId, 'kind': kind});
+}) => jsonEncode({'sid': sessionId, 'rid': requestId, 'kind': kind});
 
 /// Decode a notification payload. Handles three cases:
 ///   - JSON object → structured [NotificationPayload]

app/lib/transport/ws_client.dart

@@ -147,13 +147,7 @@ class WsClient implements Transport {
       cancelOnError: true,
     );
 
-    _send(
-      Envelope(
-        t: MsgType.hello,
-        id: Ulid().toString(),
-        body: _helloBody,
-      ),
-    );
+    _send(Envelope(t: MsgType.hello, id: Ulid().toString(), body: _helloBody));
 
     _attempt = 0;
     _setState(WsState.connected);

app/lib/ui/widgets/srv_request_handler.dart

@@ -142,7 +142,6 @@ class _SrvRequestHandlerState extends ConsumerState<SrvRequestHandler>
   Future<void> _presentDialog(Envelope env) async {
     final kind = env.body['kind'] as String? ?? 'unknown';
 
-
     // Use the router's Navigator, not this widget's context — we're above it.
     final navCtx = pinoNavigatorKey.currentContext;
     if (navCtx == null) return;

app/test/notification_request_test.dart

@@ -17,35 +17,39 @@ void main() {
       expect(n.body, contains('rm -rf build/'));
     });
 
-    test('askUserQuestion (single form) → question category + first question',
-        () {
-      final n = notificationForRequest(
-        kind: 'askUserQuestion',
-        body: {'kind': 'askUserQuestion', 'question': 'Deploy to prod?'},
-        label: label,
-      );
-      expect(n, isNotNull);
-      expect(n!.category, kQuestionCategoryId);
-      expect(n.body, contains('Deploy to prod?'));
-    });
-
-    test('askUserQuestion (wizard form) → question category + first question',
-        () {
-      final n = notificationForRequest(
-        kind: 'askUserQuestion',
-        body: {
-          'kind': 'askUserQuestion',
-          'questions': [
-            {'question': 'Pick a branch'},
-            {'question': 'ignored second'},
-          ],
-        },
-        label: label,
-      );
-      expect(n, isNotNull);
-      expect(n!.category, kQuestionCategoryId);
-      expect(n.body, contains('Pick a branch'));
-    });
+    test(
+      'askUserQuestion (single form) → question category + first question',
+      () {
+        final n = notificationForRequest(
+          kind: 'askUserQuestion',
+          body: {'kind': 'askUserQuestion', 'question': 'Deploy to prod?'},
+          label: label,
+        );
+        expect(n, isNotNull);
+        expect(n!.category, kQuestionCategoryId);
+        expect(n.body, contains('Deploy to prod?'));
+      },
+    );
+
+    test(
+      'askUserQuestion (wizard form) → question category + first question',
+      () {
+        final n = notificationForRequest(
+          kind: 'askUserQuestion',
+          body: {
+            'kind': 'askUserQuestion',
+            'questions': [
+              {'question': 'Pick a branch'},
+              {'question': 'ignored second'},
+            ],
+          },
+          label: label,
+        );
+        expect(n, isNotNull);
+        expect(n!.category, kQuestionCategoryId);
+        expect(n.body, contains('Pick a branch'));
+      },
+    );
 
     test('input kind → null', () {
       final n = notificationForRequest(
@@ -157,10 +161,7 @@ void main() {
 
     test('approve/deny require confirmAction kind → mismatch is null', () {
       expect(
-        responseForAction(
-          kind: 'askUserQuestion',
-          actionId: kApproveActionId,
-        ),
+        responseForAction(kind: 'askUserQuestion', actionId: kApproveActionId),
         isNull,
       );
       expect(

app/test/notification_service_test.dart

@@ -21,20 +21,22 @@ void main() {
       expect(decoded['actionId'], 'pino_approve');
     });
 
-    test('caps the queue to the most recent kMaxPendingActions entries',
-        () async {
-      for (var i = 0; i < kMaxPendingActions + 20; i++) {
-        await persistPendingActionForTest('p$i', 'pino_approve', null);
-      }
+    test(
+      'caps the queue to the most recent kMaxPendingActions entries',
+      () async {
+        for (var i = 0; i < kMaxPendingActions + 20; i++) {
+          await persistPendingActionForTest('p$i', 'pino_approve', null);
+        }
 
-      final prefs = await SharedPreferences.getInstance();
-      final queue = prefs.getStringList(kPendingActionsKey)!;
-      expect(queue, hasLength(kMaxPendingActions));
-      // Oldest entries dropped; newest retained.
-      final first = jsonDecode(queue.first) as Map<String, dynamic>;
-      final last = jsonDecode(queue.last) as Map<String, dynamic>;
-      expect(first['payload'], 'p20');
-      expect(last['payload'], 'p${kMaxPendingActions + 19}');
-    });
+        final prefs = await SharedPreferences.getInstance();
+        final queue = prefs.getStringList(kPendingActionsKey)!;
+        expect(queue, hasLength(kMaxPendingActions));
+        // Oldest entries dropped; newest retained.
+        final first = jsonDecode(queue.first) as Map<String, dynamic>;
+        final last = jsonDecode(queue.last) as Map<String, dynamic>;
+        expect(first['payload'], 'p20');
+        expect(last['payload'], 'p${kMaxPendingActions + 19}');
+      },
+    );
   });
 }

app/test/srv_request_handler_notify_test.dart

@@ -123,11 +123,7 @@ _FakeSecureStorage _seeded() => _FakeSecureStorage({
 
 void main() {
   Future<
-    (
-      _EmittingTransport,
-      _RecordingNotificationService,
-      ConnectionController,
-    )
+    (_EmittingTransport, _RecordingNotificationService, ConnectionController)
   >
   pumpHandler(
     WidgetTester tester, {
@@ -138,8 +134,8 @@ void main() {
     final controller = ConnectionController(
       _seeded(),
       transportFactory: () => transport,
-      browseLan:
-          ({Duration timeout = const Duration(seconds: 3)}) async => const [],
+      browseLan: ({Duration timeout = const Duration(seconds: 3)}) async =>
+          const [],
       rediscoverStall: Duration.zero,
     );
 
@@ -290,40 +286,39 @@ void main() {
     },
   );
 
-  testWidgets(
-    'backgrounded confirmAction is replayed as a dialog on resume',
-    (tester) async {
-      final (transport, notifications, _) = await pumpHandler(tester);
-      tester.binding.handleAppLifecycleStateChanged(AppLifecycleState.paused);
-      await tester.pump();
+  testWidgets('backgrounded confirmAction is replayed as a dialog on resume', (
+    tester,
+  ) async {
+    final (transport, notifications, _) = await pumpHandler(tester);
+    tester.binding.handleAppLifecycleStateChanged(AppLifecycleState.paused);
+    await tester.pump();
 
-      transport.emit(
-        Envelope(
-          t: MsgType.srvRequest,
-          id: 'req-replay',
-          body: {
-            'kind': 'confirmAction',
-            'action': 'rm -rf build/',
-            'sessionId': 's1',
-          },
-        ),
-      );
-      await tester.pump();
-      await tester.pump();
+    transport.emit(
+      Envelope(
+        t: MsgType.srvRequest,
+        id: 'req-replay',
+        body: {
+          'kind': 'confirmAction',
+          'action': 'rm -rf build/',
+          'sessionId': 's1',
+        },
+      ),
+    );
+    await tester.pump();
+    await tester.pump();
 
-      // Notification shown, dialog withheld while backgrounded.
-      expect(notifications.shown, hasLength(1));
-      expect(find.text('Approve'), findsNothing);
+    // Notification shown, dialog withheld while backgrounded.
+    expect(notifications.shown, hasLength(1));
+    expect(find.text('Approve'), findsNothing);
 
-      // Resume and pump past the drain delay → dialog now presented.
-      tester.binding.handleAppLifecycleStateChanged(AppLifecycleState.resumed);
-      await tester.pump();
-      await tester.pump(const Duration(milliseconds: 500));
-      await tester.pump();
+    // Resume and pump past the drain delay → dialog now presented.
+    tester.binding.handleAppLifecycleStateChanged(AppLifecycleState.resumed);
+    await tester.pump();
+    await tester.pump(const Duration(milliseconds: 500));
+    await tester.pump();
 
-      expect(find.text('Approve'), findsOneWidget);
-    },
-  );
+    expect(find.text('Approve'), findsOneWidget);
+  });
 
   testWidgets(
     'answered-from-notification request is not double-prompted on resume',

app/test/store_reducer_test.dart

@@ -187,7 +187,9 @@ void main() {
                 }),
               }),
               transportFactory: () => transport,
-              browseLan: ({Duration timeout = const Duration(seconds: 3)}) async => const [],
+              browseLan:
+                  ({Duration timeout = const Duration(seconds: 3)}) async =>
+                      const [],
               rediscoverStall: const Duration(seconds: 30),
             ),
           ),
@@ -226,7 +228,9 @@ void main() {
                 }),
               }),
               transportFactory: () => transport,
-              browseLan: ({Duration timeout = const Duration(seconds: 3)}) async => const [],
+              browseLan:
+                  ({Duration timeout = const Duration(seconds: 3)}) async =>
+                      const [],
               rediscoverStall: const Duration(seconds: 30),
             ),
           ),

docs/ARCHITECT-SLICE-1.md

@@ -47,23 +47,23 @@
 test("pi adapter's askUser callback sends srv.request", async () => {
   const sent: OutgoingFrame[] = [];
   const rpc = new ReverseRpc({ send: (f) => sent.push(f) });
-  
+
   const adapter = new PiAdapter();
-  
+
   // Inject askUser callback that feeds to rpc
   await adapter.start({
     cwd: "/tmp",
     sessionId: "sess-1",
     askUser: (req) => rpc.askDevice(req, "sess-1"),
   });
-  
+
   // Fake pi emitting a ui.select
   adapter.emit("event", {
     kind: "agent.message",
     payload: { text: "... choices: [a, b]" },
   });
   // TODO: simulate the actual pi ui.select RPC that triggers askUser
-  
+
   const req = sent.find((f) => f.t === "srv.request");
   assert.ok(req);
   assert.deepEqual(req.body.questions[0].options, ["a", "b"]);
@@ -92,7 +92,7 @@ const resp = await this.askUser?.({
 ```dart
 test("ApprovalDialog renders on srv.request frame", () {
   final tester = WidgetTester();
-  
+
   // Inject a fake StoreController
   final controller = FakeStoreController()
     ..incomingFrames.add(Envelope(
@@ -106,11 +106,11 @@ test("ApprovalDialog renders on srv.request frame", () {
         }],
       },
     ));
-  
+
   await tester.pumpWidget(MaterialApp(
     home: ApprovalDialog(),
   ));
-  
+
   expect(find.text("Approve tool call?"), findsOneWidget);
   expect(find.byType(ElevatedButton), findsWidgets(count: 2));
 });
@@ -131,11 +131,11 @@ test("ApprovalDialog sends srv.response on button tap", () async {
   final outgoing = <Envelope>[];
   final controller = FakeStoreController()
     ..onSend = (env) => outgoing.add(env);
-  
+
   await tester.pumpWidget(/* as above */);
   await tester.tap(find.text("Approve"));
   await tester.pumpAndSettle();
-  
+
   final resp = outgoing.whereType<Envelope>().firstWhere(
     (e) => e.t == MsgType.srvResponse && e.id == "req-1"
   );
@@ -151,7 +151,7 @@ test("ApprovalDialog sends srv.response on button tap", () async {
 
 #### Test: `app/test/e2e/approval_e2e_test.dart` — `session awaiting approval flows end-to-end`
 
-**Red:** 
+**Red:**
 1. Stub server emits `session.status { status: "awaiting-approval" }`
 2. App renders approval UI
 3. User taps "Approve"
@@ -162,7 +162,7 @@ test("e2e: awaiting-approval → approval UI → srv.response → session contin
   // 1. Spin up stub server + app
   final stubServer = StubServer();
   await app.connect(stubServer.url);
-  
+
   // 2. Stub: emit session.status awaiting-approval + srv.request
   stubServer.emitEvent(SessionEvent(
     kind: "session.status",
@@ -173,15 +173,15 @@ test("e2e: awaiting-approval → approval UI → srv.response → session contin
     id: "req-123",
     body: { "questions": [{ "options": ["Yes", "No"] }] },
   ));
-  
+
   // 3. App renders approval UI
   await tester.pumpAndSettle();
   expect(find.text("Approve?"), findsOneWidget);
-  
+
   // 4. User taps
   await tester.tap(find.text("Yes"));
   await tester.pumpAndSettle();
-  
+
   // 5. Verify stub server saw srv.response
   final resp = stubServer.capturedFrames.whereType<Envelope>()
     .firstWhere((e) => e.t == MsgType.srvResponse && e.id == "req-123");