Skip to content

File logging#314

Merged
mkholt merged 4 commits intomasterfrom
file-logging
Mar 4, 2026
Merged

File logging#314
mkholt merged 4 commits intomasterfrom
file-logging

Conversation

@mkholt
Copy link
Member

@mkholt mkholt commented Mar 4, 2026

This pull request introduces a new logging system to XrmMockup, enabling logging of the startup sequence and Plugin/API detection to a file. The logging infrastructure is integrated throughout the core initialization and plugin/custom API management, providing better visibility into system operations and diagnostics. The changes include the addition of a file-based logger, updates to the core and manager classes to accept logger instances, and the exposure of relevant logging settings in the configuration.

Logging System Integration:

  • Added a file-based logging system (FileLogger and FileLoggerFactory) that writes logs to a specified file, with support for configurable log levels. [1] [2]
  • Updated CoreInitializationData and core initialization logic to accept and propagate an ILoggerFactory, resolving it based on settings (using a file logger if a file path is provided). [1] [2] [3] [4] [5]
  • Core initialization now logs timing and summary information about plugin, workflow, and custom API registration. [1] [2] [3]

Plugin and Custom API Manager Enhancements:

  • PluginManager and CustomApiManager now accept and use ILogger instances, logging detailed information about registration, cache usage, and missing registrations. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]
  • Logging is added throughout the custom API scanning and registration process to provide insight into which APIs are registered, loaded from cache, or missing registration. [1] [2] [3] [4] [5]

Documentation:

  • Updated RELEASE_NOTES.md to announce the new logging feature.

mkholt added 4 commits March 4, 2026 08:24
- Add MinLogLevel to XrmMockupSettings for file logger filtering
- FileLogger now respects minimum log level and improves flushing
- Add detailed diagnostic logging to plugin/custom API discovery and registration
- Inject ILogger into registration strategies for better traceability
- Update unit tests to verify log level filtering and debug output
- Improve documentation for new logging options and behaviors
@mkholt mkholt merged commit faec06d into master Mar 4, 2026
2 checks passed
@mkholt mkholt deleted the file-logging branch March 4, 2026 08:15
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