From da83019d85d6182c81c6e3d8b15c496628c0c228 Mon Sep 17 00:00:00 2001
From: Taha Hawa <taha@taha.dev>
Date: Wed, 23 Apr 2025 14:53:36 -0400
Subject: [PATCH] remove need for debug in harmony-cli

---
 harmony_cli/src/lib.rs | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/harmony_cli/src/lib.rs b/harmony_cli/src/lib.rs
index 3d1bd72..9ecb8fc 100644
--- a/harmony_cli/src/lib.rs
+++ b/harmony_cli/src/lib.rs
@@ -42,7 +42,7 @@ pub struct Args {
     list: bool,
 }
 
-fn maestro_scores_filter<T: Topology + std::fmt::Debug + Send + Sync + 'static>(
+fn maestro_scores_filter<T: Topology>(
     maestro: &harmony::maestro::Maestro<T>,
     all: bool,
     filter: Option<String>,
@@ -71,9 +71,7 @@ fn maestro_scores_filter<T: Topology + std::fmt::Debug + Send + Sync + 'static>(
 }
 
 // TODO: consider adding doctest for this function
-fn list_scores_with_index<T: Topology + std::fmt::Debug + Send + Sync + 'static>(
-    scores_vec: &Vec<Box<dyn Score<T>>>,
-) -> String {
+fn list_scores_with_index<T: Topology>(scores_vec: &Vec<Box<dyn Score<T>>>) -> String {
     let mut display_str = String::new();
     for (i, s) in scores_vec.iter().enumerate() {
         let name = s.name();
@@ -82,7 +80,7 @@ fn list_scores_with_index<T: Topology + std::fmt::Debug + Send + Sync + 'static>
     return display_str;
 }
 
-pub async fn init<T: Topology + std::fmt::Debug + Send + Sync + 'static>(
+pub async fn init<T: Topology + Send + Sync + 'static>(
     maestro: harmony::maestro::Maestro<T>,
     args_struct: Option<Args>,
 ) -> Result<(), Box<dyn std::error::Error>> {
@@ -293,8 +291,6 @@ mod test {
 
         let res = crate::maestro_scores_filter(&maestro, false, None, 0);
 
-        println!("{:#?}", res);
-
         assert!(res.len() == 1);
 
         assert!(
@@ -311,8 +307,6 @@ mod test {
 
         let res = crate::maestro_scores_filter(&maestro, false, None, 11);
 
-        println!("{:#?}", res);
-
         assert!(res.len() == 0);
     }
 }