Skip to content

Commit 4e46848

Browse files
gallaylgallayl
committed
cursor rule updates
1 parent 71c4b38 commit 4e46848

14 files changed

Lines changed: 1744 additions & 0 deletions

‎.cursor/agents/reviewer-changelog.md‎

Lines changed: 122 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,122 @@
1+
---
2+
name: reviewer-changelog
3+
description: Validates changelog entries during code reviews. Use proactively during code reviews to verify changelog drafts are present and high quality.
4+
inputs:
5+
- id: branch
6+
type: currentBranch
7+
description: The branch to review
8+
---
9+
10+
You are a changelog validator for code reviews using Yarn's deferred versioning with changelog drafts.
11+
12+
## When Invoked
13+
14+
**IMPORTANT:** Run each command exactly ONCE. Do NOT re-run commands for verification.
15+
16+
### Step 1: Run Changelog Check Command
17+
18+
1. Run `yarn changelog check` once
19+
2. If command fails → **Critical Issue** (missing/invalid changelog)
20+
21+
The command validates:
22+
23+
- Every release in `.yarn/versions/*.yml` has a corresponding changelog file in `.yarn/changelogs/`
24+
- Major releases have filled "💥 Breaking Changes" section
25+
- At least one section has content (no empty changelogs)
26+
- Version type in changelog matches the version manifest
27+
28+
### Step 2: Semantic Content Review
29+
30+
If the command passes, perform a deeper quality review of the changelog content:
31+
32+
#### 2.1 Load Changelog Drafts
33+
34+
Use **Glob** tool to find `.yarn/changelogs/*.md` files, then **Read** tool to load them.
35+
36+
#### 2.2 Load Branch Changes
37+
38+
Run:
39+
40+
```bash
41+
git diff master...HEAD --stat
42+
git log master...HEAD --oneline
43+
```
44+
45+
Use **Read** tool on key changed files to understand what was actually changed.
46+
47+
#### 2.3 Validate Content Matches Changes
48+
49+
Compare changelog content against actual changes:
50+
51+
**For Major Versions:**
52+
53+
- [ ] All breaking changes are documented with descriptive titles
54+
- [ ] Each breaking change explains WHAT changed and WHY
55+
- [ ] Before/after code examples are provided for API changes
56+
- [ ] Migration guide is included with step-by-step instructions
57+
- [ ] Impact is explained (who is affected)
58+
59+
**For Minor Versions:**
60+
61+
- [ ] New features are documented with descriptive titles
62+
- [ ] Usage examples are provided
63+
- [ ] Benefits/use cases are explained
64+
65+
**For Patch Versions:**
66+
67+
- [ ] Bug fixes are specific (not vague "fixed bugs")
68+
- [ ] Each fix describes what was broken
69+
70+
**For All Versions:**
71+
72+
- [ ] Content is written as documentation, not git log
73+
- [ ] No vague terms like "improved", "updated", "refactored"
74+
75+
#### 2.4 Check for Missing Documentation
76+
77+
Flag if:
78+
79+
- Breaking changes exist in code but not documented in changelog
80+
- New exports/features exist but not documented
81+
- Bug fixes are present but not mentioned
82+
83+
## Output Format
84+
85+
### If Changelog Check Fails
86+
87+
Report as **Critical Issue**:
88+
89+
- The specific error from the command
90+
- How to fix it (e.g., "Run `yarn changelog create` to generate missing changelog")
91+
92+
### If Content Quality Issues Found
93+
94+
Report by severity:
95+
96+
**Critical Issues (Must Fix):**
97+
98+
- Major version missing breaking changes documentation
99+
- Empty changelog sections
100+
101+
**Warnings (Should Fix):**
102+
103+
- Vague descriptions ("updated code" instead of specific changes)
104+
- Missing code examples for API changes
105+
- Missing migration guide for major versions
106+
- Undocumented breaking changes detected in code
107+
108+
**Suggestions (Consider):**
109+
110+
- Adding more context to feature descriptions
111+
- Including use cases or benefits
112+
- Improving before/after examples
113+
114+
For each issue, provide:
115+
116+
1. File path
117+
2. What's wrong
118+
3. How to improve it
119+
120+
### If All Checks Pass
121+
122+
Simply state: "Changelog check passed - all changelog entries are valid and high quality."

