Fix: M3 lazy tool input review feedback

[ashleeradka]

Summary

Addresses review feedback on #9229. Three fixes:

1. Cap inputRawDict at storage point — In populateFromHistory, the raw tool input dictionary is now size-checked before storage. If the JSON-encoded size exceeds 10KB, the input is formatted eagerly (with the existing truncation logic in formatAllToolInput) and stored in inputFull instead of retaining the unbounded raw dictionary in memory.

2. Cache resolvedInputFull with @State — Both ToolCallChip and UsedToolsRow now cache the formatted input in a @State property that is populated once via .onAppear when the user first expands the tool call. This prevents formatAllToolInput from re-running on every SwiftUI render pass while expanded.

3. Remove dead ensureInputFullFormatted() — The mutating method on ToolCallData was never called anywhere in the codebase. The @State caching approach is simpler for SwiftUI views, so this dead code was removed.

Files Changed

• clients/shared/Features/Chat/ChatViewModel.swift — Size-check raw dict before storing
• clients/shared/Features/Chat/ToolCallChip.swift — Add @State cache + .onAppear
• clients/macos/vellum-assistant/Features/Chat/UsedToolsList.swift — Add @State cache + .onAppear
• clients/shared/Features/Chat/ChatMessage.swift — Remove dead ensureInputFullFormatted()

🤖 Generated with Claude Code

---

[devin-ai-integration]

Devin Review found 1 potential issue.

View 4 additional findings in Devin Review.

[ashleeradka]

Addressed in #9325