Merge pull request #32 from neuromechanist/develop

Fix CI: use testing API key and optimize integration test runs

Author: neuromechanist

.github/workflows/test.yml

@@ -128,21 +128,19 @@ jobs:
           pip install -e ".[dev]"
           pip install pytest-timeout
 
-      - name: Run integration tests
+      - name: Run integration tests only
         env:
           OPENROUTER_API_KEY_FOR_TESTING: ${{ secrets.OPENROUTER_API_KEY_FOR_TESTING }}
         run: |
           # Only run if the secret is available
           if [ -n "$OPENROUTER_API_KEY_FOR_TESTING" ]; then
             echo "Running integration tests with OpenRouter..."
-            pytest tests/test_integration_openrouter.py -v -m integration --timeout=180 --cov=src --cov-report=xml:coverage-integration.xml --cov-report=term-missing
+            pytest tests/ -v -m integration --timeout=180 --cov=src --cov-report=xml:coverage-integration.xml --cov-report=term-missing
           else
             echo "OPENROUTER_API_KEY_FOR_TESTING not set, skipping integration tests"
           fi
 
       - name: Upload integration coverage to Codecov
-        # Always upload coverage, even if some tests fail
-        # This ensures partial coverage is still counted
         if: always()
         uses: codecov/codecov-action@v4
         with:

codecov.yml

@@ -11,10 +11,6 @@ coverage:
         # Allow coverage to drop by 2% without failing
         threshold: 2%
         if_ci_failed: error
-        # Only report status after all flags are uploaded
-        flags:
-          - unittests
-          - integration
     patch:
       default:
         # New code should have coverage, but be lenient
@@ -55,5 +51,5 @@ flags:
     paths:
       - src/
     carryforward: true
-    # Integration tests run less frequently (only on merge with src changes)
-    # Carry forward ensures previous results are used when skipped
+    # Integration tests run less frequently (only when related files change)
+    # Carryforward ensures previous coverage is used when skipped

src/scripts/process_feedback.py

@@ -47,12 +47,12 @@ async def process_feedback_file(
     from src.utils.github_client import GitHubClient
     from src.utils.openrouter_llm import create_openrouter_llm
 
-    # Get API keys from environment
-    openrouter_key = os.getenv("OPENROUTER_API_KEY")
+    # Get API keys from environment (prefer testing key for CI/tests)
+    openrouter_key = os.getenv("OPENROUTER_API_KEY_FOR_TESTING") or os.getenv("OPENROUTER_API_KEY")
     github_token = os.getenv("GITHUB_TOKEN")
 
     if not openrouter_key:
-        logger.error("OPENROUTER_API_KEY not set")
+        logger.error("OPENROUTER_API_KEY or OPENROUTER_API_KEY_FOR_TESTING not set")
         sys.exit(1)
 
     # Get model configuration from environment