From af598b3deeccb66d864d3947d161c1d8951f6feb Mon Sep 17 00:00:00 2001 From: Cal Corum Date: Fri, 28 Nov 2025 12:14:50 -0600 Subject: [PATCH] CLAUDE: Remove hit location requirement from lineout outcomes MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Lineouts are simple outs with no runner advancement logic - the backend doesn't use hit_location for them (unlike flyouts for sac flies or groundballs for double plays). Updated OUTCOMES_REQUIRING_HIT_LOCATION to exclude lineout and added documentation explaining the rationale. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- frontend-sba/constants/outcomes.ts | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/frontend-sba/constants/outcomes.ts b/frontend-sba/constants/outcomes.ts index 92db227..56f91ed 100644 --- a/frontend-sba/constants/outcomes.ts +++ b/frontend-sba/constants/outcomes.ts @@ -69,7 +69,13 @@ export const OUTCOME_CATEGORIES = [ /** * Outcomes that require a hit location to be specified - * These outcomes involve defensive plays where location matters + * These outcomes involve defensive plays where location matters for game logic: + * - Groundballs: determines double play eligibility + * - Flyouts: determines sac fly advancement opportunities + * - Uncapped hits/errors: determines runner advancement + * + * Note: lineout and popout are NOT included because they are simple outs + * with no runner advancement logic - the backend doesn't use hit_location for them */ export const OUTCOMES_REQUIRING_HIT_LOCATION = [ 'groundball_a', @@ -78,7 +84,6 @@ export const OUTCOMES_REQUIRING_HIT_LOCATION = [ 'flyout_a', 'flyout_b', 'flyout_c', - 'lineout', 'single_uncapped', 'double_uncapped', 'error',