feat: Implement user password change functionality

[pulashkir]

Overview: This PR implements the functionality to allow users to change their password, addressing the existing TODO in src/actions/user.ts.

Changes

• Replaced the password change stub in src/actions/user.ts with actual logic.
• Integrated bcryptjs for secure password hashing and comparison.
• Added validation for the current password before allowing a change.
• Ensured a clear Result type response for success and error scenarios.
• Updated error handling and unit test stubs in src/actions/user.ts to reflect the new implementation.

Summary by CodeRabbit

• Style
  • Updated error messages for improved clarity during password change operations.

[coderabbitai]

Walkthrough

The pull request updates two error message strings in the changePassword action within src/actions/user.ts. The messages are revised for clarity: "User not found or password not set" becomes "User not found or password not configured," and "Invalid current password" becomes "Incorrect current password."

Changes

Cohort / File(s)	Change Summary
Error message refinements src/actions/user.ts	Updated two error message strings in the changePassword function for improved clarity and consistency in user-facing feedback.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• feat: Implement comprehensive user profile system #39: Added scaffolding for the changePassword action in src/actions/user.ts that this PR now refines with updated error messaging.
• feat: Implement password change submit flow and validation #49: Modified the same changePassword function to which this PR applies error message string updates.

Suggested reviewers

• fehranbit

Poem

🐰 A message refined, so clear and so keen,
"Incorrect" now rings where "Invalid" had been,
With words more precise, the user shall see,
What went wrong with their password authenticity!
Small tweaks, large impact—the art of the phrase,
Hops onward to better, more user-kind ways! 🥕

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The pull request title "feat: Implement user password change functionality" is directly related to the changeset. The raw_summary shows modifications to the changePassword function in src/actions/user.ts, including updates to error messages within this functionality, which aligns with the stated objective of implementing user password change functionality. The title accurately reflects the main focus of the PR, which is the password change feature, even though the specific changes captured are error message refinements. The title is clear, concise, and specific rather than vague.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/implement-password-change

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

src/actions/user.ts (1)

50-50: LGTM! Error message clarity improved.

The change from "not set" to "not configured" is clearer and more user-friendly.

Optionally, you could split this into two separate error messages for better debugging clarity (e.g., distinguishing between "User not found" and "Password not configured"), but the current combined message is acceptable for user-facing errors.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7c83d68 and 6f64b7b.

📒 Files selected for processing (1)

• src/actions/user.ts (1 hunks)

🔇 Additional comments (1)

src/actions/user.ts (1)

56-56: LGTM! More user-friendly error message.

The change from "Invalid" to "Incorrect" is an improvement. "Incorrect" more clearly communicates that the password doesn't match, whereas "Invalid" could be misinterpreted as a formatting issue.