ReviewOS

stacks/ts-analytics

Conversions overcount per instance: goal dedup is an in-process Map

#174
Closed glennmichael123 opened this 9 days ago ·
9 days ago

hasConverted/markConverted is a per-process Map capped at 1000 sessions with arbitrary eviction — the same session converts once per Lambda instance and again after restarts. Revenue-grade metric, inflated. Fix: conditional put on CONV#{sessionId}#{goalId} with attribute_not_exists + TTL.

From the production-readiness audit (docs/research/production-readiness.md, eacf996). Verified with live probes where stated.

9 days ago

Fixed — durable CONVLOCK conditional put, verified cross-process (CLAIMED/DUPLICATE).

Sign in to comment on this issue.