Add buffer handler to reduce database write operations for performance.

Introduces Monolog's BufferHandler for database logging and adds the 'wpgraphql_logging_default_buffer_limit' filter to customize buffer size. Updates tests to flush buffer before asserting log counts and documents the new filter. Also adds an index for 'level_name' in the database entity.

Author: colinmurphy

plugins/wpgraphql-logging/docs/reference/logging.md

@@ -39,6 +39,27 @@ add_filter( 'wpgraphql_logging_default_processors', function( array $processors
 });
 ```
 
+#### Filter: `wpgraphql_logging_default_buffer_limit`
+Filters the default buffer limit for the BufferHandler.
+
+Parameters:
+- `$buffer_limit` (int) Current buffer limit (default: 50)
+
+Returns: int
+
+Example:
+```php
+
+add_filter( 'wpgraphql_logging_default_buffer_limit', function( int $buffer_limit ) {
+    // Increase buffer limit for high-traffic sites
+    return 100;
+});
+
+
+```
+
+
+
 #### Filter: `wpgraphql_logging_default_handlers`
 Filters the default handler list.
 

plugins/wpgraphql-logging/src/Logger/Database/DatabaseEntity.php

@@ -366,6 +366,7 @@ public static function get_schema(): string {
 		  datetime DATETIME NOT NULL,
 		  PRIMARY KEY  (id),
 		  INDEX channel_index (channel),
+		  INDEX level_name_index (level_name)
 		  INDEX level_index (level),
 		  INDEX datetime_index (datetime)
 	   ) {$charset_collate};

plugins/wpgraphql-logging/src/Logger/LoggerService.php

@@ -4,6 +4,7 @@
 
 namespace WPGraphQL\Logging\Logger;
 
+use Monolog\Handler\BufferHandler;
 use Monolog\Handler\HandlerInterface;
 use Monolog\Logger;
 use Monolog\Processor\MemoryPeakUsageProcessor;
@@ -212,6 +213,15 @@ public function log( $level, string $message, array $context = [] ): void {
 		$this->monolog->log( $level, $message, array_merge( $this->default_context, $context ) );
 	}
 
+	/**
+	 * Gets the Monolog logger instance.
+	 *
+	 * @return \Monolog\Logger The Monolog logger instance.
+	 */
+	public function get_monolog(): Logger {
+		return $this->monolog;
+	}
+
 	/**
 	 * Returns an array of default processors.
 	 *
@@ -241,8 +251,11 @@ public static function get_default_processors(): array {
 	 * @return array<\Monolog\Handler\AbstractProcessingHandler>
 	 */
 	public static function get_default_handlers(): array {
+
+		$buffer_limit     = apply_filters( 'wpgraphql_logging_default_buffer_limit', 50 );
+		$database_handler = new BufferHandler( new WordPressDatabaseHandler(), $buffer_limit );
 		$default_handlers = [
-			new WordPressDatabaseHandler(),
+			$database_handler,
 		];
 
 		// Filter for users to add their own handlers.

plugins/wpgraphql-logging/tests/wpunit/Events/QueryActionLoggerTest.php

@@ -5,6 +5,7 @@
 namespace WPGraphQL\Logging\Tests\Events;
 
 use WPGraphQL\Logging\Plugin;
+use Monolog\Handler\BufferHandler;
 use lucatume\WPBrowser\TestCase\WPTestCase;
 use WPGraphQL\Logging\Events\QueryActionLogger;
 use WPGraphQL\Logging\Events\Events;
@@ -53,6 +54,15 @@ public function get_log_count(): int {
 	}
 
 	public function assert_log_count(int $expected_count): void {
+
+		// Flush the buffer handler to ensure the log count is accurate.
+		$handlers = $this->logger->get_monolog()->getHandlers();
+		foreach ($handlers as $handler) {
+			if ($handler instanceof BufferHandler) {
+				$handler->flush();
+			}
+		}
+
 		$actual_count = $this->get_log_count();
 		$this->assertEquals($expected_count, $actual_count, "Expected log count to be {$expected_count}, but got {$actual_count}.");
 	}

plugins/wpgraphql-logging/tests/wpunit/Events/QueryFilterLoggerTest.php

@@ -6,6 +6,7 @@
 
 
 use WPGraphQL\Logging\Plugin;
+use Monolog\Handler\BufferHandler;
 use lucatume\WPBrowser\TestCase\WPTestCase;
 use WPGraphQL\Logging\Events\QueryFilterLogger;
 use WPGraphQL\Logging\Events\Events;
@@ -55,6 +56,15 @@ public function get_log_count(): int {
 	}
 
 	public function assert_log_count(int $expected_count): void {
+
+		// Flush the buffer handler to ensure the log count is accurate.
+		$handlers = $this->logger->get_monolog()->getHandlers();
+		foreach ($handlers as $handler) {
+			if ($handler instanceof BufferHandler) {
+				$handler->flush();
+			}
+		}
+
 		$actual_count = $this->get_log_count();
 		$this->assertEquals($expected_count, $actual_count, "Expected log count to be {$expected_count}, but got {$actual_count}.");
 	}