From 73157b4e0fdf41b03b69415fcba0869874b9a211 Mon Sep 17 00:00:00 2001
From: "Jean P.D. Meijer" <jean@analoginterface.io>
Date: Mon, 16 Jun 2025 11:42:36 +0200
Subject: [PATCH 1/3] fix(client): resolve error when env variable is undefined

---
 .../analytics/src/client/simple-analytics.tsx |  6 +--
 packages/analytics/src/client/utils.ts        | 39 +++++++------------
 2 files changed, 16 insertions(+), 29 deletions(-)

diff --git a/packages/analytics/src/client/simple-analytics.tsx b/packages/analytics/src/client/simple-analytics.tsx
index 77f8e4b..ddda2ed 100644
--- a/packages/analytics/src/client/simple-analytics.tsx
+++ b/packages/analytics/src/client/simple-analytics.tsx
@@ -1,6 +1,6 @@
 "use client";
 
-import React, { Suspense } from "react";
+import * as React from "react";
 import Script from "next/script";
 import type { AnalyticsMetadata } from "../interfaces";
 import { parseDataProps } from "./utils";
@@ -32,9 +32,9 @@ export const SimpleAnalytics = (props: SimpleAnalyticsProps) => {
   const dataProps = parseDataProps(props);
 
   return (
-    <Suspense fallback={null}>
+    <React.Suspense fallback={null}>
       <Script {...dataProps} src="/proxy.js" />
-    </Suspense>
+    </React.Suspense>
   );
 };
 
