@output.ai/core 0.1.7 → 0.1.8-dev.pr156.05c9aa2

package/package.json

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

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

@@ -27,7 +27,17 @@ export const init = () => {
   cleanupOldTempFiles();
 };
 
-const getOutputDir = workflowName => join( process.argv[2], 'logs', 'runs', workflowName );
+/**
+ * Get the base path for trace file storage
+ * @returns {string} The base path where traces should be written
+ */
+const getBasePath = () => {
+  return process.env.HOST_TRACE_PATH || join( process.cwd(), 'logs' );
+};
+
+const getOutputDir = workflowName => {
+  return join( getBasePath(), 'runs', workflowName );
+};
 
 const buildOutputFileName = ( { startTime, workflowId } ) => {
   const timestamp = new Date( startTime ).toISOString().replace( /[:T.]/g, '-' );
@@ -56,12 +66,13 @@ export const exec = ( { entry, executionContext } ) => {
 };
 
 /**
- * Returns where the trace is saved
+ * Returns where the trace is saved as an absolute path
  * @param {object} args
  * @param {string} args.startTime - The start time of the workflow
  * @param {string} args.workflowId - The id of the workflow execution
  * @param {string} args.workflowName - The name of the workflow
- * @returns {string} The path where the trace will be saved
+ * @returns {string} The absolute path where the trace will be saved
  */
-export const getDestination = ( { startTime, workflowId, workflowName } ) =>
-  join( getOutputDir( workflowName ), buildOutputFileName( { workflowId, startTime } ) );
+export const getDestination = ( { startTime, workflowId, workflowName } ) => {
+  return join( getOutputDir( workflowName ), buildOutputFileName( { workflowId, startTime } ) );
+};

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

@@ -60,8 +60,23 @@ describe( 'tracing/processors/local', () => {
 
     expect( writeFileSyncMock ).toHaveBeenCalledTimes( 3 );
     const [ writtenPath, content ] = writeFileSyncMock.mock.calls.at( -1 );
-    expect( writtenPath ).toMatch( /\/logs\/runs\/WF\// );
+    // Changed: Now uses process.cwd() + '/logs' fallback when HOST_TRACE_PATH not set
+    expect( writtenPath ).toMatch( /\/runs\/WF\// );
     expect( JSON.parse( content.trim() ).count ).toBe( 3 );
   } );
+
+  it( 'getDestination(): returns absolute path', async () => {
+    const { getDestination } = await import( './index.js' );
+
+    const startTime = Date.parse( '2020-01-02T03:04:05.678Z' );
+    const workflowId = 'workflow-id-123';
+    const workflowName = 'test-workflow';
+
+    const destination = getDestination( { startTime, workflowId, workflowName } );
+
+    // Should return an absolute path
+    expect( destination ).toMatch( /^\/|^[A-Z]:\\/i ); // Starting with / or Windows drive letter
+    expect( destination ).toContain( '/logs/runs/test-workflow/2020-01-02-03-04-05-678Z_workflow-id-123.json' );
+  } );
 } );