Commit efa60cc
docs: document intentional git scope limitation in workflow commands (#139)
## Summary
- Document that the `allowed-tools` git scope (`git init` only) in
workflow commands is intentional
- Add post-workflow git operation examples for users to run after
completion
## Problem
Fixes #136
The workflow commands (`create-plugin.md` and `create-marketplace.md`)
only allow `Bash(git init:*)` in their `allowed-tools`. Issue #136
raised whether this should be expanded to include `git add`, `git
status`, `git commit`, etc.
## Solution
After analysis, the current limitation is **intentionally minimal**
(Option C from the issue). This is the correct approach because:
1. **Write tool handles file creation** - Files are created via the
Write tool, not bash commands, so `git add` isn't needed mid-workflow
2. **Respects user preferences** - Commit messages, timing, and git
workflow vary by team/user
3. **Focus on structure** - The workflow's purpose is structure
creation, not managing git history
Rather than expanding git permissions, this PR documents the intentional
design and provides post-workflow git examples for users.
### Alternatives Considered
- **Option A** (`Bash(git:*)`): Too permissive - allows destructive
operations like `push --force`, `reset --hard`
- **Option B** (add specific subcommands): Unnecessary since Write tool
handles file creation
## Changes
- `plugins/plugin-dev/commands/create-plugin.md`: Updated Phase 4 step 7
with explanation, added post-workflow git examples
- `plugins/plugin-dev/commands/create-marketplace.md`: Updated Phase 4
step 5 with explanation, added post-workflow git examples
## Testing
- [x] markdownlint passes
- [x] Changes limited to documentation only
- [x] No functional changes to workflow behavior
---
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-authored-by: Claude <noreply@anthropic.com>1 parent 3572b8a commit efa60cc
2 files changed
Lines changed: 16 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
158 | 158 | | |
159 | 159 | | |
160 | 160 | | |
161 | | - | |
| 161 | + | |
162 | 162 | | |
163 | 163 | | |
164 | 164 | | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
165 | 172 | | |
166 | 173 | | |
167 | 174 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
145 | 145 | | |
146 | 146 | | |
147 | 147 | | |
148 | | - | |
| 148 | + | |
149 | 149 | | |
150 | 150 | | |
151 | 151 | | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
152 | 159 | | |
153 | 160 | | |
154 | 161 | | |
| |||
0 commit comments