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

package/package.json

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

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

@@ -1,12 +1,12 @@
 import { appendFileSync, mkdirSync, readdirSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
-import { dirname, join } from 'node:path';
-import { fileURLToPath } from 'url';
+import { join } from 'node:path';
 import buildTraceTree from '../../tools/build_trace_tree.js';
 import { EOL } from 'node:os';
 
 const oneWeekInMS = 1000 * 60 * 60 * 24 * 7;
-const __dirname = dirname( fileURLToPath( import.meta.url ) );
-const tempDir = join( __dirname, 'temp', 'traces' );
+// 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 accumulate = ( { entry, executionContext: { workflowId, startTime } } ) => {
   const path = join( tempDir, `${startTime}_${workflowId}.trace` );
@@ -28,31 +28,25 @@ export const init = () => {
 };
 
 /**
- * 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
+ * Get the local file system path for ALL file I/O operations (read/write)
+ * Uses the project root path passed as argv[2], falls back to cwd
+ * @param {string} workflowName - The name of the workflow
+ * @returns {string} The local filesystem path for file operations
  */
-const getBasePath = () => {
-  return process.env.HOST_TRACE_PATH || join( process.cwd(), 'logs' );
+const getLocalOutputDir = workflowName => {
+  const root = process.argv[2] || process.cwd();
+  return join( root, 'logs', 'runs', workflowName );
 };
 
 /**
- * Get the output directory for trace files
+ * Get the host path for reporting trace file locations to users
+ * Uses HOST_TRACE_PATH if set (for Docker), otherwise uses project root
  * @param {string} workflowName - The name of the workflow
- * @param {boolean} useContainerPath - If true, uses container path for writing; otherwise uses host path for reporting
- * @returns {string} The output directory path
+ * @returns {string} The path to report to users/API
  */
-const getOutputDir = ( workflowName, useContainerPath = false ) => {
-  const basePath = useContainerPath ? getContainerBasePath() : getBasePath();
+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 );
 };
 
@@ -75,8 +69,8 @@ export const exec = ( { entry, executionContext } ) => {
   const { workflowId, workflowName, startTime } = executionContext;
   const content = buildTraceTree( accumulate( { entry, executionContext } ) );
 
-  // Use container path for writing files (true flag)
-  const dir = getOutputDir( workflowName, true );
+  // Always use local path for writing files
+  const dir = getLocalOutputDir( workflowName );
   const path = join( dir, buildOutputFileName( { startTime, workflowId } ) );
 
   mkdirSync( dir, { recursive: true } );
@@ -92,5 +86,6 @@ export const exec = ( { entry, executionContext } ) => {
  * @returns {string} The absolute path where the trace will be saved
  */
 export const getDestination = ( { startTime, workflowId, workflowName } ) => {
-  return join( getOutputDir( workflowName ), buildOutputFileName( { workflowId, startTime } ) );
+  // Use report path for reporting to users/API
+  return join( getReportOutputDir( workflowName ), buildOutputFileName( { workflowId, startTime } ) );
 };

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

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