alibaba__open-code-review

mirror of https://github.com/alibaba/open-code-review.git synced 2026-09-14 19:59:52 +08:00

Files

T

History

超級の新人 c590b6b606 fix(launcher): forward SIGINT/SIGTERM signals to child process (#1145 )

The Node.js launcher wrapper (bin/ocr.js) was not forwarding signals
to the Go binary child process, causing orphaned processes when the
parent received SIGINT or SIGTERM.

Changes:
- Replace spawnSync with spawn for asynchronous child process handling
- Add signal handlers for SIGINT and SIGTERM that forward signals to child
- Add error event handler for spawn failures (ENOENT, EACCES)
- Properly propagate child exit status including signal-based termination
- Add integration tests to verify signal forwarding behavior

Fixes issue where Ctrl+C or kill commands left the Go process running
after the Node.js wrapper exited.

Resolves: #1142

2026-09-07 14:19:25 +08:00

action-contract.yml

fix(launcher): forward SIGINT/SIGTERM signals to child process (#1145 )

2026-09-07 14:19:25 +08:00

ci.yml

fix(launcher): forward SIGINT/SIGTERM signals to child process (#1145 )

2026-09-07 14:19:25 +08:00

codeql.yml

Create codeql.yml (#714 )

2026-08-04 19:51:27 +08:00

deploy-pages.yml

feat(pages): serve install scripts from custom domain (#797 )