fix(tests): Address CodeRabbit feedback

- Update SPEC status from PLANNED to IMPLEMENTED (PR #125)
- Move `import os` to module level per best practices

SPEC_ARCHITECTURAL_DEBT.md

@@ -230,9 +230,9 @@ class WorkflowState:
 
 **CRITICAL**: Each phase MUST pass smoke tests before merge. Unit tests alone are insufficient.
 
-> **Implementation Status**: PLANNED (Part of Priority 8)
-> The smoke tests below are documented but NOT YET IMPLEMENTED.
-> They should be created as a prerequisite before starting Priority 4-8 refactors.
+> **Implementation Status**: IMPLEMENTED (PR #125)
+> Smoke tests are now live in `tests/e2e/test_smoke.py` with Makefile targets.
+> Run `make smoke-free` or `make smoke-paid` before any refactoring PR.
 
 ### Smoke Test Infrastructure
 

tests/e2e/test_smoke.py

@@ -9,6 +9,8 @@ Usage:
     make smoke-paid   # Test Paid Tier (OpenAI BYOK)
 """
 
+import os
+
 import pytest
 
 from src.orchestrators.advanced import AdvancedOrchestrator
@@ -47,8 +49,6 @@ async def test_free_tier_synthesis():
 @pytest.mark.timeout(300)  # 5 minute timeout for Paid Tier
 async def test_paid_tier_synthesis():
     """Verify Paid Tier (BYOK) produces synthesis."""
-    import os
-
     api_key = os.environ.get("OPENAI_API_KEY")
     if not api_key:
         pytest.skip("OPENAI_API_KEY not set")