refactor: split bridge_flutter_handlers into 8 sub-handlers

bridge_flutter_handlers.dart (989→35 lines) now delegates to:
- bf_inspection.dart: inspect, snapshot, widget tree (172 lines)
- bf_interaction.dart: tap, enter_text, swipe, gesture (155 lines)
- bf_screenshot.dart: screenshot, visual verify/diff (245 lines)
- bf_navigation.dart: route, go_back, nav stack (23 lines)
- bf_logging.dart: logs, errors, network monitoring (131 lines)
- bf_batch.dart: execute_batch, coordinate actions (103 lines)
- bf_assertions.dart: assert_visible/text/count (78 lines)
- bf_state.dart: page state, diagnostics, plugins (61 lines)

dart analyze: 0 errors.

Author: GitLab CI

lib/src/cli/server.dart

@@ -28,6 +28,14 @@ part 'tool_handlers/flutter_helpers.dart';
 part 'tool_handlers/plugin_handlers.dart';
 part 'tool_handlers/discovery_helpers.dart';
 part 'tool_handlers/bridge_flutter_handlers.dart';
+part 'tool_handlers/bf_inspection.dart';
+part 'tool_handlers/bf_interaction.dart';
+part 'tool_handlers/bf_screenshot.dart';
+part 'tool_handlers/bf_navigation.dart';
+part 'tool_handlers/bf_logging.dart';
+part 'tool_handlers/bf_batch.dart';
+part 'tool_handlers/bf_assertions.dart';
+part 'tool_handlers/bf_state.dart';
 part 'tool_handlers/connection_handlers.dart';
 part 'tool_handlers/cdp_connection_handlers.dart';
 part 'tool_handlers/dev_tool_handlers.dart';

lib/src/cli/tool_handlers/bf_assertions.dart

@@ -0,0 +1,88 @@
+part of '../server.dart';
+
+extension _BfAssertions on FlutterMcpServer {
+  Future<dynamic> _handleAssertionTool(String name, Map<String, dynamic> args, AppDriver? client) async {
+    switch (name) {
+      case 'assert_batch':
+        final assertions = (args['assertions'] as List<dynamic>?) ?? [];
+        final results = <Map<String, dynamic>>[];
+        int passed = 0;
+        int failed = 0;
+        for (final assertion in assertions) {
+          final a = assertion as Map<String, dynamic>;
+          final aType = a['type'] as String;
+          try {
+            final toolName = aType == 'visible' ? 'assert_visible'
+                : aType == 'not_visible' ? 'assert_not_visible'
+                : aType == 'text' ? 'assert_text'
+                : aType == 'element_count' ? 'assert_element_count'
+                : aType;
+            final toolArgs = <String, dynamic>{
+              if (a['key'] != null) 'key': a['key'],
+              if (a['text'] != null) 'text': a['text'],
+              if (a['expected'] != null) 'expected': a['expected'],
+              if (a['count'] != null) 'expected_count': a['count'],
+            };
+            final result = await _executeToolInner(toolName, toolArgs);
+            final success = result is Map && result['success'] == true;
+            if (success) passed++; else failed++;
+            results.add({'type': aType, 'success': success, 'result': result});
+          } catch (e) {
+            failed++;
+            results.add({'type': aType, 'success': false, 'error': e.toString()});
+          }
+        }
+        return {
+          'success': failed == 0,
+          'total': assertions.length,
+          'passed': passed,
+          'failed': failed,
+          'results': results,
+        };
+
+      // === NEW: Assertions ===
+      case 'assert_visible':
+        if (client is BridgeDriver) {
+          final found = await client.findElement(key: args['key'], text: args['text']);
+          final isVisible = found.isNotEmpty && found['found'] == true;
+          return {"success": isVisible, "visible": isVisible, "message": isVisible ? "Element is visible" : "Element not found"};
+        }
+        final fc = _asFlutterClient(client!, 'assert_visible');
+        return await _assertVisible(args, fc, shouldBeVisible: true);
+
+      case 'assert_not_visible':
+        if (client is BridgeDriver) {
+          final found = await client.findElement(key: args['key'], text: args['text']);
+          final isGone = found.isEmpty || found['found'] != true;
+          return {"success": isGone, "visible": !isGone, "message": isGone ? "Element is not visible" : "Element is still visible"};
+        }
+        final fc = _asFlutterClient(client!, 'assert_not_visible');
+        return await _assertVisible(args, fc, shouldBeVisible: false);
+
+      case 'assert_text':
+        if (client is BridgeDriver) {
+          final actual = await client.getText(key: args['key']);
+          final expected = args['expected'] as String?;
+          final matches = actual == expected;
+          return {"success": matches, "actual": actual, "expected": expected, "message": matches ? "Text matches" : "Text mismatch"};
+        }
+        final fc = _asFlutterClient(client!, 'assert_text');
+        return await _assertText(args, fc);
+
+      case 'assert_element_count':
+        if (client is BridgeDriver) {
+          final elements = await client.getInteractiveElements();
+          final count = elements.length;
+          final expected = args['expected'] as int?;
+          final matches = expected == null || count == expected;
+          return {"success": matches, "count": count, "expected": expected, "message": matches ? "Count matches" : "Expected $expected but found $count"};
+        }
+        final fc = _asFlutterClient(client!, 'assert_element_count');
+        return await _assertElementCount(args, fc);
+
+      // === NEW: Page State ===
+      default:
+        return null;
+    }
+  }
+}

