From 5aef87df286b70ffdb158dcec178565ffdb11ac3 Mon Sep 17 00:00:00 2001 From: Ed_ Date: Wed, 17 Jun 2026 15:53:07 -0400 Subject: [PATCH] docs(track): result_migration_review_pass decisions for src/gui_2.py INTERNAL_RETHROW (2 compliant + 0 migration-target) --- docs/reports/RESULT_MIGRATION_REVIEW_PASS_20260617.md | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) 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. ---