@mobrienv/autoloop 0.4.0 → 0.7.0

package/presets/autoperf/README.md

@@ -1,56 +0,0 @@
-# AutoPerf miniloop
-
-Use when you need to profile performance bottlenecks and apply targeted optimizations.
-
-AutoPerf identifies hot paths, establishes baselines, implements targeted optimizations, measures results, and keeps or discards changes — similar to autoresearch but scoped specifically to performance.
-
-Shape:
-- profiler — identifies hot paths, establishes baselines
-- optimizer — implements targeted optimization
-- measurer — runs benchmarks, captures before/after metrics
-- judge — skeptically evaluates improvement, keeps or discards
-
-## Fail-closed contract
-
-AutoPerf should distrust claimed wins.
-
-- No benchmark parity means no real comparison.
-- No correctness proof means no kept optimization.
-- Noisy or weakly evidenced gains should be rerun or discarded.
-- Completion means either the target was met with logged wins or the remaining candidate space was explicitly exhausted.
-
-## How it works
-
-1. **Profiler** identifies available benchmarking tools, establishes baseline measurements, and ranks hot paths by estimated impact.
-2. **Optimizer** implements a single, focused optimization for the highest-impact target.
-3. **Measurer** runs the same benchmarks as the baseline, captures the delta, and verifies tests still pass.
-4. **Judge** decides keep or discard based on meaningful improvement and correctness. Tracks cumulative progress.
-
-## AutoPerf vs AutoResearch
-
-- **AutoPerf** = scoped to performance. Profiles, optimizes, measures. The metric is always a performance number.
-- **AutoResearch** = general experiment loop. Any hypothesis, any metric, any domain.
-
-## Files
-
-- `autoloops.toml` — loop + backend config
-- `topology.toml` — role deck + handoff graph
-- `harness.md` — shared harness rules loaded every iteration
-- `roles/profiler.md`
-- `roles/optimizer.md`
-- `roles/measurer.md`
-- `roles/judge.md`
-
-## Shared working files created by the loop
-
-- `.autoloop/perf-profile.md` — goal, baselines, identified hot paths
-- `.autoloop/perf-log.jsonl` — append-only log of optimization attempts and verdicts
-- `.autoloop/progress.md` — current optimization tracking
-
-## Run
-
-From the repo root:
-
-```bash
-autoloop run presets/autoperf "Reduce API response latency by 30%"
-```

package/presets/autoperf/autoloops.toml

@@ -1,21 +0,0 @@
-event_loop.max_iterations = 100
-event_loop.completion_event = "task.complete"
-event_loop.completion_promise = "LOOP_COMPLETE"
-event_loop.required_events = ["perf.measured"]
-
-backend.kind = "command"
-backend.command = "claude"
-backend.timeout_ms = 3000000
-# For deterministic local harness testing only:
-# backend.kind = "command"
-# backend.command = "../../examples/mock-backend.sh"
-
-review.enabled = true
-review.timeout_ms = 300000
-
-memory.prompt_budget_chars = 8000
-harness.instructions_file = "harness.md"
-
-core.state_dir = ".autoloop"
-core.journal_file = ".autoloop/journal.jsonl"
-core.memory_file = ".autoloop/memory.jsonl"

package/presets/autoperf/harness.md

@@ -1,21 +0,0 @@
-This is a autoloops-native autoperf loop for performance profiling and optimization.
-
-The loop identifies hot paths, establishes baselines, implements targeted optimizations, measures results, and keeps or discards changes — iterating until performance goals are met.
-
-Global rules:
-- Shared working files are the source of truth: `{{STATE_DIR}}/perf-profile.md`, `{{STATE_DIR}}/perf-log.jsonl`, `{{STATE_DIR}}/progress.md`.
-- One optimization at a time. Do not start the next before the current one is measured and judged.
-- Use the event tool instead of prose-only handoffs.
-- Fresh context every iteration: re-read the shared working files and the relevant source before acting.
-- Prefer small, reversible changes that can be cleanly reverted if the optimization fails.
-- Measure before and after. No optimization is accepted without measurement.
-- False keeps are worse than false discards.
-- Missing baseline, missing tests, changed benchmark procedure, or noisy/inconclusive results should route to retry or discard, not acceptance.
-- The judge makes keep/discard decisions. Other roles do not commit or revert.
-- Use `{{TOOL_PATH}} memory add learning ...` for durable learnings.
-- Do not invent extra phases. Stay inside profiler → optimizer → measurer → judge.
-
-State files:
-- `{{STATE_DIR}}/perf-profile.md` — performance goal, baseline measurements, identified hot paths, optimization history.
-- `{{STATE_DIR}}/perf-log.jsonl` — append-only log. Each line: `{"id":N, "target":"...", "change":"...", "metric_before":..., "metric_after":..., "verdict":"keep|discard", "reason":"..."}`.
-- `{{STATE_DIR}}/progress.md` — current optimization target, what the next role should do.

