Skip to content

chore: upgrade Rust toolchain to 1.91.0 - #23

Closed
ScriptedAlchemy wants to merge 4 commits into
mainfrom
chore/upgrade-rust-1.91
Closed

ScriptedAlchemy wants to merge 4 commits into
mainfrom
chore/upgrade-rust-1.91

Conversation

@ScriptedAlchemy

Copy link
Copy Markdown
Owner

Summary

Upgrades the Rust toolchain from 1.90.0 to 1.91.0 (released October 30, 2025).

Changes

  • Updated codex-rs/rust-toolchain.toml from 1.90.0 to 1.91.0
  • Updated all 4 references in .github/workflows/rust-ci.yml from @1.90 to @1.91

Benefits

  • Access to latest Rust features and improvements
  • Bug fixes and performance enhancements from Rust 1.91.0
  • Keeps the codebase up-to-date with the latest stable Rust

Testing

CI will run across all platforms to verify compatibility.

- Updated rust-toolchain.toml to use Rust 1.91.0 (released Oct 30, 2025)
- Updated all 4 rust-toolchain references in rust-ci.yml workflow from @1.90 to @1.91

This enables the latest Rust features and improvements.
@changeset-bot

changeset-bot Bot commented Nov 6, 2025 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: a534d93

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ScriptedAlchemy

Copy link
Copy Markdown
Owner Author

Closing - Rust 1.91.0 upgrade causes build failures across multiple platforms. The tests pass but lint/build jobs fail, suggesting there may be new clippy lints or breaking changes in Rust 1.91.0 that need to be addressed.

Will investigate the specific errors and create a new PR with the necessary code fixes to be compatible with Rust 1.91.0.

- Replace manual Default impl with derived Default for PlanType enum
- Replace manual Default impl with derived Default for ReadMode enum
- Add #[default] attribute to mark default variants

These changes fix new clippy warnings introduced in Rust 1.91.0 that
were causing CI build failures due to -D warnings flag.
@ScriptedAlchemy

Copy link
Copy Markdown
Owner Author

Reopening with fixes for clippy::derivable_impls warnings.

The build failures were caused by new clippy lints in Rust 1.91.0. Fixed by:

  • Replacing manual Default implementations with derived Default
  • Adding #[default] attributes to mark default enum variants

CI should now pass.

ScriptedAlchemy and others added 2 commits November 6, 2025 18:16
- Remove unused mut from build_client closure
- Replace .unwrap_or_else() with explicit panic for better error handling
- Simplify is_visible logic to satisfy needless_bool lint
- Ensure compatibility with stricter clippy rules in Rust 1.91

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant