⚡ Bolt: optimize format reports by eliminating redundant String allocations

[badMade]

💡 What: Replaced String cloning with &str borrowing in agent_detail and render_skills_report when pushing string parts to a Vec for join().

🎯 Why: To reduce unnecessary heap allocations caused by calling .clone() and .to_string() on string components that were just going to be immediately concatenated into a final string.

📊 Impact: Reduces intermediate heap allocations for every agent and skill rendered in CLI reports, speeding up output rendering and lowering memory usage profile during those actions.

🔬 Measurement: Verifiable by the lack of regressions in rendering tests and through system memory profiling or benchmark micro-measurements on formatting large numbers of agents/skills.

---

PR created automatically by Jules for task 10108857645099812862 started by @badMade

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[Copilot]

Pull request overview

Optimizes CLI report formatting by reducing intermediate heap allocations when composing agent and skill detail strings.

Changes:

• Switched agent_detail to build Vec<&str> via .as_str() instead of cloning Strings before join().
• Switched skills report formatting to use borrowed &str parts (including removing to_string() for &'static str origin labels).
• Added a short Jules “bolt” note documenting the optimization pattern.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
rust/crates/commands/src/lib.rs	Eliminates redundant String allocations by joining borrowed &str parts for agent/skill report lines.
.jules/bolt.md	Documents the Vec<&str> + join() pattern to avoid unnecessary cloning in Rust formatting code.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[gemini-code-assist]

Code Review

This pull request introduces a new documentation entry in .jules/bolt.md explaining a Rust string cloning optimization. The optimization involves using Vec<&str> with borrowed string slices instead of Vec<String> with cloned strings to avoid unnecessary heap allocations when constructing strings from parts. This optimization is then applied in the agent_detail and render_skills_report functions within rust/crates/commands/src/lib.rs, replacing clone() and to_string() calls with as_str() or direct &str usage. The review comments suggest further refactoring in both agent_detail and render_skills_report functions to use an iterator-based approach with Option<&str> and flatten() for more concise, idiomatic, and potentially more efficient code.