diff --git a/assets/agents/architect/config.yaml b/assets/agents/architect/config.yaml index 9e5e92f..762cf83 100644 --- a/assets/agents/architect/config.yaml +++ b/assets/agents/architect/config.yaml @@ -6,7 +6,7 @@ description: | plan-conformance check (plus a black-box usage-pattern probe for consumer-facing surface), and finishes with ONE draft PR (CI checks watched to green) plus tracked follow-up tasks. Task state lives on disk in a plans directory, so runs survive context compression. -version: 2.5.0 +version: 2.6.0 agent_session: temp auto_continue: true max_auto_continues: 100 @@ -373,6 +373,12 @@ instructions: | `## Review round N @ ` (lane table, verdicts, what changed since the prior round) — never by rewriting the PR description, which carries only the stable summary and the required digest sections; comments are the append-only review log. + The same rule covers EVERY later push to an already-open PR (addressing human review + comments, a reviewer bot's findings, CI failures): the resumed Sisyphus session posts a + NEW top-level comment titled `## Update @ ` mapping each addressed + comment/finding to its commit. The description is edited only when the PR's scope or + behavior changed enough that it would otherwise misdescribe the PR — never as a + changelog of the in-between work. - **Spawn `adversary`** with the diff base and the criteria pasted in: ``` @@ -622,6 +628,10 @@ instructions: | an unprobeable consumer-facing change is a plan gap to fix, not a checkbox to skip. - Code comments referencing the plan/design doc/phases/steps/TASK numbers → docs drift, comments rot; plan references live in commit messages only. + - Review-round or update summaries appended to a PR description → the description stays a stable + description of the PR; every round/update lands as a NEW top-level PR comment + (`## Review round N @ ` / `## Update @ `), so each in-between commit's purpose is + visible in the PR timeline instead of bloating the description. - A `.env` file landing in any repo → only `.env.example` with placeholder keys is committable; `.env` stays gitignored and real values are a follow-up. - Dropping a Sisyphus-reported follow-up (not logged in the task's log.md, not in the PR's diff --git a/assets/agents/sisyphus/config.yaml b/assets/agents/sisyphus/config.yaml index d7c2c21..ae966a4 100644 --- a/assets/agents/sisyphus/config.yaml +++ b/assets/agents/sisyphus/config.yaml @@ -1,6 +1,6 @@ name: sisyphus description: OpenCode-style orchestrator - classifies intent, delegates to specialists, tracks progress with todos, enforces OMO-grade verification discipline -version: 3.13.0 +version: 3.14.0 agent_session: temp auto_continue: true @@ -709,6 +709,13 @@ instructions: | description: in-place edits destroy the review history and hide which commits each round actually reviewed. Comments are the append-only review log; the SHA in the title names exactly what was reviewed. + The same discipline covers EVERY later push to a PR you (or your run) opened — addressing + human review comments, a reviewer bot's findings, CI failures, or any other PR event: post + a NEW top-level PR comment titled `## Update @ ` mapping each addressed + comment/finding to the commit that addresses it (one line each), plus anything + deliberately NOT addressed and why. NEVER append update summaries to the description. + Edit the description ONLY when the PR's actual scope or behavior changed enough that it + would otherwise misdescribe the PR — it stays a description, never a changelog. **Close the loop.** When a decision DID reach the user (step 4), suggest in your final report that they record the resolution with `coyote --macro record-ruling` so that class of decision