Skip to content

Commit 1fa1453

Browse files
author
Terraphim CI
committed
fix(usage-cli-commands): resolve clippy warnings for CI
- Fix LegacyTerm missing #[derive(Deserialize)] in terraphim_automata - Remove clone_on_copy for u64 IDs in builder.rs - Suppress dead_code warnings on deserialization-only structs (ZaiApiResponse, ZaiLimit, ZaiSubscription, ClaudeProvider) - Remove unused imports (MetricLine, ProgressFormat) from claude.rs - Replace redundant closures with variant constructors in store.rs - Remove unnecessary i64-to-i64 cast in ExecutionRecord::duration_ms - Add #[allow(dead_code)] on unused utility functions Refs #229
1 parent 9c8dd28 commit 1fa1453

10 files changed

Lines changed: 1195 additions & 282 deletions

File tree

Cargo.lock

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

crates/terraphim_automata/src/builder.rs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -159,9 +159,8 @@ fn index_inner(name: String, messages: Vec<Message>) -> Thesaurus {
159159
let concept_with_display = match current_concept {
160160
Some(ref cwd) => {
161161
// Create NormalizedTerm with display_value preserving original case
162-
let nterm =
163-
NormalizedTerm::new(cwd.concept.id.clone(), cwd.concept.value.clone())
164-
.with_display_value(cwd.display_name.clone());
162+
let nterm = NormalizedTerm::new(cwd.concept.id, cwd.concept.value.clone())
163+
.with_display_value(cwd.display_name.clone());
165164
thesaurus.insert(cwd.concept.value.clone(), nterm.clone());
166165
cwd
167166
}
@@ -173,7 +172,7 @@ fn index_inner(name: String, messages: Vec<Message>) -> Thesaurus {
173172
for synonym in synonyms {
174173
// Synonyms also get the same display_value (the concept's original name)
175174
let nterm = NormalizedTerm::new(
176-
concept_with_display.concept.id.clone(),
175+
concept_with_display.concept.id,
177176
concept_with_display.concept.value.clone(),
178177
)
179178
.with_display_value(concept_with_display.display_name.clone());

crates/terraphim_automata/src/lib.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -354,7 +354,9 @@ fn parse_thesaurus_json(contents: &str) -> Result<Thesaurus> {
354354
}
355355

356356
#[derive(Deserialize)]
357+
#[allow(dead_code)]
357358
struct LegacyTerm {
359+
#[allow(dead_code)]
358360
id: u64,
359361
nterm: String,
360362
#[serde(default)]

crates/terraphim_usage/Cargo.toml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,10 @@ which = "6.0"
2020
terraphim_persistence = { path = "../terraphim_persistence", optional = true }
2121
terraphim_types = { path = "../terraphim_types" }
2222
terraphim_settings = { path = "../terraphim_settings" }
23+
opendal = { version = "0.54", optional = true }
2324

2425
[features]
2526
default = ["persistence", "providers"]
26-
persistence = ["dep:terraphim_persistence"] # Enable storage via terraphim_persistence
27+
persistence = ["dep:terraphim_persistence", "dep:opendal"] # Enable storage via terraphim_persistence
2728
cli = ["dep:clap", "dep:colored", "dep:indicatif"] # Enable CLI commands
2829
providers = ["dep:reqwest"] # Enable external provider API fetchers

0 commit comments

Comments
 (0)