mirror of
https://github.com/yusufipk/OpenFrame.git
synced 2026-09-11 17:46:06 +00:00
feat(analytics): record where paying customers actually came from
Adds first-party acquisition attribution and a sixteen-event funnel, written to this deployment's own database and read back on /admin/growth. Nothing is sent anywhere else, and the whole subsystem is off unless OPENFRAME_ENABLE_ANALYTICS is set, so a self-hosted instance carries the tables empty and pays nothing. The proxy gives a visitor an anonymous id and stores what brought them in two first-party cookies; signup copies that onto the account and claims the events the visitor produced before they had one, which is what joins the two halves of the funnel. Recording happens where each step actually happens rather than in the browser: an ad blocker cannot undercount landing views, and blocking rates differ by channel, so an undercounted denominator would have made GitHub traffic look like it converts better than it does. Every event carries a dedupe key on a UNIQUE column, so "recorded exactly once" is a property of the schema rather than of fifteen call sites. Subscription events are derived by comparing the row being overwritten with the row being written inside the existing Stripe sync, which makes them order-independent and replay-safe. The scoreboard reports step-to-step conversion with the denominator beside it, and splits by source over a rolling 28-day window rather than a week: at this volume a weekly per-source cell holds single digits, and a percentage computed from three visits reads exactly as confidently as one computed from three hundred. "How did you hear about us?" is asked on the first onboarding screen, not on the registration form. The number being measured is the signup conversion rate, and a question added to that form would move it.
This commit is contained in:
@@ -44,6 +44,7 @@ import {
|
||||
} from '../factories';
|
||||
|
||||
import * as adminFeedbackRoute from '@/app/api/admin/feedback/[feedbackId]/route';
|
||||
import * as adminGrowthRoute from '@/app/api/admin/growth/route';
|
||||
import * as adminRefreshR2Route from '@/app/api/admin/stats/refresh-r2/route';
|
||||
import * as approvalCancelRoute from '@/app/api/approvals/[requestId]/cancel/route';
|
||||
import * as approvalDecisionRoute from '@/app/api/approvals/[requestId]/decision/route';
|
||||
@@ -54,6 +55,7 @@ import * as commentRoute from '@/app/api/comments/[commentId]/route';
|
||||
import * as feedbackRoute from '@/app/api/feedback/route';
|
||||
import * as feedbackUploadRoute from '@/app/api/feedback/upload/route';
|
||||
import * as onboardingCompleteRoute from '@/app/api/onboarding/complete/route';
|
||||
import * as onboardingSourceRoute from '@/app/api/onboarding/source/route';
|
||||
import * as approvalCandidatesRoute from '@/app/api/projects/[projectId]/approval-candidates/route';
|
||||
import * as projectDownloadRoute from '@/app/api/projects/[projectId]/download/route';
|
||||
import * as projectInvitationRoute from '@/app/api/projects/[projectId]/members/invitations/[invitationId]/route';
|
||||
@@ -143,7 +145,7 @@ vi.mock('@/lib/r2', async (importOriginal) => {
|
||||
// The count guard
|
||||
// ---------------------------------------------------------------------------
|
||||
// Bump this only together with a new entry in ROUTE_CASES or in PUBLIC_ROUTES.
|
||||
const EXPECTED_ROUTE_MODULE_COUNT = 60;
|
||||
const EXPECTED_ROUTE_MODULE_COUNT = 63;
|
||||
|
||||
/**
|
||||
* Routes that are public by design, and why. Everything else must reject an
|
||||
@@ -176,6 +178,15 @@ const PUBLIC_ROUTES: ReadonlyMap<string, string> = new Map([
|
||||
// so it cannot be used to enumerate accounts.
|
||||
'resend of the verification email, for users who cannot sign in yet',
|
||||
],
|
||||
[
|
||||
'events/route.ts',
|
||||
// The CTA-click beacon. Its whole job is to hear from visitors who have no
|
||||
// account yet, so a session cannot be the guard. It is bounded three ways
|
||||
// instead: same-origin only, IP rate limited, and it accepts exactly one
|
||||
// event name, so nothing a caller sends can forge a signup or a payment.
|
||||
// Covered in tests/api/analytics-events.test.ts.
|
||||
'anonymous CTA beacon, restricted to one event name and to same-origin callers',
|
||||
],
|
||||
[
|
||||
'stripe/webhook/route.ts',
|
||||
// Called by Stripe, not by a browser. Authenticated by the HMAC signature
|
||||
@@ -350,6 +361,11 @@ const ROUTE_CASES: readonly RouteCase[] = [
|
||||
url: (f) => `/api/admin/feedback/${f.feedbackId}`,
|
||||
params: (f) => ({ feedbackId: f.feedbackId }),
|
||||
},
|
||||
{
|
||||
file: 'admin/growth/route.ts',
|
||||
module: adminGrowthRoute,
|
||||
url: () => '/api/admin/growth',
|
||||
},
|
||||
{
|
||||
file: 'admin/stats/refresh-r2/route.ts',
|
||||
module: adminRefreshR2Route,
|
||||
@@ -405,6 +421,12 @@ const ROUTE_CASES: readonly RouteCase[] = [
|
||||
module: onboardingCompleteRoute,
|
||||
url: () => '/api/onboarding/complete',
|
||||
},
|
||||
{
|
||||
file: 'onboarding/source/route.ts',
|
||||
module: onboardingSourceRoute,
|
||||
url: () => '/api/onboarding/source',
|
||||
body: { source: 'GITHUB' },
|
||||
},
|
||||
{
|
||||
file: 'projects/[projectId]/approval-candidates/route.ts',
|
||||
module: approvalCandidatesRoute,
|
||||
|
||||
Reference in New Issue
Block a user