Cursor rule

1---
2description: Quality standards that every contribution must meet before PR submission
3globs: ["contributions/**", "scripts/**"]
4alwaysApply: false
5---
6 
7# Contribution Quality Standards
8 
9## Pre-Submit Checklist (ALL must pass)
10 
111. **Guidelines Compliance**
12   - CONTRIBUTING.md read and all rules followed
13   - PR template filled completely (every section, no placeholders)
14   - Branch naming matches repo convention
15   - Commit messages match repo format exactly
16 
172. **Code Quality**
18   - Matches repo's exact code style (indentation, naming, imports)
19   - Uses repo's existing abstractions and utilities
20   - Proper error handling following repo patterns
21   - No hardcoded values that should be configurable
22   - No commented-out code unless repo does this
23 
243. **Testing**
25   - All existing tests pass
26   - New tests added for new/changed behavior
27   - Tests follow repo's testing patterns exactly
28   - Edge cases covered (especially ones competing PRs missed)
29 
304. **Tooling**
31   - Linter passes with zero errors (repo's config, not defaults)
32   - Formatter has been run (repo's formatter)
33   - Type checker passes (if typed language)
34   - Build succeeds
35 
365. **Security**
37   - No secrets, API keys, tokens in code
38   - No personal file paths or system-specific references
39   - No PII (personal identifiable information)
40   - Dependencies are from official sources only
41 
426. **Diff Quality**
43   - Minimal diff -- only changes necessary for the fix/feature
44   - No unrelated formatting changes
45   - No whitespace-only changes outside the fix area
46   - Clear and atomic -- one concern per commit
47 
48## Competitive Advantage Requirements
49 
50Before submitting, verify your PR is SUPERIOR to all competing PRs:
51- Addresses issues that competing PRs ignored
52- Includes tests that competing PRs skipped
53- Follows conventions that competing PRs violated
54- Has a more complete/detailed PR description
55- Handles edge cases others missed
56 
57## PR Description Template (adapt to repo's template)
58 
59If the repo has no template, use this minimum:
60 
61```markdown
62## Summary
63{One paragraph explaining what this PR does and why}
64 
65## Changes
66- {Bullet list of specific changes}
67 
68## Testing
69- {How you tested this}
70- {New tests added}
71 
72## Related Issues
73Fixes #{issue_number}
74 
75## Checklist
76- [ ] Tests pass
77- [ ] Linting passes
78- [ ] Documentation updated (if applicable)
79- [ ] Breaking changes: None / {described}
80```
81