Chore/data transformation layer (#153)

* chore: adding encoding and decoding layers for frontend and backend

* fixing linting issues

* formatting and adding face_expression_model-weights_manifest to the git ignore

* adding test cases and ensuring that the feature works

* updating test scripts with "jest --detectOpenHandles --forceExit " to avoid hanging workflows

* bug fixes

* updating learner side and tests

* bug fixes

* updates

Author: MmagdyHafezZ

.gitignore

@@ -70,7 +70,7 @@ dev.op.env
 */mock.jwt.cookie.auth.guard.ts
 
 # manifest files
-face_expresstion_model-weights_manifest.json
+face_expression_model-weights_manifest.json
 tiny_face_detector_model-weights_manifest.json
 
 *.env

apps/api-gateway/package.json

@@ -17,7 +17,7 @@
     "start:debug": "yarn start --debug --watch",
     "start:dev": "yarn start --watch",
     "start:prod": "node dist/main",
-    "test": "jest",
+    "test": "jest --detectOpenHandles --forceExit",
     "test:cov": "jest --coverage",
     "test:debug": "node --inspect-brk -r tsconfig-paths/register -r ts-node/register node_modules/.bin/jest --runInBand",
     "test:e2e": "jest --config ./test/jest-e2e.json",

apps/api/package.json

@@ -17,7 +17,7 @@
     "start:debug": "yarn start --debug --watch",
     "start:dev": "yarn start --watch",
     "start:prod": "node dist/main",
-    "test": "jest",
+    "test": "jest --detectOpenHandles --forceExit",
     "test:cov": "jest --coverage",
     "test:debug": "node --inspect-brk -r tsconfig-paths/register -r ts-node/register node_modules/.bin/jest --runInBand",
     "test:e2e": "jest --config ./test/jest-e2e.json",

apps/api/src/api/assignment/attempt/dto/assignment-attempt/get.assignment.attempt.response.dto.ts

@@ -51,6 +51,14 @@ export class AssignmentAttemptResponseDto {
     required: false,
   })
   expiresAt: Date | null;
+
+  @ApiProperty({
+    description: "The DateTime at which the attempt was created",
+    type: Date,
+    example: "2023-12-31T10:00:00Z",
+    required: true,
+  })
+  createdAt: Date;
 }
 
 export class GetAssignmentAttemptResponseDto extends AssignmentAttemptResponseDto {

apps/api/src/api/assignment/v2/controllers/version-management.controller.ts

@@ -89,13 +89,6 @@ export class VersionManagementController {
     @Param("assignmentId", ParseIntPipe) assignmentId: number,
     @Req() request: UserSessionRequest,
   ): Promise<VersionSummary[]> {
-    console.log(
-      "Received assignmentId:",
-      assignmentId,
-      "type:",
-      typeof assignmentId,
-    );
-
     if (!request?.userSession) {
       throw new Error("User session is required");
     }
@@ -274,12 +267,6 @@ export class VersionManagementController {
     @Param("assignmentId", ParseIntPipe) assignmentId: number,
     @Req() request: UserSessionRequest,
   ) {
-    console.log(
-      "getVersionHistory called with assignmentId:",
-      assignmentId,
-      "type:",
-      typeof assignmentId,
-    );
     return await this.versionService.getVersionHistory(
       assignmentId,
       request.userSession,

apps/api/src/api/llm/features/grading/services/presentation-grading.service.ts

@@ -161,7 +161,6 @@ export class PresentationGradingService implements IPresentationGradingService {
     try {
       // Parse the LLM output to get points & feedback
       const parsedResponse = await parser.parse(response);
-      console.log("Parsed Response:", parsedResponse);
 
       // Combine the AEEG components into comprehensive feedback
       const aeegFeedback = `

apps/api/src/api/user/controllers/chat.controller.ts

@@ -30,10 +30,17 @@ export class ChatController {
 
   @Post("today")
   @UseGuards(ChatAccessControlGuard)
-  async getTodayChat(@Body() body: { userId: string; assignmentId?: number }) {
+  async getTodayChat(@Body() body: { header: string; body: string }) {
+    let newBody: { userId: string; assignmentId?: number };
+    if (typeof body.body === "string") {
+      newBody = JSON.parse(body.body) as {
+        userId: string;
+        assignmentId?: number;
+      };
+    }
     return this.chatService.getOrCreateTodayChat(
-      body.userId,
-      body.assignmentId,
+      newBody.userId,
+      newBody.assignmentId,
     );
   }
 

apps/api/src/app.module.ts

@@ -18,6 +18,7 @@ import { HealthModule } from "./health/health.module";
 import { winstonOptions } from "./logger/config";
 import { LoggerMiddleware } from "./logger/logger.middleware";
 import { MessagingModule } from "./messaging/messaging.module";
+import { DataTransformMiddleware } from "./middleware/data-transform.middleware";
 import { routes } from "./routes";
 
 @Module({
@@ -38,7 +39,7 @@ import { routes } from "./routes";
 export class AppModule implements NestModule {
   configure(consumer: MiddlewareConsumer) {
     consumer
-      .apply(LoggerMiddleware)
+      .apply(DataTransformMiddleware, LoggerMiddleware)
       .forRoutes({ path: "*", method: RequestMethod.ALL })
       .apply(UserSessionMiddleware)
       .forRoutes(

apps/api/src/common/common.module.ts

@@ -0,0 +1,18 @@
+import { Module, Global } from "@nestjs/common";
+import { APP_INTERCEPTOR } from "@nestjs/core";
+import { DataTransformInterceptor } from "./interceptors/data-transform.interceptor";
+
+/**
+ * Global module for common functionality including data transformation
+ */
+@Global()
+@Module({
+  providers: [
+    {
+      provide: APP_INTERCEPTOR,
+      useClass: DataTransformInterceptor,
+    },
+  ],
+  exports: [DataTransformInterceptor],
+})
+export class CommonModule {}