fix(waitlist): queue + retry failed signups instead of silently opening mailto (#165)
A signup that hit a network error, the 8s timeout or a 5xx was handed straight
to a `mailto:` composer. That path is lossy by design: it only works if the user
actually presses send, and if we keep reconciling the support inbox into Brevo
list 4 forever (AGE-61's hourly job). 20 of 21 signups were lost that way before
that reconciler existed, and Play's active base is ~100% on v0.4.10+ — so this
was current builds leaking, not just the ~436 stale sideloads.
Now:
- Failed-but-retryable signups are persisted on-device
(`opencode.waitlist.pending.v1`, AsyncStorage) and retried on every app
foreground (`app/_layout.tsx`) and on the Add Connection screen mount.
- 4xx stays non-retryable: the server will never accept that address, so we ask
the user to fix it instead of queueing garbage forever.
- `mailto:` is now only ever opened by an explicit user tap ("Still not working?
Email us instead"), shown after 3 failed attempts, or offered in an alert when
device storage itself refuses the write — never as the silent default.
- The UI tells the truth: "Saved on this device — we'll finish signing you up as
soon as you're back online" instead of implying it was sent.
- `WaitlistResult.fallback` -> `retryable`, `shouldFallbackToMailto` ->
`isRetryableFailure`: the decision is about retry, not about mail.
Queue policy: dedupe by email, cap 5 entries, 30-day TTL, corrupt/foreign JSON
is discarded rather than replayed. Storage and the clock are injected so the
whole thing runs under `node --test` (16 new tests, incl. the acceptance case:
offline signup -> queued -> reconnect -> reaches the server, no mail client).
Also commits the AGE-61 measurement artifacts that were only ever local
(`distribution/waitlist-signup-path-coverage.md`, `scripts/play-version-share.mjs`)
and updates the doc's "current builds still leak" section, which this fixes.
Refs AGE-87, AGE-61.
Co-authored-by: engineer <engineer@macbookpro.lan>
This commit is contained in:
@@ -19,6 +19,7 @@ import * as notifications from "../src/lib/notifications"
|
||||
import { addBreadcrumb, wrap } from "../src/lib/sentry"
|
||||
import { loadTelemetryConsent, setTelemetryConsent } from "../src/lib/telemetry"
|
||||
import { initAnalytics, trackAppOpened } from "../src/lib/analytics"
|
||||
import { flushPendingSignups } from "../src/lib/waitlist-queue-storage"
|
||||
|
||||
const queryClient = new QueryClient()
|
||||
|
||||
@@ -95,6 +96,30 @@ function RootLayout() {
|
||||
return () => sub.remove()
|
||||
}, [])
|
||||
|
||||
// Retry any waitlist signup that couldn't reach the server when the user
|
||||
// tapped Join (AGE-87). Runs at cold start and on every foreground, which is
|
||||
// the cheapest reliable proxy for "connectivity may have come back" — it is a
|
||||
// no-op (single storage read, no network) when the queue is empty, and it
|
||||
// replaces the old silent mailto: fallback that lost 20 of 21 signups.
|
||||
useEffect(() => {
|
||||
const flush = () => {
|
||||
void flushPendingSignups()
|
||||
.then((outcome) => {
|
||||
if (outcome.synced.length > 0) {
|
||||
addBreadcrumb({ category: "waitlist", message: `retried ${outcome.synced.length} queued signup(s)` })
|
||||
}
|
||||
})
|
||||
.catch(() => {
|
||||
// Best effort: the entry stays queued for the next foreground.
|
||||
})
|
||||
}
|
||||
flush()
|
||||
const sub = AppState.addEventListener("change", (next) => {
|
||||
if (next === "active") flush()
|
||||
})
|
||||
return () => sub.remove()
|
||||
}, [])
|
||||
|
||||
// Connect/disconnect SSE and load catalog when client changes
|
||||
useEffect(() => {
|
||||
if (client && !sseStarted.current) {
|
||||
|
||||
Reference in New Issue
Block a user