chore: minor improvements

Author: UlisesGascon

src/__tests__/cli-commands.test.ts

@@ -527,7 +527,7 @@ describe('CLI Commands', () => {
       // Verify the result
       expect(result.success).toBe(true)
       expect(result.messages).toHaveLength(5) // 5 messages with details
-      expect(result.messages[0]).toContain('Workflow executed successfully in 2500 ms')
+      expect(result.messages[0]).toContain('Workflow executed successfully in 2.50 seconds')
       expect(result.messages[1]).toContain('Status: completed')
       expect(result.messages[2]).toContain('Started:')
       expect(result.messages[3]).toContain('Finished:')
@@ -547,7 +547,7 @@ describe('CLI Commands', () => {
       // Verify the result
       expect(result.success).toBe(true)
       expect(result.messages).toHaveLength(5) // 5 messages with details
-      expect(result.messages[0]).toContain('Workflow executed unsuccessfully in 2500 ms')
+      expect(result.messages[0]).toContain('Workflow executed unsuccessfully in 2.50 seconds')
       expect(result.messages[1]).toContain('Status: failed')
       expect(result.messages[2]).toContain('Started:')
       expect(result.messages[3]).toContain('Finished:')

src/cli-commands.ts

@@ -148,8 +148,9 @@ export const executeWorkflow = async (workflowId: string, data: any): Promise<Co
     const startTime = new Date(results.started)
     const endTime = new Date(results.finished)
     const duration = endTime.getTime() - startTime.getTime()
+    const durationStr = duration < 1000 ? `${duration} ms` : `${(duration / 1000).toFixed(2)} seconds`
 
-    messages.push(`Workflow executed ${results.result.success ? 'successfully' : 'unsuccessfully'} in ${duration} ms`)
+    messages.push(`Workflow executed ${results.result.success ? 'successfully' : 'unsuccessfully'} in ${durationStr}`)
     messages.push(`- Status: ${results.status}`)
     messages.push(`- Started: ${startTime}`)
     messages.push(`- Finished: ${endTime}`)

src/index.ts

@@ -56,13 +56,18 @@ workflow
     }
     // Check if workflow requires additional data and if it is provided or requires collection
     if (workflow.isRequiredAdditionalData && (!options.data && !options.file)) {
-      throw new Error('Workflow does not require additional data. Please remove -d or -f options')
+      throw new Error('Workflow requires additional data. Please provide data using -d or -f option')
     } else if (options.data && options.file) {
       throw new Error('Please provide either -d or -f, not both')
     } else if (options.data) {
       data = options.data
     } else if (options.file) {
-      data = JSON.parse(fs.readFileSync(options.file, 'utf-8'))
+      try {
+        const fileContent = fs.readFileSync(options.file, 'utf-8')
+        data = JSON.parse(fileContent)
+      } catch (error) {
+        throw new Error(`Failed to read or parse file: ${error instanceof Error ? error.message : 'Unknown error'}`)
+      }
     }
 
     // If data is provided, validate against JSON Schema