refactor: split harness/runner.py into focused modules (QUAL2)#34
Merged
Conversation
5 tasks
- Extract delivery-gate logic to _delivery_gate.py - Extract convergence loop + finding management to _convergence.py - Extract probe dispatch + step execution to _probe_dispatch.py - Extract result formatting + context resolution + replay to _result_fmt.py - Rewrite runner.py as thin orchestrator (149 lines, under 150-line limit) - All 124 tests pass, all 4 harness contract scripts exit 0" Agent-Logs-Url: https://github.com/markhazleton/devspark/sessions/d6521197-2cb6-4176-9d9f-40791d0ff87b Co-authored-by: markhazleton <223906+markhazleton@users.noreply.github.com>
…try param properly Agent-Logs-Url: https://github.com/markhazleton/devspark/sessions/d6521197-2cb6-4176-9d9f-40791d0ff87b Co-authored-by: markhazleton <223906+markhazleton@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Refactor: Split harness/runner.py into focused modules
refactor: split harness/runner.py into focused modules (QUAL2)
May 5, 2026
…ner methods Re-adds thin class-level delegate methods that were present on main-branch HarnessRunner but removed by the module split refactor: - resolve_step_adapter → _probe_dispatch.resolve_step_adapter - get_adapter → _probe_dispatch.get_adapter - build_retry_prompt → _probe_dispatch.build_retry_prompt - pause_for_human_review → _probe_dispatch.pause_for_human_review - write_supporting_artifacts → _result_fmt.write_supporting_artifacts - write_result → _result_fmt.write_result - write_lifecycle_artifacts → _result_fmt.write_lifecycle_artifacts All 124 tests pass; all 4 harness contract scripts exit 0. Agent-Logs-Url: https://github.com/markhazleton/devspark/sessions/3ae95ff6-afb3-4411-9a4c-4828ce717b5d Co-authored-by: markhazleton <223906+markhazleton@users.noreply.github.com>
markhazleton
approved these changes
May 5, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
_delivery_gate.py,_convergence.py,_probe_dispatch.py,_result_fmt.pyrunner.pyas thin orchestratorHarnessRunnermethods (resolve_step_adapter,get_adapter,write_supporting_artifacts,write_result,write_lifecycle_artifacts,build_retry_prompt,pause_for_human_review)