lib/src/cli/tool_handlers/bf_batch.dart

@@ -0,0 +1,113 @@
+part of '../server.dart';
+
+extension _BfBatch on FlutterMcpServer {
+  Future<dynamic> _handleBatchCoordTool(String name, Map<String, dynamic> args, AppDriver? client) async {
+    switch (name) {
+      case 'execute_batch':
+        if (client is BridgeDriver) {
+          final actions = args['actions'] as List? ?? [];
+          final results = <Map<String, dynamic>>[];
+          for (final action in actions) {
+            if (action is Map<String, dynamic>) {
+              final toolName = action['tool'] as String?;
+              final toolArgs = Map<String, dynamic>.from(action['args'] as Map? ?? {});
+              if (toolName != null) {
+                try {
+                  final result = await client.callMethod(toolName, toolArgs);
+                  results.add({'tool': toolName, 'success': true, 'result': result});
+                } catch (e) {
+                  results.add({'tool': toolName, 'success': false, 'error': e.toString()});
+                }
+              }
+            }
+          }
+          return {"success": true, "results": results, "count": results.length};
+        }
+        final fc = _asFlutterClient(client!, 'execute_batch');
+        return await _executeBatch(args, fc);
+
+      // === NEW: Coordinate-based Actions ===
+      case 'tap_at':
+        if (client is BridgeDriver) {
+          await client.callMethod('tap_at', {'x': (args['x'] as num).toDouble(), 'y': (args['y'] as num).toDouble()});
+          return {"success": true, "action": "tap_at", "x": args['x'], "y": args['y']};
+        }
+        final fc = _asFlutterClient(client!, 'tap_at');
+        final x = (args['x'] as num).toDouble();
+        final y = (args['y'] as num).toDouble();
+        await fc.tapAt(x, y);
+        return {"success": true, "action": "tap_at", "x": x, "y": y};
+
+      case 'long_press_at':
+        if (client is BridgeDriver) {
+          await client.callMethod('long_press_at', {'x': (args['x'] as num).toDouble(), 'y': (args['y'] as num).toDouble(), 'duration': args['duration'] ?? 500});
+          return {"success": true, "action": "long_press_at", "x": args['x'], "y": args['y']};
+        }
+        final fc = _asFlutterClient(client!, 'long_press_at');
+        final x = (args['x'] as num).toDouble();
+        final y = (args['y'] as num).toDouble();
+        final duration = args['duration'] ?? 500;
+        await fc.longPressAt(x, y, duration: duration);
+        return {"success": true, "action": "long_press_at", "x": x, "y": y};
+
+      case 'swipe_coordinates':
+        if (client is BridgeDriver) {
+          await client.callMethod('swipe_coordinates', {
+            'start_x': ((args['start_x'] ?? args['startX']) as num).toDouble(),
+            'start_y': ((args['start_y'] ?? args['startY']) as num).toDouble(),
+            'end_x': ((args['end_x'] ?? args['endX']) as num).toDouble(),
+            'end_y': ((args['end_y'] ?? args['endY']) as num).toDouble(),
+            'duration': args['duration'] ?? args['durationMs'] ?? 300,
+          });
+          return {"success": true, "action": "swipe_coordinates"};
+        }
+        final fc = _asFlutterClient(client!, 'swipe_coordinates');
+        final startX = ((args['start_x'] ?? args['startX']) as num).toDouble();
+        final startY = ((args['start_y'] ?? args['startY']) as num).toDouble();
+        final endX = ((args['end_x'] ?? args['endX']) as num).toDouble();
+        final endY = ((args['end_y'] ?? args['endY']) as num).toDouble();
+        final duration = args['duration'] ?? 300;
+        await fc.swipeCoordinates(startX, startY, endX, endY,
+            duration: duration);
+        return {"success": true, "action": "swipe_coordinates"};
+
+      case 'edge_swipe':
+        if (client is BridgeDriver) {
+          return await client.callMethod('edge_swipe', {'edge': args['edge'], 'direction': args['direction'], 'distance': (args['distance'] as num?)?.toDouble() ?? 200});
+        }
+        final fc = _asFlutterClient(client!, 'edge_swipe');
+        final edge = args['edge'] as String;
+        final direction = args['direction'] as String;
+        final distance = (args['distance'] as num?)?.toDouble() ?? 200;
+        final result = await fc.edgeSwipe(
+            edge: edge, direction: direction, distance: distance);
+        return result;
+
+      case 'gesture':
+        if (client is BridgeDriver) {
+          return await client.callMethod('gesture', args);
+        }
+        final fc = _asFlutterClient(client!, 'gesture');
+        return await _performGesture(args, fc);
+
+      case 'wait_for_idle':
+        if (client is BridgeDriver) {
+          return {"success": true, "message": "Bridge platform ready"};
+        }
+        final fc = _asFlutterClient(client!, 'wait_for_idle');
+        return await _waitForIdle(args, fc);
+
+      // === NEW: Smart Scroll ===
+      case 'scroll_until_visible':
+        if (client is BridgeDriver) {
+          return await client.callMethod('scroll_until_visible', {'key': args['key'], 'text': args['text'], 'direction': args['direction'] ?? 'down', 'max_scrolls': args['max_scrolls'] ?? 10});
+        }
+        final fc = _asFlutterClient(client!, 'scroll_until_visible');
+        return await _scrollUntilVisible(args, fc);
+
+      // === Batch Assertions ===
+      default:
+        return null;
+    }
+  }
+}