fix: do not lowercase all items in tool(like) lists (#344)

* fix: do not lowercase all items in tool(like) lists

When importing things from github or other case sensitive location
the previous functionality would lowercase everything causing
clones from VCS to fail.

This fix removes the calls to lowercase tools and contexts along with
the other places they occur.

Signed-off-by: Bill Maxwell <[email protected]>

* fix: the tool name was not used in the prompt

When the next tool chats, the name is lost when the resp variable is
reinitialized in the for loop. It always appeared to the user they
were chatting with the main entrypoint tool.

This changes it so the previous response is saved, and passed to the
set prompt function. So now if a tool is communicating the user
knows which tool is handling the chat turn.

Signed-off-by: Bill Maxwell <[email protected]>

---------

Signed-off-by: Bill Maxwell <[email protected]>

Author: cloudnautique

pkg/chat/chat.go

@@ -41,6 +41,8 @@ func Start(ctx context.Context, prevState runner.ChatState, chatter Chatter, prg
 	}
 	defer prompter.Close()
 
+	// We will want the tool name to be displayed in the prompt
+	var prevResp runner.ChatResponse
 	for {
 		var (
 			input string
@@ -53,7 +55,7 @@ func Start(ctx context.Context, prevState runner.ChatState, chatter Chatter, prg
 			return err
 		}
 
-		prompter.SetPrompt(getPrompt(prg, resp))
+		prompter.SetPrompt(getPrompt(prg, prevResp))
 
 		if startInput != "" {
 			input = startInput
@@ -79,5 +81,6 @@ func Start(ctx context.Context, prevState runner.ChatState, chatter Chatter, prg
 		}
 
 		prevState = resp.State
+		prevResp = resp
 	}
 }

pkg/parser/parser.go

@@ -101,15 +101,15 @@ func isParam(line string, tool *types.Tool) (_ bool, err error) {
 		}
 		tool.Parameters.Chat = v
 	case "export":
-		tool.Parameters.Export = append(tool.Parameters.Export, csv(strings.ToLower(value))...)
+		tool.Parameters.Export = append(tool.Parameters.Export, csv(value)...)
 	case "tool", "tools":
-		tool.Parameters.Tools = append(tool.Parameters.Tools, csv(strings.ToLower(value))...)
+		tool.Parameters.Tools = append(tool.Parameters.Tools, csv(value)...)
 	case "globaltool", "globaltools":
-		tool.Parameters.GlobalTools = append(tool.Parameters.GlobalTools, csv(strings.ToLower(value))...)
+		tool.Parameters.GlobalTools = append(tool.Parameters.GlobalTools, csv(value)...)
 	case "exportcontext":
-		tool.Parameters.ExportContext = append(tool.Parameters.ExportContext, csv(strings.ToLower(value))...)
+		tool.Parameters.ExportContext = append(tool.Parameters.ExportContext, csv(value)...)
 	case "context":
-		tool.Parameters.Context = append(tool.Parameters.Context, csv(strings.ToLower(value))...)
+		tool.Parameters.Context = append(tool.Parameters.Context, csv(value)...)
 	case "args", "arg", "param", "params", "parameters", "parameter":
 		if err := addArg(value, tool); err != nil {
 			return false, err

pkg/tests/testdata/TestCase/call1.golden

@@ -5,7 +5,7 @@
     {
       "function": {
         "toolID": "testdata/TestCase/test.gpt:6",
-        "name": "bob",
+        "name": "Bob",
         "description": "I'm Bob, a friendly guy.",
         "parameters": {
           "properties": {

pkg/tests/testdata/TestCase2/call1.golden

@@ -5,7 +5,7 @@
     {
       "function": {
         "toolID": "testdata/TestCase2/test.gpt:6",
-        "name": "bob",
+        "name": "Bob",
         "description": "I'm Bob, a friendly guy.",
         "parameters": {
           "properties": {