Skip to content

[WIP] Transform hardcoded logging into configurable multi-language system#60

Closed
Copilot wants to merge 1 commit into
mainfrom
copilot/transform-logging-system
Closed

[WIP] Transform hardcoded logging into configurable multi-language system#60
Copilot wants to merge 1 commit into
mainfrom
copilot/transform-logging-system

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Oct 15, 2025

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Original prompt

Transform the current hardcoded logging system into a fully configurable, multi-language system with customizable message templates and Discord webhook integration.

Current System Analysis:
SpawnerEventType enum contains hardcoded descriptions like "Spawner placed", "Spawner broken", etc.
SpawnerLogEntry uses eventType.getDescription() for logging messages
Discord logging appears to be missing or basic
No internationalization support currently exists
Required Implementation:
Create Language System Structure
Existing language structure: language/{locale}/messages.yml, gui.yml, formatting.yml, items.yml
Add to existing messages.yml in each language folder:

# Spawner Logging Messages
logging:
  events:
    SPAWNER_PLACE:
...
  # Discord embed configurations
  discord_embeds:
    SPAWNER_PLACE:
      title:

Update main config.yml to:

logging:
  discord:
    enabled: true
    webhook_url: "your_webhook_url_here"
    events:
      - SPAWNER_PLACE
      - SPAWNER_BREAK
...

make the default message


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI requested a review from ptthanh02 October 15, 2025 01:51
@ptthanh02 ptthanh02 closed this Oct 15, 2025
@ptthanh02 ptthanh02 deleted the copilot/transform-logging-system branch October 15, 2025 02:19
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.

2 participants