Add MCP preflight workflow and update NL suite

Author: dsarno

.github/workflows/claude-mcp-preflight.yml

@@ -0,0 +1,55 @@
+name: Claude MCP Preflight (no Unity)
+
+on: [workflow_dispatch]
+
+permissions:
+  contents: read
+
+jobs:
+  mcp-preflight:
+    runs-on: ubuntu-latest
+    timeout-minutes: 15
+
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - uses: astral-sh/setup-uv@v4
+        with:
+          python-version: "3.11"
+
+      - name: Install MCP server deps
+        run: |
+          set -eux
+          uv venv
+          echo "VIRTUAL_ENV=$GITHUB_WORKSPACE/.venv" >> "$GITHUB_ENV"
+          echo "$GITHUB_WORKSPACE/.venv/bin" >> "$GITHUB_PATH"
+          if [ -f Server/pyproject.toml ]; then
+            uv pip install -e Server
+          elif [ -f Server/requirements.txt ]; then
+            uv pip install -r Server/requirements.txt
+          else
+            echo "No MCP Python deps found" >&2
+            exit 1
+          fi
+
+      - name: Preflight MCP server (stdio)
+        env:
+          PYTHONUNBUFFERED: "1"
+          MCP_LOG_LEVEL: debug
+          UNITY_PROJECT_ROOT: ${{ github.workspace }}/TestProjects/UnityMCPTests
+          UNITY_MCP_STATUS_DIR: ${{ github.workspace }}/.unity-mcp-dummy
+          UNITY_MCP_HOST: 127.0.0.1
+        run: |
+          set -euxo pipefail
+          mkdir -p "$UNITY_MCP_STATUS_DIR"
+          # Create a dummy status file with an unreachable port; help should not require it
+          cat > "$UNITY_MCP_STATUS_DIR/unity-mcp-status-dummy.json" <<JSON
+          { "unity_port": 0, "reason": "dummy", "reloading": false, "project_path": "$UNITY_PROJECT_ROOT/Assets" }
+          JSON
+          uv run --active --directory Server mcp-for-unity --transport stdio --help \
+            > /tmp/mcp-preflight.log 2>&1 || { cat /tmp/mcp-preflight.log; exit 1; }
+          cat /tmp/mcp-preflight.log
+
+

.github/workflows/claude-nl-suite.yml

@@ -120,7 +120,7 @@ jobs:
           UNITY_PASSWORD: ${{ secrets.UNITY_PASSWORD }}
           UNITY_SERIAL: ${{ secrets.UNITY_SERIAL }}
         run: |
-          set -euxo pipefail
+          set -euo pipefail
           # host dirs to receive the full Unity config and local-share
           mkdir -p "$RUNNER_TEMP/unity-config" "$RUNNER_TEMP/unity-local"
 
@@ -156,7 +156,7 @@ jobs:
 
       # ---------- Warm up project (import Library once) ----------
       - name: Warm up project (import Library once)
-        if: steps.lic.outputs.use_ulf == 'true' || steps.lic.outputs.use_ebl == 'true'
+        if: steps.detect.outputs.anthropic_ok == 'true' && (steps.lic.outputs.use_ulf == 'true' || steps.lic.outputs.use_ebl == 'true')
         shell: bash
         env:
           UNITY_IMAGE: ${{ env.UNITY_IMAGE }}
@@ -172,6 +172,7 @@ jobs:
             -v "${{ github.workspace }}:/workspace" -w /workspace \
             -v "$RUNNER_TEMP/unity-config:/root/.config/unity3d" \
             -v "$RUNNER_TEMP/unity-local:/root/.local/share/unity3d" \
+            -v "$RUNNER_TEMP/unity-cache:/root/.cache/unity3d" \
             "$UNITY_IMAGE" /opt/unity/Editor/Unity -batchmode -nographics -logFile - \
               -projectPath /workspace/TestProjects/UnityMCPTests \
               "${manual_args[@]}" \
@@ -186,7 +187,7 @@ jobs:
 
       # ---------- Start headless Unity (persistent bridge) ----------
       - name: Start Unity (persistent bridge)
-        if: steps.lic.outputs.use_ulf == 'true' || steps.lic.outputs.use_ebl == 'true'
+        if: steps.detect.outputs.anthropic_ok == 'true' && (steps.lic.outputs.use_ulf == 'true' || steps.lic.outputs.use_ebl == 'true')
         shell: bash
         env:
           UNITY_IMAGE: ${{ env.UNITY_IMAGE }}
@@ -218,10 +219,11 @@ jobs:
 
       # ---------- Wait for Unity bridge ----------
       - name: Wait for Unity bridge (robust)
+        if: steps.detect.outputs.anthropic_ok == 'true' && (steps.lic.outputs.use_ulf == 'true' || steps.lic.outputs.use_ebl == 'true')
         shell: bash
         run: |
           set -euo pipefail
-          deadline=$((SECONDS+900))          # 15 min max
+          deadline=$((SECONDS+600))          # 10 min max
           fatal_after=$((SECONDS+120))       # give licensing 2 min to settle
 
           # Fail fast only if container actually died
@@ -272,7 +274,7 @@ jobs:
 
       # ---------- Debug Unity bridge status ----------
       - name: Debug Unity bridge status
-        if: always()
+        if: always() && (steps.lic.outputs.use_ulf == 'true' || steps.lic.outputs.use_ebl == 'true')
         shell: bash
         run: |
           set -euxo pipefail
@@ -341,11 +343,11 @@ jobs:
             "permissions": {
               "allow": [
                 "mcp__unity",
-                "Edit(reports/**)"
+                "Edit(reports/**)",
+                "MultiEdit(reports/**)"
               ],
               "deny": [
                 "Bash",
-                "MultiEdit",
                 "WebFetch",
                 "WebSearch",
                 "Task",
@@ -357,7 +359,7 @@ jobs:
           }
           JSON
 
-      # ---------- Reports & helper ----------
+          # ---------- Reports & helper ----------
       - name: Prepare reports and dirs
         run: |
           set -eux
@@ -405,7 +407,7 @@ jobs:
           export UNITY_PROJECT_ROOT="$GITHUB_WORKSPACE/TestProjects/UnityMCPTests"
           export UNITY_MCP_STATUS_DIR="$HOME/.unity-mcp"
           export UNITY_MCP_HOST=127.0.0.1
-          uv run --active --directory Server mcp-for-unity --help \
+          uv run --active --directory Server mcp-for-unity --transport stdio --help \
             > /tmp/mcp-preflight.log 2>&1 || { cat /tmp/mcp-preflight.log; exit 1; }
           cat /tmp/mcp-preflight.log