@actions/http-client - unnecessary dependency upon undici

[Jason3S]

When updating today, I noticed that size of my action bloated by 15k lines because undici was included as a dependency.

undici should have been in devDependencies, but was accidentally included in dependencies.

Originally posted by @Jason3S in #1547 (comment)

  /@actions/core@1.10.1:
    resolution: {integrity: sha512-3lBR9EDAY+iYIpTnTIXmWcNbX3T2kCkAEQGIQx4NVQ0575nk2k3GRZDTPQG+vVtS2izSLmINlxXf0uLtnrTP+g==}
    dependencies:
      '@actions/http-client': 2.2.0
      uuid: 8.3.2
    dev: false

  /@actions/http-client@2.2.0:
    resolution: {integrity: sha512-q+epW0trjVUUHboliPb4UF9g2msf+w61b32tAkFEwL/IwP0DQWgbCMM0Hbe3e3WXSKz5VcUXbzJQgy8Hkra/Lg==}
    dependencies:
      tunnel: 0.0.6
      undici: 5.26.3
    dev: false

[jcesarmobile]

I don't think it can be a dev dependency since it's imported here

toolkit/packages/http-client/src/index.ts

Line 9 in fe3e7ce

import {ProxyAgent} from 'undici'

but the undici addition is breaking an action we have, not sure why

[Jason3S]

@jcesarmobile,

You are right. I'm still not seeing why it needs to be added to @actions/http-client.