enable select queries

Author: devksingh4

src/api/routes/roomRequests.ts

@@ -30,6 +30,11 @@ import { FastifyZodOpenApiTypeProvider } from "fastify-zod-openapi";
 import { z } from "zod";
 import { buildAuditLogTransactPut } from "api/functions/auditLog.js";
 import { Modules } from "common/modules.js";
+import {
+  generateProjectionParams,
+  getDefaultFilteringQuerystring,
+  nonEmptyCommaSeparatedStringSchema,
+} from "common/utils.js";
 
 const roomRequestRoutes: FastifyPluginAsync = async (fastify, _options) => {
   await fastify.register(rateLimiter, {
@@ -182,12 +187,19 @@ const roomRequestRoutes: FastifyPluginAsync = async (fastify, _options) => {
               example: "sp25",
             }),
           }),
+          querystring: z.object(
+            getDefaultFilteringQuerystring({
+              defaultSelect: ["requestId", "title"],
+            }),
+          ),
         }),
       ),
       onRequest: fastify.authorizeFromSchema,
     },
     async (request, reply) => {
       const semesterId = request.params.semesterId;
+      const { ProjectionExpression, ExpressionAttributeNames } =
+        generateProjectionParams({ userFields: request.query.select });
       if (!request.username) {
         throw new InternalServerError({
           message: "Could not retrieve username.",
@@ -198,7 +210,8 @@ const roomRequestRoutes: FastifyPluginAsync = async (fastify, _options) => {
         command = new QueryCommand({
           TableName: genericConfig.RoomRequestsTableName,
           KeyConditionExpression: "semesterId = :semesterValue",
-          ProjectionExpression: "requestId, host, title, semester",
+          ProjectionExpression,
+          ExpressionAttributeNames,
           ExpressionAttributeValues: {
             ":semesterValue": { S: semesterId },
           },
@@ -210,8 +223,9 @@ const roomRequestRoutes: FastifyPluginAsync = async (fastify, _options) => {
             "semesterId = :semesterValue AND begins_with(#sortKey, :username)",
           ExpressionAttributeNames: {
             "#sortKey": "userId#requestId",
+            ...ExpressionAttributeNames,
           },
-          ProjectionExpression: "requestId, host, title, semester",
+          ProjectionExpression,
           ExpressionAttributeValues: {
             ":semesterValue": { S: semesterId },
             ":username": { S: request.username },

src/common/utils.ts

@@ -1,3 +1,4 @@
+import { z } from "zod";
 export function transformCommaSeperatedName(name: string) {
   if (name.includes(",")) {
     try {
@@ -12,3 +13,42 @@ export function transformCommaSeperatedName(name: string) {
   }
   return name;
 }
+
+type GenerateProjectionParamsInput = {
+  userFields?: string[];
+}
+/**
+ * Generates DynamoDB projection parameters for select filters, while safely handle reserved keywords.
+ */
+export const generateProjectionParams = ({ userFields }: GenerateProjectionParamsInput) => {
+  const attributes = userFields || [];
+  const expressionAttributeNames: Record<string, string> = {};
+  const projectionExpression = attributes
+    .map((attr, index) => {
+      const placeholder = `#proj${index}`;
+      expressionAttributeNames[placeholder] = attr;
+      return placeholder;
+    })
+    .join(',');
+  return {
+    ProjectionExpression: projectionExpression,
+    ExpressionAttributeNames: expressionAttributeNames,
+  };
+};
+
+
+export const nonEmptyCommaSeparatedStringSchema = z.preprocess(
+  (val) => String(val).split(',').map(item => item.trim()),
+  z.array(z.string()).nonempty()
+);
+
+type GettDefaultFilteringQuerystringInput = {
+  defaultSelect: string[];
+}
+export const getDefaultFilteringQuerystring = ({ defaultSelect }: GettDefaultFilteringQuerystringInput) => {
+  return {
+    select: z.optional(nonEmptyCommaSeparatedStringSchema).default(defaultSelect.join(',')).openapi({
+      description: "Comma-seperated list of attributes to return",
+    })
+  }
+}

src/ui/pages/roomRequest/RoomRequestLanding.page.tsx

@@ -11,6 +11,7 @@ import {
   RoomRequestFormValues,
   RoomRequestGetAllResponse,
   RoomRequestPostResponse,
+  type RoomRequestStatus,
 } from "@common/types/roomRequest";
 
 export const ManageRoomRequestsPage: React.FC = () => {
@@ -29,8 +30,15 @@ export const ManageRoomRequestsPage: React.FC = () => {
   const getRoomRequests = async (
     semester: string,
   ): Promise<RoomRequestGetAllResponse> => {
-    const response = await api.get(`/api/v1/roomRequests/${semester}`);
-    return response.data;
+    const response = await api.get<
+      {
+        requestId: string;
+        title: string;
+        host: string;
+        status: RoomRequestStatus;
+      }[]
+    >(`/api/v1/roomRequests/${semester}?select=requestId,title,host,status`);
+    return response.data.map((x) => ({ ...x, semester }));
   };
 
   useEffect(() => {