fix(harness): preserve explicit sandbox snapshot specs

[March-77]

AgentScope-Java Version

2.0.1-SNAPSHOT

Description

Background

Filesystem-specific snapshotSpec(...) overrides stored the value only in each subclass. The base SandboxFilesystemSpec.snapshotSpecOverride therefore remained null, allowing HarnessAgent to replace an explicit filesystem snapshot configuration with the DistributedStore default.

Changes

• Synchronize the base explicit snapshot override from all five production covariant setters: Docker, Daytona, Kubernetes, AgentRun, and E2B.
• Preserve each subclass's existing snapshot field and fluent return type.
• Add a harness regression test proving an explicit LocalSnapshotSpec takes precedence over a distributed store's NoopSnapshotSpec.
• Add module-local regression tests for AgentRun, Daytona, E2B, and Kubernetes so each covariant setter is directly covered.

Fixes #2254

Testing

Passed:

mvn -pl agentscope-harness -Dtest=HarnessAgentDistributedSandboxTest test
mvn -pl agentscope-extensions/agentscope-extensions-sandbox/agentscope-extensions-sandbox-daytona,agentscope-extensions/agentscope-extensions-sandbox/agentscope-extensions-sandbox-kubernetes,agentscope-extensions/agentscope-extensions-sandbox/agentscope-extensions-sandbox-agentrun,agentscope-extensions/agentscope-extensions-sandbox/agentscope-extensions-sandbox-e2b test
mvn -pl agentscope-harness,agentscope-extensions/agentscope-extensions-sandbox/agentscope-extensions-sandbox-daytona,agentscope-extensions/agentscope-extensions-sandbox/agentscope-extensions-sandbox-kubernetes,agentscope-extensions/agentscope-extensions-sandbox/agentscope-extensions-sandbox-agentrun,agentscope-extensions/agentscope-extensions-sandbox/agentscope-extensions-sandbox-e2b spotless:check
mvn -pl agentscope-harness,agentscope-extensions/agentscope-extensions-sandbox/agentscope-extensions-sandbox-daytona,agentscope-extensions/agentscope-extensions-sandbox/agentscope-extensions-sandbox-kubernetes,agentscope-extensions/agentscope-extensions-sandbox/agentscope-extensions-sandbox-agentrun,agentscope-extensions/agentscope-extensions-sandbox/agentscope-extensions-sandbox-e2b -DskipTests package

The four extension modules now pass 52 tests in total (24 Kubernetes, 17 AgentRun, 1 Daytona, and 10 E2B).

A full agentscope-harness module test run executed 649 tests with 0 assertion failures, but Maven reported 5 unrelated Windows JUnit temporary-directory cleanup errors (DirectoryNotEmptyException).

Checklist

Please check the following items before code is ready to be reviewed.

• Code has been formatted with mvn spotless:apply
• All tests are passing (mvn test)
• Javadoc comments are complete and follow project conventions (no new public API; inherited method documentation remains applicable)
• Related documentation has been updated (not applicable; no documented behavior or API surface changed)
• Code is ready for review (opened as a draft)

[CLAassistant]

All committers have signed the CLA.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[oss-maintainer]

CLA Not Signed

The Contributor License Agreement (CLA) check is currently pending on this PR (license/cla: Contributor License Agreement is not signed yet.). This PR cannot be merged until the CLA is signed.

@March-77 please sign the CLA via the CLA assistant badge in the comment above, or visit https://cla-assistant.io/agentscope-ai/agentscope-java. Once signed, the license/cla status will turn green.

---

Automated check by github-manager-bot