Merge pull request #206 from Frostday/main

Fix for issue #134

Author: gltanaka

.pdd/meta/admin_get_users_python_run.json

@@ -1,5 +1,5 @@
 {
-  "timestamp": "2025-12-15T23:18:07.519096+00:00",
+  "timestamp": "2025-12-20T02:05:12.779846+00:00",
   "exit_code": 0,
   "tests_passed": 1,
   "tests_failed": 0,

.pdd/meta/my_module_python_run.json

@@ -1,5 +1,5 @@
 {
-  "timestamp": "2025-12-15T23:18:07.516976+00:00",
+  "timestamp": "2025-12-20T02:05:12.777629+00:00",
   "exit_code": 0,
   "tests_passed": 1,
   "tests_failed": 0,

.pdd/meta/simple_math_python.json

@@ -1,6 +1,6 @@
 {
-  "pdd_version": "0.0.84",
-  "timestamp": "2025-12-15T23:18:14.786118+00:00",
+  "pdd_version": "0.0.87",
+  "timestamp": "2025-12-20T02:05:18.076987+00:00",
   "command": "test",
   "prompt_hash": "d4357fbb6b142ab7feb3a75a575e46dae19c83cf493765bda3922bd98a3a7ea5",
   "code_hash": null,

.pdd/meta/simple_math_python_run.json

@@ -1,8 +1,8 @@
 {
-  "timestamp": "2025-12-15T23:18:14.409837+00:00",
-  "exit_code": 0,
-  "tests_passed": 2,
-  "tests_failed": 0,
-  "coverage": 100.0,
+  "timestamp": "2025-12-20T02:05:17.514948+00:00",
+  "exit_code": 2,
+  "tests_passed": 0,
+  "tests_failed": 1,
+  "coverage": 0.0,
   "test_hash": "cb09a5c55340648bf4da079f83efa25b815ec557b01971793c35336b0b925be4"
 }

.pdd/meta/test_broken_fixture_python_run.json

@@ -1,5 +1,5 @@
 {
-  "timestamp": "2025-12-15T23:18:06.698878+00:00",
+  "timestamp": "2025-12-20T02:05:11.783112+00:00",
   "exit_code": 1,
   "tests_passed": 1,
   "tests_failed": 1,

.pdd/meta/test_mixed_python_run.json

@@ -1,5 +1,5 @@
 {
-  "timestamp": "2025-12-15T23:18:07.108222+00:00",
+  "timestamp": "2025-12-20T02:05:12.273198+00:00",
   "exit_code": 1,
   "tests_passed": 1,
   "tests_failed": 2,

pdd/setup_tool.py

@@ -11,6 +11,7 @@
 import requests
 import csv
 import importlib.resources
+import shlex
 from pathlib import Path
 from typing import Dict, Optional, Tuple, List
 
@@ -309,24 +310,23 @@ def get_shell_init_file(shell: str) -> str:
     return str(shell_files.get(shell, home / '.bashrc'))
 
 def create_api_env_script(keys: Dict[str, str], shell: str) -> str:
-    """Create shell-appropriate environment script"""
+    """Create shell-appropriate environment script with proper escaping"""
     valid_keys = {k: v for k, v in keys.items() if v}
+    lines = []
 
-    if shell == 'fish':
-        lines = []
-        for key, value in valid_keys.items():
-            lines.append(f'set -gx {key} "{value}"')
-        return '\n'.join(lines) + '\n'
-    elif shell in ['csh', 'tcsh']:
-        lines = []
-        for key, value in valid_keys.items():
-            lines.append(f'setenv {key} "{value}"')
-        return '\n'.join(lines) + '\n'
-    else:  # bash, zsh, ksh, sh and others
-        lines = []
-        for key, value in valid_keys.items():
-            lines.append(f'export {key}="{value}"')
-        return '\n'.join(lines) + '\n'
+    for key, value in valid_keys.items():
+        # shlex.quote is designed for POSIX shells (sh, bash, zsh, ksh)
+        # It also works reasonably well for fish and csh for simple assignments
+        quoted_val = shlex.quote(value)
+        
+        if shell == 'fish':
+            lines.append(f'set -gx {key} {quoted_val}')
+        elif shell in ['csh', 'tcsh']:
+            lines.append(f'setenv {key} {quoted_val}')
+        else:  # bash, zsh, ksh, sh and others
+            lines.append(f'export {key}={quoted_val}')
+            
+    return '\n'.join(lines) + '\n'
 
 def save_configuration(valid_keys: Dict[str, str]) -> Tuple[List[str], bool, Optional[str]]:
     """Save configuration to ~/.pdd/ directory"""
@@ -645,4 +645,4 @@ def main():
         main()
     except KeyboardInterrupt:
         print_colored("\n\nSetup cancelled.", YELLOW)
-        sys.exit(0)
+        sys.exit(0)