@mondaydotcomorg/z2h-shared-utils 0.3.3 → 0.3.4

package/dist/esm/observability/__tests__/tracker.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=tracker.test.d.ts.map

package/dist/esm/observability/__tests__/tracker.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"tracker.test.d.ts","sourceRoot":"","sources":["../../../../src/observability/__tests__/tracker.test.ts"],"names":[],"mappings":""}

package/dist/esm/observability/tracker.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"tracker.d.ts","sourceRoot":"","sources":["../../../src/observability/tracker.ts"],"names":[],"mappings":"AAWA;;;;GAIG;AACH,wBAAgB,UAAU,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAAE,MAAM,EAAE,MAAM,GAAG,IAAI,CAU5F"}
+{"version":3,"file":"tracker.d.ts","sourceRoot":"","sources":["../../../src/observability/tracker.ts"],"names":[],"mappings":"AAWA;;;;GAIG;AACH,wBAAgB,UAAU,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAAE,MAAM,EAAE,MAAM,GAAG,IAAI,CAW5F"}

package/dist/esm/observability/tracker.mjs

@@ -13,13 +13,14 @@ function getUserContext() {
  * standalone processes (e.g. CLIs) that never bootstrap the Trident runtime.
  */
 function trackEvent(name, data, source) {
-    const sendBiEvent = Api.getPart('bigbrain')?.tracker?.sendBiEvent;
-    if (!sendBiEvent) {
+    const tracker = Api.getPart('bigbrain')?.tracker;
+    if (!tracker) {
         // eslint-disable-next-line no-console
         console.warn('[z2h-shared-utils] trackEvent: bigbrain API part unavailable, was Runtime.init() called? Dropping event.');
         return;
     }
-    sendBiEvent({ name, source, data: { ...getUserContext(), ...data } });
+    // Call through `tracker`, not a destructured reference — sendBiEvent relies on `this` internally.
+    tracker.sendBiEvent({ name, source, data: { ...getUserContext(), ...data } });
 }
 
 export { trackEvent };

package/dist/observability/__tests__/tracker.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=tracker.test.d.ts.map

package/dist/observability/__tests__/tracker.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"tracker.test.d.ts","sourceRoot":"","sources":["../../../src/observability/__tests__/tracker.test.ts"],"names":[],"mappings":""}

package/dist/observability/tracker.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"tracker.d.ts","sourceRoot":"","sources":["../../src/observability/tracker.ts"],"names":[],"mappings":"AAWA;;;;GAIG;AACH,wBAAgB,UAAU,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAAE,MAAM,EAAE,MAAM,GAAG,IAAI,CAU5F"}
+{"version":3,"file":"tracker.d.ts","sourceRoot":"","sources":["../../src/observability/tracker.ts"],"names":[],"mappings":"AAWA;;;;GAIG;AACH,wBAAgB,UAAU,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAAE,MAAM,EAAE,MAAM,GAAG,IAAI,CAW5F"}

package/dist/observability/tracker.js

