Repository navigation
chore(vendor): move tinystoragedrivers to zstd 0.14.3 and drop zstd 0.13 from both lockfiles - #8446
Conversation
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 reviewReviewed head: Verdict: Ready for maintainer review · Priority: none Diagnostics and supporting detailsFull summaryPull request purpose (title): chore(vendor): move tinystoragedrivers to zstd 0.14.3 and drop zstd 0.13 from both lockfiles. Tiny Sweeper reviewed this change across 5 lane(s) and found 0 active actionable finding(s). Detailed lane evidence and any incomplete work are listed below. Review snapshot
Completeness: Complete 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
Evidence and run details
|
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Update the submodule reference vendor/tinystoragedrivers |
The submodule pointer changes to commit 19be64111fbecb4b11f3752432a0356029a83328. |
Priority:
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Other
Possibly related issues
- Epic: split openhuman-core into parallel-compiling crates (clean build 157 s -> target under 70 s) #8175: The PR performs the listed gitlink bump, but the available summary does not show the requested mono-item measurement.
Possibly related PRs
- tinyhumansai/openhuman#8325: Both PRs update the
vendor/tinystoragedriversrevision. - tinyhumansai/openhuman#8390: Both PRs advance the same submodule and relate to its zstd dependency line.
- tinyhumansai/openhuman#8423: Both PRs update zstd to 0.14.3; this PR also advances the vendor revision.
Suggested reviewers: tinysweeper
Merge Risk: 
Minimal · up to 7c888
-
No actionable merge risk remains after verifying the vendored revision and lockfile dependency transition.
Pre-merge checks: 
4
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
A rabbit checks the pointer’s place
One commit shifts with quiet grace
The vendor link now points ahead
No extra changes need be read
I nibble clover, pleased instead
Comment @coderabbitai help to get the list of available commands.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 41f1a9d196
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Summary
vendor/tinystoragedriversfrom 29471a1 to 19be641, the merge commit of fix(deps): move zstd to 0.14.3 for RUSTSEC-2026-0391/0398/0403 tinystoragedrivers#88, which bumps its workspacezstdto 0.14.3. (This PR first pinned the feat(channels): add channel schema, RPC controllers, and definition-driven UI #88 head e513fc1; the merge commit has the identical tree, b0e273c.)Cargo.lockandcrates/openhuman-app/Cargo.lock) drop zstd 0.13.3 and zstd-safe 7.3.0. The remaining zstd/zstd-safe references become bare names, and no package is added.Problem
openhuman-coreto zstd 0.14, buttinystoragedrivers-document(always) andtinystoragedrivers-connect(undersqlite) still required zstd ^0.13. So zstd 0.13.3 / zstd-safe 7.3.0 still shipped, along with RUSTSEC-2026-0391, RUSTSEC-2026-0398 and RUSTSEC-2026-0403.Solution
[patch."https://github.com/tinyhumansai/tinystoragedrivers"]points every tsd consumer (tinyagents, tinyflows, tinyinference) at the vendored copy, so this one gitlink clears zstd 0.13 from the whole graph.cargo update -p zstd@0.13.3and not hand-edited.Submission Checklist
## Related. No feature rows are affected.Impact
cargo check --locked: root-p openhuman-cli --features $CLI_FEATURES --all-targets, root--workspace, and app. All exit 0.cargo tree --locked -i zstd/-i zstd-safe: only zstd 0.14.3 / zstd-safe 8.2.0, in both root and app.scripts/check-kernel-floor.sh:profile=flows packages=341/341 names=312/312 native=3/3, down from 343.check-module-boundaries: 0 violations.tinystoragedrivers-*packages.Related
AI Authored PR Metadata (required for Codex/Linear PRs)
Linear Issue
Commit & Branch
Validation Run
pnpm --filter openhuman-app format:check. No frontend change.pnpm typecheck. No frontend change.cargo check --locked(root + app) and kernel-floor, see Impactcargo check --lockedpasses. No Rust source changed.Validation Blocked
Behavior Changes
Parity Contract
Duplicate / Superseded PR Handling
Summary by CodeRabbit