fix(main): print startup banner as single block instead of per-line logger calls

Author: FASTSHIFT

Tools/WebServer/main.py

@@ -346,15 +346,15 @@ def main():
     max_val_len = max(len(v) for _, v in info_lines)
     inner_width = max(len(title) + 8, label_cols + max_val_len + 2)
 
-    logger.info("")
-    logger.info(f"  ╔{'═' * inner_width}╗")
-    logger.info(f"  ║{title:^{inner_width}s}║")
-    logger.info(f"  ╠{'═' * inner_width}╣")
+    banner_lines = []
+    banner_lines.append(f"  ╔{'═' * inner_width}╗")
+    banner_lines.append(f"  ║{title:^{inner_width}s}║")
+    banner_lines.append(f"  ╠{'═' * inner_width}╣")
     val_width = inner_width - label_cols
     for label, value in info_lines:
-        logger.info(f"  ║  {label} {value:<{val_width}s}║")
-    logger.info(f"  ╚{'═' * inner_width}╝")
-    logger.info("")
+        banner_lines.append(f"  ║  {label} {value:<{val_width}s}║")
+    banner_lines.append(f"  ╚{'═' * inner_width}╝")
+    print("\n" + "\n".join(banner_lines) + "\n", flush=True)
 
     if not args.no_browser:
         threading.Timer(1.0, webbrowser.open, args=[local_url]).start()

Tools/WebServer/tests/test_main.py

@@ -535,13 +535,12 @@ def test_startup_banner_logged(
         )
         mock_create.return_value = Mock()
 
-        with patch("main.logger") as mock_logger:
+        with patch("builtins.print") as mock_print:
             main.main()
 
-        calls = [str(c) for c in mock_logger.info.call_args_list]
-        log_output = "\n".join(calls)
-        self.assertIn("FPBInject Web Server Started", log_output)
-        self.assertIn("http://127.0.0.1:5500", log_output)
+        printed = "\n".join(str(c) for c in mock_print.call_args_list)
+        self.assertIn("FPBInject Web Server Started", printed)
+        self.assertIn("http://127.0.0.1:5500", printed)
 
     @patch("main.threading.Timer")
     @patch("main.create_app")
@@ -568,12 +567,11 @@ def test_startup_banner_with_custom_port(
         )
         mock_create.return_value = Mock()
 
-        with patch("main.logger") as mock_logger:
+        with patch("builtins.print") as mock_print:
             main.main()
 
-        calls = [str(c) for c in mock_logger.info.call_args_list]
-        log_output = "\n".join(calls)
-        self.assertIn("http://127.0.0.1:8080", log_output)
+        printed = "\n".join(str(c) for c in mock_print.call_args_list)
+        self.assertIn("http://127.0.0.1:8080", printed)
 
     @patch("main.threading.Timer")
     @patch("main.create_app")
@@ -601,15 +599,14 @@ def test_startup_banner_shows_lan_ip(
         mock_sock = Mock()
         mock_sock.getsockname.return_value = ("192.168.1.100", 0)
 
-        with patch("main.logger") as mock_logger, patch(
+        with patch("builtins.print") as mock_print, patch(
             "main.socket.socket", return_value=mock_sock
         ):
             main.main()
 
-        calls = [str(c) for c in mock_logger.info.call_args_list]
-        log_output = "\n".join(calls)
-        self.assertIn("http://192.168.1.100:5500", log_output)
-        self.assertIn("Network", log_output)
+        printed = "\n".join(str(c) for c in mock_print.call_args_list)
+        self.assertIn("http://192.168.1.100:5500", printed)
+        self.assertIn("Network", printed)
 
     @patch("main.threading.Timer")
     @patch("main.create_app")
@@ -637,14 +634,13 @@ def test_startup_banner_lan_ip_unavailable(
         mock_sock = Mock()
         mock_sock.connect.side_effect = OSError("Network unreachable")
 
-        with patch("main.logger") as mock_logger, patch(
+        with patch("builtins.print") as mock_print, patch(
             "main.socket.socket", return_value=mock_sock
         ):
             main.main()
 
-        calls = [str(c) for c in mock_logger.info.call_args_list]
-        log_output = "\n".join(calls)
-        self.assertIn("unavailable", log_output)
+        printed = "\n".join(str(c) for c in mock_print.call_args_list)
+        self.assertIn("unavailable", printed)
 
     @patch("main.threading.Timer")
     @patch("main.create_app")