Trim leading whitespace in messages sent in agent panel

[JosephTLyons]

Summary

We currently trim trailing whitespace, but do not trim leading whitespace.

Zed: v0.211.0 (Zed Nightly 05c2cc02542cdfa55787923f6411e49d9c4943c3) 
OS: macOS 26.0
Memory: 64 GiB
Architecture: aarch64

[babalolajnr]

Can I take this on?

[JosephTLyons]

Yeah! I'll assign you! Im not working on the agent, so I'm not sure where the best place to put it is, but I'm assuming we just want to add a trim before it ever gets sent to off, as soon as possible. Mostly just finding where the existing trimming is happening for trailing whitespace and adjusting there.

[JosephTLyons]

Weirdly enough, it seems like we might already be doing that, but I might have the wrong code here, just did a quick 15-second search.

zed/crates/agent_ui/src/acp/thread_view.rs

Line 1053 in 55bc679

let text = text.trim();

[zeld-a]

I see now that this was assigned right after I started working on it. If @babalolajnr does end up making a pr, please take his instead.

[babalolajnr]

Weirdly enough, it seems like we might already be doing that, but I might have the wrong code here, just did a quick 15-second search.

zed/crates/agent_ui/src/acp/thread_view.rs

Line 1053 in 55bc679
let text = text.trim();

I am seeing this too. Any idea on how to confirm if this is actually working?

[seanstrom]

@babalolajnr I took a look at this code here:

zed/crates/agent_ui/src/acp/thread_view.rs

Line 1053 in 55bc679

let text = text.trim();

It seems that code is only trimming the content to check for some /login or /logout commands, but it doesn't seem to affect the message content sent to the agent.

---

However, I poked around and I think this is where we could possibly trim the text:

zed/crates/agent_ui/src/acp/message_editor.rs

Lines 704 to 708 in 45983e1

	let result = editor.update(cx, |editor, cx| {
	let mut ix = 0;
	let mut chunks: Vec<acp::ContentBlock> = Vec::new();
	let text = editor.text(cx);
	editor.display_map.update(cx, |map, cx| {

In this code we're processing the text content before sending it to the agent, so it seems like we can possibly change the code to something like:

  let editor_text = editor.text(cx);
  let text = editor_text.trim().to_string();

Though I'm not sure if that's the best way to handle this kind of change, so feel free to choose what's best!

[babalolajnr]

Thank you very much @seanstrom Your suggestion worked. I created a pull request here #41203.