feat: add bot detection to avoid segment tracking (#958)

* feat: add bot detection to avoid segment tracking

* fix: always trigger callback after segment or track

---------

Co-authored-by: Braian Mellor <braianj@gmail.com>

package.json

@@ -148,6 +148,7 @@
     "html-escaper": "^3.0.3",
     "htmlmin": "0.0.7",
     "http-proxy-middleware": "^2.0.6",
+    "isbot": "^5.1.22",
     "juice": "^8.0.0",
     "lodash": "^4.17.21",
     "mime": "^3.0.0",

src/utils/development/segment.ts

@@ -1,3 +1,5 @@
+import { isbot } from 'isbot'
+
 import { getMouseEventData, getMouseEventName, isMeta } from '../dom/events'
 import { isBlankTarget, isLocalLink } from '../dom/links'
 import once from '../function/once'
@@ -42,7 +44,11 @@ const getContext = once((): TrackContext => {
 })
 
 export default function segment(tracker: Tracker, callback?: () => void) {
-  if (typeof window !== 'undefined' && window.analytics) {
+  const userAgent = window.navigator.userAgent
+
+  const isBot = isbot(userAgent)
+
+  if (typeof window !== 'undefined' && window.analytics && !isBot) {
     tracker(window.analytics, getContext(), callback ?? emptyCallback)
   } else if (callback) {
     Promise.resolve().then(() => callback())
@@ -54,7 +60,11 @@ export function track(
   data: Record<string, any> = {},
   callback?: () => void
 ) {
-  if (typeof window !== 'undefined' && window.analytics) {
+  const userAgent = window.navigator.userAgent
+
+  const isBot = isbot(userAgent)
+
+  if (typeof window !== 'undefined' && window.analytics && !isBot) {
     const analytics = window.analytics
     analytics.track(event, { ...getContext(), ...data }, callback)
   } else if (callback) {