@output.ai/core 0.1.8-dev.pr156.c8e7f40 → 0.1.8-dev.pr156.f70e0a1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@output.ai/core",
-  "version": "0.1.8-dev.pr156.c8e7f40",
+  "version": "0.1.8-dev.pr156.f70e0a1",
   "description": "The core module of the output framework",
   "type": "module",
   "exports": {

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

@@ -1,29 +1,48 @@
 import { appendFileSync, mkdirSync, readdirSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
-import { join } from 'node:path';
+import { dirname, join } from 'node:path';
+import { fileURLToPath } from 'url';
 import buildTraceTree from '../../tools/build_trace_tree.js';
 import { EOL } from 'node:os';
 
-const oneWeekInMS = 1000 * 60 * 60 * 24 * 7;
-// Use project root (passed as argv[2]) for temp files, fallback to cwd
-const projectRoot = process.argv[2] || process.cwd();
-const tempDir = join( projectRoot, 'logs', 'temp', 'traces' );
+const __dirname = dirname( fileURLToPath( import.meta.url ) );
+
+const PURGE_TEMP_FILES_THRESHOLD = 1000 * 60 * 60 * 24 * 7; // 1 week in milliseconds
+
+// The path to the project root
+const LOCAL_PROJECT_ROOT_PATH = process.argv[2] || process.cwd();
+
+// The path to the local trace logs
+const LOCAL_TRACE_LOG_PATH = join( LOCAL_PROJECT_ROOT_PATH, 'logs' );
+
+// The path to the temporary trace logs
+const TMP_TRACE_LOG_PATH = join( __dirname, 'temp', 'traces' );
 
 const accumulate = ( { entry, executionContext: { workflowId, startTime } } ) => {
-  const path = join( tempDir, `${startTime}_${workflowId}.trace` );
+  const path = join( TMP_TRACE_LOG_PATH, `${startTime}_${workflowId}.trace` );
   appendFileSync( path, JSON.stringify( entry ) + EOL, 'utf-8' );
   return readFileSync( path, 'utf-8' ).split( EOL ).slice( 0, -1 ).map( v => JSON.parse( v ) );
 };
 
-const cleanupOldTempFiles = ( threshold = Date.now() - oneWeekInMS ) =>
-  readdirSync( tempDir )
+const cleanupOldTempFiles = ( threshold = Date.now() - PURGE_TEMP_FILES_THRESHOLD ) =>
+  readdirSync( TMP_TRACE_LOG_PATH )
     .filter( f => +f.split( '_' )[0] < threshold )
-    .forEach( f => rmSync( join( tempDir, f ) ) );
+    .forEach( f => rmSync( join( TMP_TRACE_LOG_PATH, f ) ) );
+
+/**
+ * Get the host trace log path, which is used for reporting trace locations.
+ * In containerized environments (e.g., Docker), this can be different from the local path
+ * to map container paths to host filesystem paths.
+ * @returns {string} The host trace log path from HOST_TRACE_PATH env var, or local path as fallback
+ */
+const getHostTraceLogPath = () => {
+  return process.env.HOST_TRACE_PATH || LOCAL_TRACE_LOG_PATH;
+};
 
 /**
  * Init this processor
  */
 export const init = () => {
-  mkdirSync( tempDir, { recursive: true } );
+  mkdirSync( TMP_TRACE_LOG_PATH, { recursive: true } );
   cleanupOldTempFiles();
 };
 
@@ -34,8 +53,7 @@ export const init = () => {
  * @returns {string} The local filesystem path for file operations
  */
 const getLocalOutputDir = workflowName => {
-  const root = process.argv[2] || process.cwd();
-  return join( root, 'logs', 'runs', workflowName );
+  return join( LOCAL_PROJECT_ROOT_PATH, 'logs', 'runs', workflowName );
 };
 
 /**
@@ -45,9 +63,7 @@ const getLocalOutputDir = workflowName => {
  * @returns {string} The path to report to users/API
  */
 const getReportOutputDir = workflowName => {
-  const root = process.argv[2] || process.cwd();
-  const basePath = process.env.HOST_TRACE_PATH || join( root, 'logs' );
-  return join( basePath, 'runs', workflowName );
+  return join( getHostTraceLogPath(), 'runs', workflowName );
 };
 
 const buildOutputFileName = ( { startTime, workflowId } ) => {

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

@@ -40,8 +40,8 @@ describe( 'tracing/processors/local', () => {
 
     init();
 
-    // Should create temp dir under process.cwd()/logs, not module-relative
-    expect( mkdirSyncMock ).toHaveBeenCalledWith( expect.stringMatching( /logs\/temp\/traces$/ ), { recursive: true } );
+    // Should create temp dir relative to module location using __dirname
+    expect( mkdirSyncMock ).toHaveBeenCalledWith( expect.stringMatching( /temp\/traces$/ ), { recursive: true } );
     expect( rmSyncMock ).toHaveBeenCalledTimes( 1 );
   } );