Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
2 changes: 1 addition & 1 deletion docs/guides/agent-development.md
Original file line number Diff line number Diff line change
Expand Up @@ -318,7 +318,7 @@ Agent prompts are built with role-appropriate context via `_build_role_context()

**Interface role** (overseer) receives pipeline state, health alerts, and agent logs.

When adding a new execution role, `_build_role_context()` will automatically provide the summarized context and filter tasks by role. If the role needs phase-specific instructions (like the tester's "Focus your testing on..." or the documenter's "Focus your documentation on..."), add a condition in `_build_role_context()` for the new role.
When adding a new execution role, `_build_role_context()` will automatically provide the summarized context and filter tasks by role. If the role needs phase-specific instructions (like the tester's "Focus your testing on..." or the documenter's "Document the current state of the code in the areas these tasks touch..."), add a condition in `_build_role_context()` for the new role.

**All producer roles** (analysis + execution; not reviewers or utility) also receive a `## Subagent use for exploration` directive via the shared `_EXPLORATION_SUBAGENT_GUIDANCE` constant in `orchestrator/routes/pipelines.py`. New producer prompt builders should include `lines.extend(_EXPLORATION_SUBAGENT_GUIDANCE)` to inherit this directive automatically ([#2814](https://github.com/jwbron/egg/issues/2814)).

Expand Down
2 changes: 1 addition & 1 deletion docs/guides/sdlc-pipeline.md
Original file line number Diff line number Diff line change
Expand Up @@ -630,7 +630,7 @@ Agents are organized into five categories (execution, analysis, review, utility,
|------|----------|---------|-------------|
| **Coder** | Execution | Implements code changes and authors its own tests | All files except docs, `.egg-state/`, `.github/` (blocklist-complement; tests are coder-writable and overlap the tester — see [Agent Roles Reference](../reference/agent-roles.md#coder)) |
| **Tester** | Execution | Reviews-and-hardens the coder's tests, adds missing regression and adversarial coverage, runs linters, NACKs with a failing test when a bug is found | Test files and infrastructure only: `tests/`, `test/`, `**/test_*.py`, `**/*_test.go`, `**/*.test.{ts,tsx,js,jsx}`, `**/*.spec.{ts,tsx,js,jsx}`, `**/conftest.py` (see [Agent Roles Reference](../reference/agent-roles.md#tester)) |
| **Documenter** | Execution | Updates documentation | Documentation and markdown only: `docs/`, `**/*.md`, `**/README.md` (see [Agent Roles Reference](../reference/agent-roles.md#documenter)) |
| **Documenter** | Execution | Documents the current state of the code | Documentation and markdown only: `docs/`, `**/*.md`, `**/README.md` (see [Agent Roles Reference](../reference/agent-roles.md#documenter)) |
| **Autofixer** | Utility | Auto-fixes lint/format/type-check issues | Source and config files (no docs or contracts) |
| **Conflict Resolver** | Utility | Resolves merge and inter-agent conflicts | Source, test, doc, and config files (no `.egg-state/`) |
| **Reviewer (Code)** | Review | Reviews code for security, correctness | Review verdicts only |
Expand Down
2 changes: 1 addition & 1 deletion docs/reference/agent-roles.md
Original file line number Diff line number Diff line change
Expand Up @@ -305,7 +305,7 @@ tests, and its mandate is two-fold: **(1) comprehensive regression coverage**

### `documenter`

**Purpose**: Update documentation and READMEs.
**Purpose**: Document the current state of the code.

**File access** (defaults — overridable per-repo via `role_patterns:` in `repositories.yaml`):
- Owned scope (allowed writes): **documentation and markdown only.**
Expand Down
8 changes: 4 additions & 4 deletions orchestrator/routes/pipelines.py
Original file line number Diff line number Diff line change
Expand Up @@ -13620,7 +13620,7 @@ def _build_brc_preamble(
"results, gap reports back to the coder",
),
"documenter": (
"Updates documentation for changes",
"Documents the current state of the code",
"doc files, README updates, inline documentation",
),
"refiner": (
Expand Down Expand Up @@ -14798,10 +14798,10 @@ def _build_agent_prompt(
"- Describe what the code does now, not what changed or when it changed.",
"- NEVER reference SDLC artifacts — slice numbers, TASK-N ids, phase or "
"HITL iteration numbers — in any doc, docstring, or inline comment you write.",
"- Include historical context (issue links, \"previously X\" rationale, "
'- Include historical context (issue links, "previously X" rationale, '
"migration notes) ONLY when it is tangibly valuable to a reader of the "
"current system, and prefer rationale (\"why it is this way\") over "
"chronology (\"what it used to be / when it changed\").",
'current system, and prefer rationale ("why it is this way") over '
'chronology ("what it used to be / when it changed").',
"- When updating an existing doc, fold the new state into the snapshot and "
"REMOVE now-stale ledger or historical entries rather than appending "
"another layer.",
Expand Down
Loading