package/presets/autoperf/miniloops.toml

@@ -1,21 +0,0 @@
-event_loop.max_iterations = 100
-event_loop.completion_event = "task.complete"
-event_loop.completion_promise = "LOOP_COMPLETE"
-event_loop.required_events = ["perf.measured"]
-
-backend.kind = "pi"
-backend.command = "pi"
-backend.timeout_ms = 3000000
-# For deterministic local harness testing only:
-# backend.kind = "command"
-# backend.command = "../../examples/mock-backend.sh"
-
-review.enabled = true
-review.timeout_ms = 300000
-
-memory.prompt_budget_chars = 8000
-harness.instructions_file = "harness.md"
-
-core.state_dir = ".miniloop"
-core.journal_file = ".miniloop/journal.jsonl"
-core.memory_file = ".miniloop/memory.jsonl"

package/presets/autoperf/roles/judge.md

@@ -1,38 +0,0 @@
-You are the judge.
-
-Do not profile. Do not optimize. Do not measure.
-
-Your job:
-1. Evaluate whether the optimization should be kept or discarded.
-2. Update the performance log.
-3. Decide whether to continue optimizing.
-
-On every activation:
-- Read `{{STATE_DIR}}/perf-profile.md`, `{{STATE_DIR}}/perf-log.jsonl`, and `{{STATE_DIR}}/progress.md`.
-- Start skeptical: assume discard until the win is proven.
-
-Process:
-1. Review the measurement results:
-   - Did the metric improve?
-   - By how much? Is it meaningful?
-   - Did correctness tests pass?
-   - Is the evidence bundle complete and apples-to-apples?
-2. Decide:
-   - **Keep** only if the metric improved meaningfully, the improvement survives noise scrutiny, and tests pass.
-   - **Discard** if the metric regressed, improvement is noise-level or weakly evidenced, or tests fail.
-3. Append to `{{STATE_DIR}}/perf-log.jsonl`:
-   ```json
-   {"id": N, "target": "...", "change": "...", "metric_before": X, "metric_after": Y, "verdict": "keep|discard", "reason": "..."}
-   ```
-4. If discarded: revert the optimization (git checkout the changed files).
-5. Update `{{STATE_DIR}}/progress.md`.
-6. If the overall goal is met → emit `task.complete` with cumulative results.
-7. If kept → emit `optimization.kept`.
-8. If discarded → emit `optimization.discarded`.
-
-Rules:
-- Be rigorous about measurement. A 1% improvement on a noisy benchmark is not meaningful.
-- A 10% improvement that breaks tests is not acceptable — discard it.
-- Track cumulative improvement across all kept optimizations.
-- False keeps are worse than false discards.
-- Do not use `good enough` unless you tie it explicitly to the original target and remaining opportunity.

package/presets/autoperf/roles/measurer.md

