wiggum-cli 0.13.0 → 0.13.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.
|
@@ -64,27 +64,21 @@ EOF
|
|
|
64
64
|
)"
|
|
65
65
|
```
|
|
66
66
|
|
|
67
|
-
### Step 4: Request
|
|
67
|
+
### Step 4: Request Claude Code Review
|
|
68
68
|
|
|
69
|
-
Run automated code review using
|
|
69
|
+
Run automated code review using Claude Code CLI:
|
|
70
70
|
|
|
71
71
|
```bash
|
|
72
|
-
# Check if
|
|
73
|
-
if ! command -v
|
|
74
|
-
echo "WARNING:
|
|
75
|
-
cd {{appDir}} && gh pr comment --body "Manual review requested -
|
|
72
|
+
# Check if Claude Code CLI is installed
|
|
73
|
+
if ! command -v claude &> /dev/null; then
|
|
74
|
+
echo "WARNING: Claude Code CLI not installed. Manual review needed."
|
|
75
|
+
cd {{appDir}} && gh pr comment --body "Manual review requested - Claude Code CLI not available. Install: https://docs.anthropic.com/en/docs/claude-code/overview"
|
|
76
76
|
else
|
|
77
|
-
echo "Running
|
|
78
|
-
# Get diff summary for context
|
|
79
|
-
DIFF_SUMMARY=$(cd {{appDir}} && git diff main --stat | head -50)
|
|
77
|
+
echo "Running Claude Code review..."
|
|
80
78
|
|
|
81
|
-
|
|
82
|
-
echo "Code Review Request for $FEATURE feature.
|
|
79
|
+
cd {{appDir}} && claude -p "You are reviewing a PR for the $FEATURE feature.
|
|
83
80
|
|
|
84
|
-
|
|
85
|
-
$DIFF_SUMMARY
|
|
86
|
-
|
|
87
|
-
## Review Checklist:
|
|
81
|
+
Review the git diff against main and check:
|
|
88
82
|
- Code quality and patterns consistency
|
|
89
83
|
- Test coverage adequacy
|
|
90
84
|
- Potential bugs or edge cases
|
|
@@ -92,19 +86,21 @@ $DIFF_SUMMARY
|
|
|
92
86
|
- Performance implications
|
|
93
87
|
- Error handling completeness
|
|
94
88
|
|
|
95
|
-
|
|
89
|
+
Run: git diff main
|
|
90
|
+
|
|
91
|
+
Respond with:
|
|
96
92
|
- APPROVED if everything looks good
|
|
97
|
-
- Or list specific issues that need to be fixed"
|
|
93
|
+
- Or list specific issues with file:line references that need to be fixed"
|
|
98
94
|
fi
|
|
99
95
|
```
|
|
100
96
|
|
|
101
97
|
**Handle review feedback:**
|
|
102
|
-
- If
|
|
103
|
-
- If
|
|
98
|
+
- If Claude outputs "APPROVED" -> Proceed to Step 5 (rebase and merge)
|
|
99
|
+
- If Claude lists issues:
|
|
104
100
|
1. Address each issue with code fixes
|
|
105
101
|
2. Commit: `git -C {{appDir}} add -A && git -C {{appDir}} commit -m "fix($FEATURE): address review feedback"`
|
|
106
102
|
3. Push: `git -C {{appDir}} push origin feat/$FEATURE`
|
|
107
|
-
4. Re-run the
|
|
103
|
+
4. Re-run the Claude review command above
|
|
108
104
|
- Max 3 review iterations before requiring manual intervention
|
|
109
105
|
|
|
110
106
|
### Step 5: Rebase Before Merge (for Parallel Execution)
|
|
@@ -126,7 +122,7 @@ cd {{appDir}} && git push --force-with-lease origin feat/$FEATURE
|
|
|
126
122
|
```
|
|
127
123
|
|
|
128
124
|
### Step 6: Merge PR
|
|
129
|
-
When
|
|
125
|
+
When Claude review is approved and branch is rebased:
|
|
130
126
|
```bash
|
|
131
127
|
cd {{appDir}} && gh pr merge --squash --delete-branch
|
|
132
128
|
```
|
|
@@ -145,7 +141,7 @@ cd {{appDir}} && gh pr merge --squash --delete-branch
|
|
|
145
141
|
Note: Spec status updates are handled in the Spec Verification phase before PR creation.
|
|
146
142
|
|
|
147
143
|
## Rules
|
|
148
|
-
- Do NOT merge without
|
|
144
|
+
- Do NOT merge without Claude Code approval
|
|
149
145
|
- Address ALL review comments before merging
|
|
150
146
|
- Use squash merge to keep history clean
|
|
151
147
|
- If gh CLI fails, check authentication: `gh auth status`
|
|
@@ -155,7 +151,7 @@ Note: Spec status updates are handled in the Spec Verification phase before PR c
|
|
|
155
151
|
- **gh: command not found** -> Install GitHub CLI: `brew install gh`
|
|
156
152
|
- **gh auth error** -> Run: `gh auth login`
|
|
157
153
|
- **PR already exists** -> Use: `gh pr view` to see status
|
|
158
|
-
- **
|
|
154
|
+
- **Claude Code CLI not installed** -> Install: `npm install -g @anthropic-ai/claude-code`
|
|
159
155
|
- **Rebase conflicts** -> Resolve carefully, re-run all tests after
|
|
160
156
|
|
|
161
157
|
## Learning Capture
|
package/package.json
CHANGED
|
@@ -64,27 +64,21 @@ EOF
|
|
|
64
64
|
)"
|
|
65
65
|
```
|
|
66
66
|
|
|
67
|
-
### Step 4: Request
|
|
67
|
+
### Step 4: Request Claude Code Review
|
|
68
68
|
|
|
69
|
-
Run automated code review using
|
|
69
|
+
Run automated code review using Claude Code CLI:
|
|
70
70
|
|
|
71
71
|
```bash
|
|
72
|
-
# Check if
|
|
73
|
-
if ! command -v
|
|
74
|
-
echo "WARNING:
|
|
75
|
-
cd {{appDir}} && gh pr comment --body "Manual review requested -
|
|
72
|
+
# Check if Claude Code CLI is installed
|
|
73
|
+
if ! command -v claude &> /dev/null; then
|
|
74
|
+
echo "WARNING: Claude Code CLI not installed. Manual review needed."
|
|
75
|
+
cd {{appDir}} && gh pr comment --body "Manual review requested - Claude Code CLI not available. Install: https://docs.anthropic.com/en/docs/claude-code/overview"
|
|
76
76
|
else
|
|
77
|
-
echo "Running
|
|
78
|
-
# Get diff summary for context
|
|
79
|
-
DIFF_SUMMARY=$(cd {{appDir}} && git diff main --stat | head -50)
|
|
77
|
+
echo "Running Claude Code review..."
|
|
80
78
|
|
|
81
|
-
|
|
82
|
-
echo "Code Review Request for $FEATURE feature.
|
|
79
|
+
cd {{appDir}} && claude -p "You are reviewing a PR for the $FEATURE feature.
|
|
83
80
|
|
|
84
|
-
|
|
85
|
-
$DIFF_SUMMARY
|
|
86
|
-
|
|
87
|
-
## Review Checklist:
|
|
81
|
+
Review the git diff against main and check:
|
|
88
82
|
- Code quality and patterns consistency
|
|
89
83
|
- Test coverage adequacy
|
|
90
84
|
- Potential bugs or edge cases
|
|
@@ -92,19 +86,21 @@ $DIFF_SUMMARY
|
|
|
92
86
|
- Performance implications
|
|
93
87
|
- Error handling completeness
|
|
94
88
|
|
|
95
|
-
|
|
89
|
+
Run: git diff main
|
|
90
|
+
|
|
91
|
+
Respond with:
|
|
96
92
|
- APPROVED if everything looks good
|
|
97
|
-
- Or list specific issues that need to be fixed"
|
|
93
|
+
- Or list specific issues with file:line references that need to be fixed"
|
|
98
94
|
fi
|
|
99
95
|
```
|
|
100
96
|
|
|
101
97
|
**Handle review feedback:**
|
|
102
|
-
- If
|
|
103
|
-
- If
|
|
98
|
+
- If Claude outputs "APPROVED" -> Proceed to Step 5 (rebase and merge)
|
|
99
|
+
- If Claude lists issues:
|
|
104
100
|
1. Address each issue with code fixes
|
|
105
101
|
2. Commit: `git -C {{appDir}} add -A && git -C {{appDir}} commit -m "fix($FEATURE): address review feedback"`
|
|
106
102
|
3. Push: `git -C {{appDir}} push origin feat/$FEATURE`
|
|
107
|
-
4. Re-run the
|
|
103
|
+
4. Re-run the Claude review command above
|
|
108
104
|
- Max 3 review iterations before requiring manual intervention
|
|
109
105
|
|
|
110
106
|
### Step 5: Rebase Before Merge (for Parallel Execution)
|
|
@@ -126,7 +122,7 @@ cd {{appDir}} && git push --force-with-lease origin feat/$FEATURE
|
|
|
126
122
|
```
|
|
127
123
|
|
|
128
124
|
### Step 6: Merge PR
|
|
129
|
-
When
|
|
125
|
+
When Claude review is approved and branch is rebased:
|
|
130
126
|
```bash
|
|
131
127
|
cd {{appDir}} && gh pr merge --squash --delete-branch
|
|
132
128
|
```
|
|
@@ -145,7 +141,7 @@ cd {{appDir}} && gh pr merge --squash --delete-branch
|
|
|
145
141
|
Note: Spec status updates are handled in the Spec Verification phase before PR creation.
|
|
146
142
|
|
|
147
143
|
## Rules
|
|
148
|
-
- Do NOT merge without
|
|
144
|
+
- Do NOT merge without Claude Code approval
|
|
149
145
|
- Address ALL review comments before merging
|
|
150
146
|
- Use squash merge to keep history clean
|
|
151
147
|
- If gh CLI fails, check authentication: `gh auth status`
|
|
@@ -155,7 +151,7 @@ Note: Spec status updates are handled in the Spec Verification phase before PR c
|
|
|
155
151
|
- **gh: command not found** -> Install GitHub CLI: `brew install gh`
|
|
156
152
|
- **gh auth error** -> Run: `gh auth login`
|
|
157
153
|
- **PR already exists** -> Use: `gh pr view` to see status
|
|
158
|
-
- **
|
|
154
|
+
- **Claude Code CLI not installed** -> Install: `npm install -g @anthropic-ai/claude-code`
|
|
159
155
|
- **Rebase conflicts** -> Resolve carefully, re-run all tests after
|
|
160
156
|
|
|
161
157
|
## Learning Capture
|