Address PR review feedback for issue #2169

- Use existing strip_ansi helper from conftest instead of duplicating ANSI escape pattern
- Properly escape ai_commands_dir with shlex.quote() to handle paths with spaces
- Add shlex import to support proper command-line argument escaping

Author: mnriem

src/specify_cli/__init__.py

@@ -33,6 +33,7 @@
 import json
 import json5
 import stat
+import shlex
 import yaml
 from pathlib import Path
 from typing import Any, Optional
@@ -102,7 +103,7 @@ def _build_integration_equivalent(
     parts = [f"--integration {integration_key}"]
     if integration_key == "generic" and ai_commands_dir:
         parts.append(
-            f'--integration-options="--commands-dir {ai_commands_dir}"'
+            f'--integration-options="--commands-dir {shlex.quote(ai_commands_dir)}"'
         )
     return " ".join(parts)
 

tests/integrations/test_cli.py

@@ -2,14 +2,15 @@
 
 import json
 import os
-import re
 
 import yaml
 
+from tests.conftest import strip_ansi
+
 
 def _normalize_cli_output(output: str) -> str:
-    output = re.sub(r"\x1b\[[0-9;]*m", "", output)
-    output = re.sub(r"\s+", " ", output)
+    output = strip_ansi(output)
+    output = " ".join(output.split())
     return output.strip()