fix(agentrun): unwrap data envelopes

[guslegend0510]

AgentScope-Java Version

2.0.0-SNAPSHOT

Description

This PR fixes AgentRun data-plane JSON handling when successful responses are wrapped in a data envelope.

Background:

• AgentRunDataPlaneHttp previously parsed response JSON from the root node.
• AgentRun create/get responses return payloads under data, so fields like id and status were not visible to the client code.

Changes made:

• Added unwrapData(...) in AgentRunDataPlaneHttp and applied it in getJson() and postJson().
• Updated AgentRunDataPlaneHttpTest to mock wrapped responses and verify parsing still works.

Related issues:

• Closes [bug] AgentRunDataPlaneHttp 响应 JSON data 信封未解包 #1907
• Related to [Bug]: AgentRunDataPlaneHttp 使用错误的 API 版本前缀 /2025-09-10 导致 "Invalid request uri" #1889

How to test:

• mvn -pl agentscope-extensions/agentscope-extensions-sandbox/agentscope-extensions-sandbox-agentrun -am -Dtest=AgentRunDataPlaneHttpTest test

[codecov]

Codecov Report

❌ Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
...nsions/sandbox/agentrun/AgentRunDataPlaneHttp.java	60.00%	1 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

[itxaiohanglover]

Nice fix — the unwrapData helper is clean and the null/hasNonNull guard is good. One thing to keep an eye on: if any legitimate response body has a top-level data field that isn't an envelope, this would strip it. The test covering responses without the envelope is a nice touch for backward compat. Looks good overall.

[itxaiohanglover]

Clean fix! Using hasNonNull("data") is more concise than the issue suggestion. Applying unwrap in both getJson and postJson covers all entry points. Test with both wrapped and unwrapped responses is good for backward compatibility.