Why use window instead of globalThis as the global object? Is it because events can be listened on window?

We already use patch-package on the polyfill, can you please apply the patch there instead?

Why use window instead of globalThis as the global object? Is it because events can be listened on window?

Out of habit, but yeah this should be globalThis.

We already use patch-package on the polyfill, can you please apply the patch there instead?

done

The behavior of patch-package is a bit strange, I may need to investigate further, and it seems that after installing patch, the compilation result of make js && make does not change, which is a bit strange

Yeah when the polyfills change the Makefile doesn't quite pick it up because the dependencies across files are not detected.

You can force it by deleting the .c file.

What's strange is that this patch doesn't seem to modify the source code, so why is it added here?
Is it because the editor automatically formats it?

@@ -639,4 +640,4 @@ function UTF8Encoder(options) {
 }
 
 exports.TextEncoder = TextEncoder;
-exports.TextDecoder = TextDecoder;
\ No newline at end of file
+exports.TextDecoder = TextDecoder;

I only added an alias utf8, which should cover most usage scenarios.

Thank you!