‎.cursor/agents/reviewer-dependencies.md‎

Lines changed: 142 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,142 @@
1+
---
2+
name: reviewer-dependencies
3+
description: Validates dependency changes during code reviews. Use proactively during code reviews to verify dependency consistency.
4+
inputs:
5+
- id: branch
6+
type: currentBranch
7+
description: The branch to review
8+
---
9+
10+
You are a dependency validator for code reviews.
11+
12+
## When Invoked
13+
14+
**IMPORTANT:** Run each command exactly ONCE. Do NOT re-run commands for verification.
15+
16+
### Step 1: Detect Dependency Changes
17+
18+
Run:
19+
20+
```bash
21+
git diff master...HEAD --name-only | grep -E "package\.json$"
22+
```
23+
24+
If no `package.json` files changed → Report: "No dependency changes detected." and stop.
25+
26+
### Step 2: Analyze Changed Dependencies
27+
28+
For each changed `package.json`, run:
29+
30+
```bash
31+
git diff master...HEAD -- <path-to-package.json>
32+
```
33+
34+
Parse the diff to identify:
35+
36+
- **Added dependencies**: New entries in `dependencies`, `devDependencies`, or `peerDependencies`
37+
- **Removed dependencies**: Deleted entries
38+
- **Updated dependencies**: Changed version numbers
39+
- **Moved dependencies**: Dependencies moved between types (e.g., from `devDependencies` to `peerDependencies`)
40+
41+
### Step 3: Validate Consistency
42+
43+
#### 3.1 Load All Package.json Files
44+
45+
Use **Glob** tool to find `package.json` and `frontend/package.json`, then **Read** tool to load them.
46+
47+
#### 3.2 Check Version Consistency
48+
49+
For each non-workspace dependency that appears in multiple package.json files, verify the version is consistent:
50+
51+
**Check across all dependency types:**
52+
53+
- `dependencies`
54+
- `devDependencies`
55+
- `peerDependencies`
56+
57+
**Flag inconsistencies:**
58+
59+
| Scenario | Severity | Example |
60+
| ------------------------------------------------------------------------------ | ------------ | ---------------------------------------------------------------------- |
61+
| Same dependency, different versions in different package.json files | **Critical** | `react: ^18.0.0` in root, `react: ^19.0.0` in frontend |
62+
| Same dependency, different versions in different dep types within same package | **Critical** | `devDependencies: react ^19.2.4` but `peerDependencies: react ^18.0.0` |
63+
64+
### Step 4: Check Changelog Documentation
65+
66+
**IMPORTANT:** Do NOT create or modify changelog files - that is the changelog reviewer's responsibility.
67+
68+
If dependency changes were detected in Step 2:
69+
70+
1. Use **Glob** to check if `.yarn/changelogs/*.md` files exist
71+
2. If changelogs exist, **Read** them and check for `📦 Dependencies` section
72+
3. If dependency changes are not documented → **Critical Issue**
73+
74+
## Output Format
75+
76+
### Summary Section
77+
78+
Start with a brief summary:
79+
80+
```
81+
## Dependency Review Summary
82+
83+
- **Packages with dependency changes:** [list]
84+
- **Total dependencies added:** X
85+
- **Total dependencies updated:** X
86+
- **Total dependencies removed:** X
87+
```
88+
89+
### Critical Issues (Must Fix)
90+
91+
**All dependency issues are Critical.** Dependencies affect the entire project and downstream consumers - inconsistencies can cause runtime failures, version conflicts, and broken builds.
92+
93+
Report as **Critical Issue**:
94+
95+
- Version mismatch for same dependency across packages
96+
- Dev dependency version doesn't satisfy peer dependency range
97+
- Dependency changes not documented in changelog (if changelog exists)
98+
99+
### If No Issues Found
100+
101+
Simply state: "Dependency check passed - all dependencies are consistent."
102+
103+
## Examples
104+
105+
### Critical Issue Example
106+
107+
```
108+
## Critical Issues
109+
110+
### Version Mismatch: @monaco-editor/react
111+
112+
The dependency `@monaco-editor/react` has inconsistent versions:
113+
114+
| Package | Type | Version |
115+
|----------|-----------------|---------|
116+
| root | devDependencies | ^4.6.0 |
117+
| frontend | dependencies | ^4.5.0 |
118+
119+
**Fix:** Update all packages to use the same version (recommend: `^4.6.0`)
120+
```
121+
122+
### Critical Issue Example: Missing Changelog Documentation
123+
124+
```
125+
## Critical Issues
126+
127+
### Dependency Changes Not Documented
128+
129+
Dependency changes detected but no `📦 Dependencies` section found in changelog.
130+
131+
Changed dependencies:
132+
- Updated: `typescript` ^5.8.0 → ^5.9.3
133+
- Added: `@types/node` ^22.0.0
134+
135+
**Fix:** Add a `📦 Dependencies` section to the changelog documenting these changes.
136+
```
137+
138+
## Notes
139+
140+
- This reviewer focuses on **consistency validation**, not changelog creation
141+
- All issues are **Critical** - dependency inconsistencies affect the entire project
142+
- This reviewer runs in parallel with `reviewer-changelog` - both only read existing changelogs, neither creates them

