Refactor logger sink wrappers to share level-based dispatch

[Copilot]

The logger package had parallel LogXxx and LogXxxToMarkdown wrapper families with the same per-level delegation shape, which made sink changes fan out across multiple files. This change keeps the public logging APIs intact while centralizing file/markdown dispatch behind shared sink helpers.

• Shared sink dispatch

  • Added a generic level-sink abstraction in internal/logger/global_state.go.
  • Introduced a single dispatcher that fans one log event out to multiple sinks in order.

• File and markdown wrappers

  • Reworked logWithLevel to dispatch through the shared sink helper instead of owning its own global logger access path.
  • Reworked logWithMarkdown to use the same dispatcher for unified-file + markdown fan-out.

• Behavior preserved

  • Public wrappers such as LogInfo, LogWarn, LogInfoToMarkdown, and LogWarnToMarkdown remain unchanged.
  • Existing file/markdown dual-write behavior continues to flow through the same top-level APIs.

• Focused coverage

  • Added a targeted unit test for the shared sink dispatcher to lock in ordering and multi-sink fan-out behavior.
  • Updated package docs to reflect the centralized dispatch path.

func logWithMarkdown(level LogLevel, category, format string, args ...interface{}) {
	dispatchLevelToSinks(level, category, format, args, fileLevelSink, markdownLevelSink)
}

• Fixes [duplicate-code] Duplicate Code Pattern: Parallel LogXxx / LogXxxToMarkdown Function Families #9679

[Copilot]

Pull request overview

Centralizes file and markdown logger dispatch while preserving public APIs and existing dual-write behavior.

Changes:

• Adds reusable level-based sink adapters and ordered fan-out.
• Refactors file and markdown wrappers to use shared dispatch.
• Adds dispatcher coverage and updates documentation.

Show a summary per file

File	Description
internal/logger/global_state.go	Adds shared sink abstraction and dispatcher.
internal/logger/file_logger.go	Routes file logging through shared dispatch.
internal/logger/markdown_logger.go	Shares file/markdown fan-out logic.
internal/logger/helper_functions_test.go	Tests ordered multi-sink dispatch and nil handling.
internal/logger/doc.go	Documents centralized dispatch.

Review details

Tip

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

• Files reviewed: 5/5 changed files
• Comments generated: 0
• Review effort level: Medium