@@ -15,13 +15,14 @@ function getUserContext() {
  * standalone processes (e.g. CLIs) that never bootstrap the Trident runtime.
  */
 function trackEvent(name, data, source) {
-    const sendBiEvent = tridentBackendApi.Api.getPart('bigbrain')?.tracker?.sendBiEvent;
-    if (!sendBiEvent) {
+    const tracker = tridentBackendApi.Api.getPart('bigbrain')?.tracker;
+    if (!tracker) {
         // eslint-disable-next-line no-console
         console.warn('[z2h-shared-utils] trackEvent: bigbrain API part unavailable, was Runtime.init() called? Dropping event.');
         return;
     }
-    sendBiEvent({ name, source, data: { ...getUserContext(), ...data } });
+    // Call through `tracker`, not a destructured reference — sendBiEvent relies on `this` internally.
+    tracker.sendBiEvent({ name, source, data: { ...getUserContext(), ...data } });
 }
 
 exports.trackEvent = trackEvent;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mondaydotcomorg/z2h-shared-utils",
-  "version": "0.3.3",
+  "version": "0.3.4",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
   "exports": {
@@ -29,6 +29,7 @@
     ]
   },
   "scripts": {
+    "test": "trident-library test",
     "build": "trident-library build",
     "watch": "trident-library build -w",
     "lint": "trident-library lint"
@@ -50,7 +51,8 @@
     "@mondaydotcomorg/trident-backend-api": "^0.24.3",
     "@mondaydotcomorg/trident-library": "^1.2.7",
     "@types/node": "^22.19.0",
-    "typescript": "^5.7.2"
+    "typescript": "^5.7.2",
+    "vitest": "^2.1.8"
   },
   "engines": {
     "node": ">=24"

package/src/observability/__tests__/tracker.test.ts

@@ -0,0 +1,71 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import { trackEvent } from '../tracker';
+
+const { getPartMock } = vi.hoisted(() => ({ getPartMock: vi.fn() }));
+vi.mock('@mondaydotcomorg/trident-backend-api', () => ({ Api: { getPart: getPartMock } }));
+
+/**
+ * Mirrors @mondaydotcomorg/bigbrain-tracker-node: sendBiEvent is a prototype method that
+ * delegates to this.sendBiEventAsync. A plain-object double would pass even when trackEvent
+ * detaches the method from its tracker, which is how that bug reached production —
+ * it only surfaces once Runtime.init() has supplied a real bigbrain part.
+ */
+class FakeTracker {
+  readonly sent: { name: string; source: string; data: Record<string, unknown> }[] = [];
+
+  async sendBiEventAsync(event: { name: string; source: string; data: Record<string, unknown> }): Promise<void> {
+    this.sent.push(event);
+  }
+
+  sendBiEvent(event: { name: string; source: string; data: Record<string, unknown> }): void {
+    void this.sendBiEventAsync(event);
+  }
+}
+
+function mockParts({ tracker, user }: { tracker?: FakeTracker; user?: { name: string } }) {
+  getPartMock.mockImplementation((part: string) => {
+    if (part === 'bigbrain') {
+      return tracker ? { tracker } : undefined;
+    }
+    if (part === 'context') {
+      return { execution: { get: () => (user ? { user } : undefined) } };
+    }
+    return undefined;
+  });
+}
+
+describe('trackEvent', () => {
+  beforeEach(() => {
+    getPartMock.mockReset();
+  });
+
+  it('sends the event through the tracker', () => {
+    const tracker = new FakeTracker();
+    mockParts({ tracker });
+
+    trackEvent('z2h_agent_tool_call', { tool_name: 'write_file' }, 'z2h-agent');
+
+    expect(tracker.sent).toEqual([
+      { name: 'z2h_agent_tool_call', source: 'z2h-agent', data: { tool_name: 'write_file' } },
+    ]);
+  });
+
+  it('attaches the current user to the event', () => {
+    const tracker = new FakeTracker();
+    mockParts({ tracker, user: { name: 'Arielle' } });
+
+    trackEvent('z2h_agent_tool_call', { tool_name: 'execute' }, 'z2h-agent');
+
+    expect(tracker.sent[0].data).toEqual({ user_name: 'Arielle', tool_name: 'execute' });
+  });
+
+  it('drops the event with a warning when the bigbrain part is unavailable', () => {
+    mockParts({});
+    const warn = vi.spyOn(console, 'warn').mockImplementation(() => {});
+
+    expect(() => trackEvent('z2h_agent_tool_call', { tool_name: 'task' }, 'z2h-agent')).not.toThrow();
+
+    expect(warn).toHaveBeenCalledOnce();
+    warn.mockRestore();
+  });
+});

package/src/observability/tracker.ts

@@ -15,13 +15,14 @@ function getUserContext(): { user_name?: string } {
  * standalone processes (e.g. CLIs) that never bootstrap the Trident runtime.
  */
 export function trackEvent(name: string, data: Record<string, unknown>, source: string): void {
-  const sendBiEvent = Api.getPart('bigbrain')?.tracker?.sendBiEvent;
-  if (!sendBiEvent) {
+  const tracker = Api.getPart('bigbrain')?.tracker;
+  if (!tracker) {
     // eslint-disable-next-line no-console
     console.warn(
       '[z2h-shared-utils] trackEvent: bigbrain API part unavailable, was Runtime.init() called? Dropping event.'
     );
     return;
   }
-  sendBiEvent({ name, source, data: { ...getUserContext(), ...data } });
+  // Call through `tracker`, not a destructured reference — sendBiEvent relies on `this` internally.
+  tracker.sendBiEvent({ name, source, data: { ...getUserContext(), ...data } });
 }