Skip to content

Add Dynamo renderer transport#1287

Draft
AmeenP wants to merge 2 commits intomainfrom
codex/dynamo-renderer-transport
Draft

Add Dynamo renderer transport#1287
AmeenP wants to merge 2 commits intomainfrom
codex/dynamo-renderer-transport

Conversation

@AmeenP
Copy link
Copy Markdown
Collaborator

@AmeenP AmeenP commented May 5, 2026

Summary

Adds a renderer transport selector to verifiers so RendererClient can choose between the existing vLLM /inference/v1/generate wire shape and Dynamo's /chat/completions + nvext.token_data wire shape.

  • adds ClientConfig.renderer_transport, defaulting to prime_vllm_generate
  • forwards the selected transport into renderers.client.generate(...)
  • pins the standalone PrimeIntellect-ai/renderers repo to the Dynamo transport PR head (7ca1ab3, renderers Question: Completion IDs for multi-turn conversations. #11)
  • adds unit coverage that the renderer client passes the configured transport through

Context

Latest verifiers/main now consumes renderers from the standalone repo, not packages/renderers. This PR uses that latest path and pins the companion renderers PR directly.

Companion PR: PrimeIntellect-ai/renderers#11

Validation

  • uv lock --check
  • uv run pytest tests/test_renderer_client.py
  • uv run ruff check pyproject.toml verifiers/types.py verifiers/clients/renderer_client.py tests/test_renderer_client.py
  • uv run ruff format --check pyproject.toml verifiers/types.py verifiers/clients/renderer_client.py tests/test_renderer_client.py

@AmeenP AmeenP force-pushed the codex/dynamo-renderer-transport branch from e746897 to 02256e0 Compare May 5, 2026 05:57
@AmeenP AmeenP changed the title [codex] Add Dynamo renderer transport Add Dynamo renderer transport May 5, 2026
@AmeenP AmeenP force-pushed the codex/dynamo-renderer-transport branch from f6b72e9 to f6f8224 Compare May 8, 2026 09:53
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