Copilot security review: main 968c7d6..79ffa9a

[judah-caruso]

Automated review of commits pushed directly to main, which Copilot cannot see otherwise.

Range: 968c7d6..79ffa9a (2 files)
Mode: security — Security review

This pull request is scratch and is closed automatically. Do not merge it.

[Copilot]

🔵 Needs a closer look

The PR is explicitly described as a scratch/closed automated review artifact for direct-to-main commits and is not intended to be merged.

Pull request overview

This PR updates Zebra’s transparent-script caching benchmark setup by replacing the Criterion-based benchmark harness with an interactive, stdin-driven measurement binary, and adding a companion script to build/run paired measurements across two fixed revisions.

Changes:

• Reworked zebra-consensus script benchmark into a main() program that prints READY, accepts batch sizes on stdin, and returns per-batch average timings (with optional cfg(cache_enabled) hit/miss behavior).
• Added zebra/bench-script-cache.sh to automate ABBA-style paired measurements across two revisions using git worktree, cargo rustc, and CSV/summary output generation.

File summaries

File	Description
zebra/zebra-consensus/benches/script.rs	Converts the benchmark from Criterion to an interactive timing harness supporting paired cache hit/miss measurements via cfg(cache_enabled).
zebra/bench-script-cache.sh	Adds an automation script that builds two benchmark binaries (with/without cache) from pinned revisions and collects paired timing observations into CSV + summary files.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[github-actions]

Recorded in the workflow run summary. Closing; this pull request is never merged.