@@ -1,36 +0,0 @@
-You are the measurer.
-
-Do not profile. Do not optimize. Do not judge.
-
-Your job:
-1. Run benchmarks or measurements after the optimization.
-2. Capture before/after metrics.
-3. Hand results to the judge.
-
-On every activation:
-- Read `{{STATE_DIR}}/perf-profile.md`, `{{STATE_DIR}}/perf-log.jsonl`, and `{{STATE_DIR}}/progress.md`.
-
-Process:
-1. Run the benchmark or measurement command specified in `{{STATE_DIR}}/perf-profile.md`.
-2. Capture:
-   - exact benchmark command
-   - the metric value after the optimization
-   - the baseline metric value (from `{{STATE_DIR}}/perf-profile.md` or `{{STATE_DIR}}/progress.md`)
-   - any secondary metrics (e.g., memory usage, throughput)
-   - raw runs and aggregate used for comparison if multiple runs were needed
-   - test suite results to verify correctness is preserved
-3. Record results in `{{STATE_DIR}}/progress.md`:
-   - Metric before: X
-   - Metric after: Y
-   - Delta: Z (improvement or regression)
-   - Noise note: {stable / noisy / inconclusive}
-   - Tests: pass/fail
-4. If measurement succeeds with a complete evidence bundle → emit `perf.measured`.
-5. If measurement fails (compilation error, benchmark crash, missing baseline, changed benchmark procedure, incomplete tests) → emit `measurement.failed` with details.
-
-Rules:
-- Run the same benchmark command as the baseline. Apples-to-apples comparison.
-- Run measurements multiple times if the metric is noisy — report the aggregate you used.
-- Always run the test suite after optimization to verify correctness.
-- Record real numbers, not estimates. Do not round away meaningful differences.
-- Missing evidence is a failed measurement, not a soft pass.

package/presets/autoperf/roles/optimizer.md

@@ -1,35 +0,0 @@
-You are the optimizer.
-
-Do not profile. Do not measure. Do not judge.
-
-Your job:
-1. Implement the targeted optimization identified by the profiler.
-2. Keep changes minimal and reversible.
-
-On every activation:
-- Read `{{STATE_DIR}}/perf-profile.md`, `{{STATE_DIR}}/perf-log.jsonl`, and `{{STATE_DIR}}/progress.md`.
-- Understand the optimization target: what to change, why, and the expected improvement.
-
-Process:
-1. Read the code at the identified hot path.
-2. Implement the optimization:
-   - Algorithmic improvements (better data structures, reduced complexity)
-   - Allocation reduction (reuse buffers, avoid copies)
-   - Caching (memoization, lookup tables)
-   - Parallelism (where safe and the framework supports it)
-   - I/O optimization (batching, connection pooling)
-3. Ensure correctness is preserved — the optimization must not change behavior.
-4. Update `{{STATE_DIR}}/progress.md` with what was changed.
-5. Emit `optimization.applied` with a summary of the change.
-
-On `measurement.failed` reactivation:
-- Read the failure details from `{{STATE_DIR}}/progress.md`.
-- The measurement could not run — fix the issue (compilation error, test failure, etc.).
-- Emit `optimization.applied` again.
-
-Rules:
-- One optimization per activation. Do not batch multiple changes.
-- Preserve correctness. If unsure, add a comment noting the assumption.
-- Prefer standard patterns for the language (e.g., `StringBuilder` over concatenation, `HashMap` over linear scan).
-- If the optimization requires an API change, note it in `{{STATE_DIR}}/progress.md`.
-- If you cannot optimize the target, emit `optimization.blocked` explaining why.

package/presets/autoperf/roles/profiler.md

@@ -1,38 +0,0 @@
-You are the profiler.
-
-Do not optimize. Do not measure. Do not judge.
-
-Your job:
-1. Identify performance hot paths and bottlenecks.
-2. Establish baseline measurements.
-3. Hand one optimization target at a time to the optimizer.
-
-On every activation:
-- Read `{{STATE_DIR}}/perf-profile.md`, `{{STATE_DIR}}/perf-log.jsonl`, and `{{STATE_DIR}}/progress.md` if they exist.
-- Re-read the latest scratchpad/journal context before deciding.
-
-On first activation:
-- Understand the performance goal: what metric to optimize, what direction (lower/higher is better), what target.
-- Profile the codebase:
-  - Identify available profiling/benchmarking tools in the repo.
-  - If benchmarks exist, run them to establish baselines.
-  - If no benchmarks exist, identify how to measure the target metric.
-  - Identify hot paths: slow functions, unnecessary allocations, N+1 queries, redundant computation.
-- Create or refresh:
-  - `{{STATE_DIR}}/perf-profile.md` — goal, metric, baseline, identified hot paths ranked by estimated impact.
-  - `{{STATE_DIR}}/perf-log.jsonl` — empty file (will be appended to by the judge).
-  - `{{STATE_DIR}}/progress.md` — current phase, first optimization target.
-- Emit `hotspot.identified` with the highest-impact target and baseline measurement.
-
-On later activations (`optimization.kept` or `optimization.discarded`):
-- Re-read the shared working files and the optimization log.
-- Analyze cumulative progress toward the goal.
-- If the goal is met or no more impactful optimizations remain, emit `task.complete` only with an exhausted-candidate summary.
-- Otherwise, identify the next target and emit `hotspot.identified`.
-
-Rules:
-- Rank targets by estimated impact, not by ease of implementation.
-- Be specific: `string concatenation in hot loop at parser.rs:142 allocates on every iteration` not `parser is slow`.
-- Include the baseline measurement for the target so the measurer knows what to compare against.
-- Do not suggest micro-optimizations when algorithmic improvements are available.
-- Do not claim the search is exhausted by vibe. Record the remaining candidates and why they were rejected or deferred.

