request: cloudflare worker support

[mandar1jn]

request: cloudflare worker support

[favna]

I'm not sure where you draw the conclusion from that node_compat is discouraged? The docs don't mention any such thing: https://developers.cloudflare.com/workers/runtime-apis/nodejs

[SuperchupuDev]

@favna the wrangler CLI mentions that node_compat has serious tradeoffs

also the plugin it uses is unmaintained

• ionic-team/rollup-plugin-node-polyfills#28

[favna]

Regarding Wrangler using code from ionic, you'll have to take that up with Cloudflare. As for the usage of node:util in this lib, we are very much aware of the issue, but at the same time personally, I have no idea how we can replace it. Possibly @kyranet who designed that part of the code can toss in some ideas. Alternatively, I'm not entirely sure why using @imranbarbhuiya's esbuild-plugins-node-modules-polyfill does not replace var util = require('util'); / import { inspect } from 'util'; from the CJS and ESM builds respectively when the plugin is activated for those bundles. That might be a way to achieve wrangler support as well.

That all said, I'm not sure if just replacing util is enough, because we also depend on:

import get from 'lodash/get.js';
import fastDeepEqual from 'fast-deep-equal/es6/index.js';
import uniqWith from 'lodash/uniqWith.js';

And outside of bundling all the code into the shapeshift bundle I don't see any way to replace those.

[imranbarbhuiya]

wrangler is also planning to use this plugin, maybe it'll solve the issue. see cloudflare/workers-sdk#3832 but they want to ship it as a breaking change so I am not sure if it'll happen anytime soon

[imranbarbhuiya]

Alternatively, I'm not entirely sure why using @imranbarbhuiya's esbuild-plugins-node-modules-polyfill does not replace var util = require('util'); / import { inspect } from 'util'; from the CJS and ESM builds respectively when the plugin is activated for those bundles.

We only enabled it for browser environments as node already have it. We can still make it to bundle for node env too