Skip to content

Migrate R write_stdin cases to external runner#128

Merged
t-kalinowski merged 3 commits into
mainfrom
migrate-r-write-stdin-api-tests
Jun 19, 2026
Merged

Migrate R write_stdin cases to external runner#128
t-kalinowski merged 3 commits into
mainfrom
migrate-r-write-stdin-api-tests

Conversation

@t-kalinowski

@t-kalinowski t-kalinowski commented Jun 18, 2026

Copy link
Copy Markdown
Member

Summary

Move a cohesive R write_stdin public-behavior cluster from Rust integration tests into the external Python public API runner.

This keeps the external runner migration moving by verifying the real mcp-repl binary over MCP stdio while removing duplicate Rust coverage for the same public scenarios.

User Facing Changes

  • No runtime behavior changes are expected.
  • Public API coverage now exercises more R write_stdin behavior through the same external runner used in CI.

Internal Changes

  • Added external runner cases for multiple calls, timeout polling, error recovery, huge echo-only input, and files-mode echo trimming.
  • Removed matching Rust integration tests and now-unused helper code from tests/write_stdin_batch.rs.
  • Stabilized two Windows prepared-launch sandbox tests by using the existing guarded temp-root helper for session temp setup.
  • Updated the active public API runner plan with the migrated cluster.

@t-kalinowski t-kalinowski marked this pull request as ready for review June 19, 2026 02:19
@t-kalinowski t-kalinowski merged commit 98c9c3b into main Jun 19, 2026
5 checks passed
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