Merge pull request #2 from palfrey/fix-workflows

Author: MarcusSorealheis

.github/workflows/ci.yml

@@ -8,7 +8,7 @@ on:
 
 jobs:
   build:
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
 
     strategy:
       matrix:
@@ -35,13 +35,13 @@ jobs:
       run: node dist/index.js --help
 
     - name: Upload build artifacts
-      uses: actions/upload-artifact@v3
+      uses: actions/upload-artifact@v4
       with:
         name: dist-${{ matrix.node-version }}
         path: dist/
 
   test-mcp:
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
     needs: build
 
     steps:
@@ -61,6 +61,7 @@ jobs:
     - name: Test with MCP Inspector
       run: |
         npx @modelcontextprotocol/inspector node dist/index.js &
-        sleep 5
-        kill %1
-      continue-on-error: true
+        curl -O https://raw.githubusercontent.com/vishnubob/wait-for-it/refs/heads/master/wait-for-it.sh
+        chmod +x wait-for-it.sh
+        ./wait-for-it.sh --timeout=240 -h localhost -p 6274
+        curl -v http://localhost:6277/config

.github/workflows/publish.yml

@@ -6,7 +6,7 @@ on:
 
 jobs:
   publish:
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
 
     steps:
     - uses: actions/checkout@v3

.gitignore

@@ -29,8 +29,8 @@ logs/
 # Testing
 coverage/
 .nyc_output/
+wait-for-it.sh
 
 # Package manager
-package-lock.json
 yarn.lock
 pnpm-lock.yaml