Skip to content

Commit f729a8d

Browse files
committed
🔧 Fix stdio2sse release workflow hanging on server health check
- Replace curl health check with process existence check - SSE endpoints don't respond to simple HTTP requests - Increase startup wait time from 3s to 5s for stability - Add clear success/failure messages for server startup
1 parent e0ba875 commit f729a8d

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

.github/workflows/release-stdio2sse.yml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,9 +61,13 @@ jobs:
6161
SIDEKICK_PID=$!
6262
echo "SIDEKICK_PID=$SIDEKICK_PID" >> $GITHUB_ENV
6363
# Wait for server to start
64-
sleep 3
65-
# Check if server is running
66-
curl -f http://localhost:5050/mcp/sse || (echo "Failed to start sidekick server" && exit 1)
64+
sleep 5
65+
# Check if process is still running
66+
if ! ps -p $SIDEKICK_PID > /dev/null; then
67+
echo "❌ Sidekick server failed to start"
68+
exit 1
69+
fi
70+
echo "✅ Sidekick server started successfully (PID: $SIDEKICK_PID)"
6771
6872
- name: Run stdio2sse tests
6973
working-directory: ./stdio2sse

0 commit comments

Comments
 (0)