Add devices command and refactor build/debug initialization

Introduces a new 'devices' command to list connected devices via Flutter. Refactors build and debug commands to use a unified 'initialize_command' method, moving progress and verbose handling to the base class. Updates CLI registration to include the new devices command.

Author: FeodorFitsner

sdk/python/packages/flet-cli/src/flet_cli/cli.py

@@ -5,6 +5,7 @@
 import flet_cli.commands.build
 import flet_cli.commands.create
 import flet_cli.commands.debug
+import flet_cli.commands.devices
 import flet_cli.commands.doctor  # Adding the doctor command
 import flet_cli.commands.pack
 import flet_cli.commands.publish
@@ -83,6 +84,7 @@ def get_parser() -> argparse.ArgumentParser:
     flet_cli.commands.pack.Command.register_to(sp, "pack")
     flet_cli.commands.publish.Command.register_to(sp, "publish")
     flet_cli.commands.serve.Command.register_to(sp, "serve")
+    flet_cli.commands.devices.Command.register_to(sp, "devices")
     flet_cli.commands.doctor.Command.register_to(sp, "doctor")
 
     # set "run" as the default subparser

sdk/python/packages/flet-cli/src/flet_cli/commands/build.py

@@ -2,7 +2,6 @@
 
 from rich.console import Group
 from rich.live import Live
-from rich.progress import Progress
 
 from flet_cli.commands.build_base import BaseBuildCommand, console
 
@@ -42,11 +41,8 @@ def handle(self, options: argparse.Namespace) -> None:
             f"[bold blue]Initializing {self.target_platform} build...",
             spinner="bouncingBall",
         )
-        self.progress = Progress(transient=True)
-        self.no_rich_output = self.no_rich_output or self.options.no_rich_output
-        self.verbose = self.options.verbose
         with Live(Group(self.status, self.progress), console=console) as self.live:
-            self.initialize_build()
+            self.initialize_command()
             self.validate_target_platform()
             self.validate_entry_point()
             self.setup_template_data()

sdk/python/packages/flet-cli/src/flet_cli/commands/build_base.py

@@ -17,13 +17,11 @@
 import flet.version
 import flet_cli.utils.processes as processes
 from flet.utils import copy_tree, is_windows, slugify
