Fixed some github CI yaml errors

Author: hillarysanders

.github/workflows/test.yml

@@ -1,8 +1,6 @@
 name: MCP Tests
 
 on:
-  push:
-    branches:
   pull_request:
 
 jobs:
@@ -11,8 +9,6 @@ jobs:
   ###########################################################################
   local:
     runs-on: ubuntu-latest
-
-    # Dummy key lets the clients authenticate against the local servers.
     env:
       API_KEY: ci-test-key
 
@@ -23,7 +19,6 @@ jobs:
         with:
           python-version: "3.11"
 
-      # Optional: cache wheels to speed up numpy / scipy installs
       - uses: actions/cache@v4
         with:
           path: ~/.cache/pip
@@ -33,47 +28,45 @@ jobs:
         run: |
           python -m pip install --upgrade pip
           pip install -r requirements.txt
-          pip install -r requirements-dev.txt
 
       - name: Run pytest (local transports)
         run: pytest -q
 
-
   ###########################################################################
-  # 2 - Remote smoke-test (only when secrets are set)
-  #
-  #    • Put MCP_SERVER_URL  → “https://<app>.herokuapp.com”
-  #    • Put API_KEY         → same key you set as Heroku config var
-  #
-  #    The fixture auto-skips the remote case if these are missing, so
-  #    the job is conditionally *created* only when both secrets exist.
+  # 2 - Remote smoke-test (skipped if ENV vars are missing)
   ###########################################################################
   remote:
-    if: ${{ secrets.MCP_SERVER_URL != '' && secrets.API_KEY != '' }}
     runs-on: ubuntu-latest
-
     env:
       MCP_SERVER_URL: ${{ secrets.MCP_SERVER_URL }}
-      API_KEY:       ${{ secrets.API_KEY }}
+      API_KEY: ${{ secrets.API_KEY }}
+      SHOULD_RUN_REMOTE: ${{ secrets.MCP_SERVER_URL != '' && secrets.API_KEY != '' }}
 
     steps:
+      - name: Skip if secrets are missing
+        if: env.SHOULD_RUN_REMOTE != 'true'
+        run: echo "Skipping remote tests due to missing secrets."
+
       - uses: actions/checkout@v4
+        if: env.SHOULD_RUN_REMOTE == 'true'
 
       - uses: actions/setup-python@v5
+        if: env.SHOULD_RUN_REMOTE == 'true'
         with:
           python-version: "3.11"
 
       - uses: actions/cache@v4
+        if: env.SHOULD_RUN_REMOTE == 'true'
         with:
           path: ~/.cache/pip
           key: ${{ runner.os }}-pip-${{ hashFiles('**/requirements*.txt') }}
 
       - name: Install dependencies
+        if: env.SHOULD_RUN_REMOTE == 'true'
         run: |
           python -m pip install --upgrade pip
           pip install -r requirements.txt
 
-      # We reuse the *same* test-suite; the fixture detects MCP_SERVER_URL
-      # and adds the “remote” parameter automatically.
       - name: Run pytest (remote smoke)
+        if: env.SHOULD_RUN_REMOTE == 'true'
         run: pytest -q