From 9d8be94edff503110ecdc1ede08398cffb61a79f Mon Sep 17 00:00:00 2001 From: Ed_ Date: Wed, 17 Jun 2026 15:55:10 -0400 Subject: [PATCH] docs(track): result_migration_review_pass decisions for src/models.py INTERNAL_RETHROW (1 compliant + 0 migration-target) --- docs/reports/RESULT_MIGRATION_REVIEW_PASS_20260617.md | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/docs/reports/RESULT_MIGRATION_REVIEW_PASS_20260617.md b/docs/reports/RESULT_MIGRATION_REVIEW_PASS_20260617.md index c90111f7..a34a7c82 100644 --- a/docs/reports/RESULT_MIGRATION_REVIEW_PASS_20260617.md +++ b/docs/reports/RESULT_MIGRATION_REVIEW_PASS_20260617.md @@ -206,7 +206,11 @@ Other audit findings (unchanged by this review pass): ### 2.12 `src/models.py` — INTERNAL_RETHROW site (1) -*(filled in Task 3.6)* +| Line | Context | Snippet | Decision | Pattern / Rationale | +|---|---|---|---|---| +| 268 | `models.__getattr__` (module-level PEP 562) | `raise AttributeError(f"module {__name__!r} has no attribute {name!r}")` | **compliant** | Standard module-level `__getattr__` pattern (PEP 562): handles `PROVIDERS` and `_PYDANTIC_CLASS_FACTORIES` lookups, then raises AttributeError for everything else. Python idiom | + +**Subtotals:** 1 compliant + 0 migration-target. ---