[Repo Assist] refactor(logger): eliminate withLock duplication across 4 logger types

internal/logger/file_logger.go

@@ -59,9 +59,7 @@ func InitFileLogger(logDir, fileName string) error {
 // withLock acquires fl.mu, executes fn, then releases fl.mu.
 // Use this in methods that return an error to avoid repeating the lock/unlock preamble.
 func (fl *FileLogger) withLock(fn func() error) error {
-	fl.mu.Lock()
-	defer fl.mu.Unlock()
-	return fn()
+	return withMutexLock(&fl.mu, fn)
 }
 
 // Close closes the log file

internal/logger/global_helpers.go

@@ -16,6 +16,15 @@ package logger
 
 import "sync"
 
+// withMutexLock acquires mu, calls fn, and releases mu.
+// It is a package-level helper to avoid repeating the lock/unlock preamble in
+// per-type withLock methods.
+func withMutexLock(mu *sync.Mutex, fn func() error) error {
+	mu.Lock()
+	defer mu.Unlock()
+	return fn()
+}
+
 // closableLogger is a constraint for types that have a Close method.
 // This is satisfied by *FileLogger, *JSONLLogger, *MarkdownLogger, *ServerFileLogger, and *ToolsLogger.
 type closableLogger interface {

internal/logger/jsonl_logger.go

@@ -70,9 +70,7 @@ func InitJSONLLogger(logDir, fileName string) error {
 // withLock acquires jl.mu, executes fn, then releases jl.mu.
 // Use this in methods that return an error to avoid repeating the lock/unlock preamble.
 func (jl *JSONLLogger) withLock(fn func() error) error {
-	jl.mu.Lock()
-	defer jl.mu.Unlock()
-	return fn()
+	return withMutexLock(&jl.mu, fn)
 }
 
 // Close closes the JSONL log file

internal/logger/markdown_logger.go

@@ -71,9 +71,7 @@ func (ml *MarkdownLogger) initializeFile() error {
 // withLock acquires ml.mu, executes fn, then releases ml.mu.
 // Use this in methods that return an error to avoid repeating the lock/unlock preamble.
 func (ml *MarkdownLogger) withLock(fn func() error) error {
-	ml.mu.Lock()
-	defer ml.mu.Unlock()
-	return fn()
+	return withMutexLock(&ml.mu, fn)
 }
 
 // Close closes the log file and writes the closing details tag

internal/logger/tools_logger.go

@@ -84,9 +84,7 @@ func InitToolsLogger(logDir, fileName string) error {
 // withLock acquires tl.mu, executes fn, then releases tl.mu.
 // Use this in methods that return an error to avoid repeating the lock/unlock preamble.
 func (tl *ToolsLogger) withLock(fn func() error) error {
-	tl.mu.Lock()
-	defer tl.mu.Unlock()
-	return fn()
+	return withMutexLock(&tl.mu, fn)
 }
 
 // LogTools logs the tools for a specific server