@pbvision/cloud-run-service 0.0.31 → 0.0.32

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pbvision/cloud-run-service",
-  "version": "0.0.31",
+  "version": "0.0.32",
   "description": "fastify-firestore-service Web Framework on Cloud Run",
   "license": "Apache-2.0",
   "type": "module",

package/src/analytics.js

@@ -1,5 +1,5 @@
 import assert from 'node:assert'
-import { randomUUID } from 'node:crypto'
+import crypto, { randomUUID } from 'node:crypto'
 
 import { DatabaseAPI, EXCEPTIONS } from '@pbvision/fastify-firestore-service'
 import UAParser from 'ua-parser-js'
@@ -29,14 +29,21 @@ export class DatabaseAPIWithAnalytics extends DatabaseAPI {
     return super.postCommit(respData)
   }
 
-  logAnalyticsEvent (mixpanelUserId, eventName, inputProperties, deviceId = null) {
+  logAnalyticsEvent (mixpanelUserId, eventName, inputProperties = {}, deviceId = null, insertId = null) {
+    if (insertId) {
+      // insert id must be <= 36 chars & only have alphanumeric & hyphen chars
+      insertId = crypto.createHash('md5').update(insertId).digest('hex')
+    } else {
+      insertId = randomUUID()
+    }
+
     this.__analyticsEvents.push({
       event: eventName,
       properties: addSenderId(mixpanelUserId, {
         ...inputProperties,
         token: mixpanelToken,
         time: new Date().getTime(),
-        $insert_id: randomUUID(),
+        $insert_id: insertId,
         ip: this.req.ip
       }, deviceId)
     })

package/test/unit-test-analytics.js

@@ -1,3 +1,5 @@
+import crypto from 'node:crypto'
+
 import { mixpanelToken } from '../src/analytics.js'
 
 import { AppTest, runTests } from './base-test.js'
@@ -139,6 +141,47 @@ class AnalyticsTest extends AppTest {
     }])
   }
 
+  async testCustomInsertionId () {
+    await this.app.post('/analytics')
+      .send({
+        eventCalls: [
+          ['some uid', 'some event', { cool: 1, hi: 'world' }, null, 'xx']
+        ]
+      }).expect(200)
+    const calls = this.fetchMock.mock.calls
+    expect(calls.length).toBe(1)
+    const actualBody = JSON.parse(calls[0][1].body)
+    expect(actualBody).toEqual([{
+      event: 'some event',
+      properties: expect.objectContaining({
+        cool: 1,
+        hi: 'world',
+        $user_id: 'some uid',
+        token: mixpanelToken,
+        $insert_id: crypto.createHash('md5').update('xx').digest('hex')
+      })
+    }])
+  }
+
+  async testOmittingEventProperties () {
+    await this.app.post('/analytics')
+      .send({
+        eventCalls: [
+          ['some uid', 'some event']
+        ]
+      }).expect(200)
+    const calls = this.fetchMock.mock.calls
+    expect(calls.length).toBe(1)
+    const actualBody = JSON.parse(calls[0][1].body)
+    expect(actualBody).toEqual([{
+      event: 'some event',
+      properties: expect.objectContaining({
+        $user_id: 'some uid',
+        token: mixpanelToken
+      })
+    }])
+  }
+
   async testUserAgent () {
     await this.sendBasicEvent('Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/122.0.0.0 Safari/537.36')
     const calls = this.fetchMock.mock.calls