reviewlisted

# Reviewer skill — agent-orchestrator ## CI / lint Reject (or request fixes) if any of these are red: - `ruff check orchestrator tests`. Common offenders to look for explicitly: - **F401** — unused import on the facade. If the import is intended as a re-export from `workflow.py`, it must be aliased `from X import Y as Y`. A bare import will not survive ruff. - **F541** — f-strings without placeholders, typically in newly-added test files. - **F841** — unused local in tests. - **E402** — import after non-import code. - `git diff --check origin/main...HEAD` — trailing whitespace and blank lines at EOF. Check it even if everything else looks clean. - Full `pytest` run is referenced in the PR description and passes end-to-end. Reject "known failure" hand-waves; if the PR claims a baseline failure, the description must include a reproduction on `origin/main` at the branch point. Otherwise the developer must fix it. ## Behavior preservation For any refactor: - Workflow labels, pinned-state JSON keys, comment marker text, watermark fields, and event-emission shape must match `main` exactly. Issues already in flight depend on these — a rename is a migration, not a refactor. - Spot-check that moved code still routes through the same auth / fetch / push / retry helpers. A refactor is not allowed to silently change side effects. - Squash-on-approval, auto-merge gates, retry budgets, and stale-session detection are easy to break by accident during a move; verify their call