Skip to content

Add AI Search oauth scopes#12064

Merged
petebacondarwin merged 1 commit intocloudflare:mainfrom
G4brym:add-ai-search-oauth-scopes
Jan 29, 2026
Merged

Add AI Search oauth scopes#12064
petebacondarwin merged 1 commit intocloudflare:mainfrom
G4brym:add-ai-search-oauth-scopes

Conversation

@G4brym
Copy link
Copy Markdown
Member

@G4brym G4brym commented Jan 23, 2026 •
edited
Loading

This pr adds AI Search oauth scopes

  • 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: does not affect users directly

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

Open with Devin

@G4brym G4brym requested a review from a team as a code owner January 23, 2026 13:46
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jan 23, 2026 •
edited
Loading

🦋 Changeset detected

Latest commit: bfc43cc

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

@github-project-automation github-project-automation Bot moved this to Untriaged in workers-sdk Jan 23, 2026
devin-ai-integration[bot]
devin-ai-integration Bot reviewed Jan 23, 2026
View reviewed changes
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 issue and 2 additional flags in Devin Review.

Open in Devin Review

Comment thread packages/wrangler/src/user/user.ts
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Jan 23, 2026 •
edited
Loading

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

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

wrangler

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

commit: bfc43cc

@G4brym G4brym force-pushed the add-ai-search-oauth-scopes branch 3 times, most recently from ca35a09 to 0221684 Compare January 23, 2026 14:33
@github-actions github-actions Bot mentioned this pull request Jan 26, 2026
Closed
@edmundhung edmundhung mentioned this pull request Jan 26, 2026
Merged
5 tasks
edmundhung
edmundhung reviewed Jan 26, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@edmundhung edmundhung left a comment

Choose a reason for hiding this comment

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

You might need to update wrangler client's permission to request that scope. I tried running wrangler login with the pre-release and it fails with ERROR invalid_scope.

@G4brym G4brym force-pushed the add-ai-search-oauth-scopes branch from 0221684 to 255e561 Compare January 28, 2026 17:11
@G4brym
Copy link
Copy Markdown
Member Author

G4brym commented Jan 28, 2026

Hey @edmundhung ive now fixed the issue on oauth server, and login does work on preview package now
Can you take a look at the pr again?

edmundhung
edmundhung approved these changes Jan 28, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@edmundhung edmundhung left a comment

Choose a reason for hiding this comment

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

Thank you!

@github-project-automation github-project-automation Bot moved this from Untriaged to Approved in workers-sdk Jan 28, 2026
@edmundhung edmundhung enabled auto-merge January 28, 2026 17:14
@edmundhung edmundhung disabled auto-merge January 28, 2026 17:15
@edmundhung edmundhung enabled auto-merge January 28, 2026 18:26
auto-merge was automatically disabled January 28, 2026 19:15

Head branch was pushed to by a user without write access

@G4brym G4brym force-pushed the add-ai-search-oauth-scopes branch from 255e561 to bfc43cc Compare January 28, 2026 19:15
@G4brym
Copy link
Copy Markdown
Member Author

G4brym commented Jan 29, 2026

CI tests in windows appear to be failing on a flaky test, can we merge this anyway?

@petebacondarwin
Copy link
Copy Markdown
Contributor

petebacondarwin commented Jan 29, 2026

Re-running the flaky test and clicked the merge-when-ready button.

@petebacondarwin petebacondarwin added this pull request to the merge queue Jan 29, 2026
Merged via the queue into cloudflare:main with commit 964a39d Jan 29, 2026
37 of 38 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Done in workers-sdk Jan 29, 2026
@workers-devprod workers-devprod mentioned this pull request Jan 29, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devin-ai-integration devin-ai-integration[bot] devin-ai-integration[bot] left review comments

@edmundhung edmundhung edmundhung approved these changes

Assignees

No one assigned

Labels

None yet

Projects

workers-sdk
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@G4brym @petebacondarwin @edmundhung @workers-devprod