Switch from webextension-polyfill-ts to @types/webextension-polyfill

[twschiller]

[email protected]: This project has moved to @types/webextension-polyfill

[fregante]

My bad 😃 fregante/Awesome-WebExtensions#61 (comment)

By the way I suggest making the browser variable global while we're at it, it also simplifies debugging because it lets us run promisified commands in the dev tools.

[twschiller]

Haha. How would you make browser global, via webpack provides? I suspect depending on how you make it global it might break something (e.g., tests, or maybe something else) because the underlying Mozilla library does a context check

[fregante]

How would you make browser global

By loading the polyfill on its own like you would without a build:

https://github.com/sindresorhus/refined-github/blob/99e81baaac8c787d1a59b74194c9437e9c29ffdf/distribution/manifest.json#L38

and have webpack copy the raw file over:

https://github.com/sindresorhus/refined-github/blob/99e81baaac8c787d1a59b74194c9437e9c29ffdf/webpack.config.ts#L54-L58

We'd add the import exactly like we're doing with vendor.js

via webpack provides

Unfortunately not fully supported: mozilla/webextension-polyfill#156

But at the end of that thread there's an alternative solution, I just never used it though