Repository navigation
build(deps): bump tinybus to v0.3.6, tinytools v0.6.0, tinystoragedrivers v2.1.1; fix rustdoc links - #180
Merged
Conversation
Update the vendored tinybus submodule to the latest upstream commit. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Updated the lockfile and workspace manifests to newer versions of several transitive and direct dependencies, including aws-lc-rs, cc, hyper, and the icu crates. This keeps the dependency graph current and picks up upstream fixes. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Add a [patch] entry for tinybus and tinybus-module so the whole dependency graph resolves to the same revision. tinystoragedrivers v2.1.1 still pins an older rev, which made tinybus::Connection two distinct types and broke compilation. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Point the tinybus and tinybus-module patch entries at local vendored crates instead of the pinned git revision, so the dependency graph resolves against the in-tree sources. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Replace the vendored path patches for tinybus and tinybus-module with git sources pinned to the same revision, and add the .git suffix to the remaining git URLs so they resolve consistently. This keeps the dependency graph unified on a single tinybus revision without relying on a vendored checkout. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Replace the pinned tinybus git revisions with the released v0.3.6 tag across the workspace patch entries and the storage dev-dependency. This unifies the dependency graph onto a single released version so tinybus::Connection resolves to one type. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Replace the v0.3.6 tag patches with explicit git revisions so the workspace and tinystoragedrivers resolve tinybus to a single source, keeping tinybus::Connection one type across the dependency graph. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Reverted a batch of transitive crates in Cargo.lock to their earlier versions, including aws-lc-rs, cc, hyper, and the icu family. The lockfile had drifted ahead of the versions the project is pinned to. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Replaced bracketed intra-doc links with plain backtick code spans across the graph, node, store, and copilot modules so rustdoc no longer emits broken-link warnings for items that are not in scope. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Repair intra-doc links that rustdoc could not resolve, replacing them with plain code spans or shortened paths so documentation builds without warnings. No runtime behaviour changes. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Several doc comments referenced items that are not in scope for rustdoc linking, which produced broken-link warnings. The affected references are now written as plain code spans so the documentation builds cleanly. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Rustdoc resolves intra-doc links by name within scope, so the explicit `(crate::...)` targets on these references were redundant. Dropping them keeps the doc comments shorter and lets rustdoc pick the target itself. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
This reverts commit da4f66a. Co-authored-by: Medulla <medulla@tinyhumans.ai>
Rustdoc resolves intra-doc links to items in scope without an explicit path, so the redundant `(crate::...)` targets were dropped from the doc comments. This is a cosmetic cleanup with no behaviour change. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
The link target was written as a path-qualified reference that rustdoc could not resolve, so the rendered docs showed a broken link. It now points at the plain `engine::resume` item. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Several doc comments referenced items by bare path, which rustdoc cannot resolve into links. Each reference now carries an explicit crate-relative target so the generated documentation links to the right item. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Tiny Sweeper reviewFindingsNo active actionable findings. Advisory notes
Verdict: Incomplete · Priority: medium Diagnostics and supporting detailsFull summaryPull request purpose (title): build(deps): bump tinybus to v0.3.6, tinytools v0.6.0, tinystoragedrivers v2.1.1; fix rustdoc links. Tiny Sweeper completed its review; deterministic results follow. Unreviewed paths
Partial review reasons
Review snapshot
Completeness: Incomplete What changedUpdates patch/rev pins for tinybus (workspace `[patch.crates-io]` and tinyflows-module's version range), tinytools, and tinystoragedrivers across crate manifests, and refreshes Cargo.lock for those packages. It also rewrites many rustdoc comments in the tinyflows crates and copilot tools (intra-doc links to backticks or shortened links), adds a note comment in crates/tinyflows-storage/Cargo.toml, and leaves tinyflows-storage's tinybus-sdk on the older 0de2d28 rev to match tinystoragedrivers' busproxy pin. Features
TestsNo supported feature-to-test mapping was produced. Test execution is not inferred. Before merge
Agent review detailscritique
security
tests
commits
description
Evidence and run details
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Update dependency pins Cargo.toml, vendor/tinybus, crates/tinyflows-*/Cargo.toml |
Manifests update tinybus, tinytools, and storage-driver dependency versions and revisions. The tinybus submodule pointer is updated to the pinned revision. The tinybus version range in tinyflows-module also changes. |
Documentation Updates
| Layer / File(s) | Summary |
|---|---|
Revise non-graph module documentation crates/tinyflows-adaptive/src/evals/mod.rs, crates/tinyflows-bus/*, crates/tinyflows-copilot/src/*, crates/tinyflows-drivers/src/store/mod.rs, crates/tinyflows-schedule/src/schedule.rs, crates/tinyflows-storage/src/primitives/leases.rs, crates/tinyflows/src/{authoring_report.rs,caps/agent.rs,catalog.rs,compat.rs,gates/*,interception.rs,lib.rs,model/*,nodes/*,observe.rs,preflight.rs,store/*,testkit/*,triggers.rs,validate.rs} |
Rust documentation references are changed between links, plain text, and inline code formatting. The copilot URL example is enclosed in angle brackets. |
Revise graph module documentation crates/tinyflows/src/graph/* |
Documentation references across graph modules are reformatted. Descriptions of graph fields, errors, and behavior remain unchanged. |
Priority:
Estimated code review effort: 2 (Simple) | ~12 minutes
Change: Other
Possibly related issues
- Epic: tinystoragedrivers as a hardened static primitive (types + API + per-primitive crates, engines as features) openhuman#8131 — This pull request updates tinyflows to
tinystoragedriversv2.1.1, matching the downstream adoption objective described in the issue.
Suggested reviewers: sanil-23
Merge Risk: 
5ca41
-
The dependency versions align at the inspected storage boundary, and the documentation typo does not obscure the retry behavior. No identified issue blocks merging after normal checks.
Pre-merge checks: 
-
Check name Status Explanation Description Check Check skipped - CodeRabbit’s high-level summary is enabled. Title check The title names the main change: dependency bumps to tinybus v0.3.6, tinytools v0.6.0, and tinystoragedrivers v2.1.1. It also names the rustdoc link fixes. The raw summary and PR objectives confirm th… Docstring Coverage Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 8 functions across 50 files. (15 skipped: … Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Full details: Title check
-
Explanation
The title names the main change: dependency bumps to tinybus v0.3.6, tinytools v0.6.0, and tinystoragedrivers v2.1.1. It also names the rustdoc link fixes. The raw summary and PR objectives confirm these changes. The title is long (99 characters), but it is specific and accurate, and it does not use vague terms.
Full details: Docstring Coverage
-
Explanation
Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 8 functions across 50 files. (15 skipped: 7 unsupported, 8 over the file limit.)
-
Finishing Touches
-
📝 Generate docstrings
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
A rabbit checks the pins in place
Then tidies links with careful grace
The graph docs hop from page to page
While versions turn a newer age
The burrow rests; no runtime change
Comment @coderabbitai help to get the list of available commands.
![tinysweeper[bot]](https://avatars.githubusercontent.com/in/4526585?s=60&v=4){kind=small}
tinysweeper
Bot
added
the
priority: p2
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
This was referenced Oct 26, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps the nested
vendor/tinybusgitlink to v0.3.6 (6ce06bc) and the tinybus git/crates-io patch revs, the tinytools rev to v0.6.0 (41cc80a) and tinystoragedrivers to v2.1.1 (bf1f0a9);tinyflows-modulenow requires tinybus>=0.3.6, <0.4. Lockfile refreshed only for those packages.Notes:
tinyflows-storagekeepstinybus-sdkon rev 0de2d28, the rev tinystoragedrivers v2.1.1 pins, sotinybus::Connectionstays one type with its busproxy (cargo cannot patch a git source onto the same URL). Move it when tinystoragedrivers releases against 0.3.x.RUSTDOCFLAGS=-D warnings cargo docerror (broken, private and redundant intra-doc links).cargo updatepulls jaq-std 3.0.4, which breaks=.x | length(expr doctests fail); the lock stays on 3.0.3. Worth a follow-up.Local: fmt, clippy (all features, schedule-only), tests, doc tests, rustdoc -D warnings, MSRV 1.88 check, module artifact verification all pass.
Summary by CodeRabbit