-from flet.utils.platform_utils import get_bool_env_var
 from flet.version import update_version
 from flet_cli.commands.flutter_base import (
     BaseFlutterCommand,
     console,
     error_style,
-    no_rich_output,
     verbose1_style,
     verbose2_style,
     warning_style,
@@ -48,34 +46,22 @@ class BaseBuildCommand(BaseFlutterCommand):
     def __init__(self, parser: argparse.ArgumentParser) -> None:
         super().__init__(parser)
 
-        self.env = {}
         self.pubspec_path = None
         self.rel_out_dir = None
         self.assets_path = None
         self.target_platform = None
-        self.debug_platform = None
-        self.package_platform = None
-        self.config_platform = None
         self.flutter_dependencies = {}
         self.package_app_path = None
-        self.options = None
         self.template_data = None
         self.python_module_filename = None
         self.out_dir = None
         self.python_module_name = None
         self.get_pyproject = None
         self.python_app_path = None
-        self.emojis = {}
-        self.dart_exe = None
-        self.verbose = False
         self.build_dir = None
         self.flutter_dir: Optional[Path] = None
         self.flutter_packages_dir = None
         self.flutter_packages_temp_dir = None
-        self.flutter_exe = None
-        self.skip_flutter_doctor = get_bool_env_var("FLET_CLI_SKIP_FLUTTER_DOCTOR")
-        self.no_rich_output = no_rich_output
-        self.current_platform = platform.system()
         self.platforms = {
             "windows": {
                 "package_platform": "Windows",
@@ -607,16 +593,17 @@ def handle(self, options: argparse.Namespace) -> None:
         if "target_platform" in self.options:
             self.target_platform = self.options.target_platform
 
-    def initialize_build(self):
+    def initialize_command(self):
         assert self.options
         assert self.target_platform
-        super().initialize_build()
-
-        self.python_app_path = Path(self.options.python_app_path).resolve()
 
         self.package_platform = self.platforms[self.target_platform]["package_platform"]
         self.config_platform = self.platforms[self.target_platform]["config_platform"]
 
+        super().initialize_command()
+
+        self.python_app_path = Path(self.options.python_app_path).resolve()
+
         if not (
             os.path.exists(self.python_app_path) or os.path.isdir(self.python_app_path)
         ):

sdk/python/packages/flet-cli/src/flet_cli/commands/debug.py

@@ -5,7 +5,6 @@
 
 from rich.console import Group
 from rich.live import Live
-from rich.progress import Progress
 
 from flet_cli.commands.build_base import BaseBuildCommand, console, verbose2_style
 
@@ -17,7 +16,6 @@ class Command(BaseBuildCommand):
 
     def __init__(self, parser: argparse.ArgumentParser) -> None:
         super().__init__(parser)
-
         self.debug_platforms = {
             "windows": {"target_platform": "windows", "device_id": "windows"},
             "macos": {"target_platform": "macos", "device_id": "macos"},
@@ -26,6 +24,7 @@ def __init__(self, parser: argparse.ArgumentParser) -> None:
             "ios": {"target_platform": "ipa", "device_id": None},
             "android": {"target_platform": "apk", "device_id": None},
         }
+        self.debug_platform = None
         self.device_id = None
 
     def add_arguments(self, parser: argparse.ArgumentParser) -> None:
@@ -86,12 +85,9 @@ def handle(self, options: argparse.Namespace) -> None:
             f"[bold blue]Initializing {self.target_platform} debug session...",
             spinner="bouncingBall",
         )
-        self.progress = Progress(transient=True)
-        self.no_rich_output = self.no_rich_output or self.options.no_rich_output
-        self.verbose = self.options.verbose
         with Live(Group(self.status, self.progress), console=console) as self.live:
             self.check_device_id()
-            self.initialize_build()
+            self.initialize_command()
             if self.options.show_devices:
                 self.run_flutter_devices()
                 self.live.update("", refresh=True)

sdk/python/packages/flet-cli/src/flet_cli/commands/devices.py

@@ -0,0 +1,89 @@
+import argparse
+import os
+
+from rich.console import Group
+from rich.live import Live
+
+from flet_cli.commands.build_base import BaseFlutterCommand, console, verbose2_style
+
+
+class Command(BaseFlutterCommand):
+    """
+    List all connected devices.
+    """
+
+    def __init__(self, parser: argparse.ArgumentParser) -> None:
+        super().__init__(parser)
+        self.devices_platform = None
+
+    def add_arguments(self, parser: argparse.ArgumentParser) -> None:
+        parser.add_argument(
+            "platform",
+            type=str,
+            nargs="?",
+            choices=["ios", "android", "web"],
+            help="The target platform to list devices for. "
+            "If not specified, lists all platforms.",
+        )
+        # parser.add_argument(
+        #     "--device-id",
+        #     "-d",
+        #     type=str,
+        #     dest="device_id",
+        #     help="Device ID to run the app on for iOS and Android builds.",
+        # )
+        # parser.add_argument(
+        #     "--show-devices",
+        #     dest="show_devices",
+        #     action="store_true",
+        #     default=False,
+        #     help="Show connected devices for iOS and Android builds.",
+        # )
+        # parser.add_argument(
+        #     "--release",
+        #     dest="release",
+        #     action="store_true",
+        #     default=False,
+        #     help="Build the app in release mode.",
+        # )
+        # parser.add_argument(
+        #     "--route",
+        #     type=str,
+        #     dest="route",
+        #     help="Route to open the app on for web, iOS and Android builds.",
+        # )
+        super().add_arguments(parser)
+
+    def handle(self, options: argparse.Namespace) -> None:
+        super().handle(options)
+        if self.options and "platform" in self.options and self.options.platform:
+            self.devices_platform = self.options.platform
+
+        self.status = console.status(
+            "[bold blue]Initializing environment...",
+            spinner="bouncingBall",
+        )
+        with Live(Group(self.status, self.progress), console=console) as self.live:
+            self.initialize_command()
+            # if self.options.show_devices:
+            #     self.run_flutter_devices()
+            #     self.live.update("", refresh=True)
+            #     return
+            self.run_flutter_devices()
+            self.cleanup(0, message=(""))
+
+    def initialize_command(self):
+        self.package_platform = ""
+        self.config_platform = ""
+
+        super().initialize_command()
+
+    def run_flutter_devices(self):
+        self.update_status("[bold blue]Checking connected devices...")
+        flutter_devices = self.run(
+            [self.flutter_exe, "devices", "--no-version-check", "--suppress-analytics"],
+            cwd=os.getcwd(),
+            capture_output=True,
+        )
+        if flutter_devices.returncode == 0 and flutter_devices.stdout:
+            console.log(flutter_devices.stdout, style=verbose2_style)

sdk/python/packages/flet-cli/src/flet_cli/commands/flutter_base.py

@@ -9,6 +9,7 @@
 from packaging import version
 from rich.console import Console, Group
 from rich.panel import Panel
+from rich.progress import Progress
 from rich.style import Style
 from rich.theme import Theme
 
@@ -49,9 +50,12 @@ def __init__(self, parser: argparse.ArgumentParser) -> None:
         self.dart_exe = None
         self.flutter_exe = None
         self.verbose = False
+        self.package_platform = None
+        self.config_platform = None
         self.skip_flutter_doctor = get_bool_env_var("FLET_CLI_SKIP_FLUTTER_DOCTOR")
         self.no_rich_output = no_rich_output
         self.current_platform = platform.system()
+        self.progress = Progress(transient=True)
 
     def add_arguments(self, parser: argparse.ArgumentParser) -> None:
         parser.add_argument(
@@ -69,10 +73,11 @@ def add_arguments(self, parser: argparse.ArgumentParser) -> None:
 
     def handle(self, options: argparse.Namespace) -> None:
         self.options = options
+        self.no_rich_output = self.no_rich_output or self.options.no_rich_output
+        self.verbose = self.options.verbose
 
-    def initialize_build(self):
+    def initialize_command(self):
         assert self.options
-        assert self.target_platform
         self.emojis = {
             "checkmark": "[green]OK[/]" if self.no_rich_output else "✅",
             "loading": "" if self.no_rich_output else "⏳",
@@ -238,8 +243,7 @@ def run(self, args, cwd, env: Optional[dict] = None, capture_output=True):
 
     def cleanup(self, exit_code: int, message: Optional[str] = None):
         if exit_code == 0:
-            msg = message or f"Success! {self.emojis['success']}"
-            self.live.update(Panel(msg), refresh=True)
+            self.live.update(Panel(message) if message else "", refresh=True)
         else:
             msg = (
                 message