@outputai/core 0.14.0 → 0.14.1-next.51e8690.0

package/src/temporal/client.spec.js

@@ -1,124 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest';
-
-const loadMock = vi.fn();
-vi.mock( '#async_storage', () => ( {
-  Storage: { load: loadMock }
-} ) );
-
-const getHandleMock = vi.fn();
-const clientConstructorMock = vi.fn( function () {
-  this.workflow = { getHandle: getHandleMock };
-} );
-vi.mock( '@temporalio/client', () => ( {
-  Client: clientConstructorMock
-} ) );
-
-const connection = {
-  fake: 'native-connection',
-  close: vi.fn().mockResolvedValue( undefined )
-};
-const activityContext = {
-  activityInfo: {
-    workflowExecution: {
-      workflowId: 'workflow-1',
-      runId: 'run-1'
-    }
-  }
-};
-
-describe( 'Temporal client', () => {
-  beforeEach( () => {
-    vi.clearAllMocks();
-    vi.resetModules();
-  } );
-
-  describe( 'createTemporalClient', () => {
-    it( 'throws a FatalError outside the worker runtime', async () => {
-      const { createTemporalClient } = await import( './client.js' );
-      const { FatalError } = await import( '#errors' );
-
-      expect( () => createTemporalClient() ).toThrow( FatalError );
-    } );
-
-    it( 'creates a client with the worker connection and namespace', async () => {
-      const { createTemporalClient, setupClientConfig } = await import( './client.js' );
-      setupClientConfig( { connection, namespace: 'namespace-1' } );
-
-      const client = createTemporalClient();
-
-      expect( clientConstructorMock ).toHaveBeenCalledWith( {
-        connection: expect.anything(),
-        namespace: 'namespace-1'
-      } );
-      expect( clientConstructorMock.mock.calls[0][0].connection ).not.toBe( connection );
-      expect( client ).toBeInstanceOf( clientConstructorMock );
-    } );
-
-    it( 'delegates connection access but prevents closing the worker connection', async () => {
-      const { createTemporalClient, setupClientConfig } = await import( './client.js' );
-      const { FatalError } = await import( '#errors' );
-      setupClientConfig( { connection, namespace: 'namespace-1' } );
-
-      createTemporalClient();
-      const clientConnection = clientConstructorMock.mock.calls[0][0].connection;
-
-      expect( clientConnection.fake ).toBe( 'native-connection' );
-      await expect( clientConnection.close() ).rejects.toThrow( FatalError );
-      expect( connection.close ).not.toHaveBeenCalled();
-    } );
-
-    it( 'returns a new client on each call', async () => {
-      const { createTemporalClient, setupClientConfig } = await import( './client.js' );
-      setupClientConfig( { connection, namespace: 'namespace-1' } );
-
-      expect( createTemporalClient() ).not.toBe( createTemporalClient() );
-      expect( clientConstructorMock ).toHaveBeenCalledTimes( 2 );
-    } );
-  } );
-
-  describe( 'getCurrentWorkflowHandle', () => {
-    it( 'throws a FatalError outside a Temporal activity context', async () => {
-      loadMock.mockReturnValue( undefined );
-      const { getCurrentWorkflowHandle, setupClientConfig } = await import( './client.js' );
-      const { FatalError } = await import( '#errors' );
-      setupClientConfig( { connection, namespace: 'namespace-1' } );
-
-      expect( () => getCurrentWorkflowHandle() ).toThrow( FatalError );
-    } );
-
-    it( 'throws a FatalError outside the worker runtime', async () => {
-      loadMock.mockReturnValue( activityContext );
-      const { getCurrentWorkflowHandle } = await import( './client.js' );
-      const { FatalError } = await import( '#errors' );
-
-      expect( () => getCurrentWorkflowHandle() ).toThrow( FatalError );
-    } );
-
-    it( 'returns the handle for the current workflow execution', async () => {
-      const handle = { signal: vi.fn() };
-      loadMock.mockReturnValue( activityContext );
-      getHandleMock.mockReturnValue( handle );
-      const { getCurrentWorkflowHandle, setupClientConfig } = await import( './client.js' );
-      setupClientConfig( { connection, namespace: 'namespace-1' } );
-
-      expect( getCurrentWorkflowHandle() ).toBe( handle );
-      expect( clientConstructorMock ).toHaveBeenCalledWith( {
-        connection: expect.anything(),
-        namespace: 'namespace-1'
-      } );
-      expect( getHandleMock ).toHaveBeenCalledWith( 'workflow-1', 'run-1' );
-    } );
-
-    it( 'reuses its client across calls', async () => {
-      loadMock.mockReturnValue( activityContext );
-      const { getCurrentWorkflowHandle, setupClientConfig } = await import( './client.js' );
-      setupClientConfig( { connection, namespace: 'namespace-1' } );
-
-      getCurrentWorkflowHandle();
-      getCurrentWorkflowHandle();
-
-      expect( clientConstructorMock ).toHaveBeenCalledTimes( 1 );
-      expect( getHandleMock ).toHaveBeenCalledTimes( 2 );
-    } );
-  } );
-} );

