fix(clippy): clear the workspace clippy failures on #707 - #719
Merged
ScriptedAlchemy merged 6 commits intoAug 24, 2026
Merged
Conversation
CI clippy has been red on this branch across roughly ten commits. Groups publish_verified_inner's three mode parameters into a struct rather than passing eight positionally, and clears field-reassign-after-default, redundant clone-to-slice, redundant deref, and a no-op drop of a type with no Drop impl. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Names the retired-owner return shape, iterates the owner map by keys, collapses a nested if, elides two needless lifetimes, backticks two doc identifiers, and states the idle recheck interval in minutes. cargo clippy --workspace --all-targets --locked -- -D warnings is clean. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
…plan-reopened' into claude/fix-ci-clippy
The streaming publication path put three 64 KiB scratch buffers on the stack, four times clippy's large-stack-array ceiling. They are I/O scratch, so the heap is the right home and no frame pays for them. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…plan-reopened' into claude/fix-ci-clippy
ScriptedAlchemy
merged commit Aug 24, 2026
29d8866
into
codex/tracedecay-total-redesign-plan-reopened
4 checks passed
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.
What this fixes
Clippyhas been failing on #707 continuously — it failed one2430e639, was not re-attempted while the light 8-check gate ran, and reproduced identically when the wide run fanned out again onfac184df5. Roughly ten commits landed in between without it being addressed, because the intervening runs never executed it.Reproduced locally with the exact CI command (
cargo clippy --workspace --all-targets --locked -- -D warnings) in an isolated worktree, so nothing touched the shared checkout's in-progress files.The one that needed a judgement call
publish_verified_innerwas at 8 arguments (7 allowed), with three of them a manifest option plus two adjacent bools. Rather than raise the threshold or#[allow]it, the three mode choices are grouped intoGraphPublishModeV1. That also fixes the readability problem the lint is pointing at:..., false, true)at a call site said nothing about which knob was which, and the four call sites now name each field.The rest
writer/worker/mod.rsDefault::default()(the WAL-budget test)codex/tests.rs×2CodexReplayIndexlcm/gc/tests.rs×2clone()to build a slice — usesslice::from_reflcm/query.rssession_temporal/projection/tests.rsdrop()on a type with noDropimpldatabase_owner_registry.rs.iter().map(|(k,_)|…)→.keys()code_index_scheduler/tests.rsifdashboard.rs×2bootstrap.rs,invocation_state.rssession_temporal_refresh_scheduler/worker.rsfrom_secs(60)→from_mins(1)No
#[allow]was added anywhere.Verification
cargo clippy --workspace --all-targets --locked -- -D warnings— clean.Note:
Hawk,DashboardandBuild Windows testsalso fail on the same wide run and are not addressed here. They are separate failures needing their own fixes.🤖 Generated with Claude Code