Skip to content

fix: handle nitro-cloudflare-dev Promise bindings#742

Closed
onmax wants to merge 1 commit intonuxt-hub:mainfrom
onmax:fix/nitro-cloudflare-dev-bindings
Closed

fix: handle nitro-cloudflare-dev Promise bindings#742
onmax wants to merge 1 commit intonuxt-hub:mainfrom
onmax:fix/nitro-cloudflare-dev-bindings

Conversation

@onmax
Copy link
Copy Markdown
Contributor

@onmax onmax commented Dec 19, 2025 •
edited
Loading

Superseded

This PR has been superseded by:

@vercel
Copy link
Copy Markdown

vercel bot commented Dec 19, 2025

@onmax is attempting to deploy a commit to the NuxtLabs Team on Vercel.

A member of the Team first needs to authorize it.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Dec 19, 2025 •
edited
Loading 

npm i https://pkg.pr.new/@nuxthub/core@742

commit: 657c0f4

@onmax onmax mentioned this pull request Dec 19, 2025
Closed
@onmax onmax force-pushed the fix/nitro-cloudflare-dev-bindings branch from 4d07903 to 757e725 Compare December 19, 2025 15:57
@atinux
Copy link
Copy Markdown
Contributor

atinux commented Dec 19, 2025

Thanks for the help on this @onmax

Did you see that with latest Nitro compat, we don't need to install nitro-cloudflare-dev anymore?
https://nitro.build/deploy/providers/cloudflare#dev-preset

@onmax
Copy link
Copy Markdown
Contributor Author

onmax commented Dec 19, 2025 •
edited
Loading

I will be offline for the next week. Please feel free to iterate or close this PR if a better solution emerges.

Merry Christmas to the whole team! And well done on this new NuxtHub; I'm really excited about everything that's happening in the Nuxt ecosystem! 2026 will be 🔥

@atinux atinux requested a review from RihanArfan January 9, 2026 09:32
@atinux
Copy link
Copy Markdown
Contributor

atinux commented Jan 9, 2026

I will let @RihanArfan have a deep review at it once he's back from holidays :)

@onmax onmax force-pushed the fix/nitro-cloudflare-dev-bindings branch from 07128ba to 5617264 Compare January 9, 2026 10:39
This was referenced Jan 10, 2026
Closed
Open
Merged
Closed
@adinvadim
Copy link
Copy Markdown

adinvadim commented Jan 12, 2026

I think release 0.10 without this PR is beta, because many feature still not migrated without this PR

@onmax onmax mentioned this pull request Jan 16, 2026
Closed
@onmax
fix: handle nitro-cloudflare-dev Promise bindings
657c0f4 
- Add `hub.remote` option to use production Cloudflare bindings during dev
- Fix Promise bindings from nitro-cloudflare-dev via getPlatformProxy
- Change db import to dynamic to handle async bindings
- Add isRemoteDev helper to reduce duplication
- Fail loudly when remote bindings fail (no silent stub fallback)
- Fix proxy.dispose() to actually call the function

Fixes nuxt-hub#741
Fixes nuxt-hub#743
@onmax onmax force-pushed the fix/nitro-cloudflare-dev-bindings branch from 5617264 to 657c0f4 Compare January 16, 2026 13:36
@onmax
Copy link
Copy Markdown
Contributor Author

onmax commented Jan 19, 2026

Superseded by #787 (feature) + #788 (bug fix)

@onmax onmax closed this Jan 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@RihanArfan RihanArfan Awaiting requested review from RihanArfan

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@onmax @atinux @adinvadim