From 448319f8223f607a4ef9db908ba9d7c94e3f28fb Mon Sep 17 00:00:00 2001
From: Ed_ <edwardgz@gmail.com>
Date: Sat, 20 Jun 2026 08:37:21 -0400
Subject: [PATCH] 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.