fix: potential garbling

[SNWCreations]

closes #60

[jpenilla]

What exactly does this fix? Old Minecraft versions not properly specifying UTF-8 for the terminal? It seems like it could have other unintended side-effects, and would mean the development environment is different from a standard one. Unless there is some other context I'm missing.

[SNWCreations]

Specifying the UTF8 as console charset would avoid errored console output on operating systems which does not use UTF8 as its native file encoding (e.g. Windows builds for Simplified Chinese language, as it uses GBK) This fix applys to any version of Minecraft.

[jpenilla]

Is setting this property commonplace in the Chinese community? Maybe we should adjust something in Paper to avoid needing it on newer versions? I am mainly concerned about inconsistency between dev and standard user environment, and the potential to hide bugs or oversights in plugins. Since this applies to more than just the console.

[SNWCreations]

Yes. This is common as far as I know, and I rarely see charset issues from plugins or server as modern plugins uses UTF8 by default in their code and modern editors like VSCode supports autodetect charset, unlike the old Windows notepad and old days (I remember that I was tired to deal with GBK and UTF8 file conversions). So it is safe to use file.encoding imo. However, it would be good to see a better solution to be used in Paper. To make sure this fix applys to older MC, you may need to do some version checking stuff.

…

---Original--- From: "Jason ***@***.***> Date: Sun, Jul 6, 2025 11:46 AM To: ***@***.***>; Cc: ***@***.******@***.***>; Subject: Re: [jpenilla/run-task] fix: potential garbling (PR #61) jpenilla left a comment (jpenilla/run-task#61) Is setting this property commonplace in the Chinese community? Maybe we should adjust something in Paper to avoid needing it on newer versions? I am mainly concerned about inconsistency between dev and standard user environment, and the potential to hide bugs or oversights in plugins. Since this applies to more than just the console. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: ***@***.***>

[jpenilla]

Approved, with a note:

Thanks for the PR. I'm going to merge this — it's a pragmatic fix for developers on non-UTF-8 system locales.

That said, if you think this file.encoding default should be handled more broadly (e.g. at the Paper server level so it also benefits production servers), please open an issue on the Paper repository to discuss it upstream. That would be the right place to fix it for everyone, not just dev environments using run-task.

[SNWCreations]

Approved, with a note:

Thanks for the PR. I'm going to merge this — it's a pragmatic fix for developers on non-UTF-8 system locales.

That said, if you think this file.encoding default should be handled more broadly (e.g. at the Paper server level so it also benefits production servers), please open an issue on the Paper repository to discuss it upstream. That would be the right place to fix it for everyone, not just dev environments using run-task.

I'm surprised to receive comment from this PR because I'm totally forget about this.
Thanks for your suggestion, will open one for it when I'm available and give my think there.