@checkstack/common 0.10.0 → 0.11.0

package/CHANGELOG.md

@@ -1,5 +1,59 @@
 # @checkstack/common
 
+## 0.11.0
+
+### Minor Changes
+
+- f23f3c9: Add the canonical `PaginationInput` zod schema and `PaginatedResult`
+  output factory in `@checkstack/common`. `PaginationInput` is an
+  integer-clamped `{ limit: 1-100 (default 20), offset: >= 0 (default 0) }`
+  shape that composes with `.extend({...})` for domain-specific filters
+  (e.g. `unreadOnly` on notifications). `PaginatedResult(itemSchema)`
+  returns the standard `{ items, total, limit, offset }` envelope. The
+  existing `PaginationInputSchema` / `paginatedOutput` / `PaginatedResponse`
+  exports are now marked `@deprecated` and will be removed once the
+  follow-up sweep migrates every `*-common` consumer to the canonical
+  contract.
+- f23f3c9: Sweep every paginated `*-common` contract onto the canonical
+  `PaginationInput` / `PaginatedResult` from `@checkstack/common` and
+  remove the now-unused legacy exports.
+
+  **BREAKING CHANGE** - `@checkstack/common` drops the deprecated
+  `PaginationInputSchema`, `paginatedOutput`, and `PaginatedResponse`
+  symbols. Callers must consume `PaginationInput` (input) and
+  `PaginatedResult(itemSchema)` (output) instead. The canonical input is
+  `{ limit (1-100, default 20), offset (>= 0, default 0) }`; the
+  canonical output envelope is
+  `{ items, total, limit, offset }`.
+
+  **BREAKING CHANGE** - `@checkstack/notification-common` migrates
+  `getNotifications` off the legacy `PaginationInputSchema`
+  (`{ limit, offset, unreadOnly }` with output `{ notifications, total }`)
+  onto `ListNotificationsInputSchema =
+PaginationInput.extend({ unreadOnly })` and
+  `PaginatedResult(NotificationSchema)`. The output key changes from
+  `notifications` to `items`, and `limit` / `offset` are now echoed on
+  the response. The `PaginationInput` type alias previously exported
+  from `notification-common` is removed - use `ListNotificationsInput`
+  or the canonical `PaginationInput` from `@checkstack/common`.
+
+  **BREAKING CHANGE** - `@checkstack/integration-common` migrates
+  `listSubscriptions` (inline `{ page, pageSize, ... }` -> output
+  `{ subscriptions, total }`) and `getDeliveryLogs` (via
+  `DeliveryLogQueryInputSchema` `{ subscriptionId?, eventType?, status?,
+page, pageSize }` -> output `{ logs, total }`) onto the canonical
+  `PaginationInput.extend({...})` input and
+  `PaginatedResult(itemSchema)` output. External callers must switch
+  from `{ page, pageSize }` to `{ limit, offset }` and read response
+  items from `data.items` (no more `data.subscriptions` / `data.logs`).
+
+  The matching `*-backend` handlers were updated to consume the new
+  input shape (`offset` arithmetic in lieu of `(page - 1) * pageSize`)
+  and to echo `limit` / `offset` on the response. The `*-frontend` call
+  sites in `NotificationsPage`, `NotificationBell`, `IntegrationsPage`,
+  and `DeliveryLogsPage` were updated to send the new input shape and
+  read `data.items`.
+
 ## 0.10.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/common",
-  "version": "0.10.0",
+  "version": "0.11.0",
   "license": "Elastic-2.0",
   "type": "module",
   "main": "./src/index.ts",
