Merge pull request #260 from ibuildthecloud/skipnode

bug: replace - with _ in tool names sent to LLM

Author: ibuildthecloud

pkg/tests/testdata/TestExport/call1.golden

@@ -23,7 +23,7 @@
     {
       "function": {
         "toolID": "testdata/TestExport/parent.gpt:11",
-        "name": "parent-local",
+        "name": "parent_local",
         "parameters": {
           "properties": {
             "defaultPromptParameter": {

pkg/tests/testdata/TestExport/call3.golden

@@ -23,7 +23,7 @@
     {
       "function": {
         "toolID": "testdata/TestExport/parent.gpt:11",
-        "name": "parent-local",
+        "name": "parent_local",
         "parameters": {
           "properties": {
             "defaultPromptParameter": {

pkg/types/toolname.go

@@ -9,8 +9,8 @@ import (
 )
 
 var (
-	validToolName = regexp.MustCompile("^[a-zA-Z0-9_-]{1,64}$")
-	invalidChars  = regexp.MustCompile("[^a-zA-Z0-9_-]+")
+	validToolName = regexp.MustCompile("^[a-zA-Z0-9_]{1,64}$")
+	invalidChars  = regexp.MustCompile("[^a-zA-Z0-9_]+")
 )
 
 func ToolNormalizer(tool string) string {

pkg/types/toolname_test.go

@@ -0,0 +1,11 @@
+package types
+
+import (
+	"testing"
+
+	"github.com/hexops/autogold/v2"
+)
+
+func TestToolNormalizer(t *testing.T) {
+	autogold.Expect("bob_tool").Equal(t, ToolNormalizer("bob-tool"))
+}