@output.ai/core 0.1.8-dev.pr156.696d4dd → 0.1.8-dev.pr156.ac53caf

package/package.json

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

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

@@ -1,11 +1,12 @@
 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 local working directory for temp files, not module-relative path
-const tempDir = join( process.cwd(), 'logs', 'temp', 'traces' );
+const __dirname = dirname( fileURLToPath( import.meta.url ) );
+const tempDir = join( __dirname, 'temp', 'traces' );
 
 const accumulate = ( { entry, executionContext: { workflowId, startTime } } ) => {
   const path = join( tempDir, `${startTime}_${workflowId}.trace` );
@@ -27,23 +28,31 @@ export const init = () => {
 };
 
 /**
- * Get the local file system path for ALL file I/O operations (read/write)
- * Always uses the container's local working directory
- * @param {string} workflowName - The name of the workflow
- * @returns {string} The local filesystem path for file operations
+ * Get the container base path for writing trace files
+ * Always writes to the local working directory, regardless of HOST_TRACE_PATH
+ * @returns {string} The base path where traces should be written in the container
+ */
+const getContainerBasePath = () => {
+  return join( process.cwd(), 'logs' );
+};
+
+/**
+ * Get the base path for reporting trace file locations
+ * Uses HOST_TRACE_PATH if set (for reporting to host), otherwise uses local path
+ * @returns {string} The base path where traces can be accessed from the host
  */
-const getLocalOutputDir = workflowName => {
-  return join( process.cwd(), 'logs', 'runs', workflowName );
+const getBasePath = () => {
+  return process.env.HOST_TRACE_PATH || join( process.cwd(), 'logs' );
 };
 
 /**
- * Get the host path for reporting trace file locations to users
- * Uses HOST_TRACE_PATH if set (for Docker), otherwise uses local path
+ * Get the output directory for trace files
  * @param {string} workflowName - The name of the workflow
- * @returns {string} The path to report to users/API
+ * @param {boolean} useContainerPath - If true, uses container path for writing; otherwise uses host path for reporting
+ * @returns {string} The output directory path
  */
-const getReportOutputDir = workflowName => {
-  const basePath = process.env.HOST_TRACE_PATH || join( process.cwd(), 'logs' );
+const getOutputDir = ( workflowName, useContainerPath = false ) => {
+  const basePath = useContainerPath ? getContainerBasePath() : getBasePath();
   return join( basePath, 'runs', workflowName );
 };
 
@@ -66,8 +75,8 @@ export const exec = ( { entry, executionContext } ) => {
   const { workflowId, workflowName, startTime } = executionContext;
   const content = buildTraceTree( accumulate( { entry, executionContext } ) );
 
-  // Always use local path for writing files
-  const dir = getLocalOutputDir( workflowName );
+  // Use container path for writing files (true flag)
+  const dir = getOutputDir( workflowName, true );
   const path = join( dir, buildOutputFileName( { startTime, workflowId } ) );
 
   mkdirSync( dir, { recursive: true } );
@@ -83,6 +92,5 @@ export const exec = ( { entry, executionContext } ) => {
  * @returns {string} The absolute path where the trace will be saved
  */
 export const getDestination = ( { startTime, workflowId, workflowName } ) => {
-  // Use report path for reporting to users/API
-  return join( getReportOutputDir( workflowName ), buildOutputFileName( { workflowId, startTime } ) );
+  return join( getOutputDir( workflowName ), buildOutputFileName( { workflowId, startTime } ) );
 };

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

@@ -40,8 +40,7 @@ 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 } );
+    expect( mkdirSyncMock ).toHaveBeenCalledWith( expect.stringMatching( /temp\/traces$/ ), { recursive: true } );
     expect( rmSyncMock ).toHaveBeenCalledTimes( 1 );
   } );