From 3da6c9e1eed66d3b3f074f484ae1ffac7054a660 Mon Sep 17 00:00:00 2001 From: Adam Perry Date: Sun, 14 Jun 2026 01:46:40 +0000 Subject: [PATCH] tests: split remote environment follow-up TODOs --- .../tests/remote_env_windows/remote_env_windows_test.rs | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/codex-rs/core/tests/remote_env_windows/remote_env_windows_test.rs b/codex-rs/core/tests/remote_env_windows/remote_env_windows_test.rs index ede1bac933..0237c38b30 100644 --- a/codex-rs/core/tests/remote_env_windows/remote_env_windows_test.rs +++ b/codex-rs/core/tests/remote_env_windows/remote_env_windows_test.rs @@ -266,11 +266,13 @@ async fn app_server_starts_thread_with_windows_environment_native_cwd() -> Resul let response: ThreadStartResponse = to_response(response)?; assert!(!response.thread.id.is_empty()); let host_cwd = codex_home.path().to_path_buf().abs(); - // TODO(anp): Return environment-native values here once these app-server response - // fields support foreign paths and remote instruction/profile discovery. + // TODO(anp): Return the selected environment's native cwd from thread/start. assert_eq!(response.cwd, host_cwd); + // TODO(anp): Derive runtime workspace roots from the selected remote environment. assert_eq!(response.runtime_workspace_roots, vec![host_cwd]); + // TODO(anp): Discover and report instruction sources from the remote filesystem. assert_eq!(response.instruction_sources, Vec::new()); + // TODO(anp): Report the implicit built-in permission profile instead of None. assert_eq!(response.active_permission_profile, None); let turn_request_id = app_server