diff --git a/docs/reports/RESULT_MIGRATION_REVIEW_PASS_20260617.md b/docs/reports/RESULT_MIGRATION_REVIEW_PASS_20260617.md index ba59e709..9e4c4da7 100644 --- a/docs/reports/RESULT_MIGRATION_REVIEW_PASS_20260617.md +++ b/docs/reports/RESULT_MIGRATION_REVIEW_PASS_20260617.md @@ -184,7 +184,12 @@ Other audit findings (unchanged by this review pass): ### 2.10 `src/gui_2.py` — INTERNAL_RETHROW sites (2) -*(filled in Task 3.4)* +| Line | Context | Snippet | Decision | Pattern / Rationale | +|---|---|---|---|---| +| 757 | `App.__getattr__` (controller guard) | `if name == 'controller': raise AttributeError(name)` | **compliant** | Standard `__getattr__` + delegation pattern: the App class delegates to the controller; the `controller` attribute is set externally, so `__getattr__` raises AttributeError when it's not yet set (Python idiom for "not initialized yet") | +| 760 | `App.__getattr__` (default fallback) | `raise AttributeError(name)` (end of `__getattr__`) | **compliant** | Standard `__getattr__` pattern (same as app_controller L1224, L1250): raise AttributeError for any name that's not in the controller's interface | + +**Subtotals:** 2 compliant + 0 migration-target. ---