Skip to content

ci: add an independent gate for eval harness tests #569

Description

@kjgbot

Why

PR #566 adds the durability eval runner and its fail-closed unit tests. Those tests are intentionally not wired into CI by that same PR because AGENTS.md forbids editing a gate that judges your own work. A separate, independently reviewed change should own the gate.

Scope

  • Run node --test scripts/run-evals.test.mjs scripts/benchmark-protocol.test.mjs in CI.
  • Keep the gate independent of PR feat: add defensible durability eval foundation #566 and do not weaken its assertions.
  • Confirm failures in timeout classification, execution witnesses, dirty provenance, output confinement, or competitive-protocol drift fail the job.
  • Avoid running the 210-trial publication benchmark on every PR unless runtime/cost is explicitly accepted; the fast harness tests are the required gate.

Current evidence

At PR #566 head c4092182870fe148afe673da38358ad7dd20c1e3, the literal local command reports 17 tests passed, 0 failed. The publication eval separately reports 210/210 at the same clean head.

Acceptance

A PR independent from #566 lands a required CI job that runs the two Node test files and demonstrates the job fails when one of their assertions is intentionally broken.

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