feature: support physical android devices (#5)

Author: dungngminh

.github/workflows/deploy-homebrew.yaml

@@ -3,7 +3,6 @@ name: Deploy to Homebrew
 on:
   release:
     types: [released]
-  workflow_call:
   workflow_dispatch:
 
 jobs:

.github/workflows/deploy-pub-dev.yaml

@@ -1,7 +1,9 @@
 name: Deploy to pub.dev
 
 on:
-  workflow_call:
+  push:
+    tags:
+      - 'v*'
   workflow_dispatch:
 
 jobs:

.github/workflows/deploy-winget.yaml

@@ -1,8 +1,8 @@
 name: Deploy to WinGet
 
 on:
-  release:
-    types: [released]
+  # release:
+  #   types: [released]
   workflow_call:
   workflow_dispatch:
 

.github/workflows/release.yaml

@@ -103,12 +103,6 @@ jobs:
             artifacts/*.zip
             artifacts/checksums.txt
 
-  # deploy-pub-dev:
-  #   name: Deploy to pub.dev
-  #   needs: release
-  #   uses: ./.github/workflows/deploy-pub-dev.yaml
-  #   secrets: inherit
-
   # deploy-homebrew:
   #   name: Deploy to Homebrew
   #   needs: release

CHANGELOG.md

@@ -5,7 +5,15 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-<!-- ## [Unreleased] -->
+## [0.1.0] - 2026-03-20
+
+### Added
+
+- Support Android physical devices
+
+### Changed
+
+- Reopen ADB tools menu
 
 ## [0.0.4] - 2026-03-17
 

lib/components/device_detail_panel.dart

@@ -25,11 +25,11 @@ class DeviceDetailPanel extends StatelessComponent {
     return Column(
       crossAxisAlignment: CrossAxisAlignment.start,
       children: [
-        _row(st, 'Name', device.name),
-        _row(st, 'ID', device.id),
-        _row(st, 'Platform', device.platform),
-        _row(st, 'Type', device.os.name),
-        _row(st, 'State', device.state.label),
+        _row(st, label: 'Name', value: device.name),
+        _row(st, label: 'ID', value: device.id),
+        _row(st, label: 'Platform', value: device.platform),
+        _row(st, label: 'Type', value: device.os.label),
+        _row(st, label: 'State', value: device.state.label),
       ],
     );
   }
@@ -40,7 +40,11 @@ class DeviceDetailPanel extends StatelessComponent {
     );
   }
 
-  Component _row(SimutilTheme st, String label, String value) {
+  Component _row(
+    SimutilTheme st, {
+    required String label,
+    required String value,
+  }) {
     return Row(
       children: [
         SizedBox(width: 12, child: Text(label, style: st.label)),

lib/components/device_list_component.dart

@@ -2,13 +2,15 @@ import 'package:nocterm/nocterm.dart';
 import 'package:simutil/components/simutil_icons.dart';
 import 'package:simutil/components/simutil_theme.dart';
 import 'package:simutil/models/device.dart';
+import 'package:simutil/models/device_type.dart';
 
 class DeviceListComponent extends StatefulComponent {
   const DeviceListComponent({
     super.key,
     required this.devices,
     this.focused = false,
     this.selectedIndex = 0,
+    this.scrollBufferItems = 2,
     this.onSelectionChanged,
     this.onDeviceLaunch,
     this.onDeviceShowOptions,
@@ -20,6 +22,7 @@ class DeviceListComponent extends StatefulComponent {
   final List<Device> devices;
   final bool focused;
   final int selectedIndex;
+  final int scrollBufferItems;
   final ValueChanged<int>? onSelectionChanged;
   final ValueChanged<Device>? onDeviceLaunch;
   final ValueChanged<Device>? onDeviceShowOptions;
@@ -32,6 +35,14 @@ class DeviceListComponent extends StatefulComponent {
 }
 
 class _DeviceListComponentState extends State<DeviceListComponent> {
+  late final ScrollController _scrollController = ScrollController();
+
+  @override
+  void dispose() {
+    _scrollController.dispose();
+    super.dispose();
+  }
+
   @override
   Component build(BuildContext context) {
     final st = SimutilTheme.of(context);
@@ -60,6 +71,7 @@ class _DeviceListComponentState extends State<DeviceListComponent> {
       focused: component.focused,
       onKeyEvent: _handleKeyEvent,
       child: ListView.builder(
+        controller: _scrollController,
         itemCount: component.devices.length,
         itemBuilder: (context, index) {
           final device = component.devices[index];
@@ -76,44 +88,64 @@ class _DeviceListComponentState extends State<DeviceListComponent> {
 
   bool _handleKeyEvent(KeyboardEvent event) {
     if (component.devices.isEmpty) return false;
-
-    if (event.logicalKey == LogicalKey.arrowUp) {
-      final newIndex = (component.selectedIndex - 1).clamp(
-        0,
-        component.devices.length - 1,
-      );
-      component.onSelectionChanged?.call(newIndex);
-      return true;
+    switch (event.logicalKey) {
+      case LogicalKey.arrowUp:
+        _handleArrowUp();
+        return true;
+      case LogicalKey.arrowDown:
+        _handleArrowDown();
+        return true;
+      case LogicalKey.enter:
+        _handleEnter();
+        return true;
+      case LogicalKey.space:
+        _handleSpace();
+        return true;
+      default:
+        return false;
     }
+  }
 
-    if (event.logicalKey == LogicalKey.arrowDown) {
-      final newIndex = (component.selectedIndex + 1).clamp(
-        0,
-        component.devices.length - 1,
-      );
-      component.onSelectionChanged?.call(newIndex);
-      return true;
-    }
+  void _handleArrowUp() {
+    final newIndex = (component.selectedIndex - 1).clamp(
+      0,
+      component.devices.length - 1,
+    );
+    component.onSelectionChanged?.call(newIndex);
+    final scrollTarget = (newIndex - component.scrollBufferItems).clamp(
+      0,
+      component.devices.length - 1,
+    );
+    _scrollController.ensureIndexVisible(index: scrollTarget);
+  }
 
-    if (event.logicalKey == LogicalKey.enter) {
-      if (component.selectedIndex < component.devices.length) {
-        component.onDeviceLaunch?.call(
-          component.devices[component.selectedIndex],
-        );
-      }
-      return true;
-    }
+  void _handleArrowDown() {
+    final newIndex = (component.selectedIndex + 1).clamp(
+      0,
+      component.devices.length - 1,
+    );
+    component.onSelectionChanged?.call(newIndex);
+    final scrollTarget = (newIndex + component.scrollBufferItems).clamp(
+      0,
+      component.devices.length - 1,
+    );
+    _scrollController.ensureIndexVisible(index: scrollTarget);
+  }
 
-    if (event.logicalKey == LogicalKey.space) {
-      if (component.selectedIndex < component.devices.length) {
-        component.onDeviceShowOptions?.call(
-          component.devices[component.selectedIndex],
-        );
-      }
-      return true;
+  void _handleEnter() {
+    if (component.selectedIndex < component.devices.length) {
+      component.onDeviceLaunch?.call(
+        component.devices[component.selectedIndex],
+      );
     }
+  }
 
-    return false;
+  void _handleSpace() {
+    if (component.selectedIndex < component.devices.length) {
+      component.onDeviceShowOptions?.call(
+        component.devices[component.selectedIndex],
+      );
+    }
   }
 }
 
@@ -135,7 +167,10 @@ class _DeviceRow extends StatelessComponent {
           child: Text(device.name, style: isSelected ? st.selected : st.body),
         ),
         Text('${device.platform} ', style: st.muted),
-        Text('${device.state.label} ', style: stateStyle),
+        if (device.type == DeviceType.simulator)
+          Text('${device.state.label} ', style: stateStyle)
+        else
+          Text('Physical', style: st.muted),
       ],
     );
   }

lib/models/device.dart

@@ -1,6 +1,6 @@
 import 'package:simutil/models/device_state.dart';
 import 'package:simutil/models/device_type.dart';
-import 'package:simutil/models/os.dart';
+import 'package:simutil/models/device_os.dart';
 
 class Device {
   const Device({
@@ -12,11 +12,44 @@ class Device {
     required this.type,
   });
 
+  factory Device.android({
+    required String id,
+    required String name,
+    required DeviceState state,
+    required DeviceType type,
+  }) {
+    return Device(
+      id: id,
+      name: name,
+      platform: 'Android',
+      os: DeviceOs.android,
+      state: state,
+      type: type,
+    );
+  }
+
+  factory Device.ios({
+    required String id,
+    required String name,
+    String? platform,
+    required DeviceState state,
+    required DeviceType type,
+  }) {
+    return Device(
+      id: id,
+      name: name,
+      platform: platform ?? 'iOS',
+      os: DeviceOs.ios,
+      state: state,
+      type: type,
+    );
+  }
+
   factory Device.fromJson(Map<String, dynamic> json) {
     return Device(
       id: json['id'] as String,
       name: json['name'] as String,
-      os: Os.values.byName(json['type'] as String),
+      os: DeviceOs.values.byName(json['type'] as String),
       platform: json['platform'] as String? ?? '',
       state: DeviceState.fromString(json['state'] as String? ?? 'Shutdown'),
       type: DeviceType.values.byName(json['type'] as String),
@@ -27,7 +60,7 @@ class Device {
 
   final String name;
 
-  final Os os;
+  final DeviceOs os;
 
   final String platform;
 
@@ -40,7 +73,7 @@ class Device {
   Device copyWith({
     String? id,
     String? name,
-    Os? os,
+    DeviceOs? os,
     String? platform,
     DeviceState? state,
     DeviceType? type,
@@ -59,9 +92,10 @@ class Device {
     return {
       'id': id,
       'name': name,
-      'type': os.name,
+      'os': os.name,
       'platform': platform,
       'state': state.label,
+      'type': type.name,
     };
   }
 

lib/models/device_os.dart

@@ -0,0 +1,13 @@
+enum DeviceOs {
+  android,
+  ios;
+
+  String get label {
+    switch (this) {
+      case DeviceOs.android:
+        return 'Android';
+      case DeviceOs.ios:
+        return 'iOS';
+    }
+  }
+}

lib/models/device_type.dart

@@ -1,5 +1,9 @@
 enum DeviceType {
   physical,
 
-  simulator,
+  simulator;
+
+  bool get isPhysical => this == DeviceType.physical;
+  
+  bool get isSimulator => this == DeviceType.simulator;
 }