Skip to content

Improve logging quality: stack traces, exception context, and resilience#5

Draft
Copilot wants to merge 4 commits intomasterfrom
copilot/improve-error-message-logging
Draft

Improve logging quality: stack traces, exception context, and resilience#5
Copilot wants to merge 4 commits intomasterfrom
copilot/improve-error-message-logging

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 31, 2026 •
edited
Loading

  • Add error(String, Throwable) and warning(String, Throwable) overloads to LoggingHelper
  • Improve error logging call sites with context (player name, file path, cause)
  • Fix exception chaining in UpdateChecker
  • Prevent plugin crash on update check failure (log warning instead)
  • Remove API key from debug log (security)
  • Add 1.3.0 changelog entries for all logging improvements and security fix

@ToothyDev ToothyDev linked an issue Mar 31, 2026 that may be closed by this pull request
Improve error handling and logging #4
Open
Copilot AI and others added 2 commits March 31, 2026 22:27
@ToothyDev
Initial plan for logging improvements
9d4bd43 
Agent-Logs-Url: https://github.com/PixelMindMC/PixelChatGuardian/sessions/8b5fe721-1499-4743-9b3c-220834894807

Co-authored-by: ToothyDev <55001472+ToothyDev@users.noreply.github.com>
@ToothyDev
Improve logging: add Throwable overloads, fix crash on update check, …
dcc2f37 
…improve error context

Agent-Logs-Url: https://github.com/PixelMindMC/PixelChatGuardian/sessions/8b5fe721-1499-4743-9b3c-220834894807

Co-authored-by: ToothyDev <55001472+ToothyDev@users.noreply.github.com>
Copilot AI changed the title [WIP] Improve error message logging with appropriate log levels Improve logging quality: stack traces, exception context, and resilience Mar 31, 2026
Copilot AI requested a review from ToothyDev March 31, 2026 22:33
@ToothyDev
Copy link
Copy Markdown
Member

ToothyDev commented Mar 31, 2026

spigotmc.org was added to the allowlist, do what you want with that power that you previously couldn't @copilot

Copy link
Copy Markdown
Author

Copilot AI commented Mar 31, 2026

spigotmc.org was added to the allowlist, do what you want with that power that you previously couldn't @copilot

Thanks! The build now succeeds cleanly with no warnings — all changes compile fine against the Spigot API (BUILD SUCCESSFUL in 34s, producing PixelChatGuardian-1.2.0.jar).

@ToothyDev ToothyDev added this to the 1.3.0 milestone Mar 31, 2026
@ToothyDev
Copy link
Copy Markdown
Member

ToothyDev commented Mar 31, 2026

@copilot Also add this to the changelog, planned to be part of the upcoming 1.3.0 release

Copy link
Copy Markdown
Author

Copilot AI commented Mar 31, 2026

@copilot Also add this to the changelog, planned to be part of the upcoming 1.3.0 release

Done in 9c1dea4. Added entries to the [1.3.0] section under 🔧 Changed (improved error logging, resilience against update check failures, proper exception chaining) and a new 🔒 Security section (API key no longer logged).

@ToothyDev ToothyDev removed this from the 1.3.0 milestone Mar 31, 2026
@ToothyDev ToothyDev requested a review from Gaming12846 March 31, 2026 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ToothyDev ToothyDev Awaiting requested review from ToothyDev

@Gaming12846 Gaming12846 Awaiting requested review from Gaming12846

At least 1 approving review is required to merge this pull request.

Assignees

@ToothyDev ToothyDev

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Improve error handling and logging

2 participants

@ToothyDev