Add Cursor Rules Pack v2 β 7 sample production-tested rules#232
Add Cursor Rules Pack v2 β 7 sample production-tested rules#232oliviacraft wants to merge 4 commits intoPatrickJS:mainfrom
Conversation
|
No actionable comments were generated in the recent review. π βΉοΈ Recent review infoβοΈ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: π Files selected for processing (1)
π§ Files skipped from review as they are similar to previous changes (1)
π WalkthroughWalkthroughAdded a new "Cursor Rules Pack v2" sample directory with a Changes
Estimated code review effortπ― 2 (Simple) | β±οΈ ~10 minutes Suggested reviewers
Poem
π₯ Pre-merge checks | β 3β Passed checks (3 passed)
βοΈ Tip: You can configure your own custom pre-merge checks in the settings. β¨ Finishing Touchesπ§ͺ Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
π€ Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@README.md`:
- Line 164: Replace the external Cursor Rules Pack v2 link with the
repository-relative path to the in-repo rule (use the correct relative markdown
link for "Cursor Rules Pack v2") and move that list item so it is alphabetically
ordered within the "Mobile Development" section; locate the list item with the
text "Cursor Rules Pack v2" in README.md and update its URL to the in-repo
relative path and reposition it among the other Mobile Development entries to
maintain alphabetical order.
πͺ Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
βΉοΈ Review info
βοΈ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 40857b66-95d8-43c2-a1d2-a696b0c88a1a
π Files selected for processing (3)
README.mdrules/cursor-rules-pack-v2-cursorrules-prompt-file/.cursorrulesrules/cursor-rules-pack-v2-cursorrules-prompt-file/README.md
β¦ent alphabetically Addresses CodeRabbit review feedback: - Replace external GitHub link with relative in-repo path - Remove commercial CTA from description - Sort Mobile Development section alphabetically
|
Thanks for the review. This PR adds 7 production-tested Cursor rules covering TypeScript strict mode, Next.js App Router, Prisma patterns, and API security β with before/after examples for each rule. |
|
Hi @PatrickJS β this PR has been open for about 10 days. All checks pass and the CodeRabbit feedback was addressed. Would appreciate a review when you have a moment. Thanks! |
|
Hi @PatrickJS β following up again. It's been 14 days since this PR was opened. All CodeRabbit feedback was addressed on April 6th (relative path, no commercial CTA, alphabetical sorting). Checks are passing and the PR is mergeable. The 7 rules cover practical patterns: TypeScript strict mode, Next.js App Router, Prisma, API security β each with before/after examples. Happy to make any additional changes if needed. Thanks for maintaining this list! |
|
Friendly bump β this PR has been open 19 days and remains mergeable with no conflicts. Happy to adjust the format or content if anything doesn't fit the submission guidelines. Let me know! |
|
Hi! Just checking in on this PR β any updates on the review? Happy to make any adjustments if needed. The Cursor Rules Pack v2 has been tested in production across several real projects. Let me know if there is anything blocking the merge. Thanks! |
Cursor Rules Pack v2
Adds a new entry to the Packs section with 7 production-tested sample rules.
What's included
Folder:
rules/cursor-rules-pack-v2-cursorrules-prompt-file/.cursorrulesβ 7 rules ready to copy into any projectREADME.mdβ usage guide and link to full packThe 7 sample rules
Source
Summary by CodeRabbit