chore: update Rust to 1.97.1 and refresh dependencies and linters - #412
Merged
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Deploying ryl-docs with
|
| Latest commit: |
f8dd707
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://51b012d3.ryl-docs.pages.dev |
| Branch Preview URL: | https://chore-rust-1-97-1-dependency.ryl-docs.pages.dev |
Test Coverage (nextest + cargo-llvm-cov)Missed Lines (per file) |
Replaces the stalled #398 refresh, which has been frozen at its 2026-07-19 state since the bot lost push access. - Rust 1.96.0 -> 1.97.1 (`rust-toolchain.toml`, `clippy.toml` msrv, AGENTS.md) - `cargo update` - `prek autoupdate --cooldown-days 7`: rumdl 0.2.52, ruff 0.16.2, typos 1.49.0, zizmor 1.29.0 - Allow the `syn` duplicate: `cargo update` moved the build graph to syn 3.x, but synstructure 0.13 still pins syn 2.x and reaches the graph only through the `jsonschema` dev-dependency, so `clippy::multiple_crate_versions` fires on a duplicate no dependency bump of ours can resolve. - ruff 0.16's RUF105/RUF106 rewrote one suppression comment. Rust 1.97 adds nothing this codebase can use: the stabilised APIs are integer bit-manipulation helpers, and `char::is_control`'s new const-ness is unusable at our call sites, which all sit in non-const predicates. Clippy under `msrv = "1.97.1"` reports no modernisation opportunities.
owenlamont
force-pushed
the
chore/rust-1.97.1-dependency-refresh
branch
from
August 16, 2026 11:34
e6f9cc8 to
f8dd707
Compare
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
1 participant
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.
Replaces the stalled #398, which has been frozen at its 2026-07-19 state since the refresh bot lost push access (fixed separately in #411).
Rust 1.96.0 → 1.97.1
rust-toolchain.toml,clippy.tomlmsrv, and the AGENTS.md version line, kept in sync per the repo's own rule.Audit for new-in-1.97 features this codebase could adopt — nothing applies:
isolate_highest_one/lowest_one/bit_widthetc.1 << 0) infloat_values.rschar::is_controlnow const-stableCow/ using.contains()— cannot be exploitedstd::chardeprecationrustc 1.97.1 -D warnings: no warning fires today.decoder.rsimportsstd::charredundantly (the inherentchar::from_u32needs no import) — worth removing later, not in this PRbuild.warningsstabilised-D warningsduplication inprek.tomlandci.yml, but it targets rustc warnings rather than clippy lints, so it deserves its own changeClippy under
msrv = "1.97.1"reports no modernisation opportunities, which corroborates the manual pass.Dependency and linter refresh
cargo updateprek autoupdate --cooldown-days 7→ rumdl 0.2.52, ruff 0.16.2, typos 1.49.0, zizmor 1.29.0scripts/update_yamllint_schemastore_snapshot.pyThe
synduplicatecargo updatemoves the build graph to syn 3.x, butsynstructure0.13 still pins syn 2.x, soclippy::cargo'smultiple_crate_versionsfails the build. syn 2.x enters only through thejsonschemadev-dependency:No bump on our side resolves it, so
synjoinsbitflagsinallowed-duplicate-crates, following the existing precedent of whitelisting individual crates rather than disabling the lint.Also included
pymdown-extensions→ 11.0.1. This is the same lockfile change as #411 and exists here only so this branch is independently green; it rebases away once #411 merges.Verification
prek run --all-files— exit 0cargo clippy --all-targets -- -D warnings— cleancargo clippy --all-targets --no-default-features -- -D warnings— cleancargo nextest run --all-features— 1752 passed, 0 skipped