Repository navigation
chore(deps): pin tinystoragedrivers at its tinybus 0.3.6 commit - #182
Conversation
tinystoragedrivers 2.1.1 at bf1f0a9 pinned tinybus 0.1.6 (git 0de2d28), so the lock carried two tinybus series: 0.3.6 for the module and 0.1.6 for the storage bus and busproxy. tinystoragedrivers#87 (29471a1) moves that pin to tinybus 0.3.6 at 6ce06bc, the rev this workspace already patches crates-io to. Pin every tinystoragedrivers crate at 29471a1 and the tinyflows-storage `tinybus-sdk` test dependency at 6ce06bc, so the lock resolves one tinybus.
Tiny Sweeper reviewReviewed head: Verdict: Ready for maintainer review · Priority: none Diagnostics and supporting detailsFull summaryPull request purpose (title): chore(deps): pin tinystoragedrivers at its tinybus 0.3.6 commit. This pull request realigns tinyflows' dependency pins so the module release resolves a single tinybus series. Per the title and description, tinyflows main locked two tinybus series: the module uses tinybus 0.3.6 at 6ce06bc, while tinystoragedrivers 2.1.1 at bf1f0a9 pinned tinybus 0.1.6 at 0de2d28 for tinystoragedrivers-bus and -busproxy, and tinyflows-storage's tinybus-sdk dev-dependency was pinned to the same 0.1.6 rev to keep Connection one type with busproxy. Because openhuman's loader accepts only modules built in the host's tinybus series, the module release has to resolve one tinybus. The change moves every tinystoragedrivers dependency from bf1f0a9 to 29471a1 (the merge of tinystoragedrivers#87, which moves that pin to tinybus 0.3.6 at 6ce06bc), moves the tinyflows-storage tinybus-sdk dev-dependency from 0de2d28 to 6ce06bc, and, per the description, updates Cargo.lock so the 11 tinystoragedrivers sources move to the new rev, the tinybus 0.1.6 package is removed, and the five tinybus 0.1.6 / 0.3.6 references become a single tinybus, with no dependency orphaned. The description lane notes Cargo.lock was not shown in the diff despite the body's lock claims about an 11-source move and removal of the tinybus 0.1.6 package, so those claims could not be verified. Unreviewed paths
Review snapshot
Completeness: Complete What changedThe diff bumps git revision pins across four workspace manifests: crates/tinyflows-adaptive/Cargo.toml, crates/tinyflows-drivers/Cargo.toml, crates/tinyflows-module/Cargo.toml and crates/tinyflows-storage/Cargo.toml. tinystoragedrivers-api, tinystoragedrivers-types, tinystoragedrivers-testkit and tinystoragedrivers-busproxy references move from rev bf1f0a97fbe6165a6edd9521659c7f6859700761 to rev 29471a164619bcec03e5858a7d7ee0c65f52bf22 while keeping version 2.1.1. In crates/tinyflows-storage/Cargo.toml the tinybus-sdk dev-dependency (package tinybus) moves from rev 0de2d283f35cbbff08efad6fbe05537ffb74611b to rev 6ce06bc674e4f4d718aa51e643ef62c1f372e270. The pull request description also states Cargo.lock changes (11 tinystoragedrivers sources move, the tinybus 0.1.6 package is removed, the five tinybus references collapse to one, and nothing is orphaned), but Cargo.lock is not present in the supplied diff evidence, so those lock edits are described rather than observed here. No source code is changed. Features
TestsNo supported feature-to-test mapping was produced. Test execution is not inferred. FindingsNo active actionable findings. Before mergeNone. Agent review detailscritique
security
tests
commits
description
Evidence and run details
|
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Update dependency pins crates/tinyflows-adaptive/Cargo.toml, crates/tinyflows-drivers/Cargo.toml, crates/tinyflows-module/Cargo.toml, crates/tinyflows-storage/Cargo.toml |
The manifests pin tinystoragedrivers dependencies to revision 29471a164619bcec03e5858a7d7ee0c65f52bf22. The storage manifest also pins tinybus-sdk to revision 6ce06bc674e4f4d718aa51e643ef62c1f372e270. |
Priority:
Estimated code review effort: 1 (Trivial) | ~4 minutes
Change: Other
Possibly related issues
tinyhumansai/openhuman#8131— The issue identifies thistinystoragedriverspin correction as a remaining close condition, while covering broader restructuring beyond this PR.
Possibly related PRs
- tinyhumansai/tinyflows#180 — This PR continues the dependency alignment by updating the
tinystoragedriverspin and the storagetinybus-sdkpin.
Suggested reviewers: senamakel
Merge Risk: 
Minimal · up to 4cd72
-
The pin updates are consistent with the stated goal of using one tinybus series and present no actionable merge risk.
Pre-merge checks: 
5
-
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (4 skipped: 4 unsupported.)
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
A rabbit checks each revision in a row,
New pins take root where Cargo winds blow.
The storage crate gets a tinybus tune,
Four manifests align beneath the moon.
I nibble the diff, then hop away.
Comment @coderabbitai help to get the list of available commands.
tinyflows
mainlocks two tinybus series. The module uses 0.3.6 (6ce06bc). tinystoragedrivers 2.1.1 atbf1f0a9pins tinybus 0.1.6 (0de2d28) fortinystoragedrivers-busand-busproxy, andtinyflows-storagepinstinybus-sdkto the same 0.1.6 rev so thatConnectionstays one type with busproxy. openhuman's loader accepts only modules built in the host's tinybus series, so the module release has to resolve a single tinybus.tinystoragedrivers#87 (merge
29471a1) moves that pin to tinybus 0.3.6 at6ce06bc. No other tinystoragedrivers manifest changes betweenbf1f0a9and29471a1.Changes
bf1f0a9to29471a1(adaptive, drivers, module, storage).tinyflows-storagetinybus-sdkdev-dependency moves from0de2d28to6ce06bc.Cargo.lock: the 11 tinystoragedrivers sources move to the new rev, and thetinybus 0.1.6package is removed. All fivetinybus 0.1.6/tinybus 0.3.6references become a singletinybus.CI's
cargo check --lockedchecks the lock edit.Summary by CodeRabbit