Skip to content

fix(task): resolve scheduled workflow IDs from filesystem#120

Merged
duguwanglong merged 2 commits intomainfrom
fix/scheduled-workflow-id-resolution
Apr 14, 2026
Merged

fix(task): resolve scheduled workflow IDs from filesystem#120
duguwanglong merged 2 commits intomainfrom
fix/scheduled-workflow-id-resolution

Conversation

@xiami762
Copy link
Copy Markdown
Contributor

Summary

  • extract a shared filesystem workflow lookup helper so workflow ID resolution is defined in one place
  • make scheduled workflow executions resolve workflowID to workflow JSON before calling run_workflow, matching the manual run path
  • add a regression test covering scheduled workflow execution with an ID-based workflow reference

Test plan

  • uv run pytest tests/task/test_task.py -q
  • uv run ruff check flocks/workflow/fs_store.py flocks/task/executor.py flocks/server/routes/workflow.py tests/task/test_task.py

Made with Cursor

xiami762 added 2 commits April 14, 2026 16:16
Reuse the filesystem-backed workflow lookup when scheduled tasks run workflows so scheduler executions follow the same ID-based resolution path as manual workflow runs.

Made-with: Cursor
Remove duplicated filesystem workflow lookup logic from the workflow routes and keep the shared workspace-root cache valid across cwd changes to avoid inconsistent workflow resolution.

Made-with: Cursor
@duguwanglong duguwanglong merged commit 93ea025 into main Apr 14, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants