awesome-slash 3.8.0 → 3.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/marketplace.json +10 -10
- package/.claude-plugin/plugin.json +1 -1
- package/CHANGELOG.md +7 -0
- package/README.md +1 -2
- package/adapters/opencode-plugin/index.ts +1 -1
- package/bin/cli.js +2 -0
- package/meta/skills/maintain-cross-platform/SKILL.md +1 -3
- package/package.json +1 -1
- package/plugins/audit-project/.claude-plugin/plugin.json +1 -1
- package/plugins/deslop/.claude-plugin/plugin.json +1 -1
- package/plugins/drift-detect/.claude-plugin/plugin.json +1 -1
- package/plugins/enhance/.claude-plugin/plugin.json +1 -1
- package/plugins/next-task/.claude-plugin/plugin.json +1 -1
- package/plugins/next-task/agents/implementation-agent.md +2 -2
- package/plugins/next-task/agents/simple-fixer.md +1 -1
- package/plugins/next-task/agents/test-coverage-checker.md +1 -1
- package/plugins/next-task/commands/next-task.md +80 -48
- package/plugins/next-task/hooks/hooks.json +1 -1
- package/plugins/next-task/skills/orchestrate-review/SKILL.md +1 -1
- package/plugins/perf/.claude-plugin/plugin.json +1 -1
- package/plugins/repo-map/.claude-plugin/plugin.json +1 -1
- package/plugins/ship/.claude-plugin/plugin.json +1 -1
- package/plugins/ship/commands/ship.md +1 -1
- package/plugins/sync-docs/.claude-plugin/plugin.json +1 -1
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "awesome-slash",
|
|
3
3
|
"description": "9 specialized plugins for AI workflow automation - task orchestration, PR workflow, slop detection, code review, drift detection, enhancement analysis, documentation sync, repo mapping, and perf investigations",
|
|
4
|
-
"version": "3.8.
|
|
4
|
+
"version": "3.8.1",
|
|
5
5
|
"owner": {
|
|
6
6
|
"name": "Avi Fenesh",
|
|
7
7
|
"url": "https://github.com/avifenesh"
|
|
@@ -26,63 +26,63 @@
|
|
|
26
26
|
"name": "next-task",
|
|
27
27
|
"source": "./plugins/next-task",
|
|
28
28
|
"description": "Master workflow orchestrator: autonomous workflow with model optimization (opus/sonnet/haiku), two-file state management, workflow enforcement gates, 14 specialist agents",
|
|
29
|
-
"version": "3.8.
|
|
29
|
+
"version": "3.8.1",
|
|
30
30
|
"category": "productivity"
|
|
31
31
|
},
|
|
32
32
|
{
|
|
33
33
|
"name": "ship",
|
|
34
34
|
"source": "./plugins/ship",
|
|
35
35
|
"description": "Complete PR workflow: commit to production, skips review when called from next-task, removes task from registry on cleanup, automatic rollback",
|
|
36
|
-
"version": "3.8.
|
|
36
|
+
"version": "3.8.1",
|
|
37
37
|
"category": "deployment"
|
|
38
38
|
},
|
|
39
39
|
{
|
|
40
40
|
"name": "deslop",
|
|
41
41
|
"source": "./plugins/deslop",
|
|
42
42
|
"description": "3-phase AI slop detection: regex patterns (HIGH), multi-pass analyzers (MEDIUM), CLI tools (LOW)",
|
|
43
|
-
"version": "3.8.
|
|
43
|
+
"version": "3.8.1",
|
|
44
44
|
"category": "development"
|
|
45
45
|
},
|
|
46
46
|
{
|
|
47
47
|
"name": "audit-project",
|
|
48
48
|
"source": "./plugins/audit-project",
|
|
49
49
|
"description": "Multi-agent iterative code review until zero issues remain",
|
|
50
|
-
"version": "3.8.
|
|
50
|
+
"version": "3.8.1",
|
|
51
51
|
"category": "development"
|
|
52
52
|
},
|
|
53
53
|
{
|
|
54
54
|
"name": "drift-detect",
|
|
55
55
|
"source": "./plugins/drift-detect",
|
|
56
56
|
"description": "Deep repository analysis to realign project plans with code reality - detects drift, gaps, and creates prioritized reconstruction plans",
|
|
57
|
-
"version": "3.8.
|
|
57
|
+
"version": "3.8.1",
|
|
58
58
|
"category": "productivity"
|
|
59
59
|
},
|
|
60
60
|
{
|
|
61
61
|
"name": "enhance",
|
|
62
62
|
"source": "./plugins/enhance",
|
|
63
63
|
"description": "Master enhancement orchestrator: parallel analyzer execution for plugins, agents, docs, CLAUDE.md, and prompts with unified reporting",
|
|
64
|
-
"version": "3.8.
|
|
64
|
+
"version": "3.8.1",
|
|
65
65
|
"category": "development"
|
|
66
66
|
},
|
|
67
67
|
{
|
|
68
68
|
"name": "sync-docs",
|
|
69
69
|
"source": "./plugins/sync-docs",
|
|
70
70
|
"description": "Standalone documentation sync: find outdated refs, update CHANGELOG, flag stale examples based on code changes",
|
|
71
|
-
"version": "3.8.
|
|
71
|
+
"version": "3.8.1",
|
|
72
72
|
"category": "development"
|
|
73
73
|
},
|
|
74
74
|
{
|
|
75
75
|
"name": "repo-map",
|
|
76
76
|
"source": "./plugins/repo-map",
|
|
77
77
|
"description": "AST-based repository map generation using ast-grep with incremental updates for faster drift analysis",
|
|
78
|
-
"version": "3.8.
|
|
78
|
+
"version": "3.8.1",
|
|
79
79
|
"category": "development"
|
|
80
80
|
},
|
|
81
81
|
{
|
|
82
82
|
"name": "perf",
|
|
83
83
|
"source": "./plugins/perf",
|
|
84
84
|
"description": "Rigorous performance investigation workflow with baselines, profiling, hypotheses, and evidence-backed decisions",
|
|
85
|
-
"version": "3.8.
|
|
85
|
+
"version": "3.8.1",
|
|
86
86
|
"category": "development"
|
|
87
87
|
}
|
|
88
88
|
]
|
package/CHANGELOG.md
CHANGED
|
@@ -7,6 +7,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
7
7
|
|
|
8
8
|
## [Unreleased]
|
|
9
9
|
|
|
10
|
+
## [3.8.1] - 2026-02-04
|
|
11
|
+
|
|
12
|
+
### Fixed
|
|
13
|
+
- **Stop Point Options** - Restored missing "Deployed" and "Production" options in OpenCode embedded policy
|
|
14
|
+
- **Phase 9 Review Loop** - Rewrote instructions to explicitly require spawning 4 parallel reviewer agents (code-quality, security, performance, test-coverage) instead of single generic reviewer
|
|
15
|
+
- **Agent Naming Consistency** - Standardized all references from legacy `deslop-work` to `deslop:deslop-agent` across 12 files (docs, configs, agent prompts)
|
|
16
|
+
|
|
10
17
|
## [3.8.0] - 2026-02-02
|
|
11
18
|
|
|
12
19
|
### Added
|
package/README.md
CHANGED
|
@@ -603,7 +603,7 @@ Every command works standalone. [`/deslop`](#deslop) cleans code without needing
|
|
|
603
603
|
|
|
604
604
|
## Installation
|
|
605
605
|
|
|
606
|
-
### Claude Code (Recommended)
|
|
606
|
+
### Claude Code (Recommended way)
|
|
607
607
|
|
|
608
608
|
```bash
|
|
609
609
|
/plugin marketplace add avifenesh/awesome-slash
|
|
@@ -700,7 +700,6 @@ This project is built on research, not guesswork.
|
|
|
700
700
|
|-------|------|
|
|
701
701
|
| Slop Patterns | [docs/reference/SLOP-PATTERNS.md](./docs/reference/SLOP-PATTERNS.md) |
|
|
702
702
|
| Agent Reference | [docs/reference/AGENTS.md](./docs/reference/AGENTS.md) |
|
|
703
|
-
| MCP Tools | [docs/reference/MCP-TOOLS.md](./docs/reference/MCP-TOOLS.md) |
|
|
704
703
|
|
|
705
704
|
---
|
|
706
705
|
|
|
@@ -28,7 +28,7 @@ const AGENT_THINKING_CONFIG: Record<string, { budget: number; description: strin
|
|
|
28
28
|
|
|
29
29
|
// Analysis tier - moderate thinking (12k)
|
|
30
30
|
"exploration-agent": { budget: 12000, description: "Codebase exploration" },
|
|
31
|
-
"deslop-
|
|
31
|
+
"deslop-agent": { budget: 12000, description: "Slop detection" },
|
|
32
32
|
"test-coverage-checker": { budget: 12000, description: "Coverage analysis" },
|
|
33
33
|
"ci-fixer": { budget: 12000, description: "CI problem solving" },
|
|
34
34
|
|
package/bin/cli.js
CHANGED
|
@@ -513,6 +513,8 @@ Ask the user these questions using AskUserQuestion:
|
|
|
513
513
|
- Merged - Until PR is merged to main
|
|
514
514
|
- PR Created - Stop after creating PR
|
|
515
515
|
- Implemented - Stop after local implementation
|
|
516
|
+
- Deployed - Deploy to staging
|
|
517
|
+
- Production - Full production deployment
|
|
516
518
|
|
|
517
519
|
After user answers, proceed to Phase 2 with the selected policy.
|
|
518
520
|
|
|
@@ -274,8 +274,7 @@ Same as RC but:
|
|
|
274
274
|
1. **mcp-server/index.js** - Add to TOOLS array and toolHandlers
|
|
275
275
|
2. **.claude-plugin/marketplace.json** - Add to `mcpServer.tools` array
|
|
276
276
|
3. **bin/cli.js** - Update MCP tools console output (OpenCode + Codex)
|
|
277
|
-
4. **
|
|
278
|
-
5. **README.md** - Add to MCP tools table if user-facing
|
|
277
|
+
4. **README.md** - Add to MCP tools table if user-facing
|
|
279
278
|
|
|
280
279
|
### If Library Module Changed
|
|
281
280
|
|
|
@@ -693,7 +692,6 @@ const toolHandlers = {
|
|
|
693
692
|
```
|
|
694
693
|
|
|
695
694
|
**3. Update docs:**
|
|
696
|
-
- `docs/reference/MCP-TOOLS.md` - Full tool documentation
|
|
697
695
|
- `README.md` - Add to features if user-facing
|
|
698
696
|
|
|
699
697
|
---
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "awesome-slash",
|
|
3
|
-
"version": "3.8.
|
|
3
|
+
"version": "3.8.1",
|
|
4
4
|
"description": "9 specialized plugins for AI workflow automation (drift detection + AST repo maps) - works with Claude Code, OpenCode, and Codex CLI",
|
|
5
5
|
"main": "lib/platform/detect-platform.js",
|
|
6
6
|
"type": "commonjs",
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "drift-detect",
|
|
3
|
-
"version": "3.8.
|
|
3
|
+
"version": "3.8.1",
|
|
4
4
|
"description": "Deep repository analysis to realign project plans with actual code reality - discovers drift, gaps, and produces prioritized reconstruction plans",
|
|
5
5
|
"author": {
|
|
6
6
|
"name": "Avi Fenesh",
|
|
@@ -377,7 +377,7 @@ implementation-agent (YOU ARE HERE)
|
|
|
377
377
|
↓
|
|
378
378
|
SubagentStop hook triggers automatically
|
|
379
379
|
↓
|
|
380
|
-
Pre-review gates: deslop-
|
|
380
|
+
Pre-review gates: deslop:deslop-agent + test-coverage-checker
|
|
381
381
|
↓
|
|
382
382
|
Phase 9 review loop (must approve)
|
|
383
383
|
↓
|
|
@@ -424,7 +424,7 @@ ${gitLog}
|
|
|
424
424
|
|
|
425
425
|
---
|
|
426
426
|
[STOP] STOPPING HERE - SubagentStop hook will trigger pre-review gates
|
|
427
|
-
→ deslop-
|
|
427
|
+
→ deslop:deslop-agent + test-coverage-checker (parallel)
|
|
428
428
|
→ Phase 9 review loop
|
|
429
429
|
→ delivery-validator
|
|
430
430
|
→ sync-docs:sync-docs-agent
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: simple-fixer
|
|
3
|
-
description: Execute simple, pre-defined code fixes. Use this agent when deslop-
|
|
3
|
+
description: Execute simple, pre-defined code fixes. Use this agent when deslop:deslop-agent or sync-docs:sync-docs-agent has a list of straightforward edits to apply.
|
|
4
4
|
tools: Read, Edit, Bash(git:*)
|
|
5
5
|
model: haiku
|
|
6
6
|
---
|
|
@@ -423,7 +423,7 @@ ${summary.recommendation}
|
|
|
423
423
|
## Integration Points
|
|
424
424
|
|
|
425
425
|
This agent is called:
|
|
426
|
-
1. **Before first review round** - In parallel with deslop-
|
|
426
|
+
1. **Before first review round** - In parallel with deslop:deslop-agent
|
|
427
427
|
2. Results passed to Phase 9 review loop for context
|
|
428
428
|
|
|
429
429
|
## Success Criteria
|
|
@@ -324,64 +324,96 @@ Use Edit tool to apply. Commit message: "fix: clean up AI slop"`
|
|
|
324
324
|
|
|
325
325
|
**Blocking gate** - Must run iterations before delivery validation.
|
|
326
326
|
|
|
327
|
-
|
|
327
|
+
**CRITICAL**: You MUST spawn multiple parallel reviewer agents. Do NOT use a single generic reviewer.
|
|
328
328
|
|
|
329
|
-
|
|
330
|
-
workflowState.startPhase('review-loop');
|
|
331
|
-
|
|
332
|
-
// REQUIRED: Run the review loop with MINIMUM 1 iteration
|
|
333
|
-
// Orchestrator may override after 3+ iterations if issues are non-blocking
|
|
334
|
-
let iteration = 0;
|
|
335
|
-
const MAX_ITERATIONS = 5;
|
|
336
|
-
const MIN_ITERATIONS = 1;
|
|
337
|
-
|
|
338
|
-
while (iteration < MAX_ITERATIONS) {
|
|
339
|
-
iteration++;
|
|
340
|
-
console.log(`[REVIEW] Iteration ${iteration}`);
|
|
341
|
-
|
|
342
|
-
// Run review passes (see orchestrate-review skill)
|
|
343
|
-
const findings = await runReviewPasses();
|
|
344
|
-
|
|
345
|
-
if (findings.openCount === 0) {
|
|
346
|
-
workflowState.updateFlow({
|
|
347
|
-
reviewResult: { approved: true, iterations: iteration }
|
|
348
|
-
});
|
|
349
|
-
break;
|
|
350
|
-
}
|
|
329
|
+
### Step 1: Get Changed Files
|
|
351
330
|
|
|
352
|
-
|
|
353
|
-
|
|
354
|
-
|
|
355
|
-
|
|
356
|
-
|
|
357
|
-
|
|
358
|
-
|
|
359
|
-
|
|
360
|
-
|
|
361
|
-
|
|
362
|
-
|
|
363
|
-
|
|
364
|
-
|
|
365
|
-
|
|
331
|
+
```bash
|
|
332
|
+
git diff --name-only main...HEAD
|
|
333
|
+
```
|
|
334
|
+
|
|
335
|
+
### Step 2: Detect Signals for Conditional Specialists
|
|
336
|
+
|
|
337
|
+
Based on changed files, detect which additional specialists are needed:
|
|
338
|
+
|
|
339
|
+
| Signal | Pattern | Specialist |
|
|
340
|
+
|--------|---------|------------|
|
|
341
|
+
| hasDb | `/(db\|migrations?\|schema\|prisma\|sql)/i` | database specialist |
|
|
342
|
+
| hasApi | `/(api\|routes?\|controllers?\|handlers?)/i` | api designer |
|
|
343
|
+
| hasFrontend | `/\.(tsx\|jsx\|vue\|svelte)$/` | frontend specialist |
|
|
344
|
+
| hasBackend | `/(server\|backend\|services?\|domain)/i` | backend specialist |
|
|
345
|
+
| hasDevops | `/(\.github\/workflows\|Dockerfile\|k8s\|terraform)/i` | devops reviewer |
|
|
346
|
+
| needsArchitecture | 20+ changed files | architecture reviewer |
|
|
347
|
+
|
|
348
|
+
### Step 3: Spawn ALL Reviewer Agents in Parallel
|
|
349
|
+
|
|
350
|
+
**MANDATORY**: Spawn these 4 core reviewers (ALWAYS) + any conditional specialists detected above.
|
|
351
|
+
|
|
352
|
+
```javascript
|
|
353
|
+
// 4 CORE REVIEWERS - ALWAYS SPAWN ALL 4 IN PARALLEL
|
|
354
|
+
const coreReviewers = [
|
|
355
|
+
{ role: 'code quality reviewer', focus: 'Style, best practices, bugs, error handling, duplication' },
|
|
356
|
+
{ role: 'security reviewer', focus: 'Auth flaws, input validation, injection, secrets exposure' },
|
|
357
|
+
{ role: 'performance reviewer', focus: 'N+1 queries, blocking ops, hot path issues, memory leaks' },
|
|
358
|
+
{ role: 'test coverage reviewer', focus: 'Missing tests, edge cases, test quality, mock appropriateness' }
|
|
359
|
+
];
|
|
360
|
+
|
|
361
|
+
// Spawn ALL 4 core reviewers in parallel using Task tool
|
|
362
|
+
const reviewResults = await Promise.all([
|
|
363
|
+
Task({ subagent_type: 'general-purpose', model: 'sonnet',
|
|
364
|
+
prompt: `You are a code quality reviewer. Review these files: ${files.join(', ')}
|
|
365
|
+
Focus: Style and consistency, Best practices, Bugs and logic errors, Error handling, Maintainability, Duplication
|
|
366
|
+
Return findings as JSON array with: file, line, severity (critical/high/medium/low), description, suggestion` }),
|
|
367
|
+
Task({ subagent_type: 'general-purpose', model: 'sonnet',
|
|
368
|
+
prompt: `You are a security reviewer. Review these files: ${files.join(', ')}
|
|
369
|
+
Focus: Auth/authz flaws, Input validation, Injection risks, Secrets exposure, Insecure defaults
|
|
370
|
+
Return findings as JSON array with: file, line, severity (critical/high/medium/low), description, suggestion` }),
|
|
371
|
+
Task({ subagent_type: 'general-purpose', model: 'sonnet',
|
|
372
|
+
prompt: `You are a performance reviewer. Review these files: ${files.join(', ')}
|
|
373
|
+
Focus: N+1 queries, Blocking operations, Hot path inefficiencies, Memory leaks
|
|
374
|
+
Return findings as JSON array with: file, line, severity (critical/high/medium/low), description, suggestion` }),
|
|
375
|
+
Task({ subagent_type: 'general-purpose', model: 'sonnet',
|
|
376
|
+
prompt: `You are a test coverage reviewer. Review these files: ${files.join(', ')}
|
|
377
|
+
Focus: Missing tests, Edge case coverage, Test quality, Integration needs, Mock appropriateness
|
|
378
|
+
Return findings as JSON array with: file, line, severity (critical/high/medium/low), description, suggestion` })
|
|
379
|
+
]);
|
|
366
380
|
|
|
367
|
-
//
|
|
368
|
-
console.log(`[VERIFIED] Review: ${iteration} iterations, approved: ${reviewResult.approved}`);
|
|
381
|
+
// Add conditional specialists based on signals (spawn in parallel with appropriate prompts)
|
|
369
382
|
```
|
|
370
383
|
|
|
384
|
+
### Step 4: Aggregate Findings
|
|
385
|
+
|
|
386
|
+
Combine all reviewer findings, deduplicate by file+line+description, group by severity.
|
|
387
|
+
|
|
388
|
+
### Step 5: Fix Issues (severity order: critical -> high -> medium -> low)
|
|
389
|
+
|
|
390
|
+
For each finding, use Edit tool to apply the suggested fix. Commit after each batch.
|
|
391
|
+
|
|
392
|
+
### Step 6: Iterate Until Clean (max 5 iterations)
|
|
393
|
+
|
|
394
|
+
Repeat steps 3-5 until:
|
|
395
|
+
- `openCount === 0` (all issues resolved) -> approved
|
|
396
|
+
- 3+ iterations with only medium/low issues -> orchestrator may override
|
|
397
|
+
- 5 iterations reached -> blocked
|
|
398
|
+
|
|
371
399
|
### Review Iteration Rules
|
|
372
|
-
-
|
|
400
|
+
- MUST run at least 1 full iteration with ALL 4 core reviewers
|
|
401
|
+
- Do NOT use a single generic reviewer - spawn all specialists in parallel
|
|
373
402
|
- Orchestrator may override after 3+ iterations if only medium/low issues remain
|
|
374
403
|
- Do not skip directly to delivery validation
|
|
375
|
-
- Do not claim "review passed" without
|
|
376
|
-
|
|
377
|
-
### Review Decision Gate
|
|
404
|
+
- Do not claim "review passed" without spawning the reviewer agents
|
|
378
405
|
|
|
379
|
-
|
|
380
|
-
1. **Critical/high issues OR security/performance/architecture** -> re-run review
|
|
381
|
-
2. **Medium/low code-quality only after 3+ iterations** -> may override
|
|
382
|
-
3. **Unclear** -> re-run review
|
|
406
|
+
### Verification Output (MANDATORY)
|
|
383
407
|
|
|
384
|
-
|
|
408
|
+
After review loop completes, output:
|
|
409
|
+
```
|
|
410
|
+
[VERIFIED] Review Loop Complete
|
|
411
|
+
- Iterations: N
|
|
412
|
+
- Core reviewers spawned: code-quality, security, performance, test-coverage
|
|
413
|
+
- Conditional specialists: [list any that were added]
|
|
414
|
+
- Findings resolved: X critical, Y high, Z medium
|
|
415
|
+
- Status: approved | blocked
|
|
416
|
+
```
|
|
385
417
|
</phase-9>
|
|
386
418
|
|
|
387
419
|
## Phase 10: Delivery Validation
|
|
@@ -5,7 +5,7 @@
|
|
|
5
5
|
"hooks": [
|
|
6
6
|
{
|
|
7
7
|
"type": "prompt",
|
|
8
|
-
"prompt": "<subagent-stop-hook>\n## Workflow Enforcement - SubagentStop Hook\n\nA subagent has completed. Determine and execute the next workflow phase.\n\n<verification-gates>\n### Verification Gates\n\nBefore proceeding to any next phase, verify the required previous steps completed.\n\n---\n\n## Gate 1: Before Exploration (Phase 4)\n\n**Required**: Worktree must have been created via `next-task:worktree-manager` agent.\n\n```javascript\nconst state = workflowState.readState();\nif (!state.worktree || !state.worktree.path) {\n console.error('[BLOCKED] Cannot proceed to exploration - worktree not created');\n return { blocked: true, reason: 'worktree-missing' };\n}\nconsole.log(`[VERIFIED] Worktree exists: ${state.worktree.path}`);\n```\n\n**Forbidden**:\n- Using `git checkout -b` directly\n- Using `git branch` directly\n- Proceeding to exploration without worktree verification\n\n---\n\n## Gate 2: Before Delivery Validation (Phase 10)\n\n**Required**: Review loop must have run with proper iterations.\n\n```javascript\nconst state = workflowState.readState();\nconst reviewResult = state.phases?.reviewLoop;\nif (!reviewResult || reviewResult.iterations < 1) {\n console.error('[BLOCKED] Review loop must run at least 1 iteration');\n return { blocked: true, reason: 'review-iterations-zero' };\n}\nif (!reviewResult.approved && !reviewResult.orchestratorOverride) {\n console.error('[BLOCKED] Review not approved');\n return { blocked: true, reason: 'review-not-approved' };\n}\nconsole.log(`[VERIFIED] Review: ${reviewResult.iterations} iterations`);\n```\n\n**Forbidden**:\n- Skipping to delivery without running review loop\n- Running review with 0 iterations\n\n---\n\n## Gate 3: Before /ship Merge\n\n**Required**: All PR comments must be addressed.\n\nEnforced in /ship command:\n1. Phase 4 CI & Review Monitor Loop must run\n2. 3-minute wait for auto-reviewers must complete\n3. All comments must be addressed before merge\n</verification-gates>\n\n---\n\n<decision-tree>\n### Decision Tree\n\n1. **worktree-manager completed**: Verify worktree path, then run exploration-agent\n2. **implementation-agent completed**: Run deslop-
|
|
8
|
+
"prompt": "<subagent-stop-hook>\n## Workflow Enforcement - SubagentStop Hook\n\nA subagent has completed. Determine and execute the next workflow phase.\n\n<verification-gates>\n### Verification Gates\n\nBefore proceeding to any next phase, verify the required previous steps completed.\n\n---\n\n## Gate 1: Before Exploration (Phase 4)\n\n**Required**: Worktree must have been created via `next-task:worktree-manager` agent.\n\n```javascript\nconst state = workflowState.readState();\nif (!state.worktree || !state.worktree.path) {\n console.error('[BLOCKED] Cannot proceed to exploration - worktree not created');\n return { blocked: true, reason: 'worktree-missing' };\n}\nconsole.log(`[VERIFIED] Worktree exists: ${state.worktree.path}`);\n```\n\n**Forbidden**:\n- Using `git checkout -b` directly\n- Using `git branch` directly\n- Proceeding to exploration without worktree verification\n\n---\n\n## Gate 2: Before Delivery Validation (Phase 10)\n\n**Required**: Review loop must have run with proper iterations.\n\n```javascript\nconst state = workflowState.readState();\nconst reviewResult = state.phases?.reviewLoop;\nif (!reviewResult || reviewResult.iterations < 1) {\n console.error('[BLOCKED] Review loop must run at least 1 iteration');\n return { blocked: true, reason: 'review-iterations-zero' };\n}\nif (!reviewResult.approved && !reviewResult.orchestratorOverride) {\n console.error('[BLOCKED] Review not approved');\n return { blocked: true, reason: 'review-not-approved' };\n}\nconsole.log(`[VERIFIED] Review: ${reviewResult.iterations} iterations`);\n```\n\n**Forbidden**:\n- Skipping to delivery without running review loop\n- Running review with 0 iterations\n\n---\n\n## Gate 3: Before /ship Merge\n\n**Required**: All PR comments must be addressed.\n\nEnforced in /ship command:\n1. Phase 4 CI & Review Monitor Loop must run\n2. 3-minute wait for auto-reviewers must complete\n3. All comments must be addressed before merge\n</verification-gates>\n\n---\n\n<decision-tree>\n### Decision Tree\n\n1. **worktree-manager completed**: Verify worktree path, then run exploration-agent\n2. **implementation-agent completed**: Run deslop:deslop-agent + test-coverage-checker (parallel)\n3. **pre-review gates completed**: Run review loop (min 1 iteration), then delivery-validator\n4. **delivery-validator completed**: If approved, run sync-docs:sync-docs-agent. If not, return to implementation.\n5. **sync-docs:sync-docs-agent completed**: Invoke /ship command\n</decision-tree>\n\n---\n\n<enforcement>\n### Enforcement\n\nEvery step exists for a reason. Taking shortcuts defeats the purpose of automation.\n\n- Do not skip worktree-manager (enables parallel task isolation)\n- Do not skip review iterations (catches bugs humans miss)\n- Do not skip 3-minute wait in /ship (auto-reviewers need time)\n- Do not skip addressing PR comments (blocks merge)\n\nIf you think a step is unnecessary, you are wrong.\n</enforcement>\n\n---\n\n<workflow-sequence>\n### Workflow Sequence\n\n1. task-discoverer\n2. [GATE] worktree-manager (must use agent)\n3. [VERIFY] worktree exists\n4. exploration-agent\n5. planning-agent\n6. implementation-agent\n7. pre-review gates (deslop + test-coverage)\n8. review loop (1+ iterations)\n9. [GATE] delivery-validator\n10. sync-docs:sync-docs-agent\n11. /ship command (must run Phase 4 loop)\n</workflow-sequence>\n\nReturn: {\"ok\": true, \"nextPhase\": \"<phase-name>\", \"verified\": [\"<gate-name>\"]}\n</subagent-stop-hook>"
|
|
9
9
|
}
|
|
10
10
|
]
|
|
11
11
|
}
|
|
@@ -187,7 +187,7 @@ while (iteration <= MAX_ITERATIONS) {
|
|
|
187
187
|
exec(`git add . && git commit -m "fix: review feedback (iteration ${iteration})"`);
|
|
188
188
|
|
|
189
189
|
// 6. Post-iteration deslop
|
|
190
|
-
Task({ subagent_type: '
|
|
190
|
+
Task({ subagent_type: 'deslop:deslop-agent', model: 'sonnet' });
|
|
191
191
|
|
|
192
192
|
// 7. Stall detection
|
|
193
193
|
const hash = crypto.createHash('sha256')
|
|
@@ -56,7 +56,7 @@ Before proceeding to merge, output:
|
|
|
56
56
|
|
|
57
57
|
When called from `/next-task` workflow (via `--state-file`):
|
|
58
58
|
- **SKIPS Phase 5** internal review agents (already done by Phase 9 review loop)
|
|
59
|
-
- **SKIPS deslop/docs** (already done by deslop-
|
|
59
|
+
- **SKIPS deslop/docs** (already done by deslop:deslop-agent, sync-docs:sync-docs-agent)
|
|
60
60
|
- **Trusts** that all quality gates passed
|
|
61
61
|
|
|
62
62
|
**CRITICAL: Phase 4 ALWAYS runs** - even from /next-task. External auto-reviewers (Gemini, Copilot, CodeRabbit) comment AFTER PR creation and must be addressed.
|