style(server): collapse nested if conditions via clippy
This commit is contained in:
1 parent
f37a2f8313
commit
9d7cd347c6
1 file changed
+6
-9
@@ -77,12 +77,11 @@ impl McpTool for DeleteTaskHandler {
|
||||
}
|
||||
|
||||
for (idx, t) in tasks.iter().enumerate() {
|
||||
if let Some(pid) = &t.parent_id {
|
||||
if let Some(&parent_idx) = id_to_index.get(pid.as_str()) {
|
||||
if let Some(pid) = &t.parent_id
|
||||
&& let Some(&parent_idx) = id_to_index.get(pid.as_str()) {
|
||||
children_map.entry(parent_idx).or_default().push(idx);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
let mut to_delete_idx = std::collections::HashSet::new();
|
||||
if let Some(&start_idx) = id_to_index.get(req.id.as_str()) {
|
||||
@@ -90,13 +89,12 @@ impl McpTool for DeleteTaskHandler {
|
||||
queue.push_back(start_idx);
|
||||
|
||||
while let Some(curr) = queue.pop_front() {
|
||||
if to_delete_idx.insert(curr) {
|
||||
if let Some(children) = children_map.get(&curr) {
|
||||
if to_delete_idx.insert(curr)
|
||||
&& let Some(children) = children_map.get(&curr) {
|
||||
queue.extend(children.iter().copied());
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
for &idx in &to_delete_idx {
|
||||
actually_deleted.push(tasks[idx].id.clone());
|
||||
@@ -165,12 +163,11 @@ impl McpTool for UpdateTaskStatusHandler {
|
||||
if !blocked {
|
||||
let mut uncompleted_deps = Vec::new();
|
||||
for dep_id in &tasks[target_idx].dependencies {
|
||||
if let Some(dep_task) = tasks.iter().find(|dt| dt.id == *dep_id) {
|
||||
if dep_task.status != "completed" && dep_task.status != "done" {
|
||||
if let Some(dep_task) = tasks.iter().find(|dt| dt.id == *dep_id)
|
||||
&& dep_task.status != "completed" && dep_task.status != "done" {
|
||||
uncompleted_deps.push(dep_task.title.as_str());
|
||||
}
|
||||
}
|
||||
}
|
||||
if !uncompleted_deps.is_empty() {
|
||||
blocked = true;
|
||||
blocker_details = format!("Blocked by dependencies: {}", uncompleted_deps.join(", "));
|
||||
|
||||
Reference in new issue
Block a user