feat: Added voice feature for end-user fix #325

.gitignore

@@ -19,6 +19,22 @@ __pycache__/
 *.py[cod]
 *$py.class
 *.so
+
+# ==============================
+# Virtual Environments
+# ==============================
+env/
+venv/
+myenv/
+venv312/
+ENV/
+env.bak/
+venv.bak/
+.venv/
+
+# ==============================
+# Distribution / Packaging
+# ==============================
 .Python
 build/
 develop-eggs/
@@ -37,7 +53,11 @@ wheels/
 *.egg
 .venv/
 venv/
+myenv/
+venv312/
 ENV/
+env.bak/
+venv.bak/
 
 # IDE
 .idea/

README.md

@@ -64,6 +64,7 @@ cortex install "tools for video compression"
 | Feature | Description |
 |---------|-------------|
 | **Natural Language** | Describe what you need in plain English |
+| **Voice Input** | Hands-free mode with Whisper speech recognition ([F9 to speak](docs/VOICE_INPUT.md)) |
 | **Dry-Run Default** | Preview all commands before execution |
 | **Sandboxed Execution** | Commands run in Firejail isolation |
 | **Full Rollback** | Undo any installation with `cortex rollback` |

cortex/api_key_detector.py

@@ -125,24 +125,21 @@ def detect(self) -> tuple[bool, str | None, str | None, str | None]:
     def _check_environment_api_keys(self) -> tuple[bool, str, str, str] | None:
         """Check for API keys in environment variables.
 
-        Respects CORTEX_PROVIDER setting when multiple keys are available.
-        Falls back to OpenAI if Anthropic is not available but OpenAI is.
+        Respects CORTEX_PROVIDER when multiple keys exist and prefers OpenAI when unspecified.
         """
-        # Check if user has explicit provider preference
-        preferred_provider = os.environ.get("CORTEX_PROVIDER", "").lower()
+        explicit_provider = os.environ.get("CORTEX_PROVIDER", "").lower()
 
-        # If provider is specified, check for that key first
-        if preferred_provider in ("anthropic", "claude"):
+        # If user explicitly set a provider, check that key first
+        if explicit_provider in ("anthropic", "claude"):
             value = os.environ.get("ANTHROPIC_API_KEY")
             if value:
                 return (True, value, "anthropic", "environment")
-        elif preferred_provider == "openai":
+        elif explicit_provider == "openai":
             value = os.environ.get("OPENAI_API_KEY")
             if value:
                 return (True, value, "openai", "environment")
 
-        # Fall back to checking all keys if no preference or preferred key not found
-        # Prefer OpenAI over Anthropic if no explicit preference (since Anthropic seems to have issues)
+        # Fallback: prefer OpenAI first, then Anthropic
         for env_var, provider in [("OPENAI_API_KEY", "openai"), ("ANTHROPIC_API_KEY", "anthropic")]:
             value = os.environ.get(env_var)
             if value:
@@ -160,7 +157,29 @@ def _check_encrypted_storage(self) -> tuple[bool, str, str, str] | None:
 
             env_mgr = get_env_manager()
 
-            # Check for API keys in encrypted storage
+            # If CORTEX_PROVIDER is explicitly set, check that provider's key first
+            explicit_provider = os.environ.get("CORTEX_PROVIDER", "").lower()
+            if explicit_provider in ["openai", "claude", "anthropic"]:
+                # Map provider names to env vars and canonical provider names
+                if explicit_provider == "openai":
+                    target_env_var = "OPENAI_API_KEY"
+                    target_provider = "openai"
+                else:  # claude or anthropic both map to ANTHROPIC_API_KEY
+                    target_env_var = "ANTHROPIC_API_KEY"
+                    target_provider = "anthropic"
+
+                value = env_mgr.get_variable(app="cortex", key=target_env_var, decrypt=True)
+                if value:
+                    os.environ[target_env_var] = value
+                    logger.debug(f"Loaded {target_env_var} from encrypted storage")
+                    return (
+                        True,
+                        value,
+                        target_provider,
+                        "encrypted storage (~/.cortex/environments/)",
+                    )
+
+            # Check for API keys in encrypted storage (default order)
             for env_var, provider in ENV_VAR_PROVIDERS.items():
                 value = env_mgr.get_variable(app="cortex", key=env_var, decrypt=True)
                 if value:

cortex/branding.py

@@ -11,12 +11,15 @@
 - Consistent visual language
 """
 
+import sys
+
 from rich import box
 from rich.console import Console
 from rich.panel import Panel
 from rich.table import Table
 
-console = Console()
+# Use force_terminal for consistent styled output across environments
+console = Console(force_terminal=True)
 
 # Brand colors
 CORTEX_CYAN = "cyan"
@@ -70,13 +73,23 @@ def cx_print(message: str, status: str = "info"):
     """
     badge = "[bold white on dark_cyan] CX [/bold white on dark_cyan]"
 
-    status_icons = {
-        "info": "[dim]│[/dim]",
-        "success": "[green]✓[/green]",
-        "warning": "[yellow]⚠[/yellow]",
-        "error": "[red]✗[/red]",
-        "thinking": "[cyan]⠋[/cyan]",  # Spinner frame
-    }
+    # Use ASCII-only icons on Windows for better compatibility
+    if sys.platform == "win32":
+        status_icons = {
+            "info": "[dim]|[/dim]",
+            "success": "[green]+[/green]",
+            "warning": "[yellow]![/yellow]",
+            "error": "[red]x[/red]",
+            "thinking": "[cyan]*[/cyan]",
+        }
+    else:
+        status_icons = {
+            "info": "[dim]│[/dim]",
+            "success": "[green]✓[/green]",
+            "warning": "[yellow]⚠[/yellow]",
+            "error": "[red]✗[/red]",
+            "thinking": "[cyan]⠋[/cyan]",  # Spinner frame
+        }
 
     icon = status_icons.get(status, status_icons["info"])
     console.print(f"{badge} {icon} {message}")
@@ -86,18 +99,20 @@ def cx_step(step_num: int, total: int, message: str):
     """
     Print a numbered step with the CX badge.
 
-    Example: CX │ [1/4] Updating package lists...
+    Example: CX | [1/4] Updating package lists...
     """
     badge = "[bold white on dark_cyan] CX [/bold white on dark_cyan]"
-    console.print(f"{badge} [dim]│[/dim] [{step_num}/{total}] {message}")
+    separator = "|" if sys.platform == "win32" else "│"
+    console.print(f"{badge} [dim]{separator}[/dim] [{step_num}/{total}] {message}")
 
 
 def cx_header(title: str):
     """
     Print a section header.
     """
     console.print()
-    console.print(f"[bold cyan]━━━ {title} ━━━[/bold cyan]")
+    separator = "---" if sys.platform == "win32" else "━━━"
+    console.print(f"[bold cyan]{separator} {title} {separator}[/bold cyan]")
     console.print()