Commit 94aa984
docs(hook-development): add PermissionRequest hook event documentation (#69)
## Summary
- Added documentation for the PermissionRequest hook event
- Updated validate-hook-schema.sh to recognize PermissionRequest as a
valid event
- PermissionRequest allows automatic allow/deny of permission dialogs
## Problem
Fixes #67
The hook-development skill was missing documentation for
PermissionRequest, which is a valid hook event in the official Claude
Code documentation. This caused:
1. Users not knowing about this capability
2. The validation script incorrectly warning about PermissionRequest
being unknown
## Solution
Added comprehensive documentation for PermissionRequest:
- New section in Hook Events with example configuration and output
format
- Added to skill description trigger list
- Added to Quick Reference table
- Added to event-specific input fields
- Added to VALID_EVENTS in validate-hook-schema.sh
### Alternatives Considered
- **Document as "not supported for plugins"**: Rejected because official
docs show it's available
- **Minimal one-line addition**: Rejected because consistent
documentation requires the same treatment as other events
## Changes
- `plugins/plugin-dev/skills/hook-development/SKILL.md`: Added
PermissionRequest documentation (+50 lines)
-
`plugins/plugin-dev/skills/hook-development/scripts/validate-hook-schema.sh`:
Added PermissionRequest to VALID_EVENTS
## Testing
- [x] Markdownlint passes
- [x] Shellcheck shows only pre-existing warnings (not related to this
change)
- [x] Documentation follows existing patterns
---
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-authored-by: Claude <noreply@anthropic.com>1 parent 87a7c20 commit 94aa984
2 files changed
Lines changed: 52 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
| 3 | + | |
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| |||
159 | 159 | | |
160 | 160 | | |
161 | 161 | | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
| 185 | + | |
| 186 | + | |
| 187 | + | |
| 188 | + | |
| 189 | + | |
| 190 | + | |
| 191 | + | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
| 198 | + | |
| 199 | + | |
| 200 | + | |
| 201 | + | |
| 202 | + | |
| 203 | + | |
| 204 | + | |
| 205 | + | |
| 206 | + | |
| 207 | + | |
| 208 | + | |
| 209 | + | |
162 | 210 | | |
163 | 211 | | |
164 | 212 | | |
| |||
327 | 375 | | |
328 | 376 | | |
329 | 377 | | |
330 | | - | |
| 378 | + | |
331 | 379 | | |
332 | 380 | | |
333 | 381 | | |
| |||
596 | 644 | | |
597 | 645 | | |
598 | 646 | | |
| 647 | + | |
599 | 648 | | |
600 | 649 | | |
601 | 650 | | |
| |||
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
41 | | - | |
| 41 | + | |
42 | 42 | | |
43 | 43 | | |
44 | 44 | | |
| |||
0 commit comments