@@ -20,7 +20,7 @@
   "devDependencies": {
     "typescript": "^5.7.2",
     "@checkstack/tsconfig": "0.0.7",
-    "@checkstack/scripts": "0.3.1"
+    "@checkstack/scripts": "0.3.2"
   },
   "scripts": {
     "typecheck": "tsgo -b",

package/src/pagination.test.ts

@@ -1,89 +1,156 @@
 import { describe, it, expect } from "bun:test";
-import { PaginationInputSchema, paginatedOutput } from "./pagination";
 import { z } from "zod";
+import { PaginationInput, PaginatedResult } from "./pagination";
 
-describe("PaginationInputSchema", () => {
-  it("should accept valid pagination input", () => {
-    const result = PaginationInputSchema.parse({
-      limit: 20,
-      offset: 40,
-    });
-
+describe("PaginationInput (canonical)", () => {
+  it("applies default limit and offset when omitted", () => {
+    const result = PaginationInput.parse({});
     expect(result.limit).toBe(20);
-    expect(result.offset).toBe(40);
+    expect(result.offset).toBe(0);
   });
 
-  it("should use default values when not provided", () => {
-    const result = PaginationInputSchema.parse({});
+  it("accepts valid limit and offset values", () => {
+    const result = PaginationInput.parse({ limit: 50, offset: 100 });
+    expect(result.limit).toBe(50);
+    expect(result.offset).toBe(100);
+  });
 
-    expect(result.limit).toBe(10);
-    expect(result.offset).toBe(0);
+  it("rejects limit below 1", () => {
+    expect(() => PaginationInput.parse({ limit: 0 })).toThrow();
+  });
+
+  it("rejects limit above 100", () => {
+    expect(() => PaginationInput.parse({ limit: 101 })).toThrow();
   });
 
-  it("should reject limit below 1", () => {
-    expect(() => PaginationInputSchema.parse({ limit: 0 })).toThrow();
+  it("rejects negative offset", () => {
+    expect(() => PaginationInput.parse({ offset: -1 })).toThrow();
   });
 
-  it("should reject limit above 100", () => {
-    expect(() => PaginationInputSchema.parse({ limit: 101 })).toThrow();
+  it("rejects non-integer limit values", () => {
+    expect(() => PaginationInput.parse({ limit: 10.5 })).toThrow();
   });
 
-  it("should reject negative offset", () => {
-    expect(() => PaginationInputSchema.parse({ offset: -1 })).toThrow();
+  it("rejects non-integer offset values", () => {
+    expect(() => PaginationInput.parse({ offset: 1.5 })).toThrow();
+  });
+
+  it("supports `.extend({...})` for domain extras", () => {
+    const ListNotificationsInput = PaginationInput.extend({
+      unreadOnly: z.boolean().default(false),
+    });
+
+    const parsed = ListNotificationsInput.parse({});
+    expect(parsed.limit).toBe(20);
+    expect(parsed.offset).toBe(0);
+    expect(parsed.unreadOnly).toBe(false);
+
+    const parsedWithExtras = ListNotificationsInput.parse({
+      limit: 5,
+      unreadOnly: true,
+    });
+    expect(parsedWithExtras.limit).toBe(5);
+    expect(parsedWithExtras.unreadOnly).toBe(true);
+  });
+
+  it("infers a typed PaginationInput", () => {
+    const sample: PaginationInput = { limit: 10, offset: 0 };
+    expect(sample.limit).toBe(10);
+    expect(sample.offset).toBe(0);
   });
 });
 
-describe("paginatedOutput", () => {
+describe("PaginatedResult (canonical)", () => {
   const ItemSchema = z.object({
     id: z.string(),
     name: z.string(),
   });
 
-  it("should create correct output schema structure", () => {
-    const outputSchema = paginatedOutput(ItemSchema);
-
-    const result = outputSchema.parse({
+  it("produces an { items, total, limit, offset } shape", () => {
+    const schema = PaginatedResult(ItemSchema);
+    const result = schema.parse({
       items: [
-        { id: "1", name: "Item 1" },
-        { id: "2", name: "Item 2" },
+        { id: "1", name: "First" },
+        { id: "2", name: "Second" },
       ],
-      total: 100,
+      total: 42,
+      limit: 20,
+      offset: 0,
     });
 
     expect(result.items).toHaveLength(2);
-    expect(result.total).toBe(100);
+    expect(result.total).toBe(42);
+    expect(result.limit).toBe(20);
+    expect(result.offset).toBe(0);
   });
 
-  it("should reject invalid items", () => {
-    const outputSchema = paginatedOutput(ItemSchema);
-
+  it("rejects items that do not match the inner schema", () => {
+    const schema = PaginatedResult(ItemSchema);
     expect(() =>
-      outputSchema.parse({
-        items: [{ id: "1" }], // Missing 'name'
+      schema.parse({
+        items: [{ id: "1" }],
         total: 1,
-      })
+        limit: 20,
+        offset: 0,
+      }),
     ).toThrow();
   });
 
-  it("should reject missing total", () => {
-    const outputSchema = paginatedOutput(ItemSchema);
+  it("rejects a missing total", () => {
+    const schema = PaginatedResult(ItemSchema);
+    expect(() =>
+      schema.parse({
+        items: [],
+        limit: 20,
+        offset: 0,
+      }),
+    ).toThrow();
+  });
 
+  it("rejects a missing limit/offset", () => {
+    const schema = PaginatedResult(ItemSchema);
     expect(() =>
-      outputSchema.parse({
-        items: [{ id: "1", name: "Item 1" }],
-      })
+      schema.parse({
+        items: [],
+        total: 0,
+      }),
     ).toThrow();
   });
 
-  it("should accept empty items array", () => {
-    const outputSchema = paginatedOutput(ItemSchema);
+  it("rejects a negative total", () => {
+    const schema = PaginatedResult(ItemSchema);
+    expect(() =>
+      schema.parse({
+        items: [],
+        total: -1,
+        limit: 20,
+        offset: 0,
+      }),
+    ).toThrow();
+  });
 
-    const result = outputSchema.parse({
+  it("accepts an empty items array", () => {
+    const schema = PaginatedResult(ItemSchema);
+    const result = schema.parse({
       items: [],
       total: 0,
+      limit: 20,
+      offset: 0,
     });
-
     expect(result.items).toEqual([]);
     expect(result.total).toBe(0);
   });
+
+  it("infers a typed result via z.infer", () => {
+    const schema = PaginatedResult(ItemSchema);
+    type Result = z.infer<typeof schema>;
+    const value: Result = {
+      items: [{ id: "x", name: "X" }],
+      total: 1,
+      limit: 20,
+      offset: 0,
+    };
+    expect(value.items[0]?.id).toBe("x");
+  });
 });
+

package/src/pagination.ts

@@ -1,55 +1,55 @@
 import { z } from "zod";
 
 /**
- * Standard pagination input schema for RPC procedures.
- * Use with paginatedOutput() for consistent pagination patterns.
+ * Canonical pagination input schema for RPC procedures.
+ *
+ * This is the single source of truth for paginated list inputs across
+ * the platform. Domain-specific extras (e.g. `unreadOnly` on
+ * notifications) must compose with `.extend({...})` — do NOT redefine
+ * the base fields.
  *
  * @example
  * ```typescript
- * // In your contract:
- * import { PaginationInputSchema, paginatedOutput } from "@checkstack/common";
+ * import { PaginationInput, PaginatedResult } from "@checkstack/common";
+ *
+ * const ListNotificationsInput = PaginationInput.extend({
+ *   unreadOnly: z.boolean().default(false),
+ * });
  *
- * const contract = {
- *   getItems: _base
- *     .input(PaginationInputSchema.extend({ search: z.string().optional() }))
- *     .output(paginatedOutput(ItemSchema)),
- * };
+ * const ListNotificationsOutput = PaginatedResult(NotificationSchema);
  * ```
  */
-export const PaginationInputSchema = z.object({
-  /** Number of items per page (1-100) */
-  limit: z.number().min(1).max(100).default(10),
-  /** Number of items to skip */
-  offset: z.number().min(0).default(0),
+export const PaginationInput = z.object({
+  /** Number of items per page (1-100). */
+  limit: z.number().int().min(1).max(100).default(20),
+  /** Number of items to skip from the start of the result set. */
+  offset: z.number().int().min(0).default(0),
 });
 
-export type PaginationInput = z.infer<typeof PaginationInputSchema>;
+export type PaginationInput = z.infer<typeof PaginationInput>;
 
 /**
- * Creates a paginated output schema wrapper.
- * Returns { items: T[], total: number } structure that works with usePagination hook.
+ * Canonical paginated response factory.
+ *
+ * Wraps an item schema in the standard `{ items, total, limit, offset }`
+ * shape. The echoed `limit` / `offset` mirror what the server actually
+ * applied so clients can render correct pagination controls even when
+ * defaults were used.
  *
  * @example
  * ```typescript
- * // Contract definition:
- * getUsers: _base
- *   .input(PaginationInputSchema)
- *   .output(paginatedOutput(UserSchema)),
+ * // In a contract:
+ * getNotifications: _base
+ *   .input(PaginationInput)
+ *   .output(PaginatedResult(NotificationSchema)),
  *
- * // Returns: { items: User[], total: number }
+ * // Returns: { items: Notification[], total: number, limit: number, offset: number }
  * ```
  */
-export function paginatedOutput<T extends z.ZodTypeAny>(itemSchema: T) {
-  return z.object({
+export const PaginatedResult = <T extends z.ZodTypeAny>(itemSchema: T) =>
+  z.object({
     items: z.array(itemSchema),
-    total: z.number(),
+    total: z.number().int().min(0),
+    limit: z.number().int(),
+    offset: z.number().int(),
   });
-}
-
-/**
- * Type helper for paginated responses
- */
-export type PaginatedResponse<T> = {
-  items: T[];
-  total: number;
-};