Skip to content

fix(scripts): read config.yaml as UTF-8 in discord-voice-doctor - #8435

Closed
Tianworld wants to merge 1 commit into
NousResearch:mainfrom
Tianworld:fix/discord-voice-doctor-config-encoding
Closed

fix(scripts): read config.yaml as UTF-8 in discord-voice-doctor#8435
Tianworld wants to merge 1 commit into
NousResearch:mainfrom
Tianworld:fix/discord-voice-doctor-config-encoding

Conversation

@Tianworld

Copy link
Copy Markdown
Contributor

Open Hermes config.yaml with explicit UTF-8 so yaml.safe_load does not depend on the Windows code page.

Made with Cursor

Avoid Windows locale default when parsing Hermes config for voice diagnostics.

Made-with: Cursor
@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/cli CLI entry point, hermes_cli/, setup wizard platform/discord Discord bot adapter labels Apr 28, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Likely duplicate of #8453 — same UTF-8 encoding fix for discord-voice-doctor config.yaml read.

@teknium1

Copy link
Copy Markdown
Contributor

Closing as already fixed on main.

Triage notes (high confidence):
scripts/discord-voice-doctor.py:245 on main already opens config.yaml with encoding='utf-8'.

If you still see this on the latest version, please reopen with reproduction steps.

(Bulk-closed during a CLI triage sweep.)

@teknium1 teknium1 closed this May 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have platform/discord Discord bot adapter type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants