Skip to content

feat: add continuation diagnostics - #8

Merged
HisenWeb merged 3 commits into
mainfrom
p1-continuation-logs
Jun 24, 2026
Merged

feat: add continuation diagnostics#8
HisenWeb merged 3 commits into
mainfrom
p1-continuation-logs

Conversation

@HisenWeb

Copy link
Copy Markdown
Owner

Summary

Adds P1-4 diagnostics around Codex tool-result continuation and stored-state recovery.

This is logging/documentation-only. It does not change protocol behavior, state lookup rules, fallback behavior, or Responses output shape.

Changes

  • Logs stateless_tool_history_bypass_state_lookup when full stateless history bypasses stored-state lookup.
  • Logs stored_response_not_found when a requested stored response ID is missing from non-expired state.
  • Logs tool_history_unique_fallback_hit when continuation is recovered via unique pending call_id fallback.
  • Logs tool_history_call_id_ambiguous and tool_history_response_ambiguous when fallback is refused due ambiguity.
  • Logs tool_history_call_id_not_found when fallback cannot match all requested call IDs.
  • Adds docs/p1-continuation-diagnostics.md with event names and operational interpretation.

Validation

Edited via GitHub connector. Please run locally before merge:

cargo fmt --check
cargo test --lib
cargo test

Notes

I intentionally avoided modifying server.rs; diagnostics were added in the narrow conversion/state layers that own the relevant decisions.

@HisenWeb
HisenWeb marked this pull request as ready for review June 24, 2026 23:24
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@HisenWeb
HisenWeb merged commit 8c72d27 into main Jun 24, 2026
1 check passed
@HisenWeb
HisenWeb deleted the p1-continuation-logs branch June 24, 2026 23:24
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