fix(deps): update dependency @netlify/plugin-edge-handlers to v3

[renovate]

This PR contains the following updates:

Package	Change	Age	Adoption	Passing	Confidence
@netlify/plugin-edge-handlers	^2.0.0 -> ^3.0.0

---

Release Notes

netlify/netlify-plugin-edge-handlers

v3.0.0

Compare Source

⚠ BREAKING CHANGES

• use pure ES modules (#​689)

Miscellaneous Chores

• use pure ES modules (#​689) (3e23fd4)

---

Configuration

📅 Schedule: At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

---

• If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by WhiteSource Renovate. View repository job log here.

[github-actions]

📊 Benchmark results

Comparing with 95164c7

Package size: 354 MB

⬇️ 0.02% decrease vs. 95164c7

^                          363 MB  363 MB  363 MB  363 MB  363 MB  363 MB  363 MB  363 MB  363 MB  354 MB 
│  354 MB  354 MB  354 MB   ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐  
│   ┌──┐    ┌──┐    ┌──┐    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
└───┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴──>
    T-12    T-11    T-10    T-9     T-8     T-7     T-6     T-5     T-4     T-3     T-2     T-1      T    

Legend

• T-17 (95164c7): 354 MB
• T-16 (ede3514): 354 MB
• T-15 (08f1b50): 354 MB
• T-14 (87ba101): 354 MB
• T-13 (ed1ae44): 354 MB
• T-12 (aeb4d99): 354 MB
• T-11 (6fa8bc8): 354 MB
• T-10 (a920292): 354 MB
• T-9 (f196c10): 363 MB
• T-8 (4ecbf11): 363 MB
• T-7 (569369b): 363 MB
• T-6 (1435482): 363 MB
• T-5 (b4c7bb7): 363 MB
• T-4 (fa829d3): 363 MB
• T-3 (79ca026): 363 MB
• T-2 (1ba0087): 363 MB
• T-1 (f44155b): 363 MB
• T (current commit): 354 MB

[ehmicky]

@NeoLegends This is using the new version of @netlify/plugin-edge-handlers which uses pure ES modules (netlify/netlify-plugin-edge-handlers#689).

Netlify CLI is resolving the absolute file path to that plugin's CLI file.

cli/src/utils/traffic-mesh.js

Line 17 in e227baa

const EDGE_HANDLERS_BUNDLER_CLI_PATH = path.resolve(require.resolve('@netlify/plugin-edge-handlers'), '..', 'cli.js')

Then it passes it to the --bundler argument of @netlify/routing-local-proxy's binary.

cli/src/utils/traffic-mesh.js

Lines 37 to 38 in e227baa

	'--bundler',
	EDGE_HANDLERS_BUNDLER_CLI_PATH,

Note: pure ES modules require Node 12.20.0, 14.14.0, 16.0.0 or higher.

The automated tests seem to pass but could you please confirm whether this should work with @netlify/routing-local-proxy before we merge this PR? Thanks!

[NeoLegends]

Hey @ehmicky thanks for letting me know!

routing-local-proxy is spawning the cli.js by running node <CLI_PATH> build. For this to work node needs to know the CLI is an ES module. I believe it's working fine as it is since the plugin's package.json specifies "type": "module" and that makes nodejs load the file in an ES-compatible way automatically.

I ran a small test to confirm this behavior:

With "type": "module":

Without "type": "module":

Alternatively we could change the file extension from the cli.js to cli.mjs to make sure node picks it up correctly even if it doesn't look at package.json for some reason.

[ehmicky]

Thanks for confirming this @NeoLegends!
The .mjs file extension should not be needed here since the package.json type: 'module' field should always be looked up by Node.js. If we are directly calling this file using node path/to/cli.js, then we should be good 👍

[ehmicky]

CI tests are currently failing, but this is unrelated to this PR, since the main branch is currently failing as well.
Still, we should hold on merging until that test is fixed.

Edit: fixed now.

[kodiakhq]

This PR currently has a merge conflict. Please resolve this and then re-add the automerge label.