diff --git a/conductor/tracks/code_path_audit_phase_3_provider_state_20260624/plan.md b/conductor/tracks/code_path_audit_phase_3_provider_state_20260624/plan.md index a8a37214..0bafde6c 100644 --- a/conductor/tracks/code_path_audit_phase_3_provider_state_20260624/plan.md +++ b/conductor/tracks/code_path_audit_phase_3_provider_state_20260624/plan.md @@ -98,7 +98,7 @@ - WHAT: delete the 12 alias declarations. Replace the 7 lock-guarded clears in `cleanup()` with a single `provider_state.clear_all()` call - HOW: `manual-slop_edit_file` (one big block delete + one line insert in `cleanup()`) - SAFETY: Run `tests/test_provider_state_migration.py` + all 7 per-provider test files. The `clear_all()` call iterates `_PROVIDER_HISTORIES.values()` and calls `.clear()` on each (with the RLock acquired per-history). Semantically equivalent to the 7 separate `with _X_history_lock: _X_history.clear()` blocks. -- [x] **COMMIT:** `refactor(ai_client): remove 12 module-level provider_state aliases; cleanup() uses clear_all()` (Tier 3, atomic) +- [x] **COMMIT:** `refactor(ai_client): remove 12 module-level provider_state aliases; cleanup() uses clear_all()` [da66adf] (Tier 3, atomic) - [x] **GIT NOTE:** 12 module-level aliases deleted. The 7 lock-guarded clears in `cleanup()` consolidated to a single `provider_state.clear_all()` call. Net diff: -10 lines (12 alias deletions - 2 added imports/comments). ## Phase 8: Verification + end-of-track (1 task, 3 commits)