Repository navigation
Conversation
Author
|
Follow-up after adversarial review: I updated the benchmark runner to use autocannon 8.0.0 as the configured load engine, added route coverage enforcement against mounted Express /api routes, made benchmark count env vars integer-validated, generated per-run unique payload IDs, kept the rate-limiter bypass behind BENCHMARK_DISABLE_RATE_LIMIT, and uploaded smoke reports as CI artifacts. Fresh local validation is in the PR body: smoke 20 endpoints/80 requests/0% errors and full 20 endpoints/1600 requests/0% errors. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
/claim #30
Closes #30.
Summary
Adds a reproducible
autocannonbenchmark suite underbenchmarks/for every mounted/api/route in the Express API. The runner can either start the local app in benchmark mode or target an existing host withBENCHMARK_TARGET_URL.What is included:
npm run benchmarkfor the full local/staging benchmark suitenpm run benchmark:smokefor low-concurrency CI regression checksautocannon8.0.0 as the configured benchmark engine/apiroutes/api/admin/metricsbenchmarks/results/benchmarks/thresholds.jsonGenerated result files are intentionally ignored by git; the latest local Markdown summary is pasted below for review.
Validation
node --check benchmarks/run-benchmark.js node --check benchmarks/routes.js node --check apps/api/src/app.js npm run benchmark:smoke npm run benchmark node --test apps/api/src/tests/*.test.js git diff --checkResults:
npm run benchmark:smoke: 20 endpoints, 80 requests, 0% error rate, all thresholds passednpm run benchmark: 20 endpoints, 1600 requests, 0% error rate, all thresholds passednode --test apps/api/src/tests/*.test.js: 1 passing health testBENCHMARK_REQUESTS=0.5now fail validation instead of producing corrupt reportsDemo
Short demo video: https://github.com/adityawrk/bug-bounty/releases/download/bounty-30-demo-20260530/securebanana-30-demo.mp4
Benchmark Summary
API Benchmark Report
Generated: 2026-05-30T13:13:17.669Z
Mode: default
Target: local loopback app
Environment
Summary
Endpoint Metrics
Threshold result: all configured thresholds passed.
Benchmark Environment
Hardware
Runtime
If submitted by or with an AI agent