From aa916b088e7b89403e6d33b64a60fb90a5e447b8 Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Thu, 28 May 2026 21:32:57 +0000 Subject: [PATCH] =?UTF-8?q?=E2=9A=A1=20Bolt:=20[performance=20improvement]?= =?UTF-8?q?=20optimize=20output=20parsing=20in=20python=20generator?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: cloudesize67-cmd <237356855+cloudesize67-cmd@users.noreply.github.com> --- xdk-build/src/python.rs | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/xdk-build/src/python.rs b/xdk-build/src/python.rs index 1d15d6f0..43c606ec 100644 --- a/xdk-build/src/python.rs +++ b/xdk-build/src/python.rs @@ -159,16 +159,16 @@ fn run_formatter(output_dir: &Path, venv_python_path: &Path, script_path: &Path) let stdout = String::from_utf8_lossy(&output.stdout); if !stdout.trim().is_empty() { println!(); - stdout - .split("\n") - .collect::>() - .into_iter() - .for_each(|line| { - let parts = line.split(" ").collect::>(); - if parts.len() == 2 { - log_info!("{} {}", parts[0], parts[1].magenta()); - } - }); + // Bolt performance optimization: + // Avoided creating intermediate `Vec` allocations during string processing + // and used character-based splitting `split('\n')` instead of `split("\n")`. + // Extracted elements using pattern matching directly on `parts.next()`. + stdout.split('\n').for_each(|line| { + let mut parts = line.split(' '); + if let (Some(p0), Some(p1), None) = (parts.next(), parts.next(), parts.next()) { + log_info!("{} {}", p0, p1.magenta()); + } + }); println!(); }