search

cloudflare / workers-sdk

⛅️ Home to Wrangler, the CLI for Cloudflare Workers®
https://developers.cloudflare.com/workers/
Apache License 2.0
2.62k stars 692 forks source link

maintenance: wrangler depends on deprecated npm package sourcemap-codec #3612

Open samj opened 1 year ago

samj commented 1 year ago

Which Cloudflare product(s) does this pertain to?

Wrangler

What version of Wrangler are you using?

3.2.0

What operating system are you using?

Windows

Describe the Bug

Install wrangler (npm i wrangler -g) on recent Node/npm (20.4.0/9.7.2).

Observe WARNing:

npm WARN deprecated sourcemap-codec@1.4.8: Please use @jridgewell/sourcemap-codec instead

Also warns about rollup-plugin-inject which was reported in #1232

geoquant commented 1 year ago

thank you for opening 🤘🏼

CyberFlameGO commented 1 year ago

For those interested: image

CyberFlameGO commented 1 year ago

also https://github.com/cloudflare/workers-sdk/discussions/1975

ivanafanasyeu commented 9 months ago

and one more lib, rollup is deprecated as well

danwulff commented 4 months ago

Still happening with wrangler@3.57.0

npm install warning:

npm WARN deprecated sourcemap-codec@1.4.8: Please use @jridgewell/sourcemap-codec instead

npm list results

└─┬ wrangler@3.57.0
  ├─┬ @esbuild-plugins/node-modules-polyfill@0.2.2
  │ └─┬ rollup-plugin-node-polyfills@0.2.1
  │   └─┬ rollup-plugin-inject@3.0.2
  │     ├─┬ magic-string@0.25.9
  │     │ └── sourcemap-codec@1.4.8
danwulff commented 4 months ago

It looks like this might end up fixed in wrangler v4: https://github.com/cloudflare/workers-sdk/issues/1232#issuecomment-2003697274