Repository navigation
chore(vendor): bump tinybus for owner-only Windows module cache - #7238
Conversation
Update the vendored tinybus submodule to a newer revision. Auto-committed-on: dragonfly Co-authored-by: Medulla <medulla@tinyhumans.ai>
|
Warning Review limit reached
This review includes 1 billable file and costs up to $0.25. Or wait 4 minutes for your next included review. View limit details
Comment |
Pins vendor/tinybus to TinyBus v0.1.3. Co-authored-by: Medulla <medulla@tinyhumans.ai>
Update the locked tinybus and tinybus-macros versions from 0.1.2 to 0.1.3. Auto-committed-on: dragonfly Co-authored-by: Medulla <medulla@tinyhumans.ai>
Update the locked tinybus and tinybus-macros versions from 0.1.2 to 0.1.3. Auto-committed-on: dragonfly Co-authored-by: Medulla <medulla@tinyhumans.ai>
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. |
Tiny Sweeper reviewTiny Sweeper reviewed this change across 6 lane(s) and found 0 active actionable finding(s). Detailed lane evidence and any incomplete work are listed below. State: Ready for maintainer review Review snapshot
Completeness: Complete What changedThe review could not produce a supported behavioral summary; inspect the cited changed surface and lane details below. FeaturesNone identified with supported citations. 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
e2e
Evidence and run details
|
Summary
Moves the
vendor/tinybusgitlink to pick up tinyhumansai/tinybus#40, which creates the Windows module release cache with an owner-only protected DACL and repairs an existing cache that inherits a group write ACE. The strict directory gate is unchanged.The fix is owned by TinyBus (module admission and cache staging), so OpenHuman needs only the pin bump:
crates/openhuman-core/src/modules/ops.rsalready installs into the persistent cache underdirs::cache_dir()and calls into tinybus'load_first_admitted, which runs the repair before loading.Pinned to the release
tinybus#40 merged (c19d78fcdd), passed its Windows CI jobs (
modulesandportabilityon windows-latest, which run thecfg(windows)ACL tests), and shipped in v0.1.3.vendor/tinybusnow points at the v0.1.3 commit (e231bf3).Validation
cargo check --manifest-path Cargo.toml -p openhuman: ok with the bumped pinCloses #6008
Repin validation
cargo check -p openhuman --locked: okcargo test -p openhuman --lib modules::registry: 11 passed