Fix isLimitSubset to correctly handle unlimited subsets

Author: kevin-dp

packages/db/src/query/predicate-utils.ts

@@ -769,9 +769,10 @@ export function isLimitSubset(
   subset: number | undefined,
   superset: number | undefined
 ): boolean {
-  // No limit requirement is always satisfied
+  // If requesting all data (no limit), we need unlimited data to satisfy it
   if (subset === undefined) {
-    return true
+    // Only unlimited data can satisfy unlimited request
+    return superset === undefined
   }
 
   // Unlimited superset satisfies any limit requirement

packages/db/tests/predicate-utils.test.ts

@@ -621,8 +621,12 @@ describe(`isOrderBySubset`, () => {
 })
 
 describe(`isLimitSubset`, () => {
-  it(`should return true for undefined subset`, () => {
-    expect(isLimitSubset(undefined, 10)).toBe(true)
+  it(`should return false for undefined subset with limited superset (requesting all data but only have limited)`, () => {
+    expect(isLimitSubset(undefined, 10)).toBe(false)
+  })
+
+  it(`should return true for undefined subset with undefined superset (requesting all data and have all data)`, () => {
+    expect(isLimitSubset(undefined, undefined)).toBe(true)
   })
 
   it(`should return true for undefined superset`, () => {