chore: update Supabase dependencies and improve e2e test infrastructure #292

jumski · 2025-11-02T00:40:15Z

Update Supabase JS SDK and Improve Edge Worker E2E Testing

This PR updates the Supabase JS SDK from version 2.55.0 to 2.74.0 and significantly improves the edge worker E2E testing infrastructure.

Key changes:

Added a new script sync-e2e-deps.sh that copies built packages to a vendor directory for E2E tests
Created a new import map in supabase/functions/deno.json to properly resolve dependencies during tests
Added a new serve:functions:e2e task that uses the vendor directory for testing
Removed SQL views and functions that are no longer needed (active_workers, inactive_workers, spawn_worker)
Updated worker detection logic to use direct queries instead of views
Improved worker startup logic with retry mechanism for better test reliability
Added E2E test environment variables to .env file
Added --no-verify-jwt flag to function serving to simplify testing

These changes make the E2E tests more reliable and easier to run, while also keeping the codebase up-to-date with the latest Supabase SDK.

changeset-bot · 2025-11-02T00:40:18Z

⚠️ No Changeset found

Latest commit: 1265cb0

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

jumski · 2025-11-02T00:40:29Z

chore: update Supabase dependencies and improve e2e test infrastructure #292 👈 (View in Graphite)
main

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

merge:queue - adds this PR to the back of the merge queue
hotfix:queue - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

_{An organization admin has enabled the Graphite Merge Queue in this repository.} _{Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.}

This stack of pull requests is managed by Graphite. Learn more about stacking.

nx-cloud · 2025-11-02T00:41:14Z

🤖 Nx Cloud AI Fix Eligible

An automatically generated fix could have helped fix failing tasks for this run, but Self-healing CI is disabled for this workspace. Visit workspace settings to enable it and get automatic fixes in future runs.

_{To disable these notifications, a workspace admin can disable them in workspace settings.}

View your CI Pipeline Execution ↗ for commit 1265cb0

Command	Status	Duration	Result
`nx affected -t lint typecheck test --parallel -...`	❌ Failed	7m 15s	View ↗
`nx affected -t test:e2e --parallel --base=b8aba...`	✅ Succeeded	6m 17s	View ↗

☁️ Nx Cloud last updated this comment at 2025-11-02 15:22:32 UTC

graphite-app · 2025-11-02T00:42:51Z

pkgs/edge-worker/scripts/sync-e2e-deps.sh

+find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.ts" -type f -exec sed -i 's/\.js"/\.ts"/g' {} +
+find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.ts" -type f -exec sed -i "s/\.js'/\.ts'/g" {} +


Critical: Platform compatibility issue with sed -i

The sed -i command behaves differently on macOS vs Linux. On macOS, sed -i requires a backup extension argument (even if empty), while Linux doesn't. This script will fail on macOS systems.

Fix:

# Use platform-independent approach find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.ts" -type f -exec sed -i.bak 's/\.js"/\.ts"/g' {} + find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.ts" -type f -exec sed -i.bak "s/\.js'/\.ts'/g" {} + find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.bak" -type f -delete

Or use a more portable solution:

find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.ts" -type f | while read -r file; do sed 's/\.js"/\.ts"/g; s/\.js'\''/.ts'\'''/g' "$file" > "$file.tmp" && mv "$file.tmp" "$file" done

Suggested change

find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.ts" -type f -exec sed -i 's/\.js"/\.ts"/g' {} +

find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.ts" -type f -exec sed -i "s/\.js'/\.ts'/g" {} +

find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.ts" -type f -exec sed -i.bak 's/\.js"/\.ts"/g' {} +

find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.ts" -type f -exec sed -i.bak "s/\.js'/\.ts'/g" {} +

find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.bak" -type f -delete

Spotted by Graphite Agent

Is this helpful? React 👍 or 👎 to let us know.

github-actions · 2025-11-02T00:54:43Z

🔍 Preview Deployment: Playground

❌ Deployment failed!

Please check the workflow logs for details.

pkgs/edge-worker/tests/e2e/_helpers.ts

pkgs/edge-worker/CLAUDE.md

github-actions · 2025-11-02T13:44:34Z

🔍 Preview Deployment: Website

✅ Deployment successful!

🔗 Preview URL: https://pr-292.pgflow.pages.dev

📝 Details:

Branch: fix-e2e-edg-worker-tests
Commit: 8e12e9daa42018dc49847cbaafd092236eff11dc
View Logs

_Last updated: _

…encies - Added a new script to synchronize vendor dependencies for e2e tests - Updated Deno lock file to include std library and remove outdated supabase-js entries - Modified project.json to include a new sync-e2e-deps target and adjust test dependencies - Removed obsolete import mappings from deno.json files in functions - Added .gitignore for vendor directory to prevent committing vendor files - Updated test command to use a different config file for e2e testing - Ensured build verification before copying dependencies in the sync script

graphite-app · 2025-11-02T15:13:49Z

Merge activity

