fixup: ruff format + address Gemini review comments

- ruff format all changed files (CI enforces ruff format --check)
- _process_menu_flow_result: str | None type hint for intro_flow_id
- _process_menu_flow_result: narrow except to (HomeAssistantAPIError, TimeoutError)
- fetch_helper_flow_info: remove `and intro_flow_id` guard from MENU branch
  (restores menu_options surfacing when intro_flow_id is absent)
- _try_raw_cdn: narrow except to httpx.RequestError
- test_tools_updates: use httpx.ConnectError in exception-continue test

Author: Sergey

src/ha_mcp/tools/best_practice_checker.py

@@ -279,10 +279,7 @@ def _check_template_string(
     # reframes #695 from "enumerate bad shapes" to "surface every template
     # in a logic position". Specific detectors above keep their tailored
     # messages.
-    if (
-        len(warnings) == initial_count
-        and _RE_ANY_TEMPLATE.search(template)
-    ):
+    if len(warnings) == initial_count and _RE_ANY_TEMPLATE.search(template):
         warnings.append(
             f"Template detected in {position} — if this maps to a native option "
             "(`numeric_state`, `state`, `time`, `sun`, `zone`, `device`), use that "
@@ -304,9 +301,7 @@ def _check_choose_actions(
             _check_condition_templates(
                 option.get("conditions", []), warnings, skill_prefix
             )
-            _check_action_tree(
-                option.get("sequence", []), warnings, skill_prefix
-            )
+            _check_action_tree(option.get("sequence", []), warnings, skill_prefix)
 
 
 def _check_repeat_actions(
@@ -445,15 +440,19 @@ def _check_target_dict(
                     f"hardcode the literal value instead. The self-reference is always "
                     f"resolvable at write time, so the template adds runtime cost without "
                     f"any flexibility."
-                    + _ref(skill_prefix, "template-guidelines.md#when-to-avoid-templates")
+                    + _ref(
+                        skill_prefix, "template-guidelines.md#when-to-avoid-templates"
+                    )
                 )
             else:
                 warnings.append(
                     f"Action `target.{field}` uses a template — prefer a hardcoded literal, "
                     f"or use a `choose` action with native conditions to dispatch to different "
                     f"hardcoded targets. Templates in target fields fail silently if they "
                     f"resolve to a non-existent entity."
-                    + _ref(skill_prefix, "template-guidelines.md#when-to-avoid-templates")
+                    + _ref(
+                        skill_prefix, "template-guidelines.md#when-to-avoid-templates"
+                    )
                 )
 
 

src/ha_mcp/tools/tools_bug_report.py

@@ -423,17 +423,21 @@ def _build_formatted_report(
         for key, value in config_toggles.items():
             report_lines.append(f"  {key}: {value}")
     if startup_logs:
-        report_lines.extend([
-            "",
-            f"=== Startup Logs ({len(startup_logs)} entries) ===",
-            startup_log_summary,
-        ])
+        report_lines.extend(
+            [
+                "",
+                f"=== Startup Logs ({len(startup_logs)} entries) ===",
+                startup_log_summary,
+            ]
+        )
     if recent_logs:
-        report_lines.extend([
-            "",
-            f"=== Recent Tool Calls ({len(recent_logs)} entries) ===",
-            log_summary,
-        ])
+        report_lines.extend(
+            [
+                "",
+                f"=== Recent Tool Calls ({len(recent_logs)} entries) ===",
+                log_summary,
+            ]
+        )
     if addon_logs:
         report_lines.extend(["", "=== Add-on Container Logs ===", addon_logs])
     return "\n".join(report_lines)