Skip to content

refactor: decouple delivery agents to prepare-delivery plugin + repo-intel rename #26

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
avifenesh merged 6 commits into from
Mar 25, 2026
Merged

refactor: decouple delivery agents to prepare-delivery plugin + repo-intel rename #26

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
044c48e
chore: update plugin references from git-map/repo-map to repo-intel
avifenesh Mar 22, 2026
95edaad
refactor: decouple delivery agents to prepare-delivery plugin
avifenesh Mar 25, 2026
ec118b7
fix: delete orphaned agent/skill files after decoupling
avifenesh Mar 25, 2026
1010cbd
fix: update remaining stale agent references after decoupling
avifenesh Mar 25, 2026
5cc1968
fix: update delivery-approval.md reference to prepare-delivery:delive…
avifenesh Mar 25, 2026
028d574
docs: sync documentation with code changes
avifenesh Mar 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 8 additions & 4 deletions AGENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# next-task

Check warning on line 1 in AGENTS.md

View workflow job for this annotation

GitHub Actions / agnix / agnix 

Missing project context section in AGENTS instruction file [AGM-004]

> Master workflow orchestrator with autonomous task-to-production automation, quality gates, and multi-agent review

Expand All @@ -6,20 +6,24 @@

- ci-fixer
- ci-monitor
- delivery-validator
- exploration-agent
- implementation-agent
- planning-agent
- simple-fixer
- task-discoverer
- test-coverage-checker
- worktree-manager

## Skills

- discover-tasks
- orchestrate-review
- validate-delivery

## Cross-Plugin Agents (from prepare-delivery)

Phases 8-11 use agents/skills from the prepare-delivery plugin:
- `prepare-delivery:delivery-validator`
- `prepare-delivery:test-coverage-checker`
- `prepare-delivery:orchestrate-review` (skill)
- `prepare-delivery:validate-delivery` (skill)

## Commands

Expand Down
4 changes: 4 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@

### Changed

- Decouple delivery-validator and test-coverage-checker agents and orchestrate-review and validate-delivery skills to the prepare-delivery plugin - next-task now references these as `prepare-delivery:*`
- Update all internal cross-plugin references from `next-task:delivery-validator` and `next-task:test-coverage-checker` to `prepare-delivery:*` equivalents
- Rename git-map/repo-map plugin references to repo-intel across agent prompts and user-facing suggestions
- Integrate repo-intel across /next-task workflow (hotspots, bugspots, diff-risk, test-gaps)
- Downgrade exploration-agent from Opus to Sonnet - pre-fetched repo-intel data makes the agent a data curator rather than a deep analyst; planning-agent remains Opus
- Replace inline state dir detections with `getStateDirPath()` from agent-core for consistent platform-aware paths
Expand All @@ -23,6 +26,7 @@

### Docs

