Fix clippy lints, dead-code warnings, and refresh quality budgets#326
Merged
Conversation
- Remove unused_mut (input.rs, turborag.rs) - Gate test-only functions with #[cfg(test)] (hash_window, active_at_token, suggest_at_path) - Fix ClipboardCommand visibility (private_interfaces warning) - Allow dead_code on AtPicker public API stubs kept for upcoming wiring - Fix unused variable _end in test assertions - Apply cargo clippy --fix for needless_borrow, manual_char_comparison, etc. - Fix manual_clamp in acp.rs - Fix doc_overindented_list_items in args.rs, terminal.rs - Replace vec_init_then_push with vec![] literal in export.rs - Allow too_many_arguments, enum_variant_names on anthropic.rs internals - Refresh code_size, test_size, panic, swallowed_error budget baselines
- Add #[allow(clippy::await_holding_lock)] to test fns that hold lock_test_env() across await (intentional test serialization) - Fix if_same_then_else in state_ui_input_helpers.rs (simplify redundant conditional to direct assignment)
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.
Summary
Fixes all clippy errors and compiler warnings blocking Quality Guardrails CI on master. Refreshes ratcheting budget baselines.
Changes:
9 await_holding_lock clippy lints remain in test code (follow-up).
Review & Testing Checklist for Human
Notes