devs-webhook 1.0.0__tar.gz → 1.0.2__tar.gz

{devs_webhook-1.0.0 → devs_webhook-1.0.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devs-webhook
-Version: 1.0.0
+Version: 1.0.2
 Summary: GitHub webhook handler for automated devcontainer operations with Claude Code
 Author: Dan Lester
 License-Expression: MIT

{devs_webhook-1.0.0 → devs_webhook-1.0.2}/devs_webhook/cli/worker.py

@@ -29,7 +29,8 @@ logger = structlog.get_logger()
 @click.option('--repo-path', required=True, help='Path to repository on host')
 @click.option('--task-type', default='claude', help='Task type: claude or tests (default: claude)')
 @click.option('--timeout', default=3600, help='Task timeout in seconds (default: 3600)')
-def worker(task_id: str, dev_name: str, repo_name: str, repo_path: str, task_type: str, timeout: int):
+@click.option('--worker-logs-dir', default=None, help='Directory for worker log files (enables file logging)')
+def worker(task_id: str, dev_name: str, repo_name: str, repo_path: str, task_type: str, timeout: int, worker_logs_dir: Optional[str]):
     """Process a single webhook task in an isolated subprocess.
     
     This command runs the complete task processing logic in a separate process to provide
@@ -47,15 +48,30 @@ def worker(task_id: str, dev_name: str, repo_name: str, repo_path: str, task_typ
     """
     # Set environment variable to redirect console output to stderr
     os.environ['DEVS_WEBHOOK_MODE'] = '1'
-    
-    # Configure structured logging for subprocess to output to stderr
+
+    # Configure structured logging for subprocess
     import logging
+
+    # Build list of handlers - always include stderr
+    handlers = [logging.StreamHandler(sys.stderr)]
+
+    # Add file handler if worker_logs_dir is provided
+    log_file_path = None
+    if worker_logs_dir:
+        log_dir = Path(worker_logs_dir)
+        log_dir.mkdir(parents=True, exist_ok=True)
+        log_file_path = log_dir / f"{task_id}.log"
+        file_handler = logging.FileHandler(log_file_path, encoding='utf-8')
+        file_handler.setLevel(logging.INFO)
+        file_handler.setFormatter(logging.Formatter('%(message)s'))
+        handlers.append(file_handler)
+
     logging.basicConfig(
-        stream=sys.stderr,
+        handlers=handlers,
         level=logging.INFO,
         format='%(message)s'
     )
-    
+
     structlog.configure(
         processors=[
             structlog.stdlib.filter_by_level,
@@ -79,7 +95,8 @@ def worker(task_id: str, dev_name: str, repo_name: str, repo_path: str, task_typ
                 repo_name=repo_name,
                 repo_path=repo_path,
                 timeout=timeout,
-                pid=os.getpid())
+                pid=os.getpid(),
+                log_file=str(log_file_path) if log_file_path else None)
     
     try:
         # Read payload from stdin

{devs_webhook-1.0.0 → devs_webhook-1.0.2}/devs_webhook/config.py

@@ -109,6 +109,16 @@ class WebhookConfig(BaseSettings, BaseConfig):
         description="Enable writing container output to log files"
     )
 
+    # Worker process logging (captures full worker subprocess logs)
+    worker_logs_dir: Path = Field(
+        default_factory=lambda: Path("/var/log/devs-webhook/workers"),
+        description="Directory for worker subprocess logs"
+    )
+    worker_logs_enabled: bool = Field(
+        default=True,
+        description="Enable writing worker subprocess logs to files (recommended)"
+    )
+
     # Task source configuration
     task_source: str = Field(
         default="webhook",
@@ -212,7 +222,10 @@ class WebhookConfig(BaseSettings, BaseConfig):
         # Container logs directory (if enabled)
         if self.container_logs_enabled:
             self.container_logs_dir.mkdir(parents=True, exist_ok=True)
-    
+        # Worker logs directory (if enabled)
+        if self.worker_logs_enabled:
+            self.worker_logs_dir.mkdir(parents=True, exist_ok=True)
+
     def validate_required_settings(self) -> None:
         """Validate that required settings are present."""
         missing = []

{devs_webhook-1.0.0 → devs_webhook-1.0.2}/devs_webhook/core/container_pool.py

@@ -575,12 +575,22 @@ class ContainerPool:
                 "--task-type", queued_task.task_type,
                 "--timeout", str(3600)  # 60 minute timeout
             ]
-            
+
+            # Add worker logs directory if enabled
+            if self.config.worker_logs_enabled:
+                cmd.extend(["--worker-logs-dir", str(self.config.worker_logs_dir)])
+
+            # Determine log file path for logging
+            worker_log_file = None
+            if self.config.worker_logs_enabled:
+                worker_log_file = str(self.config.worker_logs_dir / f"{queued_task.task_id}.log")
+
             logger.info("Launching worker subprocess",
                        task_id=queued_task.task_id,
                        container=dev_name,
                        command_length=len(' '.join(cmd)),
-                       stdin_payload_size=len(stdin_json))
+                       stdin_payload_size=len(stdin_json),
+                       worker_log_file=worker_log_file)
 
             # Launch subprocess with timeout
             # Set environment to suppress console output

{devs_webhook-1.0.0 → devs_webhook-1.0.2}/devs_webhook/core/test_dispatcher.py

@@ -479,8 +479,8 @@ class TestDispatcher(BaseDispatcher):
         
         try:
             # Execute command in the container
-            # Use same pattern as devs shell: bash execs into interactive zsh
-            shell_cmd = f"cd {container_workspace_dir} && exec /bin/zsh -ic '{command}'"
+            # Use login shell (-l) for proper PATH/env, but not interactive (-i) to avoid ZLE errors
+            shell_cmd = f"cd {container_workspace_dir} && exec /bin/zsh -lc '{command}'"
             cmd = [
                 'docker', 'exec', '-i',  # -i for stdin, no TTY
                 container_name,

{devs_webhook-1.0.0 → devs_webhook-1.0.2}/devs_webhook.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devs-webhook
-Version: 1.0.0
+Version: 1.0.2
 Summary: GitHub webhook handler for automated devcontainer operations with Claude Code
 Author: Dan Lester
 License-Expression: MIT

{devs_webhook-1.0.0 → devs_webhook-1.0.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "devs-webhook"
-version = "1.0.0"
+version = "1.0.2"
 description = "GitHub webhook handler for automated devcontainer operations with Claude Code"
 readme = "README.md"
 requires-python = ">=3.8"