fix: suppress dead code warnings in fly_api.rs and ops_mcp.rs

CI treats warnings as errors. These files contain stub implementations
for future cloud CLI and memory MCP features. Add #[allow(dead_code)]
to unused FlyMachinesClient methods (new, create_machine,
get_deployment_status, get_logs, destroy_machines) and response structs
(CreateMachineResponse, DeploymentStatusResponse, Machine, LogsResponse,
DestroyResponse) in crates/orchestrator-cli/src/services/fly_api.rs.

Also add #[allow(dead_code)] to MemoryMcpServer struct and memory_root
method in crates/orchestrator-cli/src/services/operations/ops_mcp.rs.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: Shooksie

crates/orchestrator-cli/src/services/fly_api.rs

@@ -12,11 +12,13 @@ pub struct FlyMachinesClient {
 }
 
 impl FlyMachinesClient {
+    #[allow(dead_code)]
     pub fn new(api_token: String) -> Self {
         FlyMachinesClient { api_token }
     }
 
     /// Create a new machine on Fly.io
+    #[allow(dead_code)]
     pub async fn create_machine(&self, app_name: &str, region: &str, _image: &str) -> Result<CreateMachineResponse> {
         // This would use the Fly.io GraphQL API to create a machine
         // For now, we return a placeholder response
@@ -29,6 +31,7 @@ impl FlyMachinesClient {
     }
 
     /// Get deployment status from Fly.io
+    #[allow(dead_code)]
     pub async fn get_deployment_status(&self, app_name: &str) -> Result<DeploymentStatusResponse> {
         // This would query the Fly.io API for the current deployment status
         // For now, we return a placeholder response
@@ -41,6 +44,7 @@ impl FlyMachinesClient {
     }
 
     /// Stream logs from a Fly.io machine
+    #[allow(dead_code)]
     pub async fn get_logs(&self, app_name: &str, _lines: Option<usize>, _follow: bool) -> Result<LogsResponse> {
         // This would query the Fly.io logs API
         Ok(LogsResponse {
@@ -50,6 +54,7 @@ impl FlyMachinesClient {
     }
 
     /// Destroy a deployment on Fly.io
+    #[allow(dead_code)]
     pub async fn destroy_machines(&self, app_name: &str) -> Result<DestroyResponse> {
         // This would call the Fly.io API to destroy all machines for the app
         Ok(DestroyResponse { app_name: app_name.to_string(), status: "destroyed".to_string(), machines_destroyed: 0 })
@@ -80,6 +85,7 @@ impl FlyMachinesClient {
     }
 }
 
+#[allow(dead_code)]
 #[derive(Debug, Serialize, Deserialize)]
 pub struct CreateMachineResponse {
     pub id: String,
@@ -88,6 +94,7 @@ pub struct CreateMachineResponse {
     pub app: String,
 }
 
+#[allow(dead_code)]
 #[derive(Debug, Serialize, Deserialize)]
 pub struct DeploymentStatusResponse {
     pub app_name: String,
@@ -96,19 +103,22 @@ pub struct DeploymentStatusResponse {
     pub updated_at: String,
 }
 
+#[allow(dead_code)]
 #[derive(Debug, Serialize, Deserialize)]
 pub struct Machine {
     pub id: String,
     pub status: String,
     pub region: String,
 }
 
+#[allow(dead_code)]
 #[derive(Debug, Serialize, Deserialize)]
 pub struct LogsResponse {
     pub app_name: String,
     pub logs: Vec<String>,
 }
 
+#[allow(dead_code)]
 #[derive(Debug, Serialize, Deserialize)]
 pub struct DestroyResponse {
     pub app_name: String,

crates/orchestrator-cli/src/services/operations/ops_mcp.rs

@@ -212,12 +212,14 @@ fn normalize_non_empty(value: Option<String>) -> Option<String> {
 }
 
 #[derive(Debug, Clone)]
+#[allow(dead_code)]
 struct MemoryMcpServer {
     default_project_root: String,
     tool_router: ToolRouter<Self>,
 }
 
 impl MemoryMcpServer {
+    #[allow(dead_code)]
     fn memory_root(&self) -> std::path::PathBuf {
         protocol::scoped_state_root(std::path::Path::new(&self.default_project_root))
             .unwrap_or_else(|| std::path::PathBuf::from(&self.default_project_root).join(".ao"))