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]>

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/loader/github/github.go

@@ -41,7 +41,7 @@ func getCommitLsRemote(ctx context.Context, account, repo, ref string) (string,
 }
 
 // regexp to match a git commit id
-var commitRegexp = regexp.MustCompile("^[a-z0-9]{40}$")
+var commitRegexp = regexp.MustCompile("^[a-f0-9]{40}$")
 
 func getCommit(ctx context.Context, account, repo, ref string) (string, error) {
 	if commitRegexp.MatchString(ref) {