Nov 2, 3:13 PM UTC: jumski added this pull request to the Graphite merge queue.
Nov 2, 3:14 PM UTC: CI is running for this pull request on a draft pull request (#295) due to your merge queue CI optimization settings.
Nov 2, 3:15 PM UTC: Merged by the Graphite merge queue via draft PR: #295.

…re (#292) # Update Supabase JS SDK and Improve Edge Worker E2E Testing This PR updates the Supabase JS SDK from version 2.55.0 to 2.74.0 and significantly improves the edge worker E2E testing infrastructure. Key changes: - Added a new script `sync-e2e-deps.sh` that copies built packages to a vendor directory for E2E tests - Created a new import map in `supabase/functions/deno.json` to properly resolve dependencies during tests - Added a new `serve:functions:e2e` task that uses the vendor directory for testing - Removed SQL views and functions that are no longer needed (`active_workers`, `inactive_workers`, `spawn_worker`) - Updated worker detection logic to use direct queries instead of views - Improved worker startup logic with retry mechanism for better test reliability - Added E2E test environment variables to `.env` file - Added `--no-verify-jwt` flag to function serving to simplify testing These changes make the E2E tests more reliable and easier to run, while also keeping the codebase up-to-date with the latest Supabase SDK.

graphite-app bot reviewed Nov 2, 2025

View reviewed changes

jumski had a problem deploying to preview November 2, 2025 00:47 — with GitHub Actions Failure

jumski temporarily deployed to preview November 2, 2025 00:47 — with GitHub Actions Inactive

jumski force-pushed the fix-e2e-edg-worker-tests branch 2 times, most recently from e933914 to 238cf24 Compare November 2, 2025 01:21

jumski temporarily deployed to preview November 2, 2025 01:30 — with GitHub Actions Inactive

jumski force-pushed the fix-e2e-edg-worker-tests branch from 238cf24 to ab368d4 Compare November 2, 2025 01:44

graphite-app bot reviewed Nov 2, 2025

View reviewed changes

pkgs/edge-worker/tests/e2e/_helpers.ts Outdated Show resolved Hide resolved

jumski force-pushed the fix-e2e-edg-worker-tests branch 2 times, most recently from ff76958 to 8be0970 Compare November 2, 2025 12:42

graphite-app bot reviewed Nov 2, 2025

View reviewed changes

pkgs/edge-worker/CLAUDE.md Show resolved Hide resolved

jumski temporarily deployed to preview November 2, 2025 12:51 — with GitHub Actions Inactive

jumski force-pushed the fix-e2e-edg-worker-tests branch from 8be0970 to 4c7428e Compare November 2, 2025 13:13

jumski temporarily deployed to preview November 2, 2025 13:22 — with GitHub Actions Inactive

jumski force-pushed the fix-e2e-edg-worker-tests branch from 4c7428e to c5371c5 Compare November 2, 2025 13:34

jumski temporarily deployed to preview November 2, 2025 13:43 — with GitHub Actions Inactive

jumski force-pushed the fix-e2e-edg-worker-tests branch from c5371c5 to 1265cb0 Compare November 2, 2025 15:13

graphite-app bot closed this Nov 2, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

chore: update Supabase dependencies and improve e2e test infrastructure #292

chore: update Supabase dependencies and improve e2e test infrastructure #292

Uh oh!

jumski commented Nov 2, 2025

Uh oh!

changeset-bot bot commented Nov 2, 2025 •

edited

Loading

Uh oh!

jumski commented Nov 2, 2025

Uh oh!

nx-cloud bot commented Nov 2, 2025 •

edited

Loading

Uh oh!

graphite-app bot Nov 2, 2025

Uh oh!

github-actions bot commented Nov 2, 2025

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Nov 2, 2025

Uh oh!

graphite-app bot commented Nov 2, 2025 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.ts" -type f -exec sed -i 's/\.js"/\.ts"/g' {} +
		find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.ts" -type f -exec sed -i "s/\.js'/\.ts'/g" {} +

-find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.ts" -type f -exec sed -i 's/\.js"/\.ts"/g' {} +
-find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.ts" -type f -exec sed -i "s/\.js'/\.ts'/g" {} +
+find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.ts" -type f -exec sed -i.bak 's/\.js"/\.ts"/g' {} +
+find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.ts" -type f -exec sed -i.bak "s/\.js'/\.ts'/g" {} +
+find "$VENDOR_DIR/@pgflow/edge-worker" -name "*.bak" -type f -delete

chore: update Supabase dependencies and improve e2e test infrastructure #292

chore: update Supabase dependencies and improve e2e test infrastructure #292

Uh oh!

Conversation

jumski commented Nov 2, 2025

Update Supabase JS SDK and Improve Edge Worker E2E Testing

Uh oh!

changeset-bot bot commented Nov 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

⚠️ No Changeset found

Uh oh!

jumski commented Nov 2, 2025

How to use the Graphite Merge Queue

Uh oh!

nx-cloud bot commented Nov 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🤖 Nx Cloud AI Fix Eligible

Uh oh!

graphite-app bot Nov 2, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Nov 2, 2025

🔍 Preview Deployment: Playground

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Nov 2, 2025

🔍 Preview Deployment: Website

Uh oh!

graphite-app bot commented Nov 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge activity

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

changeset-bot bot commented Nov 2, 2025 •

edited

Loading

nx-cloud bot commented Nov 2, 2025 •

edited

Loading

graphite-app bot commented Nov 2, 2025 •

edited

Loading