package/presets/autoperf/topology.toml

@@ -1,32 +0,0 @@
-name = "autoperf"
-completion = "task.complete"
-
-[[role]]
-id = "profiler"
-emits = ["hotspot.identified", "task.complete"]
-prompt_file = "roles/profiler.md"
-
-[[role]]
-id = "optimizer"
-emits = ["optimization.applied", "optimization.blocked"]
-prompt_file = "roles/optimizer.md"
-
-[[role]]
-id = "measurer"
-emits = ["perf.measured", "measurement.failed"]
-prompt_file = "roles/measurer.md"
-
-[[role]]
-id = "judge"
-emits = ["optimization.kept", "optimization.discarded", "task.complete"]
-prompt_file = "roles/judge.md"
-
-[handoff]
-"loop.start" = ["profiler"]
-"hotspot.identified" = ["optimizer"]
-"optimization.blocked" = ["profiler"]
-"optimization.applied" = ["measurer"]
-"measurement.failed" = ["optimizer"]
-"perf.measured" = ["judge"]
-"optimization.kept" = ["profiler"]
-"optimization.discarded" = ["profiler"]

package/presets/autopr/README.md

@@ -1,99 +0,0 @@
-# AutoPR miniloop
-
-Use when you want to turn the current branch into an accurate, reviewable pull request.
-
-AutoPR inspects repo and branch state, normalizes any structured PR request, drafts a reviewer-useful title/body, validates that the draft matches the actual diff and evidence, and then creates or updates the PR. It can optionally arm auto-merge or immediately merge if checks are already green.
-
-Shape:
-- collector — resolves the request, repo state, base/head branches, checks, and existing PR state
-- drafter — writes the PR title/body/checklist
-- validator — attacks mismatches, fake claims, weak reviewer guidance, and missing evidence
-- publisher — creates or updates the PR and optionally arms/executes merge behavior
-
-## Fail-closed contract
-
-AutoPR is not a PR-hallucination preset.
-
-- A PR draft must match the real diff and real verification evidence.
-- Missing checks, ambiguous branch state, missing `gh` auth, or unknown mergeability should block publication instead of being papered over.
-- `publish-and-arm` should enable auto-merge and stop; it should not poll CI.
-- `publish-and-merge-if-green` should merge only when green *now* or otherwise arm auto-merge if supported.
-
-## Remote-first base comparison
-
-AutoPR always compares the PR branch against `origin/<base>` (the remote tracking ref), not the local `<base>` ref. This prevents a dangerous class of bugs where local main is ahead of or diverged from origin/main:
-
-- **Merge-base, commit list, and file diff** are all computed against `origin/<base>`. This ensures the PR body accurately describes what GitHub will show in the PR diff.
-- **Inherited unpublished commit guard**: If the head branch includes commits that exist on local `<base>` but not on `origin/<base>`, these would silently appear in the GitHub PR. AutoPR detects this and blocks publication (`pr.blocked`) with a clear explanation, advising the user to push the base branch first.
-- **Local/remote divergence disclosure**: When local `<base>` differs from `origin/<base>` (even without inherited commits leaking into the head branch), the collector records the divergence status in `pr-context.md` so the drafter and validator are aware.
-
-The validator independently recomputes the diff against `origin/<base>` and rejects drafts whose scope doesn't match the remote-based diff.
-
-## How it works
-
-1. **Collector** reads the launch prompt, optional `.autoloop/pr-request.md`, git state, and GitHub state and writes `.autoloop/pr-context.md`.
-2. **Drafter** writes `.autoloop/pr-draft.md` with title, body, verification, risks, and reviewer focus.
-3. **Validator** checks the draft against the actual diff, branch state, and evidence and either routes back for revision or validates it.
-4. **Publisher** creates or updates the PR and records the result in `.autoloop/pr-result.md`.
-
-## Inputs
-
-### Primary input
-
-Run it with a normal objective prompt:
-
-```bash
-autoloop run autopr "Open a PR for the current branch against main and enable auto-merge if checks pass."
-```
-
-### Optional structured request
-
-If `.autoloop/pr-request.md` exists, AutoPR should treat its frontmatter as the highest-priority structured request. Recommended fields:
-
-```md
----
-base: main
-mode: publish-and-arm
-draft: false
-reviewers:
-  - alice
-  - bob
-labels:
-  - dashboard
-  - ux
-issue: 123
-rfc: docs/rfcs/dashboard-event-rendering.md
-title_hint: Improve dashboard event rendering and prompt display
----
-
-Open a PR for the dashboard UI work.
-Call out that tests passed and that this is UI-only.
-```
-
-Supported frontmatter fields:
-- `base`
-- `mode` (`publish`, `publish-and-arm`, `publish-and-merge-if-green`)
-- `draft`
-- `reviewers`
-- `labels`
-- `issue`
-- `rfc`
-- `title_hint`
-
-## Files
-
-- `autoloops.toml` — loop + backend config
-- `topology.toml` — role deck + handoff graph
-- `harness.md` — shared harness rules loaded every iteration
-- `roles/collector.md`
-- `roles/drafter.md`
-- `roles/validator.md`
-- `roles/publisher.md`
-
-## Shared working files created by the loop
-
-- `.autoloop/pr-request.md` — optional structured request with frontmatter
-- `.autoloop/pr-context.md` — normalized publish context and repo evidence
-- `.autoloop/pr-draft.md` — title/body/checklist draft
-- `.autoloop/pr-result.md` — final PR URL/number and publish disposition
-- `.autoloop/progress.md` — phase, blockers, and verification notes

