@output.ai/core 0.3.7 → 0.3.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@output.ai/core",
-  "version": "0.3.7",
+  "version": "0.3.8",
   "description": "The core module of the output framework",
   "type": "module",
   "exports": {

package/src/consts.js

@@ -10,7 +10,3 @@ export const ComponentType = {
   INTERNAL_STEP: 'internal_step',
   STEP: 'step'
 };
-
-const DEFAULT_HEARTBEAT_INTERVAL_MS = 2 * 60 * 1000; // 2 minutes
-export const HEARTBEAT_INTERVAL_MS = Number( process.env.OUTPUT_HEARTBEAT_INTERVAL_MS ) || DEFAULT_HEARTBEAT_INTERVAL_MS;
-export const HEARTBEAT_ENABLED = process.env.OUTPUT_HEARTBEAT_ENABLED !== 'false'; // on by default, set to 'false' to disable

package/src/worker/interceptors/activity.js

@@ -2,7 +2,14 @@ import { Context } from '@temporalio/activity';
 import { Storage } from '#async_storage';
 import { addEventStart, addEventEnd, addEventError } from '#tracing';
 import { headersToObject } from '../sandboxed_utils.js';
-import { METADATA_ACCESS_SYMBOL, HEARTBEAT_INTERVAL_MS, HEARTBEAT_ENABLED } from '#consts';
+import { METADATA_ACCESS_SYMBOL } from '#consts';
+
+// Heartbeat config lives here (not in #consts) because consts.js is bundled into
+// the Temporal workflow sandbox where `process` is not available.
+// Activity interceptors run in the worker context where process.env is safe.
+const DEFAULT_HEARTBEAT_INTERVAL_MS = 2 * 60 * 1000; // 2 minutes
+const HEARTBEAT_INTERVAL_MS = Number( process.env.OUTPUT_HEARTBEAT_INTERVAL_MS ) || DEFAULT_HEARTBEAT_INTERVAL_MS;
+const HEARTBEAT_ENABLED = process.env.OUTPUT_HEARTBEAT_ENABLED !== 'false'; // on by default, set to 'false' to disable
 
 /*
   This interceptor wraps every activity execution with cross-cutting concerns:

package/src/worker/interceptors/activity.spec.js

@@ -37,17 +37,9 @@ vi.mock( '../sandboxed_utils.js', () => ( {
   headersToObject: () => ( { executionContext: { workflowId: 'wf-1' } } )
 } ) );
 
-const mockConfig = { heartbeatEnabled: true, heartbeatIntervalMs: 50 };
-
 vi.mock( '#consts', () => ( {
   get METADATA_ACCESS_SYMBOL() {
     return METADATA_ACCESS_SYMBOL;
-  },
-  get HEARTBEAT_ENABLED() {
-    return mockConfig.heartbeatEnabled;
-  },
-  get HEARTBEAT_INTERVAL_MS() {
-    return mockConfig.heartbeatIntervalMs;
   }
 } ) );
 
@@ -64,12 +56,15 @@ describe( 'ActivityExecutionInterceptor', () => {
   beforeEach( () => {
     vi.clearAllMocks();
     vi.useFakeTimers();
-    mockConfig.heartbeatEnabled = true;
-    mockConfig.heartbeatIntervalMs = 50;
+    vi.resetModules();
+    // Default: heartbeat enabled with 50ms interval for fast tests
+    vi.stubEnv( 'OUTPUT_HEARTBEAT_ENABLED', 'true' );
+    vi.stubEnv( 'OUTPUT_HEARTBEAT_INTERVAL_MS', '50' );
   } );
 
   afterEach( () => {
     vi.useRealTimers();
+    vi.unstubAllEnvs();
   } );
 
   it( 'records trace start and end events on successful execution', async () => {
@@ -158,7 +153,7 @@ describe( 'ActivityExecutionInterceptor', () => {
   } );
 
   it( 'does not heartbeat when HEARTBEAT_ENABLED is false', async () => {
-    mockConfig.heartbeatEnabled = false;
+    vi.stubEnv( 'OUTPUT_HEARTBEAT_ENABLED', 'false' );
     const { ActivityExecutionInterceptor } = await import( './activity.js' );
     const interceptor = new ActivityExecutionInterceptor( makeActivities() );