comment and test tidy

Author: danmrichards

internal/api/client.go

@@ -185,7 +185,7 @@ func (c *Client) Run(ctx context.Context, model string, args []string, opts RunO
 	}
 
 	// Parse args against the real schema so type coercion is schema-driven.
-	// Protected fields (taskType, model, deliveryMethod) are rejected here.
+	// Protected fields are rejected here.
 	payload := make(map[string]any, len(args)+4)
 	payload[fieldModel] = model
 	for _, a := range args {

internal/api/run_test.go

@@ -426,7 +426,11 @@ func TestClientRun_UserProvidedTaskUUID(t *testing.T) {
 	if len(mock.captured) == 0 {
 		t.Fatal("no transport calls captured")
 	}
-	taskBytes, err := json.Marshal(mock.captured[0][0])
+	tasks := mock.captured[0]
+	if len(tasks) == 0 {
+		t.Fatal("submitted tasks slice is empty")
+	}
+	taskBytes, err := json.Marshal(tasks[0])
 	if err != nil {
 		t.Fatalf("marshal captured task: %v", err)
 	}