Unnecessary polyfill is included in client js by using Buffer API

[nissy-dev]

next-themes uses Buffer API for base64 encode when server-side rendering.

next-themes/src/index.tsx

Line 304 in 6f6e0ff

return isServer ? Buffer.from(str).toString('base64') : btoa(str)

However, this line increases a bundle size of client js. The polyfill of Buffer API is not necessary, but _app.js includes it.

I suggest two ways of solving this issue.

1. Stop using base64. I seem the PR v0.2 #89 will solve.
2. Drop supporting Node.js v14 and below. atob/btoa is available in Node.js v16 and later.

@pacocoursey
Will you continue to work about #89 ?

[pacocoursey]

Yes, I plan to move away from the base64 approach. Now that Next.js 12.1.6 is out, we can go back to using an inline script with beforeInteractive.

So this should be resolved with #89.