Skip to content

test(protocol): preserve provider transport diagnostics - #55

Merged
lyc-aon merged 1 commit into
mainfrom
codex/protocol-transport-diagnostics-sentry
Jul 18, 2026
Merged

test(protocol): preserve provider transport diagnostics#55
lyc-aon merged 1 commit into
mainfrom
codex/protocol-transport-diagnostics-sentry

Conversation

@wolfiesch

Copy link
Copy Markdown
Collaborator

Summary

  • add an app-wire upgrade sentry for additive provider transport diagnostics
  • cover full session lists, direct list decoders, incremental session deltas, and typed list/create responses
  • preserve the complete redacted diagnostic object across every session-bearing decode path

Verification

  • mise exec node@24.17.0 -- pnpm --filter @t4-code/protocol test
  • mise exec node@24.17.0 -- pnpm check
  • mise exec node@24.17.0 -- pnpm test (1,481 tests passed)
  • git diff --check

Copilot AI review requested due to automatic review settings July 18, 2026 19:37

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds regression coverage to ensure provider transport diagnostic evidence (liveState.providerTransport) is preserved across all session-bearing decode paths in the @omp/protocol app-wire facade.

Changes:

  • Introduces a frozen providerTransport diagnostic fixture used across decode tests.
  • Adds tests verifying providerTransport survives decoding through session list frames, session list results, session deltas, and typed response results.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@lyc-aon
lyc-aon merged commit 5682b79 into main Jul 18, 2026
5 checks passed
@lyc-aon
lyc-aon deleted the codex/protocol-transport-diagnostics-sentry branch July 18, 2026 19:56
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.

3 participants