chore(deps): consolidate PRs #128-137, fix h2/lru advisories - #138
Merged
Conversation
Consolidates 8 of the 11 open Dependabot PRs into a single branch: - deps(rust): serde_json 1.0.150 -> 1.0.151 (#128) - deps(rust): calamine 0.36.0 -> 0.36.1 (#131) - deps(rust): http-body-util 0.1.4 -> 0.1.5 (#132) - deps(ts): js-yaml 5.2.2 -> 5.2.3 (#130) - deps(ts): vite 8.0.16 -> 8.2.1 (#133) - deps(ts): prettier 3.8.3 -> 3.9.6 (#134) - deps(ts): recharts 3.8.1 -> 3.10.1 (#135) - deps(actions): taiki-e/install-action 2.85.8 -> 2.86.3 (#137) Deferred (left open, not applied): - #116/#126 (candle-core/candle-transformers 0.10.2 -> 0.11.0): still blocked on a mistralrs release supporting candle 0.11 (mistralrs is still pinned at 0.8.1); would create duplicate candle-core/candle-nn builds, same as previously deferred in #127. - #136 (@tanstack/react-table 8.21.3 -> 9.1.2): 8.21.3 is still the latest 8.x release, so this is a major-version rewrite, not a patch bump — same situation as previously-deferred #119. Pre-existing/upgrade-caused fixes: - Prettier 3.9 changed union-type-alias wrapping ("don't break union type when it can fit"), which flipped two already-passing files to fail the format-check gate. Reformatted with `prettier --write` (RecurringPage.tsx, models.ts) — no logic changes. Security audit (cargo audit / pnpm audit / GitHub Dependabot alerts): - RUSTSEC-2026-0258 (h2 0.4.14, unbounded empty DATA frames DoS): fixed by bumping h2 to 0.4.16 (in-range lockfile-only update via aws-smithy-http-client/hyper's transitive dependency). - RUSTSEC-2026-0253 (lru 0.16.4, use-after-free risk in LruCache::pop() under panic): no fix available — aws-sdk-s3 (latest 1.143.0 checked) still hard-pins `lru = "^0.16.3"`; the fix requires lru 0.18, a semver-major bump upstream hasn't adopted. Documented and ignored in audit-ignore / .cargo/audit.toml pending an aws-sdk-s3 release. - RUSTSEC-2026-0235 (rkyv 0.7.46, pre-existing ignore): also added to .cargo/audit.toml, which was missing it despite audit-ignore already covering it — brings the two files back in sync. - pnpm audit: no known vulnerabilities. - GitHub Dependabot alerts: 0 open. Verification (mirrors CI exactly): - Frontend: pnpm install --frozen-lockfile, lint, tsc --noEmit, prettier --check, vitest --run, build — all pass. - Backend: cargo fmt --all --check, cargo clippy --workspace --all-targets -- -D warnings, cargo test --workspace (incl. DB integration tests via docker-compose.test.yml) — all pass. - Security: bash .github/scripts/cargo-audit.sh (CI's exact gate) — passes with 9 documented exceptions.
This was referenced Aug 25, 2026
Owner
Author
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
This was referenced Aug 25, 2026
…rge_err CI's floating `dtolnay/rust-toolchain@stable` action advanced to a newer Rust/clippy since the last green run on main, and the newer clippy now flags StorageError's Err variants (392-432 bytes, dominated by aws_sdk_s3::error::SdkError) under result_large_err. Box each SdkError source field to shrink the enum; no behavior change.
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.
Consolidates 8 of the 11 open Dependabot PRs into a single branch, applied as latest-compatible-within-major, plus a security-driven fix for two advisories found during the audit pass.
Superseded PRs
Deferred (left open)
mistralrs(pinned at 0.8.1) has not shipped a release supporting candle 0.11; bumping alone would create duplicate candle-core/candle-nn builds in the workspace, which the existingCargo.tomlcomment explicitly says the 0.10 pin exists to avoid. Same blocker as the last consolidation (chore(deps): consolidate PRs #116-126, fix rkyv/brace-expansion/nanoid advisories #127).@tanstack/react-table8.21.3 → 9.1.2): 8.21.3 is still the latest published 8.x release, so this is a major-version rewrite, not a patch/minor bump — same situation as the previously-deferred deps(ts): bump @tanstack/react-table from 8.21.3 to 9.0.0 in /frontend #119.Pre-existing / upgrade-caused fixes
prettier --checkgate under the new formatter version. Reformatted withprettier --write(RecurringPage.tsx,models.ts) — no logic changes, formatting only.Security audit
Ran
cargo audit(via the CI's exact.github/scripts/cargo-audit.shgate),pnpm audit, and checked GitHub Dependabot alerts.h2to 0.4.16 via an in-range, lockfile-only update (transitive throughaws-smithy-http-client/hyper).LruCache::pop()under panic): deferred, documented.aws-sdk-s3(checked up to its latest 1.143.0 release) hard-pinslru = "^0.16.3"; the fix landed in lru 0.18.0, a semver-major bump upstream hasn't adopted yet. No safe override exists without breakingaws-sdk-s3's own dependency graph. Added toaudit-ignoreand.cargo/audit.tomlwith rationale; will resolve onceaws-sdk-s3bumps itslrurequirement..cargo/audit.toml, which was missing this entry despiteaudit-ignorealready covering it (the header comment says the two files should mirror each other) — brings them back in sync. No behavior change.pnpm audit: no known vulnerabilities.Verification
Mirrors the CI gates in
.github/workflows/ci.ymlexactly.Frontend:
Backend:
Security:
🤖 Generated with Claude Code