Replies: 1 comment
-
|
@thezzisu I totally have during the session if it is a session that was originally used through my app i use my app as the source of truth if it is a session i'm replaying that was not from my app I just inget all of the events, I figure all of the events with the exception of usage which I don't want two of are mostly harmless and I"m sure I'll have enough data to figure out how to de-dup them. |
Beta Was this translation helpful? Give feedback.
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.
Uh oh!
There was an error while loading. Please reload this page.
-
Feedback on ACP Session Loading: Metadata Persistence and State Semantics
Hi everyone,
While implementing session loading as per the ACP Protocol Spec, I’ve encountered a few challenges regarding how agents handle the
session/loadRPC response. I’d love to get your thoughts on the following:1. Missing Session Metadata in Responses
According to the spec, the agent can respond with
result: null. However, I've noticed some ACP Agents fail to echo back themode,model, andconfigOptionsduring the load process.session/load.2. Inconsistent Config Options
I've observed some strange behavior in
codex-acp 0.10.0andcodex-cli 0.117.0. When loading a session, the returned configurable options seem inconsistent with the expected session state.mergesemantics instead ofreplacesemantics.session/loadshould fully synchronize the state. Usingmergeleads to stale or orphaned configuration options.Questions for the Community:
session/loadshould strictly follow replace semantics?Looking forward to your insights and seeing how others are handling these edge cases!
transcript_session_new.json
transcript_session_load.json
JSONND Transcript files for issue 2
Update: After upgraded codex to 0.117.0 issue 2 seems to be fixed. But still, I think there is improvement space for the session/load spec.
Beta Was this translation helpful? Give feedback.
All reactions