Skip to content

fix: defer incomplete Codex turn exports#31

Open
NeuralEmpowerment wants to merge 1 commit into
langfuse:mainfrom
NeuralEmpowerment:fix/completion-gated-turn-export
Open

fix: defer incomplete Codex turn exports#31
NeuralEmpowerment wants to merge 1 commit into
langfuse:mainfrom
NeuralEmpowerment:fix/completion-gated-turn-export

Conversation

@NeuralEmpowerment

Copy link
Copy Markdown

Summary

  • defer a trailing source turn until its completed or aborted lifecycle event is present
  • prevent separate provisional and finalized OTEL trees for one codex.turn_id
  • retain completed-turn sidecar ledger behavior and bump plugin to 0.1.1

Validation

  • pnpm lint
  • pnpm test (36 tests)

Rationale

The Stop hook can read the rollout before Codex appends task_complete. The previous behavior exported that provisional tree without a ledger entry and exported it again later once completed, producing duplicate Langfuse traces/cost/tool data.

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