Refactor Flutter command execution in CLI commands

Replaces 'flutter_build' with 'run_flutter' in build and debug commands, consolidating command execution logic. Updates method names for clarity and moves status updates and logging to command-specific implementations. Improves code maintainability and consistency across build and debug workflows.

Author: FeodorFitsner

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

@@ -57,7 +57,7 @@ def handle(self, options: argparse.Namespace) -> None:
                 self.update_flutter_dependencies()
             self.customize_icons()
             self.customize_splash_images()
-            self.flutter_build()
+            self.run_flutter()
             self.copy_build_output()
 
             self.cleanup(
@@ -77,7 +77,7 @@ def handle(self, options: argparse.Namespace) -> None:
                 ),
             )
 
-    def add_flutter_build_args(self, args: list[str]):
+    def add_flutter_command_args(self, args: list[str]):
         assert self.options
         assert self.build_dir
         assert self.get_pyproject
@@ -122,3 +122,19 @@ def add_flutter_build_args(self, args: list[str]):
             or []
         ):
             args.append(arg)
+
+    def run_flutter(self):
+        assert self.platforms
+        assert self.target_platform
+
+        self.update_status(
+            f"[bold blue]Building [cyan]"
+            f"{self.platforms[self.target_platform]['status_text']}[/cyan]..."
+        )
+
+        self._run_flutter_command()
+
+        console.log(
+            f"Built [cyan]{self.platforms[self.target_platform]['status_text']}"
+            f"[/cyan] {self.emojis['checkmark']}",
+        )

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

@@ -1,4 +1,5 @@
 import argparse
+import contextlib
 import os
 import platform
 
@@ -98,24 +99,8 @@ def handle(self, options: argparse.Namespace) -> None:
                 self.update_flutter_dependencies()
             self.customize_icons()
             self.customize_splash_images()
-            self.flutter_build()
-
-            self.cleanup(
-                0,
-                message=(
-                    f"Successfully built your [cyan]"
-                    f"{self.platforms[self.target_platform]['status_text']}"
-                    f"[/cyan]! {self.emojis['success']} "
-                    f"Find it in [cyan]{self.rel_out_dir}[/cyan] directory. "
-                    f"{self.emojis['directory']}"
-                    + (
-                        "\nRun [cyan]flet serve[/cyan] command to start a web server "
-                        "with your app. "
-                        if self.target_platform == "web"
-                        else ""
-                    )
-                ),
-            )
+            self.run_flutter()
+            self.cleanup(0, message=("Debug session ended."))
 
     def check_device_id(self):
         if self.device_id is None and self.debug_platform in [
@@ -130,10 +115,20 @@ def check_device_id(self):
                 "Use --show-devices option to list connected devices.",
             )
 
-    def add_flutter_build_args(self, args: list[str]):
+    def add_flutter_command_args(self, args: list[str]):
         assert self.device_id
         args.extend(["run", "-d", self.device_id])
 
+    def run_flutter(self):
+        assert self.platforms
+        assert self.target_platform
+        self.update_status(
+            f"[bold blue]Running the app on [cyan]{self.debug_platform}[/cyan]..."
+        )
+
+        with contextlib.suppress(KeyboardInterrupt):
+            self._run_flutter_command()
+
     def run_flutter_devices(self):
         self.update_status("[bold blue]Checking connected devices...")
         flutter_devices = self.run(

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

@@ -1939,24 +1939,22 @@ def get_bool_setting(self, cli_option, pyproj_setting, default_value):
             )
         )
 
-    def add_flutter_build_args(self, args: list[str]):
+    def add_flutter_command_args(self, args: list[str]):
         pass
 
-    def flutter_build(self):
+    def run_flutter(self):
+        self._run_flutter_command()
+
+    def _run_flutter_command(self):
         assert self.options
         assert self.build_dir
         assert self.get_pyproject
         assert self.template_data
         assert self.target_platform
 
-        self.update_status(
-            f"[bold blue]Building [cyan]"
-            f"{self.platforms[self.target_platform]['status_text']}[/cyan]..."
-        )
-
         # flutter build
         build_args = [self.flutter_exe]
-        self.add_flutter_build_args(build_args)
+        self.add_flutter_command_args(build_args)
         build_args.extend(
             [
                 "--no-version-check",
@@ -2027,10 +2025,6 @@ def flutter_build(self):
             if build_result.stderr:
                 console.log(build_result.stderr, style=error_style)
             self.cleanup(build_result.returncode if build_result.returncode else 1)
-        console.log(
-            f"Built [cyan]{self.platforms[self.target_platform]['status_text']}"
-            f"[/cyan] {self.emojis['checkmark']}",
-        )
 
     def copy_build_output(self):
         assert self.template_data