package/src/tracing/processors/local/index.spec.js

@@ -1,204 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-
-// In-memory fs mock store
-const store = { files: new Map() };
-const mkdirSyncMock = vi.fn();
-const writeFileSyncMock = vi.fn();
-const appendFileSyncMock = vi.fn( ( path, data ) => {
-  const prev = store.files.get( path ) ?? '';
-  store.files.set( path, prev + data );
-} );
-const readFileSyncMock = vi.fn( path => store.files.get( path ) ?? '' );
-const readdirSyncMock = vi.fn( () => [] );
-const rmSyncMock = vi.fn();
-const createWriteStreamMock = vi.fn( path => ( { path } ) );
-
-vi.mock( 'node:fs', () => ( {
-  mkdirSync: mkdirSyncMock,
-  writeFileSync: writeFileSyncMock,
-  appendFileSync: appendFileSyncMock,
-  readFileSync: readFileSyncMock,
-  readdirSync: readdirSyncMock,
-  rmSync: rmSyncMock,
-  createWriteStream: createWriteStreamMock
-} ) );
-
-const pipelineMock = vi.fn( async ( source, destination ) => {
-  const chunks = [];
-  for await ( const chunk of source ) {
-    chunks.push( Buffer.isBuffer( chunk ) ? chunk : Buffer.from( chunk ) );
-  }
-  store.files.set( destination.path, Buffer.concat( chunks ).toString( 'utf8' ) );
-} );
-vi.mock( 'node:stream/promises', () => ( { pipeline: pipelineMock } ) );
-
-vi.mock( 'json-stream-stringify', async () => {
-  const { Readable } = await import( 'node:stream' );
-  return {
-    JsonStreamStringify: class extends Readable {
-      constructor( body ) {
-        super();
-        this.body = body;
-      }
-
-      _read() {
-        this.push( JSON.stringify( this.body ) );
-        this.push( null );
-      }
-    }
-  };
-} );
-
-const buildTraceTreeMock = vi.fn( entries => ( { count: entries.length } ) );
-vi.mock( '../../tools/build_trace_tree.js', () => ( { default: buildTraceTreeMock } ) );
-
-/** Flush happens when the root id matches runId and action is not 'start', or when action is 'error'. */
-const rootStart = ( runId, ts ) => ( { id: runId, action: 'start', timestamp: ts } );
-const rootEnd = ( runId, ts ) => ( { id: runId, action: 'end', timestamp: ts } );
-const childTick = ( id, ts ) => ( { id, action: 'tick', timestamp: ts } );
-
-describe( 'tracing/processors/local', () => {
-  const startTime = Date.parse( '2020-01-02T03:04:05.678Z' );
-  const traceInfo = {
-    workflowId: 'id1',
-    runId: 'run-1',
-    workflowType: 'WF',
-    startTime
-  };
-
-  beforeEach( () => {
-    vi.clearAllMocks();
-    store.files.clear();
-    process.argv[2] = '/tmp/project';
-    delete process.env.OUTPUT_TRACE_HOST_PATH;
-  } );
-
-  it( 'init(): creates temp dir and cleans up old files', async () => {
-    const { init } = await import( './index.js' );
-
-    const now = Date.now();
-    readdirSyncMock.mockReturnValue( [ `${now - ( 8 * 24 * 60 * 60 * 1000 )}_old.trace`, `${now}_new.trace` ] );
-
-    init();
-
-    expect( mkdirSyncMock ).toHaveBeenCalledWith( expect.stringMatching( /temp\/traces$/ ), { recursive: true } );
-    expect( rmSyncMock ).toHaveBeenCalledTimes( 1 );
-  } );
-
-  it( 'exec(): appends each entry and writes aggregated tree once on root workflow end', async () => {
-    const { exec, init } = await import( './index.js' );
-    init();
-
-    await exec( { traceInfo, entry: rootStart( traceInfo.runId, startTime ) } );
-    await exec( { traceInfo, entry: childTick( 'child-1', startTime + 1 ) } );
-    await exec( { traceInfo, entry: rootEnd( traceInfo.runId, startTime + 2 ) } );
-
-    expect( buildTraceTreeMock ).toHaveBeenCalledTimes( 1 );
-    expect( buildTraceTreeMock.mock.calls[0][0] ).toHaveLength( 3 );
-
-    expect( createWriteStreamMock ).toHaveBeenCalledTimes( 1 );
-    expect( pipelineMock ).toHaveBeenCalledTimes( 1 );
-    const [ writtenPath ] = createWriteStreamMock.mock.calls[0];
-    expect( writtenPath ).toMatch( /\/tmp\/project\/logs\/runs\/WF\// );
-    expect( JSON.parse( store.files.get( writtenPath ) ).count ).toBe( 3 );
-  } );
-
-  it( 'exec(): does not build or write on non-flush entries', async () => {
-    const { exec, init } = await import( './index.js' );
-    init();
-
-    await exec( { traceInfo, entry: rootStart( traceInfo.runId, startTime ) } );
-    await exec( { traceInfo, entry: childTick( 'child-1', startTime + 1 ) } );
-
-    expect( buildTraceTreeMock ).not.toHaveBeenCalled();
-    expect( writeFileSyncMock ).not.toHaveBeenCalled();
-    expect( createWriteStreamMock ).not.toHaveBeenCalled();
-    expect( pipelineMock ).not.toHaveBeenCalled();
-  } );
-
-  it( 'exec(): flushes on error action before root end', async () => {
-    const { exec, init } = await import( './index.js' );
-    init();
-
-    await exec( { traceInfo, entry: rootStart( traceInfo.runId, startTime ) } );
-    await exec( { traceInfo, entry: { id: 'step-1', action: 'error', timestamp: startTime + 1 } } );
-
-    expect( buildTraceTreeMock ).toHaveBeenCalledTimes( 1 );
-    expect( buildTraceTreeMock.mock.calls[0][0] ).toHaveLength( 2 );
-    expect( createWriteStreamMock ).toHaveBeenCalledTimes( 1 );
-    expect( pipelineMock ).toHaveBeenCalledTimes( 1 );
-  } );
-
-  it( 'getDestination(): returns absolute path under callerDir logs', async () => {
-    const { getDestination } = await import( './index.js' );
-
-    const destination = getDestination( {
-      ...traceInfo,
-      workflowId: 'workflow-id-123',
-      workflowType: 'test-workflow'
-    } );
-
-    expect( destination ).toMatch( /^\/|^[A-Z]:\\/i );
-    expect( destination ).toBe(
-      '/tmp/project/logs/runs/test-workflow/2020-01-02-03-04-05-678Z_workflow-id-123.json'
-    );
-  } );
-
-  it( 'exec(): writes under process.argv[2] logs even when OUTPUT_TRACE_HOST_PATH is set', async () => {
-    const { exec, init } = await import( './index.js' );
-
-    process.env.OUTPUT_TRACE_HOST_PATH = '/host/path/logs';
-
-    init();
-
-    await exec( { traceInfo, entry: rootStart( traceInfo.runId, startTime ) } );
-    await exec( { traceInfo, entry: rootEnd( traceInfo.runId, startTime + 1 ) } );
-
-    expect( createWriteStreamMock ).toHaveBeenCalledTimes( 1 );
-    const [ writtenPath ] = createWriteStreamMock.mock.calls[0];
-
-    expect( writtenPath ).not.toContain( '/host/path/logs' );
-    expect( writtenPath ).toMatch( /\/tmp\/project\/logs\/runs\/WF\// );
-  } );
-
-  it( 'getDestination(): returns OUTPUT_TRACE_HOST_PATH when set', async () => {
-    const { getDestination } = await import( './index.js' );
-
-    process.env.OUTPUT_TRACE_HOST_PATH = '/host/path/logs';
-
-    const destination = getDestination( {
-      ...traceInfo,
-      workflowId: 'workflow-id-123',
-      workflowType: 'test-workflow'
-    } );
-
-    expect( destination ).toBe( '/host/path/logs/runs/test-workflow/2020-01-02-03-04-05-678Z_workflow-id-123.json' );
-  } );
-
-  it( 'separation of write and report paths works correctly', async () => {
-    const { exec, getDestination, init } = await import( './index.js' );
-
-    process.env.OUTPUT_TRACE_HOST_PATH = '/Users/ben/project/logs';
-
-    init();
-
-    const testTraceInfo = {
-      ...traceInfo,
-      workflowId: 'workflow-id-123',
-      runId: 'run-123',
-      workflowType: 'test-workflow'
-    };
-
-    await exec( { traceInfo: testTraceInfo, entry: rootStart( testTraceInfo.runId, startTime ) } );
-    await exec( { traceInfo: testTraceInfo, entry: rootEnd( testTraceInfo.runId, startTime + 1 ) } );
-
-    const destination = getDestination( testTraceInfo );
-
-    const [ writtenPath ] = createWriteStreamMock.mock.calls[0];
-    expect( writtenPath ).not.toContain( '/Users/ben/project' );
-    expect( writtenPath ).toMatch( /\/tmp\/project\/logs\/runs\/test-workflow\// );
-    expect( JSON.parse( store.files.get( writtenPath ) ).count ).toBe( 2 );
-
-    expect( destination ).toBe( '/Users/ben/project/logs/runs/test-workflow/2020-01-02-03-04-05-678Z_workflow-id-123.json' );
-  } );
-} );

package/src/tracing/processors/s3/configs.spec.js

@@ -1,64 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-
-async function loadModule() {
-  vi.resetModules();
-  return import( './configs.js' );
-}
-
-describe( 'tracing/processors/s3/configs', () => {
-  const required = {
-    OUTPUT_AWS_REGION: 'us-east-1',
-    OUTPUT_AWS_ACCESS_KEY_ID: 'id',
-    OUTPUT_AWS_SECRET_ACCESS_KEY: 'sek',
-    OUTPUT_TRACE_REMOTE_S3_BUCKET: 'bkt',
-    OUTPUT_REDIS_URL: 'redis://localhost:6379'
-  };
-
-  beforeEach( () => {
-    vi.stubEnv( 'OUTPUT_AWS_REGION', required.OUTPUT_AWS_REGION );
-    vi.stubEnv( 'OUTPUT_AWS_ACCESS_KEY_ID', required.OUTPUT_AWS_ACCESS_KEY_ID );
-    vi.stubEnv( 'OUTPUT_AWS_SECRET_ACCESS_KEY', required.OUTPUT_AWS_SECRET_ACCESS_KEY );
-    vi.stubEnv( 'OUTPUT_TRACE_REMOTE_S3_BUCKET', required.OUTPUT_TRACE_REMOTE_S3_BUCKET );
-    vi.stubEnv( 'OUTPUT_REDIS_URL', required.OUTPUT_REDIS_URL );
-  } );
-
-  afterEach( () => {
-    vi.unstubAllEnvs();
-  } );
-
-  it( 'loadEnv() throws when required env vars are missing', async () => {
-    vi.stubEnv( 'OUTPUT_REDIS_URL', undefined );
-    const { loadEnv } = await loadModule();
-    expect( () => loadEnv() ).toThrow( /OUTPUT_REDIS_URL/ );
-  } );
-
-  it( 'loadEnv() populates getVars() with parsed env', async () => {
-    const { loadEnv, getVars } = await loadModule();
-    loadEnv();
-    const vars = getVars();
-    expect( vars.awsRegion ).toBe( required.OUTPUT_AWS_REGION );
-    expect( vars.awsAccessKeyId ).toBe( required.OUTPUT_AWS_ACCESS_KEY_ID );
-    expect( vars.awsSecretAccessKey ).toBe( required.OUTPUT_AWS_SECRET_ACCESS_KEY );
-    expect( vars.remoteS3Bucket ).toBe( required.OUTPUT_TRACE_REMOTE_S3_BUCKET );
-    expect( vars.redisUrl ).toBe( required.OUTPUT_REDIS_URL );
-    expect( vars.redisIncompleteWorkflowsTTL ).toBe( 60 * 60 * 24 * 7 );
-  } );
-
-  it( 'loadEnv() uses OUTPUT_REDIS_TRACE_TTL when set', async () => {
-    vi.stubEnv( 'OUTPUT_REDIS_TRACE_TTL', '3600' );
-    const { loadEnv, getVars } = await loadModule();
-    loadEnv();
-    expect( getVars().redisIncompleteWorkflowsTTL ).toBe( 3600 );
-  } );
-
-  it( 'getVars() throws when loadEnv() was not called', async () => {
-    const { getVars } = await loadModule();
-    expect( () => getVars() ).toThrow( 'Env vars not loaded. Use loadEnv() first.' );
-  } );
-
-  it( 'loadEnv() throws when OUTPUT_REDIS_TRACE_TTL is invalid', async () => {
-    vi.stubEnv( 'OUTPUT_REDIS_TRACE_TTL', 'not-a-number' );
-    const { loadEnv } = await loadModule();
-    expect( () => loadEnv() ).toThrow();
-  } );
-} );

package/src/tracing/processors/s3/index.spec.js

@@ -1,185 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-
-const loadEnvMock = vi.fn();
-const getVarsMock = vi.fn( () => ( {
-  remoteS3Bucket: 'bkt',
-  redisIncompleteWorkflowsTTL: 3600
-} ) );
-vi.mock( './configs.js', () => ( { loadEnv: loadEnvMock, getVars: getVarsMock } ) );
-
-const redisMulti = {
-  zAdd: vi.fn().mockReturnThis(),
-  expire: vi.fn().mockReturnThis(),
-  exec: vi.fn()
-};
-const zRangeMock = vi.fn();
-const delMock = vi.fn().mockResolvedValue( undefined );
-const getRedisClientMock = vi.fn( async () => ( {
-  multi: () => redisMulti,
-  zRange: zRangeMock,
-  del: delMock
-} ) );
-vi.mock( './redis_client.js', () => ( { getRedisClient: getRedisClientMock } ) );
-
-const uploadMock = vi.fn();
-vi.mock( './s3_client.js', () => ( { upload: uploadMock } ) );
-
-const buildTraceTreeMock = vi.fn( entries => ( { count: entries.length } ) );
-vi.mock( '../../tools/build_trace_tree.js', () => ( { default: buildTraceTreeMock } ) );
-
-vi.mock( 'json-stream-stringify', async () => {
-  const { Readable } = await import( 'node:stream' );
-  return {
-    JsonStreamStringify: class extends Readable {
-      constructor( body ) {
-        super();
-        this.body = body;
-      }
-
-      _read() {
-        this.push( JSON.stringify( this.body ) );
-        this.push( null );
-      }
-    }
-  };
-} );
-
-const streamToString = async stream => {
-  const chunks = [];
-  for await ( const chunk of stream ) {
-    chunks.push( Buffer.isBuffer( chunk ) ? chunk : Buffer.from( chunk ) );
-  }
-  return Buffer.concat( chunks ).toString( 'utf8' );
-};
-
-describe( 'tracing/processors/s3', () => {
-  const startTime = Date.parse( '2020-01-02T03:04:05.678Z' );
-  const traceInfo = {
-    workflowId: 'id1',
-    runId: 'run-1',
-    workflowType: 'WF',
-    startTime
-  };
-
-  beforeEach( () => {
-    vi.useFakeTimers();
-    vi.clearAllMocks();
-    getVarsMock.mockReturnValue( { remoteS3Bucket: 'bkt', redisIncompleteWorkflowsTTL: 3600, traceUploadDelayMs: 10_000 } );
-  } );
-
-  afterEach( () => {
-    vi.useRealTimers();
-  } );
-
-  it( 'init(): loads config and ensures redis client is created', async () => {
-    const { init } = await import( './index.js' );
-    await init();
-    expect( loadEnvMock ).toHaveBeenCalledTimes( 1 );
-    expect( getRedisClientMock ).toHaveBeenCalledTimes( 1 );
-  } );
-
-  it( 'exec(): accumulates via redis, uploads only on root workflow end', async () => {
-    const { exec } = await import( './index.js' );
-
-    redisMulti.exec.mockResolvedValue( [] );
-
-    const workflowStart = { id: 'run-1', name: 'WF', kind: 'workflow', action: 'start', details: {}, timestamp: startTime };
-    const activityStart = {
-      id: 'act-1',
-      name: 'DoSomething',
-      kind: 'step',
-      parentId: 'run-1',
-      action: 'start',
-      details: {},
-      timestamp: startTime + 1
-    };
-    const workflowEnd = { id: 'run-1', action: 'end', details: { ok: true }, timestamp: startTime + 2 };
-    zRangeMock.mockResolvedValue( [
-      JSON.stringify( workflowStart ),
-      JSON.stringify( activityStart ),
-      JSON.stringify( workflowEnd )
-    ] );
-
-    await exec( { traceInfo, entry: workflowStart } );
-    await exec( { traceInfo, entry: activityStart } );
-    // Root end: id matches runId and not start — triggers the 10s delay before upload
-    const endPromise = exec( { traceInfo, entry: workflowEnd } );
-    await vi.advanceTimersByTimeAsync( 10_000 );
-    await endPromise;
-
-    expect( redisMulti.zAdd ).toHaveBeenCalledTimes( 3 );
-    expect( buildTraceTreeMock ).toHaveBeenCalledTimes( 1 );
-    expect( zRangeMock ).toHaveBeenCalledTimes( 1 );
-    expect( uploadMock ).toHaveBeenCalledTimes( 1 );
-    const { key, content } = uploadMock.mock.calls[0][0];
-    expect( key ).toMatch( /^WF\/2020\/01\/02\// );
-    expect( JSON.parse( await streamToString( content ) ).count ).toBe( 3 );
-    expect( delMock ).toHaveBeenCalledTimes( 1 );
-    expect( delMock ).toHaveBeenCalledWith( 'traces/run-1' );
-  } );
-
-  it( 'getDestination(): returns S3 URL using bucket and key from getVars', async () => {
-    getVarsMock.mockReturnValue( { remoteS3Bucket: 'my-bucket', redisIncompleteWorkflowsTTL: 3600, traceUploadDelayMs: 10_000 } );
-    const { getDestination } = await import( './index.js' );
-    const url = getDestination( traceInfo );
-    expect( getVarsMock ).toHaveBeenCalled();
-    expect( url ).toBe(
-      'https://my-bucket.s3.amazonaws.com/WF/2020/01/02/2020-01-02-03-04-05-678Z_id1.json'
-    );
-  } );
-
-  it( 'exec(): sets expiry on the redis key for each entry', async () => {
-    const { exec } = await import( './index.js' );
-
-    redisMulti.exec.mockResolvedValue( [] );
-    const workflowStart = {
-      kind: 'workflow', id: 'run-1', name: 'WF', parentId: undefined, action: 'start', details: {}, timestamp: startTime
-    };
-    zRangeMock.mockResolvedValue( [ JSON.stringify( workflowStart ) ] );
-
-    await exec( { traceInfo, entry: workflowStart } );
-
-    expect( redisMulti.expire ).toHaveBeenCalledTimes( 1 );
-    expect( redisMulti.expire ).toHaveBeenCalledWith( 'traces/run-1', 3600 );
-  } );
-
-  it( 'exec(): does not treat a non-root end (e.g. step without parentId) as root workflow end — regression for wrong root detection', async () => {
-    const { exec } = await import( './index.js' );
-
-    redisMulti.exec.mockResolvedValue( [] );
-    const workflowStart = { id: 'run-1', name: 'WF', kind: 'workflow', action: 'start', details: {}, timestamp: startTime };
-    const stepEndNoParent = { id: 'step-1', action: 'end', details: { done: true }, timestamp: startTime + 1 };
-    zRangeMock.mockResolvedValue( [
-      JSON.stringify( workflowStart ),
-      JSON.stringify( stepEndNoParent )
-    ] );
-
-    await exec( { traceInfo, entry: workflowStart } );
-    await exec( { traceInfo, entry: stepEndNoParent } );
-
-    expect( redisMulti.zAdd ).toHaveBeenCalledTimes( 2 );
-    expect( buildTraceTreeMock ).not.toHaveBeenCalled();
-    expect( uploadMock ).not.toHaveBeenCalled();
-    expect( delMock ).not.toHaveBeenCalled();
-  } );
-
-  it( 'exec(): when buildTraceTree returns null (incomplete tree), does not upload or bust cache', async () => {
-    const { exec } = await import( './index.js' );
-
-    redisMulti.exec.mockResolvedValue( [] );
-    const workflowEnd = {
-      kind: 'workflow', id: 'run-1', name: 'WF', parentId: undefined, action: 'end', details: {}, timestamp: startTime
-    };
-    zRangeMock.mockResolvedValue( [ JSON.stringify( workflowEnd ) ] );
-    buildTraceTreeMock.mockReturnValueOnce( null );
-
-    const endPromise = exec( { traceInfo, entry: workflowEnd } );
-    await vi.advanceTimersByTimeAsync( 10_000 );
-    await endPromise;
-
-    expect( buildTraceTreeMock ).toHaveBeenCalledTimes( 1 );
-    expect( uploadMock ).not.toHaveBeenCalled();
-    expect( delMock ).not.toHaveBeenCalled();
-  } );
-} );
-