From 4a55a14fc05910816da41e66c5f0f4fe0a93d734 Mon Sep 17 00:00:00 2001 From: Ed_ Date: Tue, 16 Jun 2026 10:26:10 -0400 Subject: [PATCH] conductor: register result_migration_20260616 in tracks.md (umbrella + 5 sub-tracks) --- conductor/tracks.md | 28 ++++++++++++++++++++++++++++ 1 file changed, 28 insertions(+) diff --git a/conductor/tracks.md b/conductor/tracks.md index 72828c78..4beb840c 100644 --- a/conductor/tracks.md +++ b/conductor/tracks.md @@ -24,6 +24,7 @@ Tracks that are unblocked and ready to start. Ordered by **dependency** (blocked | 6a | A | [Public API Migration + UI Polish Test Cleanup](#track-public-api-migration--ui-polish-test-cleanup) | spec ✓, plan ✓, shipped 2026-06-15 (13 pre-existing failures fixed; 3 RAG failures deferred to `rag_test_failures_20260615`) | (none — independent; **NEW 2026-06-15**; combined stability track) | | 6b | A | [RAG Test Failures Fix](#track-rag-test-failures-fix-new-2026-06-15) | spec ✓, plan ✓, shipped 2026-06-15 (3 RAG tests fixed; first fully green baseline 1288 + 4 + 0) | (none — independent; **NEW 2026-06-15**; small bug-fix track) | | 6c | B | [Exception Handling Audit (Convention Compliance + Doc Clarification)](#track-exception-handling-audit-convention-compliance--doc-clarification) | spec ✓, plan ✓, shipped 2026-06-16 (211 violations identified across 42 files; 5 doc gaps closed) | (none — independent; **NEW 2026-06-16**; audit + doc track; identifies the migration target for `data_structure_strengthening_20260606` and the user's `send_result` → `send` rename) | +| 6d | A | [Result Migration (5 sub-tracks)](#track-result-migration-5-sub-tracks-new-2026-06-16) | umbrella spec ✓; 5 sub-tracks pending (sub-track 1: `result_migration_review_pass`) | `exception_handling_audit_20260616`; identifies the migration target | (none — independent; **NEW 2026-06-16**; refactor phase; 5 sub-tracks eliminate the 268 "bad" sites per the audit; sub-tracks use the consistent `result_migration_*` prefix) | | 7 | — | [UI Polish (Five Issues)](#track-ui-polish-five-issues) | spec ✓, plan ✓, ready to start (Phases 1/4/5 shipped; Phases 2/3 code shipped but tests broken — fixed by track 6a) | (none — independent) | | 7a | B | [SQLite-Granularity Inline Docs for gui_2.py](#track-sqlite-granularity-inline-docs-for-gui_2py) | spec ✓, plan ✓, complete | (none — independent) | | 7b | B | [Continued SQLite-Granularity Inline Docs for gui_2.py](#track-continued-sqlite-granularity-inline-docs-for-gui_2py) | spec ✓, plan ✓, complete | (none — independent) | @@ -697,6 +698,33 @@ Lightweight chronology; full spec/plan/state per track is in the linked folder. *Out of scope (deferred to separate tracks): the `send_result` → `send` mass rename (user's stated manual refactor), 23 lower-impact weak-type files (`data_structure_strengthening_20260606`), `live_gui_mock_injection_20260615` infrastructure (separate track), RAG test quality cleanup (poll loops; separate track), and — most importantly — **any production code refactor** (this track is informational; the user decides what to migrate).* +#### Track: Result Migration (5 sub-tracks) `[track-created: 2026-06-16]` +*Link: [./tracks/result_migration_20260616/](./tracks/result_migration_20260616/), Spec: [./tracks/result_migration_20260616/spec.md](./tracks/result_migration_20260616/spec.md), Plan: [./tracks/result_migration_20260616/plan.md](./tracks/result_migration_20260616/plan.md), Metadata: [./tracks/result_migration_20260616/metadata.json](./tracks/result_migration_20260616/metadata.json), Audit: [../../docs/reports/EXCEPTION_HANDLING_AUDIT_20260616.md](../../docs/reports/EXCEPTION_HANDLING_AUDIT_20260616.md)* + +*Status: 2026-06-16 — Umbrella track; spec/plan/metadata planned. 5 sub-tracks pending. The umbrella specifies the sequence and scope of the 5 sub-tracks; each sub-track gets its own spec/plan/metadata when it starts.* + +*Goal: Eliminate all 211 violations + 25 suspicious + 32 unclear = **268 "bad" sites** across 42 files (per the `exception_handling_audit_20260616` report). After all 5 sub-tracks ship, the data-oriented error handling convention is fully applied to all 65 `src/` files, and the `audit_exception_handling.py --strict` mode can be wired into CI as a pre-commit gate.* + +*5 sub-tracks (consistent `result_migration_*` prefix):* + +| # | Sub-track | T-shirt | Scope | Why this position | +|---|---|---|---|---| +| 1 | `result_migration_review_pass` | S | 57 sites (32 UNCLEAR + 25 INTERNAL_RETHROW) across 15 files | First: human review + audit script heuristic updates inform all later sub-tracks | +| 2 | `result_migration_small_files` | L | 37 files (35 SMALL + 2 MEDIUM from `--by-size`); 72 V+S sites | Second: quick wins; doesn't depend on the orchestrator or GUI; can run in parallel with 3-4 | +| 3 | `result_migration_app_controller` | XL | 56 sites in `src/app_controller.py` (166KB; 13 FastAPI boundary stay as-is) | Third: high coordination with Hook API + MMA + RAG; gates the GUI migration | +| 4 | `result_migration_gui_2` | XL | 54 sites in `src/gui_2.py` (260KB) | Fourth: depends on 3 for clean API; the largest file | +| 5 | `result_migration_baseline_cleanup` | L | 112 sites in 3 refactored files (mcp_client.py, ai_client.py, rag_engine.py) | Fifth: closes the gaps in the convention reference; parent's Path C deferred work | + +*Total: 5 sub-tracks, 268 sites across 42 files, ~2100 lines changed.* + +*NO day estimates (per the new Tier 1 rule added 2026-06-16). Effort is measured by scope (N files, M sites) and T-shirt size (S/M/L/XL). The user / Tier 2 agent decides the actual pacing.* + +*Sequence: 1 (review) -> 2 (small files) -> 3 (app_controller) -> 4 (gui_2) -> 5 (baseline cleanup). Tracks 2 + 5 can run in parallel; tracks 3 + 4 must be sequential (the GUI calls controller methods); track 1 is independent.* + +*`blocks: data_structure_strengthening_20260606` (parallel track; uses the cleaner Result API from this phase) and the user's stated `send_result` → `send` mass rename.* + +*Out of scope (deferred to separate tracks): the `send_result` → `send` mass rename (user's stated manual refactor; post-this-phase), 23 lower-impact weak-type files (`data_structure_strengthening_20260606`), `live_gui_mock_injection_20260615` infrastructure (separate track), RAG test quality cleanup (poll loops; separate track), and **any audit script changes that belong in the review pass (sub-track 1)** — those are detailed in `conductor/tracks/result_migration_20260616/plan.md`.* + --- ## Phase 9: Chore Tracks