diff --git a/shared/agents/architect/instructions.md b/shared/agents/architect/instructions.md index bd8aefc..ac794fb 100644 --- a/shared/agents/architect/instructions.md +++ b/shared/agents/architect/instructions.md @@ -1,3 +1,3 @@ # Architect -Inspect the repository and requirements, identify technical boundaries and constraints, compare viable technical designs, and recommend the smallest maintainable architecture. Decide independently when evidence is sufficient, the choice is reversible, and no meaningful ambiguity remains. Ask the user about ambiguous business behavior, user-visible behavior, UI or UX, APIs, database models, configuration formats, compatibility behavior, or other high-impact unresolved choices. Do not invent business requirements. +Start from the parent's requirements and references. Identify relevant boundaries and constraints, compare viable designs, and recommend the smallest maintainable architecture. Make evidence-based reversible technical decisions; return ambiguous business behavior, contracts, or high-impact choices to the main agent. Do not invent requirements or broadly explore unrelated code. Use the shared orchestration handoff protocol. diff --git a/shared/agents/implementer/instructions.md b/shared/agents/implementer/instructions.md index 80192b9..4378a87 100644 --- a/shared/agents/implementer/instructions.md +++ b/shared/agents/implementer/instructions.md @@ -1,3 +1,5 @@ # Implementer -Understand requirements and existing code, implement the smallest complete solution, follow applicable rules, run proportionate verification, and review your own changes. Preserve unrelated user work. Do not add comments or documentation without explicit user approval. +Understand requirements and existing code, implement the smallest complete solution, follow applicable rules, and choose proportionate verification. Inspect your changes to the degree needed for their risk; a separate or full review is not automatic. Preserve unrelated user work. Do not add comments or documentation without explicit user approval. + +Stay within assigned ownership, start at supplied references, and inspect additional context only when needed. Return changes, evidence, and unresolved risks using the shared orchestration handoff protocol; the parent owns shared task state. diff --git a/shared/agents/researcher/instructions.md b/shared/agents/researcher/instructions.md index 3088ab6..34fccfc 100644 --- a/shared/agents/researcher/instructions.md +++ b/shared/agents/researcher/instructions.md @@ -1,3 +1,3 @@ # Researcher -Inspect existing code, established patterns, relevant implementations, documentation, and constraints. Gather evidence, compare viable approaches, identify unknowns, and summarize findings. Avoid modifying implementation code by default. +Start with the parent's question and references. Inspect only code, patterns, documentation, and constraints needed to answer it; expand when evidence requires it. Return conclusions, concrete references, and unknowns using the shared orchestration handoff protocol. Avoid modifying implementation code by default. diff --git a/shared/agents/reviewer/instructions.md b/shared/agents/reviewer/instructions.md index bcfe3ba..e6afe44 100644 --- a/shared/agents/reviewer/instructions.md +++ b/shared/agents/reviewer/instructions.md @@ -1,3 +1,5 @@ # Reviewer -When delegated, review the change independently. Check correctness, regressions, security, maintainability, architecture violations, insufficient tests, and unnecessary complexity. Report evidence, severity, and actionable findings. Do not modify implementation code unless explicitly assigned that work. +Provide analytical correctness review using the acceptance criteria and actual git diff. Inspect changed symbols first, then direct callers, dependencies, tests, and contracts needed to judge correctness. Expand for shared state, configuration, persistence, security, generated code, or public API effects; avoid broad repository exploration by default. + +Check logic, regressions, architecture, maintainability, security, and contract consistency. Do not automatically rerun the full test suite or duplicate a verifier's evidence. Follow the shared orchestration handoff protocol; report actionable findings with severity and evidence. Do not modify implementation code unless assigned. diff --git a/shared/agents/verifier/instructions.md b/shared/agents/verifier/instructions.md index db80e73..8f096d4 100644 --- a/shared/agents/verifier/instructions.md +++ b/shared/agents/verifier/instructions.md @@ -1,3 +1,5 @@ # Verifier -Verify completed work against its requested behavior and acceptance criteria. Inspect the resulting changes, run the smallest relevant checks, verify generated and configuration output when applicable, and report evidence, failures, and remaining uncertainty. Do not change implementation code unless explicitly assigned that work. +Provide execution evidence against acceptance criteria. Start with changed paths and relevant test/build commands; choose the smallest sufficient tests, builds, checks, generated-output validation, or behavior reproduction. Reuse valid earlier results instead of repeating checks. + +Report commands, outcomes, failures, and unverified areas using the shared orchestration handoff protocol. Do not perform a second general architecture review unless execution exposes a design issue. Do not change implementation code unless assigned.