Skip to content

Fix AI Search rpc binding#12543

Merged
ascorbic merged 1 commit intocloudflare:mainfrom
G4brym:fix/ai-search-rpc
Feb 16, 2026
Merged

Fix AI Search rpc binding#12543
ascorbic merged 1 commit intocloudflare:mainfrom
G4brym:fix/ai-search-rpc

Conversation

@G4brym
Copy link
Copy Markdown
Member

@G4brym G4brym commented Feb 12, 2026

Fixes #[insert GH or internal issue link(s)].

AI Search binding is of type RPC, and with current implementation it did not work


  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: docs will only be added when we announce the new binding

A picture of a cute animal (not mandatory, but encouraged)


Open with Devin

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Feb 12, 2026

🦋 Changeset detected

Latest commit: 6232b3b

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

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

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Feb 12, 2026

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@12543

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@12543

miniflare

npm i https://pkg.pr.new/miniflare@12543

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@12543

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@12543

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@12543

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@12543

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@12543

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@12543

wrangler

npm i https://pkg.pr.new/wrangler@12543

commit: 6232b3b

@G4brym G4brym marked this pull request as ready for review February 12, 2026 15:18
@G4brym G4brym requested a review from a team as a code owner February 12, 2026 15:18
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

View 4 additional findings in Devin Review.

Open in Devin Review

Copy link
Copy Markdown

@JoaquinGimenez1 JoaquinGimenez1 left a comment

Choose a reason for hiding this comment

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

LGTM

@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Feb 16, 2026
@ascorbic ascorbic merged commit 5a868a0 into cloudflare:main Feb 16, 2026
37 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Done in workers-sdk Feb 16, 2026
penalosa added a commit that referenced this pull request Feb 26, 2026
The workaround added MF-Binding-Type threading and a plain-object
wrapper in ProxyServerWorker to work around capnweb classifying the
raw AI binding prototype as unsupported. This has since been fixed
properly upstream, so remove the workaround cleanly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants