Merge pull request #98 from cagostino/chris/calc_tool_bug

Chris/calc tool bug

Author: cagostino

npcsh/llm_funcs.py

@@ -1051,14 +1051,14 @@ def handle_tool_call(
     for inp in required_inputs:
         if not isinstance(inp, dict):
             # dicts contain the keywords so its fine if theyre missing from the inputs.
-            if inp not in input_values:
+            if inp not in input_values or input_values[inp] == "":
                 missing_inputs.append(inp)
     if len(missing_inputs) > 0:
         # print(f"Missing required inputs for tool '{tool_name}': {missing_inputs}")
         if attempt < n_attempts:
             print(f"attempt {attempt+1} to generate inputs failed, trying again")
             print("missing inputs", missing_inputs)
-            print("llm response", response)
+            #print("llm response", response)
             print("input values", input_values)
             return handle_tool_call(
                 command,
@@ -1080,6 +1080,7 @@ def handle_tool_call(
         }
 
     # try:
+    print("Executing tool with input values:", input_values)
     tool_output = tool.execute(
         input_values,
         npc.all_tools_dict,

npcsh/npc_compiler.py

@@ -222,6 +222,11 @@ def execute_step(
                         context["results"] = exec_env["output"]
             except NameError as e:
                 print(f"NameError: {e} , on the following tool code: ", rendered_code)
+            except SyntaxError as e:
+                print(f"SyntaxError: {e} , on the following tool code: ", rendered_code)
+            except Exception as e:
+                print(f"Error executing Python code: {e}")
+
         return context
 
     def to_dict(self):

setup.py

@@ -57,7 +57,7 @@ def get_setup_message():
 
 setup(
     name="npcsh",
-    version="0.3.8",
+    version="0.3.9",
     packages=find_packages(exclude=["tests*"]),
     install_requires=[
         "redis",