style: apply ruff formatting

Author: bx33661

src/wireshark_mcp/installer.py

@@ -586,7 +586,11 @@ def _print_rows(rows: list[dict[str, str]]) -> None:
 
 def _summarize_status_rows(rows: list[dict[str, str]], statuses: tuple[str, ...]) -> dict[str, int]:
     """Summarize row counts for the requested statuses."""
-    return {status: sum(row["status"] == status for row in rows) for status in statuses if any(row["status"] == status for row in rows)}
+    return {
+        status: sum(row["status"] == status for row in rows)
+        for status in statuses
+        if any(row["status"] == status for row in rows)
+    }
 
 
 def _collect_client_rows(selected_clients: list[str] | None = None) -> list[dict[str, str]]:
@@ -794,7 +798,12 @@ def install_mcp_servers(*, uninstall: bool = False, selected_clients: list[str]
 
         if not os.path.exists(config_dir):
             result_rows.append(
-                {"marker": "[SKIP]", "name": name, "detail": f"{action_word} skipped (config dir not found)", "path": config_path}
+                {
+                    "marker": "[SKIP]",
+                    "name": name,
+                    "detail": f"{action_word} skipped (config dir not found)",
+                    "path": config_path,
+                }
             )
             skipped += 1
             continue
@@ -808,7 +817,9 @@ def install_mcp_servers(*, uninstall: bool = False, selected_clients: list[str]
                 continue
 
             done_word = "uninstalled" if uninstall else "installed"
-            result_rows.append({"marker": "[OK]", "name": name, "detail": f"{done_word} (restart required)", "path": config_path})
+            result_rows.append(
+                {"marker": "[OK]", "name": name, "detail": f"{done_word} (restart required)", "path": config_path}
+            )
             installed += 1
             continue
 
@@ -827,7 +838,9 @@ def install_mcp_servers(*, uninstall: bool = False, selected_clients: list[str]
         _write_json_config(config_path, config)
 
         done_word = "uninstalled" if uninstall else "installed"
-        result_rows.append({"marker": "[OK]", "name": name, "detail": f"{done_word} (restart required)", "path": config_path})
+        result_rows.append(
+            {"marker": "[OK]", "name": name, "detail": f"{done_word} (restart required)", "path": config_path}
+        )
         installed += 1
 
     _print_rows(result_rows)

src/wireshark_mcp/tools/registry.py

@@ -159,7 +159,9 @@ def recommended_tools_for_protocols(self, detected_protocols: set[str]) -> list[
 
         recommended = sorted(tool_name for tool_name in tools_to_recommend if tool_name in self._contextual_catalog)
 
-        missing_tools = sorted(tool_name for tool_name in tools_to_recommend if tool_name not in self._contextual_catalog)
+        missing_tools = sorted(
+            tool_name for tool_name in tools_to_recommend if tool_name not in self._contextual_catalog
+        )
         for tool_name in missing_tools:
             logger.warning("Tool %s is in PROTOCOL_TOOL_MAP but not in catalog", tool_name)
 
@@ -271,7 +273,9 @@ async def wireshark_open_file(pcap_file: str) -> str:
                 output_parts.append(f"  • {tool_name}: {doc}")
         else:
             output_parts.append("\n--- No protocol-specific recommendations ---")
-            output_parts.append("The core tools should be enough to start, and all contextual tools remain available if needed.")
+            output_parts.append(
+                "The core tools should be enough to start, and all contextual tools remain available if needed."
+            )
 
         output_parts.append(
             "\n💡 Tip: Start broad with wireshark_quick_analysis or wireshark_get_packet_list, then narrow using the recommended tools above."

tests/test_registry.py

@@ -188,7 +188,9 @@ def test_open_file_recommends_tools_without_mutating_tool_surface(self, mock_cli
         from mcp.server.fastmcp import FastMCP
 
         async def fake_get_protocol_stats(_pcap_file: str) -> str:
-            return success_response("eth  frames:10 bytes:100\n  ip  frames:10 bytes:90\n    tcp  frames:5 bytes:50\n      http  frames:5 bytes:50\n")
+            return success_response(
+                "eth  frames:10 bytes:100\n  ip  frames:10 bytes:90\n    tcp  frames:5 bytes:50\n      http  frames:5 bytes:50\n"
+            )
 
         async def fake_get_file_info(_pcap_file: str) -> str:
             return success_response("file name: test.pcap\n")

tests/test_security.py

@@ -29,7 +29,7 @@ async def extract_fields(
 
         if fields == ["http.request.full_uri", "http.host", "http.request.uri"] and display_filter == "http.request":
             return success_response(
-                'http.request.full_uri\thttp.host\thttp.request.uri\n'
+                "http.request.full_uri\thttp.host\thttp.request.uri\n"
                 '"http://bad.example/evil"\t"bad.example"\t"/evil"\n'
                 '""\t"only-host.example"\t"/download"\n'
             )