fix: prevent data loss in BracketWriter on write errors

Move buffer advancement after the successful write to the underlying
writer. Previously, the line was removed from the buffer before the
write, so a failed write (e.g., full disk) would lose the log entry.

Also avoids an unnecessary string allocation by passing the byte
slice directly to fmt.Fprintf.

Author: ilopezluna

pkg/logging/logging.go

@@ -114,13 +114,15 @@ func (bw *BracketWriter) Write(p []byte) (int, error) {
 		if idx < 0 {
 			break
 		}
-		line := string(bw.buf[:idx])
-		bw.buf = append(bw.buf[:0], bw.buf[idx+1:]...)
+		line := bw.buf[:idx]
 
 		ts := time.Now().UTC().Format("2006-01-02T15:04:05.000000000Z")
 		if _, err := fmt.Fprintf(bw.w, "[%s] %s\n", ts, line); err != nil {
 			return n, err
 		}
+		// Advance the buffer only after a successful write to
+		// avoid losing the line on transient I/O errors.
+		bw.buf = append(bw.buf[:0], bw.buf[idx+1:]...)
 	}
 	return n, nil
 }