Skip to content

test(ai-gateway-provider): cover unified dynamic routes - #619

Open
dhruvil-codes wants to merge 1 commit into
cloudflare:mainfrom
dhruvil-codes:investigate/dynamic-workers-ai-route
Open

test(ai-gateway-provider): cover unified dynamic routes#619
dhruvil-codes wants to merge 1 commit into
cloudflare:mainfrom
dhruvil-codes:investigate/dynamic-workers-ai-route

Conversation

@dhruvil-codes

Copy link
Copy Markdown

Summary

Adds live E2E coverage for AI Gateway unified dynamic routes.

The existing ai-gateway-provider E2E suite covers provider/BYOK routing through the universal endpoint, but does not exercise a dynamic route resolving to a Workers AI model.

This adds a gated E2E case using:

aigateway(unified(DYNAMIC_ROUTE))

and verifies that generateText successfully returns the expected response through the universal-endpoint REST path.

Motivation

Issue #617 reports that a dynamic route targeting Workers AI fails through the AI Gateway universal endpoint while the same request succeeds through /compat/chat/completions.

This test exercises the affected ai-gateway-provider path and provides regression coverage for the universal-endpoint dynamic-route flow.

Testing

  • pnpm --filter ai-gateway-provider type-check
  • vitest --config vitest.e2e.config.ts --watch=false test/e2e/gateway.e2e.test.ts
  • oxfmt --check test/e2e/gateway.e2e.test.ts

All checks pass locally. The live dynamic-route case is gated by CLOUDFLARE_AI_GATEWAY_DYNAMIC_ROUTE.

Refs #617

@changeset-bot

changeset-bot Bot commented Jul 15, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 8b115e8

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

@ArshVermaGit ArshVermaGit left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice test addition. Covering the unified dynamic route improves confidence in the AI Gateway routing flow and helps catch regressions in the affected path. The changes are focused and fit well with the existing test suite. LGTM.

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