Bug fix

NOTES.md

@@ -0,0 +1,3 @@
+I have plan to fix all the errors. I've already made modifications for all the issues found.
+I used Sonnet model, because my task wasn't very complex. I didn't split commits by each issue. Rather I've commited all at once.
+Review caught several issues. I fixed all of them.  

db/store.js

@@ -24,4 +24,20 @@ function createUser({ name, email }) {
   return user;
 }
 
-module.exports = { getAllUsers, getUserById, createUser };
+function putUserById(id, { name, email }) {
+  // Find the existing user by id
+  const user = users.find(u => u.id === id);
+
+  // If the user doesn't exist, you can return null or handle the error
+  if (!user) {
+    return null;
+  }
+
+  // Update the existing user's properties
+  user.name = name;
+  user.email = email;
+
+  return user;
+}
+
+module.exports = { getAllUsers, getUserById, createUser, putUserById };

routes/users.js

@@ -32,4 +32,22 @@ router.post("/", (req, res) => {
   res.status(201).json(user);
 });
 
+// PUT /users/:id — Update a single user, or 404 if it doesn't exist
+router.put("/:id", (req, res) => {
+  const id = Number(req.params.id);
+  const { name, email } = req.body;
+
+  if (!name || !email) {
+    return res.status(400).json({ error: "Name and email are required" });
+  }
+
+  const user = store.putUserById(id, { name, email });
+
+   if (!user) {
+    return res.status(404).json({ error: "User not found" });
+  }
+
+  res.json(user);
+});
+
 module.exports = router;