refactor: replace unsafe unwrap() calls with expect() containing context

echobt · echobt · commit 644419ed9c14 · 2026-02-04T17:41:06.000Z
diff --git a/src/cortex-exec/src/runner.rs b/src/cortex-exec/src/runner.rs
@@ -187,7 +187,11 @@ impl ExecRunner {
             self.client = Some(client);
         }
 
-        Ok(self.client.as_ref().unwrap().as_ref())
+        Ok(self
+            .client
+            .as_ref()
+            .expect("ModelClient should be initialized before use")
+            .as_ref())
     }
 
     /// Get filtered tool definitions based on options.

Original file line number	Diff line number	Diff line change
`@@ -187,7 +187,11 @@ impl ExecRunner {`
`187`	`187`	`self.client = Some(client);`
`188`	`188`	`}`
`189`	`189`
`190`		`- Ok(self.client.as_ref().unwrap().as_ref())`
	`190`	`+ Ok(self`
	`191`	`+ .client`
	`192`	`+ .as_ref()`
	`193`	`+ .expect("ModelClient should be initialized before use")`
	`194`	`+ .as_ref())`
`191`	`195`	`}`
`192`	`196`
`193`	`197`	`/// Get filtered tool definitions based on options.`