Skip to content

[v5] reviewer lane follow-ups deferred from #5480 delivery #5617

Description

@clubanderson

The #5480 reviewer lane shipped (#5485 implementation, #5515 ledger-sync fix proven by the #5511 Spin model, #5588 adjudication audit events) and is live on a hosted spoke. Deferred items from the delivery, none blocking:

  1. G2 — pending-wipe: escalation.Sweep treats CI-pending as not-red and deletes ledger entries, making the distinct-SHA breaker probabilistic. Remains the one pinned expected-fail in src/formal/escalation (witness at 7,706 states). Fix + flip the model expectation.
  2. Embedded kick template: reviewer lane uses the hardcoded builder fallback; ship a reviewer-lane.md template per the Phase-4 template plan.
  3. Structured hand-off notes: post-reviewer re-escalations go to a human with only labels as context; add a structured note (what the reviewer tried, why it failed).
  4. True creation-time ordering: work list orders escalated PRs by PR number as an age proxy; thread real timestamps through ci-failing.json rows.

Refs #5480

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions