Improve readability of the errIsValidTimeout logic

confirm/command.go

@@ -31,11 +31,12 @@ func (o Options) Run() error {
 		WithShowHelp(o.ShowHelp).
 		Run()
 
-	// Should the form time out it will return an error with this string:
-	const timeoutErrMsg = "timeout"
+	if err != nil {
+		allowErr := o.errIsValidTimeout(err)
 
-	if err != nil && err.Error() != timeoutErrMsg {
-		return fmt.Errorf("unable to run confirm: %w", err)
+		if !allowErr {
+			return fmt.Errorf("unable to run confirm: %w", err)
+		}
 	}
 
 	if !choice {
@@ -44,3 +45,11 @@ func (o Options) Run() error {
 
 	return nil
 }
+
+// errIsValidTimeout returns true iff an error is a timeout error AND the user specified a nonzero o.Timeout.
+func (o Options) errIsValidTimeout(err error) bool {
+	errWasTimeout := err.Error() == huh.ErrTimeout.Error()
+	timeoutsExpected := o.Timeout > 0
+
+	return errWasTimeout && timeoutsExpected
+}