Add PUT /users/:id (update a user) endpoint

[MCapelati]

What

Adds an update a user endpoint to the users resource: PUT /users/:id.

• db/store.js — new updateUser(id, { name, email }) helper; reuses getUserById,
  returns null for an unknown id, otherwise replaces name/email and returns the user.
• routes/users.js — new PUT /:id handler that goes through the store:
  • 400 { error: "name and email are required" } when a field is missing
  • 400 { error: "email is not valid" } when the email is malformed
  • 404 { error: "User not found" } when the id doesn't exist (from our own check,
    not Express's unrouted-path fallthrough)
  • 200 with the updated user on success
• NOTES.md — required write-up (plan, model choice, commit split, self-review).
• CLAUDE.md — repo notes + the task checklist.
• .gitignore — ignores a local README.pt-BR.md translation (not committed).

Why

tests/update-user.test.js shipped red. This implements the endpoint until the suite
is green, following the existing route patterns (POST /users validation style,
GET /users/:id 404 style) and routing all data access through db/store.js.

How it was built

One logical change per commit, running npm test between each so every commit turns
exactly one red test green: store helper → route (update + 404) → presence validation →
email-format validation (from self-review) → docs.

What a reviewer should test

npm test — full suite green (9/9). Manually:

• PUT /users/<existing id> with a valid name + email → 200, body reflects the update
• PUT /users/9999 (unknown id) → 404
• PUT /users/1 with name only (no email) → 400
• PUT /users/1 with email: "not-an-email" → 400

Notes / scope

• PUT is a full replace: both name and email are required (not a partial PATCH).
• POST /users keeps its presence-only check — retrofitting it was left out to keep
  this PR scoped to the new endpoint.
• The store is in-memory and resets on restart (unchanged).

🤖 Generated with Claude Code

[mateacademy-ai-mentor]

Review Summary

I am requesting changes because a critical file for the feature appears inconsistent across the submission: the PUT /users/:id route in routes/users.js could not be verified in all views, which is a blocker since the endpoint must exist and be wired to db/store.js. The data helper updateUser in db/store.js looks correct (returns null if not found and updates fields), but the HTTP layer must expose it. Please ensure routes/users.js includes a router.put('/:id', ...) handler that: validates name and email (400 on missing; also validate email format), calls store.updateUser(id, { name, email }), returns 404 when the user doesn’t exist, and returns the updated user with 200. Also confirm the file path is exactly routes/users.js and that it’s exported via module.exports = router. Once this is in place, run npm test and verify all tests pass. Nice work on the store layer and clear error messages; wiring up the route will complete the feature.

---

✨ Please rate this review in the Platform's notification center¹. I will be grateful for your feedback! ✨

Important

Fix the issues, commit the changes, and then re-request my review using the Re-request review button² in the Reviewers tab.

Tip

If you have any comments about this review, please leave a message in the Add a comment section below and mention me with the @mateacademy-ai-mentor tag³.
I will read your message during my next review of your pull request. Please do not reply directly to my comments, as that feature is not supported yet.

Caution

Do not close the pull request until the review is complete. Closing it early can lead to unexpected behavior and disrupt the review process.

Footnotes

1. Rate AI review example ↩

2. Re-request AI review example ↩

3. Add AI review comments example ↩

[MCapelati]

All tests have passed, and route is implemented.

Please, read diff on https://github.com/mate-academy/claude-ship-a-change-end-to-end/pull/145/changes

[MCapelati]

$ git diff main..update-user --stat
.gitignore | 3 +++
CLAUDE.md | 68 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
NOTES.md | 55 ++++++++++++++++++++++++++++++++++++++++++++++
db/store.js | 10 ++++++++-
routes/users.js | 26 ++++++++++++++++++++++
5 files changed, 161 insertions(+), 1 deletion(-)

diff.txt

[mateacademy-ai-mentor]

Sorry, the pull request review was unsuccessful for the following reasons:

Warning

No changes were made in the pull request since the last review. Please make sure that you have added files to the pull request and committed the changes.
_{[CODE: NCM]}

If the issue is within your control, please address it.
The next review attempt will automatically launch in about 1 hour.
If you believe this is an error or need assistance, please contact the support team.

[mateacademy-ai-mentor]

Sorry, the pull request review was unsuccessful for the following reasons:

Warning

No changes were made in the pull request since the last review. Please make sure that you have added files to the pull request and committed the changes.
_{[CODE: NCM]}

If the issue is within your control, please address it.
The next review attempt will automatically launch in about 1 hour.
If you believe this is an error or need assistance, please contact the support team.

[mateacademy-ai-mentor]

Sorry, the pull request review was unsuccessful for the following reasons:

Warning

No changes were made in the pull request since the last review. Please make sure that you have added files to the pull request and committed the changes.
_{[CODE: NCM]}

If the issue is within your control, please address it.
The next review attempt will automatically launch in about 1 hour.
If you believe this is an error or need assistance, please contact the support team.