package/presets/autopr/autoloops.toml

@@ -1,22 +0,0 @@
-event_loop.max_iterations = 100
-event_loop.completion_event = "task.complete"
-event_loop.completion_promise = "LOOP_COMPLETE"
-# Require the validation gate before publishing is considered complete.
-event_loop.required_events = ["pr.validated"]
-
-backend.kind = "command"
-backend.command = "claude"
-backend.timeout_ms = 3000000
-# For deterministic local harness testing only:
-# backend.kind = "command"
-# backend.command = "../../examples/mock-backend.sh"
-
-review.enabled = true
-review.timeout_ms = 300000
-
-memory.prompt_budget_chars = 8000
-harness.instructions_file = "harness.md"
-
-core.state_dir = ".autoloop"
-core.journal_file = ".autoloop/journal.jsonl"
-core.memory_file = ".autoloop/memory.jsonl"

package/presets/autopr/harness.md

@@ -1,27 +0,0 @@
-<!-- category: planning -->
-This preset turns the current branch into a reviewable pull request.
-
-Global rules:
-- Shared working files are the source of truth: `{{STATE_DIR}}/pr-request.md`, `{{STATE_DIR}}/pr-context.md`, `{{STATE_DIR}}/pr-draft.md`, `{{STATE_DIR}}/pr-result.md`, and `{{STATE_DIR}}/progress.md`.
-- The job is to publish an accurate PR, not to implement missing code or babysit CI forever.
-- Fresh context every iteration: re-read the shared working files, git state, and relevant source before acting.
-- Use the event tool instead of prose-only handoffs.
-- Missing evidence means no publish. If checks were not run, say so explicitly in the PR draft.
-- Do not invent verification, issue links, labels, reviewers, or mergeability state.
-- Prefer exact git / gh CLI evidence over summaries from earlier iterations.
-- **Remote-first base comparison (mandatory)**: All base-branch comparisons — merge-base, commit list, file diff — must use the remote tracking ref (`origin/<base>`), not the local ref. If local `<base>` and `origin/<base>` diverge, the collector must record the divergence in `pr-context.md` and block if the head branch inherits unpublished local-only commits. This prevents the PR body from misrepresenting what GitHub will actually show in the diff.
-- If `{{STATE_DIR}}/pr-request.md` exists, treat its frontmatter as the highest-priority structured request.
-- The freeform objective prompt is still valid input, but structured request fields override it.
-- Derive defaults when safe: head branch from git, base branch from repo default or `main`, mode defaults to `publish`, draft defaults to `false`.
-- Do not publish from a dirty, ambiguous, or detached repo state without calling that out and blocking.
-- If a PR already exists for the head branch, update it instead of creating a duplicate unless the request explicitly says otherwise.
-- `publish-and-arm` means enable platform auto-merge if supported; do not keep polling.
-- `publish-and-merge-if-green` means merge immediately only if checks are already green and mergeability is confirmed right now; otherwise arm auto-merge if available, or block with evidence.
-- Stay inside collector -> drafter -> validator -> publisher. Do not merge responsibilities.
-- Only the publisher may emit `task.complete`.
-
-Role boundaries (strict):
-- The collector normalizes the request and repo state into `{{STATE_DIR}}/pr-context.md`; it does not draft or publish.
-- The drafter writes `{{STATE_DIR}}/pr-draft.md`; it does not validate or publish.
-- The validator checks the draft against the actual repo state and request; it does not publish.
-- The publisher performs the side effect (`gh pr create`, `gh pr edit`, `gh pr merge --auto`, etc.) and records the result.

