TODO: Determine if it makes sense to have a negative int32 for maxTokens and what a real maximum is

[maxnystrom]

For the initial release, my instinct was that it didn't make sense to have a maximum number of tokens returned to be negative. Likewise, since the value for the key is defined as int32 in the documentation, I left the maximum as the true maximum value. Not sure if this makes sense either.

The task here is to set real bounds on the maxTokens value.

together-go/completions.go

Lines 54 to 56 in c60d6e1

	if maxTokens < 1 {
	return CompletionsResponse{}, fmt.Errorf("maxTokens must be greater than 0 and less than 2147483647") // Not sure it makes sense to have negative tokens?
	}