Skip to content

fix(storage): preserve developer messages in session snapshots - #260

Closed
KaiEureka wants to merge 1 commit into
mainfrom
codex/fix-audit-developer-snapshots-20261025
Closed

KaiEureka wants to merge 1 commit into
mainfrom
codex/fix-audit-developer-snapshots-20261025

Conversation

@KaiEureka

Copy link
Copy Markdown
Collaborator

A valid Responses session containing developer messages could complete a model turn but fail during autosave or restore because the session store rejected that role.

Include developer in the existing role allowlist and apply the same text-content validation used for system and user messages. Preserve its original role through snapshots, journals, checkpoints, legacy JSON and JSONL, exports and subsequent model requests.

The new regressions passed all 48 cases and related persistence tests passed all 204 cases. The native SDK before-and-after probe verifies autosave, restore, a second user turn, cumulative usage and session identity. Unknown roles and malformed message content remain rejected. Repository Ruff, import constraints and dependency checks passed.

Validate developer history as instruction content during snapshot and journal loading. Preserve its role through autosave, exports, and resumed Responses requests.

Cover native SDK continuation, saved identity and usage, legacy formats, and malformed message rejection.
@KaiEureka KaiEureka closed this Oct 25, 2026
@KaiEureka
KaiEureka deleted the codex/fix-audit-developer-snapshots-20261025 branch October 25, 2026 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant