Merge in Q2: Remove undocumented AnalyticsNode from @segment/analytics-next

.changeset/plenty-peaches-roll.md

@@ -0,0 +1,6 @@
+---
+'@segment/analytics-next': major
+---
+
+Remove experimental/undocumented AnalyticsNode library from browser. @segment/analytics-node should be used.
+Should also fix https://github.com/segmentio/analytics-next/issues/837.

packages/browser/package.json

@@ -10,12 +10,6 @@
   "main": "./dist/cjs/index.js",
   "module": "./dist/pkg/index.js",
   "types": "./dist/types/index.d.ts",
-  "browser": {
-    "./dist/cjs/node": "./dist/cjs/node/node.browser.js",
-    "./dist/cjs/node.js": "./dist/cjs/node/node.browser.js",
-    "./dist/pkg/node": "./dist/pkg/node/node.browser.js",
-    "./dist/pkg/node.js": "./dist/pkg/node/node.browser.js"
-  },
   "files": [
     "dist/",
     "src/",
@@ -54,13 +48,13 @@
     "@segment/tsub": "1.0.1",
     "dset": "^3.1.2",
     "js-cookie": "3.0.1",
-    "node-fetch": "^2.6.7",
     "spark-md5": "^3.0.1",
     "tslib": "^2.4.1",
     "unfetch": "^4.1.0"
   },
   "devDependencies": {
     "@internal/config": "0.0.0",
+    "@segment/analytics-node": "*",
     "@segment/analytics.js-integration": "^3.3.3",
     "@segment/analytics.js-integration-amplitude": "^3.3.3",
     "@size-limit/preset-big-lib": "^7.0.8",
@@ -88,6 +82,7 @@
     "log-update": "^4.0.0",
     "micro-memoize": "^4.0.9",
     "mime": "^2.4.6",
+    "node-fetch": "^2.6.7",
     "node-gyp": "^9.0.0",
     "playwright": "^1.28.1",
     "serve-handler": "^6.1.3",

packages/browser/qa/lib/stats.ts

@@ -1,51 +1,44 @@
-import { Analytics } from '../../src/index'
-import { Context } from '../../src/core/context'
-import { AnalyticsNode } from '../../src/node'
+import { Analytics, Context } from '@segment/analytics-node'
 import ex from 'execa'
 
-let analytics!: Analytics
-
-
 const getBranch = async () =>
   (await ex('git', ['branch', '--show-current'])).stdout
 
-async function client(): Promise<Analytics> {
-
+function client(): Analytics {
   if (!process.env.STATS_WRITEKEY) {
     throw new Error('no process.env.STATS_WRITEKEY')
   }
 
-  if (analytics) {
-    return analytics
-  }
-
-  const [nodeAnalytics] = await AnalyticsNode.load({
+  const analytics = new Analytics({
     writeKey: process.env.STATS_WRITEKEY,
+    maxEventsInBatch: 1,
   })
 
-  analytics = nodeAnalytics
   return analytics
 }
 
 export async function gauge(
   metric: string,
   value: number = 0,
   tags: string[] = []
-): Promise<Context> {
-  const ajs = await client()
+) {
+  const ajs = client()
 
   const branch = process.env.BUILDKITE_BRANCH || (await getBranch())
 
-  const ctx = await ajs.track(
-    metric,
-    {
-      value,
-      tags: [...tags, `branch:${branch}`],
-      type: 'gauge',
-    },
-    {
-      userId: 'system',
-    }
+  const ctx = await new Promise<Context>((resolve, reject) =>
+    ajs.track(
+      {
+        event: metric,
+        properties: {
+          value,
+          tags: [...tags, `branch:${branch}`],
+          type: 'gauge',
+        },
+        userId: 'system',
+      },
+      (err, res) => (err ? reject(err) : resolve(res!))
+    )
   )
 
   return ctx

packages/browser/src/index.ts

@@ -1,6 +1,5 @@
 export * from './core/analytics'
 export * from './browser'
-export * from './node'
 
 export * from './core/context'
 export * from './core/events'

yarn.lock

@@ -1733,6 +1733,7 @@ __metadata:
     "@internal/config": 0.0.0
     "@lukeed/uuid": ^2.0.0
     "@segment/analytics-core": 1.2.4
+    "@segment/analytics-node": "*"
     "@segment/analytics.js-integration": ^3.3.3
     "@segment/analytics.js-integration-amplitude": ^3.3.3
     "@segment/analytics.js-video-plugins": ^0.2.1