posthog-node 4.2.2 → 4.2.3

package/lib/posthog-core/src/index.d.ts

@@ -6,7 +6,7 @@ import { SimpleEventEmitter } from './eventemitter';
 export declare abstract class PostHogCoreStateless {
     readonly apiKey: string;
     readonly host: string;
-    private flushAt;
+    readonly flushAt: number;
     private maxBatchSize;
     private maxQueueSize;
     private flushInterval;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "posthog-node",
-  "version": "4.2.2",
+  "version": "4.2.3",
   "description": "PostHog Node.js integration",
   "repository": {
     "type": "git",

package/src/posthog-node.ts

@@ -174,12 +174,19 @@ export class PostHog extends PostHogCoreStateless implements PostHogNodeV1 {
 
   identify({ distinctId, properties, disableGeoip }: IdentifyMessage): void {
     // Catch properties passed as $set and move them to the top level
-    const personProperties = properties?.$set || properties
+
+    // promote $set and $set_once to top level
+    const userPropsOnce = properties?.$set_once
+    delete properties?.$set_once
+
+    // if no $set is provided we assume all properties are $set
+    const userProps = properties?.$set || properties
 
     super.identifyStateless(
       distinctId,
       {
-        $set: personProperties,
+        $set: userProps,
+        $set_once: userPropsOnce,
       },
       { disableGeoip }
     )

package/test/posthog-node.spec.ts

@@ -154,9 +154,9 @@ describe('PostHog Node.js', () => {
       ])
     })
 
-    it('should handle identify mistakenly using $set', async () => {
+    it('should handle identify using $set and $set_once', async () => {
       expect(mockedFetch).toHaveBeenCalledTimes(0)
-      posthog.identify({ distinctId: '123', properties: { foo: 'bar', $set: { foo: 'other' } } })
+      posthog.identify({ distinctId: '123', properties: { $set: { foo: 'bar' }, $set_once: { vip: true } } })
       jest.runOnlyPendingTimers()
       await waitForPromises()
       const batchEvents = getLastBatchEvents()
@@ -166,7 +166,33 @@ describe('PostHog Node.js', () => {
           event: '$identify',
           properties: {
             $set: {
-              foo: 'other',
+              foo: 'bar',
+            },
+            $set_once: {
+              vip: true,
+            },
+            $geoip_disable: true,
+          },
+        },
+      ])
+    })
+
+    it('should handle identify using $set_once', async () => {
+      expect(mockedFetch).toHaveBeenCalledTimes(0)
+      posthog.identify({ distinctId: '123', properties: { foo: 'bar', $set_once: { vip: true } } })
+      jest.runOnlyPendingTimers()
+      await waitForPromises()
+      const batchEvents = getLastBatchEvents()
+      expect(batchEvents).toMatchObject([
+        {
+          distinct_id: '123',
+          event: '$identify',
+          properties: {
+            $set: {
+              foo: 'bar',
+            },
+            $set_once: {
+              vip: true,
             },
             $geoip_disable: true,
           },