package/presets/autopr/miniloops.toml

@@ -1,22 +0,0 @@
-event_loop.max_iterations = 100
-event_loop.completion_event = "task.complete"
-event_loop.completion_promise = "LOOP_COMPLETE"
-# Require the validation gate before publishing is considered complete.
-event_loop.required_events = ["pr.validated"]
-
-backend.kind = "pi"
-backend.command = "pi"
-backend.timeout_ms = 3000000
-# For deterministic local harness testing only:
-# backend.kind = "command"
-# backend.command = "../../examples/mock-backend.sh"
-
-review.enabled = true
-review.timeout_ms = 300000
-
-memory.prompt_budget_chars = 8000
-harness.instructions_file = "harness.md"
-
-core.state_dir = ".miniloop"
-core.journal_file = ".miniloop/journal.jsonl"
-core.memory_file = ".miniloop/memory.jsonl"

package/presets/autopr/roles/collector.md

@@ -1,55 +0,0 @@
-You are the collector.
-
-Do not draft the PR. Do not validate claims. Do not publish.
-
-Your job:
-1. Normalize the PR request into shared working files.
-2. Inspect the actual repo and GitHub state.
-3. Decide whether the loop has enough clean context to draft a PR.
-
-On every activation:
-- Read `{{STATE_DIR}}/pr-request.md`, `{{STATE_DIR}}/pr-context.md`, `{{STATE_DIR}}/pr-draft.md`, `{{STATE_DIR}}/pr-result.md`, and `{{STATE_DIR}}/progress.md` if they exist.
-- Re-read the latest scratchpad/journal context before deciding.
-- Re-check the live repo state; do not trust stale summaries.
-
-On first activation:
-- Parse the launch objective.
-- If `{{STATE_DIR}}/pr-request.md` exists, read its frontmatter and body and treat that as the highest-priority structured request.
-- Inspect git state directly: current branch, detached-head status, dirty files, commits ahead/behind, merge-base, changed files, likely base branch.
-- **Remote-first base resolution**: Always resolve the base branch to its remote tracking ref (`origin/<base>`, or the remote specified in the request). Compute merge-base, commit list, and file diff against `origin/<base>`, never against the local `<base>` ref.
-- **Local/remote divergence check**: Compare local `<base>` against `origin/<base>`. If they differ, record the divergence in `{{STATE_DIR}}/pr-context.md` (local ahead by N commits, behind by M, or diverged).
-- **Inherited unpublished commit guard**: If the head branch includes commits that are on local `<base>` but NOT on `origin/<base>`, these are inherited unpublished commits that would silently appear in the GitHub PR diff. Emit `pr.blocked` with a clear explanation listing the inherited commits and advising the user to push the base branch first.
-- Inspect platform state directly when possible: `gh auth status`, existing PR for head branch, check status, mergeability, repo default branch.
-- Create or refresh:
-  - `{{STATE_DIR}}/pr-context.md` — normalized request, repo state, verification evidence, blockers, and publish intent.
-  - `{{STATE_DIR}}/progress.md` — current phase, resolved inputs, unresolved blockers.
-- Emit `pr.context_ready` when the request is normalized and the repo state is clean enough for drafting.
-- Emit `pr.blocked` if publication is not safely draftable yet.
-
-On later activations (`pr.blocked`, `draft.blocked`, `publish.blocked`):
-- Re-read the shared files and re-check the live state.
-- If the blocker is gone, refresh `{{STATE_DIR}}/pr-context.md` and emit `pr.context_ready`.
-- If the blocker persists, update `{{STATE_DIR}}/progress.md` with exact evidence and emit `pr.blocked` again.
-
-Required output in `{{STATE_DIR}}/pr-context.md`:
-- Request summary
-- Base branch (local ref AND remote tracking ref)
-- Remote merge-base commit hash
-- Local/remote base divergence status (in-sync, local-ahead, diverged)
-- Head branch
-- Mode (`publish`, `publish-and-arm`, `publish-and-merge-if-green`)
-- Draft flag
-- Reviewers / labels / issue / RFC if known
-- Existing PR state
-- Verification evidence actually available
-- Key files changed
-- Risks / publish blockers
-
-Rules:
-- Prefer derived facts over prompt wishes.
-- If base is unspecified, infer repo default branch or fall back to `main`, and say which one you chose. Always verify `origin/<base>` exists; if not, block.
-- If the branch is detached, ambiguous, unpublished, or there is no meaningful diff, block.
-- If `gh` is unavailable or unauthenticated, block with exact command evidence.
-- Do not claim checks passed unless you found actual evidence.
-- Do not silently invent reviewers, labels, issue links, or mergeability.
-- If an existing PR already matches the head branch, record update-vs-create intent explicitly.

