Merge pull request #77 from okekefrancis112/pagination

Pagination

Author: greatest0fallt1me

jest.config.js jest.config.cjsjest.config.js renamed to jest.config.cjs

@@ -2,6 +2,7 @@ export default {
   preset: 'ts-jest/presets/default-esm',
   testEnvironment: 'node',
   extensionsToTreatAsEsm: ['.ts'],
+  testMatch: ['**/?(*.)+(spec|test).ts'],
   moduleNameMapper: {
     '^(\\.{1,2}/.*)\\.js$': '$1',
   },
@@ -26,5 +27,4 @@ export default {
       statements: 95,
     },
   },
-  testMatch: ['**/tests/integration/**/*.test.ts'],
 };

src/app.ts

@@ -13,6 +13,8 @@ import { apiStatusEnum, type ApiStatus } from './db/schema.js';
 import { requireAuth, type AuthenticatedLocals } from './middleware/requireAuth.js';
 import { buildDeveloperAnalytics } from './services/developerAnalytics.js';
 import { errorHandler } from './middleware/errorHandler.js';
+import adminRouter from './routes/admin.js';
+import { parsePagination, paginatedResponse } from './lib/pagination.js';
 import { InMemoryVaultRepository, type VaultRepository } from './repositories/vaultRepository.js';
 import { DepositController } from './controllers/depositController.js';
 import { TransactionBuilderService } from './services/transactionBuilder.js';
@@ -83,6 +85,7 @@ export const createApp = (dependencies?: Partial<AppDependencies>) => {
   }
 
   app.use(requestLogger);
+
   const allowedOrigins = (process.env.CORS_ALLOWED_ORIGINS ?? 'http://localhost:5173')
     .split(',')
     .map((o) => o.trim());
@@ -109,8 +112,9 @@ export const createApp = (dependencies?: Partial<AppDependencies>) => {
     res.json({ status: 'ok', service: 'callora-backend' });
   });
 
