safeword 0.8.3 → 0.8.5
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/dist/{check-Y3AQWR4Y.js → check-QZ3ZAHZY.js} +3 -3
- package/dist/{chunk-SIK3BC7F.js → chunk-62YXVEKM.js} +27 -9
- package/dist/{chunk-SIK3BC7F.js.map → chunk-62YXVEKM.js.map} +1 -1
- package/dist/{chunk-VLNT6YSH.js → chunk-SIEJPWQA.js} +3 -2
- package/dist/{chunk-VLNT6YSH.js.map → chunk-SIEJPWQA.js.map} +1 -1
- package/dist/{chunk-QYCKBF57.js → chunk-VZPFU4YI.js} +2 -2
- package/dist/cli.js +6 -6
- package/dist/{diff-OITRHARU.js → diff-YXUSIILN.js} +3 -3
- package/dist/{reset-D6HUKH4Z.js → reset-HQXT7SCI.js} +3 -3
- package/dist/{setup-DW7OG2QD.js → setup-JX476EH3.js} +3 -3
- package/dist/sync-A7VWZKQD.js +9 -0
- package/dist/{upgrade-FDZLLLT2.js → upgrade-45E6255Q.js} +4 -4
- package/package.json +15 -14
- package/templates/cursor/rules/safeword-brainstorming.mdc +198 -0
- package/templates/cursor/rules/safeword-debugging.mdc +202 -0
- package/templates/cursor/rules/safeword-enforcing-tdd.mdc +207 -0
- package/templates/cursor/rules/safeword-quality-reviewer.mdc +158 -0
- package/templates/cursor/rules/safeword-refactoring.mdc +175 -0
- package/templates/scripts/lint-md.sh +0 -0
- package/templates/skills/safeword-brainstorming/SKILL.md +210 -0
- package/templates/skills/{safeword-systematic-debugger → safeword-debugging}/SKILL.md +1 -71
- package/templates/skills/{safeword-tdd-enforcer → safeword-enforcing-tdd}/SKILL.md +17 -17
- package/templates/skills/safeword-quality-reviewer/SKILL.md +17 -67
- package/dist/sync-AOKWEHCY.js +0 -9
- /package/dist/{check-Y3AQWR4Y.js.map → check-QZ3ZAHZY.js.map} +0 -0
- /package/dist/{chunk-QYCKBF57.js.map → chunk-VZPFU4YI.js.map} +0 -0
- /package/dist/{diff-OITRHARU.js.map → diff-YXUSIILN.js.map} +0 -0
- /package/dist/{reset-D6HUKH4Z.js.map → reset-HQXT7SCI.js.map} +0 -0
- /package/dist/{setup-DW7OG2QD.js.map → setup-JX476EH3.js.map} +0 -0
- /package/dist/{sync-AOKWEHCY.js.map → sync-A7VWZKQD.js.map} +0 -0
- /package/dist/{upgrade-FDZLLLT2.js.map → upgrade-45E6255Q.js.map} +0 -0
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: quality-reviewer
|
|
3
|
-
description: Deep code
|
|
3
|
+
description: Deep code review with web research to verify against latest ecosystem. Use when user says 'double check against latest', 'verify versions', 'check security', 'review against docs', or needs deep analysis beyond automatic quality hook.
|
|
4
4
|
allowed-tools: '*'
|
|
5
5
|
---
|
|
6
6
|
|
|
@@ -8,7 +8,7 @@ allowed-tools: '*'
|
|
|
8
8
|
|
|
9
9
|
Deep quality review with web research to verify code against the latest ecosystem state.
|
|
10
10
|
|
|
11
|
-
**Primary differentiator**: Web research
|
|
11
|
+
**Primary differentiator**: Web research to verify against current versions, documentation, and best practices.
|
|
12
12
|
|
|
13
13
|
**Triggers**:
|
|
14
14
|
|
|
@@ -16,7 +16,6 @@ Deep quality review with web research to verify code against the latest ecosyste
|
|
|
16
16
|
- **Deep dive needed**: User wants analysis beyond automatic hook (performance, architecture alternatives, trade-offs)
|
|
17
17
|
- **No SAFEWORD.md/CLAUDE.md**: Projects without context files (automatic hook won't run, manual review needed)
|
|
18
18
|
- **Pre-change review**: User wants review before making changes (automatic hook only triggers after changes)
|
|
19
|
-
- **Model-invoked**: Claude determines web research would be valuable
|
|
20
19
|
|
|
21
20
|
**Relationship to automatic quality hook**:
|
|
22
21
|
|
|
@@ -43,7 +42,6 @@ Read relevant standards:
|
|
|
43
42
|
|
|
44
43
|
- `CLAUDE.md` or `SAFEWORD.md` - Project-specific guidelines
|
|
45
44
|
- `ARCHITECTURE.md` - Architectural principles
|
|
46
|
-
- `@./.safeword/guides/code-philosophy.md` - Core coding principles
|
|
47
45
|
|
|
48
46
|
### 3. Evaluate Correctness
|
|
49
47
|
|
|
@@ -92,14 +90,12 @@ Read relevant standards:
|
|
|
92
90
|
- Are we using libraries correctly?
|
|
93
91
|
- Are we following official documentation?
|
|
94
92
|
|
|
95
|
-
### 7. Verify Latest Versions
|
|
93
|
+
### 7. Verify Latest Versions - PRIMARY VALUE
|
|
96
94
|
|
|
97
95
|
**CRITICAL**: This is your main differentiator from automatic hook. ALWAYS check versions.
|
|
98
96
|
|
|
99
|
-
|
|
100
|
-
|
|
101
|
-
WebSearch: "[library name] security vulnerabilities"
|
|
102
|
-
```
|
|
97
|
+
Search for: "[library name] latest stable version 2025"
|
|
98
|
+
Search for: "[library name] security vulnerabilities"
|
|
103
99
|
|
|
104
100
|
**Flag if outdated:**
|
|
105
101
|
|
|
@@ -110,25 +106,17 @@ WebSearch: "[library name] security vulnerabilities"
|
|
|
110
106
|
|
|
111
107
|
**Common libraries**: React, TypeScript, Vite, Next.js, Node.js, Vitest, Playwright, Jest, esbuild
|
|
112
108
|
|
|
113
|
-
|
|
114
|
-
|
|
115
|
-
### 8. Verify Latest Documentation ⭐ **PRIMARY VALUE**
|
|
109
|
+
### 8. Verify Latest Documentation - PRIMARY VALUE
|
|
116
110
|
|
|
117
111
|
**CRITICAL**: This is your main differentiator from automatic hook. ALWAYS verify against current docs.
|
|
118
112
|
|
|
119
|
-
|
|
120
|
-
WebFetch: https://react.dev (for React)
|
|
121
|
-
WebFetch: https://vitejs.dev (for Vite)
|
|
122
|
-
WebFetch: https://www.electronjs.org/docs (for Electron)
|
|
123
|
-
```
|
|
113
|
+
Fetch and check official documentation sites for the libraries in use.
|
|
124
114
|
|
|
125
115
|
**Look for:**
|
|
126
116
|
|
|
127
117
|
- Are we using deprecated APIs?
|
|
128
118
|
- Are there newer, better patterns?
|
|
129
|
-
- Did the library's recommendations change
|
|
130
|
-
|
|
131
|
-
**Cache results**: If you checked docs recently in this session, don't re-fetch.
|
|
119
|
+
- Did the library's recommendations change recently?
|
|
132
120
|
|
|
133
121
|
## Output Format
|
|
134
122
|
|
|
@@ -147,8 +135,8 @@ WebFetch: https://www.electronjs.org/docs (for Electron)
|
|
|
147
135
|
**Anti-Bloat:** [✓/⚠️/❌] [Brief assessment]
|
|
148
136
|
**Elegance:** [✓/⚠️/❌] [Brief assessment]
|
|
149
137
|
**Standards:** [✓/⚠️/❌] [Brief assessment]
|
|
150
|
-
**Versions:** [✓/⚠️/❌] [Latest version check
|
|
151
|
-
**Documentation:** [✓/⚠️/❌] [Current docs check
|
|
138
|
+
**Versions:** [✓/⚠️/❌] [Latest version check]
|
|
139
|
+
**Documentation:** [✓/⚠️/❌] [Current docs check]
|
|
152
140
|
|
|
153
141
|
**Verdict:** [APPROVE / REQUEST CHANGES / NEEDS DISCUSSION]
|
|
154
142
|
|
|
@@ -156,52 +144,14 @@ WebFetch: https://www.electronjs.org/docs (for Electron)
|
|
|
156
144
|
**Suggested improvements:** [List or "None"]
|
|
157
145
|
```
|
|
158
146
|
|
|
159
|
-
Use structured format for "double check"/"critique". Use brief format for specific questions.
|
|
160
|
-
|
|
161
|
-
## Example: Full Review
|
|
162
|
-
|
|
163
|
-
```markdown
|
|
164
|
-
## Quality Review
|
|
165
|
-
|
|
166
|
-
**Correctness:** ✓ Logic sound, edge cases covered, error handling adequate
|
|
167
|
-
**Anti-Bloat:** ✓ Minimal dependencies, appropriate abstractions
|
|
168
|
-
**Elegance:** ✓ Clear code, good naming, well-structured
|
|
169
|
-
**Standards:** ✓ Follows CLAUDE.md patterns
|
|
170
|
-
**Versions:** ✓ React 19.0.0 (latest stable), TypeScript 5.7.2 (latest)
|
|
171
|
-
**Documentation:** ✓ Using current React patterns per https://react.dev
|
|
172
|
-
|
|
173
|
-
**Verdict:** APPROVE - Production ready
|
|
174
|
-
|
|
175
|
-
**Critical issues:** None
|
|
176
|
-
**Suggested improvements:** None
|
|
177
|
-
```
|
|
178
|
-
|
|
179
147
|
## Critical Reminders
|
|
180
148
|
|
|
181
|
-
1. **Primary value: Web research** -
|
|
149
|
+
1. **Primary value: Web research** - Verify against current ecosystem (versions, docs, security)
|
|
182
150
|
2. **Complement automatic hook** - Hook does fast check with existing knowledge, you do deep dive with web research
|
|
183
151
|
3. **Explicit triggers matter** - "double check against latest docs", "verify versions", "check security" = invoke web research
|
|
184
|
-
4. **
|
|
185
|
-
5. **Always
|
|
186
|
-
6. **
|
|
187
|
-
7. **
|
|
188
|
-
8. **
|
|
189
|
-
9. **
|
|
190
|
-
10. **Separate critical vs nice-to-have** - User needs to know what's blocking vs optional
|
|
191
|
-
|
|
192
|
-
## Non-Obvious Edge Cases
|
|
193
|
-
|
|
194
|
-
**User requests review after automatic hook ran:**
|
|
195
|
-
|
|
196
|
-
- Acknowledge hook ran: "The automatic quality hook already did a fast check. I'll now do deeper analysis with web research..."
|
|
197
|
-
- Focus on what automatic hook doesn't do: fetch latest docs, verify versions, security checks, performance analysis
|
|
198
|
-
|
|
199
|
-
**WebSearch/WebFetch fails:**
|
|
200
|
-
|
|
201
|
-
- Continue review without version/docs checks
|
|
202
|
-
- Note: "Couldn't verify latest versions/docs, skipping that check"
|
|
203
|
-
|
|
204
|
-
**Project has no CLAUDE.md/SAFEWORD.md:**
|
|
205
|
-
|
|
206
|
-
- Use `@./.safeword/guides/code-philosophy.md` as fallback
|
|
207
|
-
- Note: "No project-specific standards found, using general best practices"
|
|
152
|
+
4. **Always check latest docs** - Verify patterns are current, not outdated
|
|
153
|
+
5. **Always verify versions** - Flag outdated dependencies
|
|
154
|
+
6. **Be thorough but concise** - Cover all areas but keep explanations brief
|
|
155
|
+
7. **Provide actionable feedback** - Specific line numbers, concrete suggestions
|
|
156
|
+
8. **Clear verdict** - Always end with APPROVE/REQUEST CHANGES/NEEDS DISCUSSION
|
|
157
|
+
9. **Separate critical vs nice-to-have** - User needs to know what's blocking vs optional
|
package/dist/sync-AOKWEHCY.js
DELETED
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|