@reventlessdev/reventless-aws 3.0.0-alpha.322 → 3.0.0-alpha.323

package/CHANGELOG.md

@@ -3,6 +3,13 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# 3.0.0-alpha.323 (2026-08-23)
+
+### Bug Fixes
+
+* **querydb:** serve the previous page instead of refusing it ([48341c9](https://github.com/ReventlessDev/reventless-core/commit/48341c9f61b5155b441deeb0edc9abf461948346))
+
+
 # 3.0.0-alpha.322 (2026-08-23)
 
 ### Bug Fixes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reventlessdev/reventless-aws",
-  "version": "3.0.0-alpha.322",
+  "version": "3.0.0-alpha.323",
   "description": "AWS adapters for Reventless",
   "license": "Apache-2.0",
   "bin": {
@@ -16,17 +16,17 @@
     "sury": "11.0.0-rc.2",
     "uuid": "^13.0.0",
     "@reventlessdev/rescript-aws-sdk": "3.0.0-alpha.13",
-    "@reventlessdev/rescript-jest": "1.0.0-alpha.10",
-    "@reventlessdev/rescript-effect": "0.1.0-alpha.32",
     "@reventlessdev/rescript-node": "2.0.0-alpha.8",
-    "@reventlessdev/rescript-pulumi-aws": "3.0.0-alpha.4",
+    "@reventlessdev/rescript-effect": "0.1.0-alpha.32",
+    "@reventlessdev/rescript-pulumi-aws": "3.0.0-alpha.5",
+    "@reventlessdev/rescript-jest": "1.0.0-alpha.10",
     "@reventlessdev/rescript-pulumi-pulumi": "2.3.0-alpha.19",
-    "@reventlessdev/reventless-core": "3.0.0-alpha.249",
-    "@reventlessdev/reventless-infra": "3.0.0-alpha.151",
     "@reventlessdev/rescript-uuid": "2.0.0-alpha.0",
     "@reventlessdev/reventless-interop": "3.0.0-alpha.33",
-    "@reventlessdev/reventless-postgres": "3.0.0-alpha.113",
-    "@reventlessdev/reventless-spec": "3.0.0-alpha.123"
+    "@reventlessdev/reventless-postgres": "3.0.0-alpha.114",
+    "@reventlessdev/reventless-core": "3.0.0-alpha.250",
+    "@reventlessdev/reventless-spec": "3.0.0-alpha.123",
+    "@reventlessdev/reventless-infra": "3.0.0-alpha.151"
   },
   "devDependencies": {
     "rescript": "12.3.0",

package/tests/QueryDbResolvers_AppSyncTest.res

@@ -51,16 +51,32 @@ describe("listAllItemsConnection — paging", () => {
   })
 
   testSync("a filtered read examines more rows than it serves", () => {
-    expect(code->String.includes("parts.length > 0 ? (_first > 1000 ? _first : 1000)"))->toBe(true)
-    expect(code->String.includes("const _page = _rest.slice(0, _first);"))->toBe(true)
+    // The budget reads as deep as the page needs, which for a backward cut is
+    // `_upTo` rather than `_first + _from`.
+    expect(
+      code->String.includes("parts.length > 0 ? ((_backward ? _upTo : _first + _from) > 1000"),
+    )->toBe(true)
+    expect(code->String.includes("const _page = _backward ? _rest : _rest.slice(0, _first);"))
+    ->toBe(true)
     expect(code->String.includes("hasNextPage: _more || !!_next,"))->toBe(true)
   })
 
-  testSync("backward paging (last/before) is rejected, not silently mishandled", () => {
-    expect(code->String.includes("ctx.args.before != null || ctx.args.last != null"))->toBe(true)
+  // `before` is served by re-reading the window the cursor names; only a previous
+  // page lying in an EARLIER window is out of reach, and `last` — which needs the
+  // end of the list — stays refused.
+  testSync("backward paging is served, and its one limit is named", () => {
+    expect(code->String.includes("ctx.args.before != null && ctx.args.before !== ''"))->toBe(true)
+    expect(code->String.includes("_backward && _upTo <= 0 && _window !== null"))->toBe(true)
     expect(code->String.includes("UnsupportedPagination"))->toBe(true)
   })
 
+  // The door must not advertise a previous page it cannot serve — that is what
+  // put a live Prev button in front of an error.
+  testSync("only a reachable previous page is advertised", () => {
+    expect(code->String.includes("hasPreviousPage: _from > 0,"))->toBe(true)
+    expect(code->String.includes("hasPreviousPage: !!ctx.args.after,"))->toBe(false)
+  })
+
   testSync("a window emptied by the filter still yields a resumable boundary cursor", () => {
     expect(
       code->String.includes(

package/tests/QueryDbResolvers_AppSyncTest.res.mjs

@@ -37,14 +37,19 @@ globalThis.describe("listAllItemsConnection — paging", () => {
     globalThis.expect(code.includes("ctx.args.after + '_' + i")).toBe(false);
   });
   globalThis.test("a filtered read examines more rows than it serves", () => {
-    globalThis.expect(code.includes("parts.length > 0 ? (_first > 1000 ? _first : 1000)")).toBe(true);
-    globalThis.expect(code.includes("const _page = _rest.slice(0, _first);")).toBe(true);
+    globalThis.expect(code.includes("parts.length > 0 ? ((_backward ? _upTo : _first + _from) > 1000")).toBe(true);
+    globalThis.expect(code.includes("const _page = _backward ? _rest : _rest.slice(0, _first);")).toBe(true);
     globalThis.expect(code.includes("hasNextPage: _more || !!_next,")).toBe(true);
   });
-  globalThis.test("backward paging (last/before) is rejected, not silently mishandled", () => {
-    globalThis.expect(code.includes("ctx.args.before != null || ctx.args.last != null")).toBe(true);
+  globalThis.test("backward paging is served, and its one limit is named", () => {
+    globalThis.expect(code.includes("ctx.args.before != null && ctx.args.before !== ''")).toBe(true);
+    globalThis.expect(code.includes("_backward && _upTo <= 0 && _window !== null")).toBe(true);
     globalThis.expect(code.includes("UnsupportedPagination")).toBe(true);
   });
+  globalThis.test("only a reachable previous page is advertised", () => {
+    globalThis.expect(code.includes("hasPreviousPage: _from > 0,")).toBe(true);
+    globalThis.expect(code.includes("hasPreviousPage: !!ctx.args.after,")).toBe(false);
+  });
   globalThis.test("a window emptied by the filter still yields a resumable boundary cursor", () => {
     globalThis.expect(code.includes("const _boundary = _next ? util.base64Encode(JSON.stringify({ t: _next, n: -1 })) : null;")).toBe(true);
     globalThis.expect(code.includes("edges.length > 0 ? edges[edges.length - 1].cursor : _boundary")).toBe(true);