@open-mercato/shared 0.6.8-develop.6987.1.02f619470c → 0.6.8-develop.6992.1.00c90fecff

package/.turbo/turbo-build.log

@@ -1,2 +1,2 @@
-[build:shared] found 260 entry points
+[build:shared] found 261 entry points
 [build:shared] built successfully

package/dist/lib/pagination/load-more.js

@@ -0,0 +1,7 @@
+function hasMoreFromPage(servedCount, pageSize) {
+  return servedCount >= pageSize;
+}
+export {
+  hasMoreFromPage
+};
+//# sourceMappingURL=load-more.js.map

package/dist/lib/pagination/load-more.js.map

@@ -0,0 +1,7 @@
+{
+  "version": 3,
+  "sources": ["../../../src/lib/pagination/load-more.ts"],
+  "sourcesContent": ["/**\n * Termination signal for incremental \"load more\" affordances.\n *\n * A page that comes back full is the only reliable \"there may be more\" signal.\n * A server-reported `total` / `totalPages` can under-report \u2014 rows inserted\n * between two requests do it today, and a capped list count will do it by\n * design once `OM_LIST_COUNT_CAP` lands (see\n * `.ai/specs/2026-07-27-list-count-strategies.md`) \u2014 and gating the affordance\n * on one strands every row past the reported end behind a button that has\n * already disappeared.\n *\n * `>=` rather than `===` so an endpoint that serves more than it was asked for\n * still terminates correctly.\n *\n * Two obligations come with this, and both have already caused real defects:\n *\n * 1. **Measure what the server served**, not what survived client-side dedupe\n *    or filtering. A deduped length shorter than the served page terminates the\n *    sequence early \u2014 reintroducing exactly the bug this replaces.\n *\n * 2. **Only for endpoints whose page past the end comes back short.** That\n *    holds for every list built on the query engine, which computes an\n *    unclamped offset. An endpoint that instead clamps the requested page to\n *    the last page re-serves that page in full, forever; a caller of one must\n *    also check it was served the page it asked for before appending. See\n *    `AttachmentsSection`, whose endpoint clamps.\n */\nexport function hasMoreFromPage(servedCount: number, pageSize: number): boolean {\n  return servedCount >= pageSize\n}\n"],
+  "mappings": "AA2BO,SAAS,gBAAgB,aAAqB,UAA2B;AAC9E,SAAO,eAAe;AACxB;",
+  "names": []
+}

package/dist/lib/version.js

@@ -1,4 +1,4 @@
-const APP_VERSION = "0.6.8-develop.6987.1.02f619470c";
+const APP_VERSION = "0.6.8-develop.6992.1.00c90fecff";
 const appVersion = APP_VERSION;
 export {
   APP_VERSION,

package/dist/lib/version.js.map

@@ -1,7 +1,7 @@
 {
   "version": 3,
   "sources": ["../../src/lib/version.ts"],
-  "sourcesContent": ["// Build-time generated version\nexport const APP_VERSION = '0.6.8-develop.6987.1.02f619470c';\nexport const appVersion = APP_VERSION;\n"],
+  "sourcesContent": ["// Build-time generated version\nexport const APP_VERSION = '0.6.8-develop.6992.1.00c90fecff';\nexport const appVersion = APP_VERSION;\n"],
   "mappings": "AACO,MAAM,cAAc;AACpB,MAAM,aAAa;",
   "names": []
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-mercato/shared",
-  "version": "0.6.8-develop.6987.1.02f619470c",
+  "version": "0.6.8-develop.6992.1.00c90fecff",
   "license": "MIT",
   "type": "module",
   "main": "./dist/index.js",
@@ -105,7 +105,7 @@
     "@mikro-orm/core": "^7.1.8",
     "@mikro-orm/decorators": "^7.1.8",
     "@mikro-orm/postgresql": "^7.1.8",
-    "@open-mercato/cache": "0.6.8-develop.6987.1.02f619470c",
+    "@open-mercato/cache": "0.6.8-develop.6992.1.00c90fecff",
     "@types/sanitize-html": "^2.16.1",
     "dotenv": "^17.4.2",
     "pino": "^10.3.1",

package/src/lib/pagination/__tests__/load-more.test.ts

@@ -0,0 +1,29 @@
+import { hasMoreFromPage } from '../load-more'
+
+describe('hasMoreFromPage', () => {
+  it('reports more when the page came back full', () => {
+    expect(hasMoreFromPage(24, 24)).toBe(true)
+  })
+
+  it('reports more when the endpoint served more than it was asked for', () => {
+    // `>=` rather than `===`, so an over-serving endpoint still terminates.
+    expect(hasMoreFromPage(25, 24)).toBe(true)
+  })
+
+  it('terminates on a short page', () => {
+    expect(hasMoreFromPage(23, 24)).toBe(false)
+  })
+
+  it('terminates on an empty page', () => {
+    expect(hasMoreFromPage(0, 24)).toBe(false)
+  })
+
+  // The served count is the contract (obligation 1 in the docstring): callers
+  // must not pass a client-deduped length. Nothing here can enforce that, but
+  // pinning the boundary keeps the `>=` from drifting to `===` or `>`.
+  it('treats one row short of the page size as the end, one row over as more', () => {
+    expect(hasMoreFromPage(1, 2)).toBe(false)
+    expect(hasMoreFromPage(2, 2)).toBe(true)
+    expect(hasMoreFromPage(3, 2)).toBe(true)
+  })
+})

package/src/lib/pagination/load-more.ts

@@ -0,0 +1,30 @@
+/**
+ * Termination signal for incremental "load more" affordances.
+ *
+ * A page that comes back full is the only reliable "there may be more" signal.
+ * A server-reported `total` / `totalPages` can under-report — rows inserted
+ * between two requests do it today, and a capped list count will do it by
+ * design once `OM_LIST_COUNT_CAP` lands (see
+ * `.ai/specs/2026-07-27-list-count-strategies.md`) — and gating the affordance
+ * on one strands every row past the reported end behind a button that has
+ * already disappeared.
+ *
+ * `>=` rather than `===` so an endpoint that serves more than it was asked for
+ * still terminates correctly.
+ *
+ * Two obligations come with this, and both have already caused real defects:
+ *
+ * 1. **Measure what the server served**, not what survived client-side dedupe
+ *    or filtering. A deduped length shorter than the served page terminates the
+ *    sequence early — reintroducing exactly the bug this replaces.
+ *
+ * 2. **Only for endpoints whose page past the end comes back short.** That
+ *    holds for every list built on the query engine, which computes an
+ *    unclamped offset. An endpoint that instead clamps the requested page to
+ *    the last page re-serves that page in full, forever; a caller of one must
+ *    also check it was served the page it asked for before appending. See
+ *    `AttachmentsSection`, whose endpoint clamps.
+ */
+export function hasMoreFromPage(servedCount: number, pageSize: number): boolean {
+  return servedCount >= pageSize
+}