@clipboard-health/ai-rules 2.9.0 → 2.9.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/package.json CHANGED
@@ -1,7 +1,7 @@
1
1
  {
2
2
  "name": "@clipboard-health/ai-rules",
3
3
  "description": "Pre-built AI agent rules for consistent coding standards.",
4
- "version": "2.9.0",
4
+ "version": "2.9.2",
5
5
  "bugs": "https://github.com/ClipboardHealth/core-utils/issues",
6
6
  "keywords": [
7
7
  "ai",
@@ -32,20 +32,16 @@ Then, for EVERY comment (both `unresolvedComments` AND `nitpickComments`):
32
32
 
33
33
  1. Group comments by file path and read each file once (not per-comment)
34
34
  2. If a file no longer exists, note that the comment may be outdated
35
- 3. Assess the comment against the current code and provide your opinion:
36
- - **Agree**: Explain why and offer to fix it
37
- - **Disagree**: Explain why the current code is acceptable
38
- - **Already fixed**: Note that the code already addresses this concern
39
- 4. When multiple comments appear at the same file and line, they are part of the same review thread — read them together as a conversation and assess the original feedback
40
- 5. Present your assessment in list format (renders reliably in terminals):
35
+ 3. Assess each comment against the current code. When multiple comments appear at the same file and line, they are part of the same review thread — read them together as a conversation
36
+ 4. Group your assessment as follows:
41
37
 
42
- **1. `src/api.ts:118`** - Wrap JSON.parse in try-catch
43
- **Verdict: Already fixed** - Try-catch added in recent commit
38
+ **Should address**
39
+ - Description of the comment with file path(s)
40
+ - Why it's a real issue (bug, a11y, UX, etc.)
44
41
 
45
- **2. `src/config.ts:23`** - Use `const` instead of `let`
46
- **Verdict: Disagree** - Value is reassigned on L31
42
+ **Can ignore**
43
+ - Description of the comment with file path(s)
44
+ - Why: already fixed, not actionable, not worth addressing in this PR, etc.
47
45
 
48
- **3. `src/utils.ts:42`** - Add null check for user input
49
- **Verdict: Agree** - Input isn't validated, could cause runtime error
50
-
51
- Then, offer to fix any issues where you agreed.
46
+ **Net**
47
+ Summary of how many are worth fixing and what kind of issues they are. Offer to fix, but **do NOT start until the user confirms**.