From 1f4d35f8a34f1793e76e33cb7f2e80c0d8f021e0 Mon Sep 17 00:00:00 2001
From: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Date: Fri, 13 Dec 2024 11:21:35 +0000
Subject: [PATCH] fix: properly handle object type parameters in tools

- Add special handling for object type parameters
- Parse JSON input for object parameters
- Maintain raw input if JSON parsing fails
- Fixes #110

Co-Authored-By: ashwin@anthropic.com <ashwin@anthropic.com>
---
 client/src/components/ToolsTab.tsx | 53 ++++++++++++++++++++++++------
 1 file changed, 43 insertions(+), 10 deletions(-)

diff --git a/client/src/components/ToolsTab.tsx b/client/src/components/ToolsTab.tsx
index 085f76a..43b0668 100644
--- a/client/src/components/ToolsTab.tsx
+++ b/client/src/components/ToolsTab.tsx
@@ -159,13 +159,15 @@ const ToolsTab = ({
                       {key}
                     </Label>
                     {
-                      /* @ts-expect-error value type is currently unknown */
+                      // @ts-expect-error Tool schema types are not fully typed
                       value.type === "string" ? (
                         <Textarea
                           id={key}
                           name={key}
-                          // @ts-expect-error value type is currently unknown
-                          placeholder={value.description}
+                          placeholder={
+                            // @ts-expect-error Tool schema types are not fully typed
+                            value.description
+                          }
                           onChange={(e) =>
                             setParams({
                               ...params,
@@ -174,19 +176,50 @@ const ToolsTab = ({
                           }
                           className="mt-1"
                         />
-                      ) : (
-                        <Input
-                          // @ts-expect-error value type is currently unknown
-                          type={value.type === "number" ? "number" : "text"}
+                      ) :
+                      // @ts-expect-error Tool schema types are not fully typed
+                      value.type === "object" ? (
+                        <Textarea
                           id={key}
                           name={key}
-                          // @ts-expect-error value type is currently unknown
-                          placeholder={value.description}
+                          placeholder={
+                            // @ts-expect-error Tool schema types are not fully typed
+                            value.description
+                          }
+                          onChange={(e) => {
+                            try {
+                              const parsed = JSON.parse(e.target.value);
+                              setParams({
+                                ...params,
+                                [key]: parsed,
+                              });
+                            } catch (err) {
+                              // If invalid JSON, store as string - will be validated on submit
+                              setParams({
+                                ...params,
+                                [key]: e.target.value,
+                              });
+                            }
+                          }}
+                          className="mt-1"
+                        />
+                      ) : (
+                        <Input
+                          type={
+                            // @ts-expect-error Tool schema types are not fully typed
+                            value.type === "number" ? "number" : "text"
+                          }
+                          id={key}
+                          name={key}
+                          placeholder={
+                            // @ts-expect-error Tool schema types are not fully typed
+                            value.description
+                          }
                           onChange={(e) =>
                             setParams({
                               ...params,
                               [key]:
-                                // @ts-expect-error value type is currently unknown
+                                // @ts-expect-error Tool schema types are not fully typed
                                 value.type === "number"
                                   ? Number(e.target.value)
                                   : e.target.value,