fix: configure worker deploy for GitHub Actions

- Enable preview_urls in wrangler configuration
- Remove invalid --out json flag from wrangler deploy
- Extract preview URL from wrangler deploy console output
- Update redirect tests to check for non-redirect status codes
  instead of expecting 200 (since there's no backend to proxy to)

The worker now properly deploys to a preview environment and
tests verify that redirects work as expected.

Signed-off-by: Mark Phelps <[email protected]>

Author: markphelps

.github/workflows/worker.yml

@@ -27,7 +27,19 @@ jobs:
         id: deploy
         working-directory: ./worker
         run: |
-          PREVIEW_URL=$(wrangler deploy --config wrangler.jsonc --env preview --dry-run=remote --out json | jq -r '.previews[0].url')
+          # Deploy the worker to preview environment
+          OUTPUT=$(wrangler deploy --config wrangler.jsonc --env preview 2>&1 | tee /dev/tty)
+          
+          # Extract the deployed URL from the output
+          # Look for the worker URL pattern
+          PREVIEW_URL=$(echo "$OUTPUT" | grep -o 'https://docs-preview[^[:space:]]*' | head -1)
+          
+          # If not found, try a more general pattern
+          if [ -z "$PREVIEW_URL" ]; then
+            PREVIEW_URL=$(echo "$OUTPUT" | grep -o 'https://[^[:space:]]*workers.dev' | head -1)
+          fi
+          
+          echo "Extracted preview URL: $PREVIEW_URL"
           echo "preview_url=$PREVIEW_URL" >> $GITHUB_OUTPUT
 
       - name: Run redirect tests
@@ -47,15 +59,30 @@ jobs:
           test "$LOCATION" = "$BASE_URL/v1/getting-started"
 
           echo "Testing already versioned path (/v1) → no redirect"
-          STATUS=$(curl -s -o /dev/null -w "%{http_code}" $BASE_URL/v1/getting-started)
-          test "$STATUS" -eq 200
+          STATUS=$(curl -s -o /dev/null -w "%{http_code}" --max-redirs 0 $BASE_URL/v1/getting-started)
+          # Should not get a redirect (301/302), might get 404 or 522 since there's no backend
+          if [ "$STATUS" -eq 301 ] || [ "$STATUS" -eq 302 ]; then
+            echo "ERROR: /v1/ path should not redirect but got status $STATUS"
+            exit 1
+          fi
+          echo "✓ Got status $STATUS (not a redirect)"
 
           echo "Testing already versioned path (/v2) → no redirect"
-          STATUS=$(curl -s -o /dev/null -w "%{http_code}" $BASE_URL/v2/getting-started)
-          test "$STATUS" -eq 200
+          STATUS=$(curl -s -o /dev/null -w "%{http_code}" --max-redirs 0 $BASE_URL/v2/getting-started)
+          # Should not get a redirect (301/302), might get 404 or 522 since there's no backend
+          if [ "$STATUS" -eq 301 ] || [ "$STATUS" -eq 302 ]; then
+            echo "ERROR: /v2/ path should not redirect but got status $STATUS"
+            exit 1
+          fi
+          echo "✓ Got status $STATUS (not a redirect)"
 
           echo "Testing excluded path (.css) → no redirect"
-          STATUS=$(curl -s -o /dev/null -w "%{http_code}" $BASE_URL/styles.css)
-          test "$STATUS" -eq 200
+          STATUS=$(curl -s -o /dev/null -w "%{http_code}" --max-redirs 0 $BASE_URL/styles.css)
+          # Should not get a redirect (301/302), might get 404 or 522 since there's no backend
+          if [ "$STATUS" -eq 301 ] || [ "$STATUS" -eq 302 ]; then
+            echo "ERROR: .css path should not redirect but got status $STATUS"
+            exit 1
+          fi
+          echo "✓ Got status $STATUS (not a redirect)"
 
           echo "✅ All redirect tests passed!"

worker/wrangler.jsonc

@@ -10,6 +10,7 @@
 	"observability": {
 		"enabled": true
 	},
+	"preview_urls": true,
 	"env": {
 		"preview": {
 			"name": "docs-preview"