Skip to content

chore(deps): bump vendor/tinyflows to 7ba0b918 - #5717

Closed
senamakel wants to merge 1 commit into
tinyhumansai:mainfrom
senamakel:bump-tinyflows-run-executor
Closed

chore(deps): bump vendor/tinyflows to 7ba0b918#5717
senamakel wants to merge 1 commit into
tinyhumansai:mainfrom
senamakel:bump-tinyflows-run-executor

Conversation

@senamakel

@senamakel senamakel commented Aug 24, 2026

Copy link
Copy Markdown
Member

Summary

Bumps vendor/tinyflows from 4e2e0426 to 7ba0b918 (current tip of tinyflows main).

This is the middle link in a three-repo chain fixing workflow run records left behind by a killed process:

  1. feat(store): record which process owns a run, and enumerate unsettled runs tinyflows#76 — merged. Adds RunExecutor and cancel_requested to RunRecord, and WorkflowStore::unsettled_runs().
  2. This PR — makes those available to anything vendoring OpenHuman.
  3. tinyhumansai/medulla#280 — the host half (reconciliation sweep + cross-process cancel). It cannot compile until this lands, because it imports RunExecutor and unsettled_runs() through the OpenHuman-vendored tinyflows.

What is in the bump

4e2e0426..7ba0b918 is tinyflows#76 plus the already-merged #74 (workdir resolution) and its followups. It lands on tinyflows main'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:

  • No struct literals of RunRecord. New required fields would break those, but grep -rn "RunRecord" src/ returns nothing — OpenHuman never names the type. (The new fields are #[serde(default)] regardless, so serialized records round-trip both ways.)
  • No impl WorkflowStore. A new trait method would break an implementor, but there are none in src/, and unsettled_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 whole tinyflows dependency (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 against origin/main before pushing.

Summary by CodeRabbit

  • Chores
    • Updated the underlying workflow components to the latest available version.
    • No user-facing features or behavior changes are expected from this update.
    • Existing functionality remains unchanged.

Picks up tinyhumansai/tinyflows#76, which adds RunExecutor and
cancel_requested to RunRecord and WorkflowStore::unsettled_runs().

Co-authored-by: Medulla <medulla@tinyhumans.ai>
@senamakel
senamakel requested a review from a team August 24, 2026 07:37
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for security reviews. Please try again later.

@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 8fbcceee-f591-433e-9278-eb17d99e9fd3

📥 Commits

Reviewing files that changed from the base of the PR and between 1f332dd and 815237a.

📒 Files selected for processing (1)
  • vendor/tinyflows

📝 Walkthrough

Walkthrough

The pull request updates the vendor/tinyflows submodule reference from one commit to another. No exported or public declarations change.

Changes

tinyflows update

Layer / File(s) Summary
Submodule reference update
vendor/tinyflows
The submodule now points to commit 7ba0b918f2d6beb09b8cd2ee92ee16f71c8311e0 instead of 4e2e04267ae62f04a3f8faccaf0bdad9f17a9c05.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: graycyrus

Poem

A rabbit hops through vendor space,
Tinyflows shifts one commit in place.
No public names change or grow,
Just a newer pointer now we show.
“Hop approved!” the rabbit sings.

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@senamakel

Copy link
Copy Markdown
Member Author

Closing as a duplicate of #5716, which was raised 20 minutes earlier with a byte-identical diff (vendor/tinyflows 4e2e0426 → 7ba0b918, one file). Two of us converged on the same bump; #5716 has priority. Compile evidence carried over there.

@senamakel senamakel closed this Aug 24, 2026
@tinysweeper tinysweeper Bot added the priority: p3 Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect. label Aug 24, 2026

@tinysweeper tinysweeper Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tinysweeper found nothing blocking. Approving.

$0.0000 · 0 in / 0 out · 16 embedded · openrouter/openai/text-embedding-3-small

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: p3 Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant