Skip to content

test: cover litellm proxy hooks#171

Merged
rlippmann merged 1 commit into
mainfrom
codex/codex-gap-check2
Jun 4, 2026
Merged

test: cover litellm proxy hooks#171
rlippmann merged 1 commit into
mainfrom
codex/codex-gap-check2

Conversation

@rlippmann

Copy link
Copy Markdown
Owner

What changed

  • add hermetic tests for the LiteLLM proxy hook examples in tests/test_litellm_proxy_hooks.py
  • cover unsupported call types, clarify blocking behavior, system-contract injection, mixed-content transcript extraction, and the replay-input versus forwarded-message boundary in the preprocessor hook
  • protect the rule that preprocessor drafting may normalize replay input for compiler evaluation but must not rewrite the request messages forwarded upstream

Why

Refs #111.

This adds hermetic coverage for the LiteLLM proxy-hook behavior identified in #111. Runtime smoke/E2E coverage remains out of scope.

Checklist

  • pre-commit run (uv run pre-commit run --all-files)
  • tests pass (uv run pytest)

@rlippmann rlippmann merged commit 3b3218d into main Jun 4, 2026
14 checks passed
@rlippmann rlippmann deleted the codex/codex-gap-check2 branch June 4, 2026 06:54
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