Skip to content

feat: add solver supervisor stop dry run#227

Merged
SaJaToGu merged 1 commit into
developfrom
ai/fix-issue-223-stop-dry-run
Jun 5, 2026
Merged

feat: add solver supervisor stop dry run#227
SaJaToGu merged 1 commit into
developfrom
ai/fix-issue-223-stop-dry-run

Conversation

@SaJaToGu
Copy link
Copy Markdown
Owner

@SaJaToGu SaJaToGu commented Jun 5, 2026

Refs #223

Third small PR for the solver process supervisor split.

Scope:

  • adds solver_supervisor.py stop --dry-run
  • supports selecting exactly one run by run id, repo plus issue, or worker pid
  • previews known runner and worker PIDs plus child process tree
  • sends no signals and does not stop processes
  • rejects non-dry-run stop attempts in this slice

Validation:

  • ./.venv/bin/python -m unittest tests.test_solver_supervisor
  • ./.venv/bin/python scripts/solver_supervisor.py stop --run-id 20260604-232546-262166-ai-issue-solver-issue-189 --dry-run
  • ./.venv/bin/python -m unittest discover -s tests
  • git diff --check

Follow-up PRs for #223 should add real safe stop with worktree preservation and dashboard integration separately.

@SaJaToGu SaJaToGu merged commit 482ed16 into develop Jun 5, 2026
2 checks passed
@SaJaToGu SaJaToGu deleted the ai/fix-issue-223-stop-dry-run branch June 5, 2026 16:12
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