Skip to content

fix missing benchmark suite - #1

Open
Svector-anu wants to merge 2 commits into
mainfrom
fix/restore-benchmark-suite
Open

fix missing benchmark suite#1
Svector-anu wants to merge 2 commits into
mainfrom
fix/restore-benchmark-suite

Conversation

@Svector-anu

@Svector-anu Svector-anu commented Aug 25, 2026

Copy link
Copy Markdown
Owner

why

Commit 3de875d moved internal research out of the public repository and deleted research/benchmarks/benchmark-suite-v1.json, but the public CLI kept exposing benchmark commands whose runner still requires that file. As a result, benchmark run, invariants, gate, and determinism all failed before evaluating anything.

fix

Restore the executable benchmark suite under packaged data/benchmarks/ and make npm stage it before creating a tarball. Internal planning notes, session files, and benchmark snapshots remain private.

verification

  • npm test — 22/22 passed
  • npm run build — passed
  • rightstack benchmark invariants --json — executes and reports corpus findings
  • rightstack benchmark determinism --runs 2 --json — deterministic, no divergences
  • Built and installed the actual npm tarball, then ran determinism from the clean installation — passed
  • rightstack benchmark gate --min-pass-rate 0.95 --json — executes correctly and exposes current quality debt: 38/50 pass, 12 partial, 0 fail; Q010 is the only partial golden query

The quality-gate failures are deliberately not hidden or changed in this restoration PR.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant