Receipt · 6084d1ea-1
Synchronous child_process calls block the Node event loop on every request
performancemediumclosed in 4b9b492closed in 1 hour
repo 6f7fc663·PR #3·reviewed 2 days ago·2 days ago
The finding
- dashboard/app/api/secrets/route.ts:33-50
- dashboard/app/api/secrets/route.ts:96-99
- dashboard/app/api/secrets/route.ts:115
Every request invokes `execSync`/`execFileSync` (auth-status check plus list/set/delete), blocking the single Node worker thread for the full duration of the `gh` invocation (often hundreds of ms to seconds because gh performs network calls). In a Next.js server this stalls all concurrent requests on that worker.
Fix
Use `child_process.execFile` with a promise wrapper (`util.promisify`) and `await` it in the async handlers.
Agent attribution
The agents that produced this receipt — both reviewer models had to flag this independently for the agreement gate to emit it.
anthropic
gpt-5
63.7s · error
openai
claude-opus-4-7
70.3s · error
Total
wall-clock review time · est. inference cost
70.3s · $0.40
Sweeper
closed at SHA 4b9b492
closed in 1 hour
internal review id · 6084d1ea
Third-party witnesses
Everything below lives on GitHub's event log, not ours. Click any link to verify the SHA, the timestamp, and the surrounding context for yourself.
Closure receipt comment
https://github.com/AntFleet/aeon-bench/pull/3#issuecomment-4476008154Original review comment
https://github.com/AntFleet/aeon-bench/pull/3#issuecomment-4475352768The pull request
https://github.com/AntFleet/aeon-bench/pull/3