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

[echobt]

Replaces PR #26 which had merge conflicts.

This PR replaces the unsafe .unwrap() call in runner.rs with .expect() containing context message for better debugging when the assertion fails.

Closes#26

[greptile-apps]

Greptile Overview

Greptile Summary

replaced unsafe .unwrap() with .expect() containing descriptive context message in the init_client method at src/cortex-exec/src/runner.rs:190-194

• improved error messages for debugging when ModelClient initialization assertion fails
• maintains the same runtime behavior while providing better panic messages
• the change is safe because the client is guaranteed to be initialized in the preceding if block (lines 171-187)

Confidence Score: 5/5

• This PR is completely safe to merge with no risk
• The change is a simple refactoring that replaces .unwrap() with .expect() containing a descriptive error message. This improves error handling without changing runtime behavior or logic. The code is guaranteed to be safe due to the initialization check in the preceding block.
• No files require special attention

Important Files Changed

Filename	Overview
src/cortex-exec/src/runner.rs	replaced .unwrap() with .expect() containing context message in init_client method for better error debugging

Sequence Diagram

sequenceDiagram
participant Caller
participant ExecRunner
participant ModelClient
Caller->>ExecRunner: init_client()
ExecRunner->>ExecRunner: Check if client is None
alt client is None
ExecRunner->>ModelClient: create_client()
ModelClient-->>ExecRunner: Return new client
ExecRunner->>ExecRunner: Set self.client = Some(client)
end
ExecRunner->>ExecRunner: expect("ModelClient should be initialized")
ExecRunner-->>Caller: Return &dyn ModelClient

Loading

[greptile-apps]

_{1 file reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}