From 12b06ce66f89b9606aea368dbe2f764e0ef0b4ad Mon Sep 17 00:00:00 2001 From: airano-ir Date: Tue, 31 Mar 2026 17:52:54 +0200 Subject: [PATCH] fix: black formatting for CI (Python 3.12 compat) Co-Authored-By: Claude Opus 4.6 (1M context) --- plugins/supabase/handlers/auth.py | 2 - plugins/wordpress/wp_cli.py | 69 +++++++++++++++++++++++-------- tests/test_admin_system.py | 1 - 3 files changed, 51 insertions(+), 21 deletions(-) diff --git a/plugins/supabase/handlers/auth.py b/plugins/supabase/handlers/auth.py index d657e5e..a4bb174 100644 --- a/plugins/supabase/handlers/auth.py +++ b/plugins/supabase/handlers/auth.py @@ -483,8 +483,6 @@ async def delete_user_factor(client: SupabaseClient, user_id: str, factor_id: st return json.dumps({"success": False, "error": str(e)}, indent=2, ensure_ascii=False) - - async def search_users( client: SupabaseClient, query: str, page: int = 1, per_page: int = 50 ) -> str: diff --git a/plugins/wordpress/wp_cli.py b/plugins/wordpress/wp_cli.py index 30493b3..5b263f4 100644 --- a/plugins/wordpress/wp_cli.py +++ b/plugins/wordpress/wp_cli.py @@ -77,8 +77,12 @@ class WPCLIManager: try: # First, test if we have Docker socket access test_process = await asyncio.create_subprocess_exec( - "docker", "version", "--format", "{{.Server.Version}}", - stdout=asyncio.subprocess.PIPE, stderr=asyncio.subprocess.PIPE + "docker", + "version", + "--format", + "{{.Server.Version}}", + stdout=asyncio.subprocess.PIPE, + stderr=asyncio.subprocess.PIPE, ) test_stdout, test_stderr = await asyncio.wait_for( @@ -99,10 +103,15 @@ class WPCLIManager: # Now check for our specific container using exact name match process = await asyncio.create_subprocess_exec( - "docker", "ps", "--all", - "--filter", f"name=^{self.container_name}$", - "--format", "{{.Names}}|{{.Status}}", - stdout=asyncio.subprocess.PIPE, stderr=asyncio.subprocess.PIPE + "docker", + "ps", + "--all", + "--filter", + f"name=^{self.container_name}$", + "--format", + "{{.Names}}|{{.Status}}", + stdout=asyncio.subprocess.PIPE, + stderr=asyncio.subprocess.PIPE, ) stdout, stderr = await asyncio.wait_for(process.communicate(), timeout=5.0) @@ -117,8 +126,13 @@ class WPCLIManager: if not output: # Container not found - get list of available containers for helpful error list_process = await asyncio.create_subprocess_exec( - "docker", "ps", "--all", "--format", "{{.Names}}", - stdout=asyncio.subprocess.PIPE, stderr=asyncio.subprocess.PIPE + "docker", + "ps", + "--all", + "--format", + "{{.Names}}", + stdout=asyncio.subprocess.PIPE, + stderr=asyncio.subprocess.PIPE, ) list_stdout, _ = await list_process.communicate() available = list_stdout.decode().strip().split("\n") if list_stdout else [] @@ -170,9 +184,14 @@ class WPCLIManager: try: # Try to run wp --version process = await asyncio.create_subprocess_exec( - "docker", "exec", self.container_name, - "wp", "--version", "--allow-root", - stdout=asyncio.subprocess.PIPE, stderr=asyncio.subprocess.PIPE + "docker", + "exec", + self.container_name, + "wp", + "--version", + "--allow-root", + stdout=asyncio.subprocess.PIPE, + stderr=asyncio.subprocess.PIPE, ) stdout, stderr = await asyncio.wait_for(process.communicate(), timeout=5.0) @@ -291,7 +310,9 @@ class WPCLIManager: ) # 4. Build docker exec command (split command into args to prevent shell injection) - cmd_parts = ["docker", "exec", self.container_name, "wp"] + command.split() + ["--allow-root"] + cmd_parts = ( + ["docker", "exec", self.container_name, "wp"] + command.split() + ["--allow-root"] + ) self.logger.info(f"Executing: {' '.join(cmd_parts)}") @@ -582,17 +603,29 @@ class WPCLIManager: try: # Try GNU stat first, fall back to BSD stat process = await asyncio.create_subprocess_exec( - "docker", "exec", self.container_name, - "stat", "-c", "%s", export_path, - stdout=asyncio.subprocess.PIPE, stderr=asyncio.subprocess.PIPE + "docker", + "exec", + self.container_name, + "stat", + "-c", + "%s", + export_path, + stdout=asyncio.subprocess.PIPE, + stderr=asyncio.subprocess.PIPE, ) stdout, _ = await asyncio.wait_for(process.communicate(), timeout=5.0) if process.returncode != 0: # BSD stat fallback process = await asyncio.create_subprocess_exec( - "docker", "exec", self.container_name, - "stat", "-f", "%z", export_path, - stdout=asyncio.subprocess.PIPE, stderr=asyncio.subprocess.PIPE + "docker", + "exec", + self.container_name, + "stat", + "-f", + "%z", + export_path, + stdout=asyncio.subprocess.PIPE, + stderr=asyncio.subprocess.PIPE, ) stdout, _ = await asyncio.wait_for(process.communicate(), timeout=5.0) diff --git a/tests/test_admin_system.py b/tests/test_admin_system.py index 8f8b6e3..94ad5f7 100644 --- a/tests/test_admin_system.py +++ b/tests/test_admin_system.py @@ -1,6 +1,5 @@ """Tests for Admin System Unification (F.4).""" - from core.dashboard.auth import ( DashboardSession, get_session_display_info,