Skip to content

feat(ci): add CSP benchmarks and C-API static library#77

Merged
MatteoMer merged 4 commits into
mainfrom
feat/csp-benchmarks-ci
Apr 9, 2026
Merged

feat(ci): add CSP benchmarks and C-API static library#77
MatteoMer merged 4 commits into
mainfrom
feat/csp-benchmarks-ci

Conversation

@MatteoMer

Copy link
Copy Markdown
Owner

Summary

  • Add src/c_api.zig as a zolt_capi static library target in build.zig (produces libzolt_capi.a for FFI from Rust)
  • Add a non-blocking CI job that clones privacy-ethereum/csp-benchmarks and runs SHA-256 criterion benchmarks in test mode on macOS ARM64
  • Metrics (proof size, preprocessing size) are posted to the GitHub step summary and uploaded as artifacts

Test plan

  • zig build -Doptimize=ReleaseFast produces zig-out/lib/libzolt_capi.a
  • CI job succeeds with CSP_BENCHMARKS_TOKEN secret configured

🤖 Generated with Claude Code

MatteoMer and others added 4 commits April 10, 2026 00:14
Wire src/c_api.zig into build.zig as a separate static library target
(libzolt_capi.a) for FFI consumption. Add a non-blocking CI job that
clones csp-benchmarks and runs the SHA-256 criterion benchmarks in test
mode on macOS ARM64.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The nested checkout caused the csp-benchmarks directory to not exist.
Check out both repos at the workspace root so paths resolve correctly.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Point to MatteoMer/csp-benchmarks fork (feat/add-zolt-bench branch)
which has the zolt-bench crate. No token needed since the fork is public.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add run_csp_benchmarks input toggle, matching the pattern used by
the extended proof jobs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@MatteoMer MatteoMer merged commit 0324b34 into main Apr 9, 2026
34 checks passed
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