Update to Express 5 and fix breaking changes

Author: cbullinger

mflix/server/js-express/package.json

@@ -29,9 +29,9 @@
   },
   "devDependencies": {
     "@types/cors": "^2.8.19",
-    "@types/express": "^4.17.25",
+    "@types/express": "^5.0.6",
     "@types/jest": "^30.0.0",
-    "@types/node": "^25.2.1",
+    "@types/node": "^25.2.2",
     "@types/supertest": "^6.0.3",
     "@types/swagger-jsdoc": "^6.0.4",
     "@types/swagger-ui-express": "^4.1.8",

mflix/server/js-express/src/controllers/movieController.ts

@@ -172,8 +172,8 @@ export async function getDistinctGenres(
 export async function getMovieById(req: Request, res: Response): Promise<void> {
   const { id } = req.params;
 
-  // Validate ObjectId format
-  if (!ObjectId.isValid(id)) {
+  // Validate id is a string and ObjectId format
+  if (typeof id !== "string" || !ObjectId.isValid(id)) {
     res
       .status(400)
       .json(
@@ -305,8 +305,8 @@ export async function updateMovie(req: Request, res: Response): Promise<void> {
   const { id } = req.params;
   const updateData: UpdateMovieRequest = req.body;
 
-  // Validate ObjectId format
-  if (!ObjectId.isValid(id)) {
+  // Validate id is a string and ObjectId format
+  if (typeof id !== "string" || !ObjectId.isValid(id)) {
     res
       .status(400)
       .json(
@@ -426,8 +426,8 @@ export async function updateMoviesBatch(
 export async function deleteMovie(req: Request, res: Response): Promise<void> {
   const { id } = req.params;
 
-  // Validate ObjectId format
-  if (!ObjectId.isValid(id)) {
+  // Validate id is a string and ObjectId format
+  if (typeof id !== "string" || !ObjectId.isValid(id)) {
     res
       .status(400)
       .json(
@@ -521,8 +521,8 @@ export async function findAndDeleteMovie(
 ): Promise<void> {
   const { id } = req.params;
 
-  // Validate ObjectId format
-  if (!ObjectId.isValid(id)) {
+  // Validate id is a string and ObjectId format
+  if (typeof id !== "string" || !ObjectId.isValid(id)) {
     res
       .status(400)
       .json(