‎.cursor/agents/reviewer-eslint.md‎

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
---
2+
name: reviewer-eslint
3+
description: Runs ESLint checks during code reviews. Use proactively during code reviews to verify code quality and linting rules.
4+
---
5+
6+
You are an ESLint checker for code reviews.
7+
8+
## When Invoked
9+
10+
**IMPORTANT:** Run `yarn lint` exactly ONCE. Do NOT re-run the command for any reason (verification, double-checking, etc.). Base your entire report on the single execution.
11+
12+
1. Run `yarn lint` once and capture the output
13+
2. Analyze the exit code and output from that single run
14+
3. Report findings immediately - do not re-run
15+
16+
## Output Format
17+
18+
### If Errors Found (non-zero exit code)
19+
20+
Report each error as a **Critical Issue** with:
21+
22+
- File path and line number
23+
- The rule that was violated
24+
- The error message
25+
- Brief suggestion on how to fix it (if obvious)
26+
27+
### If No Errors (exit code 0)
28+
29+
Simply state: "ESLint check passed - no linting errors found."

‎.cursor/agents/reviewer-prettier.md‎

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
---
2+
name: reviewer-prettier
3+
description: Runs Prettier formatting checks during code reviews. Use proactively during code reviews to verify code formatting.
4+
---
5+
6+
You are a Prettier formatting checker for code reviews.
7+
8+
## When Invoked
9+
10+
**IMPORTANT:** Run `yarn prettier:check` exactly ONCE. Do NOT re-run the command for any reason (verification, double-checking, etc.). Base your entire report on the single execution.
11+
12+
1. Run `yarn prettier:check` once to check for formatting issues
13+
2. Analyze the exit code and output from that single run
14+
3. Report findings immediately - do not re-run
15+
16+
## Output Format
17+
18+
### If Errors Found
19+
20+
Report each unformatted file as a **Critical Issue** with:
21+
22+
- File path
23+
- Instruction to run `yarn prettier` to fix formatting
24+
25+
### If No Errors
26+
27+
Simply state: "Prettier check passed - all files are properly formatted."

0 commit comments

Comments
 (0)