Perf test fixes

Author: huntharo

.github/workflows/build.yml

@@ -744,12 +744,15 @@ jobs:
 
           # Run performance tests and save results
           echo "Running latency test (20 concurrent, 60s)..."
-          oha -j --no-tui -c 20 -z 60s ${{ needs.deploy.outputs.url }}/ping > test-results/latency.json
+          oha --no-tui -j -c 20 -z 60s ${{ needs.deploy.outputs.url }}/ping > test-results/latency.json
+          cat test-results/latency.json
 
           echo "Running throughput test (100 concurrent, 60s)..."
-          oha -j --no-tui -c 100 -z 60s ${{ needs.deploy.outputs.url }}/ping > test-results/throughput.json
+          oha --no-tui -j -c 100 -z 60s ${{ needs.deploy.outputs.url }}/ping > test-results/throughput.json
+          cat test-results/throughput.json
 
           # Parse results and create markdown table
+          echo "Generating markdown table..."
           node -e '
             const fs = require("fs");
             const latencyResults = JSON.parse(fs.readFileSync("test-results/latency.json"));
@@ -762,12 +765,12 @@ jobs:
               "",
               "| Metric | Latency Test (20 concurrent) | Throughput Test (100 concurrent) |",
               "|--------|----------------------------|--------------------------------|",
-              `| Total Requests | ${formatNumber(latencyResults.total)} | ${formatNumber(throughputResults.total)} |`,
-              `| Requests/sec | ${formatNumber(latencyResults.rps)} | ${formatNumber(throughputResults.rps)} |`,
-              `| Mean Latency | ${formatNumber(latencyResults.latency_percentiles["50"])}ms | ${formatNumber(throughputResults.latency_percentiles["50"])}ms |`,
-              `| p95 Latency | ${formatNumber(latencyResults.latency_percentiles["95"])}ms | ${formatNumber(throughputResults.latency_percentiles["95"])}ms |`,
-              `| p99 Latency | ${formatNumber(latencyResults.latency_percentiles["99"])}ms | ${formatNumber(throughputResults.latency_percentiles["99"])}ms |`,
-              `| Max Latency | ${formatNumber(latencyResults.latency_percentiles["100"])}ms | ${formatNumber(throughputResults.latency_percentiles["100"])}ms |`,
+              `| Total Requests | ${formatNumber(latencyResults.summary.total)} | ${formatNumber(throughputResults.summary.total)} |`,
+              `| Requests/sec | ${formatNumber(latencyResults.summary.rps)} | ${formatNumber(throughputResults.summary.rps)} |`,
+              `| Mean Latency | ${formatNumber(latencyResults.summary.average)}ms | ${formatNumber(throughputResults.summary.average)}ms |`,
+              `| p95 Latency | ${formatNumber(latencyResults.summary.distribution["95"])}ms | ${formatNumber(throughputResults.summary.distribution["95"])}ms |`,
+              `| p99 Latency | ${formatNumber(latencyResults.summary.distribution["99"])}ms | ${formatNumber(throughputResults.summary.distribution["99"])}ms |`,
+              `| Max Latency | ${formatNumber(latencyResults.summary.slowest)}ms | ${formatNumber(throughputResults.summary.slowest)}ms |`,
               "",
               "_Note: Tests run against the /ping endpoint for 60 seconds each._",
               "",