@lvlup-sw/exarchos 2.7.1 → 2.8.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/.claude-plugin/plugin.json +3 -2
- package/agents/fixer.md +2 -1
- package/agents/implementer.md +2 -1
- package/agents/reviewer.md +2 -1
- package/agents/scaffolder.md +55 -0
- package/commands/discover.md +5 -0
- package/commands/reload.md +0 -1
- package/commands/tdd.md +0 -1
- package/dist/exarchos.js +314 -335
- package/package.json +1 -1
- package/skills/claude/cleanup/SKILL.md +10 -4
- package/skills/claude/cleanup/references/merge-verification.md +8 -9
- package/skills/claude/debug/references/thorough-track.md +5 -2
- package/skills/claude/debug/references/troubleshooting.md +1 -1
- package/skills/claude/delegation/SKILL.md +4 -3
- package/skills/claude/delegation/references/agent-teams-saga.md +1 -3
- package/skills/claude/delegation/references/fixer-prompt.md +0 -1
- package/skills/claude/delegation/references/implementer-prompt.md +0 -1
- package/skills/claude/delegation/references/parallel-strategy.md +6 -11
- package/skills/claude/delegation/references/worked-example.md +3 -3
- package/skills/claude/delegation/references/workflow-steps.md +0 -2
- package/skills/claude/delegation/references/worktree-enforcement.md +1 -1
- package/skills/claude/discovery/SKILL.md +95 -0
- package/skills/claude/dogfood/SKILL.md +8 -2
- package/skills/claude/dogfood/references/report-template.md +4 -4
- package/skills/claude/oneshot-workflow/SKILL.md +9 -3
- package/skills/claude/prune-workflows/SKILL.md +6 -0
- package/skills/claude/quality-review/references/auto-transition.md +1 -1
- package/skills/claude/refactor/references/overhaul-track.md +1 -1
- package/skills/claude/shepherd/SKILL.md +14 -5
- package/skills/claude/shepherd/references/assess-checklist.md +7 -7
- package/skills/claude/shepherd/references/fix-strategies.md +4 -4
- package/skills/claude/synthesis/SKILL.md +27 -8
- package/skills/claude/synthesis/references/github-native-stacking.md +26 -30
- package/skills/claude/synthesis/references/merge-ordering.md +1 -1
- package/skills/claude/synthesis/references/synthesis-steps.md +6 -6
- package/skills/claude/synthesis/references/troubleshooting.md +11 -18
- package/skills/codex/cleanup/SKILL.md +10 -4
- package/skills/codex/cleanup/references/merge-verification.md +8 -9
- package/skills/codex/debug/references/thorough-track.md +5 -2
- package/skills/codex/debug/references/troubleshooting.md +1 -1
- package/skills/codex/delegation/SKILL.md +4 -3
- package/skills/codex/delegation/references/agent-teams-saga.md +1 -3
- package/skills/codex/delegation/references/fixer-prompt.md +0 -1
- package/skills/codex/delegation/references/implementer-prompt.md +0 -1
- package/skills/codex/delegation/references/parallel-strategy.md +6 -11
- package/skills/codex/delegation/references/worked-example.md +3 -3
- package/skills/codex/delegation/references/workflow-steps.md +0 -2
- package/skills/codex/delegation/references/worktree-enforcement.md +1 -1
- package/skills/codex/discovery/SKILL.md +95 -0
- package/skills/codex/dogfood/SKILL.md +8 -2
- package/skills/codex/dogfood/references/report-template.md +4 -4
- package/skills/codex/oneshot-workflow/SKILL.md +9 -3
- package/skills/codex/prune-workflows/SKILL.md +6 -0
- package/skills/codex/quality-review/references/auto-transition.md +1 -1
- package/skills/codex/refactor/references/overhaul-track.md +1 -1
- package/skills/codex/shepherd/SKILL.md +14 -5
- package/skills/codex/shepherd/references/assess-checklist.md +7 -7
- package/skills/codex/shepherd/references/fix-strategies.md +4 -4
- package/skills/codex/synthesis/SKILL.md +27 -8
- package/skills/codex/synthesis/references/github-native-stacking.md +26 -30
- package/skills/codex/synthesis/references/merge-ordering.md +1 -1
- package/skills/codex/synthesis/references/synthesis-steps.md +6 -6
- package/skills/codex/synthesis/references/troubleshooting.md +11 -18
- package/skills/copilot/cleanup/SKILL.md +10 -4
- package/skills/copilot/cleanup/references/merge-verification.md +8 -9
- package/skills/copilot/debug/references/thorough-track.md +5 -2
- package/skills/copilot/debug/references/troubleshooting.md +1 -1
- package/skills/copilot/delegation/SKILL.md +4 -3
- package/skills/copilot/delegation/references/agent-teams-saga.md +1 -3
- package/skills/copilot/delegation/references/fixer-prompt.md +0 -1
- package/skills/copilot/delegation/references/implementer-prompt.md +0 -1
- package/skills/copilot/delegation/references/parallel-strategy.md +6 -11
- package/skills/copilot/delegation/references/worked-example.md +3 -3
- package/skills/copilot/delegation/references/workflow-steps.md +0 -2
- package/skills/copilot/delegation/references/worktree-enforcement.md +1 -1
- package/skills/copilot/discovery/SKILL.md +95 -0
- package/skills/copilot/dogfood/SKILL.md +8 -2
- package/skills/copilot/dogfood/references/report-template.md +4 -4
- package/skills/copilot/oneshot-workflow/SKILL.md +9 -3
- package/skills/copilot/prune-workflows/SKILL.md +6 -0
- package/skills/copilot/quality-review/references/auto-transition.md +1 -1
- package/skills/copilot/refactor/references/overhaul-track.md +1 -1
- package/skills/copilot/shepherd/SKILL.md +14 -5
- package/skills/copilot/shepherd/references/assess-checklist.md +7 -7
- package/skills/copilot/shepherd/references/fix-strategies.md +4 -4
- package/skills/copilot/synthesis/SKILL.md +27 -8
- package/skills/copilot/synthesis/references/github-native-stacking.md +26 -30
- package/skills/copilot/synthesis/references/merge-ordering.md +1 -1
- package/skills/copilot/synthesis/references/synthesis-steps.md +6 -6
- package/skills/copilot/synthesis/references/troubleshooting.md +11 -18
- package/skills/cursor/cleanup/SKILL.md +10 -4
- package/skills/cursor/cleanup/references/merge-verification.md +8 -9
- package/skills/cursor/debug/references/thorough-track.md +5 -2
- package/skills/cursor/debug/references/troubleshooting.md +1 -1
- package/skills/cursor/delegation/SKILL.md +4 -3
- package/skills/cursor/delegation/references/agent-teams-saga.md +1 -3
- package/skills/cursor/delegation/references/fixer-prompt.md +0 -1
- package/skills/cursor/delegation/references/implementer-prompt.md +0 -1
- package/skills/cursor/delegation/references/parallel-strategy.md +6 -11
- package/skills/cursor/delegation/references/worked-example.md +3 -3
- package/skills/cursor/delegation/references/workflow-steps.md +0 -2
- package/skills/cursor/delegation/references/worktree-enforcement.md +1 -1
- package/skills/cursor/discovery/SKILL.md +95 -0
- package/skills/cursor/dogfood/SKILL.md +8 -2
- package/skills/cursor/dogfood/references/report-template.md +4 -4
- package/skills/cursor/oneshot-workflow/SKILL.md +9 -3
- package/skills/cursor/prune-workflows/SKILL.md +6 -0
- package/skills/cursor/quality-review/references/auto-transition.md +1 -1
- package/skills/cursor/refactor/references/overhaul-track.md +1 -1
- package/skills/cursor/shepherd/SKILL.md +14 -5
- package/skills/cursor/shepherd/references/assess-checklist.md +7 -7
- package/skills/cursor/shepherd/references/fix-strategies.md +4 -4
- package/skills/cursor/synthesis/SKILL.md +27 -8
- package/skills/cursor/synthesis/references/github-native-stacking.md +26 -30
- package/skills/cursor/synthesis/references/merge-ordering.md +1 -1
- package/skills/cursor/synthesis/references/synthesis-steps.md +6 -6
- package/skills/cursor/synthesis/references/troubleshooting.md +11 -18
- package/skills/generic/cleanup/SKILL.md +10 -4
- package/skills/generic/cleanup/references/merge-verification.md +8 -9
- package/skills/generic/debug/references/thorough-track.md +5 -2
- package/skills/generic/debug/references/troubleshooting.md +1 -1
- package/skills/generic/delegation/SKILL.md +4 -3
- package/skills/generic/delegation/references/agent-teams-saga.md +1 -3
- package/skills/generic/delegation/references/fixer-prompt.md +0 -1
- package/skills/generic/delegation/references/implementer-prompt.md +0 -1
- package/skills/generic/delegation/references/parallel-strategy.md +6 -11
- package/skills/generic/delegation/references/worked-example.md +3 -3
- package/skills/generic/delegation/references/workflow-steps.md +0 -2
- package/skills/generic/delegation/references/worktree-enforcement.md +1 -1
- package/skills/generic/discovery/SKILL.md +95 -0
- package/skills/generic/dogfood/SKILL.md +8 -2
- package/skills/generic/dogfood/references/report-template.md +4 -4
- package/skills/generic/oneshot-workflow/SKILL.md +9 -3
- package/skills/generic/prune-workflows/SKILL.md +6 -0
- package/skills/generic/quality-review/references/auto-transition.md +1 -1
- package/skills/generic/refactor/references/overhaul-track.md +1 -1
- package/skills/generic/shepherd/SKILL.md +14 -5
- package/skills/generic/shepherd/references/assess-checklist.md +7 -7
- package/skills/generic/shepherd/references/fix-strategies.md +4 -4
- package/skills/generic/synthesis/SKILL.md +27 -8
- package/skills/generic/synthesis/references/github-native-stacking.md +26 -30
- package/skills/generic/synthesis/references/merge-ordering.md +1 -1
- package/skills/generic/synthesis/references/synthesis-steps.md +6 -6
- package/skills/generic/synthesis/references/troubleshooting.md +11 -18
- package/skills/opencode/cleanup/SKILL.md +10 -4
- package/skills/opencode/cleanup/references/merge-verification.md +8 -9
- package/skills/opencode/debug/references/thorough-track.md +5 -2
- package/skills/opencode/debug/references/troubleshooting.md +1 -1
- package/skills/opencode/delegation/SKILL.md +4 -3
- package/skills/opencode/delegation/references/agent-teams-saga.md +1 -3
- package/skills/opencode/delegation/references/fixer-prompt.md +0 -1
- package/skills/opencode/delegation/references/implementer-prompt.md +0 -1
- package/skills/opencode/delegation/references/parallel-strategy.md +6 -11
- package/skills/opencode/delegation/references/worked-example.md +3 -3
- package/skills/opencode/delegation/references/workflow-steps.md +0 -2
- package/skills/opencode/delegation/references/worktree-enforcement.md +1 -1
- package/skills/opencode/discovery/SKILL.md +95 -0
- package/skills/opencode/dogfood/SKILL.md +8 -2
- package/skills/opencode/dogfood/references/report-template.md +4 -4
- package/skills/opencode/oneshot-workflow/SKILL.md +9 -3
- package/skills/opencode/prune-workflows/SKILL.md +6 -0
- package/skills/opencode/quality-review/references/auto-transition.md +1 -1
- package/skills/opencode/refactor/references/overhaul-track.md +1 -1
- package/skills/opencode/shepherd/SKILL.md +14 -5
- package/skills/opencode/shepherd/references/assess-checklist.md +7 -7
- package/skills/opencode/shepherd/references/fix-strategies.md +4 -4
- package/skills/opencode/synthesis/SKILL.md +27 -8
- package/skills/opencode/synthesis/references/github-native-stacking.md +26 -30
- package/skills/opencode/synthesis/references/merge-ordering.md +1 -1
- package/skills/opencode/synthesis/references/synthesis-steps.md +6 -6
- package/skills/opencode/synthesis/references/troubleshooting.md +11 -18
- package/skills/claude/delegation/references/pr-fixes-mode.md +0 -155
- package/skills/codex/delegation/references/pr-fixes-mode.md +0 -155
- package/skills/copilot/delegation/references/pr-fixes-mode.md +0 -155
- package/skills/cursor/delegation/references/pr-fixes-mode.md +0 -155
- package/skills/generic/delegation/references/pr-fixes-mode.md +0 -155
- package/skills/opencode/delegation/references/pr-fixes-mode.md +0 -155
|
@@ -1,155 +0,0 @@
|
|
|
1
|
-
# PR Feedback Mode (--pr-fixes)
|
|
2
|
-
|
|
3
|
-
When invoked with `--pr-fixes [PR_URL]`, delegation addresses human review feedback from a pull request instead of implementing from a plan.
|
|
4
|
-
|
|
5
|
-
## Priority Levels
|
|
6
|
-
|
|
7
|
-
| Priority | Source | Description |
|
|
8
|
-
|----------|--------|-------------|
|
|
9
|
-
| 1 | `coderabbit:critical` | Critical issues + SPEC COMPLIANCE failures |
|
|
10
|
-
| 2 | `human` | Human reviewer comments (authority over automation) |
|
|
11
|
-
| 3 | `coderabbit:major` | Major issues + CODE QUALITY HIGH items |
|
|
12
|
-
| 4 | `coderabbit:minor` | Minor issues |
|
|
13
|
-
|
|
14
|
-
## Process
|
|
15
|
-
|
|
16
|
-
### Step 1: Fetch All PR Feedback
|
|
17
|
-
|
|
18
|
-
```bash
|
|
19
|
-
# Get full PR details including reviews and comments
|
|
20
|
-
gh pr view <number> --json title,body,state,files,reviewDecision,reviews,comments
|
|
21
|
-
|
|
22
|
-
# Get issue-level comments (pre-merge check summaries)
|
|
23
|
-
gh issue view <number> --json comments
|
|
24
|
-
```
|
|
25
|
-
|
|
26
|
-
> Or use GitHub MCP `pull_request_read` and `issue_read` if available.
|
|
27
|
-
|
|
28
|
-
### Step 2: Parse CodeRabbit Feedback
|
|
29
|
-
|
|
30
|
-
**2a: Identify CodeRabbit comments** by author = `coderabbitai[bot]`
|
|
31
|
-
|
|
32
|
-
**2b: Parse line comments by severity label:**
|
|
33
|
-
|
|
34
|
-
| Label | Priority |
|
|
35
|
-
|-------|----------|
|
|
36
|
-
| `Critical` | 1 |
|
|
37
|
-
| `Major` | 2 |
|
|
38
|
-
| `Minor` | 3 |
|
|
39
|
-
|
|
40
|
-
Extract from each comment:
|
|
41
|
-
- Severity label (emoji + text)
|
|
42
|
-
- File path and line number
|
|
43
|
-
- Issue description
|
|
44
|
-
- Suggested fix (from Proposed fix section if present)
|
|
45
|
-
|
|
46
|
-
**2c: Parse pre-merge check summaries:**
|
|
47
|
-
- `Status: FAIL` in Spec Review -> Priority 1
|
|
48
|
-
- `Status: NEEDS_FIXES | BLOCKED` in Quality Review -> Priority 2
|
|
49
|
-
- Extract items from `Missing:`, `Untested:`, `Scope Creep:` lists
|
|
50
|
-
|
|
51
|
-
**2d: Parse reviews with `CHANGES_REQUESTED`:**
|
|
52
|
-
- Note "Actionable comments posted: N" count
|
|
53
|
-
- Cross-reference with line comments already parsed
|
|
54
|
-
|
|
55
|
-
### Step 3: Parse Human Comments (Priority 2)
|
|
56
|
-
|
|
57
|
-
For non-CodeRabbit comments, assign `priority: 2` (human authority over automation).
|
|
58
|
-
|
|
59
|
-
Skip comments that are:
|
|
60
|
-
- Purely praise/acknowledgment ("LGTM", "Nice work")
|
|
61
|
-
- Questions without action items
|
|
62
|
-
- Already marked resolved
|
|
63
|
-
- From other bots (CI notifications, etc.)
|
|
64
|
-
|
|
65
|
-
### Step 4: Create Fix Tasks
|
|
66
|
-
|
|
67
|
-
For each actionable item, create a structured fix task:
|
|
68
|
-
|
|
69
|
-
| Field | Description |
|
|
70
|
-
|-------|-------------|
|
|
71
|
-
| `id` | `fix-001`, `fix-002`, etc. |
|
|
72
|
-
| `priority` | `1` (critical), `2` (human), `3` (major), `4` (minor) |
|
|
73
|
-
| `source` | `"coderabbit:critical"`, `"human"`, `"coderabbit:major"`, `"coderabbit:minor"` |
|
|
74
|
-
| `severity` | `"Critical"`, `"Major"`, `"Minor"`, or null |
|
|
75
|
-
| `file` | File path |
|
|
76
|
-
| `line` | Line number (if line comment) |
|
|
77
|
-
| `issue` | Problem description |
|
|
78
|
-
| `action` | Required change / suggested fix |
|
|
79
|
-
|
|
80
|
-
### Step 5: Sort and Display Fix Tasks
|
|
81
|
-
|
|
82
|
-
Sort by priority (1->4), then by file path for grouping.
|
|
83
|
-
|
|
84
|
-
### Step 6: Track Fix Tasks
|
|
85
|
-
|
|
86
|
-
Use TodoWrite to track all fix tasks with priority labels.
|
|
87
|
-
|
|
88
|
-
### Step 7: Dispatch Fixes (MANDATORY - Priority Order)
|
|
89
|
-
|
|
90
|
-
**Dispatch sequence:**
|
|
91
|
-
1. Dispatch all P1 (critical) fixes in parallel
|
|
92
|
-
2. Wait for completion
|
|
93
|
-
3. Dispatch all P2 (human) fixes in parallel
|
|
94
|
-
4. Wait for completion
|
|
95
|
-
5. Dispatch all P3 (major) fixes in parallel
|
|
96
|
-
6. Wait for completion
|
|
97
|
-
7. Dispatch all P4 (minor) fixes in parallel
|
|
98
|
-
8. Wait for completion
|
|
99
|
-
|
|
100
|
-
**Task prompt template:**
|
|
101
|
-
```typescript
|
|
102
|
-
Task({
|
|
103
|
-
subagent_type: "general-purpose",
|
|
104
|
-
model: "opus",
|
|
105
|
-
description: "Fix [P{priority} {severity}]: {issue summary}",
|
|
106
|
-
prompt: `
|
|
107
|
-
# Task: Fix PR Feedback - {issue summary}
|
|
108
|
-
|
|
109
|
-
## Priority
|
|
110
|
-
{priority} - {source}
|
|
111
|
-
|
|
112
|
-
## Context
|
|
113
|
-
PR: {PR_URL}
|
|
114
|
-
Source: {source}
|
|
115
|
-
Original feedback: "{original comment text}"
|
|
116
|
-
|
|
117
|
-
## Working Directory
|
|
118
|
-
{absolute path to repo}
|
|
119
|
-
|
|
120
|
-
## Fix Required
|
|
121
|
-
File: {file path}
|
|
122
|
-
Line: {line number if applicable}
|
|
123
|
-
Issue: {issue description}
|
|
124
|
-
Action: {required change}
|
|
125
|
-
|
|
126
|
-
## TDD Requirements
|
|
127
|
-
1. Write a test that would catch this issue (if applicable)
|
|
128
|
-
2. Verify test fails
|
|
129
|
-
3. Implement the fix
|
|
130
|
-
4. Verify test passes
|
|
131
|
-
|
|
132
|
-
## Success Criteria
|
|
133
|
-
- [ ] Issue addressed per feedback
|
|
134
|
-
- [ ] Tests pass
|
|
135
|
-
- [ ] No regressions introduced
|
|
136
|
-
`
|
|
137
|
-
})
|
|
138
|
-
```
|
|
139
|
-
|
|
140
|
-
**CHECKPOINT:** Do NOT proceed to next priority level until all fixes in the current level have completed.
|
|
141
|
-
|
|
142
|
-
### Step 8: Push and Report
|
|
143
|
-
|
|
144
|
-
After all fixes complete, commit and push:
|
|
145
|
-
```bash
|
|
146
|
-
git add <fixed-files>
|
|
147
|
-
git commit -m "fix: address PR review feedback"
|
|
148
|
-
git push
|
|
149
|
-
```
|
|
150
|
-
|
|
151
|
-
Report to user: total fixes by priority, files modified. Then auto-chain to `/exarchos:synthesize` for merge confirmation.
|
|
152
|
-
|
|
153
|
-
### Handling Missing CodeRabbit Comments
|
|
154
|
-
|
|
155
|
-
If no CodeRabbit comments found, proceed with human comments only.
|
|
@@ -1,155 +0,0 @@
|
|
|
1
|
-
# PR Feedback Mode (--pr-fixes)
|
|
2
|
-
|
|
3
|
-
When invoked with `--pr-fixes [PR_URL]`, delegation addresses human review feedback from a pull request instead of implementing from a plan.
|
|
4
|
-
|
|
5
|
-
## Priority Levels
|
|
6
|
-
|
|
7
|
-
| Priority | Source | Description |
|
|
8
|
-
|----------|--------|-------------|
|
|
9
|
-
| 1 | `coderabbit:critical` | Critical issues + SPEC COMPLIANCE failures |
|
|
10
|
-
| 2 | `human` | Human reviewer comments (authority over automation) |
|
|
11
|
-
| 3 | `coderabbit:major` | Major issues + CODE QUALITY HIGH items |
|
|
12
|
-
| 4 | `coderabbit:minor` | Minor issues |
|
|
13
|
-
|
|
14
|
-
## Process
|
|
15
|
-
|
|
16
|
-
### Step 1: Fetch All PR Feedback
|
|
17
|
-
|
|
18
|
-
```bash
|
|
19
|
-
# Get full PR details including reviews and comments
|
|
20
|
-
gh pr view <number> --json title,body,state,files,reviewDecision,reviews,comments
|
|
21
|
-
|
|
22
|
-
# Get issue-level comments (pre-merge check summaries)
|
|
23
|
-
gh issue view <number> --json comments
|
|
24
|
-
```
|
|
25
|
-
|
|
26
|
-
> Or use GitHub MCP `pull_request_read` and `issue_read` if available.
|
|
27
|
-
|
|
28
|
-
### Step 2: Parse CodeRabbit Feedback
|
|
29
|
-
|
|
30
|
-
**2a: Identify CodeRabbit comments** by author = `coderabbitai[bot]`
|
|
31
|
-
|
|
32
|
-
**2b: Parse line comments by severity label:**
|
|
33
|
-
|
|
34
|
-
| Label | Priority |
|
|
35
|
-
|-------|----------|
|
|
36
|
-
| `Critical` | 1 |
|
|
37
|
-
| `Major` | 2 |
|
|
38
|
-
| `Minor` | 3 |
|
|
39
|
-
|
|
40
|
-
Extract from each comment:
|
|
41
|
-
- Severity label (emoji + text)
|
|
42
|
-
- File path and line number
|
|
43
|
-
- Issue description
|
|
44
|
-
- Suggested fix (from Proposed fix section if present)
|
|
45
|
-
|
|
46
|
-
**2c: Parse pre-merge check summaries:**
|
|
47
|
-
- `Status: FAIL` in Spec Review -> Priority 1
|
|
48
|
-
- `Status: NEEDS_FIXES | BLOCKED` in Quality Review -> Priority 2
|
|
49
|
-
- Extract items from `Missing:`, `Untested:`, `Scope Creep:` lists
|
|
50
|
-
|
|
51
|
-
**2d: Parse reviews with `CHANGES_REQUESTED`:**
|
|
52
|
-
- Note "Actionable comments posted: N" count
|
|
53
|
-
- Cross-reference with line comments already parsed
|
|
54
|
-
|
|
55
|
-
### Step 3: Parse Human Comments (Priority 2)
|
|
56
|
-
|
|
57
|
-
For non-CodeRabbit comments, assign `priority: 2` (human authority over automation).
|
|
58
|
-
|
|
59
|
-
Skip comments that are:
|
|
60
|
-
- Purely praise/acknowledgment ("LGTM", "Nice work")
|
|
61
|
-
- Questions without action items
|
|
62
|
-
- Already marked resolved
|
|
63
|
-
- From other bots (CI notifications, etc.)
|
|
64
|
-
|
|
65
|
-
### Step 4: Create Fix Tasks
|
|
66
|
-
|
|
67
|
-
For each actionable item, create a structured fix task:
|
|
68
|
-
|
|
69
|
-
| Field | Description |
|
|
70
|
-
|-------|-------------|
|
|
71
|
-
| `id` | `fix-001`, `fix-002`, etc. |
|
|
72
|
-
| `priority` | `1` (critical), `2` (human), `3` (major), `4` (minor) |
|
|
73
|
-
| `source` | `"coderabbit:critical"`, `"human"`, `"coderabbit:major"`, `"coderabbit:minor"` |
|
|
74
|
-
| `severity` | `"Critical"`, `"Major"`, `"Minor"`, or null |
|
|
75
|
-
| `file` | File path |
|
|
76
|
-
| `line` | Line number (if line comment) |
|
|
77
|
-
| `issue` | Problem description |
|
|
78
|
-
| `action` | Required change / suggested fix |
|
|
79
|
-
|
|
80
|
-
### Step 5: Sort and Display Fix Tasks
|
|
81
|
-
|
|
82
|
-
Sort by priority (1->4), then by file path for grouping.
|
|
83
|
-
|
|
84
|
-
### Step 6: Track Fix Tasks
|
|
85
|
-
|
|
86
|
-
Use TodoWrite to track all fix tasks with priority labels.
|
|
87
|
-
|
|
88
|
-
### Step 7: Dispatch Fixes (MANDATORY - Priority Order)
|
|
89
|
-
|
|
90
|
-
**Dispatch sequence:**
|
|
91
|
-
1. Dispatch all P1 (critical) fixes in parallel
|
|
92
|
-
2. Wait for completion
|
|
93
|
-
3. Dispatch all P2 (human) fixes in parallel
|
|
94
|
-
4. Wait for completion
|
|
95
|
-
5. Dispatch all P3 (major) fixes in parallel
|
|
96
|
-
6. Wait for completion
|
|
97
|
-
7. Dispatch all P4 (minor) fixes in parallel
|
|
98
|
-
8. Wait for completion
|
|
99
|
-
|
|
100
|
-
**Task prompt template:**
|
|
101
|
-
```typescript
|
|
102
|
-
Task({
|
|
103
|
-
subagent_type: "general-purpose",
|
|
104
|
-
model: "opus",
|
|
105
|
-
description: "Fix [P{priority} {severity}]: {issue summary}",
|
|
106
|
-
prompt: `
|
|
107
|
-
# Task: Fix PR Feedback - {issue summary}
|
|
108
|
-
|
|
109
|
-
## Priority
|
|
110
|
-
{priority} - {source}
|
|
111
|
-
|
|
112
|
-
## Context
|
|
113
|
-
PR: {PR_URL}
|
|
114
|
-
Source: {source}
|
|
115
|
-
Original feedback: "{original comment text}"
|
|
116
|
-
|
|
117
|
-
## Working Directory
|
|
118
|
-
{absolute path to repo}
|
|
119
|
-
|
|
120
|
-
## Fix Required
|
|
121
|
-
File: {file path}
|
|
122
|
-
Line: {line number if applicable}
|
|
123
|
-
Issue: {issue description}
|
|
124
|
-
Action: {required change}
|
|
125
|
-
|
|
126
|
-
## TDD Requirements
|
|
127
|
-
1. Write a test that would catch this issue (if applicable)
|
|
128
|
-
2. Verify test fails
|
|
129
|
-
3. Implement the fix
|
|
130
|
-
4. Verify test passes
|
|
131
|
-
|
|
132
|
-
## Success Criteria
|
|
133
|
-
- [ ] Issue addressed per feedback
|
|
134
|
-
- [ ] Tests pass
|
|
135
|
-
- [ ] No regressions introduced
|
|
136
|
-
`
|
|
137
|
-
})
|
|
138
|
-
```
|
|
139
|
-
|
|
140
|
-
**CHECKPOINT:** Do NOT proceed to next priority level until all fixes in the current level have completed.
|
|
141
|
-
|
|
142
|
-
### Step 8: Push and Report
|
|
143
|
-
|
|
144
|
-
After all fixes complete, commit and push:
|
|
145
|
-
```bash
|
|
146
|
-
git add <fixed-files>
|
|
147
|
-
git commit -m "fix: address PR review feedback"
|
|
148
|
-
git push
|
|
149
|
-
```
|
|
150
|
-
|
|
151
|
-
Report to user: total fixes by priority, files modified. Then auto-chain to `/exarchos:synthesize` for merge confirmation.
|
|
152
|
-
|
|
153
|
-
### Handling Missing CodeRabbit Comments
|
|
154
|
-
|
|
155
|
-
If no CodeRabbit comments found, proceed with human comments only.
|
|
@@ -1,155 +0,0 @@
|
|
|
1
|
-
# PR Feedback Mode (--pr-fixes)
|
|
2
|
-
|
|
3
|
-
When invoked with `--pr-fixes [PR_URL]`, delegation addresses human review feedback from a pull request instead of implementing from a plan.
|
|
4
|
-
|
|
5
|
-
## Priority Levels
|
|
6
|
-
|
|
7
|
-
| Priority | Source | Description |
|
|
8
|
-
|----------|--------|-------------|
|
|
9
|
-
| 1 | `coderabbit:critical` | Critical issues + SPEC COMPLIANCE failures |
|
|
10
|
-
| 2 | `human` | Human reviewer comments (authority over automation) |
|
|
11
|
-
| 3 | `coderabbit:major` | Major issues + CODE QUALITY HIGH items |
|
|
12
|
-
| 4 | `coderabbit:minor` | Minor issues |
|
|
13
|
-
|
|
14
|
-
## Process
|
|
15
|
-
|
|
16
|
-
### Step 1: Fetch All PR Feedback
|
|
17
|
-
|
|
18
|
-
```bash
|
|
19
|
-
# Get full PR details including reviews and comments
|
|
20
|
-
gh pr view <number> --json title,body,state,files,reviewDecision,reviews,comments
|
|
21
|
-
|
|
22
|
-
# Get issue-level comments (pre-merge check summaries)
|
|
23
|
-
gh issue view <number> --json comments
|
|
24
|
-
```
|
|
25
|
-
|
|
26
|
-
> Or use GitHub MCP `pull_request_read` and `issue_read` if available.
|
|
27
|
-
|
|
28
|
-
### Step 2: Parse CodeRabbit Feedback
|
|
29
|
-
|
|
30
|
-
**2a: Identify CodeRabbit comments** by author = `coderabbitai[bot]`
|
|
31
|
-
|
|
32
|
-
**2b: Parse line comments by severity label:**
|
|
33
|
-
|
|
34
|
-
| Label | Priority |
|
|
35
|
-
|-------|----------|
|
|
36
|
-
| `Critical` | 1 |
|
|
37
|
-
| `Major` | 2 |
|
|
38
|
-
| `Minor` | 3 |
|
|
39
|
-
|
|
40
|
-
Extract from each comment:
|
|
41
|
-
- Severity label (emoji + text)
|
|
42
|
-
- File path and line number
|
|
43
|
-
- Issue description
|
|
44
|
-
- Suggested fix (from Proposed fix section if present)
|
|
45
|
-
|
|
46
|
-
**2c: Parse pre-merge check summaries:**
|
|
47
|
-
- `Status: FAIL` in Spec Review -> Priority 1
|
|
48
|
-
- `Status: NEEDS_FIXES | BLOCKED` in Quality Review -> Priority 2
|
|
49
|
-
- Extract items from `Missing:`, `Untested:`, `Scope Creep:` lists
|
|
50
|
-
|
|
51
|
-
**2d: Parse reviews with `CHANGES_REQUESTED`:**
|
|
52
|
-
- Note "Actionable comments posted: N" count
|
|
53
|
-
- Cross-reference with line comments already parsed
|
|
54
|
-
|
|
55
|
-
### Step 3: Parse Human Comments (Priority 2)
|
|
56
|
-
|
|
57
|
-
For non-CodeRabbit comments, assign `priority: 2` (human authority over automation).
|
|
58
|
-
|
|
59
|
-
Skip comments that are:
|
|
60
|
-
- Purely praise/acknowledgment ("LGTM", "Nice work")
|
|
61
|
-
- Questions without action items
|
|
62
|
-
- Already marked resolved
|
|
63
|
-
- From other bots (CI notifications, etc.)
|
|
64
|
-
|
|
65
|
-
### Step 4: Create Fix Tasks
|
|
66
|
-
|
|
67
|
-
For each actionable item, create a structured fix task:
|
|
68
|
-
|
|
69
|
-
| Field | Description |
|
|
70
|
-
|-------|-------------|
|
|
71
|
-
| `id` | `fix-001`, `fix-002`, etc. |
|
|
72
|
-
| `priority` | `1` (critical), `2` (human), `3` (major), `4` (minor) |
|
|
73
|
-
| `source` | `"coderabbit:critical"`, `"human"`, `"coderabbit:major"`, `"coderabbit:minor"` |
|
|
74
|
-
| `severity` | `"Critical"`, `"Major"`, `"Minor"`, or null |
|
|
75
|
-
| `file` | File path |
|
|
76
|
-
| `line` | Line number (if line comment) |
|
|
77
|
-
| `issue` | Problem description |
|
|
78
|
-
| `action` | Required change / suggested fix |
|
|
79
|
-
|
|
80
|
-
### Step 5: Sort and Display Fix Tasks
|
|
81
|
-
|
|
82
|
-
Sort by priority (1->4), then by file path for grouping.
|
|
83
|
-
|
|
84
|
-
### Step 6: Track Fix Tasks
|
|
85
|
-
|
|
86
|
-
Use TodoWrite to track all fix tasks with priority labels.
|
|
87
|
-
|
|
88
|
-
### Step 7: Dispatch Fixes (MANDATORY - Priority Order)
|
|
89
|
-
|
|
90
|
-
**Dispatch sequence:**
|
|
91
|
-
1. Dispatch all P1 (critical) fixes in parallel
|
|
92
|
-
2. Wait for completion
|
|
93
|
-
3. Dispatch all P2 (human) fixes in parallel
|
|
94
|
-
4. Wait for completion
|
|
95
|
-
5. Dispatch all P3 (major) fixes in parallel
|
|
96
|
-
6. Wait for completion
|
|
97
|
-
7. Dispatch all P4 (minor) fixes in parallel
|
|
98
|
-
8. Wait for completion
|
|
99
|
-
|
|
100
|
-
**Task prompt template:**
|
|
101
|
-
```typescript
|
|
102
|
-
Task({
|
|
103
|
-
subagent_type: "general-purpose",
|
|
104
|
-
model: "opus",
|
|
105
|
-
description: "Fix [P{priority} {severity}]: {issue summary}",
|
|
106
|
-
prompt: `
|
|
107
|
-
# Task: Fix PR Feedback - {issue summary}
|
|
108
|
-
|
|
109
|
-
## Priority
|
|
110
|
-
{priority} - {source}
|
|
111
|
-
|
|
112
|
-
## Context
|
|
113
|
-
PR: {PR_URL}
|
|
114
|
-
Source: {source}
|
|
115
|
-
Original feedback: "{original comment text}"
|
|
116
|
-
|
|
117
|
-
## Working Directory
|
|
118
|
-
{absolute path to repo}
|
|
119
|
-
|
|
120
|
-
## Fix Required
|
|
121
|
-
File: {file path}
|
|
122
|
-
Line: {line number if applicable}
|
|
123
|
-
Issue: {issue description}
|
|
124
|
-
Action: {required change}
|
|
125
|
-
|
|
126
|
-
## TDD Requirements
|
|
127
|
-
1. Write a test that would catch this issue (if applicable)
|
|
128
|
-
2. Verify test fails
|
|
129
|
-
3. Implement the fix
|
|
130
|
-
4. Verify test passes
|
|
131
|
-
|
|
132
|
-
## Success Criteria
|
|
133
|
-
- [ ] Issue addressed per feedback
|
|
134
|
-
- [ ] Tests pass
|
|
135
|
-
- [ ] No regressions introduced
|
|
136
|
-
`
|
|
137
|
-
})
|
|
138
|
-
```
|
|
139
|
-
|
|
140
|
-
**CHECKPOINT:** Do NOT proceed to next priority level until all fixes in the current level have completed.
|
|
141
|
-
|
|
142
|
-
### Step 8: Push and Report
|
|
143
|
-
|
|
144
|
-
After all fixes complete, commit and push:
|
|
145
|
-
```bash
|
|
146
|
-
git add <fixed-files>
|
|
147
|
-
git commit -m "fix: address PR review feedback"
|
|
148
|
-
git push
|
|
149
|
-
```
|
|
150
|
-
|
|
151
|
-
Report to user: total fixes by priority, files modified. Then auto-chain to `/exarchos:synthesize` for merge confirmation.
|
|
152
|
-
|
|
153
|
-
### Handling Missing CodeRabbit Comments
|
|
154
|
-
|
|
155
|
-
If no CodeRabbit comments found, proceed with human comments only.
|