Skip to content

fix(cli): revert broken merge and stabilize captured process containment - #110

Open
marlon-costa-dc wants to merge 2 commits into
0.12.0-devfrom
fix/stabilize-captured-process-containment
Open

fix(cli): revert broken merge and stabilize captured process containment#110
marlon-costa-dc wants to merge 2 commits into
0.12.0-devfrom
fix/stabilize-captured-process-containment

Conversation

@marlon-costa-dc

@marlon-costa-dc marlon-costa-dc commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Reverts broken merge d58c2f2 and applies captured-process-containment runtime gates:

  • Revert merge: d58c2f2
  • Add: _runtime_process_output.py, _runtime_process_timing.py
  • Modify: runtime commands, process cleanup, execution, outcome, start, stream, threads, run_to_file, runtime.py, test_runtime_process_containment.py

Summary by cubic

Reverts the broken CLI merge that destabilized the runtime, then adds containment gates so captured subprocesses no longer leak during cleanup or streaming. The revert restores the pre-merge agent tooling and lint configuration. New _runtime_process_output.py and _runtime_process_timing.py modules centralize process handling, and containment tests now cover the new gates.

Written for commit f9c11fc. Summary will update on new commits.

Review in cubic

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
E Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant