TIER-2 READ conductor/code_styleguides/error_handling.md end-to-end before Phase 4
Re-read lines 462-540 (The Broad-Except Distinction). Same migration pattern as Phase 3 Batch A: each legacy string-returning tool function delegates to its _result variant. The try/except Exception in the legacy function is REMOVED; the new Result variant captures ErrorInfo with kind=INTERNAL and the original exception. Phase 4 = mcp_client Batch B: 8 INTERNAL_BROAD_CATCH sites (lines 473-593) - L473 get_git_diff - L492 ts_c_get_skeleton, L509 ts_c_get_code_outline, L523 ts_c_get_definition - L537 ts_c_get_signature, L555 ts_c_update_definition - L576 ts_cpp_get_skeleton, L593 ts_cpp_get_code_outline Target: mcp_client BC 32 -> 24 after Batch B.
This commit is contained in: