From d059023b51a323df027c1cf033454440bd0d7e57 Mon Sep 17 00:00:00 2001 From: Thomas Date: Fri, 19 Jun 2026 21:21:50 +0200 Subject: [PATCH] feat(analytics): migrate from Umami to PostHog server-side analytics (#5983) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * feat(analytics): migrate from Umami to PostHog server-side analytics Replace @umami/node with posthog-node for all server-side telemetry. Add singleton PostHog client, feature tracking POC, DAU heartbeat, and tRPC analytics router. Cron job is now locked (no manual trigger, no custom interval). Update consent copy to explain what we collect and why. * style: auto-fix lint and format * refactor(analytics): simplify + add client-side posthog-js init - Inline constants.ts and track.ts into client.ts (single source) - Remove dead exports (shutdownPostHogAsync, getPostHogClient) - Flatten all DB queries into one Promise.all (no waterfall) - Derive integration/widget totals from group-by (drop redundant counts) - Add posthog-js with dynamic import, gated on enableGeneral - Move Analytics component to body, pass consent from server settings * style: auto-fix lint and format * feat(cron): add hidden flag to cron jobs, hide analytics from task panel The analytics job runs silently in the background — users don't need to see or interact with it. Hidden jobs are filtered out of getAllAsync so they never appear in the management UI. * refactor(analytics): simplify to single toggle, always collect all data Remove granular enableWidgetData/enableIntegrationData/enableUserData toggles. Analytics is now a single on/off switch. When enabled, all anonymous usage data is collected in one flat Promise.all call. Side effect: existing users with enableGeneral=true who had granular flags off will now send all data. This is by design — the granular controls added complexity without meaningful privacy benefit since the data is already anonymous counts. * style: auto-fix lint and format --- apps/nextjs/package.json | 1 + apps/nextjs/src/app/[locale]/layout.tsx | 2 +- .../_components/analytics.settings.tsx | 5 +- .../src/components/layout/analytics.tsx | 30 +- package.json | 2 +- packages/analytics/package.json | 4 +- packages/analytics/src/client.ts | 23 + packages/analytics/src/constants.ts | 2 - packages/analytics/src/index.ts | 4 +- .../analytics/src/send-server-analytics.ts | 199 ++- packages/api/package.json | 1 + packages/api/src/root.ts | 1 + packages/api/src/router/analytics.ts | 51 + packages/api/src/router/serverSettings.ts | 13 +- .../src/router/test/serverSettings.spec.ts | 7 +- packages/cron-jobs-core/src/creator.ts | 2 + packages/cron-jobs/src/job-manager.ts | 22 +- packages/cron-jobs/src/jobs/analytics.ts | 9 +- packages/server-settings/src/index.ts | 4 +- packages/translation/src/lang/en.json | 16 +- packages/validation/src/settings.ts | 3 - pnpm-lock.yaml | 1150 +++++------------ pnpm-workspace.yaml | 3 +- 23 files changed, 625 insertions(+), 929 deletions(-) create mode 100644 packages/analytics/src/client.ts delete mode 100644 packages/analytics/src/constants.ts create mode 100644 packages/api/src/router/analytics.ts diff --git a/apps/nextjs/package.json b/apps/nextjs/package.json index e058f325c..cfea600e7 100644 --- a/apps/nextjs/package.json +++ b/apps/nextjs/package.json @@ -89,6 +89,7 @@ "mcp-handler": "catalog:", "next": "catalog:", "postcss-preset-mantine": "catalog:", + "posthog-js": "catalog:", "prismjs": "catalog:", "react": "catalog:", "react-dom": "catalog:", diff --git a/apps/nextjs/src/app/[locale]/layout.tsx b/apps/nextjs/src/app/[locale]/layout.tsx index a9c6b65cb..dbb81dc49 100644 --- a/apps/nextjs/src/app/[locale]/layout.tsx +++ b/apps/nextjs/src/app/[locale]/layout.tsx @@ -139,11 +139,11 @@ export default async function Layout(props: { suppressHydrationWarning > - + diff --git a/apps/nextjs/src/app/[locale]/manage/settings/_components/analytics.settings.tsx b/apps/nextjs/src/app/[locale]/manage/settings/_components/analytics.settings.tsx index ed2dfcce8..9ad50b432 100644 --- a/apps/nextjs/src/app/[locale]/manage/settings/_components/analytics.settings.tsx +++ b/apps/nextjs/src/app/[locale]/manage/settings/_components/analytics.settings.tsx @@ -17,9 +17,10 @@ interface AnalyticsSettingsProps { export const AnalyticsSettings = ({ initialData }: AnalyticsSettingsProps) => { const t = useScopedI18n("management.page.settings.section.analytics"); + const { instanceId: _, ...formDefaults } = initialData; const form = useForm({ - initialValues: initialData, - onValuesChange: (updatedValues, _) => { + initialValues: formDefaults, + onValuesChange: (updatedValues, _prev) => { if (!form.isValid()) { return; } diff --git a/apps/nextjs/src/components/layout/analytics.tsx b/apps/nextjs/src/components/layout/analytics.tsx index 8d81fa13c..a59c65b7e 100644 --- a/apps/nextjs/src/components/layout/analytics.tsx +++ b/apps/nextjs/src/components/layout/analytics.tsx @@ -1,16 +1,24 @@ -import Script from "next/script"; +"use client"; -import { UMAMI_WEBSITE_ID } from "@homarr/analytics"; -import { db } from "@homarr/db"; -import { getServerSettingByKeyAsync } from "@homarr/db/queries"; +import { useEffect } from "react"; -export const Analytics = async () => { - // For static pages it will not find any analytics data so we do not include the script on them - const analytics = await getServerSettingByKeyAsync(db, "analytics").catch(() => null); +const POSTHOG_KEY = "phc_vYBmGWNbRshvfeC7EHfeSmUm2pD2Neg5nGqzJuGvS8Hs"; +const POSTHOG_HOST = "https://hog.homarr.dev"; - if (analytics?.enableGeneral) { - return