fix: address final review findings

- Guard against division-by-zero in summary when totalRequests is 0
- Fix runner grace period: break semaphore loop on abort so in-flight
  requests can be waited on with 3s timeout
- Remove unused cli-table3 dependency

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: sup3x

package.json

@@ -20,7 +20,6 @@
   "engines": { "node": ">=18" },
   "dependencies": {
     "chalk": "^5.3.0",
-    "cli-table3": "^0.6.5",
     "commander": "^12.1.0",
     "undici": "^7.0.0"
   },

src/core/runner.ts

@@ -57,6 +57,7 @@ export async function runBenchmark(
   // Semaphore: when one completes, start another
   while (inFlight.size > 0) {
     await Promise.race(inFlight);
+    if (signal?.aborted) break;
     while (inFlight.size < effectiveConcurrency && shouldContinue()) {
       startRequest();
     }

src/output/summary.ts

@@ -13,9 +13,11 @@ export function renderSummary(stats: BenchmarkStats, url: string): string {
   lines.push(DIVIDER);
   lines.push('');
   lines.push(`  Total Requests    ${formatNumber(stats.totalRequests)}`);
-  lines.push(`  Succeeded         ${success(String(stats.succeeded))}          ${dim(`(${formatPercent((stats.succeeded / stats.totalRequests) * 100)})`)}`);
+  const successPct = stats.totalRequests > 0 ? (stats.succeeded / stats.totalRequests) * 100 : 0;
+  lines.push(`  Succeeded         ${success(String(stats.succeeded))}          ${dim(`(${formatPercent(successPct)})`)}`);
   if (stats.failed > 0) {
-    lines.push(`  Failed            ${error(String(stats.failed))}          ${dim(`(${formatPercent((stats.failed / stats.totalRequests) * 100)})`)}`);
+    const failPct = stats.totalRequests > 0 ? (stats.failed / stats.totalRequests) * 100 : 0;
+    lines.push(`  Failed            ${error(String(stats.failed))}          ${dim(`(${formatPercent(failPct)})`)}`);
   }
   lines.push(`  Total Time        ${formatMs(stats.totalTimeMs)}`);
   lines.push(`  Requests/sec      ${bold(formatNumber(stats.requestsPerSec))}`);