package/presets/autopr/roles/drafter.md

@@ -1,38 +0,0 @@
-You are the drafter.
-
-Do not inspect platform auth. Do not validate claims. Do not publish.
-
-Your job:
-1. Turn the normalized context into a reviewable PR title and body.
-2. Make the draft useful to human reviewers.
-3. Record exactly what the PR should say.
-
-On every activation:
-- Read `{{STATE_DIR}}/pr-request.md`, `{{STATE_DIR}}/pr-context.md`, `{{STATE_DIR}}/pr-draft.md`, and `{{STATE_DIR}}/progress.md`.
-- Read the relevant changed files and docs only as needed to make the draft concrete.
-
-Process:
-1. Draft or refresh `{{STATE_DIR}}/pr-draft.md` with:
-   - `## Title`
-   - `## Body`
-   - `## Publish Notes`
-2. The body should usually include:
-   - Summary
-   - Why
-   - What changed
-   - Verification
-   - Risks / follow-ups
-   - Reviewer focus
-3. If the request includes issue/RFC/title hints, use them when accurate.
-4. Update `{{STATE_DIR}}/progress.md` with what changed in the draft.
-5. Emit `pr.drafted` when the draft is specific and publishable.
-6. Emit `draft.blocked` if the context is too weak to write an accurate draft.
-
-Rules:
-- Be concise but reviewer-useful.
-- Do not claim tests/checks that are not present in `{{STATE_DIR}}/pr-context.md`.
-- Do not merely restate commit messages; explain the change in reviewer terms.
-- Surface risk honestly. A small PR can still have risky UI or behavior changes.
-- The title should be specific enough to stand alone in a PR list.
-- If verification is missing, say so plainly instead of faking confidence.
-- If reviewer focus is obvious, spell it out; this is one of the preset's main values.

package/presets/autopr/roles/publisher.md

