fix(deno-deploy): treat all https:// modules as external

[AaronDewes]

🔗 Linked issue

#1432

❓ Type of change

• 📖 Documentation (updates to the documentation or readme)
• 🐞 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)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Rollup 3.26.2 (accidentally?) made a breaking change in the handling of manualChunks. This updates rollup to a fixed version, and also changes the config so that all https:// modules in Deno are treated external.

📝 Checklist

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

[codecov]

Codecov Report

Merging #1438 (2642f93) into main (e87e2ba) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1438   +/-   ##
=======================================
  Coverage   76.17%   76.17%           
=======================================
  Files          73       73           
  Lines        7437     7437           
  Branches      728      728           
=======================================
  Hits         5665     5665           
  Misses       1771     1771           
  Partials        1        1           

Impacted Files	Coverage Δ
src/presets/deno-deploy.ts	100.00% <100.00%> (ø)

[pi0]

@AaronDewes Can you please confirm if https://github.com/rollup/rollup/releases/tag/v3.26.3 is resolving our issue?

[AaronDewes]

Yes, it is. I'll update this PR to include an update to rollup. I also changed it so Nitro now treats all "https://" imports as external (as you suggested), so this PR is still relevant.

[pi0]

LGTM. Thanks!