Skip to content

fix(server): initialize SQL parser WASM in Workers#187

Open
siisee11 wants to merge 3 commits into
mainfrom
codex/fix-sql-parser-wasm
Open

fix(server): initialize SQL parser WASM in Workers#187
siisee11 wants to merge 3 commits into
mainfrom
codex/fix-sql-parser-wasm

Conversation

@siisee11
Copy link
Copy Markdown
Contributor

@siisee11 siisee11 commented May 4, 2026

Summary

  • Initialize the SQL parser WASM through a Worker-safe dynamic import when running in Cloudflare Workers.
  • Preserve the existing Node/self-host runtime asset path for Node execution.
  • Apply formatter-required import ordering.

Validation

  • bun --filter @onequery/server typecheck
  • bun --cwd vendor/onequery/packages/server test src/services/data-source-query/execute-query.test.ts
  • bun --cwd apps/web build

Downstream

  • Velen PR: https://github.com/wordbricks/velen/pull/891
  • Velen currently points at an equivalent fix commit on top of onequery/codex/shared-connection-guide because Velen main already depends on that branch's shared connection guide export.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 4, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
onequery-landing 9f71bcf May 05 2026, 08:17 AM

@siisee11 siisee11 enabled auto-merge (squash) May 5, 2026 10:01
@lentil32
Copy link
Copy Markdown
Contributor

lentil32 commented May 6, 2026

@codex are those type guards all necessary ?

@chatgpt-codex-connector
Copy link
Copy Markdown

To use Codex here, create an environment for this repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants