@anteros/core 0.0.23 → 0.0.24

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@anteros/core",
   "module": "index.ts",
-  "version": "0.0.23",
+  "version": "0.0.24",
   "type": "module",
   "description": "Anteros Core",
   "private": false,

package/tests/api.test.ts

@@ -73,6 +73,56 @@ describe("find", () => {
     });
 });
 
+describe("$limit", () => {
+    it("$limit: 0 means no limit (returns all matching docs)", async () => {
+        const marker = `limit-zero-${crypto.randomUUID()}`;
+        await rest.bulkWrite("items", Array.from({ length: 101 }, (_, i) => ({
+            insertOne: { document: { title: marker, count: i } },
+        })) as any);
+
+        const all = await rest.find("items", { $match: { title: marker }, $limit: 0 });
+        expect(all.length).toBe(101); // the old 100 cap would have returned 100
+
+        await rest.db.collection("items").deleteMany({ title: marker });
+    });
+
+    it("accepts $limit above 1000", async () => {
+        const marker = `limit-big-${crypto.randomUUID()}`;
+        await rest.insertMany("items", [
+            { title: marker, count: 1 },
+            { title: marker, count: 2 },
+            { title: marker, count: 3 },
+        ]);
+
+        const docs = await rest.find("items", { $match: { title: marker }, $limit: 5000 });
+        expect(docs.length).toBe(3);
+
+        await rest.db.collection("items").deleteMany({ title: marker });
+    });
+
+    it("defaults to $limit 100", async () => {
+        const marker = `limit-default-${crypto.randomUUID()}`;
+        await rest.bulkWrite("items", Array.from({ length: 120 }, (_, i) => ({
+            insertOne: { document: { title: marker, count: i } },
+        })) as any);
+
+        const docs = await rest.find("items", { $match: { title: marker } });
+        expect(docs.length).toBe(100);
+
+        await rest.db.collection("items").deleteMany({ title: marker });
+    });
+
+    it("rejects negative $limit with INVALID_LIMIT", async () => {
+        const err: any = await rest.find("items", { $limit: -1 }).then(() => null, (e) => e);
+        expect(err?.code).toBe("INVALID_LIMIT");
+    });
+
+    it("rejects non-integer $limit with INVALID_LIMIT", async () => {
+        const err: any = await rest.find("items", { $limit: 1.5 }).then(() => null, (e) => e);
+        expect(err?.code).toBe("INVALID_LIMIT");
+    });
+});
+
 describe("findOne", () => {
     it("returns single doc by _id", async () => {
         const doc = await rest.insertOne("items", { title: "single" });

package/utils/func.ts

@@ -748,18 +748,20 @@ function buildPipeline(p: FindOptions, options?: {
         })
     }
 
-    // $limit — Faille 14: must be a safe integer in (0, MAX_LIMIT]; 0 would mean "no limit"
-    const MAX_LIMIT = 1000;
+    // $limit — 0 means "no limit" (the $limit stage is omitted);
+    // any positive safe integer is allowed (no 1000 cap)
     p.$limit = p?.$limit ?? 100
-    if (!Number.isSafeInteger(p.$limit) || p.$limit <= 0 || p.$limit > MAX_LIMIT) {
-        throw new AppError(`Invalid $limit (expected an integer between 1 and ${MAX_LIMIT})`, {
+    if (!Number.isSafeInteger(p.$limit) || p.$limit < 0) {
+        throw new AppError(`Invalid $limit (expected a non-negative integer; 0 = no limit)`, {
             code: 'INVALID_LIMIT',
             status: 400
         });
     }
-    pipeline.push({
-        $limit: p.$limit
-    })
+    if (p.$limit > 0) {
+        pipeline.push({
+            $limit: p.$limit
+        })
+    }
 
     // $include
     if (p?.$include && Array.isArray(p?.$include)) {