fix: hold update overlay until post-update status refresh completes; demote periodic-poll logs to DEBUG

Author: gmmcosta15

BlocksScreen/lib/panels/mainWindow.py

@@ -299,18 +299,8 @@ def show_LoadScreen(self, show: bool = True, msg: str = ""):
         if _sender is self.update_page:
             self._update_in_progress = show
         if not show and self._post_update_reconnect:
-            _logger.warning(
-                "show_LoadScreen(False) suppressed: post_update_reconnect=True update_in_progress=%s sender=%s",
-                self._update_in_progress,
-                _sender,
-            )
             return
         elif not show and self._update_in_progress:
-            _logger.warning(
-                "show_LoadScreen(False) suppressed: update_in_progress=True post_update_reconnect=%s sender=%s",
-                self._post_update_reconnect,
-                _sender,
-            )
             return
 
         if _sender == self.filamentPanel:
@@ -328,21 +318,8 @@ def show_LoadScreen(self, show: bool = True, msg: str = ""):
 
         self.loadwidget.set_status_message(msg)
         if show:
-            _logger.info(
-                "show_LoadScreen: showing msg=%r update_in_progress=%s post_update_reconnect=%s sender=%s",
-                msg,
-                self._update_in_progress,
-                self._post_update_reconnect,
-                _sender,
-            )
             self.loadscreen.show()
         else:
-            _logger.info(
-                "show_LoadScreen: hiding update_in_progress=%s post_update_reconnect=%s sender=%s",
-                self._update_in_progress,
-                self._post_update_reconnect,
-                _sender,
-            )
             self.loadscreen.hide()
 
     @QtCore.pyqtSlot(bool, name="show-update-page")
@@ -410,12 +387,10 @@ def on_updater_unavailable(self) -> None:
     @QtCore.pyqtSlot(name="on-post-update-reconnect")
     def _on_post_update_reconnect(self) -> None:
         """Called when an update finishes: hold the loading screen and retry Moonraker until it's up."""
-        _logger.info(
-            "_on_post_update_reconnect: ws.connected=%s loadscreen_visible=%s update_in_progress=%s post_update_reconnect=%s",
+        _logger.debug(
+            "_on_post_update_reconnect: ws.connected=%s update_in_progress=%s",
             self.ws.connected,
-            self.loadscreen.isVisible(),
             self._update_in_progress,
-            self._post_update_reconnect,
         )
         if self.ws.connected:
             # Moonraker never restarted — overlay will be dismissed by handle_status_ready
@@ -435,7 +410,7 @@ def _on_reconnect_timer(self) -> None:
             self._reconnect_timer.stop()
             return
         if self.ws.connected:
-            _logger.info("post-update: Moonraker already connected, hiding loadscreen")
+            _logger.debug("post-update: Moonraker reconnected, hiding loadscreen")
             self._post_update_reconnect = False
             self._update_in_progress = False
             self._reconnect_retries = 0

BlocksScreen/lib/panels/widgets/updatePage.py

@@ -225,7 +225,7 @@ def _toggle_details(self) -> None:
 
     def handle_status_ready(self, json_str: str) -> None:
         """Update component statuses from a JSON payload and refresh the list."""
-        _log.info("handle_status_ready: busy=%s", self._busy)
+        _log.debug("handle_status_ready: busy=%s", self._busy)
         try:
             data: dict[str, dict] = json.loads(json_str)
             self._statuses = {
@@ -244,7 +244,7 @@ def handle_status_ready(self, json_str: str) -> None:
                 self.call_load_panel.emit(False, "")
                 self._post_update_status_pending = False
         else:
-            _log.warning("status_ready: skipping loadscreen dismiss (busy=True)")
+            _log.debug("status_ready: skipping loadscreen dismiss (busy=True)")
         self.build_cards()
         self.update_available.emit(_update_avail)
 

BlocksScreen/lib/updater_worker.py

@@ -187,7 +187,7 @@ def trigger_status(self) -> None:
         """Request a status check; result arrives via status_ready signal."""
         if not self._require_proxy():
             return
-        _log.info("trigger_status called")
+        _log.debug("trigger_status called")
         self._submit(self._call_status())
 
     def trigger_recover(self, name: str, hard: bool) -> None:  # noqa: FBT001
@@ -265,7 +265,7 @@ async def _call_cancel(self) -> None:
 
     async def _listen_status_ready(self) -> None:
         async for json_str in self._proxy.status_ready:
-            _log.info("status_ready received (%d bytes)", len(json_str))
+            _log.debug("status_ready received (%d bytes)", len(json_str))
             self.status_ready.emit(json_str)
 
     async def _listen_step_complete(self) -> None: