Comments

A standard for communicating constructive feedback clearly and explicitly.

praise

Highlights something positive. Try to leave at least one of these per review.

praise: Great job handling that edge case! This logic is much cleaner.

nitpick

Small, non-blocking requests like typos, formatting, or variable naming.

nitpick: This variable name is a bit ambiguous. Maybe 'userCount'?

suggestion

Specific improvements to the current implementation. Explain the 'why'.

suggestion: We could memoize this value to prevent unnecessary re-renders.

issue

Highlights a bug, error, or oversight that must be addressed before merging.

issue: This will throw an error if the user is not authenticated.

question

Asking for clarification. Does not necessarily require a code change.

question: Is this timeout value sufficient for slower networks?

thought

Idea that popped up. Not blocking. Good for 'nice to have' or future refactors.

thought: In the future, we might want to move this to a utility.

chore

Maintenance tasks like updating dependencies, comments, or definitions.

chore: Can we update the JSDoc to reflect the new parameter?