Skip to content

Cleanup and reorganize logging settings - #50

Open
rknall wants to merge 6 commits into
adly98:mainfrom
rknall:feat/cleanup-log-settings
Open

Cleanup and reorganize logging settings #50
rknall wants to merge 6 commits into
adly98:mainfrom
rknall:feat/cleanup-log-settings

Conversation

@rknall

@rknall rknall commented Dec 6, 2025

Copy link
Copy Markdown
Contributor

Summary

  • Move file logging settings from Debug section to Auto.Logging for better organization
  • Rename debug logging flags to use shorter, consistent names (e.g., logAdventures → Adventures)
  • Enable geologists and explorers debug logging by default
  • Add extensive debug logging throughout the adventure queue system

Changes

  • Reorganize settings structure:
    • File logging (logToFile, maxLogFileSize, keepRotatedLogs) now under
      aSettings.defaults.Auto.Logging
    • Debug flags renamed: enableLogging → enable, logAdventures → Adventures, etc.
  • Add debug logging to adventure queue operations:
    • Step transitions and zone changes
    • Buff applications and quest completions
    • General movements and combat actions
    • Battle finished events with casualty tracking
  • Update UI to reflect new settings structure
  • Enable Geologists and Explorers debug logging by default

- Add global lz() function for zero-padding numbers
- Implement console file logger that writes to auto/logs/console.log
- Wrap console.log/error/warn/info to write to file
- Add log rotation when file exceeds maxLogFileSize
- Add session markers for easy session identification
- Add UI settings for file logging configuration
- Reset logger flag on Tools->Update for proper re-initialization
- Include path validation and directory creation
@rknall

rknall commented Dec 6, 2025

Copy link
Copy Markdown
Contributor Author

@adly98 could you please merge that, as it is the basis for the remaining issues

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.

1 participant