fix: ensure AgentRun workspace root before hydration

[guslegend0510]

Fixes #2216

Summary

• ensure AgentRun workspace root is created before hydration
• fail fast when workspace root setup returns a non-zero exit code
• add AgentRun sandbox tests for setup failure and hydration ordering

Tests

• mvn -pl agentscope-extensions/agentscope-extensions-sandbox/agentscope-extensions-sandbox-agentrun test

[AgentScopeJavaBot]

🤖 AI Review

This PR fixes issue #2216 where AgentRun sandbox hydration could fail with a tar extraction error because the workspace root directory was never created beforehand. The fix implements doSetupWorkspace() with a real mkdir -p + exit-code check, adds defensive ensureWorkspaceRoot() inside doHydrateWorkspace() for callers that bypass start(), and introduces an AgentRunExecutor inner interface to decouple AgentRunSandbox from the concrete AgentRunMcpChannel, enabling unit tests. The change is focused, well-tested, and consistent with patterns used in DaytonaSandbox and E2bSandbox.

(inline comments could not be attached — line numbers fell outside PR hunks. See archived report.)

[AgentScopeJavaBot]

🤖 AI Review

This PR fixes issue #2216 where AgentRun sandbox hydration could fail with a tar extraction error because the workspace root directory was never created beforehand. The fix implements doSetupWorkspace() with a real mkdir -p + exit-code check, adds defensive ensureWorkspaceRoot() inside doHydrateWorkspace() for callers that bypass start(), and introduces an AgentRunExecutor inner interface to decouple AgentRunSandbox from the concrete AgentRunMcpChannel, enabling unit tests. The change is focused, well-tested, and consistent with patterns used in DaytonaSandbox and E2bSandbox.

(inline comments could not be attached — line numbers fell outside PR hunks. See archived report.)

[codecov]

Codecov Report

❌ Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
...e/extensions/sandbox/agentrun/AgentRunSandbox.java	85.71%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!