Loading...
Loading...
[v3] Resolve all PR comments using parallel agents with full workflow and verification gate
npx skill4agent add skinnyandbald/fish-skills pr-resolutionDEFAULT WORKFLOW for resolving PR comments with parallel execution.
| Action | Command |
|---|---|
| Get comments | |
| Parse CodeRabbit | |
| Check CI | |
| Resolve thread | |
| Resolve all threads | |
Phase 0: Pre-Flight → GoodToGo status check (if installed, otherwise skip)
Phase 1: Discovery → Gather comments, parse bot formats, enumerate
Phase 2: Classification → Categorize by priority, group by file
Phase 3: Resolution → Launch parallel agents by file group
Phase 4: Verification → Local checks + GoodToGo gate (if installed)
Phase 5: Completion → Commit, push, resolve threadsgtgreferences/goodtogo.mdif command -v gtg &> /dev/null; then
# gtg auto-detects repo from git remote
GTG_RESULT=$(gtg $PR_NUM --format json 2>/dev/null)
GTG_STATUS=$(echo "$GTG_RESULT" | jq -r '.status')
figtgREADYCI_FAILINGACTION_REQUIREDUNRESOLVED_THREADSreferences/discovery.mdreferences/bot-formats.mdreferences/classification.md## Parallel Execution Plan
### Group A: src/api/route.ts (3 comments → 1 agent)
- #1 [blocking] Line 45 - Add error handling
- #3 [suggestion] Line 67 - Improve validation
### Group B: src/components/Button.tsx (1 comment → 1 agent)
- #2 [suggestion] Line 23 - Add prop types
### Group C: CI Failures (if any → 1 agent)
- Fix lint/type errors
Total: 3 parallel agentsAgent 1: "Fix comments on src/api/route.ts"
- Comment #1: Add error handling at line 45
- Comment #3: Improve validation at line 67
Agent 2: "Fix comments on src/components/Button.tsx"
- Comment #2: Add prop types at line 23
Agent 3: "Fix CI failures"
- Lint errors
- Type errors| Condition | Execution |
|---|---|
| Same file | → Same agent (avoid conflicts) |
| Different files | → Parallel agents |
| CI failures | → Dedicated agent |
| Questions | → Ask human first |
references/verification.mdgtgreferences/goodtogo.mdreferences/completion.md~/.claude/skills/pr-resolution/bin/resolve-all-threads $PR_NUMbin/resolve-pr-thread THREAD_ID## Discovery
1. [blocking] src/api/route.ts:45 - Security issue
2. [suggestion] src/api/route.ts:67 - Add validation
3. [suggestion] src/components/Form.tsx:23 - Add types
4. [nitpick] src/utils/format.ts:12 - Typo
5. [question] src/lib/auth.ts:89 - "Handle null?"
6. CI: Lint error
## Parallel Plan (after asking human about #5)
- Agent 1: src/api/route.ts (#1, #2)
- Agent 2: src/components/Form.tsx (#3)
- Agent 3: src/utils/format.ts (#4)
- Agent 4: src/lib/auth.ts (#5 - if fix)
- Agent 5: CI fix (#6)
## Execution
Launch agents in parallel → Wait → Verify → Commit → Push| Resource | Description |
|---|---|
| Single-threaded detailed reference |
| Clean commit workflow |