breaking change: refactor EnsureLimits method to emphasize fix error …

…limit parameters (#6)

* breaking change: Refactored EnsureLimits method to emphasize fix error parameter

* Update middleware.go

README.md

@@ -24,8 +24,8 @@ p := relay.New(
         // Keyset-based pagination
         return gormrelay.NewKeysetAdapter[*User](db)(ctx, req)
     }),
-    // maxLimit / limitIfNotSet
-    relay.EnsureLimits[*User](100, 10),
+    // defaultLimit / maxLimit
+    relay.EnsureLimits[*User](10, 100),
     // Append primary sorting fields, if any are unspecified
     relay.EnsurePrimaryOrderBy[*User](
         relay.OrderBy{Field: "ID", Desc: false},

context.go

@@ -23,11 +23,11 @@ func GetSkip(ctx context.Context) Skip {
 
 type ctxKeyNodeProcessor struct{}
 
-func WithNodeProcessor[T any](ctx context.Context, processor func(node T) T) context.Context {
+func WithNodeProcessor[T any](ctx context.Context, processor func(ctx context.Context, node T) (T, error)) context.Context {
 	return context.WithValue(ctx, ctxKeyNodeProcessor{}, processor)
 }
 
-func GetNodeProcessor[T any](ctx context.Context) func(node T) T {
-	processor, _ := ctx.Value(ctxKeyNodeProcessor{}).(func(node T) T)
+func GetNodeProcessor[T any](ctx context.Context) func(ctx context.Context, node T) (T, error) {
+	processor, _ := ctx.Value(ctxKeyNodeProcessor{}).(func(ctx context.Context, node T) (T, error))
 	return processor
 }

gormrelay/keyset.go

@@ -144,6 +144,8 @@ func scopeKeyset(after, before *map[string]any, orderBys []relay.OrderBy, limit
 
 		if limit > 0 {
 			exprs = append(exprs, clause.Limit{Limit: &limit})
+		} else {
+			db.AddError(errors.New("limit must be greater than 0"))
 		}
 
 		return db.Clauses(exprs...)