summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--.cursor/commands/review.md10
1 files changed, 10 insertions, 0 deletions
diff --git a/.cursor/commands/review.md b/.cursor/commands/review.md
index f538e9da..d7c0c45c 100644
--- a/.cursor/commands/review.md
+++ b/.cursor/commands/review.md
@@ -91,6 +91,16 @@ Review every point below carefully to ensure files follow consistent code style
- [ ] Code is scannable — use variables and helper functions to make intent clear at a glance
- [ ] Complex logic is extracted into well-named variables or helpers
+- [ ] No redundant single/plural function variants — if a function can operate on multiple items, it should accept an array and handle both cases. Don't create `doThing()` + `doThings()`.
+
+ ```tsx
+ // ❌ wrong — redundant variants
+ function updateElement({ element }: { element: Element }) { ... }
+ function updateElements({ elements }: { elements: Element[] }) { ... }
+
+ // ✅ correct — one function, accepts array
+ function updateElements({ elements }: { elements: Element[] }) { ... }
+ ```
---