Merge pull request #333 from ibuildthecloud/seq

ibuildthecloud · web-flow · commit ae4f020af996 · 2024-05-08T20:52:02.000-07:00
chore: change message of finish tool
diff --git a/pkg/builtin/builtin.go b/pkg/builtin/builtin.go
@@ -114,7 +114,7 @@ var tools = map[string]types.Tool{
 		Parameters: types.Parameters{
 			Description: "Concludes the conversation. This can not be used to ask a question.",
 			Arguments: types.ObjectSchema(
-				"summary", "A summary of the dialog",
+				"return", "The instructed value to return or a summary of the dialog if no value is instructed",
 			),
 		},
 		BuiltinFunc: SysChatFinish,
@@ -640,7 +640,7 @@ func (e *ErrChatFinish) Error() string {
 
 func SysChatFinish(ctx context.Context, env []string, input string) (string, error) {
 	var params struct {
-		Message string `json:"summary,omitempty"`
+		Message string `json:"return,omitempty"`
 	}
 	if err := json.Unmarshal([]byte(input), &params); err != nil {
 		return "", &ErrChatFinish{
diff --git a/pkg/tests/runner_test.go b/pkg/tests/runner_test.go
@@ -64,7 +64,7 @@ func TestDualSubChat(t *testing.T) {
 	r.RespondWith(tester.Result{
 		Func: types.CompletionFunctionCall{
 			Name:      types.ToolNormalizer("sys.chat.finish"),
-			Arguments: `{"summary":"Chat done"}`,
+			Arguments: `{"return":"Chat done"}`,
 		},
 	})
 
@@ -89,7 +89,7 @@ func TestDualSubChat(t *testing.T) {
 	r.RespondWith(tester.Result{
 		Func: types.CompletionFunctionCall{
 			Name:      types.ToolNormalizer("sys.chat.finish"),
-			Arguments: `{"summary":"Chat done2"}`,
+			Arguments: `{"return":"Chat done2"}`,
 		},
 	}, tester.Result{
 		Text: "And we're done",
diff --git a/pkg/tests/testdata/TestContextSubChat/call2.golden b/pkg/tests/testdata/TestContextSubChat/call2.golden
@@ -9,8 +9,8 @@
         "description": "Concludes the conversation. This can not be used to ask a question.",
         "parameters": {
           "properties": {
-            "summary": {
-              "description": "A summary of the dialog",
+            "return": {
+              "description": "The instructed value to return or a summary of the dialog if no value is instructed",
               "type": "string"
             }
           },
diff --git a/pkg/tests/testdata/TestContextSubChat/call3.golden b/pkg/tests/testdata/TestContextSubChat/call3.golden
@@ -9,8 +9,8 @@
         "description": "Concludes the conversation. This can not be used to ask a question.",
         "parameters": {
           "properties": {
-            "summary": {
-              "description": "A summary of the dialog",
+            "return": {
+              "description": "The instructed value to return or a summary of the dialog if no value is instructed",
               "type": "string"
             }
           },
diff --git a/pkg/tests/testdata/TestContextSubChat/call7.golden b/pkg/tests/testdata/TestContextSubChat/call7.golden
@@ -9,8 +9,8 @@
         "description": "Concludes the conversation. This can not be used to ask a question.",
         "parameters": {
           "properties": {
-            "summary": {
-              "description": "A summary of the dialog",
+            "return": {
+              "description": "The instructed value to return or a summary of the dialog if no value is instructed",
               "type": "string"
             }
           },
diff --git a/pkg/tests/testdata/TestContextSubChat/call8.golden b/pkg/tests/testdata/TestContextSubChat/call8.golden
@@ -9,8 +9,8 @@
         "description": "Concludes the conversation. This can not be used to ask a question.",
         "parameters": {
           "properties": {
-            "summary": {
-              "description": "A summary of the dialog",
+            "return": {
+              "description": "The instructed value to return or a summary of the dialog if no value is instructed",
               "type": "string"
             }
           },
diff --git a/pkg/tests/testdata/TestContextSubChat/step1.golden b/pkg/tests/testdata/TestContextSubChat/step1.golden
@@ -86,8 +86,8 @@
                         "description": "Concludes the conversation. This can not be used to ask a question.",
                         "parameters": {
                           "properties": {
-                            "summary": {
-                              "description": "A summary of the dialog",
+                            "return": {
+                              "description": "The instructed value to return or a summary of the dialog if no value is instructed",
                               "type": "string"
                             }
                           },
diff --git a/pkg/tests/testdata/TestContextSubChat/step3.golden b/pkg/tests/testdata/TestContextSubChat/step3.golden
@@ -130,8 +130,8 @@
                         "description": "Concludes the conversation. This can not be used to ask a question.",
                         "parameters": {
                           "properties": {
-                            "summary": {
-                              "description": "A summary of the dialog",
+                            "return": {
+                              "description": "The instructed value to return or a summary of the dialog if no value is instructed",
                               "type": "string"
                             }
                           },
diff --git a/pkg/tests/testdata/TestDualSubChat/call2.golden b/pkg/tests/testdata/TestDualSubChat/call2.golden
@@ -9,8 +9,8 @@
         "description": "Concludes the conversation. This can not be used to ask a question.",
         "parameters": {
           "properties": {
-            "summary": {
-              "description": "A summary of the dialog",
+            "return": {
+              "description": "The instructed value to return or a summary of the dialog if no value is instructed",
               "type": "string"
             }
           },
diff --git a/pkg/tests/testdata/TestDualSubChat/call3.golden b/pkg/tests/testdata/TestDualSubChat/call3.golden
@@ -9,8 +9,8 @@
         "description": "Concludes the conversation. This can not be used to ask a question.",
         "parameters": {
           "properties": {
-            "summary": {
-              "description": "A summary of the dialog",
+            "return": {
+              "description": "The instructed value to return or a summary of the dialog if no value is instructed",
               "type": "string"
             }
           },
diff --git a/pkg/tests/testdata/TestDualSubChat/call4.golden b/pkg/tests/testdata/TestDualSubChat/call4.golden
@@ -9,8 +9,8 @@
         "description": "Concludes the conversation. This can not be used to ask a question.",
         "parameters": {
           "properties": {
-            "summary": {
-              "description": "A summary of the dialog",
+            "return": {
+              "description": "The instructed value to return or a summary of the dialog if no value is instructed",
               "type": "string"
             }
           },
diff --git a/pkg/tests/testdata/TestDualSubChat/call5.golden b/pkg/tests/testdata/TestDualSubChat/call5.golden
@@ -9,8 +9,8 @@
         "description": "Concludes the conversation. This can not be used to ask a question.",
         "parameters": {
           "properties": {
-            "summary": {
-              "description": "A summary of the dialog",
+            "return": {
+              "description": "The instructed value to return or a summary of the dialog if no value is instructed",
               "type": "string"
             }
           },
diff --git a/pkg/tests/testdata/TestDualSubChat/call6.golden b/pkg/tests/testdata/TestDualSubChat/call6.golden
@@ -9,8 +9,8 @@
         "description": "Concludes the conversation. This can not be used to ask a question.",
         "parameters": {
           "properties": {
-            "summary": {
-              "description": "A summary of the dialog",
+            "return": {
+              "description": "The instructed value to return or a summary of the dialog if no value is instructed",
               "type": "string"
             }
           },
diff --git a/pkg/tests/testdata/TestDualSubChat/step1.golden b/pkg/tests/testdata/TestDualSubChat/step1.golden
@@ -123,8 +123,8 @@
                       "description": "Concludes the conversation. This can not be used to ask a question.",
                       "parameters": {
                         "properties": {
-                          "summary": {
-                            "description": "A summary of the dialog",
+                          "return": {
+                            "description": "The instructed value to return or a summary of the dialog if no value is instructed",
                             "type": "string"
                           }
                         },
@@ -189,8 +189,8 @@
                       "description": "Concludes the conversation. This can not be used to ask a question.",
                       "parameters": {
                         "properties": {
-                          "summary": {
-                            "description": "A summary of the dialog",
+                          "return": {
+                            "description": "The instructed value to return or a summary of the dialog if no value is instructed",
                             "type": "string"
                           }
                         },
diff --git a/pkg/tests/testdata/TestDualSubChat/step2.golden b/pkg/tests/testdata/TestDualSubChat/step2.golden
@@ -130,8 +130,8 @@
                       "description": "Concludes the conversation. This can not be used to ask a question.",
                       "parameters": {
                         "properties": {
-                          "summary": {
-                            "description": "A summary of the dialog",
+                          "return": {
+                            "description": "The instructed value to return or a summary of the dialog if no value is instructed",
                             "type": "string"
                           }
                         },
diff --git a/pkg/tests/testdata/TestDualSubChat/step3.golden b/pkg/tests/testdata/TestDualSubChat/step3.golden
@@ -130,8 +130,8 @@
                       "description": "Concludes the conversation. This can not be used to ask a question.",
                       "parameters": {
                         "properties": {
-                          "summary": {
-                            "description": "A summary of the dialog",
+                          "return": {
+                            "description": "The instructed value to return or a summary of the dialog if no value is instructed",
                             "type": "string"
                           }
                         },
