Skip to content

feat(5.2): Extract EvolutionOrchestrator into evolution/orchestrator.py#40

Merged
Deepfreezechill merged 2 commits intomainfrom
epic/5.2-evolution-orchestrator
Apr 3, 2026
Merged

feat(5.2): Extract EvolutionOrchestrator into evolution/orchestrator.py#40
Deepfreezechill merged 2 commits intomainfrom
epic/5.2-evolution-orchestrator

Conversation

@Deepfreezechill
Copy link
Copy Markdown
Owner

Epic 5.2: Extract EvolutionOrchestrator

Changes

  • New: \�volution/orchestrator.py\ (132 lines) — dispatch_evolution, execute_contexts, schedule_background, log_background_result
  • Modified: \�volver.py\ — evolve(), _execute_contexts(), schedule_background(), _log_background_result delegate to orchestrator
  • New: \ ests/test_evolution_orchestrator.py\ — 21 tests (dispatch routing, throttling, background tasks, delegation seam, backward compat)

Call Chain Preserved

\
_execute_contexts → evolver.evolve(c) → dispatch_evolution → _evolve_fix/derived/captured
\
Batch and single-context paths both go through \�volve(), preserving subclass/hook compatibility.

Review Results (/8eyes + /collab)

Reviewer Initial After Fix
8eyes-sec ✅ PASS
8eyes-impl ❌ FAIL (batch bypass) ✅ Fixed
/collab Opus 4.6 ⚠️ APPROVE w/ 5 findings ✅ All addressed
/collab GPT-5.4 ❌ REQUEST CHANGES (3) ✅ All addressed

1,446 tests pass, 127 skipped

Brian Krafft and others added 2 commits April 3, 2026 15:26
- New: evolution/orchestrator.py (130 lines) — dispatch_evolution, execute_contexts, schedule_background, log_background_result
- Modified: evolver.py — evolve(), _execute_contexts(), schedule_background(), _log_background_result delegate to orchestrator
- New: tests/test_evolution_orchestrator.py — 18 tests (dispatch routing, throttling, background tasks, backward compat)
- Suite: 1,443 passed, 127 skipped

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
… logger, add delegation tests

- F1 HIGH: execute_contexts now calls evolver.evolve() not dispatch_evolution() directly
- F2 MEDIUM: Logger changed to Logger.get_logger(__name__) (repo standard)
- F5 LOW: Added 3 TestDelegationSeam tests (real SkillEvolver → orchestrator path)
- Suite: 1,446 passed, 127 skipped

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 3, 2026

🔒 Phase Gate Enforcement — 🚫 FAIL

Check Result Detail
Issue linkage No linked issues found. PR must contain "Closes #N", "Fixes #N", or "Resolves #N".

Verdict: FAIL
Timestamp: 2026-04-03T22:34:08.325Z
Run: View workflow run


How to fix: Ensure all prerequisite phases are complete, or add emergency:bypass label with a ## Bypass Reason section in the PR body.

@Deepfreezechill Deepfreezechill merged commit 9df2f33 into main Apr 3, 2026
0 of 5 checks passed
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