Skip to content

Refactor RPI Workflow for Parallel Agents and Manifest-Driven Execution #10

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
SireJeff merged 1 commit into from
Mar 22, 2026
Merged

Refactor RPI Workflow for Parallel Agents and Manifest-Driven Execution #10

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
77 changes: 42 additions & 35 deletions templates/base/research/RESEARCH_TEMPLATE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
# Research: [Feature/Bug Name]

**Date:** YYYY-MM-DD
**Researcher:** Claude Code
**Researcher:** RPI-Research Agent
**Status:** IN_PROGRESS | COMPLETE | BLOCKED
**Context Budget Used:** X% of 200k
**Total Chunks:** N
**Parallel Agents Used:** N (3 minimum, up to 5 for complex features)
**Total Chunks:** 5 (Standard Parallel Set)
**Parallel Agents Used:** 5 (API, Logic, DB, External, QA)
Comment on lines +7 to +8

Copilot AI Mar 22, 2026

Copy link

Choose a reason for hiding this comment

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

This template now hard-codes “Total Chunks: 5” and a fixed 5-agent set (API/Logic/DB/External/QA). Later sections still use placeholders like “[N] research chunks” and only enumerate key files for CHUNK-R1–R3. Please make the chunk count/agent naming consistent throughout (either keep it variable everywhere or update the later placeholders/lists to reflect the fixed 5-chunk model).

Suggested change
**Total Chunks:** 5 (Standard Parallel Set)
**Parallel Agents Used:** 5 (API, Logic, DB, External, QA)
**Total Chunks:** [N] (Parallel Set)
**Parallel Agents Used:** [N] (e.g., API, Logic, DB, External, QA)

Copilot uses AI. Check for mistakes.

---

Expand All @@ -15,36 +15,37 @@

---

## Chunk Manifest
## Research Manifest

> **INTER-PHASE CONTRACT:** This manifest is designed for RPI-Plan consumption.
> RPI-Plan will process each chunk sequentially, creating a CHUNK-Pn todolist per CHUNK-Rn.
> **GENERATED BY:** Phase 1 - Parallel Search Agents
> **PURPOSE:** Drivers for Phase 2 - Sequential Deep Dive.
> **INTER-PHASE CONTRACT:** RPI-Plan reads this to understand the system structure.
Comment on lines +18 to +22

Copilot AI Mar 22, 2026

Copy link

Choose a reason for hiding this comment

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

The new “Research Manifest” describes a Phase 1→Phase 2 (parallel search → sequential deep dive) flow, but the template’s downstream contract in this same document still describes the older RPI-Plan interaction (CHUNK-Pn todolists, marking chunks as PLANNED, etc.). Please reconcile the manifest/phase language here with the later Inter-Phase Contract so the expected consumer behavior and chunk status lifecycle are consistent and machine-/human-readable.

Copilot uses AI. Check for mistakes.
Comment on lines +18 to +22

Copilot AI Mar 22, 2026

Copy link

Choose a reason for hiding this comment

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

PR description claims a broad RPI workflow refactor (updates to /rpi-research, /rpi-plan, /rpi-implement, a new rpi-engineer agent, and code-mapper/config changes). In the current branch state, those corresponding templates/docs still reference the previous “Chunk Manifest” format and 3–5 agents (e.g., templates/base/commands/rpi-research.md:85-128; templates/base/RPI_WORKFLOW_PLAN.md:40-49), and no rpi-engineer file is present under templates/base/agents/. Either include the missing changes in this PR or narrow/update the PR description so it matches what is actually being modified.

Copilot uses AI. Check for mistakes.

| Chunk ID | Domain | Status | Files | Ready for Planning |
|----------|--------|--------|-------|-------------------|
| CHUNK-R1 | API/Routes | COMPLETE | 3 | ✅ |
| CHUNK-R2 | Business Logic | COMPLETE | 4 | ✅ |
| CHUNK-R3 | Database/Storage | COMPLETE | 2 | ✅ |
| CHUNK-R4 | External Integrations | COMPLETE | 1 | ✅ |
| CHUNK-R5 | Test Coverage | COMPLETE | 3 | ✅ |
| Chunk ID | Domain | Assigned Agent | Status | Files Found | Ready for Deep Dive |
|----------|--------|----------------|--------|-------------|---------------------|
| CHUNK-R1 | API/Routes | Agent 1 (API) | PENDING | [Count] | ⏳ |
| CHUNK-R2 | Business Logic | Agent 2 (Logic) | PENDING | [Count] | ⏳ |
| CHUNK-R3 | Database/Storage | Agent 3 (DB) | PENDING | [Count] | ⏳ |
| CHUNK-R4 | Integrations | Agent 4 (Ext) | PENDING | [Count] | ⏳ |
| CHUNK-R5 | Test Coverage | Agent 5 (QA) | PENDING | [Count] | ⏳ |

