From 27153d89eae02511c8c1913d62eee8cee023eaaf Mon Sep 17 00:00:00 2001 From: Ed_ Date: Wed, 17 Jun 2026 15:56:16 -0400 Subject: [PATCH] docs(track): result_migration_review_pass decisions for src/warmup.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 a34a7c82..ab56c8b5 100644 --- a/docs/reports/RESULT_MIGRATION_REVIEW_PASS_20260617.md +++ b/docs/reports/RESULT_MIGRATION_REVIEW_PASS_20260617.md @@ -216,7 +216,11 @@ Other audit findings (unchanged by this review pass): ### 2.13 `src/warmup.py` — INTERNAL_RETHROW site (1) -*(filled in Task 3.7)* +| Line | Context | Snippet | Decision | Pattern / Rationale | +|---|---|---|---|---| +| 85 | `WarmupManager.submit` (double-submit guard) | `raise RuntimeError("WarmupManager.submit() called twice; call reset() first")` | **compliant** | Validation raise for double-submit guard: the user called `submit` twice without `reset` in between, which is a programming error (API misuse). The error message is actionable. `RuntimeError` is in `PROGRAMMER_ERROR_EXCEPTIONS` | + +**Subtotals:** 1 compliant + 0 migration-target. ---