Add error handling for user creation and improve server initialization

Author: upayanmazumder

api/controllers/createUser.js

@@ -1,9 +1,17 @@
 const User = require("../models/User.js");
 
 const createUser = async (repos) => {
-  const username = repos[0].owner.login;
-  const githubId = repos[0].owner.id;
-  const avatarUrl = repos[0].owner.avatar_url;
+  if (!repos || repos.length === 0) {
+    throw new Error("No repositories found for the user.");
+  }
+
+  const username = repos[0]?.owner?.login;
+  const githubId = repos[0]?.owner?.id;
+  const avatarUrl = repos[0]?.owner?.avatar_url;
+
+  if (!username || !githubId || !avatarUrl) {
+    throw new Error("Owner information is missing in the repository data.");
+  }
 
   const existingUser = await User.findOne({ username });
 
@@ -12,7 +20,6 @@ const createUser = async (repos) => {
     return existingUser;
   }
 
-  // Create new user
   const newUser = await User.create({
     githubId,
     username,
@@ -21,10 +28,11 @@ const createUser = async (repos) => {
       name: repo.name,
       url: repo.html_url,
     })),
-    accessToken: "", // Store access token if needed
+    accessToken: "",
   });
 
   console.log("✅ New user created with username:", username);
+  return newUser;
 };
 
 module.exports = createUser;

api/server.js

@@ -13,33 +13,37 @@ const analyzeRoutes = require("./routes/analyze");
 
 const app = express();
 
-app.use(cors());
-app.use(express.json());
-app.use(
-  session({
-    secret: process.env.SESSION_SECRET,
-    resave: false,
-    saveUninitialized: true,
-  })
-);
-app.use(passport.initialize());
-app.use(passport.session());
+try {
+  app.use(cors());
+  app.use(express.json());
+  app.use(
+    session({
+      secret: process.env.SESSION_SECRET,
+      resave: false,
+      saveUninitialized: true,
+    })
+  );
+  app.use(passport.initialize());
+  app.use(passport.session());
 
-app.use("/auth", authRoutes);
-app.use("/github", githubRoutes);
-app.use("/", analyzeRoutes);
+  app.use("/auth", authRoutes);
+  app.use("/github", githubRoutes);
+  app.use("/", analyzeRoutes);
 
-mongoose
-  .connect(process.env.MONGO_URI)
-  .then(() => console.log("MongoDB Connected"))
-  .catch((err) => console.error("MongoDB Connection Error:", err));
+  mongoose
+    .connect(process.env.MONGO_URI)
+    .then(() => console.log("MongoDB Connected"))
+    .catch((err) => console.error("MongoDB Connection Error:", err));
 
-const PORT = process.env.API_PORT || 5000;
-app.listen(PORT, () => {
-  console.log(`Server running on http://localhost:${PORT}`);
-  console.log(
-    `GitHub API status: ${
-      process.env.GITHUB_TOKEN ? "Authenticated" : "Not configured"
-    }`
-  );
-});
+  const PORT = process.env.API_PORT || 5000;
+  app.listen(PORT, () => {
+    console.log(`Server running on http://localhost:${PORT}`);
+    console.log(
+      `GitHub API status: ${
+        process.env.GITHUB_TOKEN ? "Authenticated" : "Not configured"
+      }`
+    );
+  });
+} catch (error) {
+  console.error("An error occurred while starting the server:", error);
+}