Michael BolinandGitHub aef4af1079 app-server tests: disable shell_snapshot for review suite (#11657) ...

## Why


`suite::v2::review::review_start_with_detached_delivery_returns_new_thread_id`
was failing on Windows CI due to an unrelated process crash during shell
snapshot initialization (`tokio-runtime-worker` stack overflow).

This review test suite validates review API behavior and should not
depend on shell snapshot behavior. Keeping shell snapshot enabled in
this fixture made the test flaky for reasons outside the scenario under
test.

## What Changed

- Updated the review suite test config in
`codex-rs/app-server/tests/suite/v2/review.rs` to set:
  - `shell_snapshot = false`

This keeps the review tests focused on review behavior by disabling
shell snapshot initialization in this fixture.

## Verification

- `cargo test -p codex-app-server`
- Confirmed the previously failing Windows CI job for this test now
passes on this PR.

2026-02-12 23:56:43 +00:00

..

common

app-server: add fuzzy search sessions for streaming file search (#10268)

2026-02-12 10:49:44 -08:00

suite

app-server tests: disable shell_snapshot for review suite (#11657)

2026-02-12 23:56:43 +00:00

all.rs

fix: separate codex mcp into codex mcp-server and codex app-server (#4471)

2025-09-30 07:06:18 +00:00