chore(deps): bump vendor/tinyflows to 7ba0b918 - #5717
Conversation
Picks up tinyhumansai/tinyflows#76, which adds RunExecutor and cancel_requested to RunRecord and WorkflowStore::unsettled_runs(). Co-authored-by: Medulla <medulla@tinyhumans.ai>
|
You have reached your Codex usage limits for security reviews. Please try again later. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe pull request updates the Changestinyflows update
Estimated code review effort: 1 (Trivial) | ~2 minutes Suggested reviewers: Poem
Warning Your free Security trial is over. An organization admin can activate billing to continue. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Bumps
vendor/tinyflowsfrom4e2e0426to7ba0b918(current tip of tinyflowsmain).This is the middle link in a three-repo chain fixing workflow run records left behind by a killed process:
RunExecutorandcancel_requestedtoRunRecord, andWorkflowStore::unsettled_runs().RunExecutorandunsettled_runs()through the OpenHuman-vendored tinyflows.What is in the bump
4e2e0426..7ba0b918is tinyflows#76 plus the already-merged #74 (workdir resolution) and its followups. It lands on tinyflowsmain's tip, so this is an ordinary catch-up rather than a jump to a side branch.Why this is low risk for OpenHuman
The tinyflows change is purely additive, and the two shapes that could break a consumer do not occur here:
RunRecord. New required fields would break those, butgrep -rn "RunRecord" src/returns nothing — OpenHuman never names the type. (The new fields are#[serde(default)]regardless, so serialized records round-trip both ways.)impl WorkflowStore. A new trait method would break an implementor, but there are none insrc/, andunsettled_runs()ships with a default body anyway.OpenHuman consumes the engine surface —
tinyflows::caps,::model,::nodes,::observability— not the store. The store is what #76 touched.Testing
cargo check --features flows— clean (Finished dev profile in 1m 33s). That is the feature gating the wholetinyflowsdependency (flows = ["dep:tinyflows"]), so it is the surface this bump can affect.Not run: the full test suite, since this changes no OpenHuman source — the diff is one gitlink line.
Note on the diff
Exactly one file. If you have seen sibling
vendor/*gitlinks ride along in bumps from this workspace, that is a local auto-commit hook picking up submodule drift; those were stripped here and the diff verified againstorigin/mainbefore pushing.Summary by CodeRabbit