Cleanup and simplification

mos/src/debugger/mod.rs

@@ -578,12 +578,11 @@ impl CompletionsRequestHandler {
 }
 
 impl Handler<CompletionsRequest> for CompletionsRequestHandler {
-    // TODO: prettify the signature. Why is CompletionsResponse not allowed?
     fn handle(
         &self,
         conn: &mut DebugSession,
         args: <CompletionsRequest as Request>::Arguments,
-    ) -> MosResult<<CompletionsRequest as Request>::Response> {
+    ) -> MosResult<CompletionsResponse> {
         let registers = {
             let adapter = conn.machine_adapter()?;
             adapter.registers()?
@@ -601,8 +600,6 @@ impl Handler<CompletionsRequest> for CompletionsRequestHandler {
             }
         };
 
-        println!("Completion string: {}", curr_completion);
-
         // get the completion candidates
         let candidates: Vec<String> = if curr_completion.starts_with("cpu.flags.") {
             let (_, after_dot_current) = curr_completion.split_at(10);
@@ -1060,9 +1057,6 @@ mod tests {
     fn completions() -> MosResult<()> {
         use mos_testing::assert_unordered_eq;
 
-        // TODO: completing in the middle of the word should probably not return the entire word?
-        //       or should it?
-
         let src = r#".test "a" {
                          ldx #123
                          foo: nop

mos/src/debugger/types.rs

@@ -770,7 +770,6 @@ pub struct CompletionsResponse {
 #[derive(Debug, Serialize, Deserialize, Clone)]
 #[serde(rename_all = "camelCase")]
 pub struct CompletionItem {
-    // TODO: do we need the other fields for our use case?
     pub label: String,
 }