-  app.get('/api/apis', (_req, res) => {
-    res.json({ apis: [] });
+  app.get('/api/apis', (req, res) => {
+    const { limit, offset } = parsePagination(req.query as { limit?: string; offset?: string });
+    res.json(paginatedResponse([], { limit, offset }));
   });
 
   app.get('/api/apis/:id', async (req, res) => {
@@ -149,8 +153,9 @@ export const createApp = (dependencies?: Partial<AppDependencies>) => {
     });
   });
 
-  app.get('/api/usage', (_req, res) => {
-    res.json({ calls: 0, period: 'current' });
+  app.get('/api/usage', (req, res) => {
+    const { limit, offset } = parsePagination(req.query as { limit?: string; offset?: string });
+    res.json(paginatedResponse([], { limit, offset }));
   });
 
   app.get('/api/developers/apis', requireAuth, async (req, res: express.Response<unknown, AuthenticatedLocals>) => {

src/controllers/depositController.ts

@@ -140,6 +140,11 @@ export class DepositController {
         network: unsignedTx.network,
         contractId: vault.contractId,
         amount: validation.normalizedAmount!,
+        operation: {
+          type: unsignedTx.operation.type,
+          function: unsignedTx.operation.function as 'deposit',
+          args: unsignedTx.operation.args,
+        },
         operation: unsignedTx.operation as DepositPrepareResponse['operation'],
         metadata: {
           fee: unsignedTx.fee,

src/events/event.emitter.ts

@@ -1,5 +1,6 @@
 import { EventEmitter } from 'events';
 import {
+    WebhookConfig,
     WebhookEventType,
     WebhookPayload,
     NewApiCallData,
@@ -24,7 +25,7 @@ async function handleEvent(
     };
 
     const configs = WebhookStore.getByEvent(event).filter(
-        (cfg) => cfg.developerId === developerId
+        (cfg: WebhookConfig) => cfg.developerId === developerId
     );
 
     if (configs.length > 0) {

src/index.ts

@@ -9,11 +9,8 @@ const PORT = process.env.PORT ?? 3000;
 
 // Inject the metrics middleware globally to track all incoming requests
 app.use(metricsMiddleware);
-
-// Register the securely guarded metrics endpoint
 app.get('/api/metrics', metricsEndpoint);
 
-// Execute the server only if this file is run directly
 if (process.argv[1] && fileURLToPath(import.meta.url) === process.argv[1]) {
   app.listen(PORT, () => {
     logger.info(`Callora backend listening on http://localhost:${PORT}`);

src/lib/__tests__/pagination.test.ts

@@ -0,0 +1,49 @@
+import assert from 'node:assert/strict';
+import { describe, it } from 'node:test';
+import { parsePagination, paginatedResponse } from '../pagination.js';
+
+describe('parsePagination', () => {
+  it('returns defaults when no query params given', () => {
+    assert.deepEqual(parsePagination({}), { limit: 20, offset: 0 });
+  });
+
+  it('parses valid limit and offset', () => {
+    assert.deepEqual(parsePagination({ limit: '10', offset: '30' }), { limit: 10, offset: 30 });
+  });
+
+  it('clamps limit to max 100', () => {
+    assert.deepEqual(parsePagination({ limit: '500' }), { limit: 100, offset: 0 });
+  });
+
+  it('clamps limit to min 1', () => {
+    assert.deepEqual(parsePagination({ limit: '0' }), { limit: 1, offset: 0 });
+    assert.deepEqual(parsePagination({ limit: '-5' }), { limit: 1, offset: 0 });
+  });
+
+  it('clamps offset to min 0', () => {
+    assert.deepEqual(parsePagination({ offset: '-10' }), { limit: 20, offset: 0 });
+  });
+
+  it('handles non-numeric strings gracefully', () => {
+    assert.deepEqual(parsePagination({ limit: 'abc', offset: 'xyz' }), { limit: 20, offset: 0 });
+  });
+});
+
+describe('paginatedResponse', () => {
+  it('wraps data and meta into the envelope', () => {
+    const result = paginatedResponse([{ id: '1' }], { total: 1, limit: 20, offset: 0 });
+    assert.deepEqual(result, {
+      data: [{ id: '1' }],
+      meta: { total: 1, limit: 20, offset: 0 },
+    });
+  });
+
+  it('works without total in meta', () => {
+    const result = paginatedResponse([], { limit: 20, offset: 0 });
+    assert.deepEqual(result, {
+      data: [],
+      meta: { limit: 20, offset: 0 },
+    });
+    assert.equal('total' in result.meta, false);
+  });
+});

src/lib/pagination.ts

@@ -0,0 +1,41 @@
+export interface PaginationParams {
+  limit: number;
+  offset: number;
+}
+
+export interface PaginationMeta {
+  total?: number;
+  limit: number;
+  offset: number;
+}
+
+export interface PaginatedResponse<T> {
+  data: T[];
+  meta: PaginationMeta;
+}
+
+const DEFAULT_LIMIT = 20;
+const MAX_LIMIT = 100;
+
+export function parsePagination(query: {
+  limit?: string;
+  offset?: string;
+}): PaginationParams {
+  const parsedLimit = parseInt(query.limit ?? '', 10);
+  const limit = Math.min(
+    MAX_LIMIT,
+    Math.max(1, Number.isNaN(parsedLimit) ? DEFAULT_LIMIT : parsedLimit),
+  );
+
+  const parsedOffset = parseInt(query.offset ?? '', 10);
+  const offset = Math.max(0, Number.isNaN(parsedOffset) ? 0 : parsedOffset);
+
+  return { limit, offset };
+}
+
+export function paginatedResponse<T>(
+  data: T[],
+  meta: PaginationMeta,
+): PaginatedResponse<T> {
+  return { data, meta };
+}

src/repositories/apiRepository.drizzle.ts

@@ -1,10 +1,23 @@
-import { eq, and } from 'drizzle-orm';
+import { eq, and, type SQL } from 'drizzle-orm';
 import { db, schema } from '../db/index.js';
 import type { Api } from '../db/schema.js';
 import type { ApiDetails, ApiEndpointInfo, ApiListFilters, ApiRepository } from './apiRepository.js';
 
 export class DrizzleApiRepository implements ApiRepository {
   async listByDeveloper(developerId: number, filters: ApiListFilters = {}): Promise<Api[]> {
+    const conditions: SQL[] = [eq(schema.apis.developer_id, developerId)];
+    if (filters.status) {
+      conditions.push(eq(schema.apis.status, filters.status));
+    }
+    const results = await db.select().from(schema.apis).where(and(...conditions));
+    let rows = results as Api[];
+    if (typeof filters.offset === 'number') {
+      rows = rows.slice(filters.offset);
+    }
+    if (typeof filters.limit === 'number') {
+      rows = rows.slice(0, filters.limit);
+    }
+    return rows;
     const conditions = [eq(schema.apis.developer_id, developerId)];
     if (filters.status) {
       conditions.push(eq(schema.apis.status, filters.status));

src/repositories/apiRepository.ts

@@ -1,3 +1,4 @@
+import { eq, and, type SQL } from 'drizzle-orm';
 import { eq, and } from 'drizzle-orm';
 import { db, schema } from '../db/index.js';
 import type { Api, ApiStatus } from '../db/schema.js';
@@ -40,11 +41,25 @@ export interface ApiRepository {
 
 export const defaultApiRepository: ApiRepository = {
   async listByDeveloper(developerId, filters = {}) {
+    const conditions: SQL[] = [eq(schema.apis.developer_id, developerId)];
     const conditions = [eq(schema.apis.developer_id, developerId)];
     if (filters.status) {
       conditions.push(eq(schema.apis.status, filters.status));
     }
 
+    const results = await db
+      .select()
+      .from(schema.apis)
+      .where(and(...conditions));
+
+    let rows = results as Api[];
+    if (typeof filters.offset === 'number') {
+      rows = rows.slice(filters.offset);
+    }
+    if (typeof filters.limit === 'number') {
+      rows = rows.slice(0, filters.limit);
+    }
+    return rows;
     let query = db.select().from(schema.apis).where(and(...conditions));
 
     if (typeof filters.limit === 'number') {

src/repositories/userRepository.ts

@@ -1,5 +1,15 @@
 import prisma from '../lib/prisma.js';
 import type { User } from '../generated/prisma/client.js';
+import type { PaginationParams } from '../lib/pagination.js';
+
+export type UserListItem = Pick<User, 'id' | 'stellar_address' | 'created_at'>;
+
+interface FindUsersResult {
+  users: UserListItem[];
+  total: number;
+}
+
+export async function findUsers(params: PaginationParams): Promise<FindUsersResult> {
 
 interface PaginatedUsers {
   users: Pick<User, 'id' | 'stellar_address' | 'created_at'>[];
@@ -20,12 +30,15 @@ export async function findUsers(page: number, limit: number): Promise<PaginatedU
         created_at: true,
       },
       orderBy: { created_at: 'desc' },
+      skip: params.offset,
+      take: params.limit,
       skip,
       take: limit,
     }),
     prisma.user.count(),
   ]);
 
+  return { users, total };
   return {
     users,
     total,