Bogdan Chadkin

Results 362 comments of Bogdan Chadkin

Looks like kvg namespace is defined in unusual way not supported by sax parser we use.

Missing these object-oriented features is one of the reason people choose flow over typescript.

Why even ts used to deploy node apps? Like many libraries do own bundling and use `@vercel/node` as client though they still load 70MB of typescript package ts support could...

What is the point of ignoring named exports. Can you just not export those classes?

This is default rollup output when you do not have named exports. With named exports it's not clear how interop should be handled. And look with named only exports you...

@STRML This was done because everybody asked to not check whole node_modules, but only direct and transitive dependencies. I think both alias and root resolve are bad practices because every...

This is because vite-plugin-cloudflare hardcodes export conditions and is using "workerd" instead of "cloudflare" https://github.com/cloudflare/workers-sdk/blob/main/packages/vite-plugin-cloudflare/src/cloudflare-environment.ts#L128

Reported https://github.com/cloudflare/workers-sdk/issues/9668

I fixed by patching pg-cloudflare like this ```diff --- a/package.json +++ b/package.json @@ -15,6 +15,10 @@ "import": "./esm/index.mjs", "require": "./dist/index.js" }, + "workerd": { + "import": "./esm/index.mjs", + "require": "./dist/index.js"...

Using [patch-package](https://github.com/ds300/patch-package) for npm Pnpm has [builtin command](https://pnpm.io/cli/patch) for this