diff --git a/packages/analytics/src/client/utils.ts b/packages/analytics/src/client/utils.ts
index 6d77d02..b808e62 100644
--- a/packages/analytics/src/client/utils.ts
+++ b/packages/analytics/src/client/utils.ts
@@ -22,29 +22,16 @@ function isBuildTime() {
 }
 
 export function parseDataProps(settings?: SimpleAnalyticsProps) {
-  if (!process.env.NEXT_PUBLIC_SIMPLE_ANALYTICS_HOSTNAME) {
-    console.error("No hostname provided for Simple Analytics");
-    return {};
-  }
-
-  if (isBuildTime()) {
-    return {};
-  }
-
   if (!isProduction()) {
-    console.log(
-      "Simple Analytics is disabled by default in development and preview environments, enable it by setting NEXT_PUBLIC_ENABLE_ANALYTICS_IN_DEV=1 in your environment",
-    );
     return {};
   }
 
-  if (!settings) {
-    return {
-      "data-hostname": process.env.NEXT_PUBLIC_SIMPLE_ANALYTICS_HOSTNAME,
-    };
-  }
+  // if (!process.env.NEXT_PUBLIC_SIMPLE_ANALYTICS_HOSTNAME) {
+  //   console.log("No hostname provided for Simple Analytics");
+  //   return {};
+  // }
 
-  const metrics = settings.ignoreMetrics
+  const metrics = settings?.ignoreMetrics
     ? Object.entries(settings.ignoreMetrics)
         .filter(([_, value]) => value)
         .map(([key]) => `${key}`)
@@ -52,15 +39,15 @@ export function parseDataProps(settings?: SimpleAnalyticsProps) {
     : undefined;
 
   return {
-    "data-auto-collect": settings.autoCollect,
-    "data-collect-dnt": settings.collectDnt,
+    "data-auto-collect": settings?.autoCollect,
+    "data-collect-dnt": settings?.collectDnt,
     "data-hostname":
-      settings.hostname ?? process.env.NEXT_PUBLIC_SIMPLE_ANALYTICS_HOSTNAME,
-    "data-mode": settings.mode,
+      settings?.hostname ?? process.env.NEXT_PUBLIC_SIMPLE_ANALYTICS_HOSTNAME,
+    "data-mode": settings?.mode,
     "data-ignore-metrics": metrics === "" ? undefined : metrics,
-    "data-ignore-pages": settings.ignorePages?.join(","),
-    "data-allow-params": settings.allowParams?.join(","),
-    "data-non-unique-params": settings.nonUniqueParams?.join(","),
-    "data-strict-utm": settings.strictUtm,
+    "data-ignore-pages": settings?.ignorePages?.join(","),
+    "data-allow-params": settings?.allowParams?.join(","),
+    "data-non-unique-params": settings?.nonUniqueParams?.join(","),
+    "data-strict-utm": settings?.strictUtm,
   };
 }

From d059322999bc954b9446195d88f159d6b891f9aa Mon Sep 17 00:00:00 2001
From: "Jean P.D. Meijer" <jean@analoginterface.io>
Date: Mon, 16 Jun 2025 11:43:35 +0200
Subject: [PATCH 2/3] chore(server): reduce logging frequency

---
 .../src/plugin/with-simple-analytics.ts       |  5 ++++
 .../analytics/src/server/simple-analytics.ts  | 24 +++++++++----------
 2 files changed, 17 insertions(+), 12 deletions(-)

diff --git a/packages/analytics/src/plugin/with-simple-analytics.ts b/packages/analytics/src/plugin/with-simple-analytics.ts
index b4623be..0ea44ba 100644
--- a/packages/analytics/src/plugin/with-simple-analytics.ts
+++ b/packages/analytics/src/plugin/with-simple-analytics.ts
@@ -38,6 +38,11 @@ export function withSimpleAnalytics(
 ): NextConfig {
   const hostname = options?.hostname ?? process.env.SIMPLE_ANALYTICS_HOSTNAME;
 
+  if (!hostname) {
+    console.warn("No hostname provided for Simple Analytics, plugin disabled.");
+    return nextConfig;
+  }
+
   const clientHints = buildClientHintHeaders(options?.clientHints);
 
   const nextAnalyticsConfig: NextConfig = {
diff --git a/packages/analytics/src/server/simple-analytics.ts b/packages/analytics/src/server/simple-analytics.ts
index cbae883..6a13326 100644
--- a/packages/analytics/src/server/simple-analytics.ts
+++ b/packages/analytics/src/server/simple-analytics.ts
@@ -23,10 +23,10 @@ export async function trackEvent(
   eventName: string,
   options: TrackEventOptions,
 ) {
-  const hostname = options.hostname ?? process.env.SIMPLE_ANALYTICS_HOSTNAME;
+  const hostname = options?.hostname ?? process.env.SIMPLE_ANALYTICS_HOSTNAME;
 
   if (!hostname) {
-    console.error("No hostname provided for Simple Analytics");
+    // console.error("No hostname provided for Simple Analytics");
     return;
   }
 
@@ -34,7 +34,7 @@ export async function trackEvent(
     "request" in options ? options.request.headers : options.headers;
 
   if (isDoNotTrackEnabled(headers) && !options.collectDnt) {
-    console.log("Do not track enabled, not tracking event");
+    // console.log("Do not track enabled, not tracking event");
     return;
   }
 
@@ -51,9 +51,9 @@ export async function trackEvent(
   }
 
   if (!isProduction()) {
-    console.log(
-      "Simple Analytics is disabled by default in development and preview environments, enable it by setting ENABLE_ANALYTICS_IN_DEV=1 in your environment",
-    );
+    // console.log(
+    //   "Simple Analytics is disabled by default in development and preview environments, enable it by setting ENABLE_ANALYTICS_IN_DEV=1 in your environment",
+    // );
     return;
   }
 
@@ -86,10 +86,10 @@ const PROXY_PATHS = /^\/(proxy\.js|auto-events\.js|simple\/.*)$/;
 type TrackPageviewOptions = TrackingOptions & ServerContext;
 
 export async function trackPageview(options: TrackPageviewOptions) {
-  const hostname = options.hostname ?? process.env.SIMPLE_ANALYTICS_HOSTNAME;
+  const hostname = options?.hostname ?? process.env.SIMPLE_ANALYTICS_HOSTNAME;
 
   if (!hostname) {
-    console.error("No hostname provided for Simple Analytics");
+    // console.error("No hostname provided for Simple Analytics");
     return;
   }
 
@@ -109,7 +109,7 @@ export async function trackPageview(options: TrackPageviewOptions) {
   }
 
   if (isDoNotTrackEnabled(headers) && !options.collectDnt) {
-    console.log("Do not track enabled, not tracking pageview");
+    // console.log("Do not track enabled, not tracking pageview");
     return;
   }
 
@@ -135,9 +135,9 @@ export async function trackPageview(options: TrackPageviewOptions) {
   }
 
   if (!isProduction()) {
-    console.log(
-      "Simple Analytics is disabled by default in development and preview environments, enable it by setting ENABLE_ANALYTICS_IN_DEV=1 in your environment",
-    );
+    // console.log(
+    //   "Simple Analytics is disabled by default in development and preview environments, enable it by setting ENABLE_ANALYTICS_IN_DEV=1 in your environment",
+    // );
     return;
   }
 

From 5a7b1064331268de73e9c95f3c54b58af129fcdd Mon Sep 17 00:00:00 2001
From: "Jean P.D. Meijer" <jean@analoginterface.io>
Date: Fri, 5 Sep 2025 15:04:28 +0200
Subject: [PATCH 3/3] chore: clean up and add warning

---
 packages/analytics/src/client/utils.ts        |  9 -------
 .../analytics/src/server/simple-analytics.ts  | 24 ++++++++-----------
 2 files changed, 10 insertions(+), 23 deletions(-)

diff --git a/packages/analytics/src/client/utils.ts b/packages/analytics/src/client/utils.ts
index b808e62..1b89ac2 100644
--- a/packages/analytics/src/client/utils.ts
+++ b/packages/analytics/src/client/utils.ts
@@ -17,20 +17,11 @@ export function isProduction() {
   return process.env.NEXT_PUBLIC_VERCEL_ENV === "production";
 }
 
-function isBuildTime() {
-  return process.env.NEXT_PHASE === PHASE_PRODUCTION_BUILD;
-}
-
 export function parseDataProps(settings?: SimpleAnalyticsProps) {
   if (!isProduction()) {
     return {};
   }
 
-  // if (!process.env.NEXT_PUBLIC_SIMPLE_ANALYTICS_HOSTNAME) {
-  //   console.log("No hostname provided for Simple Analytics");
-  //   return {};
-  // }
-
   const metrics = settings?.ignoreMetrics
     ? Object.entries(settings.ignoreMetrics)
         .filter(([_, value]) => value)
diff --git a/packages/analytics/src/server/simple-analytics.ts b/packages/analytics/src/server/simple-analytics.ts
index 6a13326..52e0779 100644
--- a/packages/analytics/src/server/simple-analytics.ts
+++ b/packages/analytics/src/server/simple-analytics.ts
@@ -26,7 +26,6 @@ export async function trackEvent(
   const hostname = options?.hostname ?? process.env.SIMPLE_ANALYTICS_HOSTNAME;
 
   if (!hostname) {
-    // console.error("No hostname provided for Simple Analytics");
     return;
   }
 
@@ -34,7 +33,6 @@ export async function trackEvent(
     "request" in options ? options.request.headers : options.headers;
 
   if (isDoNotTrackEnabled(headers) && !options.collectDnt) {
-    // console.log("Do not track enabled, not tracking event");
     return;
   }
 
@@ -46,14 +44,13 @@ export async function trackEvent(
     ...parseHeaders(headers, options.ignoreMetrics),
   };
 
-  if (isBuildTime()) {
+  if (isBuildTime() || !isProduction()) {
     return;
   }
 
-  if (!isProduction()) {
-    // console.log(
-    //   "Simple Analytics is disabled by default in development and preview environments, enable it by setting ENABLE_ANALYTICS_IN_DEV=1 in your environment",
-    // );
+  // Only show a warning in production.
+  if (!hostname) {
+    console.log("No hostname provided for Simple Analytics, event not tracked");
     return;
   }
 
@@ -89,7 +86,6 @@ export async function trackPageview(options: TrackPageviewOptions) {
   const hostname = options?.hostname ?? process.env.SIMPLE_ANALYTICS_HOSTNAME;
 
   if (!hostname) {
-    // console.error("No hostname provided for Simple Analytics");
     return;
   }
 
@@ -109,7 +105,6 @@ export async function trackPageview(options: TrackPageviewOptions) {
   }
 
   if (isDoNotTrackEnabled(headers) && !options.collectDnt) {
-    // console.log("Do not track enabled, not tracking pageview");
     return;
   }
 
@@ -130,14 +125,15 @@ export async function trackPageview(options: TrackPageviewOptions) {
       : {}),
   };
 
-  if (isBuildTime()) {
+  if (isBuildTime() || !isProduction()) {
     return;
   }
 
-  if (!isProduction()) {
-    // console.log(
-    //   "Simple Analytics is disabled by default in development and preview environments, enable it by setting ENABLE_ANALYTICS_IN_DEV=1 in your environment",
-    // );
+  // Only show a warning in production.
+  if (!hostname) {
+    console.log(
+      "No hostname provided for Simple Analytics, pageview not tracked",
+    );
     return;
   }