@riligar/agents-kit 1.5.0 → 1.5.1
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/.agent/skills/riligar-dev-clean-code/SKILL.md +67 -82
- package/README.md +1 -1
- package/package.json +1 -1
- /package/.agent/skills/{riligar-dev-api → riligar-dev-backend}/SKILL.md +0 -0
- /package/.agent/skills/{riligar-dev-api → riligar-dev-backend}/api-style.md +0 -0
- /package/.agent/skills/{riligar-dev-api → riligar-dev-backend}/auth.md +0 -0
- /package/.agent/skills/{riligar-dev-api → riligar-dev-backend}/documentation.md +0 -0
- /package/.agent/skills/{riligar-dev-api → riligar-dev-backend}/graphql.md +0 -0
- /package/.agent/skills/{riligar-dev-api → riligar-dev-backend}/rate-limiting.md +0 -0
- /package/.agent/skills/{riligar-dev-api → riligar-dev-backend}/response.md +0 -0
- /package/.agent/skills/{riligar-dev-api → riligar-dev-backend}/rest.md +0 -0
- /package/.agent/skills/{riligar-dev-api → riligar-dev-backend}/scripts/api_validator.py +0 -0
- /package/.agent/skills/{riligar-dev-api → riligar-dev-backend}/security-testing.md +0 -0
- /package/.agent/skills/{riligar-dev-api → riligar-dev-backend}/trpc.md +0 -0
- /package/.agent/skills/{riligar-dev-api → riligar-dev-backend}/versioning.md +0 -0
- /package/.agent/skills/{riligar-dev-react → riligar-dev-frontend}/SKILL.md +0 -0
|
@@ -1,14 +1,14 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: riligar-dev-clean-code
|
|
3
|
-
description: Pragmatic coding standards - concise, direct, no over-engineering, no unnecessary comments
|
|
3
|
+
description: Pragmatic coding standards - concise, direct, no over-engineering, no unnecessary comments (RiLiGar Standard)
|
|
4
4
|
allowed-tools: Read, Write, Edit
|
|
5
|
-
version: 2.
|
|
5
|
+
version: 2.1
|
|
6
6
|
priority: CRITICAL
|
|
7
7
|
---
|
|
8
8
|
|
|
9
|
-
# Clean Code - Pragmatic AI Coding Standards
|
|
9
|
+
# Clean Code - Pragmatic AI Coding Standards (RiLiGar)
|
|
10
10
|
|
|
11
|
-
> **CRITICAL SKILL** - Be **concise, direct, and solution-focused**.
|
|
11
|
+
> **CRITICAL SKILL** - Be **concise, direct, and solution-focused**. Follow the RiLiGar Tech Stack strictly.
|
|
12
12
|
|
|
13
13
|
---
|
|
14
14
|
|
|
@@ -24,6 +24,20 @@ priority: CRITICAL
|
|
|
24
24
|
|
|
25
25
|
---
|
|
26
26
|
|
|
27
|
+
## RiLiGar Formatting (Prettier)
|
|
28
|
+
|
|
29
|
+
| Prop | Value |
|
|
30
|
+
| ------------------ | -------- |
|
|
31
|
+
| **Indent Size** | 4 spaces |
|
|
32
|
+
| **Semicolons** | `false` |
|
|
33
|
+
| **Quotes** | Single |
|
|
34
|
+
| **Print Width** | 300 |
|
|
35
|
+
| **Trailing Comma** | `es5` |
|
|
36
|
+
|
|
37
|
+
> 🔴 **Rule:** Never use TypeScript. Always use modern JavaScript (ES6+).
|
|
38
|
+
|
|
39
|
+
---
|
|
40
|
+
|
|
27
41
|
## Naming Rules
|
|
28
42
|
|
|
29
43
|
| Element | Convention |
|
|
@@ -39,13 +53,26 @@ priority: CRITICAL
|
|
|
39
53
|
|
|
40
54
|
## Function Rules
|
|
41
55
|
|
|
42
|
-
| Rule | Description
|
|
43
|
-
| ------------------- |
|
|
44
|
-
| **Small** | Max 20 lines, ideally 5-10
|
|
45
|
-
| **One Thing** | Does one thing, does it well
|
|
46
|
-
| **One Level** | One level of abstraction per function
|
|
47
|
-
| **Few Args** | Max 3 arguments, prefer 0-2
|
|
48
|
-
| **No Side Effects** | Don't mutate inputs unexpectedly
|
|
56
|
+
| Rule | Description |
|
|
57
|
+
| ------------------- | ------------------------------------------ |
|
|
58
|
+
| **Small** | Max 20 lines, ideally 5-10 |
|
|
59
|
+
| **One Thing** | Does one thing, does it well |
|
|
60
|
+
| **One Level** | One level of abstraction per function |
|
|
61
|
+
| **Few Args** | Max 3 arguments, prefer 0-2 |
|
|
62
|
+
| **No Side Effects** | Don't mutate inputs unexpectedly |
|
|
63
|
+
| **Async/Await** | Prefer `async/await` over `.then()` chains |
|
|
64
|
+
|
|
65
|
+
---
|
|
66
|
+
|
|
67
|
+
## Component & Framework Rules (React/Elysia)
|
|
68
|
+
|
|
69
|
+
| Situation | Action |
|
|
70
|
+
| -------------------- | ------------------------------------------------ |
|
|
71
|
+
| **Complex Logic** | Extract to a Custom Hook if logic > 15 lines |
|
|
72
|
+
| **State Management** | Use **Zustand** for global, `useState` for local |
|
|
73
|
+
| **UI Components** | **ALWAYS** use Mantine. No custom/raw CSS. |
|
|
74
|
+
| **API Endpoints** | Extract complex handler logic to `services/` |
|
|
75
|
+
| **Early Returns** | Use Guard Clauses to avoid deep nesting |
|
|
49
76
|
|
|
50
77
|
---
|
|
51
78
|
|
|
@@ -72,16 +99,17 @@ priority: CRITICAL
|
|
|
72
99
|
|
|
73
100
|
## Anti-Patterns (DON'T)
|
|
74
101
|
|
|
75
|
-
| ❌ Pattern
|
|
76
|
-
|
|
|
77
|
-
| Comment every line
|
|
78
|
-
| Helper for one-liner
|
|
79
|
-
| Factory for 2 objects
|
|
80
|
-
| utils.
|
|
81
|
-
| "First we import..."
|
|
82
|
-
| Deep nesting
|
|
83
|
-
| Magic numbers
|
|
84
|
-
| God functions
|
|
102
|
+
| ❌ Pattern | ✅ Fix |
|
|
103
|
+
| ----------------------- | ------------------------------ |
|
|
104
|
+
| Comment every line | Delete obvious comments |
|
|
105
|
+
| Helper for one-liner | Inline the code |
|
|
106
|
+
| Factory for 2 objects | Direct instantiation |
|
|
107
|
+
| `utils.js` (1 function) | Put code where used |
|
|
108
|
+
| "First we import..." | Just write code |
|
|
109
|
+
| Deep nesting | Guard clauses |
|
|
110
|
+
| Magic numbers | Named constants |
|
|
111
|
+
| God functions | Split by responsibility |
|
|
112
|
+
| TypeScript / `.ts` | Use Vanilla JavaScript (`.js`) |
|
|
85
113
|
|
|
86
114
|
---
|
|
87
115
|
|
|
@@ -93,19 +121,17 @@ priority: CRITICAL
|
|
|
93
121
|
| ------------------------------- | ------------------------ |
|
|
94
122
|
| **What imports this file?** | They might break |
|
|
95
123
|
| **What does this file import?** | Interface changes |
|
|
96
|
-
| **What tests cover this?** | Tests might fail |
|
|
97
124
|
| **Is this a shared component?** | Multiple places affected |
|
|
98
125
|
|
|
99
126
|
**Quick Check:**
|
|
100
127
|
|
|
101
128
|
```
|
|
102
|
-
File to edit:
|
|
103
|
-
└── Who imports this? →
|
|
129
|
+
File to edit: user-service.js
|
|
130
|
+
└── Who imports this? → user-controller.js
|
|
104
131
|
└── Do they need changes too? → Check function signatures
|
|
105
132
|
```
|
|
106
133
|
|
|
107
134
|
> 🔴 **Rule:** Edit the file + all dependent files in the SAME task.
|
|
108
|
-
> 🔴 **Never leave broken imports or missing updates.**
|
|
109
135
|
|
|
110
136
|
---
|
|
111
137
|
|
|
@@ -128,43 +154,27 @@ File to edit: UserService.ts
|
|
|
128
154
|
|
|
129
155
|
**Before saying "task complete", verify:**
|
|
130
156
|
|
|
131
|
-
| Check
|
|
132
|
-
|
|
|
133
|
-
| ✅ **Goal met?**
|
|
134
|
-
| ✅ **Files edited?**
|
|
135
|
-
| ✅ **Code works?**
|
|
136
|
-
| ✅ **No
|
|
137
|
-
| ✅ **
|
|
138
|
-
|
|
139
|
-
> 🔴 **Rule:** If ANY check fails, fix it before completing.
|
|
157
|
+
| Check | Question |
|
|
158
|
+
| --------------------- | --------------------------------- |
|
|
159
|
+
| ✅ **Goal met?** | Did I do exactly what user asked? |
|
|
160
|
+
| ✅ **Files edited?** | Did I modify all necessary files? |
|
|
161
|
+
| ✅ **Code works?** | Did I test/verify the change? |
|
|
162
|
+
| ✅ **No TypeScript?** | Verified no `.ts` or type syntax? |
|
|
163
|
+
| ✅ **Formatting?** | 4 spaces, no semi, single quotes? |
|
|
140
164
|
|
|
141
165
|
---
|
|
142
166
|
|
|
143
167
|
## Verification Scripts (MANDATORY)
|
|
144
168
|
|
|
145
|
-
> 🔴 **CRITICAL:**
|
|
169
|
+
> 🔴 **CRITICAL:** Run only valid scripts found in `.agent/skills/*/scripts`.
|
|
146
170
|
|
|
147
171
|
### Agent → Script Mapping
|
|
148
172
|
|
|
149
|
-
| Agent
|
|
150
|
-
|
|
|
151
|
-
| **
|
|
152
|
-
| **
|
|
153
|
-
| **
|
|
154
|
-
| **mobile-developer** | Mobile Audit | `python .agent/skills/mobile-design/scripts/mobile_audit.py .` |
|
|
155
|
-
| **database-architect** | Schema Validate | `python .agent/skills/database-design/scripts/schema_validator.py .` |
|
|
156
|
-
| **security-auditor** | Security Scan | `python .agent/skills/vulnerability-scanner/scripts/security_scan.py .` |
|
|
157
|
-
| **seo-specialist** | SEO Check | `python .agent/skills/seo-fundamentals/scripts/seo_checker.py .` |
|
|
158
|
-
| **seo-specialist** | GEO Check | `python .agent/skills/geo-fundamentals/scripts/geo_checker.py .` |
|
|
159
|
-
| **performance-optimizer** | Lighthouse | `python .agent/skills/performance-profiling/scripts/lighthouse_audit.py <url>` |
|
|
160
|
-
| **test-engineer** | Test Runner | `python .agent/skills/testing-patterns/scripts/test_runner.py .` |
|
|
161
|
-
| **test-engineer** | Playwright | `python .agent/skills/webapp-testing/scripts/playwright_runner.py <url>` |
|
|
162
|
-
| **Any agent** | Lint Check | `python .agent/skills/lint-and-validate/scripts/lint_runner.py .` |
|
|
163
|
-
| **Any agent** | Type Coverage | `python .agent/skills/lint-and-validate/scripts/type_coverage.py .` |
|
|
164
|
-
| **Any agent** | i18n Check | `python .agent/skills/i18n-localization/scripts/i18n_checker.py .` |
|
|
165
|
-
|
|
166
|
-
> ❌ **WRONG:** `test-engineer` running `ux_audit.py`
|
|
167
|
-
> ✅ **CORRECT:** `frontend-specialist` running `ux_audit.py`
|
|
173
|
+
| Agent | Script | Command |
|
|
174
|
+
| ---------------------- | --------------- | ------------------------------------------------------------------------- |
|
|
175
|
+
| **backend-specialist** | API Validator | `python .agent/skills/riligar-dev-backend/scripts/api_validator.py .` |
|
|
176
|
+
| **database-architect** | Schema Validate | `python .agent/skills/riligar-dev-database/scripts/schema_validator.py .` |
|
|
177
|
+
| **Any agent** | Lint Check | `bun x eslint .` |
|
|
168
178
|
|
|
169
179
|
---
|
|
170
180
|
|
|
@@ -172,33 +182,8 @@ File to edit: UserService.ts
|
|
|
172
182
|
|
|
173
183
|
**When running a validation script, you MUST:**
|
|
174
184
|
|
|
175
|
-
1.
|
|
176
|
-
2.
|
|
177
|
-
3.
|
|
178
|
-
|
|
179
|
-
```markdown
|
|
180
|
-
## Script Results: [script_name.py]
|
|
181
|
-
|
|
182
|
-
### ❌ Errors Found (X items)
|
|
183
|
-
|
|
184
|
-
- [File:Line] Error description 1
|
|
185
|
-
- [File:Line] Error description 2
|
|
186
|
-
|
|
187
|
-
### ⚠️ Warnings (Y items)
|
|
188
|
-
|
|
189
|
-
- [File:Line] Warning description
|
|
190
|
-
|
|
191
|
-
### ✅ Passed (Z items)
|
|
192
|
-
|
|
193
|
-
- Check 1 passed
|
|
194
|
-
- Check 2 passed
|
|
195
|
-
|
|
196
|
-
**Should I fix the X errors?**
|
|
197
|
-
```
|
|
198
|
-
|
|
199
|
-
4. **Wait for user confirmation** before fixing
|
|
200
|
-
5. **After fixing** → Re-run script to confirm
|
|
185
|
+
1. **Run the script** and capture ALL output.
|
|
186
|
+
2. **Summarize results** (Errors, Warnings, Passes).
|
|
187
|
+
3. **Ask before fixing** any identified errors.
|
|
201
188
|
|
|
202
|
-
> 🔴 **VIOLATION:** Running script and ignoring output = FAILED task.
|
|
203
189
|
> 🔴 **VIOLATION:** Auto-fixing without asking = Not allowed.
|
|
204
|
-
> 🔴 **Rule:** Always READ output → SUMMARIZE → ASK → then fix.
|
package/README.md
CHANGED
package/package.json
CHANGED
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|