umputun / ralphex

79%

master: 78%

LAST BUILD ON BRANCH fix-process-group-cleanup

branch: fix-process-group-cleanup

CHANGE BRANCH

x

Reset

• fix-process-group-cleanup
• add-dirty-worktree-check
• add-reset-flag
• auto-plan-mode-detection
• configurable-colors
• dashboard-tests
• dependabot/go_modules/golang.org/x/crypto-0.45.0
• feat/scalar-config-fallback
• feature/claude-plugin-support
• fix-checkout-keep-untracked
• fix-codex-not-installed
• fix-isignored-global-patterns
• fix-scanner-buffer-size
• fix-uncommitted-changes-handling
• fix-windows-build
• fix-worktree-support
• graceful-prompt-variables
• interactive-plan-creation
• master
• project-local-config
• refs/tags/v0.1.0
• refs/tags/v0.2.0
• refs/tags/v0.2.1
• refs/tags/v0.2.2
• refs/tags/v0.2.3
• refs/tags/v0.3.0
• refs/tags/v0.4.0
• refs/tags/v0.4.1
• refs/tags/v0.4.2
• refs/tags/v0.4.3
• refs/tags/v0.4.4
• refs/tags/v0.5.0
• remove-docker
• web-planner
• web-ui
• web-ui-improvements

Build # 21342539068

Build Type

Pull #21

github

Committed by melonamin

Commit Message

fix: address codex review findings

Add pre-start context check to both Claude and Codex runners to avoid
spawning processes when the context is already canceled. This addresses
the valid finding that switching from exec.CommandContext to exec.Command
lost the pre-start cancellation behavior.

Note: The "Windows build break risk" finding was rejected as Windows
support is explicitly excluded in CLAUDE.md - the project only targets
Linux/macOS due to syscall.Flock usage for progress file locking.

Pull Request Pull Request #21: Fix: kill entire process group on context cancellation

Run Details

58 of 71 new or added lines in 3 files covered. (81.69%)

2998 of 3816 relevant lines covered (78.56%)

67.12 hits per line