chore: only allow global params in first tool

Author: ibuildthecloud

pkg/loader/loader.go

@@ -221,6 +221,14 @@ func readTool(ctx context.Context, cache *cache.Client, prg *types.Program, base
 			return types.Tool{}, parser.NewErrLine(tool.Source.Location, tool.Source.LineNo, fmt.Errorf("only the first tool in a file can have no name"))
 		}
 
+		if i != 0 && tool.Parameters.GlobalModelName != "" {
+			return types.Tool{}, parser.NewErrLine(tool.Source.Location, tool.Source.LineNo, fmt.Errorf("only the first tool in a file can have global model name"))
+		}
+
+		if i != 0 && len(tool.Parameters.GlobalTools) > 0 {
+			return types.Tool{}, parser.NewErrLine(tool.Source.Location, tool.Source.LineNo, fmt.Errorf("only the first tool in a file can have global tools"))
+		}
+
 		if targetToolName != "" && strings.EqualFold(tool.Parameters.Name, targetToolName) {
 			mainTool = tool
 		}

pkg/tests/runner_test.go

@@ -669,6 +669,15 @@ func TestCase2(t *testing.T) {
 	assert.Equal(t, "TEST RESULT CALL: 1", x)
 }
 
+func TestGlobalErr(t *testing.T) {
+	runner := tester.NewRunner(t)
+	_, err := runner.Run("", "")
+	autogold.Expect("line testdata/TestGlobalErr/test.gpt:4: only the first tool in a file can have global model name").Equal(t, err.Error())
+
+	_, err = runner.Run("test2.gpt", "")
+	autogold.Expect("line testdata/TestGlobalErr/test2.gpt:4: only the first tool in a file can have global tools").Equal(t, err.Error())
+}
+
 func TestContextArg(t *testing.T) {
 	runner := tester.NewRunner(t)
 	x, err := runner.Run("", `{

pkg/tests/testdata/TestGlobalErr/test.gpt

@@ -0,0 +1,7 @@
+first
+
+---
+name: second
+global model name: foo
+
+second

pkg/tests/testdata/TestGlobalErr/test2.gpt

@@ -0,0 +1,7 @@
+first
+
+---
+name: second
+global tools: asdf
+
+second