workflow-review
Workflow review
Read the workflow files and recent execution evidence before proposing changes.
When the workflow uses named volumes or filesystem-backed state, use workflow-volume-design and inspect volume declarations, actual usage, and the steps on the common request path.
Review the workflow as one program:
- Explain its trigger, data flow, side effects, and result.
- Test representative success, empty, malformed, duplicate, and dependency-failure inputs.
- Identify failures that can be handled locally and failures that must remain visible.
- Remove redundant work and simplify boundaries before adding retries or branches.
- Run affected steps and tests after changes.
For volume-backed workflows, check that frequent requests do not scan growing directories, mount unrelated state, or start unnecessary downstream executions. Check that scheduled writers cannot overlap on the same files, select bounded batches before scanning or parsing, retain input until derived changes are durable, and drain pending work under sustained input. Compare request latency, waiting executions, oldest pending age, and volume usage before and after a change.
Prioritize concrete issues supported by code or execution results. Preserve intentional behavior, and do not add logging, retries, or abstractions without a specific failure they address.