Update CLAUDE.md

Signed-off-by: Sergio Herrera <627709+seherv@users.noreply.github.com>

Author: seherv

CLAUDE.md

@@ -2,11 +2,13 @@
 
 Use pathlib instead of os.path.
 Use httpx instead of urllib.
+subprocess(`shell=True`) is used only when it makes the code more readable. Use either shlex or args lists.
+subprocess calls should have a reasonable timeout.
 Use modern Python (3.10+) features.
 Make all code strongly typed.
 Keep conditional nesting to a minimum, and use guard clauses when possible.
-Aim for medium "visual complexity": use intermediate variables to store results of nested/complex function calls, but don't create a new variable for everything. 
-Avoid comments unless there is an unusual gotcha, a complex algorithm or anything an experienced code reviewer needs to be aware of. Focus on making better Google-style docstrings instead.
+Aim for medium "visual complexity": use intermediate variables to store results of nested/complex function calls, but don't create a new variable for everything.
+Avoid comments unless there is a gotcha, a complex algorithm or anything an experienced code reviewer needs to be aware of. Focus on making better Google-style docstrings instead.
 
 The user is not always right. Be skeptical and do not blindly comply if something doesn't make sense.
 Code should be production-ready.

tests/integration/conftest.py

@@ -105,6 +105,7 @@ def cleanup(self) -> None:
         for client in self._clients:
             client.close()
         self._clients.clear()
+
         for proc in self._processes:
             if proc.poll() is None:
                 proc.terminate()
@@ -114,6 +115,7 @@ def cleanup(self) -> None:
                     proc.kill()
                     proc.wait()
         self._processes.clear()
+
         for log_path in self._log_files:
             log_path.unlink(missing_ok=True)
         self._log_files.clear()