---

## CHUNK-R1: API/Routes
## CHUNK-R1: API/Routes (Deep Dive)

**Status:** FOUND | COMPLETE | PLANNED | IMPLEMENTED
**Parallel Agent:** Agent 1
**Status:** SEARCHED | ANALYZED | COMPLETE
**Source:** Agent 1 (Parallel Search)
**Ready for Planning:** Yes/No

### Files Explored
> Populated by Agent 1 during Deep Dive

| File | Lines | Key Findings |
|------|-------|--------------|
| `path/to/route.ext` | XXX-YYY | [Entry point for feature] |
| `path/to/handler.ext` | XXX-YYY | [Request handling logic] |

### Code Flow Analysis

```
entry_function() [file.ext:XXX]
├─ validate_request() [file.ext:YYY]
Expand All @@ -57,21 +58,21 @@ entry_function() [file.ext:XXX]

---

## CHUNK-R2: Business Logic
## CHUNK-R2: Business Logic (Deep Dive)

**Status:** FOUND | COMPLETE | PLANNED | IMPLEMENTED
**Parallel Agent:** Agent 2
**Status:** SEARCHED | ANALYZED | COMPLETE
**Source:** Agent 2 (Parallel Search)
**Ready for Planning:** Yes/No

### Files Explored
> Populated by Agent 2 during Deep Dive

| File | Lines | Key Findings |
|------|-------|--------------|
| `path/to/service.ext` | XXX-YYY | [Core business logic] |
| `path/to/model.ext` | XXX-YYY | [Data models] |

### Code Flow Analysis

```
process_request() [service.ext:XXX]
├─ validate_data() [service.ext:YYY]
Expand All @@ -85,21 +86,21 @@ process_request() [service.ext:XXX]

---

## CHUNK-R3: Database/Storage
## CHUNK-R3: Database/Storage (Deep Dive)

**Status:** FOUND | COMPLETE | PLANNED | IMPLEMENTED
**Parallel Agent:** Agent 3
**Status:** SEARCHED | ANALYZED | COMPLETE
**Source:** Agent 3 (Parallel Search)
**Ready for Planning:** Yes/No

### Files Explored
> Populated by Agent 3 during Deep Dive

| File | Lines | Key Findings |
|------|-------|--------------|
| `path/to/crud.ext` | XXX-YYY | [Database operations] |
| `path/to/schema.ext` | XXX-YYY | [Schema definitions] |

### Database Schema Involved

| Table | Operations | Purpose |
|-------|------------|---------|
| `table_name` | READ/WRITE/UPDATE | [What data] |
Expand All @@ -110,35 +111,41 @@ process_request() [service.ext:XXX]

---

## CHUNK-R4: External Integrations
## CHUNK-R4: External Integrations (Deep Dive)

**Status:** FOUND | COMPLETE | PLANNED | IMPLEMENTED
**Parallel Agent:** Agent 4
**Status:** SEARCHED | ANALYZED | COMPLETE
**Source:** Agent 4 (Parallel Search)
**Ready for Planning:** Yes/No

### Files Explored
> Populated by Agent 4 during Deep Dive

| File | Lines | Key Findings |
|------|-------|--------------|
| `path/to/client.ext` | XXX-YYY | [External API client] |

### External Dependencies

| Dependency | Type | Purpose |
|------------|------|---------|
| [API Name] | HTTP API | [What it does] |
| [Library] | Package | [What it provides] |

---

## CHUNK-R5: Test Coverage
## CHUNK-R5: Test Coverage (Deep Dive)

**Status:** FOUND | COMPLETE | PLANNED | IMPLEMENTED
**Parallel Agent:** Agent 5
**Status:** SEARCHED | ANALYZED | COMPLETE
**Source:** Agent 5 (Parallel Search)
**Ready for Planning:** Yes/No

### Existing Tests
### Files Explored
> Populated by Agent 5 during Deep Dive

| File | Lines | Key Findings |
|------|-------|--------------|
| `tests/test_feature.ext` | XXX-YYY | [Existing tests] |

### Existing Tests
| Test File | Coverage Area |
|-----------|---------------|
| `tests/test_feature.ext` | [What scenarios] |
Expand Down Expand Up @@ -240,6 +247,6 @@ After research completes:
**Context Usage Report:**
- Files read: X
- Tokens used: ~Xk (X% of 200k)
- Parallel agents spawned: 3-5
- Chunks created: N
- Parallel agents spawned: 5
- Chunks created: 5
- Compaction needed: Yes/No
Loading