Centralize logger level wrapper registration

[Copilot]

The logger package repeated the same Info/Warn/Error/Debug level-binding boilerplate across the unified, markdown, and server logger variants. This change pulls the per-level closure registration into shared helpers while preserving the existing exported logging APIs.

• What changed

  • Added shared level-registration helpers in internal/logger/common.go for:
    • standard logger signatures
    • server-scoped logger signatures
  • Replaced per-file logInfo/logWarn/logError/logDebug var blocks with a single helper-backed struct per logger type.

• Affected logger APIs

  • Unified logger wrappers now dispatch through fileLevelLoggers
  • Markdown logger wrappers now dispatch through markdownLevelLoggers
  • Server logger wrappers now dispatch through serverLevelLoggers

• Behavioral scope

  • Exported functions remain unchanged:
    • LogInfo/LogWarn/LogError/LogDebug
    • LogInfoToMarkdown/...
    • LogInfoToServer/...
  • The refactor is limited to internal level wiring; call sites and signatures stay stable.

• Test coverage

  • Added focused tests for the shared helper constructors to verify the expected LogLevel bindings.
  • Kept the existing wrapper-coverage test to ensure all registered levels still have public wrappers.

type levelLoggerFuncs struct {
	info  func(category, format string, args ...interface{})
	warn  func(category, format string, args ...interface{})
	error func(category, format string, args ...interface{})
	debug func(category, format string, args ...interface{})
}

var fileLevelLoggers = newLevelLoggerFuncs(logWithLevel)

[Copilot]

Pull request overview

This PR centralizes the logger package’s repeated Info/Warn/Error/Debug closure wiring while preserving the existing exported logging APIs.

Changes:

• Added shared helper structs/constructors for standard and server-scoped level loggers.
• Updated file, markdown, and server logger wrappers to dispatch through centralized helper-backed structs.
• Added focused tests verifying helper constructors bind the expected LogLevel values.

Show a summary per file

File	Description
internal/logger/common.go	Adds centralized level logger helper types and constructors.
internal/logger/file_logger.go	Replaces per-level file logger closures with fileLevelLoggers.
internal/logger/markdown_logger.go	Replaces per-level markdown logger closures with markdownLevelLoggers.
internal/logger/server_file_logger.go	Replaces per-level server logger closures with serverLevelLoggers.
internal/logger/log_level_wrappers_test.go	Adds tests for centralized helper level bindings.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 5/5 changed files
• Comments generated: 0