diff options
| author | Maze Winther <mazewinther@gmail.com> | 2026-02-28 01:12:27 +0100 |
|---|---|---|
| committer | Maze Winther <mazewinther@gmail.com> | 2026-02-28 01:12:27 +0100 |
| commit | a9e93471a74a3d1978278f09f35632d4f6588a76 (patch) | |
| tree | 42e6b73147db5b990a9b2309ad78fb53ab121419 | |
| parent | 50885fb976f9a609c5f94359f548038fd44e2b3d (diff) | |
| download | opencut-a9e93471a74a3d1978278f09f35632d4f6588a76.tar.gz | |
review cursor command
| -rw-r--r-- | .cursor/commands/review.md | 15 |
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. |
