fixed arg validation

tools/picker/excerpt_modifier.go

@@ -53,7 +53,7 @@ func newExcerptModifier(args ...string) (*excerptModifier, error) {
 	}
 
 	max := cast.ToInt(args[0])
-	if max == 0 {
+	if max <= 0 {
 		return nil, errors.New("max argument must be > 0")
 	}
 

tools/picker/excerpt_modifier_test.go

@@ -28,6 +28,16 @@ func TestNewExcerptModifier(t *testing.T) {
 			[]string{"something"}, // should fallback to 0 which is not allowed
 			true,
 		},
+		{
+			"negative max argument",
+			[]string{"-1"},
+			true,
+		},
+		{
+			"zero max argument",
+			[]string{"0"},
+			true,
+		},
 		{
 			"numeric max argument",
 			[]string{"12"},