fix(e2e): fix messaging-providers and sandbox-survival nightly E2E regressions

[ericksoa]

Summary

• messaging-providers-e2e: The tier-based policy selector (feat(policy): tier-based policy selector with combined access-mode UI #1753) defaults non-interactive onboard to balanced, which excludes messaging presets. Adds NEMOCLAW_POLICY_TIER: "open" to the workflow env.
• sandbox-survival-e2e: SSH after gateway restart can fail because the sandbox SSH agent isn't immediately reachable when the gateway reports healthy (more noticeable with OpenClaw 2026.4.2). Adds a 30s retry loop for SSH connectivity. Also removes cleanup_ssh() from the failure path — deleting the SSH config was causing all subsequent phases to fail with "Can't open user config file", masking whether data actually persisted.

Test plan

• Nightly E2E messaging-providers job passes
• Nightly E2E sandbox-survival job passes (or produces real diagnostics if data is actually lost)

🤖 Generated with Claude Code

Summary by CodeRabbit

• Chores

  • Nightly end-to-end test environment configuration updated to set the test policy tier.

• Tests

  • End-to-end test flow now retries SSH connectivity multiple times and preserves connection data on failure, improving test stability and enabling richer diagnostic logs.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 136758f3-713e-470a-bbbb-a829f8729730

📥 Commits

Reviewing files that changed from the base of the PR and between 3efe7cb and 10c0b63.

📒 Files selected for processing (1)

• test/e2e/test-sandbox-survival.sh

---

📝 Walkthrough

Walkthrough

Added an environment variable NEMOCLAW_POLICY_TIER: "open" to the nightly messaging-providers-e2e job and replaced a single SSH connectivity check after gateway restart with a retry loop that attempts SSH up to 6 times, preserving SSH config and collecting handshake logs on failure.

Changes

Cohort / File(s)	Summary
Workflow Configuration .github/workflows/nightly-e2e.yaml	Added environment variable NEMOCLAW_POLICY_TIER: "open" to the messaging-providers-e2e job.
E2E Test Script test/e2e/test-sandbox-survival.sh	Replaced single-attempt SSH check with a retry loop (up to 6 attempts, 5s sleep). On failure, preserves SSH config and gathers handshake-related logs instead of calling cleanup_ssh.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 I hopped through scripts with care,
Adding retries into the air,
A policy set, a gentle nudge,
Logs kept safe, no sudden trudge,
Tests and workflows — a joyful pair 🥕

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly addresses the main changes: fixing E2E regressions in messaging-providers (via NEMOCLAW_POLICY_TIER) and sandbox-survival (via SSH retry logic).
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/e2e-messaging-tier

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}