refactor(cloudflare-kv, cloudflare-r2): move getBindings to utils and add default BUCKET for r2

[oritwoen]

🔗 Linked issue

❓ Type of change

• 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality like performance)
• ✨ New feature (a non-breaking change that adds functionality)
• 🧹 Chore (updates to the build process or auxiliary tools and libraries)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

The getBindings function was used in 2 files. This PR:

• moves the getBindings function to utils
• adds default binding value for R2
• adds additional references in the documentation

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[nuxt-studio]

✅ Live Preview ready!

Name	Edit	Preview	Latest Commit
unstorage	Edit on Studio ↗︎	View Live Preview	16ebe9b

[pi0]

(sorry for delay on review. i will do it asap in next days to release 👍🏼 overall looks nice!)

[oritwoen]

@pi0

I've been thinking about additional changes lately. There will be more HTTP drivers for Cloudflare in the future. It would be good to put general functions and types in one place:

• https://github.com/unjs/unstorage/blob/main/src/drivers/cloudflare-kv-http.ts#L62
• https://github.com/unjs/unstorage/blob/main/src/drivers/cloudflare-kv-http.ts#L95

etc.

Creating a new PR at this point seems "problematic" because an identical file would be created as in this PR. So maybe change the title of this request and include generalized http functions for cloudflare here as well?

[oritwoen]

@pi0 Can you address my comment above?

[codecov]

Codecov Report

Attention: 15 lines in your changes are missing coverage. Please review.

Comparison is base (78e1568) 64.06% compared to head (fa104f2) 64.26%.

Files	Patch %	Lines
src/drivers/utils/cloudflare.ts	63.15%	14 Missing ⚠️
src/drivers/cloudflare-r2-binding.ts	92.30%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #292      +/-   ##
==========================================
+ Coverage   64.06%   64.26%   +0.20%     
==========================================
  Files          41       42       +1     
  Lines        4121     4105      -16     
  Branches      493      490       -3     
==========================================
- Hits         2640     2638       -2     
+ Misses       1469     1455      -14     
  Partials       12       12              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pi0]

Thanks again! I have made few more refactors to make types working.

PR also welcome to prepare types refactoring we might make utils/cloudflare.types.ts to make them more maintainable 💯