chore(Makefile): update all target to include formatcheck for better code quality

style(shell_executor.py): format code for consistency and readability
style(test_shell_executor.py): format test cases for improved readability and consistency

Author: tumf

Makefile

@@ -25,4 +25,4 @@ coverage:
 # Run all checks required before pushing
 check:  lint typecheck test
 fix: check format
-all: check test coverage
+all: formatcheck test coverage

src/mcp_shell_server/shell_executor.py

@@ -269,7 +269,7 @@ def _validate_pipeline(self, commands: List[str]) -> None:
                     raise ValueError("Empty command before pipe operator")
                 self._validate_command(current_cmd)
                 current_cmd = []
-            elif token in [";" , "&&", "||"]:
+            elif token in [";", "&&", "||"]:
                 raise ValueError(f"Unexpected shell operator in pipeline: {token}")
             else:
                 current_cmd.append(token)

tests/test_shell_executor.py

@@ -429,16 +429,19 @@ def test_create_shell_command(executor):
     """Test shell command creation with various input combinations"""
     # Test basic command
     assert executor._create_shell_command(["echo", "hello"]) == "echo hello"
-    
+
     # Test command with space-only argument
     assert executor._create_shell_command(["echo", " "]) == "echo ' '"
-    
+
     # Test command with wildcards
     assert executor._create_shell_command(["ls", "*.txt"]) == "ls '*.txt'"
-    
+
     # Test command with special characters
-    assert executor._create_shell_command(["echo", "hello;", "world"]) == "echo 'hello;' world"
-    
+    assert (
+        executor._create_shell_command(["echo", "hello;", "world"])
+        == "echo 'hello;' world"
+    )
+
     # Test empty command
     assert executor._create_shell_command([]) == ""
 
@@ -447,31 +450,31 @@ def test_preprocess_command(executor):
     """Test command preprocessing for pipeline handling"""
     # Test basic command
     assert executor._preprocess_command(["ls"]) == ["ls"]
-    
+
     # Test command with separate pipe
     assert executor._preprocess_command(["ls", "|", "grep", "test"]) == [
         "ls",
         "|",
         "grep",
         "test",
     ]
-    
+
     # Test command with attached pipe
     assert executor._preprocess_command(["ls|", "grep", "test"]) == [
         "ls",
         "|",
         "grep",
         "test",
     ]
-    
+
     # Test command with special operators
     assert executor._preprocess_command(["echo", "hello", "&&", "ls"]) == [
         "echo",
         "hello",
         "&&",
         "ls",
     ]
-    
+
     # Test empty command
     assert executor._preprocess_command([]) == []