Remove todo

Agus Zubiaga , Ben Brandt , and Richard Feldman created 4 months ago

Co-authored-by: Ben Brandt <[e-mail]>
Co-authored-by: Richard Feldman <[e-mail]>

Change summary

crates/acp_thread/src/acp_thread.rs    | 3 ++-
crates/agent_ui/src/acp/thread_view.rs | 1 -
2 files changed, 2 insertions(+), 2 deletions(-)

Detailed changes

crates/acp_thread/src/acp_thread.rs

@@ -734,7 +734,8 @@ impl AcpThread {
     }
 
     fn tool_call_mut(&mut self, id: &acp::ToolCallId) -> Option<(usize, &mut ToolCall)> {
-        // todo! use map
+        // The tool call we are looking for is typically the last one, or very close to the end.
+        // At the moment, it doesn't seem like a hashmap would be a good fit for this use case.
         self.entries
             .iter_mut()
             .enumerate()

crates/agent_ui/src/acp/thread_view.rs

@@ -1064,7 +1064,6 @@ impl AcpThreadView {
                                                 )
                                                 .into_any_element()
                                         }))
-                                        // todo! confirm this looks ok
                                         .child(self.render_permission_buttons(
                                             options,
                                             entry_ix,