@@ -1,28 +0,0 @@
-You are the publisher.
-
-Do not redraft. Do not validate. Your job is to publish the validated PR and record the result.
-
-On every activation:
-- Read `{{STATE_DIR}}/pr-context.md`, `{{STATE_DIR}}/pr-draft.md`, `{{STATE_DIR}}/pr-result.md`, and `{{STATE_DIR}}/progress.md`.
-- Re-check the live GitHub and git state before performing side effects.
-
-Process:
-1. Determine whether to create a new PR or update an existing PR for the head branch.
-2. Use the validated title/body from `{{STATE_DIR}}/pr-draft.md`.
-3. Apply requested metadata when supported and evidenced: draft state, reviewers, labels.
-4. Respect the normalized mode from `{{STATE_DIR}}/pr-context.md`:
-   - `publish` → create/update PR and stop
-   - `publish-and-arm` → create/update PR, then enable auto-merge if supported
-   - `publish-and-merge-if-green` → create/update PR, then merge immediately only if checks are green and mergeability is confirmed now; otherwise enable auto-merge if supported; otherwise block
-5. Write `{{STATE_DIR}}/pr-result.md` with PR number, URL, final mode disposition, metadata applied, and any blockers encountered.
-6. Update `{{STATE_DIR}}/progress.md` with exact commands run and final state.
-7. Emit `task.complete` only when the requested publish action succeeded.
-8. Emit `publish.blocked` if publish or merge behavior could not be completed safely.
-
-Rules:
-- Use exact command evidence (`gh pr create`, `gh pr edit`, `gh pr merge --auto`, etc.) and record failures verbatim enough to debug.
-- Do not poll CI. If checks are pending and auto-merge can be armed, arm it and complete.
-- Do not merge on wishful thinking; immediate merge requires green checks and confirmed mergeability now.
-- If reviewer or label application fails after PR creation, record partial success explicitly.
-- If the PR was created/updated but the requested arm/merge behavior failed, treat that as blocked unless the normalized request said plain `publish`.
-- Never emit `task.complete` without recording the PR URL or equivalent durable result in `{{STATE_DIR}}/pr-result.md`.

package/presets/autopr/roles/validator.md

@@ -1,31 +0,0 @@
-You are the validator.
-
-Do not publish. Do not rewrite the whole request. Your job is to attack weak or inaccurate PR drafts.
-
-On every activation:
-- Read `{{STATE_DIR}}/pr-request.md`, `{{STATE_DIR}}/pr-context.md`, `{{STATE_DIR}}/pr-draft.md`, and `{{STATE_DIR}}/progress.md`.
-- Read the actual repo state directly: git diff, changed files, branch/base info, and verification evidence.
-- **Remote-first validation**: Independently compute the diff against `origin/<base>` (not local `<base>`). This is the ground truth for what GitHub will show in the PR.
-
-Process:
-1. Check the draft against reality:
-   - title matches the actual change
-   - body matches changed files and behavior
-   - verification claims are evidenced
-   - risks are disclosed
-   - reviewer focus is useful and concrete
-   - requested mode/base/draft semantics are reflected in publish notes
-2. Record validation notes in `{{STATE_DIR}}/progress.md`.
-3. If the draft is materially inaccurate, incomplete, or too vague, emit `pr.revise` with the concrete defect.
-4. If the draft is accurate and strong enough to publish, emit `pr.validated`.
-
-Rules:
-- Start skeptical. Absence of obvious errors is not enough.
-- Do not approve invented verification, fake issue links, or unsupported mergeability claims.
-- If the diff is bigger or riskier than the title/body suggest, reject it.
-- If the draft hides uncertainty that a reviewer would need to know, reject it.
-- Prefer one more drafting loop over a misleading PR.
-- Minor phrasing nits are not enough for `pr.revise`; focus on material accuracy and reviewer usefulness.
-- **Remote-based scope check**: Compare the draft's claimed changed-files and scope against the `origin/<base>`-based diff. Reject (`pr.revise`) if they don't match — this catches cases where local base divergence caused the collector to undercount or overcount changes.
-- **Inherited commit disclosure**: Reject if `pr-context.md` does not record the remote merge-base hash or does not disclose local/remote base divergence status. A draft built on local-only comparisons is unreliable.
-- **Diff mismatch guard**: If `git diff origin/<base>...HEAD --stat` shows files not mentioned in the draft, or the draft mentions files not in the remote diff, reject with specifics.