Skip to content

fix(locale): quote cs.yml island-chat-spy syntax to fix malformed YAML#58

Merged
tastybento merged 1 commit into
developfrom
fix/cs-yml-malformed-yaml
Apr 24, 2026
Merged

fix(locale): quote cs.yml island-chat-spy syntax to fix malformed YAML#58
tastybento merged 1 commit into
developfrom
fix/cs-yml-malformed-yaml

Conversation

@tastybento

Copy link
Copy Markdown
Member

Summary

  • The island-chat-spy syntax line in cs.yml was unquoted but contained : (colon + space), which YAML treats as a mapping separator — causing a ScannerException on startup
  • The equivalent team-chat-spy line on the same file was already correctly quoted; this brings island-chat-spy in line with it

Fixes BentoBoxWorld/BentoBox#2953

Test plan

  • Start a server with this cs.yml in place — no YAML parse error on load
  • Set locale to cs, use /teamchatspy and /chatspy — spy messages format correctly

🤖 Generated with Claude Code

The island-chat-spy syntax line contained an unquoted `: ` (colon+space)
which YAML interprets as a mapping separator, causing a parse error on
startup. Quoted the value to match the already-quoted team-chat-spy line.

Fixes BentoBoxWorld/BentoBox#2953

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@tastybento tastybento merged commit 11cfe39 into develop Apr 24, 2026
1 check passed
lukesky19 pushed a commit to lukesky19/BentoBox that referenced this pull request Jun 23, 2026
… error message

YamlConfiguration.loadConfiguration(File) internally catches parse
exceptions and logs them at SEVERE with a full stack trace via
Bukkit.getLogger(), bypassing BentoBox's own try-catch entirely.

Switch to new YamlConfiguration() + load(file) so BentoBox owns the
error handling. Split the catch into InvalidConfigurationException
(bad YAML syntax — tells the admin to delete/fix the file and notes
English fallback) and a generic Exception fallback. Includes the
locale folder name in the message so the admin knows which addon's
file is at fault.

Related: BentoBoxWorld/Chat#58

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Could not update locale file 'locales/cs.yml' due to it being malformed

1 participant