deps: replace webextension-polyfill-ts with webextension-polyfill + @types/webextension-polyfill

[legobeat]

This is a follow-up on #37 .

As can be seen in the lockfile diff, this implies a bump of webextension-polyfill from 0.8.0 to 0.10.0.

Keeping webextension-polyfill as a runtime dependency is intended as an intermediary measure to keep this change non-breaking, making it a peerDependency in a future update.

[socket-security]

👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

Next steps

Take a deeper look at the dependency

Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev.

Remove the package

If you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency.

Mark a package as acceptable risk

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of package-name@version specifiers. e.g. @SocketSecurity ignore foo@1.0.0 bar@* or ignore all packages with @SocketSecurity ignore-all

[legobeat]

At the point of this comment, the most recent release 2.0.2 has not been published, blocked by #44 .

The intention here is that 2.0.2 should go out as-is, with this coming in a subsequent release.

[socket-security]

New and updated dependency changes detected. Learn more about Socket for GitHub ↗︎

Packages		Version	New capabilities	Transitives1	Size	Publisher
webextension-polyfill	🆕	0.10.0	None	+0	198 kB	addons-robot
@types/webextension-polyfill	🆕	0.10.0	None	+0	472 kB	types

🚮 Removed packages: webextension-polyfill-ts@0.26.0

Footnotes

1. https://docs.socket.dev ↩

[Gudahtt]

Nit: Could we use a ^ range here instead? Seems more conventional and avoids post-v1 breaking changes in theory.

Suggested change

	"webextension-polyfill": ">=0.10.0"
	},
	"devDependencies": {
	"@metamask/eslint-config": "^4.1.0",
	"@types/webextension-polyfill": ">=0.10.0",
	"webextension-polyfill": "^0.10.0"
	},
	"devDependencies": {
	"@metamask/eslint-config": "^4.1.0",
	"@types/webextension-polyfill": "^0.10.0",

[legobeat]

Updated both to >=0.10.0 <1.0 to keep it sub-1.0 (very reasonable) but open to bringing it down to ^0.10.0 only if still preferred.

[Gudahtt]

This seems OK to me, though I'm not sure why this would be preferable to the more conventional ^0.10.0 range

[Gudahtt]

LGTM, though I did have one suggestion

[mcmire]

Makes sense.