fix(python): bump requires-python, portable sed, and signal forwarding (#1005)

* fix(python): bump requires-python, portable sed, and signal forwarding

- Bump requires-python to >=3.8 and drop importlib_metadata fallback
  since Python 3.6/3.7 are long EOL
- Replace GNU-only sed -i with redirect+mv pattern for macOS compat
- Forward SIGTERM/SIGHUP to child process using subprocess.Popen to
  prevent orphaned processes on container shutdown

Fixes #1004

Signed-off-by: Marc Nuri <marc@marcnuri.com>

* fix(python): guard SIGHUP for Windows and handle send_signal race

- Gate SIGHUP registration behind hasattr check since the constant
  is Unix-only and would crash the wrapper on Windows
- Catch OSError in send_signal calls to handle the race where the
  child exits between signal arrival and forwarding

Signed-off-by: Marc Nuri <marc@marcnuri.com>

---------

Signed-off-by: Marc Nuri <marc@marcnuri.com>

Author: manusa

build/python.mk

@@ -5,6 +5,6 @@ CLEAN_TARGETS += ./python/dist ./python/*.egg-info
 .PHONY: python-publish
 python-publish: ## Publish the python packages
 	cd ./python && \
-	sed -i "s/version = \".*\"/version = \"$(GIT_TAG_VERSION)\"/" pyproject.toml && \
+	sed "s/version = \".*\"/version = \"$(GIT_TAG_VERSION)\"/" pyproject.toml > pyproject.toml.tmp && mv pyproject.toml.tmp pyproject.toml && \
 	uv build && \
 	uv publish

python/kubernetes_mcp_server/kubernetes_mcp_server.py

@@ -1,17 +1,14 @@
 import os
 import platform
+import signal
 import subprocess
 import sys
+from importlib.metadata import version
 from pathlib import Path
 import shutil
 import tempfile
 import urllib.request
 
-if sys.version_info >= (3, 8):
-    from importlib.metadata import version
-else:
-    from importlib_metadata import version
-
 __version__ = version("kubernetes-mcp-server")
 
 def get_platform_binary():
@@ -82,8 +79,26 @@ def execute(args=None):
         cmd = [str(binary_path)] + args
 
         # Execute the binary with the provided arguments
-        process = subprocess.run(cmd)
-        return process.returncode
+        process = subprocess.Popen(cmd)
+
+        def handle_signal(signum, frame):
+            try:
+                process.send_signal(signum)
+            except OSError:
+                pass
+
+        signal.signal(signal.SIGTERM, handle_signal)
+        if hasattr(signal, "SIGHUP"):
+            signal.signal(signal.SIGHUP, handle_signal)
+
+        try:
+            return process.wait()
+        except KeyboardInterrupt:
+            try:
+                process.send_signal(signal.SIGINT)
+            except OSError:
+                pass
+            return process.wait()
     except Exception as e:
         print(f"Error executing kubernetes-mcp-server: {e}", file=sys.stderr)
         return 1

python/pyproject.toml

@@ -7,7 +7,7 @@ name = "kubernetes-mcp-server"
 version = "0.0.0"
 description = "Kubernetes MCP Server (Model Context Protocol) with special support for OpenShift"
 readme = {file="README.md", content-type="text/markdown"}
-requires-python = ">=3.6"
+requires-python = ">=3.8"
 license = "Apache-2.0"
 authors = [
     { name = "Marc Nuri", email = "marc@marcnuri.com" }