- Add prepare-delivery to Cross-Plugin Dependencies table in README
- Upgrade README with 12-phase workflow diagram, agent model table, and task sources reference ([#18](https://github.com/agent-sh/next-task/pull/18))

## [1.1.0] - 2026-03-14
Expand Down
12 changes: 8 additions & 4 deletions CLAUDE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,20 +6,24 @@

- ci-fixer
- ci-monitor
- delivery-validator
- exploration-agent
- implementation-agent
- planning-agent
- simple-fixer
- task-discoverer
- test-coverage-checker
- worktree-manager

## Skills

- discover-tasks
- orchestrate-review
- validate-delivery

## Cross-Plugin Agents (from prepare-delivery)

Phases 8-11 use agents/skills from the prepare-delivery plugin:
- `prepare-delivery:delivery-validator`
- `prepare-delivery:test-coverage-checker`
- `prepare-delivery:orchestrate-review` (skill)
- `prepare-delivery:validate-delivery` (skill)

## Commands

Expand Down
10 changes: 6 additions & 4 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,9 +45,9 @@ The workflow has 12 phases. Phases 1-6 involve the user; phases 7-12 run autonom
| 5 | Planning | planning-agent | Opus |
| 6 | User Approval | - | - |
| 7 | Implementation | implementation-agent | Opus |
| 8 | Pre-Review Gates | deslop-agent + test-coverage-checker | Sonnet |
| 8 | Pre-Review Gates | deslop:deslop-agent + prepare-delivery:test-coverage-checker | Sonnet |
| 9 | Review Loop | 4+ parallel reviewers | Sonnet |
| 10 | Delivery Validation | delivery-validator | Sonnet |
| 10 | Delivery Validation | prepare-delivery:delivery-validator | Sonnet |
| 11 | Docs Update | sync-docs-agent | Sonnet |
| 12 | Ship | ship:ship | - |

Expand Down Expand Up @@ -87,8 +87,9 @@ The state directory is platform-aware: `.claude/`, `.opencode/`, or `.codex/`.
| Skill | Purpose |
|-------|---------|
| discover-tasks | Fetch, filter, score, and present tasks for selection |
| orchestrate-review | Multi-pass parallel review with aggregation and iteration |
| validate-delivery | Autonomous completion checks - tests, build, requirements, regressions |

Phases 8-11 use skills from the [prepare-delivery](https://github.com/agent-sh/prepare-delivery) plugin:
`orchestrate-review`, `validate-delivery`, `check-test-coverage`.

## Requirements

Expand All @@ -102,6 +103,7 @@ The state directory is platform-aware: `.claude/`, `.opencode/`, or `.codex/`.
| Plugin | Used in |
|--------|---------|
| [deslop](https://github.com/agent-sh/deslop) | Phase 8 - AI slop cleanup |
| [prepare-delivery](https://github.com/agent-sh/prepare-delivery) | Phases 8-10 - test coverage, review orchestration, delivery validation |
| [sync-docs](https://github.com/agent-sh/sync-docs) | Phase 11 - documentation sync |
| [ship](https://github.com/agent-sh/ship) | Phase 12 - PR creation, CI, merge |

Expand Down
118 changes: 0 additions & 118 deletions agents/delivery-validator.md
View file
Open in desktop

This file was deleted.

4 changes: 2 additions & 2 deletions agents/exploration-agent.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ const repoMap = require(path.join(pluginRoot, 'lib/repo-map'));
const map = repoMap.load(process.cwd());

if (!map) {
console.log('Repo map not found. Consider: /repo-map init');
console.log('Repo map not found. Consider: /repo-intel init');
} else {
console.log(`Repo map loaded: ${Object.keys(map.files).length} files, ${map.stats.totalSymbols} symbols`);
}
Expand Down Expand Up @@ -109,7 +109,7 @@ if (fs.existsSync(mapFile)) {

console.log(`Repo intel loaded: hotspots=${repoIntel.hotspots?.length || 0}, bugspots=${repoIntel.bugspots?.length || 0}`);
} else {
console.log('Repo intel not found. Consider: /git-map build');
console.log('Repo intel not found. Consider: /repo-intel init');
}
```

Expand Down
10 changes: 5 additions & 5 deletions agents/implementation-agent.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ const map = repoMap.load(process.cwd());
if (map) {
console.log(`Repo map loaded: ${map.stats.totalSymbols} symbols`);
} else {
console.log('Repo map not found. Consider /repo-map init for faster lookups.');
console.log('Repo map not found. Consider /repo-intel init for faster lookups.');
}
```

Expand Down Expand Up @@ -387,11 +387,11 @@ implementation-agent (YOU ARE HERE)
↓
SubagentStop hook triggers automatically
↓
Pre-review gates: deslop:deslop-agent + test-coverage-checker
Pre-review gates: deslop:deslop-agent + prepare-delivery:test-coverage-checker
↓
Phase 9 review loop (must approve)
↓
delivery-validator (must approve)
prepare-delivery:delivery-validator (must approve)
↓
sync-docs:sync-docs-agent
↓
Expand Down Expand Up @@ -434,9 +434,9 @@ ${gitLog}

---
[STOP] STOPPING HERE - SubagentStop hook will trigger pre-review gates
→ deslop:deslop-agent + test-coverage-checker (parallel)
→ deslop:deslop-agent + prepare-delivery:test-coverage-checker (parallel)
→ Phase 9 review loop
→ delivery-validator
→ prepare-delivery:delivery-validator
→ sync-docs:sync-docs-agent
→ ship:ship
```
Expand Down
10 changes: 5 additions & 5 deletions agents/planning-agent.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ const repoMap = require(path.join(pluginRoot, 'lib/repo-map'));
const map = repoMap.load(process.cwd());

if (!map) {
console.log('Repo map missing. Suggest /repo-map init if needed.');
console.log('Repo map missing. Suggest /repo-intel init if needed.');
}
```

Expand Down Expand Up @@ -201,19 +201,19 @@ If the exploration output includes repo-intel data (hotspots, bugspots, coupling
const { getPluginRoot } = require('./lib/cross-platform');
const path = require('path');

const pluginRoot = getPluginRoot('git-map');
const pluginRoot = getPluginRoot('repo-intel');
let repoIntel = null;

if (pluginRoot) {
try {
const queries = require(path.join(pluginRoot, 'lib/git-map/queries'));
const queries = require(path.join(pluginRoot, 'lib/repo-intel/queries'));
repoIntel = queries;
console.log('[OK] repo-intel available - augmenting risk assessment');
} catch {
console.log('[WARN] git-map plugin found but queries failed to load');
console.log('[WARN] repo-intel plugin found but queries failed to load');
}
Comment on lines 212 to 214
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot Mar 25, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The catch block currently swallows the actual error, which can make debugging difficult. It's a good practice to log the error object for better diagnostics, as per our general rules.

Consider capturing and logging the error, like this:

  } catch (e) {
    console.log('[WARN] repo-intel plugin found but queries failed to load', e);
  }
Suggested change
} catch {
console.log('[WARN] git-map plugin found but queries failed to load');
console.log('[WARN] repo-intel plugin found but queries failed to load');
}
} catch (e) {
console.log('[WARN] repo-intel plugin found but queries failed to load', e);
}
References
  1. In helper functions that catch and handle errors, log the error along with relevant context, such as input arguments, to ensure failures are visible and debuggable.

} else {
console.log('[WARN] git-map plugin not installed - using heuristic risk assessment only');
console.log('[WARN] repo-intel plugin not installed - using heuristic risk assessment only');
}
```

Expand Down
Loading
Loading