[FEAT] New web bundle

[roziscoding]

New bundling script generates bundling/bundle folder which can be npm published creating @grammyjs/web.

The package.json for the new package is automatically generated from the current package.json to avoid version mismatches.

Tested with vite, but not with CF workers.

Co-authored-by: Wojciech Pawlik [email protected]

[KnorpelSenf]

1. Can we retain the README section about the CDN? We don't shut it down.
2. Should the Netlify CDN config be changed to redirect to https://bundle.deno.dev?

[codecov-commenter]

Codecov Report

Base: 36.93% // Head: 36.93% // No change to project coverage 👍

Coverage data is based on head (d05dbc0) compared to base (63ac43a).
Patch has no changes to coverable lines.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #318   +/-   ##
=======================================
  Coverage   36.93%   36.93%           
=======================================
  Files          17       17           
  Lines        4364     4364           
  Branches      157      157           
=======================================
  Hits         1612     1612           
  Misses       2750     2750           
  Partials        2        2           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[wojpawlik]

Constantly rebuilding every release is a pain. Let's just not upload new versions to CDN.

[roziscoding]

1. Can we retain the README section about the CDN? We don't shut it down.
2. Should the Netlify CDN config be changed to redirect to https://bundle.deno.dev?

1. Sure. Should we say it's not the preferred way of doing this or something?
2. I think that makes sense, but it requires 1 to explain the new URL format, right? Because we don't have separate bundles anymore

[KnorpelSenf]

1. Can we retain the README section about the CDN? We don't shut it down.
2. Should the Netlify CDN config be changed to redirect to https://bundle.deno.dev?

1. Sure. Should we say it's not the preferred way of doing this or something?
2. I think that makes sense, but it requires 1 to explain the new URL format, right? Because we don't have separate bundles anymore

1. Yes, that'd be great. Explain: Deno, Node, web through Deno, web through Node, web through CDN. I'm not sure how much of this ought to be in the README, we might as well create a new docs page for it. They should probably be mentioned in the README either way.
2. I'd just rewrite all URLs to the equivalent bundling service. Perhaps we need a tiny script for this on Deno Deploy.

Constantly rebuilding every release is a pain. Let's just not upload new versions to CDN.

I'd remove the CDN entirely and just let bundle.deno.dev handle instead what the CDN used to do.

[wojpawlik]

CDN is pointless now. It has the same code as @grammyjs/web, except it doesn't work in CFW.

[KnorpelSenf]

And yet there's no good reason to break people's setups, is there? It is a one-time action to set it up and it will require zero maintenance.

[wojpawlik]

Sure, feel free to set up a redirect. Documenting it would just cause unnecessary confusion tho.

[KnorpelSenf]

I don't like this, mostly because regex on source is very ugly and hard to maintain. The whole point is to have one clean file per platform. If we want to support different platforms that require different platform-specific code, we should create a new file and cp our way through.

[roziscoding]

can we rewrite to prevent top level awaits?

[KnorpelSenf]

If we provide a new file platform.web.ts, we can write it w/o TLA, yes.

[KnorpelSenf]

Wouldn't it be better to explicitly copy over the files we need, rather than copying over everything and then deleting some specific parts of it?

[roziscoding]

It would, but glob patterns are very sketchy and I couldn't get cp to reproduce the existing folder structure but ignore js files

[KnorpelSenf]

rsync?

[roziscoding]

I haven't used it before.
I'll look into it

[KnorpelSenf]

I want to look into this in my holidays

[KnorpelSenf]

… then there is https://deno.land/[email protected]/tools/bundler#bundling-for-the-web:

The output of deno bundle is intended for consumption in Deno and not for use in a web browser or other runtimes. That said, depending on the input it may work in other environments.

If you wish to bundle for the web, we recommend other solutions such as esbuild.

[wojpawlik]

Esbuild wasn't smart enough to omit if (false) when I last tired it. Unlike SWC used by deno bundle.

[KnorpelSenf]

if (false) is a big hack that should be avoided anyway. Do you remember how we went from bash scripts to d2n? I'd like to make a similar move here, and end up using good tools rather than writing custom scripts.

[roziscoding]

Closing in favor of #338 and #339