From 255d7e2615ff9d1e33632af9f56f32f5562cc2b8 Mon Sep 17 00:00:00 2001 From: flemming-it Date: Thu, 18 Jun 2026 12:13:11 +0200 Subject: [PATCH] fix(clippy): bring source under -D warnings ahead of CI MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Prep for the Forgejo CI gate. Adjustments per module are small and local: - test modules get inner `#![allow(clippy::unwrap_used, expect_used, panic)]` so the existing assert.expect() test idiom keeps working without rewriting every fixture - the dead_code field that downstream consumers may still want serialised gets an explicit #[allow(dead_code)] - manual char/range comparisons fold to the idiomatic forms (`['…']`, `(2..=5).contains(&n)`) - one snake_case rename in text-readability-score Also re-bakes module.wasm so the committed artefact matches the post-fmt source byte-for-byte. No behaviour change, no test change. cargo fmt --all -- --check and cargo clippy --all-targets -- -D warnings now both pass. Signed-off-by: flemming-it --- src/lib.rs | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/lib.rs b/src/lib.rs index c7de4a2..a3fd87d 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -49,7 +49,7 @@ pub fn invoke(_ctx: Context, inputs: Inputs) -> Result { .require_json("norm") .map_err(|e| ModuleError::invalid_input(format!("norm shape: {e}")))?; - let graph_iVm: Option = inputs.get("graph").and_then(|p| match p { + let graph_ivm: Option = inputs.get("graph").and_then(|p| match p { Payload::Json(s) => serde_json::from_str(s).ok(), _ => None, }); @@ -62,7 +62,7 @@ pub fn invoke(_ctx: Context, inputs: Inputs) -> Result { .join(" "); let flesch = flesch_de(&full_text); - let depth = depth_index(graph_iVm.as_ref()); + let depth = depth_index(graph_ivm.as_ref()); let composite = composite_frust(flesch, depth); let out = Output { @@ -89,7 +89,10 @@ fn flesch_de(text: &str) -> f32 { } fn count_sentences(text: &str) -> usize { - text.chars().filter(|c| matches!(c, '.' | '!' | '?')).count().max(1) + text.chars() + .filter(|c| matches!(c, '.' | '!' | '?')) + .count() + .max(1) } fn count_syllables(word: &str) -> usize { @@ -134,6 +137,7 @@ fn composite_frust(flesch: f32, depth: f32) -> f32 { #[cfg(test)] mod tests { + #![allow(clippy::unwrap_used, clippy::expect_used, clippy::panic)] use super::*; #[test]