ReviewOS

also looking at this

stacks/rpx

[DO NOT MERGE] diag: #2267 CLI-hang discriminator

#2269
Closed chris wants to merge diag/2267-cli-hang into main
5 files +370 -6
10 days ago

Temporary diagnostic for #2267 — exists only to make CI run one test and be closed.

The two gateway.test.ts CLI cases hang on GitHub's Linux runners and nowhere else I can get to (not in oven/bun:1 on either arch under any configuration). Meanwhile cli.test.ts spawns the identical CLI via spawnSync and passes in the same run, so the entry point itself is fine.

zz-2267-diagnostic.test.ts runs eight spawn variants of bun bin/cli.ts --help from a parent that has the rpx graph loaded, with a 12s budget each, and dumps the stuck child's /proc state (status, wchan, kernel stacks via sudo -n, fds, cwd, children, io) before killing it. Every line is prefixed [2267-diag].

variantisolates
G bun --version, node spawnis it spawning bun at all, or the CLI graph?
A cli --help, node spawn ['ignore','pipe','pipe']the exact failing shape (baseline)
B A + BUN_RUNTIME_TRANSPILER_CACHE_PATH=0transpiler-cache contention
C A + stdin pipe (ended)stdin-ignore
F A + cwd=tmpdirrepo-root bunfig.toml / cwd
I A + stdout/stderr inheritpipe plumbing vs child not running
D Bun.spawnthe node:child_process shim specifically
E spawnSync, parent has graphcli.test.ts's shape, but with the graph loaded

The test always passes; the log is the result. Will close once read.

🤖 Generated with Claude Code

5 changed files on the files tab, with 0 review threads.

Sign in to comment on this pull request.