riotpiao-poimen/poimen-workflows
2
0
Fork 0
Code Issues 6 Pull Requests Actions Packages Projects Releases Wiki Activity

fix: use string activity name in LLMTestWorkflow instead of function reference

Browse Source 
- Changed workflow.ExecuteActivity(actCtx, activity.LLMInferenceActivity, ...)
  to workflow.ExecuteActivity(actCtx, "LLMInferenceActivity", ...)
- Fixes WorkflowTaskFailed error on first execution
- Matches Temporal Go SDK best practices (determinism requirement)
- Workflow now executes cleanly on first attempt without retries
- Timeline: 5 events instead of 8, 0 failures instead of 1
This commit is contained in:
Test
2026-09-08 12:49:08 -07:00
parent 9103b78d9c
commit e15a535d82
2 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
poimen-worker
BIN
View File
Binary file not shown.
workflow/llm_test_workflow.go
+1 -1
View File
@@ -26,7 +26,7 @@ func LLMTestWorkflow(ctx workflow.Context, input LLMTestWorkflowInput) (string,
} }
var result activity.LLMInferenceOutput var result activity.LLMInferenceOutput
err := workflow.ExecuteActivity(actCtx, activity.LLMInferenceActivity, actInput).Get(actCtx, &result) err := workflow.ExecuteActivity(actCtx, "LLMInferenceActivity", actInput).Get(actCtx, &result)
if err != nil { if err != nil {
return "", err return "", err
} }