Skip to content

keeper: adopt the venue denial-classification seam from videre #669

Closed

Description

@mfw78

Problem. nxm-rs/videre#11 ships a venue-supplied denial-classification seam: Venue::classify_denied (default Drop) plus a classifier-parameterised keeper path, so the generic sweep and the composable-cow sweep can derive the same RetryAction from one mapping. Shepherd still derives them independently: the composable-cow sweep calls cow_venue::classification::classify_denied directly and the generic path (stop-loss) drops every denial.

Scope.

  • Bump the videre rev once keeper: let a venue supply denial classification to the generic retry path nxm-rs/videre#11 merges.
  • Override classify_denied on the CoW venue marker with cow_venue::classification::classify_denied.
  • Derive the composable-cow sweep and stop-loss retry actions through the seam (with_classifier / retry_action::<V>), removing the direct call.
  • Parity test: both paths return the same RetryAction for the same fault, including the InvalidEip1271Signature one-block grace.

Dependencies. nxm-rs/videre#11 merged and revved.

AI Assistance: Claude Fable filed this follow-up from the videre#11 design (seam here, adoption in shepherd).

Activity

  1. mfw78 commented on Oct 12, 2026

    @mfw78
    ContributorAuthor

    Superseded and done. The seam shipped in videre as a FaultPolicy (nxm-rs/videre#88) rather than Venue::classify_denied, and the CoW policy over it landed here in #691 together with the videre pin bump.

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