weiqingy commented on issue #723: URL: https://github.com/apache/flink-agents/issues/723#issuecomment-4666992203
Thanks @wenjin272 for assigning this, and @joeyutong for confirming the sequencing. I've opened the first PR: apache/flink-agents#828. **PR #1 (#828) — built-in tool-context fix (this PR):** normalizes the framework-owned tool-context payloads in `chat_model_action.py` to a primitive-only form before they reach memory, reconstructing the rich types on read — fully inside the existing helpers, no API change. Covers the three offenders Pemja would wrap and crash on restore: `ChatMessage` lists, `initial_request_id`, and `output_schema`. `prompt_args` is user-supplied and left to PR #2. **PR #2 (after #828 merges):** 1. A `set()`-time validator that rejects non-checkpoint-stable values early with a clear error, covering `prompt_args` and arbitrary user values. 2. Docs for the Python memory value contract and why it diverges from the Java side (values must materialize across Pemja first). Kept the built-in fix first since it's self-contained and independent of the validator's behavior change. Reviews on #828 welcome. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
