Replies: 2 comments
|
+1, with an embedder's data point and the workaround we ended up shipping. We drive the SDK runtime as one of several agent runtimes inside a host Two additions to the report:
Server-side load/resume of the persisted log (as proposed here) would let us |
0 replies
|
+1 This is critical for me as well. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Summary
deepseek-harness-sdk0.1.0rc6 cannot resume a persisted session from a fresh runtime process. The firstrun()on a session works, but any laterrun()with the samesession_idin a new process (e.g., after the host process restarts, or when a server runs multiple workers and the next turn lands on a different worker) ends immediately with an empty response:Expected behavior
A persisted session should be resumable from a new process. The runtime already implements
session/load/session/resume(ACP methods), so a subsequent prompt should continue the conversation with the restored context instead of failing.Actual behavior
assistant/chunkdeltas and a finalassistant/messageare produced.session_idfrom a new runtime process: immediatelyturn/endwithreason.kind == "error"and the "id collision" message above. Noassistant/chunkevents;final_responseis empty;finish_reason == "error".run()calls with the samesession_idin the same runtime process: both succeed (the in-memory session object is reused).session/load(orsession/resume) directly viaharness.client.request(...)fails withJsonRpcError: unknown DeepSeek Harness SDK runtime method: session/load— the SDK JSON-RPC server only acceptsinitialize,session/prompt, andshutdown.Reproduction
Environment:
deepseek-harness-sdk==0.1.0rc6+ bundled runtime (deepseek-harness-runtime-bin==0.1.0rc6), modeldeepseek-v4-flashvia the official provider, Python 3.12.Impact
Suggested fix
session/load(orsession/resume) through the SDK JSON-RPC server — the runtime already implements these ACP methods; orsession/promptadopt an existing persisted log when the requested session is not already live (treat "load/resume it instead of creating" as the default for existing sessions).Environment
All reactions