trim runtime storage compatibility shims

codex-rs/app-server-client/src/lib.rs

@@ -25,7 +25,6 @@ use std::io::Result as IoResult;
 use std::sync::Arc;
 use std::time::Duration;
 
-pub use codex_app_server::AppServerRuntimeStorageDeps;
 pub use codex_app_server::AppServerRuntimeStorageOverrides;
 pub use codex_app_server::ArtifactStore;
 pub use codex_app_server::LocalArtifactStore;
@@ -629,15 +628,6 @@ impl InProcessAppServerClient {
         })
     }
 
-    /// Compatibility wrapper for existing storage-deps callers.
-    #[doc(hidden)]
-    pub async fn start_with_runtime_storage_deps(
-        args: InProcessClientStartArgs,
-        runtime_storage_deps: AppServerRuntimeStorageDeps,
-    ) -> IoResult<Self> {
-        Self::start_with_runtime_storage_overrides(args, runtime_storage_deps).await
-    }
-
     pub fn request_handle(&self) -> InProcessAppServerRequestHandle {
         InProcessAppServerRequestHandle {
             command_tx: self.command_tx.clone(),

codex-rs/app-server/src/in_process.rs

@@ -381,15 +381,6 @@ pub async fn start_with_runtime_storage_overrides(
     Ok(client)
 }
 
-/// Compatibility wrapper for existing storage-deps callers.
-#[doc(hidden)]
-pub async fn start_with_runtime_storage_deps(
-    args: InProcessStartArgs,
-    runtime_storage_deps: crate::AppServerRuntimeStorageDeps,
-) -> IoResult<InProcessClientHandle> {
-    start_with_runtime_storage_overrides(args, runtime_storage_deps).await
-}
-
 async fn start_uninitialized(
     args: InProcessStartArgs,
     runtime_storage_overrides: AppServerRuntimeStorageOverrides,

codex-rs/app-server/src/lib.rs

@@ -431,10 +431,6 @@ pub struct AppServerRuntimeStorageOverrides {
     pub thread_manager: Option<ThreadManagerStorageDeps>,
 }
 
-/// Compatibility alias for existing storage-deps callers.
-#[doc(hidden)]
-pub type AppServerRuntimeStorageDeps = AppServerRuntimeStorageOverrides;
-
 #[allow(clippy::too_many_arguments)]
 pub async fn run_main_with_transport_options(
     arg0_paths: Arg0DispatchPaths,
@@ -1135,36 +1131,6 @@ pub async fn run_main_with_transport_options_and_storage_overrides(
     Ok(())
 }
 
-/// Compatibility wrapper for existing storage-deps callers.
-#[allow(clippy::too_many_arguments)]
-#[doc(hidden)]
-pub async fn run_main_with_transport_options_and_storage_deps(
-    arg0_paths: Arg0DispatchPaths,
-    cli_config_overrides: CliConfigOverrides,
-    loader_overrides: LoaderOverrides,
-    strict_config: bool,
-    default_analytics_enabled: bool,
-    transport: AppServerTransport,
-    session_source: SessionSource,
-    auth: AppServerWebsocketAuthSettings,
-    runtime_options: AppServerRuntimeOptions,
-    runtime_storage_deps: AppServerRuntimeStorageDeps,
-) -> IoResult<()> {
-    run_main_with_transport_options_and_storage_overrides(
-        arg0_paths,
-        cli_config_overrides,
-        loader_overrides,
-        strict_config,
-        default_analytics_enabled,
-        transport,
-        session_source,
-        auth,
-        runtime_options,
-        runtime_storage_deps,
-    )
-    .await
-}
-
 fn analytics_rpc_transport(transport: &AppServerTransport) -> AppServerRpcTransport {
     match transport {
         AppServerTransport::Stdio => AppServerRpcTransport::Stdio,