Skip to content

Fix Codex goal continuation transcript handling#874

Open
tpn wants to merge 3 commits into
kenn-io:mainfrom
tpn:868-codex-goal-awareness
Open

Fix Codex goal continuation transcript handling#874
tpn wants to merge 3 commits into
kenn-io:mainfrom
tpn:868-codex-goal-awareness

Conversation

@tpn

@tpn tpn commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Fixes #868.

Classifies Codex goal-continuation wrapper messages as internal/system content during parsing so they no longer persist as ordinary user turns or inflate user-message counts.

Adds read-path fallbacks for legacy stored rows by recognizing both current <codex_internal_context source="goal"> and older <goal_context> wrappers in backend/frontend system-message checks. HTML export now applies the same filtering and keeps focused transcript selection from being affected by legacy goal-context rows.

This bumps the parser data version to 55. Multi-host PostgreSQL deployments should upgrade pg serve and all pushers together, because older agentsview binaries reject PG rows written by a newer parser data version.

@tpn tpn marked this pull request as ready for review June 26, 2026 01:57
@roborev-ci

roborev-ci Bot commented Jun 26, 2026

Copy link
Copy Markdown

roborev: Combined Review (69c4679)

HTML export has one medium-severity visibility regression; security review found no issues.

Medium

  • Location: internal/server/export.go:678
    Problem: The new HTML export filter drops every db.IsSystemPrefixed user message, which also removes promoted Claude boundary rows such as continuations, interruptions, task notifications, and stop-hook feedback that the SPA intentionally keeps visible via source_subtype.
    Fix: Restrict the export filter to the Codex goal-context prefixes, or mirror the frontend visibility rules by preserving visible system subtypes before applying prefix fallback.

Panel: ci_default_security | Synthesis: codex, 6s | Members: codex_default (codex/default, done, 3m55s), codex_security (codex/security, done, 58s) | Total: 4m59s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Codex /goal continuation context renders as user messages in transcripts

1 participant