Skip to content

Isolate chat widget harness auto-review state#361

Merged
cbusillo merged 1 commit into
mainfrom
fix/auto-review-snapshot-state-isolation
Jun 3, 2026
Merged

Isolate chat widget harness auto-review state#361
cbusillo merged 1 commit into
mainfrom
fix/auto-review-snapshot-state-isolation

Conversation

@cbusillo
Copy link
Copy Markdown
Owner

@cbusillo cbusillo commented Jun 3, 2026

Summary

  • Gives ChatWidgetHarness its own temporary CODE_HOME and cwd.
  • Prevents snapshot tests from reading repo-local auto-review run state left by live harness sessions.
  • Keeps production restart recovery behavior unchanged.

Validation

  • cargo test --manifest-path code-rs/Cargo.toml -p code-tui --test vt100_chatwidget_snapshot baseline_empty_chat --features test-helpers -- --nocapture
  • cargo test --manifest-path code-rs/Cargo.toml -p code-tui --test vt100_chatwidget_snapshot --features test-helpers -- --nocapture
  • cargo test --manifest-path code-rs/Cargo.toml -p code-tui --test bottom_spacer_clip_regression --features test-helpers -- --nocapture
  • ./build-fast.sh

@cbusillo cbusillo merged commit 9fd2bec into main Jun 3, 2026
1 check passed
@cbusillo cbusillo deleted the fix/auto-review-snapshot-state-isolation branch June 3, 2026 23:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant