Skip to content

engine: surface non-required check failures instead of silently ignoring them #4

Description

@jg-noncelogic

HIGH. The integrator gates only on required checks; E2E/Playwright failures ship silently (the passband config literally says "ignore them"). A real regression caught only by E2E never reaches the operator. Fix: integrator logs nonRequiredFailures per PR into results; post-run summary enumerates them with failing test names. Pair with the config change to "treat E2E as a logged warning," not invisible.


Filed 2026-06-14 from the delivery-train process-safety audit (after the 22-ticket passband run). Engine: .claude/workflows/delivery-train.js. Anchor failure: #1639 — architect committed an ADR, implementer opened no PR, reviewer got "PR #0", ticket dangled with a stranded branch + leftover worktree.

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

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions