impr: use tsrest for dev endpoints (@fehmer)

backend/scripts/openapi.ts

@@ -105,6 +105,12 @@ export function getOpenApi(): OpenAPIObject {
           "x-displayName": "Server configuration",
           "x-public": "yes",
         },
+        {
+          name: "dev",
+          description:
+            "Development related endpoints. Only available on dev environment",
+          "x-displayName": "Development",
+        },
       ],
     },
 

backend/src/api/controllers/dev.ts

@@ -1,4 +1,4 @@
-import { MonkeyResponse } from "../../utils/monkey-response";
+import { MonkeyResponse2 } from "../../utils/monkey-response";
 import * as UserDal from "../../dal/user";
 import FirebaseAdmin from "../../init/firebase-admin";
 import Logger from "../../utils/logger";
@@ -15,10 +15,14 @@ import {
   PersonalBest,
   PersonalBests,
 } from "@monkeytype/contracts/schemas/shared";
+import {
+  GenerateDataRequest,
+  GenerateDataResponse,
+} from "@monkeytype/contracts/dev";
 
 type GenerateDataOptions = {
-  firstTestTimestamp: Date;
-  lastTestTimestamp: Date;
+  firstTestTimestamp: Date | number;
+  lastTestTimestamp: Date | number;
   minTestsPerDay: number;
   maxTestsPerDay: number;
 };
@@ -31,8 +35,8 @@ const CREATE_RESULT_DEFAULT_OPTIONS: GenerateDataOptions = {
 };
 
 export async function createTestData(
-  req: MonkeyTypes.Request
-): Promise<MonkeyResponse> {
+  req: MonkeyTypes.Request2<undefined, GenerateDataRequest>
+): Promise<GenerateDataResponse> {
   const { username, createUser } = req.body;
   const user = await getOrCreateUser(username, "password", createUser);
 
@@ -42,7 +46,7 @@ export async function createTestData(
   await updateUser(uid);
   await updateLeaderboard();
 
-  return new MonkeyResponse("test data created", { uid, email }, 200);
+  return new MonkeyResponse2("test data created", { uid, email });
 }
 
 async function getOrCreateUser(

backend/src/api/routes/dev.ts

@@ -1,35 +1,22 @@
-import { Router } from "express";
-import joi from "joi";
-import { createTestData } from "../controllers/dev";
-import { isDevEnvironment } from "../../utils/misc";
+import { devContract } from "@monkeytype/contracts/dev";
+import { initServer } from "@ts-rest/express";
 import { validate } from "../../middlewares/configuration";
-import { validateRequest } from "../../middlewares/validation";
-import { asyncHandler } from "../../middlewares/utility";
-
-const router = Router();
+import { isDevEnvironment } from "../../utils/misc";
+import * as DevController from "../controllers/dev";
+import { callController } from "../ts-rest-adapter";
 
-router.use(
-  validate({
-    criteria: () => {
-      return isDevEnvironment();
-    },
-    invalidMessage: "Development endpoints are only available in DEV mode.",
-  })
-);
+const onlyAvailableOnDev = validate({
+  criteria: () => {
+    return isDevEnvironment();
+  },
+  invalidMessage: "Development endpoints are only available in DEV mode.",
+});
 
-router.post(
-  "/generateData",
-  validateRequest({
-    body: {
-      username: joi.string().required(),
-      createUser: joi.boolean().optional(),
-      firstTestTimestamp: joi.number().optional(),
-      lastTestTimestamp: joi.number().optional(),
-      minTestsPerDay: joi.number().optional(),
-      maxTestsPerDay: joi.number().optional(),
-    },
-  }),
-  asyncHandler(createTestData)
-);
+const s = initServer();
 
-export default router;
+export default s.router(devContract, {
+  generateData: {
+    middleware: [onlyAvailableOnDev],
+    handler: async (r) => callController(DevController.createTestData)(r),
+  },
+});

backend/src/api/routes/index.ts

@@ -57,6 +57,7 @@ const router = s.router(contract, {
   leaderboards,
   results,
   configuration,
+  dev,
 });
 
 export function addApiRoutes(app: Application): void {
@@ -139,9 +140,6 @@ function applyDevApiRoutes(app: Application): void {
       }
       next();
     });
-
-    //enable dev edpoints
-    app.use("/dev", dev);
   }
 }
 

frontend/src/ts/ape/endpoints/index.ts

@@ -1,9 +1,7 @@
 import Quotes from "./quotes";
 import Users from "./users";
-import Dev from "./dev";
 
 export default {
   Quotes,
   Users,
-  Dev,
 };

frontend/src/ts/ape/index.ts

@@ -3,20 +3,22 @@ import { buildHttpClient } from "./adapters/axios-adapter";
 import { envConfig } from "../constants/env-config";
 import { buildClient } from "./adapters/ts-rest-adapter";
 import { contract } from "@monkeytype/contracts";
