Fix sprintf(): Too few arguments when color tokens in format string and colorization disabled

Co-authored-by: swissspidy <841956+swissspidy@users.noreply.github.com>

Author: Copilot

lib/cli/Streams.php

@@ -44,6 +44,8 @@ public static function render( $msg ) {
 			// Colorize the message first so sprintf doesn't bitch at us
 			if ( Colors::shouldColorize() ) {
 				$args[0] = Colors::colorize( $args[0] );
+			} else {
+				$args[0] = Colors::decolorize( $args[0] );
 			}
 
 			// Escape percent characters for sprintf

tests/Test_Cli.php

@@ -553,4 +553,20 @@ function test_safe_strlen() {
 			mb_detect_order( $mb_detect_order );
 		}
 	}
+
+	function test_render_with_color_tokens_and_sprintf_args_colors_disabled() {
+		Colors::disable( true );
+
+		// Color tokens in format string must not cause "sprintf(): Too few arguments".
+		$result = \cli\render( '[%C%k%s%N] Starting!', '2024-01-01 12:00:00' );
+		$this->assertSame( '[2024-01-01 12:00:00] Starting!', $result );
+	}
+
+	function test_render_with_color_tokens_and_sprintf_args_colors_enabled() {
+		Colors::enable( true );
+
+		$result = \cli\render( '[%C%k%s%N] Starting!', '2024-01-01 12:00:00' );
+		$this->assertStringContainsString( '2024-01-01 12:00:00', $result );
+		$this->assertStringContainsString( 'Starting!', $result );
+	}
 }