summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMaze Winther <mazewinther@gmail.com>2026-02-28 01:12:27 +0100
committerMaze Winther <mazewinther@gmail.com>2026-02-28 01:12:27 +0100
commita9e93471a74a3d1978278f09f35632d4f6588a76 (patch)
tree42e6b73147db5b990a9b2309ad78fb53ab121419
parent50885fb976f9a609c5f94359f548038fd44e2b3d (diff)
downloadopencut-a9e93471a74a3d1978278f09f35632d4f6588a76.tar.gz
review cursor command
-rw-r--r--.cursor/commands/review.md15
1 files changed, 15 insertions, 0 deletions
diff --git a/.cursor/commands/review.md b/.cursor/commands/review.md
index ec3e1a20..4361b397 100644
--- a/.cursor/commands/review.md
+++ b/.cursor/commands/review.md
@@ -147,3 +147,18 @@ Before outputting the review, list each checklist section and confirm you checke
- If something looks off but isn't covered by the checklist, you can mention it as a brief side note at the end — but keep it clearly separate from the actual review. Always default to fixing the issues covered by the checklist above, unless the user says otherwise.
> You WILL miss things if you try to review the whole file in one pass. Iterate rule by rule.
+
+---
+
+## Think Bigger
+
+After the checklist review, step back and ask the hard questions. The biggest architectural problems get solved by the biggest questions.
+
+- Does this abstraction actually need to exist? Could it be deleted entirely?
+- Is this the right layer for this logic? (wrong layer = future pain)
+- Is this solving a real problem, or a problem we invented?
+- Would a simpler data model make this whole file unnecessary?
+- Are we adding complexity to work around a bad decision made earlier?
+- Could this field be derived from other existing fields? Redundant data in a model is a source of bugs.
+
+Don't be shy about flagging these. A "why does this exist?" question is often worth more than 10 style fixes.