crates/vm/src/function/argument.rs (1)

452-467: Add the mutable by-reference iterator variant for API completeness.

The newtype preserves owned iteration and for ... in &kwargs, but not the previous IndexMap pattern for ... in &mut kwargs. While DerefMut provides mutable access to the underlying map, it does not automatically enable IntoIterator desugaring for &mut KwArgs<T>. Adding IntoIterator for &mut KwArgs<T> restores compatibility with the prior iteration pattern.

♻️ Proposed iterator completion

 impl<'a, T> IntoIterator for &'a KwArgs<T> {
     type Item = (&'a String, &'a T);
     type IntoIter = indexmap::map::Iter<'a, String, T>;

     fn into_iter(self) -> Self::IntoIter {
         self.0.iter()
     }
 }

+impl<'a, T> IntoIterator for &'a mut KwArgs<T> {
+    type Item = (&'a String, &'a mut T);
+    type IntoIter = indexmap::map::IterMut<'a, String, T>;
+
+    fn into_iter(self) -> Self::IntoIter {
+        self.0.iter_mut()
+    }
+}
+
 impl<T> IntoIterator for KwArgs<T> {
     type Item = (String, T);
     type IntoIter = indexmap::map::IntoIter<String, T>;

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@crates/vm/src/function/argument.rs` around lines 452 - 467, The KwArgs type
currently has IntoIterator implementations for immutable references and owned
values, but is missing the mutable by-reference variant. Add a third
IntoIterator implementation for &'a mut KwArgs<T> that returns an iterator over
mutable references to the key-value pairs from the underlying IndexMap. This
should follow the same pattern as the existing implementations, delegating to
self.0.iter_mut() to provide iteration over mutable references while maintaining
API compatibility with the previous IndexMap iteration pattern.