+import { devContract } from "@monkeytype/contracts/dev";
 
 const API_PATH = "";
 const BASE_URL = envConfig.backendUrl;
 const API_URL = `${BASE_URL}${API_PATH}`;
 
 const httpClient = buildHttpClient(API_URL, 10_000);
 const tsRestClient = buildClient(contract, BASE_URL, 10_000);
+const devClient = buildClient(devContract, BASE_URL, 240_000);
 
 // API Endpoints
 const Ape = {
   ...tsRestClient,
   users: new endpoints.Users(httpClient),
   quotes: new endpoints.Quotes(httpClient),
-  dev: new endpoints.Dev(buildHttpClient(API_URL, 240_000)),
+  dev: devClient,
 };
 
 export default Ape;

frontend/src/ts/modals/simple-modals.ts

@@ -34,6 +34,7 @@ import {
   TextInput,
 } from "../utils/simple-modal";
 import { ShowOptions } from "../utils/animated-modal";
+import { GenerateDataRequest } from "@monkeytype/contracts/dev";
 
 type PopupKey =
   | "updateEmail"
@@ -1308,7 +1309,7 @@ list.devGenerateData = new SimpleModal({
     minTestsPerDay,
     maxTestsPerDay
   ): Promise<ExecReturn> => {
-    const request: Ape.Dev.GenerateData = {
+    const request: GenerateDataRequest = {
       username,
       createUser: createUser === "true",
     };
@@ -1321,11 +1322,11 @@ list.devGenerateData = new SimpleModal({
     if (maxTestsPerDay !== undefined && maxTestsPerDay.length > 0)
       request.maxTestsPerDay = Number.parseInt(maxTestsPerDay);
 
-    const result = await Ape.dev.generateData(request);
+    const result = await Ape.dev.generateData({ body: request });
 
     return {
       status: result.status === 200 ? 1 : -1,
-      message: result.message,
+      message: result.body.message,
       hideOptions: {
         clearModalChain: true,
       },

packages/contracts/src/dev.ts

@@ -0,0 +1,58 @@
+import { initContract } from "@ts-rest/core";
+import { z } from "zod";
+import {
+  CommonResponses,
+  EndpointMetadata,
+  responseWithData,
+} from "./schemas/api";
+import { IdSchema } from "./schemas/util";
+
+export const GenerateDataRequestSchema = z.object({
+  username: z.string(),
+  createUser: z
+    .boolean()
+    .optional()
+    .describe(
+      "If `true` create user with <username>@example.com and password `password`. If false user has to exist."
+    ),
+  firstTestTimestamp: z.number().int().nonnegative().optional(),
+  lastTestTimestamp: z.number().int().nonnegative().optional(),
+  minTestsPerDay: z.number().int().nonnegative().optional(),
+  maxTestsPerDay: z.number().int().nonnegative().optional(),
+});
+export type GenerateDataRequest = z.infer<typeof GenerateDataRequestSchema>;
+
+export const GenerateDataResponseSchema = responseWithData(
+  z.object({
+    uid: IdSchema,
+    email: z.string().email(),
+  })
+);
+export type GenerateDataResponse = z.infer<typeof GenerateDataResponseSchema>;
+
+const c = initContract();
+export const devContract = c.router(
+  {
+    generateData: {
+      summary: "generate data",
+      description: "Generate test results for the given user.",
+      method: "POST",
+      path: "/generateData",
+      body: GenerateDataRequestSchema.strict(),
+      responses: {
+        200: GenerateDataResponseSchema,
+      },
+    },
+  },
+  {
+    pathPrefix: "/dev",
+    strictStatusCodes: true,
+    metadata: {
+      openApiTags: "dev",
+      authenticationOptions: {
+        isPublic: true,
+      },
+    } as EndpointMetadata,
+    commonResponses: CommonResponses,
+  }
+);

packages/contracts/src/index.ts

@@ -8,6 +8,7 @@ import { publicContract } from "./public";
 import { leaderboardsContract } from "./leaderboards";
 import { resultsContract } from "./results";
 import { configurationContract } from "./configuration";
+import { devContract } from "./dev";
 
 const c = initContract();
 
@@ -21,4 +22,5 @@ export const contract = c.router({
   leaderboards: leaderboardsContract,
   results: resultsContract,
   configuration: configurationContract,
+  dev: devContract,
 });

packages/contracts/src/schemas/api.ts

@@ -9,7 +9,8 @@ export type OpenApiTag =
   | "public"
   | "leaderboards"
   | "results"
-  | "configuration";
+  | "configuration"
+  | "dev";
 
 export type EndpointMetadata = {
   /** Authentication options, by default a bearer token is required. */
@@ -18,7 +19,7 @@ export type EndpointMetadata = {
 };
 
 export type RequestAuthenticationOptions = {
-  /** Endpoint is accessible without any authentication. If `false` bearer authentication is required. */
+  /** Endpoint is accessible without any authentication. If `false` bearer authentication is requuired. */
   isPublic?: boolean;
   /** Endpoint is accessible with ape key authentication in  _addition_ to the bearer authentication. */
   acceptApeKeys?: boolean;