dispatch: session close 2026-05-22 — DTS test fixes, build, deploy

Author: Douglas Jones

dispatches/2026-05-22-session-close.readout.md

@@ -0,0 +1,40 @@
+# Session Close — 2026-05-22
+
+## Session Summary
+
+Fixed 35 failing tests in the DecodeTheSign web API test suite, ran a clean build, deployed to production on Vercel, and synced state to remote.
+
+## Work Completed
+
+### DecodeTheSign — Test Suite Fixes
+
+**Root cause 1 — missing mock exports (`consumer-live-capture-route.test.ts`, `consumer-live-capture-errors.test.ts`)**
+
+The capture route imports `extractSignCondition` and `isSignConditionEligibleToWin` from `@/lib/ocr/extract`. Both test files mocked that module but only included `extractWithConfiguredProvider` and `extractDirectionalHint`. Vitest throws a hard error when a mocked module is missing exports that the module under test imports, causing all 11 capture route tests to return 500 instead of 200.
+
+Fix: added `extractSignCondition: () => "normal"` and `isSignConditionEligibleToWin: () => true` to the mock factory in both files.
+
+**Root cause 2 — rule engine cache bleeding across tests (`rule-engine.test.ts`)**
+
+`lib/rule-engine.ts` has an in-memory LRU cache (`ruleEvaluationCache`) with a 5-minute TTL. The `beforeEach` called `vi.clearAllMocks()` but never cleared the cache. Tests that ran after a green-result test would hit the cache and return the stale green result instead of evaluating the freshly mocked rules, causing 24 failures (all returning "green" when "red" was expected).
+
+Fix: imported `clearRuleEvaluationCache` from `lib/rule-engine.ts` and called it in `beforeEach`.
+
+### Result
+
+- 1007 tests passing, 6 skipped (property-based), 0 failures
+- `npm run build` clean
+- Deployed to production: https://decodethesign.com
+
+### Commits
+
+- `016c2dc7` — fix: test suite — add missing ocr/extract mock exports, clear rule engine cache between tests
+- `03a760fe` — feat(ios): ContentView, HomeView, NearbySignsCache, OnDeviceInference, LocationService updates
+
+Both pushed to `origin/main`.
+
+## State
+
+- DecodeTheSign web API: all tests green, production deployed
+- iOS native: pending changes committed (ContentView, HomeView, NearbySignsCache, OnDeviceInference, LocationService)
+- Dispatch pairs: complete

dispatches/2026-05-22-session-close.yaml

@@ -0,0 +1,39 @@
+date: "2026-05-22"
+type: session-close
+project: DecodeTheSign
+status: complete
+
+summary: >
+  Fixed 35 failing tests (missing mock exports + rule engine cache bleed),
+  ran clean build, deployed to Vercel production, pushed 2 commits to remote.
+
+commits:
+  - sha: 016c2dc7
+    repo: decodethesign
+    message: "fix: test suite — add missing ocr/extract mock exports, clear rule engine cache between tests"
+  - sha: 03a760fe
+    repo: decodethesign
+    message: "feat(ios): ContentView, HomeView, NearbySignsCache, OnDeviceInference, LocationService updates"
+
+fixes:
+  - file: tests/consumer-live-capture-route.test.ts
+    issue: "@/lib/ocr/extract mock missing extractSignCondition and isSignConditionEligibleToWin"
+    resolution: "added both to mock factory"
+  - file: tests/consumer-live-capture-errors.test.ts
+    issue: "same missing mock exports"
+    resolution: "added both to mock factory"
+  - file: tests/rule-engine.test.ts
+    issue: "ruleEvaluationCache not cleared between tests causing stale green results"
+    resolution: "import clearRuleEvaluationCache, call in beforeEach"
+
+test_result:
+  passed: 1007
+  skipped: 6
+  failed: 0
+
+deploy:
+  platform: vercel
+  url: https://decodethesign.com
+  status: success
+
+dispatch_check: complete